Skip to content

chore: ignore .DS_Store and tighten SECURITY.md - #151

Merged
mayankpande88 merged 1 commit into
mainfrom
chore/prepublic-tidy
May 12, 2026
Merged

chore: ignore .DS_Store and tighten SECURITY.md#151
mayankpande88 merged 1 commit into
mainfrom
chore/prepublic-tidy

Conversation

@mayankpande88

Copy link
Copy Markdown
Contributor

Description

Two small pre-public cleanups.

  • `.gitignore`: add `.DS_Store` so macOS Finder metadata doesn't get committed.
  • `SECURITY.md`: remove the `security@nudgebee.com` fallback. That address was named in the original draft but not verified as monitored — pointing reporters at a dead inbox is worse than relying on GitHub's private vulnerability reporting as the sole channel. PVR will be enabled via repo settings as a separate admin step.

How Has This Been Tested?

  • Both files render correctly.
  • `SECURITY.md` still has a working channel (GitHub PVR via the Security tab).

.gitignore: add .DS_Store so macOS Finder metadata doesn't end up
committed by accident.

SECURITY.md: drop the security@nudgebee.com fallback. That address
was named but not verified to be monitored, and pointing reporters
at a dead inbox is worse than leaving GitHub's private vulnerability
reporting as the sole channel.

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Code Review

This pull request updates the .gitignore file to exclude macOS .DS_Store files and simplifies the SECURITY.md file by removing the email contact option for vulnerability reporting. A suggestion was made to also include ._* files in the .gitignore to prevent other macOS-specific metadata from being committed.

Comment thread .gitignore
@mayankpande88
mayankpande88 merged commit bc408a9 into main May 12, 2026
1 check passed
@blue4209211
blue4209211 deleted the chore/prepublic-tidy branch May 22, 2026 09:27
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