Skip to content

Update Ruby gem and Github Actions packages#97

Merged
jerpelea merged 2 commits into
apache:masterfrom
btashton:dep-updates
May 15, 2023
Merged

Update Ruby gem and Github Actions packages#97
jerpelea merged 2 commits into
apache:masterfrom
btashton:dep-updates

Conversation

@btashton
Copy link
Copy Markdown
Contributor

Summary

This slims down and updates the Ruby gems that we are using. Removing the github-pages gem which was unused with they way that we upload the site meant we could move off a bunch of EOL gems and also on to the latest version on Jekell.

I did also have to update sass files, this was automated with sass-migrator https://sass-lang.com/documentation/cli/migrator

We are also running old versions of github actions which would spit out a warning about still using node 12. https://github.blog/changelog/2022-09-22-github-actions-all-actions-will-begin-running-on-node16-instead-of-node12/

Impact

Living in the future.

Testing

Passing CI should be sufficient. Local testing also passed

@btashton
Copy link
Copy Markdown
Contributor Author

@cederom this may also make it easier for you to build locally long as you have ruby >= 2.5

Note my instructions with the updated readme.

@cederom
Copy link
Copy Markdown
Contributor

cederom commented May 15, 2023

@btashton: @cederom this may also make it easier for you to build locally long as you have ruby >= 2.5
Note my instructions with the updated readme.

Cool! Thank you @btashton! :-) Yes we have ruby 3.0, 3.1, 3.2 on FBSD :-) I will verify and report back.. just need to take some sleep its 0530AM here :-P

Copy link
Copy Markdown
Contributor

@cederom cederom left a comment

Choose a reason for hiding this comment

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

Rapid Response @btashton :-)
All checks have passed :-)

@jerpelea jerpelea merged commit b538557 into apache:master May 15, 2023
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