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
fix: export decorators for backward compatibility #850
Conversation
|
||
describe('backward-compatibility', () => { | ||
it('exports functions from @loopback/openapi-v2', async () => { | ||
try { |
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.
No try-catch
is needed. The test passes if it complies.
describe('backward-compatibility', () => { | ||
it('exports functions from @loopback/openapi-v2', async () => { | ||
try { | ||
class Test { |
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.
You need to add /* tslint:disable-next-line:no-unused-variable */
to fix the style issue:
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.
Please address my comments to pass CI.
5b2bedd
to
1d718bf
Compare
@slnode test please |
@raymondfeng thank you for reviewing, feedbacks applied but after rebasing, module Triggered a new build. |
@slnode test please |
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.
Please rebase to latest master before merging.
related issue: error TS2305: Module has no exported member 'get'. strongloop/loopback#3753
test cases