Skip to content

[MAINT] Improve Import behaviour of github_default_branch#3216

Open
deiga wants to merge 10 commits intointegrations:mainfrom
F-Secure-web:fix-default-branch-import
Open

[MAINT] Improve Import behaviour of github_default_branch#3216
deiga wants to merge 10 commits intointegrations:mainfrom
F-Secure-web:fix-default-branch-import

Conversation

@deiga
Copy link
Collaborator

@deiga deiga commented Feb 21, 2026

Resolves #2613


Before the change?

  • Resource didn't use Context-aware functions
  • Resource didn't use tflog for logging
  • Missing tests for Update and Import functionality

After the change?

  • Changes to use Context-aware functions
  • Changes to use tflog for logging
  • Adds support for repository renaming, by add repository_id field and removing ForceNew from repository

Pull request checklist

  • Schema migrations have been created if needed (example)
  • 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

@github-actions
Copy link

👋 Hi! Thank you for this contribution! Just to let you know, our GitHub SDK team does a round of issue and PR reviews twice a week, every Monday and Friday! We have a process in place for prioritizing and responding to your input. Because you are a part of this community please feel free to comment, add to, or pick up any issues/PRs that are labeled with Status: Up for grabs. You & others like you are the reason all of this works! So thank you & happy coding! 🚀

@github-actions github-actions bot added the Type: Bug Something isn't working as documented label Feb 21, 2026
@deiga deiga added this to the v6.12.0 Release milestone Feb 22, 2026
deiga added 10 commits March 8, 2026 23:11
Signed-off-by: Timo Sand <timo.sand@f-secure.com>
Signed-off-by: Timo Sand <timo.sand@f-secure.com>
Signed-off-by: Timo Sand <timo.sand@f-secure.com>
Signed-off-by: Timo Sand <timo.sand@f-secure.com>
Signed-off-by: Timo Sand <timo.sand@f-secure.com>
Signed-off-by: Timo Sand <timo.sand@f-secure.com>
Signed-off-by: Timo Sand <timo.sand@f-secure.com>
Signed-off-by: Timo Sand <timo.sand@f-secure.com>
Signed-off-by: Timo Sand <timo.sand@f-secure.com>
Signed-off-by: Timo Sand <timo.sand@f-secure.com>
@deiga deiga force-pushed the fix-default-branch-import branch from 0e63837 to a31fc9f Compare March 8, 2026 21:14
@deiga deiga requested a review from stevehipwell March 8, 2026 21:15
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]: github_branch_default attempts to rename to the same name on import

1 participant