Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Fix asserts_url typo #1194

Merged
merged 4 commits into from
Nov 8, 2022
Merged

Fix asserts_url typo #1194

merged 4 commits into from
Nov 8, 2022

Conversation

shabbyrobe
Copy link
Contributor

This is an attempt to revive #448, which seems to have stalled. The change is largely the same, but I have also followed the instructions for deprecation found here.

@shabbyrobe
Copy link
Contributor Author

@kfcampbell - hope you don't mind me pushing my luck and pinging you on this one too 😄

This feels like it's relatively benign, do you think this is something that could be merged?

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.

Thanks for bringing this up! This has always bothered me.

I wouldn't mind cutting a major release to remove the deprecated version at some point in the near future as well. I'd like to get in the habit of releasing more often.

@kfcampbell
Copy link
Member

Integration tests are passing locally for me.

@kfcampbell kfcampbell merged commit baafae0 into integrations:main Nov 8, 2022
kazaker pushed a commit to auto1-oss/terraform-provider-github that referenced this pull request Dec 28, 2022
* Fix assets_url typo

* Add deprecation message

* Follow instructions more closely

Co-authored-by: Keegan Campbell <me@kfcampbell.com>
avidspartan1 pushed a commit to avidspartan1/terraform-provider-github that referenced this pull request Feb 5, 2024
* Fix assets_url typo

* Add deprecation message

* Follow instructions more closely

Co-authored-by: Keegan Campbell <me@kfcampbell.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants