This repository has been archived by the owner. It is now read-only.

Forbid semicolons after decorators in classes #352

Merged
merged 1 commit into from Feb 22, 2017

Conversation

Projects
None yet
2 participants
@bakkot
Copy link
Contributor

bakkot commented Feb 10, 2017

Q A
Bug fix? yes
Breaking change? yes
New feature? no
Deprecations? no
Spec compliancy? yes
Tests added/pass? yes
Fixed tickets #85
License MIT

The decorators proposal does not allow (and as far as I'm aware has never allowed) semicolons after decorators in class bodies, but they're currently accepted (treated as empty ClassElements, basically). This PR fixes that.

@danez

This comment has been minimized.

Copy link
Member

danez commented Feb 22, 2017

Thanks

@danez danez merged commit 561d4f7 into babel:master Feb 22, 2017

3 checks passed

codecov/patch 100% of diff hit (target 97.65%)
Details
codecov/project 97.65% (+<.01%) compared to 898c4a7
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.