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

fix: prepare next-major-spec for release #421

Merged

Conversation

jonaslagoni
Copy link
Member

@jonaslagoni jonaslagoni commented Aug 7, 2023

Description
This PR adds the final changes to next-major-spec that is needed before release, this includes:

  • Added migration guide for v6
  • Added migration guide for v5 (was missing)
  • Added v3 schemas to the all-schema store

Related issue(s)
Blocking #208

@sonarcloud
Copy link

sonarcloud bot commented Aug 7, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@jonaslagoni jonaslagoni changed the title feat: prepare next-major-spec for release fix: prepare next-major-spec for release Oct 9, 2023
@jonaslagoni
Copy link
Member Author

Adapted this PR to match the desire to not release before the spec.

Copy link
Member

@smoya smoya left a comment

Choose a reason for hiding this comment

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

@derberg @jonaslagoni Would you mind answering (maybe it has been answered already somewhere else) the following https://github.com/asyncapi/spec-json-schemas/pull/421/files/b43d3d650d24ab99c9e2a7436d1ba7d1adb60bc9..340e842a3a27498361022e743b4239fbb77d593e#r1350501180 before moving forward?

Thank you 🙏

Copy link
Member

@fmvilas fmvilas left a comment

Choose a reason for hiding this comment

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

LGTM 👍

Copy link
Collaborator

@char0n char0n left a comment

Choose a reason for hiding this comment

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

LGTM

@derberg
Copy link
Member

derberg commented Nov 28, 2023

looks like everybody agree that #421 (comment) is a direction so @jonaslagoni you can update this PR and we can merge it before release

Copy link

sonarcloud bot commented Nov 30, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

Copy link
Member

@smoya smoya left a comment

Choose a reason for hiding this comment

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

LGTM! 🚀🌔

@fmvilas
Copy link
Member

fmvilas commented Nov 30, 2023

/rtm

@asyncapi-bot asyncapi-bot merged commit 24e105a into asyncapi:next-major-spec Nov 30, 2023
11 checks passed
@derberg
Copy link
Member

derberg commented Nov 30, 2023

oh wait, so what about below @jonaslagoni

looks like everybody agree that #421 (comment) is a direction so @jonaslagoni you can update this PR and we can merge it before release

@fmvilas
Copy link
Member

fmvilas commented Nov 30, 2023

Looks like I was too fast pressing the button. @jonaslagoni feel free to create another PR to address your concerns, if any.

@jonaslagoni
Copy link
Member Author

All good 👍

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.

7 participants