Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

feat: add support for status events to baseRef validator #602

Merged

Conversation

abid-mujtaba
Copy link
Contributor

  • For status events the commit sha in the payload is used to make an
    octokit request to fetch the pull requests to which the commit
    belongs.
    The pull requests are filtered to just the open ones.
    The validator than check the base refs of these open pull requests.
  • Add unit tests to cover new functionality
  • Addresses feature request: add validator for other Github checks in a PR #395

@abid-mujtaba abid-mujtaba changed the title Add support for status events to baseRef validator feat: add support for status events to baseRef validator Dec 12, 2021
@abid-mujtaba abid-mujtaba force-pushed the feat/baseref-status-events branch from f2903b4 to 36b9106 Compare December 12, 2021 21:05
@codecov
Copy link

codecov bot commented Dec 12, 2021

Codecov Report

Merging #602 (45a15e6) into master (a430400) will increase coverage by 0.03%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #602      +/-   ##
==========================================
+ Coverage   92.83%   92.87%   +0.03%     
==========================================
  Files         106      106              
  Lines        2416     2427      +11     
  Branches      433      435       +2     
==========================================
+ Hits         2243     2254      +11     
  Misses        154      154              
  Partials       19       19              
Impacted Files Coverage Δ
lib/validators/baseRef.js 100.00% <100.00%> (ø)

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 a430400...45a15e6. Read the comment docs.

- For status events the commit sha in the payload is used to make an
  octokit request to fetch the pull requests to which the commit
  belongs.
  The pull requests are filtered to just the open ones.
  The validator than check the base refs of these open pull requests.
- Add unit tests to cover new functionality
@abid-mujtaba abid-mujtaba force-pushed the feat/baseref-status-events branch from 36b9106 to 45a15e6 Compare December 12, 2021 21:27
@shine2lay shine2lay merged commit 9677d94 into mergeability:master Dec 22, 2021
@abid-mujtaba abid-mujtaba deleted the feat/baseref-status-events branch December 23, 2021 14:09
@github-actions
Copy link

github-actions bot commented Feb 5, 2022

🎉 This PR is included in version 2.9.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants