Sync with upstream (to fix generated oneOf variable names)#7
Closed
sazapp-spectra wants to merge 466 commits intoSpectraLogic:masterfrom
Closed
Sync with upstream (to fix generated oneOf variable names)#7sazapp-spectra wants to merge 466 commits intoSpectraLogic:masterfrom
sazapp-spectra wants to merge 466 commits intoSpectraLogic:masterfrom
Conversation
* Add value types for the ASPNETCORE generator * generate samples
…alidation (#18332) * Fix java list validation * Fix java list validation * Fix java list validation * Fix java list validation
* webclient: update junit to 5.x * update samples * update samples * update samples * update samples * update samples * clean up * update samples * update pom plugin version * update tests
* fix: template * chore: samples
Depending on test exec environment, this test could linger around for ~8 minutes, b/c pinging test.com will not resolve and run into seveal timeouts.
…code (#18751) * Fix tests for parsing of additional type annotations These were comparing the same things with each other and this could not fail. Now using (as probably intended by the author) the before unused ArrayList for assertion of the expectation. * Remove Exception from signature that is never thrown * Simplify assertions using AssertJ * Replace stub implementation with abstract Mockito mock * Cache flat parsed openapi results for faster test execution Simply caching any calls to TestUtils.parseFlattenSpec that occur at least twice. * Fix some "Raw use of parameterized class 'Schema'" warnings
* Updated crystal dependencies to remedy build failure with a mutually conflicting library version. * Samples updated from running PR-suggested build.
* Even with .nil? typecheck, compiler still fails due to .empty? call on nillable type. This appears a poor implementation issue by crystal-lang, but this explicit try avoids the issue for now. * Made call more succinct.
* Fix javaJaxRS template: remove deprecated in JDK9 method usage * Fix javaJaxRS api template: regenerate samples
* Added authentication via Bearer-token api_key and basic for server and client * Improved errorhandling * Added check on oAuth audience to example * Updates of the petstore files for Rust-server * Moved module import to prevent issue in callbacks * updated samples * Fix for unused-qualifications issue * updated sampmles
Bumps [s4u/setup-maven-action](https://github.com/s4u/setup-maven-action) from 1.12.0 to 1.13.0. - [Release notes](https://github.com/s4u/setup-maven-action/releases) - [Commits](s4u/setup-maven-action@v1.12.0...v1.13.0) --- updated-dependencies: - dependency-name: s4u/setup-maven-action dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* moved service api into shared source * moved service api into shared source * This Scala cask change makes the service traits available to Both JVM and JS platforms. It does this by moving the api interfaces into the cross-platform Shared sources
…cs, maven & gradle plugin (#18568) * Mention that Golang generator also supports XML annotations Looking at src/main/resources/go/model_simple.mustache and src/main/java/org/openapitools/codegen/languages/GoClientCodegen.java the GoLang seems to cater for withXml=true * Fix maven plugin config description for `withXml` * Add basic test for global withXml setting * Use global withXml setting if not configured in ConfigOptions Resolves #3839 and #5764 * Don't generate metadata or other files when only Models are tested * Reformat table for readability * Remove global property `withXml` from generator and docs * Move WITH_XML constant out of system constants block Currently there is only a single reference to this value in the whole codebase (GoClientOptionsProvider). Maybe we should re-think how this file is organised (i.e. provide a clearer split / mapping / understanding what are system properties vs. global properties vs. configOptions and where to put them). * Remove global option `withXml` from Maven plugin (Breaking change) This is a "soft" breaking change: Plugin will no longer execute if user have this option – which is good, b/c it never worked as expected. We may want to hint this in the 8.0 release notes. * Remove global property `withXml` from Gradle plugin (Breaking change) This is a "soft" breaking change: Plugin will no longer execute if user have this option – which is good, b/c it never worked as expected. We may want to hint this in the 8.0 release notes, so they can add it to the `configOptions` map if required, or simply delete it * Update samples to reflect removed `withXml` property * Move `withXml` option into ConfigOptions for Java Microprofile sample * Remove unused local vars and parameters * Avoid repetition using fluent assertions * Remove extraneous debug output
…minator (#19219) * Add samples using `oneOf` without discriminator * Fix typescript-fetch code gen for `oneOf` without discriminator * Return `{}` instead of original `json`
* Adding decimal to template for Python generator * Rerunning the build steps * Add tests for decimal serialization and deserialization. * Move test to python not legacy pydantic sample * readd old imports --------- Co-authored-by: Adam <abolfik@pollyex.com>
…terface (#19212) * fix #19211 kotlin-spring flag delegatePattern together with skipDefaultInterface generates broken code * add kotlin-springboot-delegate-nodefaults to the workflow * fix Platform declaration clash * move kotlin-springboot-delegate-nodefaults to the kotlin-server-jdk17 file * fixed empty line
…9202) * [Kotlin][Spring] Document already supported vendor extensions * [Kotlin][Spring] Add support for x-{class,field}-extra-annotation
…9230) This commit updates the access modifier of the ExtendedCodegenModel class from its default (package-private) to public, allowing it to be accessible from outside its package. Co-authored-by: Paulo Da Rocha Filho <paulo.darochafilho@wnco.com>
* rust: support repr(int) enum * Regen APIS * remove the extra lines * merge most recent commits from master * update tests to ensure that enum compiles correctly * drop changes to kotlin files --------- Co-authored-by: Jihyun Yu <yjh0502@gmail.com>
* feat: add scope for oauth2 * fix: add parameter AlwaysMultipartFormData * fix: string to boolean * fix: optional bool * fix: null checks * Optional string * Remove all references to OAuthMultipartFormData * Remove _multipartFormData = multipartFormData; * Remove typo * Run generate-samples and export_docs_generators * Revert "Run generate-samples and export_docs_generators" This reverts commit f051f26. * Switch to string.IsNullOrEmpty, add langVersion 8 * Add langVersion 8 in ConditionalSerialization * Use regular strings for netstandard2.0 * Remove references to langVersion 8 * Fix variable * Use template engine to toggle nullable string * Trigger tests * Generate samples * Trigger build * Use {{nrt?}} * update samples --------- Co-authored-by: Morten Jansrud <morten.jansrud@snokam.no>
* Add ArrayRule * Run ./bin/utils/export_docs_generators.sh * Add ArrayValidationErrorKind * Add validation method * Run ./bin/generate-samples.sh * Add array rule property * Rename minItem and maxItem => minItems and maxItems * Fix uniqueItems template * Tweaks * Add sample property * Run ./bin/generate-samples.sh
* set style and explode * update client * fix * set the newer version
* [Kotlin][Spring] fix #19244 integer enum * fix embedded array enum
* Update users.yml Adding aspectivo users.yml * Add files via upload Adding aspectivo logo file
…9253) * [Kotlin][Client] Encode default values with kotlinx serialization * [Kotlin][Client] Encode default values with kotlinx serialization
* revert problematic java-camel XML change * generate samples
* support response ranges * revert unintended change * update try deserialize methods
…ssignment" in to_dict() (#19223) * [python] mypy fix for multiple arrays of objects * [python] mypy test for multiple arrays of objects
…cial square implementation of rxJava3 adapter (#19266)
…ned (#19232) * use default values for required vars when available * update tests to existing config file * cleanup OAS test * Remove setting value in Get. * Use Pascal Case for variable naming in Getter Function * add a CLI option * add a CLI option * add a CLI option
* update docstring in go client, better code format * update samples
I have not been active in the project for nearly 4 years, but remain as a pingable maintainer on the readme. This is updating the readme to be accurate. I only spend around 1-3 hours every other month lately on open-source, so it's just an issue of time/availability.
…'oneOf' (#19183) * fix oneOf var names when no discriminator is used * add OneOfWithComplexType to Go petstore schema - an array of strings variant should end up with the same valid name whether `useOneOfDiscriminatorLookup` is `true` or `false`
Signed-off-by: Emilien Escalle <emilien.escalle@escemi.com>
…rong derive clause on data classes which have a parent as well as are serializable. (#19283)
Collaborator
Author
|
gonna try this a different way |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
20 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
I recently got a fix merged upstream to enable us to use
oneOfin our new PATCH-based basic settings schema. This is an attempt to sync with upstream to pull in that and other fixes that don't conflict with our parameter handling changes.PR checklist
Commit all changed files.
This is important, as CI jobs will verify all generator outputs of your HEAD commit as it would merge with master.
These must match the expectations made by your contribution.
You may regenerate an individual generator by passing the relevant config(s) as an argument to the script, for example
./bin/generate-samples.sh bin/configs/java*.IMPORTANT: Do NOT purge/delete any folders/files (e.g. tests) when regenerating the samples as manually written tests may be removed.
master(upcoming 7.1.0 minor release - breaking changes with fallbacks),8.0.x(breaking changes without fallbacks)