Skip to content

consolidate workflow files to display CI workflow graph under workflow summary page#409

Closed
ilaflott wants to merge 3 commits into
NOAA-GFDL:mainfrom
ilaflott:make-CI-clearer
Closed

consolidate workflow files to display CI workflow graph under workflow summary page#409
ilaflott wants to merge 3 commits into
NOAA-GFDL:mainfrom
ilaflott:make-CI-clearer

Conversation

@ilaflott

@ilaflott ilaflott commented Jun 11, 2026

Copy link
Copy Markdown
Member

Description
collapse main, main_expensive, distcheck workflow files --> one main workflow file with the same exact task dependencies

Closes #408

How Has This Been Tested?
see checks, specifically the summary

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • Any dependent changes have been merged and published in downstream modules
  • New check tests, if applicable, are included
  • make distcheck passes

@ilaflott

Copy link
Copy Markdown
Member Author

now there's one spot to click for the action this PR launched, and in the actions tab, it looks like the attached screenshot

Screenshot 2026-06-11 at 9 12 35 AM

🤌 🤌 🤌

@ilaflott ilaflott changed the title Update main.yml consolidate workflow files to display CI workflow graph under workflow summary page Jun 11, 2026
@ilaflott ilaflott requested review from rem1776 and underwoo and removed request for rem1776 June 11, 2026 13:22
Comment on lines +64 to +66
env:
MPI: ${{ matrix.with_mpi }}
QUAD_P: ${{ matrix.enable_quad_precision }}

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

this is less indented than before. according to github doc + gemini, it appears the environment variable should be read appropriately all the same. github shoves the environment variables into the container for you by assumption.

@ilaflott

Copy link
Copy Markdown
Member Author

closing as superseded by #410

@ilaflott ilaflott closed this Jun 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

make CI jobs / workflow clearer

1 participant