Skip to content

Conversation

georgemccabe
Copy link
Collaborator

@georgemccabe georgemccabe commented Jan 24, 2024

See #2473

Also added a fix to docs/requirements.txt that was applied to develop but not main that prevents failure in ReadTheDocs builds.

@georgemccabe georgemccabe added this to the METplus-6.0.0 milestone Jan 24, 2024
Copy link
Collaborator

@JohnHalleyGotway JohnHalleyGotway left a comment

Choose a reason for hiding this comment

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

I approve of these changes.

These are the same set of changes made to the develop branch by PR #2473. I do note the doc-only update to the contents of requirements.txt. While the testing workflow has not yet finished for this PR, the documentation workflow has and it completed without error.

Copy link
Collaborator

@JohnHalleyGotway JohnHalleyGotway left a comment

Choose a reason for hiding this comment

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

I approve of these changes.

These are changes to the main_v5.1 that were made to develop via PR's #2473 and #2475. This updates to action version numbers that use Node.js 20 instead of 16, and updates the handling of error log artifacts.

@georgemccabe georgemccabe merged commit cb46573 into main_v5.1 Jan 26, 2024
@georgemccabe georgemccabe deleted the feature_met2976_main_v5.1_gha_node branch March 20, 2024 22:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Status: 🏁 Done
Development

Successfully merging this pull request may close these issues.

2 participants