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

feat: update next major spec with master #289

Conversation

jonaslagoni
Copy link
Sponsor Member

Description
This PR updates the next_major_spec branch with master commits

char0n and others added 17 commits September 29, 2022 09:01
Co-authored-by: Sergio Moya <1083296+smoya@users.noreply.github.com>
Co-authored-by: asyncapi-bot <bot+chan@asyncapi.io>
Co-authored-by: asyncapi-bot-eve <bot+eve@asyncapi.io>
Co-authored-by: Jeremy Whitlock <jwhitlock@apache.org>
Co-authored-by: derberg <lpgornicki@gmail.com>
Co-authored-by: Sergio Moya <1083296+smoya@users.noreply.github.com>
…spec_with_master

# Conflicts:
#	index.js
#	package-lock.json
#	package.json
@jonaslagoni
Copy link
Sponsor Member Author

Important that you merge this PR as merge commit and not a squash commit, this means you have to change some settings in the repo most likely!

Otherwise, the merge conflicts will come back to haunt us later on 😄

cc codeowners @fmvilas @whitlockjc @derberg @dalelane @smoya

@fmvilas
Copy link
Member

fmvilas commented Nov 21, 2022

Shouldn't this PR point to next-major-spec branch? 🤔

@jonaslagoni jonaslagoni changed the base branch from master to next-major-spec November 21, 2022 23:22
@jonaslagoni
Copy link
Sponsor Member Author

Of course @fmvilas 🤦 Changing.

fmvilas
fmvilas previously approved these changes Nov 21, 2022
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.

I'm happy to merge manually with a merge commit. Let's wait for the others to see what they have to say.

dalelane
dalelane previously approved these changes Dec 1, 2022
derberg
derberg previously approved these changes Dec 5, 2022
@jonaslagoni jonaslagoni dismissed stale reviews from derberg, dalelane, and fmvilas via a10f733 December 9, 2022 17:59
@jonaslagoni
Copy link
Sponsor Member Author

Updated with the latest master changes.

@jonaslagoni jonaslagoni requested review from fmvilas and removed request for char0n and smoya December 9, 2022 18:01
@sonarcloud
Copy link

sonarcloud bot commented Dec 9, 2022

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

@fmvilas
Copy link
Member

fmvilas commented Dec 9, 2022

@dalelane @derberg please approve again 🙏

@fmvilas fmvilas merged commit d4d5197 into asyncapi:next-major-spec Dec 12, 2022
@fmvilas
Copy link
Member

fmvilas commented Dec 12, 2022

Alright, merged with merge commit @jonaslagoni

@asyncapi-bot
Copy link
Contributor

🎉 This PR is included in version 4.1.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@asyncapi-bot
Copy link
Contributor

🎉 This PR is included in version 5.0.0-next-major-spec.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

@jonaslagoni jonaslagoni deleted the feature/update_next_major_spec_with_master branch February 7, 2023 10:05
@asyncapi-bot
Copy link
Contributor

🎉 This PR is included in version 6.0.0-next-major-spec.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

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.

None yet