doc: Explain that .gitignore is not for IDE-specific excludes #32417

pull maflcko wants to merge 1 commits into bitcoin:master from maflcko:2505-gitignoreignore changing 1 files +5 −0

maflcko commented at 4:53 PM on May 4, 2025: member
This comes up frequently, so document it.

Examples:
- #27275
- #21894
- #32392
- #17868
...
DrahtBot commented at 4:53 PM on May 4, 2025: contributor



The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.



Code Coverage & Benchmarks

For details see: https://corecheck.dev/bitcoin/bitcoin/pulls/32417.



Reviews

See the guideline for information on the review process.

Type Reviewers

ACK hebasto, l0rinc, janb84

If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.
DrahtBot added the label Docs on May 4, 2025
l0rinc commented at 5:04 PM on May 4, 2025: contributor

ACK fa9c8bc05e3bb72e2c3c726b643997cea4874158
hebasto commented at 5:04 PM on May 4, 2025: member

Concept ACK.

I would suggest expanding this list to include desktop environment–specific files, such as .DS_Store on macOS.
doc: Explain that .gitignore is not for IDE-specific excludes fada115cbe
maflcko force-pushed on May 4, 2025
hebasto approved
hebasto commented at 6:05 PM on May 4, 2025: member

ACK fada115cbeaa8c0ca3d19178499079b66ee5f499.
DrahtBot requested review from l0rinc on May 4, 2025
l0rinc commented at 6:24 PM on May 4, 2025: contributor

ACK fada115cbeaa8c0ca3d19178499079b66ee5f499

PR title and commit message still state IDE, but it's not important.
janb84 commented at 7:44 PM on May 4, 2025: contributor

ACK fada115

Will reduce issues & PR's , good to include the extra comment.
hebasto merged this on May 5, 2025
hebasto closed this on May 5, 2025
maflcko deleted the branch on May 5, 2025