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

Breaking: change no-comma-dangle to comma-dangle (fixes #1350) #1946

Merged
merged 1 commit into from Mar 6, 2015

Conversation

lo1tuma
Copy link
Member

@lo1tuma lo1tuma commented Mar 5, 2015

No description provided.

@lo1tuma
Copy link
Member Author

lo1tuma commented Mar 5, 2015

Travis build failed because of #1947

@nzakas
Copy link
Member

nzakas commented Mar 6, 2015

Okay, if you rebase it should work now.

@lo1tuma
Copy link
Member Author

lo1tuma commented Mar 6, 2015

Rebased.

@nzakas
Copy link
Member

nzakas commented Mar 6, 2015

Awesome, thanks. I know this will make @vjeux very happy. :)

Can you also add this to the tracking issue for rule removal?

nzakas added a commit that referenced this pull request Mar 6, 2015
Breaking: change no-comma-dangle to comma-dangle (fixes #1350)
@nzakas nzakas merged commit 0d08ba4 into master Mar 6, 2015
@vjeux
Copy link

vjeux commented Mar 6, 2015

Awesome, can't wait for the release so that i can enable this on react native repo :)

@lo1tuma lo1tuma deleted the issue1350 branch March 6, 2015 16:30
@eslint-deprecated eslint-deprecated bot locked and limited conversation to collaborators Feb 7, 2018
@eslint-deprecated eslint-deprecated bot added the archived due to age This issue has been archived; please open a new issue for any further discussion label Feb 7, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
archived due to age This issue has been archived; please open a new issue for any further discussion
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants