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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

馃敟 Remove pr-approvals GitHub Action as it's not compatible with forks. Use the new one #2162

Merged
merged 1 commit into from Oct 11, 2020

Conversation

tiangolo
Copy link
Owner

馃敟 Remove pr-approvals action as it is not compatible with forks

as it is not compatible with forks
@tiangolo tiangolo changed the title 馃敟 Remove pr-approvals action 馃敟 Remove pr-approvals GitHub Action as it's not compatible with forks. Use the new one Oct 11, 2020
@codecov
Copy link

codecov bot commented Oct 11, 2020

Codecov Report

Merging #2162 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##            master     #2162   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          239       239           
  Lines         7079      7079           
=========================================
  Hits          7079      7079           

Continue to review full report at Codecov.

Legend - Click here to learn more
螖 = absolute <relative> (impact), 酶 = not affected, ? = missing data
Powered by Codecov. Last update 952cd73...40c61f1. Read the comment docs.

@github-actions github-actions bot temporarily deployed to commit October 11, 2020 10:02 Inactive
@github-actions
Copy link
Contributor

@tiangolo tiangolo merged commit ad5aa33 into master Oct 11, 2020
@tiangolo tiangolo deleted the remove-pr-approvals branch October 11, 2020 10:04
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.

None yet

1 participant