Skip to content

Conversation

@kedong2002
Copy link

@kedong2002 kedong2002 commented Nov 25, 2025

Note

.gitignore expanded and reorganized to cover Node/Next.js builds, caches/temp, env files, VS Code, and Foundry broadcast artifacts (with 31337 and dry-run exclusions).

  • Git ignore rules:
    • Builds/Caches: Add node_modules/, out/, cache/, /tmp/, .DS_Store, buffer/.
    • Sensitive config: Ignore .env, .env.local, .env.testnet, .env.mainnet.
    • Tooling: Ignore .vscode/.
    • Web3/Foundry artifacts: Ignore /broadcast/ with !/broadcast re-include; exclude /broadcast/*/31337/ and /broadcast/**/dry-run/.

Written by Cursor Bugbot for commit 25cf0a9. This will update automatically on new commits. Configure here.


# Re-include the top-level broadcast directory itself so that sub-rules can work.
# NOTE: The actual broadcast outputs are handled by the exclusion patterns below.
# This line is often implicitly handled by Git but kept for clarity/compatiblity.
Copy link

Choose a reason for hiding this comment

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

Bug: Broadcast directory negation pattern ineffective

The pattern !/broadcast attempts to re-include the broadcast directory after excluding it with /broadcast/ on line 39, but this negation won't work because !/broadcast without a trailing slash only matches a file, not a directory. This prevents the subsequent exclusion patterns on lines 47 and 49 from working, as Git won't traverse into an already-excluded directory.

Fix in Cursor Fix in Web

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.

1 participant