Skip to content

fix(doxygen): adds the correct header to doxygen website#1062

Merged
kellijohnson-NOAA merged 1 commit into
mainfrom
dev
Dec 10, 2025
Merged

fix(doxygen): adds the correct header to doxygen website#1062
kellijohnson-NOAA merged 1 commit into
mainfrom
dev

Conversation

@kellijohnson-NOAA

@kellijohnson-NOAA kellijohnson-NOAA commented Dec 9, 2025

Copy link
Copy Markdown
Contributor

What is the feature?

  • Fix the header in the doxygen to match the website.

How have you implemented the solution?

  • The doxygen header was not working so this changes the reference in the GitHub workflow

Does the PR impact any other area of the project, maybe another repo?

  • No

e-perl-NOAA
e-perl-NOAA approved these changes Dec 9, 2025
View reviewed changes

@e-perl-NOAA e-perl-NOAA left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

praying that this finally works!

@codecov

codecov Bot commented Dec 9, 2025 •
edited
Loading

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.97%. Comparing base (93f48b4) to head (06bd03c).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files 
@@           Coverage Diff           @@
##             main    #1062   +/-   ##
=======================================
  Coverage   85.97%   85.97%           
=======================================
  Files          81       81           
  Lines        7884     7884           
  Branches      517      517           
=======================================
  Hits         6778     6778           
  Misses       1098     1098           
  Partials        8        8

☔ 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.

@kellijohnson-NOAA

kellijohnson-NOAA commented Dec 10, 2025

Copy link
Copy Markdown
Contributor Author

@e-perl-NOAA this PR is hung up because there have been a few packages updated on CRAN, e.g., {yaml}, that appear to not have new versions for Mac yet and are causing problems with the installation of R packages. There are rules that require passing tests for a few of these failed actions that I do not have the ability to overwrite. I am hoping that this will work itself out in the next 24 hours and then we will be able to merge this in.

@kellijohnson-NOAA kellijohnson-NOAA merged commit 5cbe0c3 into main Dec 10, 2025
39 of 45 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@e-perl-NOAA e-perl-NOAA e-perl-NOAA approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@kellijohnson-NOAA @e-perl-NOAA