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 syntax error introduced in GH-81 #82

Merged
merged 2 commits into from Dec 3, 2019

Conversation

rien333
Copy link
Contributor

@rien333 rien333 commented Dec 3, 2019

The theme fails to load due to this syntax error, please fix. (also, why is the melpa package connected to the development branch? because of things like this, it makes more sense to connect it to the master branch)

This syntax error keeps Nord from loading, please fix.
Copy link
Contributor

@arcticicestudio arcticicestudio left a comment

Choose a reason for hiding this comment

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

Hi @rien333 👋, thanks for your contribution 👍

Really good catch thanks 💪
I've also overlooked the extra brace in #81, wasn't really easy to see next to changed number.

I'm not a Emacs user, but I guess it makes sense to connect the develop branch to MELPA since Nord is available in both, the stable repository/registry as well as the bleeding edge one.

@arcticicestudio arcticicestudio changed the title Fix syntax error Fix syntax error introduced in GH-81 Dec 3, 2019
@arcticicestudio arcticicestudio merged commit 2fc8fe4 into nordtheme:develop Dec 3, 2019
@arcticicestudio arcticicestudio removed their assignment Dec 3, 2019
arcticicestudio referenced this pull request Dec 3, 2019
It now uses the whole "Frost" palette.
It used `nord8` for all levels before to match the style of Markdown
syntax in Emacs as well as other projects, but it makes sense for this
package to use different colors since users might expect it.
This also matches the default style of the Org-Mode package that makes
use of different colors.

Additionaly this comit removed a duplicate line.
arcticicestudio pushed a commit that referenced this pull request Jan 8, 2020
Fixed a syntax error introduced in GH-81 (PR) that added an extra brace. This resulted in error when trying to load the theme.
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

3 participants