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: work with cycle defendant properties #174

Merged

Conversation

Tenischev
Copy link
Member

Description

components:
  messages:
    lightMeasured:
      summary: Inform about environmental lighting conditions for a particular streetlight.
      payload:
        $ref: "#/components/schemas/lightMeasuredPayload"
  schemas:
    lightMeasuredPayload:
      type: object
      properties:
        lumens:
          type: integer
          minimum: 0
          description: Light intensity measured in lumens.
        childs:
          type: array
          items:
            $ref: "#/components/schemas/lightMeasuredPayload"

Related issue(s)
Fixes: #150

Tenischev added 30 commits April 8, 2020 21:18
# Conflicts:
#	filters/all.js
#	template/src/main/java/com/asyncapi/model/$$message$$.java
#	template/src/main/java/com/asyncapi/model/$$schema$$.java
filters/all.js Show resolved Hide resolved
@derberg
Copy link
Member

derberg commented Dec 9, 2021

@Tenischev as @magicmatatjahu recently released new parser-js with function dedicated to stringify securely (handles circular deps like a pro) AsyncAPI document, and it is already in the generator:

  • you can remove decycle
  • in package.json update that template will work only with generator "generator": ">=1.8.27 <2.0.0",
  • this is how you access the stringify function now asyncapi.constructor.stringify(asyncapi)

@derberg
Copy link
Member

derberg commented Dec 9, 2021

@Tenischev btw mate, I see you are still using the template because you are still improving it, don't you want to join as a maintainer? so except for pushing your stuff, you would have to also support us on review or other PRs and triaging existing issues

@Tenischev
Copy link
Member Author

@derberg thanks for the advice, I redid it

And regarding your suggestions, yes it would be interesting for me.

btw, Merry Christmas! 🎅

@derberg
Copy link
Member

derberg commented Dec 27, 2021

@Tenischev

Merry Christmassssss!

Please also remove json-decycle from package.json, and also changes in package-lock.json should not be reflected in this PR.

And regarding your suggestions, yes it would be interesting for me.

Perfect, do you know anyone who would like to join you, or would it have to be me until we find someone?

@derberg
Copy link
Member

derberg commented Jan 12, 2022

@Tenischev yo man, back at work already? 😄

@Tenischev
Copy link
Member Author

Not yet :), tomorrow last day of vacation
I will fix it within next days

@derberg
Copy link
Member

derberg commented Jan 12, 2022

oh lucky you!!! enjoy!

@Tenischev
Copy link
Member Author

@derberg package and package-lock are fixed.

Perfect, do you know anyone who would like to join you, or would it have to be me until we find someone?

Unfortunately, no, so it's going to be you 😄

@derberg
Copy link
Member

derberg commented Jan 17, 2022

@Tenischev 😭 😆 please open up a PR to add codeowners and add us there. Also specify if you can help us on org level and want to join TSC 😉

@derberg
Copy link
Member

derberg commented Jan 17, 2022

/rtm

@asyncapi-bot asyncapi-bot merged commit 3723efd into asyncapi:master Jan 17, 2022
@asyncapi-bot
Copy link
Contributor

🎉 This PR is included in version 0.25.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.

Recursive model
3 participants