-
-
Notifications
You must be signed in to change notification settings - Fork 3
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-oddcontrast #519
update-oddcontrast #519
Conversation
View diff of compiled files (may take a few minutes): https://github.com/oddbird/oddleventy-built/compare/main..update-oddcontrast |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me! I deleted the contrast.jpg
image, because I don't think it was needed. Let me know if it is needed, and I should restore it.
README.md
Outdated
be used automatically for any `yarn` command. To activate it, run | ||
`corepack enable` | ||
(once per local Node installation). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@SondraE Out of curiosity, why this change?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jgerigmeyer There was a line break between corepack and enable that meant the corepack enable
command wasn't highlighted. Because it wasn't highlighted, I missed it. I asked James why I was getting errors, and he directed me to it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh, in your local editor? That makes sense. Wrapping doesn't impact highlighting on GitHub: https://github.com/oddbird/oddleventy#install-node-and-yarn
Steps to test/reproduce