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

feat: add estimate time to review PR command and GH action #78

Closed
wants to merge 2 commits into from

Conversation

tnevrlka
Copy link
Contributor

Examples:

Add command for estimation of the time it takes to review a pull
request.
@tnevrlka tnevrlka requested review from psturc and Dannyb48 February 16, 2024 16:11
Add GitHub Action for running the time needed for PR review estimation
@tnevrlka tnevrlka force-pushed the estimate-review-time branch from c84519e to 6b16bbf Compare February 16, 2024 16:23
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@tnevrlka
Copy link
Contributor Author

@Dannyb48
Copy link
Collaborator

Can someone with permissions check please? https://github.com/redhat-appstudio/qe-tools/actions/runs/7933221065/job/21661485809?pr=78#step:6:27

So far all the permissions look correct so I can't tell why it is failing.

Workflow Permissions > Enabled Read and Write Permissions
Action Permissions > Enabled Allow all actions and reusable workflows

@psturc
Copy link
Member

psturc commented Feb 20, 2024

actions/first-interaction#10 (comment)

@tnevrlka I'd try to recreate this PR from an upstream branch instead of forked one

@tnevrlka
Copy link
Contributor Author

Created #86, closing this one.

@tnevrlka tnevrlka closed this Feb 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants