Skip to content

[PROF-11035] Bump to version 3.3.7#9

Merged
ivoanjo merged 3 commits into
masterfrom
ivoanjo/prof-11035-bump-version
Dec 16, 2024
Merged

[PROF-11035] Bump to version 3.3.7#9
ivoanjo merged 3 commits into
masterfrom
ivoanjo/prof-11035-bump-version

Conversation

@ivoanjo

@ivoanjo ivoanjo commented Dec 13, 2024

Copy link
Copy Markdown
Member

What does this PR do?

This PR bumps the gem to version 3.3.7 in preparation for the next release.

Motivation:

Release a new version with the Ruby 3.4.0-rc1 headers from #8.

Additional Notes:

I've also removed the CHANGELOG.md and CONTRIBUTING.md files as they were leftovers from our forking debase-ruby_core_source and aren't particularly relevant anymore -- we have our own separate changelog file, and the instructions in CONTRIBUTING.md are also in the README.md.

This PR is on top of #8 to make it easier to test the new release (and because it implicitly depends on #8 anyway); once #8 gets merged to master, this one will automatically will get changed to master as well.

How to test the change?

Here's what I did to validate this locally:

  1. Ran bundle exec rake build to package the gem
  2. Manually installed the packaged gem, and ran the profiler test suite on ruby 3.4.0-rc1 and 3.3.5 with this version.

@ivoanjo
ivoanjo requested a review from a team December 13, 2024 10:01
@ivoanjo ivoanjo assigned AlexJF and unassigned AlexJF Dec 13, 2024
@ivoanjo
ivoanjo requested a review from AlexJF December 13, 2024 10:01
Base automatically changed from ivoanjo/prof-11035-ruby-3.4.0-rc1-headers to master December 13, 2024 14:18
@ivoanjo

ivoanjo commented Dec 13, 2024

Copy link
Copy Markdown
Member Author

We should only merge this and release once #11 is merged as well.

@ivoanjo
ivoanjo enabled auto-merge December 16, 2024 12:37
@ivoanjo
ivoanjo merged commit 3747ae9 into master Dec 16, 2024
@ivoanjo
ivoanjo deleted the ivoanjo/prof-11035-bump-version branch December 16, 2024 12:39
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