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

Feature/oidf 4 #9

Closed
wants to merge 7 commits into from
Closed

Feature/oidf 4 #9

wants to merge 7 commits into from

Conversation

zoemaas
Copy link
Contributor

@zoemaas zoemaas commented Jul 9, 2024

No description provided.

@zoemaas zoemaas requested a review from jcmelati July 9, 2024 14:18

### Profiles

The Open API generator will generate only models, infrastructures and apis by default. To make it generate apis. To make
Copy link
Contributor

Choose a reason for hiding this comment

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

These sentenses stop halfway and are conflicting

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Probabably Ctrl-Z was pressed at some point. Hopefully it makes sense now

packageName.set("com.sphereon.oid.fed.openapi")
apiPackage.set("$openApiPackage.api")
modelPackage.set("$openApiPackage.models")
inputSpec.set("$rootDir/src/main/kotlin/com/sphereon/oid/fed/openapi/openapi.yaml")
Copy link
Collaborator

Choose a reason for hiding this comment

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

I had to update it to inputSpec.set("$rootDir/modules/openapi/src/main/kotlin/com/sphereon/oid/fed/openapi/openapi.yaml") for it to run

Copy link
Contributor

Choose a reason for hiding this comment

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

Depends on where you are running the command prob

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 have the old problem "works on my machine" 😄

Copy link
Contributor Author

@zoemaas zoemaas Jul 10, 2024

Choose a reason for hiding this comment

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

changing from rootDir to projectDir fixes the problem

@zoemaas
Copy link
Contributor Author

zoemaas commented Jul 16, 2024

This PR will be closed and another one will be created for feature/OIDF-33. All comments are already addressed

@zoemaas zoemaas closed this Jul 16, 2024
@zoemaas zoemaas deleted the feature/OIDF-4 branch July 16, 2024 10:58
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.

3 participants