Uh oh!
There was an error while loading. Please reload this page.
fix: Allow removing/unsetting repository custom property - #3309
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@## master #3309 +/- ##
==========================================
- Coverage 97.72% 93.05% -4.67%
==========================================
Files 153 172 +19 Lines 13390 11772 -1618 ==========================================
- Hits 13085 10955 -2130 - Misses 215 723 +508 - Partials 90 94 +4 ☔ View full report in Codecov by Sentry. |
gmlewis
commented
Oct 6, 2024
Thank you, @tomfeigin ! |
felixlut
commented
Oct 6, 2024
Could we get a new release with this change as well? I have some changes over the last couple of weeks that will unblock some stuff I'm working on in https://github.com/integrations/terraform-provider-github: |
gmlewis
commented
Oct 6, 2024
Yes, I can work on a new release later today. |
gmlewis
commented
Oct 6, 2024
This has now been released here: |
This change will allow a user to set
nullwhen calling the create/update repository custom property endpoint. As written in the docs, doing so will remove/unset the custom property. Currently doing so is not possibleNormally I think this would count as a breaking change, since this will modify the behavior when omitting the
Value, but I believe this is a straight up bug, as omitting theValueshould always result in an error. So there shouldn't be any usages that's relying on this behavior. Technically it's possible, but improbable.