Skip to content

Specify # of Increased Warnings/Remarks in GHA Check#3132

Closed
gspetro-NOAA wants to merge 5 commits intoufs-community:developfrom
gspetro-NOAA:gha-log-check
Closed

Specify # of Increased Warnings/Remarks in GHA Check#3132
gspetro-NOAA wants to merge 5 commits intoufs-community:developfrom
gspetro-NOAA:gha-log-check

Conversation

@gspetro-NOAA
Copy link
Copy Markdown
Collaborator

@gspetro-NOAA gspetro-NOAA commented Mar 8, 2026

Commit Queue Requirements:

  • This PR addresses a relevant WM issue (if not, create an issue).
  • All subcomponent pull requests (if any) have been reviewed by their code managers. N/A
  • Run the full Intel+GNU RT suite (compared to current baselines), preferably on Ursa (Derecho or Hercules are acceptable alternatives). Exceptions: documentation-only PRs, CI-only PRs, etc. N/A
    • Commit log file w/full results from RT suite run (if applicable).
    • Verify that test_changes.list indicates which tests, if any, are changed by this PR. Commit test_changes.list, even if it is empty.
  • Fill out all sections of this template.

Description:

The GitHub Action (GHA) check that reports increases in warnings or remarks will now specify how many more warnings/remarks there are compared to the previous log.

Commit Message:

* UFSWM - Enhancement to GHA warnings/remarks check

Priority:

  • Critical Bugfix: Reason
  • High: Reason
  • Normal

Git Tracking

UFSWM:

Sub component Pull Requests:

  • None

UFSWM Blocking Dependencies:

  • Blocked by #
  • None

Documentation:

  • Documentation update required.
    • Relevant updates are included with this PR.
    • A WM issue has been opened to track the need for a documentation update; a person responsible for submitting the update has been assigned to the issue (link issue).
  • Documentation update NOT required.
    • Explanation: CI improvement

Changes

Regression Test Changes (Please commit test_changes.list):

  • PR Adds New Tests/Baselines.
  • PR Updates/Changes Baselines.
  • No Baseline Changes.

Input data Changes:

  • None.
  • PR adds input data.
  • PR changes existing input data.

Library Changes/Upgrades:

  • Required
    • Library names w/versions:
    • Git Stack Issue (JCSDA/spack-stack#)
  • No Updates

Testing Log:

  • RDHPCS
    • Orion
    • Hercules
    • GaeaC6
    • Derecho
    • Ursa
  • WCOSS2
    • Dogwood/Cactus
    • Acorn
  • CI
  • opnReqTest (complete task if unnecessary)

@github-project-automation github-project-automation bot moved this to Evaluating in PRs to Process Mar 8, 2026
@gspetro-NOAA gspetro-NOAA marked this pull request as ready for review March 8, 2026 03:13
@gspetro-NOAA gspetro-NOAA moved this from Evaluating to Review in PRs to Process Mar 8, 2026
@gspetro-NOAA gspetro-NOAA added the No Baseline Change No Baseline Change label Mar 8, 2026
FernandoAndrade-NOAA pushed a commit that referenced this pull request Mar 10, 2026
…mmit (#3116) // Specify # of Increased Warnings/Remarks in GHA Check (#3132) (#3123)

* UFSWM - Bring in UFSATM documentation
* UFSWM - Enhancement to GHA warnings/remarks check
  * UFSATM - Documentation was added in doxygen format
  * MOM6 - update MOM6 to its main repository 20260302 updating
@gspetro-NOAA
Copy link
Copy Markdown
Collaborator Author

Merged via #3123

@gspetro-NOAA gspetro-NOAA deleted the gha-log-check branch March 17, 2026 01:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

No Baseline Change No Baseline Change

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

1 participant