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

[scala][client] add Scala code generation test #3859

Merged
merged 7 commits into from
Sep 12, 2019
Merged

[scala][client] add Scala code generation test #3859

merged 7 commits into from
Sep 12, 2019

Conversation

sullis
Copy link
Contributor

@sullis sullis commented Sep 8, 2019

PR checklist

  • Read the contribution guidelines.
  • Ran the shell script under ./bin/ to update Petstore sample so that CIs can verify the change. (For instance, only need to run ./bin/{LANG}-petstore.sh, ./bin/openapi3/{LANG}-petstore.sh if updating the {LANG} (e.g. php, ruby, python, etc) code generator or {LANG} client's mustache templates). Windows batch files can be found in .\bin\windows\. If contributing template-only or documentation-only changes which will change sample output, be sure to build the project first.
  • Filed the PR against the correct branch: master, 4.1.x, 5.0.x. Default: master.
  • Copied the technical committee to review the pull request if your PR is targeting a particular programming language.

Description of the PR

(details of the change, additional tests that have been done, reference to the issue for tracking, etc)

@sullis
Copy link
Contributor Author

sullis commented Sep 8, 2019

@ramzimaalej @shijinkui

@sullis
Copy link
Contributor Author

sullis commented Sep 8, 2019

@wing328

@sullis sullis changed the title [scala][client] add Scala reserved words test [scala][client] add Scala code generation test Sep 8, 2019
@sullis
Copy link
Contributor Author

sullis commented Sep 10, 2019

@ramzimaalej LGTM :)

@sullis
Copy link
Contributor Author

sullis commented Sep 11, 2019

@wing328 can you merge this?

@wing328
Copy link
Member

wing328 commented Sep 12, 2019

@sullis sure, we can. We were just busy releasing v4.1.2: https://twitter.com/oas_generator/status/1172068944355528705

@wing328 wing328 added this to the 4.1.3 milestone Sep 12, 2019
@wing328 wing328 merged commit e56bfe4 into OpenAPITools:master Sep 12, 2019
@sullis sullis deleted the add-scalaReservedWordTest branch September 12, 2019 14:20
jimschubert added a commit that referenced this pull request Sep 13, 2019
* master: (22 commits)
  various minor improvements to nim generator (#3883)
  Add Nim client code generator (#3879)
  updated google-api-client version from 1.23.0 to 1.30.2.  Bugfix #3625 (#3882)
  typescript-fetch: fix missing comma in multiple imports (#3881)
  [scala][client] add Scala code generation test (#3859)
  typescript-node: Use HttpError class when rejecting promises (#3876)
  update stable release
  update doc, samples (#3875)
  BugFix #2053 Spring Boot fails to parse LocalDate query parameter (#3860)
  fix version in readme
  Prepare v4.1.2 release (#3873)
  Rename gRPC generator to "protobuf-schema" (#3864)
  typescript-axios: Fix baseoptions (#3866)
  Fix/r/serialization fix and minor 3xx resp fix (#3817)
  [typescript-angular] allow empty string basePath (#3489)
  typescript-inversify: improve check for required parameters, support multiple media types (#3849)
  [C++][Pistache] Add missing setter for arrays (#3837)
  Adds Http Info To Dart Api (#3851)
  [Java][okhttp-gson] fix failure to deserialize floats (#3846)
  maven-plugin-plugin 3.6.0 (#3854)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants