Skip to content

Update Ruby Dependencies#1595

Merged
justin808 merged 13 commits intomasterfrom
update-dependencies
Jan 10, 2024
Merged

Update Ruby Dependencies#1595
justin808 merged 13 commits intomasterfrom
update-dependencies

Conversation

@ahangarha
Copy link
Copy Markdown
Contributor

@ahangarha ahangarha commented Jan 8, 2024

This change is Reviewable

@ahangarha ahangarha force-pushed the update-dependencies branch from 1833503 to 270507e Compare January 8, 2024 14:43
@ahangarha ahangarha force-pushed the update-dependencies branch from 25ae017 to bbb41bf Compare January 8, 2024 15:50
@ahangarha ahangarha force-pushed the update-dependencies branch from 784ab96 to 891bf53 Compare January 8, 2024 16:44
@ahangarha ahangarha force-pushed the update-dependencies branch from 891bf53 to d7ec85e Compare January 8, 2024 16:46
@ahangarha ahangarha changed the title WIP - Update dependencies UpdateRuby Dependencies Jan 8, 2024
@ahangarha ahangarha changed the title UpdateRuby Dependencies Update Ruby Dependencies Jan 8, 2024
@ahangarha ahangarha requested a review from Judahmeek January 8, 2024 17:29
@ahangarha ahangarha marked this pull request as ready for review January 8, 2024 17:29
Comment thread .circleci/config.yml
aliases:
- &docker-image
- image: circleci/ruby:2.7-node-browsers
- image: circleci/ruby:3.0-node-browsers
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

is 3.0 the newest

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Do we use circleci for ROR at all?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

No, but good to have the example file.

gem "shakapacker", "7.2.1"
gem "bootsnap", require: false
gem "rails", "~> 7.0", ">= 7.0.1"
gem "rails", "~> 7.1"
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

any updates needed for the Rails upgrade?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I didn't notice anything to apply to this project.

Copy link
Copy Markdown
Member

@justin808 justin808 left a comment

Choose a reason for hiding this comment

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

LGTM

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