Skip to content

fix: do not update/create rulesets on archived repository#2460

Merged
diofeher merged 12 commits intointegrations:mainfrom
scadu:fix/archived_ignore_attrs
Dec 2, 2025
Merged

fix: do not update/create rulesets on archived repository#2460
diofeher merged 12 commits intointegrations:mainfrom
scadu:fix/archived_ignore_attrs

Conversation

@scadu
Copy link
Copy Markdown
Contributor

@scadu scadu commented Nov 13, 2024

At the moment, provider would attempt to update github_repository_ruleset on archived repository resulting in 403 from GitHub API. Provider should not attempt updating this attribute when repository is archived (read-only).

Resolves #2459


Before the change?

  • Provider attempts to update github_repository_ruleset on archived (read-only) repository.

After the change?

  • Provider skips updating github_repository_ruleset on archived repository.

Pull request checklist

  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been reviewed and added / updated if needed (for bug fixes / features)

Does this introduce a breaking change?

Please see our docs on breaking changes to help!

  • Yes
  • No

scadu and others added 2 commits November 13, 2024 12:36
At the moment, provider would attempt to update `github_repository_ruleset` on archived repository resulting in 403 from GitHub API.
Provider should not attempt updating this attribute when repository is archived (read-only).
@scadu scadu marked this pull request as ready for review November 25, 2024 10:16
@deiga
Copy link
Copy Markdown
Collaborator

deiga commented Nov 28, 2025

Hey @scadu 👋
Thank you for your contribution!

Would you be available to resolve the merge conflicts on this PR?

Then we can try to get this reviewed and merged ☺️

@github-actions github-actions bot added the Type: Bug Something isn't working as documented label Dec 1, 2025
@scadu scadu force-pushed the fix/archived_ignore_attrs branch from dfda867 to b4f0310 Compare December 1, 2025 08:34
@scadu scadu force-pushed the fix/archived_ignore_attrs branch from 083161c to 79b52cb Compare December 1, 2025 08:42
@scadu
Copy link
Copy Markdown
Contributor Author

scadu commented Dec 1, 2025

Hey @scadu 👋 Thank you for your contribution!

Would you be available to resolve the merge conflicts on this PR?

Then we can try to get this reviewed and merged ☺️

@deiga here you go.

@diofeher
Copy link
Copy Markdown
Collaborator

diofeher commented Dec 1, 2025

There are a few lint errors here: https://github.com/integrations/terraform-provider-github/actions/runs/19816483287/job/56804316636?pr=2460

@scadu

@scadu scadu force-pushed the fix/archived_ignore_attrs branch from ee5f84c to 27ce6ee Compare December 1, 2025 18:15
fmt
@scadu scadu force-pushed the fix/archived_ignore_attrs branch from 77ad2ce to 964e7d5 Compare December 2, 2025 08:16
@scadu scadu requested a review from diofeher December 2, 2025 08:16
Copy link
Copy Markdown
Collaborator

@diofeher diofeher left a comment

Choose a reason for hiding this comment

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

Tests passed and code reviewed. Thanks for the contribution @scadu

@diofeher diofeher merged commit 6e45f0f into integrations:main Dec 2, 2025
8 checks passed
@github-project-automation github-project-automation bot moved this from 👀 In review to ✅ Done in 🧰 Octokit Active Dec 2, 2025
@scadu scadu deleted the fix/archived_ignore_attrs branch December 2, 2025 10:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Type: Bug Something isn't working as documented

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG]: updating `github_repository_ruleset

3 participants