Skip to content

Update .gitignore with entries from open PRs and AI tooling - #3495

Merged
andrew-platt merged 4 commits into
OpenFAST:rc-5.0.1from
andrew-platt:f/gitignore_update_ai
Oct 20, 2026
Merged

andrew-platt merged 4 commits into
OpenFAST:rc-5.0.1from
andrew-platt:f/gitignore_update_ai

Conversation

@andrew-platt

@andrew-platt andrew-platt commented Oct 20, 2026 •

Copy link
Copy Markdown
Collaborator

Ready to merge

Feature or improvement description
Updates .gitignore with entries proposed in open pull requests, plus common compiler, build-tool, editor, OpenFAST run, AI-tool, environment, and credential files.

  • From AeroAcoustics improvements #3449 (b/AeroAcoustics):
    • gitVersionInfo.h (generated at build time)
    • Visual Studio files: .vs, *.user, *.u2d
    • .kiro/* (per-user AI steering files)
  • From OpenFAST ZMQ connection (1 WT) #2281 (zmq-merge-dev):
    • Simulation output files: *.outb, *.pkl.gz
  • OpenFAST run byproducts: *.ech, *.chkp, *.dbg
  • Intel Fortran byproducts: *__genmod.f90, *__genmod.mod, *.i90, *.optrpt
  • CMake/clangd/coverage: compile_commands.json, CMakeUserPresets.json, .cache/, *.gcda, *.gcno
  • Visual Studio debug/link files: *.pdb, *.ilk
  • Python environments: .venv, venv/, .ipynb_checkpoints/; .env broadened to .env*
  • Editor backups: *.swo, *~
  • AI assistant files: /CLAUDE.md, .claude/, .continue/, .cursor/, .aider*, .windsurf/, AGENTS.md, GEMINI.md
  • Credential files: *credentials*.json, *serviceaccount*.json
  • Fix: __pycache__/* only matched at the repo root. Changed to __pycache__/ so it matches in all subdirectories.

Related issue, if one exists
None. Some entries come from #3449 and #2281.

Impacted areas of the software
.gitignore only. No source, build, or test changes.

Additional supporting information

  • No currently tracked file matches any of the new patterns.
  • AGENTS.md and GEMINI.md are ignored in every directory. If the project later wants shared agent instructions committed, those entries should be removed or anchored.
  • I left out /RealTimeFAST and /cJSON from OpenFAST ZMQ connection (1 WT) #2281. They only apply to that unmerged ZMQ work and can go in with it.
  • *.out from OpenFAST ZMQ connection (1 WT) #2281 is already ignored.
  • I left out *.sum, *.vtp, and *.lin because they can be legitimate example or reference files (one *.sum is already tracked).
  • I also checked every branch on andrew-platt/openfast. None had further .gitignore additions.

Generative AI usage
Claude Code (Claude Opus 5.5) found the .gitignore changes across the open PRs and branches, proposed additional patterns, checked every new pattern against the tracked files, and prepared the edits and commit messages. Some candidate entries were suggested by Google Gemini. The author reviewed and selected all entries.

Assisted-by: Claude Code:claude-opus-5-5 [Bash] [Read] [Edit]
Co-authored-by: Anthropic Claude claude@anthropic.com

Test results, if applicable
Not applicable (.gitignore only).

Collect .gitignore additions from open OpenFAST pull requests:
- PR OpenFAST#3449 (b/AeroAcoustics): gitVersionInfo.h, Visual Studio files
  (.vs, *.user, *.u2d), and .kiro/* AI steering files
- PR OpenFAST#2281 (zmq-merge-dev): simulation output files (*.outb, *.pkl.gz);
  ZMQ-specific entries (/RealTimeFAST, /cJSON) intentionally omitted

Also ignore the root CLAUDE.md used for local AI agent instructions.

Assisted-by: Claude Code:claude-opus-5-5 [Bash] [Read] [Edit]
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@andrew-platt
andrew-platt requested a review from luwang00 October 20, 2026 20:23
@andrew-platt andrew-platt self-assigned this Oct 20, 2026
@andrew-platt andrew-platt added the Code-dev: cross-merges and maintenance Cross-merges between development branches. This is mostly code development maintenance. label Oct 20, 2026

@luwang00 luwang00 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Please consider also adding .venv for virtual environments.

andrew-platt and others added 3 commits October 20, 2026 14:29
- Broaden .env to .env* to cover variants such as .env.local
- Ignore .continue/ and .cursor/ AI assistant directories
- Ignore *credentials*.json and *serviceaccount*.json to guard against
  committing secrets

Assisted-by: Claude Code:claude-opus-5-5 [Bash] [Read]
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
- Fix __pycache__/* (anchored to repo root) to __pycache__/ so it
  matches in all subdirectories
- Intel Fortran byproducts: *__genmod.f90, *__genmod.mod, *.i90, *.optrpt
- CMake/clangd/coverage: compile_commands.json, CMakeUserPresets.json,
  .cache/, *.gcda, *.gcno
- Visual Studio debug/link files: *.pdb, *.ilk
- Python: venv/, .ipynb_checkpoints/
- Editor backups: *.swo, *~
- OpenFAST run byproducts: *.ech, *.chkp, *.dbg
- AI tool files: .claude/, .aider*, .windsurf/, AGENTS.md, GEMINI.md

Assisted-by: Claude Code:claude-opus-5-5 [Bash] [Read]
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@andrew-platt

Copy link
Copy Markdown
Collaborator Author

Added a few additional suggestions from Claude and Gemini for other models and build flows.

@andrew-platt
andrew-platt merged commit bd79998 into OpenFAST:rc-5.0.1 Oct 20, 2026
28 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Code-dev: cross-merges and maintenance Cross-merges between development branches. This is mostly code development maintenance.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants