-
Notifications
You must be signed in to change notification settings - Fork 0
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
Update package.json #5
base: master
Are you sure you want to change the base?
Conversation
Signed-off-by: Per Ploug <[email protected]>
🤖 zincr found 2 problems , 0 warnings
Details on how to resolve are provided below ApprovalsAll proposed changes must be reviewed by project maintainers before they can be merged Not enough people have approved this pull request - please ensure that atleast 2 users who have not contributed to this pull request approve the changes.
Dependency LicensingAll dependencies specified in package manager files must be reviewed, banned dependency licenses will block the merge, all new dependencies introduced in this pull request will give a warning, but not block the merge Please ensure that only dependencies with licenses compatible with the license of this project is included in the pull request. |
1 similar comment
🤖 zincr found 2 problems , 0 warnings
Details on how to resolve are provided below ApprovalsAll proposed changes must be reviewed by project maintainers before they can be merged Not enough people have approved this pull request - please ensure that atleast 2 users who have not contributed to this pull request approve the changes.
Dependency LicensingAll dependencies specified in package manager files must be reviewed, banned dependency licenses will block the merge, all new dependencies introduced in this pull request will give a warning, but not block the merge Please ensure that only dependencies with licenses compatible with the license of this project is included in the pull request. |
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.
looks good to me!
🤖 zincr found 2 problems , 0 warnings
Details on how to resolve are provided below ApprovalsAll proposed changes must be reviewed by project maintainers before they can be merged Not enough people have approved this pull request - please ensure that atleast 2 users who have not contributed to this pull request approve the changes.
Dependency LicensingAll dependencies specified in package manager files must be reviewed, banned dependency licenses will block the merge, all new dependencies introduced in this pull request will give a warning, but not block the merge Please ensure that only dependencies with licenses compatible with the license of this project is included in the pull request. |
🤖 zincr found 2 problems , 0 warnings
Details on how to resolve are provided below ApprovalsAll proposed changes must be reviewed by project maintainers before they can be merged Not enough people have approved this pull request - please ensure that atleast 2 users who have not contributed to this pull request approve the changes.
Dependency LicensingAll dependencies specified in package manager files must be reviewed, banned dependency licenses will block the merge, all new dependencies introduced in this pull request will give a warning, but not block the merge Please ensure that only dependencies with licenses compatible with the license of this project is included in the pull request. |
Signed-off-by: Per Ploug [email protected] #4 .