Skip to content
This repository has been archived by the owner on Jun 24, 2022. It is now read-only.

Various Jekyll/build improvements #2194

Closed
wants to merge 10 commits into from
Closed

Various Jekyll/build improvements #2194

wants to merge 10 commits into from

Conversation

jonaharagon
Copy link
Contributor

Changes:

  • Specify specific Ruby version to use in .ruby-version, to ensure developers are using identical software to build machines and Netlify.
    • Set this value to 3.0.0, up from current builds using 2.6.2. Additionally I've added two gems that are now required for Jekyll Ruby 3 support.
    • We want to avoid issues like Update Ruby Dependencies #2168 in the future, as updating Ruby locally without coordinating with site builders can break production builds.
    • Updated Ruby installation instructions in README.
  • Upgrade Bundler to 2.2.5
    • Locking to 2.0.1 in the first place was a workaround for a Netlify build bug which is now resolved.
  • Switched from github-pages gem to jekyll gem.
  • Upgrade Jekyll to 4.2.0
    • No longer depends on ruby-sass, which is EOL.
  • Adds brotli static file generation.
  • Renames build.sh to .buildrc.
    • This new file is sourced within the build scripts on production servers. Commands should be assumed to be running in the main directory ($HOME/privacytools.io) directly following a fresh pull from GitHub.

Gusted and others added 10 commits January 12, 2021 20:27
- Compress file AOD.
- Let the server only use static files. 
- Don't add github-pages to jekyll-plugins group, due to the fact github-pages will behave differently when enabling production ENV.
- Adds an extra 10s for the `host` to use `./buid.sh` due to compressing all files.
- Resolves #2186
Locking to 2.0.1 was a necessary workaround for a Netlify bug which has finally been resolved.
@jonaharagon jonaharagon marked this pull request as draft January 15, 2021 21:23
@jonaharagon
Copy link
Contributor Author

Blocked by netlify/build-image#517

@Gusted Gusted mentioned this pull request Jan 15, 2021
3 tasks
This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant