search

ScottLogic / openapi-forge

⚒️🔥 Effortlessly create OpenAPI clients, in a range of languages, from the fiery furnace of our forge
8 stars 7 forks source link

Generator options - merge to master #160

Closed ms14981 closed 1 year ago

ms14981 commented 1 year ago

I've raised https://github.com/ScottLogic/openapi-forge/pull/162 to move generate.js back to its original location, which represents option 4 below, so that we can sort the file locations out properly in https://github.com/ScottLogic/openapi-forge/issues/158 without having to worry about it in this PR, which can then have a reduced scope (the generator-options command)

BREAKING CHANGES: This will need updates to the JS and TS generator for those generators to continue working.

Possible routes forward:

  1. Make PRs for the JS and TS changes, and merge at the same time.
  2. Address https://github.com/ScottLogic/openapi-forge/issues/158 first.
  3. Split out the commits that moves the generate.js file and merge only what's needed for the generator-options command.
  4. Put log.js and generate.js back in their original locations.
github-actions[bot] commented 1 year ago

:tada: This PR is included in version 1.4.0 :tada:

The release is available on:

Your semantic-release bot :package::rocket: