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

feat(importCDX): enhance component VCS naming convention for improved consistency #2744

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

sameed20
Copy link
Contributor

Description

This MR introduces improvements to the naming convention of components by leveraging the VCS (Version Control System) field. By utilizing this field, the component names will now be more descriptive and reflective of their origins, ensuring less duplicity of components and also maintaining the DB integrity.

@sameed20 sameed20 added needs code review needs general test This is general testing, meaning that there is no org specific issue to check for labels Nov 25, 2024
Copy link
Contributor

@heliocastro heliocastro left a comment

Choose a reason for hiding this comment

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

THIs is an implementation question. Are we plan to do some logic solution or we will keep updating checks with random urls all the time ?

@GMishx GMishx added this to the Release - 20.0.0 milestone Nov 27, 2024
@afsahsyeda
Copy link
Contributor

afsahsyeda commented Nov 27, 2024

We can extract the domain specific logic in a dedicated class file which will also allow us to add more domains in the future and/or identify commonalities between different domain repository URL structures to arrive at a more generalized solution.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs code review needs general test This is general testing, meaning that there is no org specific issue to check for
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants