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

Features/287 refactor maven metadata inferer #300

Merged
merged 9 commits into from
Dec 9, 2022

Conversation

Riduidel
Copy link
Owner

@Riduidel Riduidel commented Dec 8, 2022

Seems to look quite good. Some changes need to be made to have a more apparent structure seggregation between

  • detecting the maven pom somewhere
  • handling the peculiarities jof that model (modules, inheritence, source code location, various additionnal concerns like issue manager and scm)

Repository owner deleted a comment from github-actions bot Dec 8, 2022
@github-actions
Copy link

github-actions bot commented Dec 8, 2022

Current coverage (as computed by Jacoco)

File Coverage [52.55%] 🍏
ModelElementMavenEnhancer.java 100% 🍏
MavenPomReader.java 68.93% 🍏
MavenPomDecorator.java 60.29% 🍏
MavenDetailsInfererEnhancer.java 51.32% 🍏
AbstractContainerEnhancer.java 27.25%
SoftwareSystemEnhancer.java 15.63%
ComponentEnhancer.java 0%
ContainerEnhancer.java 0%
Total Project Coverage 25.48% 🍏

Repository owner deleted a comment from github-actions bot Dec 8, 2022
@Riduidel Riduidel merged commit eef91a3 into main Dec 9, 2022
@delete-merged-branch delete-merged-branch bot deleted the features/287-refactor-maven-metadata-inferer branch December 9, 2022 09:00
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.

1 participant