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

refactor: publish openapi artifact with api group as classifier #247

Merged
merged 1 commit into from
Jul 8, 2024

Conversation

ndr-brt
Copy link
Member

@ndr-brt ndr-brt commented Jul 8, 2024

What this PR changes/adds

Publish the openapi artifact with the apiGroup as classifier when specified.

Why it does that

It will be easier for downstream project to filter/group specs by apiGroup

Further notes

List other areas of code that have changed but are not necessarily linked to the main feature. This could be method
signature changes, package declarations, bugs that were encountered and were fixed inline, etc.

Linked Issue(s)

Part of #244

Please be sure to take a look at the contributing guidelines and our etiquette for pull requests.

@ndr-brt ndr-brt added documentation Improvements or additions to documentation build labels Jul 8, 2024
Copy link

github-actions bot commented Jul 8, 2024

Test Results

49 tests  ±0   49 ✅ ±0   20s ⏱️ ±0s
20 suites ±0    0 💤 ±0 
20 files   ±0    0 ❌ ±0 

Results for commit b6fdab9. ± Comparison against base commit 8c9baf4.

♻️ This comment has been updated with latest results.

@ndr-brt ndr-brt merged commit 9e6f3b9 into main Jul 8, 2024
19 checks passed
@ndr-brt ndr-brt deleted the 244-change-classifier branch July 8, 2024 12:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants