Skip to content
This repository has been archived by the owner on Nov 3, 2020. It is now read-only.

feat: Deprecate tslint and eslint all the things. #329

Merged
merged 2 commits into from
May 11, 2019

Conversation

beausmith
Copy link
Contributor

@beausmith beausmith commented May 9, 2019

Replace tslint with eslint, stylelint, commitlint, etc.

Sync'd with BAS linting, etc.

@netlify
Copy link

netlify bot commented May 9, 2019

Deploy preview for vw-bmd ready!

Built with commit 2ba074a

https://deploy-preview-329--vw-bmd.netlify.com

Copy link
Contributor

@benadida benadida left a comment

Choose a reason for hiding this comment

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

I'm a little annoyed by the super strict commit message formatting, it is super constraining when doing development and encourages commit squashing too early.

If it's important to you, I can live with it, but let's keep an eye on the paper cuts because I've found development on BAS to be harder as a result.

@beausmith
Copy link
Contributor Author

Yes, I agree. It's a little pre-optimized. Let's discuss and fix it.

@benadida
Copy link
Contributor

I can live with it if there is no obvious easier path.

@beausmith
Copy link
Contributor Author

I've removed commitlint and will merge this without a commit linter for now.

@beausmith beausmith merged commit 51d5e3e into master May 11, 2019
@beausmith beausmith deleted the eslint-all-the-things branch May 11, 2019 16:34
@beausmith
Copy link
Contributor Author

Captured here: #338

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants