Codecov Report

Merging #2831 (889737e) into master (bedd4e3) will decrease coverage by 0.01%.
The diff coverage is n/a.

❗ Current head 889737e differs from pull request most recent head cb37562. Consider uploading reports for the commit cb37562 to get more accurate results

@@            Coverage Diff             @@
##           master    #2831      +/-   ##
==========================================
- Coverage   98.06%   98.06%   -0.01%     
==========================================
  Files         137      137              
  Lines       12289    12287       -2     
==========================================
- Hits        12051    12049       -2     
  Misses        162      162              
  Partials       76       76              

... and 3 files with indirect coverage changes

It seems the breaking change was introduced by GitHub (point 2 here): https://github.blog/changelog/2023-06-27-repository-rules-public-beta-updates/

The latest change in master added a couple of test cases related to organization rulesets which used Int64 as opposed to pointers in ruleset ID, this was the case prior to the change I’m proposing here so I fixed that in the latest commit.

Thank you @o-sama ! LGTM.

@liaodaniel - could you please look at these breaking API changes since you are the original author of this section of code and make sure you are happy with them?

Yep, this looks good from my end. Part of me felt that I probably should have waited a bit until the feature was out of beta before introducing it but the feature seemed too compelling to wait 😄

Thank you, @liaodaniel !
Merging.