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

Add additional identifiers #6518

Merged
merged 13 commits into from
Feb 12, 2025
Merged

Conversation

tas50
Copy link
Contributor

@tas50 tas50 commented Jan 5, 2025

  • New repology / cpe identifiers for many projects
  • Add another repology identifier one for Chef Infra Client since it's in repology twice
  • Fix the incorrect repology link for Chef Infra Server

@captn3m0
Copy link
Member

captn3m0 commented Jan 6, 2025

Thanks, will review this soon. Skimmed through, and most looked good.

@captn3m0 captn3m0 self-assigned this Jan 6, 2025
@captn3m0 captn3m0 added the product-updates Product content updates that are not fixes nor releases updates label Jan 6, 2025
products/cos.md Outdated Show resolved Hide resolved
@marcwrobel marcwrobel requested a review from captn3m0 January 12, 2025 17:11
@tas50 tas50 force-pushed the tas50/identifiers branch 2 times, most recently from 9d1b0a6 to be1f994 Compare January 14, 2025 23:38
@tas50
Copy link
Contributor Author

tas50 commented Jan 14, 2025

@captn3m0 I rebased this to resolve the conflict with the gerrit changes. Any chance you can take a peak at this before more conflicts pop up

@usta
Copy link
Member

usta commented Jan 17, 2025

@tas50 could you work with @chenrui333 ? I mean after you mentioned this PR i asked him to close his overlapping PR thus i dont want any of his work gone. And thank you both of you @tas50 @chenrui333 👍 💯 🥇 ❤️

@tas50
Copy link
Contributor Author

tas50 commented Jan 18, 2025

@usta let me see what I can do to pull that same work in

tas50 added 7 commits January 17, 2025 18:34
- New repology identifiers for many projects
- Add another one for Chef Infra Client since it's in repology twice
- Fix the incorrect repology link for Chef Infra Server

Signed-off-by: Tim Smith <[email protected]>
Signed-off-by: Tim Smith <[email protected]>
Signed-off-by: Tim Smith <[email protected]>
Signed-off-by: Tim Smith <[email protected]>
Signed-off-by: Tim Smith <[email protected]>
Signed-off-by: Tim Smith <[email protected]>
@tas50 tas50 force-pushed the tas50/identifiers branch from be1f994 to 4f41912 Compare January 18, 2025 02:34
Make these consistent

Signed-off-by: Tim Smith <[email protected]>
@usta
Copy link
Member

usta commented Jan 18, 2025

[except angular because there were a small mistake in his PR about it ]

@tas50
Copy link
Contributor Author

tas50 commented Jan 29, 2025

@captn3m0 any chance you can take a look at this one?

products/dotnet.md Outdated Show resolved Hide resolved
products/elasticsearch.md Outdated Show resolved Hide resolved
products/jquery.md Show resolved Hide resolved
products/vmware-harbor-registry.md Outdated Show resolved Hide resolved
@marcwrobel
Copy link
Member

@tas50, I have a few suggestions / questions but overall really really nice work !

@tas50
Copy link
Contributor Author

tas50 commented Feb 11, 2025

@marcwrobel Suggestions LGTM. Thanks!

@tas50
Copy link
Contributor Author

tas50 commented Feb 11, 2025

@marcwrobel I have another change lined up to sort all the identifiers but I'd rather get this one in before that big shuffle change.

Copy link
Member

@marcwrobel marcwrobel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, merging.

@marcwrobel marcwrobel merged commit 5ce8def into endoflife-date:master Feb 12, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
product-updates Product content updates that are not fixes nor releases updates
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants