-
-
Notifications
You must be signed in to change notification settings - Fork 53
feat: add pre-commit hook and action for conventional commit and semantic release #32
feat: add pre-commit hook and action for conventional commit and semantic release #32
Conversation
This comment was marked as off-topic.
This comment was marked as off-topic.
8cd7b33 to
2f6bcf2
Compare
J0
left a comment
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.
Thanks for the PR! It looks good to me. While I have access to the PyPI Repository, it seems like we need to get approval to add secrets to the Github repository. Requesting for that now.
|
tests are failing and I don't really know why, could you please take a look @leynier? Thanks! |
Yes, I will. |
…for-conventional-commit
|
@dreinon already tests running fine |
|
@leynier codecov/patch is failing, is this meaningful? |
|
@leynier tests failing due to connection errors |
|
Tests are failing by this supabase/auth-js#188 (comment) |
Sourcery Code Quality Report✅ Merging this PR will increase code quality in the affected files by 0.55%.
Here are some functions in these files that still need a tune-up:
Legend and ExplanationThe emojis denote the absolute quality of the code:
The 👍 and 👎 indicate whether the quality has improved or gotten worse with this pull request. Please see our documentation here for details on how these metrics are calculated. We are actively working on this report - lots more documentation and extra metrics to come! Help us improve this quality report! |
Resolve #30
Resolve #31