Skip to content
This repository has been archived by the owner on Jul 29, 2024. It is now read-only.

feat!: support 2.1.0 version of spec #41

Merged

Conversation

magicmatatjahu
Copy link
Contributor

Description

  • change the logic for getPayloadExamples and getHeadersExamples functions to support the name and summary fields from item of message.examples
  • add tests
  • install parser-js as dev-dependency, only for testing purpose.

Related issue(s)
Part of asyncapi/spec#536

@magicmatatjahu magicmatatjahu added the enhancement New feature or request label Jun 25, 2021
@magicmatatjahu magicmatatjahu changed the title Support 2.1.0 feat!: support 2.1.0 version of spec Jun 25, 2021
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Welcome to AsyncAPI. Thanks a lot for creating your first pull request. Please check out our contributors guide and the instructions about a basic recommended setup useful for opening a pull request.

Keep in mind there are also other channels you can use to interact with AsyncAPI community. For more details check out this issue.

Copy link
Collaborator

@derberg derberg left a comment

Choose a reason for hiding this comment

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

just 2 things

src/customFilters.test.js Outdated Show resolved Hide resolved
src/customFilters.test.js Outdated Show resolved Hide resolved
src/customFilters.test.js Outdated Show resolved Hide resolved
src/customFilters.test.js Outdated Show resolved Hide resolved
@magicmatatjahu magicmatatjahu requested a review from derberg June 25, 2021 14:41
.eslintrc Outdated

overrides:
- files: "**/*.test.js"
Copy link
Collaborator

Choose a reason for hiding this comment

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

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Nope 😞 Still 36% of duplication...

@derberg
Copy link
Collaborator

derberg commented Jun 28, 2021

ok, last try, try again in esconfig but explicitly link to the test file that causes errors please

@derberg
Copy link
Collaborator

derberg commented Jun 28, 2021

you have some errors ineslint config file and pr testing is failing

@magicmatatjahu magicmatatjahu requested a review from derberg June 28, 2021 09:46
@derberg
Copy link
Collaborator

derberg commented Jun 28, 2021

👏🏼 you decided to actually refactor rather than silet the rule, bravo 👏🏼

derberg
derberg previously approved these changes Jun 28, 2021
@magicmatatjahu
Copy link
Contributor Author

@derberg Yep, but current tests with this helper function aren't readable as were.

@derberg
Copy link
Collaborator

derberg commented Jun 28, 2021

I think https://github.com/asyncapi/modelina/blob/676726ac6385afbd901a5ebc0c5ad0c2a1064499/sonar-project.properties should do the trick, I just reminded myself about it that Jonas asked me for the same some day and this is how we fixed it

@magicmatatjahu
Copy link
Contributor Author

I also had sonar.properties file and didn't work.

@derberg
Copy link
Collaborator

derberg commented Jun 28, 2021

but with full filepath like jonas?

@magicmatatjahu
Copy link
Contributor Author

@derberg Yes.

@sonarcloud
Copy link

sonarcloud bot commented Jun 29, 2021

Kudos, SonarCloud 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

@magicmatatjahu magicmatatjahu merged commit 183635d into asyncapi-archived-repos:master Jun 29, 2021
@magicmatatjahu magicmatatjahu deleted the support-2.1.0 branch June 29, 2021 11:30
@asyncapi-bot
Copy link
Contributor

🎉 This PR is included in version 2.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request released
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants