-
-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
feat: extension identification improvements #2332
feat: extension identification improvements #2332
Conversation
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.
look into the given comments
packages/hoppscotch-app/helpers/strategies/ExtensionStrategy.ts
Outdated
Show resolved
Hide resolved
This pull request introduces 3 alerts and fixes 2 when merging da9ac91 into 432337b - view on LGTM.com new alerts:
fixed alerts:
|
This pull request introduces 3 alerts and fixes 2 when merging bd93d0a into 432337b - view on LGTM.com new alerts:
fixed alerts:
|
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.
lgtm 💯
Co-authored-by: Andrew Bastin <andrewbastin.k@gmail.com>
Closes #2176
Description
Checks
Additional Information
Requires hoppscotch/hoppscotch-extension#136 to be merged