Skip to content

Add Deno.gitignore#4816

Merged
wirecat merged 1 commit intogithub:mainfrom
mvanhorn:osc/deno-gitignore
Apr 17, 2026
Merged

Add Deno.gitignore#4816
wirecat merged 1 commit intogithub:mainfrom
mvanhorn:osc/deno-gitignore

Conversation

@mvanhorn
Copy link
Copy Markdown
Contributor

Reasons for making this change

Deno is a JavaScript/TypeScript runtime with 100k+ GitHub stars. The existing Node.gitignore covers Node.js artifacts but none of Deno's unique patterns.

This addresses @wirecat's feedback on #4101, which asked for a version trimmed to Deno-specific rules only. The original PR (#4101, open since 2022) included IDE and OS rules that don't belong. The author hasn't responded since 2023. This is a fresh, focused replacement.

Included patterns:

  • .deno/ - local cache directory when using project-local DENO_DIR
  • coverage/ and *.lcov - output from deno coverage

Intentionally NOT ignored:

  • deno.lock - Deno documentation says to commit the lockfile for reproducible builds
  • vendor/ - Deno documentation says to commit vendored dependencies for hermetic builds

The coverage/ pattern matches what Deno's own standard library uses.

Links to documentation supporting these rule changes

If this is a new template

Link to application or project's homepage: https://deno.com/

Merge and Approval Steps

  • Confirm that you've read the contribution guidelines and ensured your PR aligns
  • Ensure CI is passing
  • Get a review and Approval from one of the maintainers

This contribution was developed with AI assistance (Claude Code).

@mvanhorn mvanhorn requested a review from a team as a code owner March 17, 2026 22:35
Copy link
Copy Markdown
Collaborator

@wirecat wirecat left a comment

Choose a reason for hiding this comment

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

@mvanhorn Thank you for picking this up and contributing!

@wirecat wirecat merged commit 1187b2c into github:main Apr 17, 2026
@wirecat wirecat mentioned this pull request Apr 17, 2026
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.

2 participants