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

adds openapi deprecated support as a dsl. #3497

Merged
merged 7 commits into from Mar 18, 2024
Merged

Conversation

xlanor
Copy link
Contributor

@xlanor xlanor commented Mar 18, 2024

added deprecated support as a dsl for both v2 and v3 on HTTP routes.

Fixes #3449

added deprecated support

Signed-off-by: xlanor <contact@jingk.ai>

fixed missing import

Signed-off-by: xlanor <contact@jingk.ai>

refactor v3

Signed-off-by: xlanor <contact@jingk.ai>

refactor v2

Signed-off-by: xlanor <contact@jingk.ai>
Signed-off-by: xlanor <contact@jingk.ai>
Signed-off-by: xlanor <contact@jingk.ai>
Copy link
Member

@raphael raphael left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks great! I left a few nitpicks that would be nice to fix before merging, thank you!

dsl/method.go Outdated Show resolved Hide resolved
dsl/method.go Outdated Show resolved Hide resolved
http/codegen/openapi/v2/builder.go Outdated Show resolved Hide resolved
http/codegen/openapi/v3/builder.go Outdated Show resolved Hide resolved
Signed-off-by: xlanor <contact@jingk.ai>
Signed-off-by: xlanor <contact@jingk.ai>
@xlanor xlanor requested a review from raphael March 18, 2024 16:45
@xlanor
Copy link
Contributor Author

xlanor commented Mar 18, 2024

Really not sure what's going on here because I've switched indentation to spaces and it shows up funny in github.

image

Signed-off-by: xlanor <contact@jingk.ai>
Signed-off-by: xlanor <contact@jingk.ai>
@xlanor
Copy link
Contributor Author

xlanor commented Mar 18, 2024

Should be good to go now once CI passes.

@raphael
Copy link
Member

raphael commented Mar 18, 2024

This is great, thank you!

