Skip to content

Conversation

Not-Dhananjay-Mishra
Copy link
Contributor

@Not-Dhananjay-Mishra Not-Dhananjay-Mishra commented Oct 9, 2025

Fixes: #3771.

This PR adds support for Organization Immutable Releases API

Implement

  • GET /orgs/{org}/settings/immutable-releases
  • PUT /orgs/{org}/settings/immutable-releases
  • GET /orgs/{org}/settings/immutable-releases/repositories
  • PUT /orgs/{org}/settings/immutable-releases/repositories
  • PUT /orgs/{org}/settings/immutable-releases/repositories/{repository_id}
  • DELETE /orgs/{org}/settings/immutable-releases/repositories/{repository_id}

issue - #3771

@gmlewis gmlewis added the NeedsReview PR is awaiting a review before merging. label Oct 9, 2025
Copy link

codecov bot commented Oct 9, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.44%. Comparing base (06b8b3a) to head (723c401).

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #3774      +/-   ##
==========================================
+ Coverage   91.41%   91.44%   +0.03%     
==========================================
  Files         187      188       +1     
  Lines       16822    16900      +78     
==========================================
+ Hits        15377    15455      +78     
  Misses       1257     1257              
  Partials      188      188              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Collaborator

@gmlewis gmlewis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you, @Not-Dhananjay-Mishra!
Just a few issues to address, please, then we should be ready for a second LGTM+Approval from any other contributor to this repo.

@Not-Dhananjay-Mishra
Copy link
Contributor Author

@gmlewis and @zyfy29 Thanks for the suggestions, I’ve made some changes:
EnforcedRepositories string -> *string
SelectedRepositoriesIDs -> SelectedRepositoryIDs
in SetImmutableReleasesPolicy *ImmutableReleaseRepository -> ImmutableReleaseRepository
SetImmutableReleaseRepositories now accept directly repositoryIDs []int64 rather than SelectedRepositories struct
Made method descriptions in a single paragraph
and made a few other description tweaks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
NeedsReview PR is awaiting a review before merging.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

add support for organization Immutable Releases API
3 participants