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

Npm updates #33

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Npm updates #33

wants to merge 1 commit into from

Conversation

brettz9
Copy link

@brettz9 brettz9 commented Apr 29, 2020

  • Update to Unicoode 13; closes Update identifier data to Unicode v11 #27
  • Linting: As per latest jshint (define function before called)
  • Testing: Use chai's register-expect
  • npm: Update devDeps (including moving from coffee-script to non-deprecated coffeescript)
  • npm: Simplify script running
  • npm: Avoid adding package-lock.json
  • npm: Update mocha per latest API
  • npm: Add recommended package.json fields: keywords, bugs, dependencies and change to author/contributors
  • Maintenance: Add .editorconfig

(I have a PR built on top of this I intend to submit next which replaces jshint with eslint and replaces coffeescript entirely, but trying to keep it more atomic.)

@brettz9 brettz9 mentioned this pull request Apr 29, 2020
- Linting: As per latest jshint (define function before called)
- Testing: Use chai's `register-expect`
- npm: Update devDeps (including moving from `coffee-script` to non-deprecated `coffeescript`)
- npm: Simplify script running
- npm: Avoid adding `package-lock.json`
- npm: Update mocha per latest API
- npm: Add recommended package.json fields: keywords, bugs, dependencies and change to author/contributors
- Maintenance: Add `.editorconfig`
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.

1 participant