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

refactor: add isEmpty() method to components #605

Closed
wants to merge 1 commit into from

Conversation

smoya
Copy link
Member

@smoya smoya commented Sep 9, 2022

Description

For some reason, I forgot to add the isEmpty() method to components. See https://github.com/asyncapi/parser-api/pull/71/files#diff-ac35ac6e0b0ad3629802a0e04076aa18e94da614f5392e1164dcaacdfe3ee198R84.

Related issue(s)
#401

@sonarqubecloud
Copy link

sonarqubecloud bot commented Sep 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

@magicmatatjahu
Copy link
Member

You add that method here https://github.com/asyncapi/parser-js/pull/597/files#diff-c8a73b0d91063fa284e344bc0ed3a31db68590f0832dc6b4a2e5e4007028abfbR120 Please close that PR.

@smoya
Copy link
Member Author

smoya commented Sep 12, 2022

You add that method here https://github.com/asyncapi/parser-js/pull/597/files#diff-c8a73b0d91063fa284e344bc0ed3a31db68590f0832dc6b4a2e5e4007028abfbR120 Please close that PR.

LOL i was pretty sure i did. OMG my brain...

@smoya smoya closed this Sep 12, 2022
@smoya smoya deleted the feat/isEmpty branch September 12, 2022 08:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants