Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Tooling updates #689

Merged
merged 4 commits into from
Jan 28, 2025
Merged

Conversation

maddenp-noaa
Copy link
Collaborator

@maddenp-noaa maddenp-noaa commented Jan 28, 2025

Synopsis

Two small tooling updates:

  1. Fix the format script so that, when a .json or .jsonschema file has not changed, its timestamp will not be updated by format. This has been the cause, recently, of make devshell often recreating the development environment apparently unnecessarily: The Makefile rules depend on the timestamps of meta.yaml and info.json, and the latter was being updated by format (aka make format) even when its content wasn't changing.
  2. Trim the CODEOWNERS file to those currently on the project.
  3. Remove troublesome gnu.org links from docs.
  4. Update recipe/meta.json with == in package pins.

Type

  • Tooling (CI, code-quality, packaging, revision-control, etc.)

Impact

  • This is a non-breaking change (existing functionality continues to work as expected)

Checklist

  • I have added myself and any co-authors to the PR's Assignees list.
  • I have reviewed the documentation and have made any updates necessitated by this change.

Copy link
Collaborator

@christinaholtNOAA christinaholtNOAA left a comment

Choose a reason for hiding this comment

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

LGTM.

@maddenp-noaa maddenp-noaa merged commit 8a89b21 into ufs-community:main Jan 28, 2025
2 checks passed
@maddenp-noaa maddenp-noaa deleted the tooling-updates branch January 28, 2025 20:43
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