-
Notifications
You must be signed in to change notification settings - Fork 315
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
Add a "Project" suffix to the type
of project IDs
#9392
base: main
Are you sure you want to change the base?
Conversation
ee1d841
to
b51312d
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #9392 +/- ##
=========================================
Coverage 67.91% 67.91%
Complexity 1262 1262
=========================================
Files 244 244
Lines 8724 8724
Branches 909 909
=========================================
Hits 5925 5925
Misses 2422 2422
Partials 377 377
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Signed-off-by: Sebastian Schuberth <[email protected]>
a0d2bfe
to
212fcad
Compare
To avoid one root cause of duplicate project vs package IDs, add a "Project" suffix to the `type` of project IDs. This change does not affect IDs used in package curation or package configurations as these really only apply to packages, not projects. Signed-off-by: Sebastian Schuberth <[email protected]>
212fcad
to
54aeadd
Compare
This would solve my scan issues I believe, any timeline when it will be merged? https://github.com/phoenixframework/phoenix/blob/main/package.json
private repository anonymized |
@olfkadolf, unfortunately your case wouldn't be solved by this PR, which is about package-vs-project duplicates. Your case is about a project-vs-project duplicate. |
Please have a look at the individual commit messages for the details.