-
Notifications
You must be signed in to change notification settings - Fork 19
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
feat: github comment generation and posting #12
feat: github comment generation and posting #12
Conversation
Example of successful comment posting with results: |
# Conflicts: # src/parser/permit-generation-module.ts
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice code quality per usual
@0x4007 I believe that if I had all the tests in this PR it will become enormous, maybe I ought to do it in a separate PR. |
I think it's fine. I generally scrutinize test code less than normal program code anyways. |
a466667
into
ubiquity-os-marketplace:development
Resolves #5
Depends on #8
Reviews looks gigantic but it's only due to the API snapshots I promise.