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

should we support tslint@next ? #56

Open
bhovhannes opened this issue Dec 7, 2016 · 1 comment
Open

should we support tslint@next ? #56

bhovhannes opened this issue Dec 7, 2016 · 1 comment

Comments

@bhovhannes
Copy link
Collaborator

This issue has been created to continue discussion opened at pull request #55, which tried to fix issue #53 .

In order to keep master branch clean all changes from PR #55 were moved into new dev branch.

If we decided to have 2 release channels, stable channel will be published from master branch and unstable (potentially) dev channel will be published from dev branch.
If we decided to have 1 release channel and support both stable and *-dev versions of tslint, we can just merge that dev branch into master.
If we decided to support only stable versions of tslint, we can just delete that dev branch.

@Strate
Copy link
Contributor

Strate commented Dec 7, 2016

As far as I understood from previous conversation, we do not want to get unstable version of tslint while installing tslint-loader with npm2. If this is the main reason, we could remove peerDependency of tslint at all, and force user to install preferred version of linter.

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

No branches or pull requests

2 participants