-
-
Notifications
You must be signed in to change notification settings - Fork 233
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
Hacktoberfest easy issues #1272
Comments
Hey I wanted to work on this will you please assigned to me |
@thebedigupta we don't assign issues, please go ahead and work on it :) |
Should I have to solve all issues in One PR? |
I also want to work on these issues |
No need to solve all issues in one PR instead open separate PRs resolving issues which will help maintainers to efficient review PR |
Hey I wanted to work on this issue |
Raise a PR if you think so |
Raised a PR , please let me know if any changes are required |
raised a pr for this issue |
Hey, I want to work on this issue : - https://sonarcloud.io/project/issues?open=AZD-TPs2DKL_zKkK1XuA&id=asyncapi_generator . Can I work? |
Hey folks, we do not assign issues, if you see any issue/s you're interested in and there's currently no PR for it, kindly go ahead and work on it 🚀 |
hey folks, be patient here 🙏🏼 there are many PRs that we need to review, also test runs are not fast. If you wanna speed up, make sure you run tests locally and check if the pass. I already noticed in one PR that contributor did not do it, and then merging have to take more time as they need to followup, etc |
also, folks please do not address any fixes or anything in |
Hey I wanted to work on this issue |
No need to ask for assign this issue if you think you can then go for it |
I'm not going to create separate issues as this would be too much work that is not really needed.
So let us experiment and see if one issue is enough.
SonarCloud suggestions
50 small fixes can be done to improve the quality of this project. All are listed here: https://sonarcloud.io/project/issues?issueStatuses=OPEN%2CCONFIRMED&id=asyncapi_generator
Process of work:
Tests
Use our new dev guide https://github.com/asyncapi/generator/blob/master/Development.md and install the project and run tests. At the run summary you will see the test coverage overview. You can just open a PR that adds one of the missing tests. In description just make sure you show test coverage report before and then after adding the test, so it makes review easier, and shows improvements are really there.
Should be good 😄
Happy Hacktoberfesting!
The text was updated successfully, but these errors were encountered: