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 gem quote consistency on docs #9517

Merged
merged 1 commit into from
Jan 12, 2024
Merged

Conversation

akirataguchi115
Copy link
Contributor

Summary

Changed all documentation double quotes to single quotes. In some places there already was a use of single quotes and this is what Bundler suggests to use.

Context

Documentation.

@mattr-
Copy link
Member

mattr- commented Dec 30, 2023

👋 Jekyll's style guide says to prefer double quotes over single quotes.

Rather than switch things to single quotes, would you be willing to switch things to double quotes instead?

@akirataguchi115
Copy link
Contributor Author

Good to know! And yeah sure I'm willing to amend the commit to double quotes instead of single quotes instead. Could you link me the style guide first?

@mattr-
Copy link
Member

mattr- commented Dec 30, 2023

.rubocop.yml in this repository are the rules. We don't have anything "officially" documented 😅

@akirataguchi115
Copy link
Contributor Author

That's good enough documentation for me. I changed the single quotes to double quotes on all .md pages.

Copy link
Member

@mattr- mattr- left a comment

Choose a reason for hiding this comment

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

Thanks!

@jekyllbot: merge +docs

@jekyllbot jekyllbot merged commit 01da87c into jekyll:master Jan 12, 2024
11 checks passed
jekyllbot added a commit that referenced this pull request Jan 12, 2024
github-actions bot pushed a commit that referenced this pull request Jan 12, 2024
Akira Taguchi: Fix gem quote consistency on docs (#9517)

Merge pull request 9517
@chmaynard
Copy link
Contributor

Not sure if this is related, but the Jekyll docs site is no longer working correctly. See issues #9527 and #9528.

monfresh pushed a commit to monfresh/jekyll that referenced this pull request Jun 5, 2024
monfresh pushed a commit to monfresh/jekyll that referenced this pull request Jun 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants