Skip to content

Update CI workflows, pyproject.toml, and Homebrew docs - #379

Open
mathomp4 wants to merge 7 commits into
mainfrom
mathomp4-patch-1
Open

Update CI workflows, pyproject.toml, and Homebrew docs#379
mathomp4 wants to merge 7 commits into
mainfrom
mathomp4-patch-1

Conversation

@mathomp4

@mathomp4 mathomp4 commented Jun 7, 2026

Copy link
Copy Markdown
Member

Summary

This PR brings several improvements to CI configuration, project metadata, and documentation.

CI workflow updates

  • All workflows now use actions/checkout@v7 (pinned to SHA)
  • All workflows now use astral-sh/setup-uv pinned to SHA (v8.2.0) with uv 0.11.23 and caching disabled (no uv.lock in this repo)
  • run-formatter.yaml and run-linter.yaml now use uv run instead of manually activating .venv
  • run-tests.yaml: Python version matrix updated — added 3.13, 3.14, pypy@3.11; removed pypy@3.9; added fail-fast: false

pyproject.toml

  • Migrated [tool.uv] dev-dependencies to [dependency-groups] dev (modern uv standard)
  • Fixed license field to use inline table format ({ file = "LICENSE" })
  • Moved repository URL to [project.urls]

Documentation

  • Revised Homebrew installation instructions in README.md: clarified recommended install method, added guidance on tap trust, and provided tap management commands

Code formatting

  • Reformatted multi-line help strings in src/mepo/cmdline/parser.py and print statements in src/mepo/command/config_get.py for consistency (black)
  • Removed an unnecessary blank line in tests/test_mepo_clone.py

Updated Homebrew installation instructions for 'mepo' and added tap trust information.
@mathomp4 mathomp4 self-assigned this Jun 7, 2026
@mathomp4
mathomp4 requested a review from a team as a code owner June 7, 2026 11:46
@mathomp4 mathomp4 added documentation Improvements or additions to documentation Skip Changelog Skips the Changelog Enforcer labels Jun 7, 2026
@mathomp4

mathomp4 commented Jun 7, 2026

Copy link
Copy Markdown
Member Author

@pchakraborty I used some ChatGPT suggestions to fix up the CI as it was failing in the black formatter.

Though, I wonder, should we switch to using ruff and ty etc?

@mathomp4 mathomp4 changed the title Revise Homebrew installation instructions for 'mepo' Revise Homebrew installation instructions, update black action Jun 7, 2026
@pchakraborty

Copy link
Copy Markdown
Collaborator

Why aren’t the tests running?

@mathomp4

mathomp4 commented Jun 8, 2026

Copy link
Copy Markdown
Member Author

Why aren’t the tests running?

Looks like one failed and all canceled. Let me fix that so we know which is actually having issues.

@mathomp4 mathomp4 changed the title Revise Homebrew installation instructions, update black action Update CI workflows, pyproject.toml, and Homebrew docs Jun 8, 2026
@mathomp4 mathomp4 mentioned this pull request Jun 8, 2026
@mathomp4

mathomp4 commented Jun 8, 2026

Copy link
Copy Markdown
Member Author

@pchakraborty Okay. I think things are "better" but only in that things are now failing at pytest time, see: https://github.com/GEOS-ESM/mepo/actions/runs/27146165707/job/80124406229?pr=379

I think there is maybe an update needed in what mepo sees in the tests?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation Skip Changelog Skips the Changelog Enforcer

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants