-
Notifications
You must be signed in to change notification settings - Fork 5
Conversation
Co-authored-by: Shrey Anand <shanand@redhat.com> Co-authored-by: Hema Veeradhi <hveeradh@redhat.com>
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
/lgtm 👍 |
/test pre-commit |
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.
/lgtm
@Shreyanand any reason for pre-commit failing? |
Seems like this module is not available to the pre-commit environment (source). |
/retest |
@Shreyanand seems like the module not available issue is already fixed: |
@harshad16 thanks, it seems it's fixed but I tried to |
@Shreyanand please check all ssh url to htps URL metrics/.pre-commit-config.yaml Line 3 in 848facd
to fix that authentication issue. |
Co-authored-by: Hema Veeradhi <hveeradh@redhat.com> Co-authored-by: Karanraj Chauhan <kachau@redhat.com>
New changes are detected. LGTM label has been removed. |
Thanks, that fixed it 👍 |
@Shreyanand |
Thanks this is helpful, added an issue in the project template for future projects using it. |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: MichaelClifford, oindrillac The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
For issue #22, this PR fetches events from repositories, extracts contributor information from them and adds a table to Trino for visualization.
Also, it updates the pipfile with required dependencies.
Co-authored-by: Hema Veeradhi hveeradh@redhat.com
Co-authored-by: Karanraj Chauhan kachau@redhat.com