Skip to content

Ignore dirty submodules - #140

Merged
adamdempsey90 merged 2 commits into
developfrom
dempsey/dirty
Jul 27, 2026
Merged

Ignore dirty submodules#140
adamdempsey90 merged 2 commits into
developfrom
dempsey/dirty

Conversation

@adamdempsey90

Copy link
Copy Markdown
Collaborator

Background

I don't know why we didn't do this a long time ago. This will get rid of the submodules showing up in git status or git diff if there was no change to them other than any things that changed from building the code.

We could go one step further and do ignore = untracked to ignore untracked files that may have been added, but that sounds more dangerous.

Description of Changes

Checklist

  • New features are documented
  • Tests added for bug fixes and new features
  • (@lanl.gov employees) Update copyright on changed files
  • Any contribution that was created or modified with the assistance of generative AI must have a comment disclosing this such as // This file was created in part or in whole by generative AI

@RyanWollaeger RyanWollaeger left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM - it looks like the indentation is a bit behind url - is this intentional?

@adamdempsey90

Copy link
Copy Markdown
Collaborator Author

LGTM - it looks like the indentation is a bit behind url - is this intentional?

The other lines were using \t and my new lines where using spaces. I've updated all lines to use spaces.

@adamdempsey90
adamdempsey90 enabled auto-merge July 27, 2026 16:26
@adamdempsey90
adamdempsey90 merged commit 656bb18 into develop Jul 27, 2026
6 checks passed
@adamdempsey90
adamdempsey90 deleted the dempsey/dirty branch July 27, 2026 18:48
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