go-github icon indicating copy to clipboard operation
go-github copied to clipboard

Can't remove repository ruleset's every Bypass Actors because of serialization issue

Open lado936 opened this issue 1 year ago • 4 comments

Hello all, I am using go-github repository ruleset update function from here

but issue is that, if i already have bypassactors set and want to remove all of them, it not applying changes as bypassactors has omitempty set here, if i understand correctly, omitempty must be removed if i want to pass empty list there. Proposing to remove omitempty if there is no case i can't see, why its needed.

lado936 avatar Apr 22 '24 08:04 lado936

It seems we may need a separate endpoint to explicitly remove all BypassActors since we need the omitempty for regular usage of this endpoint. Agreed?

Would you like to put together a PR for this, @lado936, or shall I open this up to other contributors to this repo?

gmlewis avatar Apr 22 '24 11:04 gmlewis

Better if someone else will work on this, new to Golang and will need way more time i guess

lado936 avatar Apr 22 '24 13:04 lado936

Better if someone else will work on this, new to Golang and will need way more time i guess

OK, no problem.

This would be a great PR for any new contributor to this repo or a new Go developer. All contributions are greatly appreciated!

Feel free to volunteer for any issue and the issue can be assigned to you so that others don't attempt to duplicate the work.

Please check out our CONTRIBUTING.md guide to get started. (In particular, please remember to go generate ./... and don't use force-push to your PRs.)

Thank you!

gmlewis avatar Apr 22 '24 14:04 gmlewis

I can take this

Matthew-Reidy avatar Apr 29 '24 21:04 Matthew-Reidy