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.

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

Bugfix/webview version upgrade #14

Merged
merged 6 commits into from
Mar 26, 2022
Merged

Conversation

CAMOBAP
Copy link
Collaborator

@CAMOBAP CAMOBAP commented Mar 24, 2022

Notes:

Unfortunatelly I wasn't able to verify the fix, npm completely ignore peerDependency check for local modules (so I wasn't able to reproduce the issue locally)

@e271828- if you know how we can 'force' this check for local modules please let me know I will update CI

@CAMOBAP CAMOBAP requested a review from e271828- March 24, 2022 07:42
@CAMOBAP CAMOBAP self-assigned this Mar 24, 2022
@CAMOBAP CAMOBAP force-pushed the bugfix/webivew-version-upgrade branch 9 times, most recently from 73e4048 to 382e898 Compare March 24, 2022 08:54
@CAMOBAP CAMOBAP force-pushed the bugfix/webivew-version-upgrade branch from 382e898 to 9ab5dfc Compare March 24, 2022 09:23
md5.js Show resolved Hide resolved
@e271828- e271828- changed the title Bugfix/webivew version upgrade Bugfix/webview version upgrade Mar 24, 2022
@e271828-
Copy link
Contributor

npm completely ignore peerDependency check for local modules

https://www.npmjs.com/package/check-peer-dependencies perhaps?

@CAMOBAP
Copy link
Collaborator Author

CAMOBAP commented Mar 24, 2022

npm completely ignore peerDependency check for local modules

https://www.npmjs.com/package/check-peer-dependencies perhaps?

Looks exactly what we need, I will update PR later

@CAMOBAP CAMOBAP force-pushed the bugfix/webivew-version-upgrade branch from 3f63068 to 9a1f535 Compare March 25, 2022 07:01
@CAMOBAP
Copy link
Collaborator Author

CAMOBAP commented Mar 25, 2022

@e271828- done

@CAMOBAP CAMOBAP requested a review from e271828- March 26, 2022 17:05
@e271828- e271828- merged commit 23826bd into master Mar 26, 2022
@e271828- e271828- deleted the bugfix/webivew-version-upgrade branch March 26, 2022 18:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants