Ignore .claude/ and pubspec.lock - #118
Conversation
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request updates the .gitignore file to exclude pubspec.lock and the .claude/ directory. Feedback suggests that pubspec.lock should be tracked rather than ignored to ensure reproducible builds for the application, and recommends adding pubspec_overrides.yaml to the ignore list instead.
| # https://dart.dev/guides/libraries/private-files | ||
| # Created by `dart pub` | ||
| .dart_tool/ | ||
| pubspec.lock |
There was a problem hiding this comment.
The Dart guide linked at the top of this file recommends committing pubspec.lock for applications and workspaces to ensure reproducible builds across different environments. Since this repository contains a command-line application (tool/generator), it is better to commit the lock file. However, you should ignore pubspec_overrides.yaml, as it contains local environment overrides that can break the build for other contributors if committed.
pubspec_overrides.yaml
1 participant
.claude/andpubspec.lockto.gitignore