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/Downgrade ruby to work on heroku #11

Closed
wants to merge 1 commit into from

Conversation

jesusMava
Copy link
Owner

Downgrade ruby from 3.1 to 3.0.3 because it was an error with prawn gem to deploy on heroku
here are the issues:
prawnpdf/prawn#1195

prawnpdf/prawn#1235

@jesusMava jesusMava force-pushed the Fix/Update-ruby-to-work-on-heroku branch from 39b661b to 3d80469 Compare January 26, 2022 03:27
@jesusMava jesusMava temporarily deployed to curiosity-fix-update-ru-akxa9y January 26, 2022 03:30 Inactive
@jesusMava jesusMava requested a review from itsmeurbi January 26, 2022 03:31
@@ -36,7 +36,7 @@ class CommentsControllerTest < ActionDispatch::IntegrationTest
test 'user should be able to edit their own comments' do
user = create(:user)
curiosity = create(:curiosity_card)
comment = create(:comment, curiosity_card: curiosity, user: )
comment = create(:comment, curiosity_card: curiosity, user: user)
Copy link
Collaborator

Choose a reason for hiding this comment

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

This changes has nothing to do with ruby version downgrade hehe

Copy link
Owner Author

Choose a reason for hiding this comment

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

actually, we need it because it's a new release from ruby 3.1.0 https://www.ruby-lang.org/en/news/2021/11/09/ruby-3-1-0-preview1-released/ hehe and my test was broken because of it :c hehe

Copy link
Collaborator

Choose a reason for hiding this comment

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

Holy smokes!
Using user: does actually looks for a user var. That's crazy dude haha

Copy link
Collaborator

@itsmeurbi itsmeurbi left a comment

Choose a reason for hiding this comment

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

Crap, didn't know about ruby 3.1.0 changes

Copy link

@luisAzcuaga luisAzcuaga left a comment

Choose a reason for hiding this comment

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

Changing the ruby version in a project in an early may be easy. But it isn't always like that; If you are sure nothing else could be done by configuring Heroku, then you have my approval :)

@jesusMava jesusMava closed this Jan 26, 2022
@jesusMava jesusMava deleted the Fix/Update-ruby-to-work-on-heroku branch January 26, 2022 20:41
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