Skip to content

Modernization#2

Open
dsblv wants to merge 1 commit into
conceptualitis:masterfrom
dsblv:patch-2
Open

Modernization#2
dsblv wants to merge 1 commit into
conceptualitis:masterfrom
dsblv:patch-2

Conversation

@dsblv
Copy link
Copy Markdown
Contributor

@dsblv dsblv commented Feb 20, 2016

This PR is about making this repo appear cooler in users' eyes. It's important if you want to dominate among other similar mdules.

It includes:

  • Addition of .editorconfig to simplify following your code style
  • Addition of .gitattributes to ensure line endings are being normalized by git
  • Removal of .npmignore in favor of files key in package.json
  • Linting with XO that is a wrap around eslint with simpler API and sensible set of rules
  • Testing with AVA. Just check new test.js out!
  • A few package.json tweaks

This one is very opinionated, so feel free to decline 😉


I also encourage you to use travis with .travis.yml like this and stick a badge to the readme so everyone knows your tests are passing.

@conceptualitis
Copy link
Copy Markdown
Owner

At a glance I'm fine with these changes. I probably won't add Travis, because I don't want to use any more resources than this joke repo is already consuming 😆

good

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