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

retire Ruby 2 + gems update + release 1.6 #91

Merged
merged 7 commits into from
Sep 14, 2023

Conversation

tdutreui-solocal
Copy link
Collaborator

@tdutreui-solocal tdutreui-solocal commented Sep 13, 2023

Ruby 2.7 life ended 5 months ago (31 Mar 2023)

working_directory: ~/repo
steps:
- build

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@thomasLeclaire do you know why this "build_2_x" step still appears in GitHub ? It's gone on CircleCI but still blocks validation on GitHub
Same on this simpler PR #92

Copy link
Contributor

Choose a reason for hiding this comment

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

it is the branches protections : defined there https://github.com/Effilab/bing_ads_ruby_sdk/settings/branch_protection_rules/4793994
just removed the 2.x one.

@thomasLeclaire
Copy link
Contributor

You should consider to publish explicitly a new gem and change the ruby_version there :

spec.required_ruby_version = ">= 2.3"
(and maybe review also the other dependency in same file)
and adapt the changelog obviously.

this gem is public and used out of solocal fyi.

@tdutreui-solocal tdutreui-solocal changed the title retire Ruby 2 + gems update retire Ruby 2 + gems update + release 1.6 Sep 14, 2023
@tdutreui-solocal tdutreui-solocal merged commit 1099e2a into master Sep 14, 2023
2 checks passed
@tdutreui-solocal tdutreui-solocal deleted the bot-ruby-update-2 branch September 14, 2023 09:47
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