Skip to content

Conversation

@sophiejanssens
Copy link
Contributor

No description provided.

Copy link
Contributor

@bartveenstra bartveenstra left a comment

Choose a reason for hiding this comment

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

The example contains backbase specfic examples. such as the bom file. Please use the bom file that is already in the examples.

Copy link
Contributor

@bartveenstra bartveenstra left a comment

Choose a reason for hiding this comment

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

Plus code coverage must be on par :)

@@ -0,0 +1,82 @@
<?xml version="1.0" encoding="UTF-8"?>
Copy link
Contributor

Choose a reason for hiding this comment

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

You must not refer to internal artifacts or specs

<modules>
<module>angular</module>
<module>java-clients</module>
<!-- <module>java-clients</module>-->
Copy link
Contributor

Choose a reason for hiding this comment

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

Why is this disabled?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sorry that was accident

<goal>doc</goal>
</goals>
<configuration>
<inputSpec>${project.basedir}/src/main/resources/contact-manager_contact-manager-service-api-v2.3.0.yaml</inputSpec>
Copy link
Contributor

Choose a reason for hiding this comment

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

this is an internal API and cannot be used

for (File f : inputSpecs) {
inputSpec = f.getPath();
output = new File(outPutDirectory.getPath(),f.getName().substring(0,f.getName().lastIndexOf(".")).concat("-docs"));
output = new File(outPutDirectory.getPath(), f.getName().substring(0, f.getName().lastIndexOf(".")).concat("-docs"));
Copy link
Contributor

Choose a reason for hiding this comment

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

Output are always docs? Also for client generation?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The directory input was only wanted for doc goal

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I can change it for others if wanted

Copy link
Contributor

Choose a reason for hiding this comment

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

Ah! I see that now! Sorry. Disregard the previous comment :)

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 2 Code Smells

81.1% 81.1% Coverage
0.0% 0.0% Duplication

@bartveenstra
Copy link
Contributor

@pa314159 Can you also review this?

@bartveenstra bartveenstra merged commit 5bc3389 into main May 11, 2021
@bartveenstra bartveenstra deleted the feature/input-maven-artifact branch July 16, 2021 08: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.

Allow the generate mojos (lint, doc, code gen, etc) to be able to read the specs from maven artefacts coordinates

3 participants