-
-
Notifications
You must be signed in to change notification settings - Fork 5.6k
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
Decorators 2 Transform [WIP] #6107
Closed
Closed
Changes from 1 commit
Commits
Show all changes
23 commits
Select commit
Hold shift + click to select a range
57cdfaf
Add syntax plugin for decorators2
peey 9904b61
WIP decorator transform initial commit
peey 1d64084
Merge branch '7.0' into decorators-2-transform-wip
peey f214d1d
wip commiting code before major changes
peey edb7ff2
Transform works great, move stuff to helpers
peey 10d763e
Clean up code
peey 358cf70
Add feature to pass array of deps to a helper
peey 626b92e
Add exec tests
peey d8cebba
Add tests for evaluation order and fix transformation
peey ef4eba5
Add tests for class decorators; minor fixes
peey b1092ca
Add test for enumerability decorator
peey a55e374
Add examples, & tests for extras, finishers
peey 5f7bf93
Merge branch '7.0' into decorators-2-transform-wip
peey 8707f4e
Update versions to make tests pass
peey 6ec36a1
Improvements in handling of computed keys
peey 2e7c6a1
Add predicate hasDecorators (for early exit)
peey 8e28ca7
use path.scope.parent.push & other review suggestions
peey 1776e24
Account for getters and setters
peey 4e527a3
Pass elementDescriptor to decorators not propertyDescriptor
peey 44465f2
configurability checks
peey f86b327
Merge getters and setters; Map->object in decorate
peey 96f51aa
Add failing test for instanceof
peey 76f5d3c
Fix parentage tests. It's not an issue
peey File filter
Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this should be fixed
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not sure if this is related to this PR. I'll investigate more