Skip to content

Comments

[FEAT] Add github_repository_pages Resource and Data Source#3168

Open
deiga wants to merge 16 commits intointegrations:mainfrom
F-Secure-web:add-github_repository_pages-resource
Open

[FEAT] Add github_repository_pages Resource and Data Source#3168
deiga wants to merge 16 commits intointegrations:mainfrom
F-Secure-web:add-github_repository_pages-resource

Conversation

@deiga
Copy link
Collaborator

@deiga deiga commented Feb 8, 2026 •
edited
Loading

Resolves #3167
Resolves #2671
Resolves #3142
Resolves #1045

Before the change?

  • A repository's Pages resource was only manageable through github_repository

After the change?

  • There is now a github_repository_pages resource to manage the pages
  • There is now a github_repository_pages data source to fetch information about the pages
  • The pages fields are now deprecated in github_repository Resource and Data Source

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

github-actions bot commented Feb 8, 2026

👋 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! 🚀

@deiga deiga added this to the v6.12.0 Release milestone Feb 8, 2026
@deiga deiga requested a review from stevehipwell February 12, 2026 11:58
@deiga deiga force-pushed the add-github_repository_pages-resource branch 2 times, most recently from e081a2d to c75d366 Compare February 13, 2026 22:14
@deiga deiga mentioned this pull request Feb 13, 2026
Open
4 tasks
stevehipwell
stevehipwell requested changes Feb 16, 2026
View reviewed changes
Copy link
Collaborator

@stevehipwell stevehipwell left a comment

Choose a reason for hiding this comment

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

This is looking really good.

stevehipwell
stevehipwell reviewed Feb 16, 2026
View reviewed changes
stevehipwell
stevehipwell reviewed Feb 16, 2026
View reviewed changes
deiga added 10 commits February 17, 2026 22:09
@deiga
Add resource for repository pages and deprecate it from repository R
88a3bf9 
Signed-off-by: Timo Sand <timo.sand@f-secure.com>
@deiga
Add data source for repository pages and deprecate it from repository DS
312823f 
Signed-off-by: Timo Sand <timo.sand@f-secure.com>
@deiga
Update formatting of docs page
cc53612 
Signed-off-by: Timo Sand <timo.sand@f-secure.com>
@deiga
Change github_repository to only Read pages info if pages is in the…
2fba4cc 
… config

Signed-off-by: Timo Sand <timo.sand@f-secure.com>
@deiga
Fix faulty test
1a949f6 
Signed-off-by: Timo Sand <timo.sand@f-secure.com>
@deiga
Update Import test
ad643d9 
Signed-off-by: Timo Sand <timo.sand@f-secure.com>
@deiga
Add repository_id diff logic and unify naming
4740338 
Signed-off-by: Timo Sand <timo.sand@f-secure.com>
@deiga
Use ConfigStateChecks in DS tests
7f9672c 
Signed-off-by: Timo Sand <timo.sand@f-secure.com>
@deiga
Refactor to use ConfigStateChecks in Resource
6ec5526 
Signed-off-by: Timo Sand <timo.sand@f-secure.com>
@deiga
Comment out owner field for now
91f6484 
Signed-off-by: Timo Sand <timo.sand@f-secure.com>
@deiga deiga force-pushed the add-github_repository_pages-resource branch from c75d366 to 91f6484 Compare February 17, 2026 20:09
@deiga deiga requested a review from stevehipwell February 17, 2026 20:10
stevehipwell
stevehipwell requested changes Feb 18, 2026
View reviewed changes
@deiga deiga mentioned this pull request Feb 18, 2026
Open
4 tasks
deiga added 2 commits February 19, 2026 11:49
@deiga
Comment out owner field for DS as well
3bc00a5 
Signed-off-by: Timo Sand <timo.sand@f-secure.com>
@deiga
Simplify import ID
e3d61b7 
Signed-off-by: Timo Sand <timo.sand@f-secure.com>
@deiga deiga requested a review from stevehipwell February 19, 2026 10:01
@healiha
Copy link

healiha commented Feb 19, 2026

Would it be possible to add visibility to github_repository_pages?
image

The documentation doesn't seem to cover this, but after testing, a PUT request to https://api.github.com/repos/<org>/<repo>/pages with {"public": true/false} works.

@deiga
Copy link
Collaborator Author

deiga commented Feb 19, 2026

@healiha I'll check that out

deiga added 4 commits February 19, 2026 22:23
@deiga
Add support for public field in pages
8445361 
Signed-off-by: Timo Sand <timo.sand@f-secure.com>
@deiga
Update docs
1e5f17d 
Signed-off-by: Timo Sand <timo.sand@f-secure.com>
@deiga
Improve field descriptions
dda2b91 
Signed-off-by: Timo Sand <timo.sand@f-secure.com>
@deiga
Only modify CNAME in state if it has a value upstream
18c8135 
Signed-off-by: Timo Sand <timo.sand@f-secure.com>
@deiga deiga changed the title [FEAT] Add github_repository_pages Reasource and Data Source [FEAT] Add github_repository_pages Resource and Data Source Feb 20, 2026
stevehipwell
stevehipwell approved these changes Feb 20, 2026
View reviewed changes
Copy link
Collaborator

@stevehipwell stevehipwell left a comment

Choose a reason for hiding this comment

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

LGTM

@stevehipwell
Copy link
Collaborator

stevehipwell commented Feb 20, 2026

@deiga could you please rebase this PR?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment