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.

Sign up for GitHub

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

Jump to bottom

refactor: add migration function from v2 to v1 API #607

Merged
merged 9 commits into from
Sep 15, 2022
Merged

refactor: add migration function from v2 to v1 API #607

merged 9 commits into from
Sep 15, 2022

Conversation

magicmatatjahu
Copy link
Member

@magicmatatjahu magicmatatjahu commented Sep 13, 2022 •
edited
Loading

Description

  • add new iterator for v1 API - due to fact that old AsyncAPIDocument exposes traverse function in one of the method we need to create another traverse function for schemas written in v1 API.
  • create migration function from v1 to v2 API and expose it .
  • add anonymous naming for schemas and messages.
  • add unit tests
  • update Readme.md.
  • return document not parsed from parser.parse() function.

cc @smoya

Related issue(s)
Part of #481
Part of #482

@magicmatatjahu magicmatatjahu added the enhancement New feature or request label Sep 13, 2022
@magicmatatjahu magicmatatjahu mentioned this pull request Sep 13, 2022
Closed
20 tasks
@magicmatatjahu magicmatatjahu marked this pull request as ready for review September 14, 2022 10:43
magicmatatjahu
magicmatatjahu commented Sep 14, 2022
View reviewed changes
src/old-api/external-docs.ts
@@ -14,4 +15,36 @@ export class ExternalDocs extends SpecificationExtensionsModel<v2.ExternalDocume
description() {
return description(this);
}

hasExtensions() {
Copy link
Member Author

Choose a reason for hiding this comment

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

I had to do that due to circular references between tag, externalDocs and mixins files.

src/old-api/tag.ts
@@ -22,4 +23,36 @@ export class Tag extends SpecificationExtensionsModel<v2.TagObject> {
hasExternalDocs() {
return hasExternalDocs(this);
}

hasExtensions() {
Copy link
Member Author

Choose a reason for hiding this comment

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

I had to do that due to circular references between tag, externalDocs and mixins files.

smoya
smoya reviewed Sep 14, 2022
View reviewed changes
README.md Outdated Show resolved Hide resolved
smoya
smoya reviewed Sep 14, 2022
View reviewed changes
README.md Show resolved Hide resolved
smoya
smoya reviewed Sep 14, 2022
View reviewed changes
src/utils.ts Show resolved Hide resolved
@sonarcloud
Copy link

sonarcloud bot commented Sep 14, 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

smoya
smoya approved these changes Sep 14, 2022
View reviewed changes
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! 🚀🌔 Great addition!

@magicmatatjahu
Copy link
Member Author

/rtm

@asyncapi-bot asyncapi-bot merged commit 1265ae6 into asyncapi:next-major Sep 15, 2022
@magicmatatjahu magicmatatjahu deleted the next/migrator-function branch September 15, 2022 07:21
magicmatatjahu added a commit to magicmatatjahu/parser-js that referenced this pull request Oct 3, 2022
@magicmatatjahu
refactor: add migration function from v2 to v1 API (asyncapi#607)
595f74d
derberg pushed a commit that referenced this pull request Oct 4, 2022
@magicmatatjahu @derberg
refactor: add migration function from v2 to v1 API (#607)
d8c3c03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@smoya smoya smoya approved these changes

@fmvilas fmvilas Awaiting requested review from fmvilas fmvilas is a code owner

@jonaslagoni jonaslagoni Awaiting requested review from jonaslagoni jonaslagoni is a code owner

@derberg derberg Awaiting requested review from derberg derberg is a code owner

@asyncapi-bot-eve asyncapi-bot-eve Awaiting requested review from asyncapi-bot-eve asyncapi-bot-eve is a code owner

Assignees
No one assigned
Labels
enhancement New feature or request ready-to-merge
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@magicmatatjahu @smoya @asyncapi-bot