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

Add nord-emacs #4655

Merged
merged 2 commits into from
Mar 28, 2017
Merged

Add nord-emacs #4655

merged 2 commits into from
Mar 28, 2017

Conversation

arcticicestudio
Copy link
Contributor

An arctic, north-bluish clean and elegant Emacs theme.

Designed for a fluent and clear workflow.
Based on the Nord color palette.

Nord Emacs is a 16 colorspace theme build to run in GUI- and terminal mode with support for many third-party syntax- and UI packages.


  • Direct link to the package repository:
    arcticicestudio/nord-emacs
  • Association with the package:
    Owner of the project
  • Relevant communications with the upstream package maintainer:
    None needed

Checklist

  • I've read CONTRIBUTING.md
  • I've used package-lint to check for packaging issues, and addressed its feedback
  • My elisp byte-compiles cleanly
  • M-x checkdoc is happy with my docstrings
  • I've built and installed the package using the instructions in the README

MELPA recipe PR
@purcell
Copy link
Member

purcell commented Mar 26, 2017

Thanks! I sent you some fixes in nordtheme/emacs#21. You'll also need to call this package nord-theme, of course, rather than nord-emacs: please rename the recipe file and adjust the contents to match

@arcticicestudio
Copy link
Contributor Author

arcticicestudio commented Mar 26, 2017

Thanks for your fast reply and the PR to fix this format issues 👍. I've reviewed and merged it already.

Do I need to update the change of the name in the recipe too or only the file name?
(nord-emacs ... -> (nord-theme ...

@purcell
Copy link
Member

purcell commented Mar 26, 2017

Do I need to update the change the name in the recipe too ot only the file name?
(nord-emacs ... -> (nord-theme ...

Yes please, both. The first element of the recipe is the package's name, which is nord-theme in this case.

@arcticicestudio
Copy link
Contributor Author

Updated the file- and package name.

@purcell purcell merged commit 31cb600 into melpa:master Mar 28, 2017
@purcell
Copy link
Member

purcell commented Mar 28, 2017

Awesome, thanks.

@arcticicestudio arcticicestudio deleted the patch-1 branch March 28, 2017 16:59
@arcticicestudio arcticicestudio mentioned this pull request Mar 28, 2017
10 tasks
microamp pushed a commit to microamp/melpa that referenced this pull request Jul 24, 2017
* Add nord-emacs theme

MELPA recipe PR

* Update file- and package name
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.

None yet

2 participants