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

Fix Circle config to use Ruby 3.1 #28

Merged
merged 1 commit into from
Apr 14, 2023
Merged

Fix Circle config to use Ruby 3.1 #28

merged 1 commit into from
Apr 14, 2023

Conversation

seven1m
Copy link
Member

@seven1m seven1m commented Apr 14, 2023

No description provided.

@seven1m seven1m requested a review from a team as a code owner April 14, 2023 17:13
@seven1m seven1m requested a review from a team April 14, 2023 17:14
@seven1m seven1m changed the title Fix Circle config to use Ruby 2.7 Fix Circle config to use Ruby 3.1 Apr 14, 2023
Copy link

@tannermares tannermares left a comment

Choose a reason for hiding this comment

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

Simple enough! What do you think about switching this our use our circleci-tools orb?

version: '2.1'
orbs:
  rails: planningcenter/rails@2.2.0
jobs:
  build:
    executor:
      name: rails/minimal
      ruby-version: '3.1'
    steps:
      - checkout
      - rails/bundle
      - rails/test:
          test-runner: rspec
workflows:
  commit:
    jobs:
      - test:
          context:
            - GitHub Packages
            - AWS ECR (circleci-tools)
  version: 2

https://circleci.com/developer/orbs/orb/planningcenter/rails

see simple-gem-setup

@seven1m
Copy link
Member Author

seven1m commented Apr 14, 2023

Huh, that didn't work. Is there a benefit to using the shared orb given this is such a simple build?

@seven1m seven1m merged commit dc91e1e into main Apr 14, 2023
@seven1m seven1m deleted the 7m/fix-circle branch April 14, 2023 22:11
@tannermares
Copy link

@seven1m not really... just that you would be using the same tooling we're using everywhere else.

@seven1m
Copy link
Member Author

seven1m commented Apr 24, 2023

Sorry I gave up 😩

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.

None yet

3 participants