Skip to content

Add "/coverage/" directory to the default .gitignore template #166909

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
doctor-g opened this issue Apr 10, 2025 · 0 comments · Fixed by #169387
Closed

Add "/coverage/" directory to the default .gitignore template #166909

doctor-g opened this issue Apr 10, 2025 · 0 comments · Fixed by #169387
Assignees
@matanlurey
Labels
c: proposal A detailed proposal for a change to Flutter P2 Important issues not at the top of the work list team-tool Owned by Flutter Tool team tool Affects the "flutter" command-line tool. See also t: labels. triaged-tool Triaged by Flutter Tool team

Comments

@doctor-g
Copy link

Use case

Running flutter test --coverage generates the file coverage/lcov.info. As a generated file, it should not be added to version control, but the default .gitignore template does not prevent this.

Proposal

I propose that /coverage/ be added to the default .gitignore template that is generated by flutter create in order to circumvent this problem.

If a team wanted to record these files in version control, they would have to remove that entry. That strikes me as a reasonable cost given that the alternative is to have a high-churn generated file not being ignored by default.
@maheshj01 maheshj01 added in triage Presently being triaged by the triage team c: proposal A detailed proposal for a change to Flutter team-tool Owned by Flutter Tool team tool Affects the "flutter" command-line tool. See also t: labels. and removed in triage Presently being triaged by the triage team labels Apr 10, 2025
@bkonyi bkonyi added P2 Important issues not at the top of the work list triaged-tool Triaged by Flutter Tool team labels Apr 22, 2025
@bkonyi bkonyi self-assigned this Apr 22, 2025
@matanlurey matanlurey assigned matanlurey and unassigned bkonyi May 23, 2025
@matanlurey matanlurey mentioned this issue May 23, 2025
Merged
github-merge-queue bot pushed a commit that referenced this issue May 26, 2025
@matanlurey
Add /coverage/ to .gitignore.tmp (#169387)
36275e1 
Closes #166909.

Made `/build/` consistent while I was here.
stan-at-work pushed a commit to stan-at-work/flutter that referenced this issue May 26, 2025
@matanlurey @stan-at-work
Add /coverage/ to .gitignore.tmp (flutter#169387)
9cdc508 
Closes flutter#166909.

Made `/build/` consistent while I was here.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@matanlurey matanlurey

Labels
c: proposal A detailed proposal for a change to Flutter P2 Important issues not at the top of the work list team-tool Owned by Flutter Tool team tool Affects the "flutter" command-line tool. See also t: labels. triaged-tool Triaged by Flutter Tool team
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add /coverage/ to .gitignore.tmp flutter/flutter
4 participants
@matanlurey @doctor-g @bkonyi @maheshj01
pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy