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

Improve repo tagging #6343

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

chrisrueger
Copy link
Contributor

@chrisrueger chrisrueger commented Oct 29, 2024

As discussed with @pkriens today:

  • The Repo Browser does not show 'resolve' tag anymore, for repos which do not have defined a tag

image

  • instead only if developer has manually entered a tag in a repo it is shown

image

image

Also fixed a bug I discovered regarding Repository implementations which currently do not implement the Tagged interface. Those repositories should still be considered for resolution by default (as if they have the resolve tag). This is e.g. the case for any custom Respository plugin in the wild which implements Repository but not Tagged (because Tagged interface is just being added in 7.1.0)

it is still possible there are custom Repository plugins which do not implement the Tagged interface yet. those repositories should still be considered for resolving

Signed-off-by: Christoph Rueger <chrisrueger@gmail.com>
This does not show 'resolve' on all repos in the Repo Browser anymore

Signed-off-by: Christoph Rueger <chrisrueger@gmail.com>
Signed-off-by: Christoph Rueger <chrisrueger@gmail.com>
Signed-off-by: Christoph Rueger <chrisrueger@gmail.com>
Signed-off-by: Christoph Rueger <chrisrueger@gmail.com>
@chrisrueger
Copy link
Contributor Author

@pkriens condition improved and Glob support added.

as suggested

javadoc

Signed-off-by: Christoph Rueger <chrisrueger@gmail.com>
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

Successfully merging this pull request may close these issues.

2 participants