Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add support for packages IP ranges in github_ip_ranges #1958

Merged
merged 7 commits into from
Feb 2, 2024

Conversation

Nmishin
Copy link
Contributor

@Nmishin Nmishin commented Oct 14, 2023

Resolves #1812

Can be merged after go-github change: google/go-github#2964


Before the change?

After the change?

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

@kfcampbell
Copy link
Member

We're waiting here on the next release of google/go-github. Thanks for the contribution!

@Nmishin
Copy link
Contributor Author

Nmishin commented Jan 9, 2024

@kfcampbell could you please review, i think we are ready to merge it

@kfcampbell
Copy link
Member

Would you mind adding appropriate checks for these fields in github/data_source_github_ip_ranges_test.go?

@Nmishin
Copy link
Contributor Author

Nmishin commented Jan 25, 2024

Test updated!

Copy link
Member

@kfcampbell kfcampbell left a comment

Choose a reason for hiding this comment

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

@Nmishin Awesome, thank you!

@kfcampbell kfcampbell merged commit 970dd20 into integrations:main Feb 2, 2024
3 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FEAT]: Add packages IP ranges to github_ip_ranges
2 participants