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

dash-licenses marks dependencies as failing #166

Open
vince-fugnitto opened this issue Apr 28, 2022 · 2 comments
Open

dash-licenses marks dependencies as failing #166

vince-fugnitto opened this issue Apr 28, 2022 · 2 comments

Comments

@vince-fugnitto
Copy link
Member

Description

We currently use dash-licenses in eclipse-theia and noticed that the tool reports errors in dependencies despite them not being changed. This happened 3 days this week so I thought I'd bring it to the project's attention (I had to submit automated reviews for these dependencies).

Is there a reason that the check passes one day, and not the next despite nothing on our side changing? Is it related to #81?

Additional Info:

Automated review requests this week:

@vince-fugnitto
Copy link
Member Author

@waynebeaton
Copy link
Member

This is not related to issue #81 (that should only impact Java).

There's at least two different things happening.

My best guess with npm/npmjs/-/eslint-module-utils/2.7.3 and npm/npmjs/-/normalize-package-data/2.5.0 is that the ClearlyDefined scores changed (dropped), so they no longer pass based on ClearlyDefined and are flagged as requiring further review. This sort of thing is expected; though my expectation is that one average, we move more from restricted to approved than the other way around.

The other two had previously been reviewed and approved by us, so they should be okay. I'll investigate.

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

No branches or pull requests

2 participants