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

Update rubygems and add a devcontainer #514

Merged
merged 3 commits into from
Jan 24, 2022

Conversation

razzeee
Copy link
Contributor

@razzeee razzeee commented Jan 23, 2022

This updates all gems, so that it can build again with the current versions of ruby and rubygem.

It also adds a devcontainer, so that people can run it without installing anything ruby on their system.

@razzeee razzeee changed the title Update rubygems Update rubygems and add a devcontainer Jan 23, 2022
@razzeee
Copy link
Contributor Author

razzeee commented Jan 23, 2022

Closes #506

gem "middleman", "~> 3"#, github: "middleman/middleman", branch: "v3-stable"
gem "middleman-matomo", github: "barthalion/middleman-matomo"
gem "middleman", "~> 4"#, github: "middleman/middleman", branch: "v3-stable"
gem 'matomo-middleman'
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This seems to work fine, as far as I can tell without deploying it. Locally everything seems to be added correctly.

But I'm not sure what the changes in @barthalion fork were

Copy link
Member

Choose a reason for hiding this comment

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

I don't actually remember at this point.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Guess we figured out why :D

@@ -16,14 +16,14 @@ gem "sass" #3.2.12
# gem "middleman-syntax"

#deployment
gem "middleman-deploy" #, "~> 2.0.0.pre.alpha"
gem "middleman-deploy", "~> 2.0.0.pre.alpha"
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Based on karlfreeman/middleman-deploy#132 (comment) doesn't work without for me

@barthalion barthalion merged commit c549c43 into flatpak:source Jan 24, 2022
@razzeee razzeee deleted the update-ruby branch January 28, 2022 18:30
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