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

chore: run eslint --fix on server files #4132

Conversation

jniles
Copy link
Collaborator

@jniles jniles commented Jan 27, 2020

This commit runs eslint --fix on server files and manually patches issues that cannot be fixed automatically.

@kwilu kwilu added the size: XL label Jan 27, 2020
This commit runs `eslint --fix` on server files and manually patches
issues that cannot be fixed automatically.
@jniles jniles force-pushed the chore-apply-eslint-fixes-to-server-files branch from 19297fa to bd0e66f Compare January 27, 2020 09:45
@jniles jniles requested a review from lomamech January 27, 2020 11:46
@jniles
Copy link
Collaborator Author

jniles commented Feb 3, 2020

bors r+

bors bot added a commit that referenced this pull request Feb 3, 2020
4131: deps: bump package versions r=jniles a=jniles

Bumps some major package versions so that they are up to date.  Notably,
this includes:
  1. `mkdirp`
  2. `mocha`
  3. `cross-env`

Additionally, I've removed lzma-native since it sometimes causes crashes
on windows and replaced it with the native zlib.  This also fits into
our regular backup strategy.

Closes #4126.
Closes #4127.
Closes #4129.

4132: chore: run eslint --fix on server files r=jniles a=jniles

This commit runs `eslint --fix` on server files and manually patches issues that cannot be fixed automatically.

Co-authored-by: Jonathan Niles <jonathanwniles@gmail.com>
@bors
Copy link
Contributor

bors bot commented Feb 3, 2020

@bors bors bot merged commit bd0e66f into Third-Culture-Software:master Feb 3, 2020
@jniles jniles deleted the chore-apply-eslint-fixes-to-server-files branch February 3, 2020 14:11
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