Skip to content

v2: Add .rgignore file#4372

Merged
mathomp4 merged 4 commits into
developfrom
feature/v2-add-rgignore
Feb 10, 2026
Merged

v2: Add .rgignore file#4372
mathomp4 merged 4 commits into
developfrom
feature/v2-add-rgignore

Conversation

@mathomp4
Copy link
Copy Markdown
Member

Types of change(s)

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Trivial change (affects only documentation or cleanup)
  • Refactor (no functional changes, no api changes)

Checklist

  • Tested this change with a run of GEOSgcm
  • Ran the Unit Tests (make tests)

Description

This is a pretty minor PR. It adds a .rgignore file in MAPL so that if you have a build or install directory in your "main" MAPL directory, ripgrep will not search inside.

Normally this wouldn't be needed since ripgrep obeys .gitignore and we add one of those to our build and install dirs at CMake time. But because of mepo, we usually run ripgrep with --no-ignore-vcs so it'll find the mepo subrepos.

I've tried to capture the main directory names I've seen people use. I'll mention @GEOS-ESM/gmao-si-team here for more thoughts and possible names.

Related Issue

@mathomp4 mathomp4 self-assigned this Feb 10, 2026
@mathomp4 mathomp4 requested a review from a team as a code owner February 10, 2026 13:15
@mathomp4 mathomp4 added the 0 Diff Trivial The changes in this pull request are trivially zero-diff (documentation, build failure, &c.) label Feb 10, 2026
@mathomp4 mathomp4 merged commit d724c43 into develop Feb 10, 2026
45 checks passed
@mathomp4 mathomp4 deleted the feature/v2-add-rgignore branch February 10, 2026 18:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

0 Diff Trivial The changes in this pull request are trivially zero-diff (documentation, build failure, &c.)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants