Skip to content

TF changes rebased#1

Open
luisgabrielroldan wants to merge 59 commits intotf-changesfrom
tf-changes-rebased
Open

TF changes rebased#1
luisgabrielroldan wants to merge 59 commits intotf-changesfrom
tf-changes-rebased

Conversation

@luisgabrielroldan
Copy link
Member

Description of the change

Rebasing tf-changes with last official changes

Type of change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Configuration change
  • Technical Debt
  • Documentation

Related tickets

Link to Shortcut/Jira Ticket Goes Here

Checklists

Development and Testing

  • Lint rules pass locally.
  • The code changed/added as part of this pull request has been covered with tests, or this PR does not alter production code.
  • All tests related to the changed code pass in development, or tests are not applicable.

Code Review

  • This pull request has a descriptive title and information useful to a reviewer. There may be a screenshot or screencast attached.
  • At least two engineers have been added as "Reviewers" on the pull request.
  • Changes have been reviewed by at least two other engineers who did not write the code.
  • This branch has been rebased off master to be current.

Tracking

  • Issue from Shortcut/Jira has a link to this pull request.
  • This PR has a link to the issue in Shortcut.

QA

  • This branch has been deployed to staging and tested.

thbar and others added 30 commits July 25, 2023 18:32
…open-api-spex#550)

* Fix deprecation warning on Elixir 1.15

* Stop testing against Elixir 1.10

* Require Elixir 1.11+ now

* Update CHANGELOG.md
…-api-spex#552)

Co-authored-by: Gianluca Nitti <gianluca.nitti@voismart.it>
…#545)

* feat: add assert_operation_response, assert_raw_schema

* make assert_operation_response pipeable

* fix return type

* automagically infer operationId in assertion

* don't need to resolve a %Schema{}

* ignore 204s

* use OpenApiSpex.OpenApi.json_encoder()

* rename test to match fn

* reorganize json_encoder check per feedback

* update json_encoder message for :jason or :poison

* use a regex to match json content types in validate_operation_response

* 💅 feedback - types, error message, module attrib for regex

* add doc for content_type_from_header

* remove no_return from spec
Co-authored-by: Alberto Sartori <alberto.sartori@hpe.com>
* Exclude empty paths from spec
* fix: assert_operation_response header lookup
…x#592)

* Add failing test

* Cast result of AllOf cast into a struct

* Shorter module name
…-spex#586)

* relax dependency on ymlr, and fix some tests

* test with more elixir versions
* Update Elixir version test matrix

* Fix map key order dependent test
…pex#593)

* Allow script and style nonces

* Allow nonces on the SwaggerUIOAuth2Redirect plug as well
…#606)

* fix: ensure operation_id is always set on conn.private when an operation is known
zachdaniel and others added 29 commits July 8, 2024 09:19
…pen-api-spex#613)

* improvement: use struct spec to avoid double `%` in struct inspect

currently, inspecting an `OpenApiSpex.Schema` looks like this:

`%OpenApiSpex.Schema%{type: :object}`

with the new change, it looks like this:

`%OpenApiSpex.Schema{type: :object}`

which allows copy/pasting the struct
…le (open-api-spex#618)

Add a --check option to run Mix tasks and compare the generated spec
with a previously generated file.

This is useful for scenarios where a CI check is desirable to catch
unwanted drifts from a validated OpenAPI spec: e.g. the OpenAPI spec is
committed and reviewed, and should not change without additional review.

Fixes open-api-spex#617

Signed-off-by: Davide Briani <davide@briani.dev>
* Fix cast x-validate when decoded schema

* fix credo complain when using apply/3

---------

Co-authored-by: Giorgio Torres <giorgio.torres@hpe.com>
* Add mix spec.yaml tasks in example applications

* Update example application dependencies
* Fix Elixir 1.18 compilation warnings

* Test against 1.18
@luisgabrielroldan luisgabrielroldan self-assigned this Jun 10, 2025
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.