Skip to content

oalders/precious #3048

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 10 commits into from
May 15, 2024
Merged

oalders/precious #3048

merged 10 commits into from
May 15, 2024

Conversation

oalders
Copy link
Member

@oalders oalders commented May 3, 2024

  • Add precious config
  • Add script to install precious in Docker

@oalders oalders force-pushed the oalders/precious branch from 558b622 to 8cf3238 Compare May 4, 2024 14:32
@oalders oalders marked this pull request as draft May 4, 2024 14:32
@oalders oalders force-pushed the oalders/precious branch 2 times, most recently from 8c408b0 to 4b2e560 Compare May 5, 2024 14:39
Copy link

codecov bot commented May 5, 2024

Codecov Report

Attention: Patch coverage is 94.11765% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 71.75%. Comparing base (4b4afe7) to head (f81d1f4).

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #3048      +/-   ##
==========================================
- Coverage   71.87%   71.75%   -0.12%     
==========================================
  Files          70       70              
  Lines        2414     2411       -3     
  Branches      338      338              
==========================================
- Hits         1735     1730       -5     
- Misses        554      555       +1     
- Partials      125      126       +1     
Files Coverage Δ
app.psgi 81.81% <100.00%> (-0.80%) ⬇️
lib/MetaCPAN/Web/Controller/About.pm 59.45% <100.00%> (ø)
lib/MetaCPAN/Web/Model/API/Pod.pm 44.11% <100.00%> (ø)
lib/MetaCPAN/Web/Model/GitHub.pm 13.23% <100.00%> (-2.48%) ⬇️
lib/MetaCPAN/Web/RenderUtil.pm 88.23% <100.00%> (+0.17%) ⬆️
lib/MetaCPAN/Web/Controller/Feed.pm 89.11% <83.33%> (-0.75%) ⬇️

... and 1 file with indirect coverage changes

@oalders oalders force-pushed the oalders/precious branch 10 times, most recently from 80c78f8 to 34b8d9e Compare May 7, 2024 21:06
@oalders oalders marked this pull request as ready for review May 7, 2024 21:06
@@ -83,3 +88,17 @@ jobs:
run: cover -report codecov
env:
CODECOV_TOKEN: ${{ secrets.CODECOV_TOKEN }}
- name: Install precious
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Don't we want this to be in a separate job, so it's shown as a separate status check?

haarg
haarg previously approved these changes May 15, 2024
@oalders oalders force-pushed the oalders/precious branch from 34b8d9e to f81d1f4 Compare May 15, 2024 15:19
@oalders oalders merged commit bdd23aa into master May 15, 2024
8 checks passed
@oalders oalders deleted the oalders/precious branch May 15, 2024 15:30
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.

2 participants