@raphael raphael merged commit f3d7996 into goadesign:v3 Mar 18, 2024
11 checks passed
ti-chi-bot bot pushed a commit to PingCAP-QE/ee-apps that referenced this pull request Apr 16, 2024
[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [goa.design/goa/v3](https://togithub.com/goadesign/goa) | `v3.14.1` ->
`v3.16.1` |
[![age](https://developer.mend.io/api/mc/badges/age/go/goa.design%2fgoa%2fv3/v3.16.1?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://developer.mend.io/api/mc/badges/adoption/go/goa.design%2fgoa%2fv3/v3.16.1?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://developer.mend.io/api/mc/badges/compatibility/go/goa.design%2fgoa%2fv3/v3.14.1/v3.16.1?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://developer.mend.io/api/mc/badges/confidence/go/goa.design%2fgoa%2fv3/v3.14.1/v3.16.1?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|

---

### Release Notes

<details>
<summary>goadesign/goa (goa.design/goa/v3)</summary>

### [`v3.16.1`](https://togithub.com/goadesign/goa/releases/tag/v3.16.1)

[Compare
Source](https://togithub.com/goadesign/goa/compare/v3.16.0...v3.16.1)

#### What's Changed

- Save r.URL.Query() in a variable by
[@&#8203;duckbrain](https://togithub.com/duckbrain) in
[goadesign/goa#3506
- chore: fix function names in comment by
[@&#8203;goodfirm](https://togithub.com/goodfirm) in
[goadesign/goa#3504
- go fmt ./... by [@&#8203;tchssk](https://togithub.com/tchssk) in
[goadesign/goa#3507
- Fix handling of custom gen packages by
[@&#8203;raphael](https://togithub.com/raphael) in
[goadesign/goa#3508

#### New Contributors

- [@&#8203;duckbrain](https://togithub.com/duckbrain) made their first
contribution in
[goadesign/goa#3506
- [@&#8203;goodfirm](https://togithub.com/goodfirm) made their first
contribution in
[goadesign/goa#3504

**Full Changelog**:
goadesign/goa@v3.16.0...v3.16.1

### [`v3.16.0`](https://togithub.com/goadesign/goa/releases/tag/v3.16.0)

[Compare
Source](https://togithub.com/goadesign/goa/compare/v3.15.2...v3.16.0)

#### What's Changed

- Add OpenAPI deprecated support to DSL. by
[@&#8203;xlanor](https://togithub.com/xlanor) in
[goadesign/goa#3497
- Customize proto message name by
[@&#8203;nitinmohan87](https://togithub.com/nitinmohan87) in
[goadesign/goa#3498
- Fix eval.IncompatibleDSL() to hide internal DSL by
[@&#8203;tchssk](https://togithub.com/tchssk) in
[goadesign/goa#3502

#### New Contributors

- [@&#8203;xlanor](https://togithub.com/xlanor) made their first
contribution in
[goadesign/goa#3497

**Full Changelog**:
goadesign/goa@v3.15.2...v3.16.0

### [`v3.15.2`](https://togithub.com/goadesign/goa/releases/tag/v3.15.2)

[Compare
Source](https://togithub.com/goadesign/goa/compare/v3.15.1...v3.15.2)

#### What's Changed

- Rename to codegen/service/templates/security_authfuncs.go.tpl by
[@&#8203;tchssk](https://togithub.com/tchssk) in
[goadesign/goa#3496

**Full Changelog**:
goadesign/goa@v3.15.1...v3.15.2

### [`v3.15.1`](https://togithub.com/goadesign/goa/releases/tag/v3.15.1)

[Compare
Source](https://togithub.com/goadesign/goa/compare/v3.15.0...v3.15.1)

#### What's Changed

- Add openapi:json:indent Meta by
[@&#8203;tchssk](https://togithub.com/tchssk) in
[goadesign/goa#3480
- Fix validation code for projected union types by
[@&#8203;raphael](https://togithub.com/raphael) in
[goadesign/goa#3487
- Return a validation error when query string is malformed by
[@&#8203;raphael](https://togithub.com/raphael) in
[goadesign/goa#3488
- Properly handle decoding of maps with nil entries by
[@&#8203;raphael](https://togithub.com/raphael) in
[goadesign/goa#3490
- Correctly handle union to union transforms by
[@&#8203;raphael](https://togithub.com/raphael) in
[goadesign/goa#3491

**Full Changelog**:
goadesign/goa@v3.15.0...v3.15.1

### [`v3.15.0`](https://togithub.com/goadesign/goa/releases/tag/v3.15.0)

[Compare
Source](https://togithub.com/goadesign/goa/compare/v3.14.6...v3.15.0)

#### What's Changed

- Standardize OpenAPI integer format by
[@&#8203;raphael](https://togithub.com/raphael) in
[goadesign/goa#3451
- Fix openapi:generate Meta to affect required validations by
[@&#8203;tchssk](https://togithub.com/tchssk) in
[goadesign/goa#3452
- Merge duplicate mustGenerate() functions by
[@&#8203;tchssk](https://togithub.com/tchssk) in
[goadesign/goa#3453
- Generate valid OpenAPI specifications by
[@&#8203;raphael](https://togithub.com/raphael) in
[goadesign/goa#3454
- Embed template files at compile time by
[@&#8203;raphael](https://togithub.com/raphael) in
[goadesign/goa#3455
- Skip flaky tests by [@&#8203;raphael](https://togithub.com/raphael) in
[goadesign/goa#3458
- No need to checkout to report test coverage by
[@&#8203;raphael](https://togithub.com/raphael) in
[goadesign/goa#3459
- Use v3 branch for pushes and feature branch for PRs for code coverage
by [@&#8203;raphael](https://togithub.com/raphael) in
[goadesign/goa#3460
- Move templates out of Go code into their own files by
[@&#8203;raphael](https://togithub.com/raphael) in
[goadesign/goa#3456
- Fix env var expansion in DeepSource GitHub action by
[@&#8203;raphael](https://togithub.com/raphael) in
[goadesign/goa#3463
- Fix godoc for codegen.UserTypeLocation() by
[@&#8203;tchssk](https://togithub.com/tchssk) in
[goadesign/goa#3464
- Use crypto/rand for sampling by
[@&#8203;nitinmohan87](https://togithub.com/nitinmohan87) in
[goadesign/goa#3466
- Fix/trace sampler by [@&#8203;raphael](https://togithub.com/raphael)
in
[goadesign/goa#3467
- Make receiver consistent across all methods of ServiceError by
[@&#8203;raphael](https://togithub.com/raphael) in
[goadesign/goa#3468
- Treat nested paths for struct:pkg:path Meta by
[@&#8203;tchssk](https://togithub.com/tchssk) in
[goadesign/goa#3469
- Validate Array and Map for struct:pkg:path Meta by
[@&#8203;tchssk](https://togithub.com/tchssk) in
[goadesign/goa#3474
- Protect against invalid route captures by
[@&#8203;raphael](https://togithub.com/raphael) in
[goadesign/goa#3476

**Full Changelog**:
goadesign/goa@v3.14.6...v3.15.0

### [`v3.14.6`](https://togithub.com/goadesign/goa/releases/tag/v3.14.6)

[Compare
Source](https://togithub.com/goadesign/goa/compare/v3.14.5...v3.14.6)

#### What's Changed

- Properly generate format for primitive types in OpenAPI specs by
[@&#8203;tchssk](https://togithub.com/tchssk) in
[goadesign/goa#3447
- Properly handle MapParams by
[@&#8203;raphael](https://togithub.com/raphael) in
[goadesign/goa#3450

**Full Changelog**:
goadesign/goa@v3.14.5...v3.14.6

### [`v3.14.5`](https://togithub.com/goadesign/goa/releases/tag/v3.14.5)

[Compare
Source](https://togithub.com/goadesign/goa/compare/v3.14.4...v3.14.5)

#### What's Changed

- Fix godoc for openapi:generate Meta by
[@&#8203;tchssk](https://togithub.com/tchssk) in
[goadesign/goa#3443
- Add Speakeasy sponsorship banner by
[@&#8203;raphael](https://togithub.com/raphael) in
[goadesign/goa#3444
- Add APIName constant to generated code by
[@&#8203;raphael](https://togithub.com/raphael) in
[goadesign/goa#3445
- Corrected MapParams Behavior to Avoid Unintended Filtering by
[@&#8203;raphael](https://togithub.com/raphael) in
[goadesign/goa#3446

**Full Changelog**:
goadesign/goa@v3.14.4...v3.14.5

### [`v3.14.4`](https://togithub.com/goadesign/goa/releases/tag/v3.14.4)

#### What's Changed

- Add openapi:generate Meta support to dsl.Attribute by
[@&#8203;tchssk](https://togithub.com/tchssk) in
[goadesign/goa#3437
- Add API version constant to generated code by
[@&#8203;raphael](https://togithub.com/raphael) in
[goadesign/goa#3439
- Deprecate obsolete instrumentation by
[@&#8203;raphael](https://togithub.com/raphael) in
[goadesign/goa#3440
- Handle inline object array and map elements by
[@&#8203;raphael](https://togithub.com/raphael) in
[goadesign/goa#3442

**Full Changelog**:
goadesign/goa@v3.14.1...v3.14.4

###
[`v3.14.2`](https://togithub.com/goadesign/goa/compare/v3.14.1...v3.14.2)

[Compare
Source](https://togithub.com/goadesign/goa/compare/v3.14.1...v3.14.2)

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you
are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://developer.mend.io/github/PingCAP-QE/ee-apps).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy4yOTMuMCIsInVwZGF0ZWRJblZlciI6IjM3LjI5My4wIiwidGFyZ2V0QnJhbmNoIjoibWFpbiIsImxhYmVscyI6W119-->

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
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.

meta for externalDocs and deprecated?
2 participants