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

Fix RustDocs & AlwaysPass codecov patch check #1870

Merged
merged 4 commits into from
Jul 2, 2024
Merged

Conversation

gpmayorga
Copy link
Contributor

@gpmayorga gpmayorga commented Jun 12, 2024

Description

https://kflabs.slack.com/archives/C04GJQAM9P0/p1718188938855999
Fix docs deployments, broken for about a month due to a GHA module upgrade

Copy link

codecov bot commented Jun 12, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 47.34%. Comparing base (753a8b5) to head (ac997eb).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1870   +/-   ##
=======================================
  Coverage   47.34%   47.34%           
=======================================
  Files         176      176           
  Lines       13305    13305           
=======================================
  Hits         6299     6299           
  Misses       7006     7006           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@lemunozm
Copy link
Contributor

How can we have the fmt job failing but passing in main? 🤔

@gpmayorga
Copy link
Contributor Author

I'm absolutely puzzled. It's complaining about code lines from 2 years ago.

@gpmayorga
Copy link
Contributor Author

#1871 should fix it...

@gpmayorga
Copy link
Contributor Author

We'll have to wait for William to be back but it's not urgent, every time this PR gets updated with the latest from main the docs are automatically deployed by the workflow as this is treated as a special branch.

@gpmayorga gpmayorga enabled auto-merge (squash) June 14, 2024 07:41
@gpmayorga gpmayorga changed the title Try to fix deploy docs Fix RustDocs & AlwaysPass codecov patch check Jun 14, 2024
Copy link
Contributor

@lemunozm lemunozm left a comment

Choose a reason for hiding this comment

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

Let's merge it!

Copy link
Contributor

@wischli wischli left a comment

Choose a reason for hiding this comment

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

Thanks for the fix!

@gpmayorga gpmayorga merged commit 2f08ae7 into main Jul 2, 2024
13 checks passed
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

3 participants