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.

Sign up for GitHub

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

Jump to bottom

chore(decorator): opt code in serverlessTrigger #3164

Merged
merged 1 commit into from
Aug 14, 2023
Merged

chore(decorator): opt code in serverlessTrigger #3164

merged 1 commit into from
Aug 14, 2023

Conversation

taosher
Copy link
Contributor

@taosher taosher commented Aug 13, 2023

target.constructor is not required because target is compatible with function attachClassMetadata

Checklist
  • npm test passes
  • tests and/or benchmarks are included
  • documentation is changed or added
  • commit message follows commit guidelines
Affected core subsystem(s)
Description of change

@taosher
chore(decorator): opt code in serverlessTrigger
740c9a3 
```target.constructor``` is not required because ```target``` is compatible with function ```attachClassMetadata```
@czy88840616 czy88840616 added the pr: maintenance This PR does not produce any behavior differences to end users when upgrading. label Aug 14, 2023
@czy88840616 czy88840616 merged commit f30ccc7 into midwayjs:main Aug 14, 2023
@open-digger-bot open-digger-bot bot mentioned this pull request Aug 21, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
pr: maintenance This PR does not produce any behavior differences to end users when upgrading.
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@taosher @czy88840616