Skip to content

Trying to remove red logo - #1119

Merged
jgabry merged 3 commits into
masterfrom
fix-pkgdown-logo
Dec 4, 2025
Merged

Trying to remove red logo#1119
jgabry merged 3 commits into
masterfrom
fix-pkgdown-logo

Conversation

@VisruthSK

@VisruthSK VisruthSK commented Dec 3, 2025

Copy link
Copy Markdown
Member

Trying to move to the new, blue logo in the website.

@VisruthSK VisruthSK changed the title Trying to remove red logo [no ci] Trying to remove red logo Dec 3, 2025
@VisruthSK
VisruthSK marked this pull request as ready for review December 3, 2025 19:04
@VisruthSK
VisruthSK requested a review from jgabry December 3, 2025 19:05
@VisruthSK

Copy link
Copy Markdown
Member Author

Simple fix, looks like the correct, blue favicon is live on dev site.

Comment thread R/model.R
@jgabry

jgabry commented Dec 3, 2025

Copy link
Copy Markdown
Member

This looks good, I just made one comment about some extra linting in model.R that's leading to a big diff.

@codecov-commenter

codecov-commenter commented Dec 3, 2025

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 87.17%. Comparing base (1d58078) to head (71ffe4e).

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1119      +/-   ##
==========================================
+ Coverage   87.15%   87.17%   +0.01%     
==========================================
  Files          14       14              
  Lines        5973     5973              
==========================================
+ Hits         5206     5207       +1     
+ Misses        767      766       -1     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@jgabry
jgabry merged commit 9ecff3b into master Dec 4, 2025
11 checks passed
@jgabry
jgabry deleted the fix-pkgdown-logo branch December 4, 2025 02:16
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.

3 participants