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: use latest ruby and newer jekyll versions #4487

Merged
merged 4 commits into from
Oct 17, 2024

Conversation

jmeridth
Copy link
Member

@jmeridth jmeridth commented Oct 12, 2024

The resulting https://localhost:4000 redirects to https://localhost:4000/feed.json and it includes the whistleblowing content. Still looking into that.

Screenshot 2024-10-12 at 1 37 37 PM

My changes listed below in each section under My Changes

Please confirm this pull request meets the following requirements:

Which change are you proposing?

  • Suggesting edits to an existing topic or collection
  • Curating a new topic or collection
  • Something that does not neatly fit into the binary options above

Editing an existing topic or collection

I'm suggesting these edits to an existing topic or collection:

  • Image (and my file is *.png, square, dimensions 288x288, size <= 75 kB)
  • Content (and my changes are in index.md)

My Changes

Curating a new topic or collection

  • I've formatted my changes as a new folder directory, named for the topic or collection as it appears in the URL on GitHub (e.g. https://github.com/topics/[NAME] or https://github.com/collections/[NAME])
  • My folder contains a *.png image (if applicable) and index.md
  • All required fields in my index.md conform to the Style Guide and API docs: https://github.com/github/explore/tree/main/docs

Please replace this line with an explanation of why you think this topic or collection should be curated.

Something that does not neatly fit into the binary options above

  • My suggested edits are not about an existing topic or collection, or at least not a single one
  • My suggested edits are not about curating a new topic or collection, or at least not a single one
  • My suggested edits conform to the Style Guide and API docs: https://github.com/github/explore/tree/main/docs

My Changes

  • upgrade to ruby 3.3.5 from 3.1.2, update .ruby-version (3.1 is end of life in 5 months)
  • use latest github-pages gem which uses Jekyll 3.10.0 (from 6/2024)
    • previous was using jekyll 2.4.0 (was from 10 years ago)
  • setup .devcontainer for local development
  • add Gemfile.lock (remove from .gitignore) so we can update dependabot config to include gems/bundler (weekly)
  • fix markdown formatting in CONTRIBUTING.md

Please note: we will close your PR without comment if you do not check the boxes above and provide ALL requested information.

- [x] ruby 3.3.5, update .ruby-version
- [x] use latest github-pages gem which uses Jekyll 3.10.0 (from 6/2024)
  - previous was using jekyll 2.4.0 (was from 10 years ago)
- [x] setup .devcontainer for local development
- [x] add Gemfile.lock (remove from .gitignore)
- [x] update dependabot config to include gems/bundler (weekly)
- [x] fix markdown formatting in CONTRIBUTING.md
- [x] fix liquid parsing issue of handlebars/index.md (backticks to quotes)
  - [Error](https://github.com/github/explore/actions/runs/11299945094/job/31431925940#step:6:82)

Signed-off-by: jmeridth <[email protected]>
@jmeridth jmeridth self-assigned this Oct 12, 2024
@jmeridth jmeridth requested a review from kenyonj October 12, 2024 18:33
@@ -10,4 +10,4 @@ related: mustache-js, ejs-templates, pug, nunjucks, jinja2, liquid, javascript,
released: February 15, 2011
github_url: https://github.com/handlebars-lang
---
**Handlebars** is a simple templating language. It uses a template and an input object to generate HTML or other text formats. Handlebars templates look like regular text with embedded Handlebars expressions. Handlebars is largely compatible with Mustache templates. In most cases it is possible to swap out Mustache with Handlebars and continue using your current templates. A handlebars expression is a `{{`, some contents, followed by a `}}`. When the template is executed, these expressions are replaced with values from an input object.
**Handlebars** is a simple templating language. It uses a template and an input object to generate HTML or other text formats. Handlebars templates look like regular text with embedded Handlebars expressions. Handlebars is largely compatible with Mustache templates. In most cases it is possible to swap out Mustache with Handlebars and continue using your current templates. A handlebars expression is a "{{", some contents, followed by a "}}". When the template is executed, these expressions are replaced with values from an input object.
Copy link
Member Author

Choose a reason for hiding this comment

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

changing from backticks to quotes fixed liquid warning

@jmeridth
Copy link
Member Author

This might be too much for one PR. 🤷

@jmeridth jmeridth marked this pull request as ready for review October 12, 2024 18:41
@jmeridth jmeridth requested a review from a team as a code owner October 12, 2024 18:41
Copy link
Contributor

@tomthorogood tomthorogood left a comment

Choose a reason for hiding this comment

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

@jmeridth jmeridth added this pull request to the merge queue Oct 17, 2024
Merged via the queue into main with commit fc52dac Oct 17, 2024
8 checks passed
@jmeridth jmeridth deleted the jm_ruby_jekyll_update_versions branch October 17, 2024 18:26
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