Make canHaveModifiers/Decorators public by rbuckton · Pull Request #50399 · microsoft/TypeScript
Navigation Menu
{{ message }}
microsoft / TypeScript Public
- Notifications You must be signed in to change notification settings
- Fork 13.3k
Merged
Conversation
Copy link
Contributor
rbuckton
commented
Aug 22, 2022
rbuckton
commented
Makes canHaveModifiers and canHaveDecorators public.
Related #49939
@DanielRosenwasser If it's not too late we should probably take this for 4.8 as well.
rbuckton
requested review from
andrewbranch and
sandersn
typescript-bot
assigned
rbuckton
typescript-bot
added
Author: Team
For Uncommitted Bug
labels
Aug 22, 2022
rbuckton
mentioned this pull request
andrewbranch approved these changes Aug 22, 2022
Copy link
Member
DanielRosenwasser
commented
Aug 22, 2022
DanielRosenwasser commented
Aug 22, 2022@typescript-bot cherry-pick this to release-4.8 and LKG
Copy link
Collaborator
Heya @DanielRosenwasser, I've started to run the task to cherry-pick this into release-4.8 on this PR at 86f4962. You can monitor the build here.
Copy link
Collaborator
typescript-bot
commented
Aug 22, 2022
typescript-bot commented
Aug 22, 2022Hey @DanielRosenwasser, I couldn't open a PR with the cherry-pick. (You can check the log here). You may need to squash and pick this PR into release-4.8 manually.
rbuckton
merged commit
aaa4f9d
into
main
rbuckton
deleted the
modifiers-decorators-test-api
branch
rbuckton
mentioned this pull request
bradennapier
mentioned this pull request
microsoft
locked as resolved and limited conversation to collaborators
This file contains hidden or 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
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
4 participants