Skip to content

chore: replace .gitignore with standard Node.js/Next.js template#100

Merged
oorjagandhi merged 2 commits into
mainfrom
chore/update-gitignore
Jun 2, 2026
Merged

chore: replace .gitignore with standard Node.js/Next.js template#100
oorjagandhi merged 2 commits into
mainfrom
chore/update-gitignore

Conversation

@joengy

@joengy joengy commented Jun 2, 2026

Copy link
Copy Markdown
Collaborator

What does this PR do?

Follow-up to #75, as requested by @oorjagandhi.

Replaces the minimal .gitignore with GitHub's official Node.gitignore template, plus a .DS_Store entry. The previous file was very bare and didn't ignore .DS_Store, which is why a macOS file slipped into a PR.

Type of change

  • Feature
  • Bug fix
  • Chore / config
  • Hotfix

Notes

  • Adds coverage for build artifacts (.next, out, dist), logs, .env* files, caches, and OS/editor junk (.DS_Store).
  • Preserves the existing project rule ignoring package-lock.json / yarn.lock so only the root pnpm-lock.yaml is committed.

Linked Issues

Related to #75

joengy added 2 commits June 2, 2026 19:06
The previous .gitignore was minimal and missing common entries. Replace it with GitHub's official Node.gitignore template plus a .DS_Store entry, covering build artifacts, logs, env files, and editor/OS junk.
Re-add the project-specific rule ignoring package-lock.json / yarn.lock so
only the root pnpm-lock.yaml is committed that got deleted when I added the github template

@oorjagandhi oorjagandhi left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

ty

@deb-coder-man deb-coder-man 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.

LGTM!!!!!

@oorjagandhi oorjagandhi merged commit 689ee80 into main Jun 2, 2026
3 checks passed
@oorjagandhi oorjagandhi deleted the chore/update-gitignore branch June 2, 2026 13:33
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.

3 participants