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

Loosen version requirement for OAuth gem #187

Merged
merged 1 commit into from
May 5, 2023
Merged

Loosen version requirement for OAuth gem #187

merged 1 commit into from
May 5, 2023

Conversation

MrSerth
Copy link

@MrSerth MrSerth commented Apr 13, 2023

Newer versions mainly include bug fixes, support newer Ruby versions, and partly drop support for older Ruby versions. Through the new version range specified with >= 0.4.5, we still allow the same OAuth gem version, but additionally allow all newer versions. Those do not contain any breaking changes.

See https://gitlab.com/oauth-xx/oauth/-/blob/main/CHANGELOG.md


The changes I performed are similar to the ones discussed here by one of the OAuth gem maintainers. According to that discussion and my own analysis, newer versions of the OAuth gem do not contain any breaking changes.

@slaughter550: I would be happy if you could review my PR and potentially draft a new release -- thanks!

Newer versions mainly include bug fixes, support newer Ruby versions, and partly drop support for older Ruby versions. Through the new version range specified with `>= 0.4.5`, we still allow the same OAuth gem version, but additionally allow all newer versions. Those do not contain any breaking changes.

See https://gitlab.com/oauth-xx/oauth/-/blob/main/CHANGELOG.md
@CLAassistant
Copy link

CLAassistant commented Apr 13, 2023

CLA assistant check
All committers have signed the CLA.

@slaughter550
Copy link
Contributor

I will review on Monday and post feedback if any. Thanks so much!

@MrSerth
Copy link
Author

MrSerth commented Apr 24, 2023

Sure, let me know if you have any questions, and I'll be happy to respond. Thanks again!

@MrSerth
Copy link
Author

MrSerth commented May 2, 2023

@slaughter550 I would appreciate your review and a new version very much; this will unblock another gem update for me. Thanks again for your time!

@slaughter550 slaughter550 merged commit 170e470 into instructure:1.2.x May 5, 2023
@MrSerth MrSerth deleted the 1.2.x branch May 5, 2023 06:47
@MrSerth
Copy link
Author

MrSerth commented May 8, 2023

Awesome, thanks for merging. I would be grateful if you could drop a new release, making those changes available for everyone and easier integration with existing projects 🙏

@MrSerth
Copy link
Author

MrSerth commented May 22, 2023

@slaughter550 Would you mind drafting a new release with this change, please? If you'd like, I can prepare a PR with a version bump, as a first step of the release process.

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.

3 participants