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

chore: add @smoya as code owner #174

Merged
merged 3 commits into from
Apr 26, 2023
Merged

Conversation

smoya
Copy link
Member

@smoya smoya commented Apr 24, 2023

Description

Adding me @smoya as code owner. Worked along with @magicmatatjahu adapting this to TS and the new Parser-JS v2 interface.

@magicmatatjahu feel free to suggest your addition as well.

Related issue(s)

@smoya
Copy link
Member Author

smoya commented Apr 25, 2023

I don't really get why the testing action fails. Mocha is installed as dev dependency. Running this locally works 🤷 cc @fmvilas

fmvilas
fmvilas previously approved these changes Apr 25, 2023
@sonarcloud
Copy link

sonarcloud bot commented Apr 26, 2023

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
0.0% 0.0% Duplication

@smoya
Copy link
Member Author

smoya commented Apr 26, 2023

I don't really get why the testing action fails. Mocha is installed as dev dependency. Running this locally works 🤷 cc @fmvilas

Fails the npm i command https:/asyncapi/raml-dt-schema-parser/actions/runs/4796162112/jobs/8557692937#step:8:128

fixed updating some deps. Can you @fmvilas approve again?

@fmvilas
Copy link
Member

fmvilas commented Apr 26, 2023

/rtm

@asyncapi-bot asyncapi-bot merged commit 0102ca5 into asyncapi:master Apr 26, 2023
@smoya smoya deleted the patch-1 branch April 27, 2023 03:59
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.

3 participants