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

docs: add spectral ruleset docs #790

Merged
merged 5 commits into from
Jun 28, 2023

Conversation

jonaslagoni
Copy link
Member

Description
This is a continuum of @magicmatatjahu PR in #700

Related issue(s)
Fixes #698

@jonaslagoni
Copy link
Member Author

Gonna close #698 after this is merged.

@jonaslagoni
Copy link
Member Author

@smoya can you have a look? 🙂

@fmvilas
Copy link
Member

fmvilas commented Jun 26, 2023

Yeah @smoya cause I have no idea 😅

@smoya
Copy link
Member

smoya commented Jun 26, 2023

Yeah @smoya cause I have no idea 😅

Gonna add to my todo list 😅 omg parental leave

README.md Outdated Show resolved Hide resolved
docs/ruleset/README.md Outdated Show resolved Hide resolved
docs/ruleset/README.md Outdated Show resolved Hide resolved
docs/ruleset/core-ruleset.md Outdated Show resolved Hide resolved
docs/ruleset/core-ruleset.md Outdated Show resolved Hide resolved
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.

I added some suggestions. Some of them more nitpick, others more importants.

@jonaslagoni jonaslagoni requested a review from smoya June 28, 2023 12:27
@sonarcloud
Copy link

sonarcloud bot commented Jun 28, 2023

Please retry analysis of this Pull-Request directly on SonarCloud.

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! 🚀🌔

Thanks @magicmatatjahu and @jonaslagoni for bringing this 🙌

@jonaslagoni jonaslagoni merged commit 6f84151 into asyncapi:master Jun 28, 2023
@jonaslagoni jonaslagoni deleted the add_ruleset_docs branch June 28, 2023 17:33
@jonaslagoni
Copy link
Member Author

Thanks for the review @smoya 💪

@asyncapi-bot
Copy link
Contributor

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

The release is available on:

Your semantic-release bot 📦🚀

@asyncapi-bot
Copy link
Contributor

🎉 This PR is included in version 2.1.0 🎉

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.

Move Spectral ruleset to this repo
4 participants