Skip to content

Moved from gh action API to gh webhook API #49

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

Merged
merged 2 commits into from
Sep 21, 2021
Merged

Conversation

filipecosta90
Copy link
Contributor

@filipecosta90 filipecosta90 commented Sep 21, 2021

Currently, using the following two events:

  • Pull request labeled ( by default using label trigger-benchmark - can be altered via ENV variable PULL_REQUEST_TRIGGER_LABEL ).
  • Git pushes to origin repo

Reference on secured webhooks: https://docs.github.com/en/developers/webhooks-and-events/webhooks/securing-your-webhooks

@filipecosta90 filipecosta90 added the tool-api Tooling changes: API label Sep 21, 2021
@codecov-commenter
Copy link

codecov-commenter commented Sep 21, 2021

Codecov Report

Merging #49 (9bb5217) into main (28b86a5) will increase coverage by 5.98%.
The diff coverage is 93.75%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main      #49      +/-   ##
==========================================
+ Coverage   69.57%   75.55%   +5.98%     
==========================================
  Files          12       11       -1     
  Lines         631      671      +40     
==========================================
+ Hits          439      507      +68     
+ Misses        192      164      -28     
Impacted Files Coverage Δ
...chmarks_specification/__common__/builder_schema.py 81.66% <80.00%> (-1.10%) ⬇️
redis_benchmarks_specification/__api__/app.py 93.02% <94.59%> (+72.56%) ⬆️
redis_benchmarks_specification/__common__/env.py 97.14% <100.00%> (+0.08%) ⬆️

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 28b86a5...9bb5217. Read the comment docs.

…r to decide uppon which PR label to use for trigger automation
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tool-api Tooling changes: API
Projects
None yet
Development

Successfully merging this pull request may close these issues.

api/gh/redis/redis should return 401 unauthorized and not 500 internal server error on bad auth requests
2 participants