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

Release 7.0.1 #591

Merged
merged 2 commits into from
Nov 21, 2024
Merged

Release 7.0.1 #591

merged 2 commits into from
Nov 21, 2024

Conversation

yunwwang
Copy link
Contributor

Release 7.0.1 incoporate US Core v0.9.2

CHANGELOG.md Outdated Show resolved Hide resolved
us_core_test_kit (0.9.1)
inferno_core (>= 0.4.37)
us_core_test_kit (0.9.2)
inferno_core (>= 0.5.0)
Copy link
Contributor

Choose a reason for hiding this comment

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

I find this a little surprising on a patch release but if @Jammjammjamm is good with this (I assume it was required here), no objections.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Before releasing 0.9.0, inferno_core dependency was updated to 0.5.0 using bundle update command. But the gemspec was not updated in that release. I think it would be good to keep gemspec updated also.

Copy link
Contributor

@Jammjammjamm Jammjammjamm Nov 20, 2024

Choose a reason for hiding this comment

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

Bumping it in the gemspec wasn't strictly necessary, but it was also one of the test kits that had a problem that was addressed in 0.5.0.

But in general, there is no expectation that the version in the gemspec match what is in Gemfile.lock.

Co-authored-by: Rob Scanlon <[email protected]>
@yunwwang yunwwang requested a review from arscan November 21, 2024 19:06
@yunwwang yunwwang merged commit 83fd7d5 into main Nov 21, 2024
3 checks passed
@yunwwang yunwwang deleted the FI-3489-release-701 branch November 21, 2024 19:13
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