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

Use openapi generate from node #21

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

Conversation

dpash
Copy link

@dpash dpash commented Feb 15, 2024

I had some issues getting the project to build, with finding the correct version of openapi generator.

To make it slightly easier to set up the environment, this commit uses @openapitools/openapi-generator-cli to fetch the relevant version. We're stopping at 6.6.0 as later versions have issues with the project. We also need Java 15 or earlier, as Java 16 closes access to a number of classes that OpenAPI uses (via the Java Moustache library).

The project was originally dependent on openapi being installed, but
that was causing issues finding the relevant version to install.

To make it slightly easier to set up the environment, this commit
uses `@openapitools/openapi-generator-cli` to fetch the relevant
version. We're stopping at 6.6.0 as later versions have issues with
the project. We also need Java 15 or earlier, as Java 16 closes access
to a number of classes that OpenAPI uses (via the Java Moustache
library).
@dpash
Copy link
Author

dpash commented Feb 15, 2024

I don't know why the quote escaping was required to get it to build for me (without it, it would generate files in ./src/ModelsCPUSFeeds/Cause.php which is obviously incorrect.

I also noticed that it's generating test files, but they're not in the repo, so I've just added that to .gitignore for now.

My alternative plan was a docker based setup, so there's no requirement for node to be installed, but obviously that requires a docker install instead.

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