Skip to content

chore: clean up temporary project files and ignore dist directories - #647

Merged
p3ris0n merged 1 commit into
mainfrom
chore/clean-up-codebase
Aug 4, 2026
Merged

chore: clean up temporary project files and ignore dist directories#647
p3ris0n merged 1 commit into
mainfrom
chore/clean-up-codebase

Conversation

@p3ris0n

@p3ris0n p3ris0n commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

Overview

This PR removes several temporary, accidental, and leftover files from the repository root and subdirectories to keep the project tree clean and organized. None of these files are referenced by any workflows, scripts, documentation, or code. It also updates .gitignore to globally ignore dist/ directories and removes previously tracked compiled artifacts (cli/dist, indexer/dist, react/dist) from version control.


Files Removed / Untracked

File / Directory Description & Reason for Removal
clippy.log A ~170 KB raw compiler warning/error log from running cargo clippy that was accidentally committed.
fix_tests.py A temporary Python script previously used to rewrite test stubs in contracts/token/src/test.rs.
PR_BODY.md A leftover draft PR description Markdown file in the project root.
IMPLEMENTATION_SUMMARY.md A temporary summary Markdown file created for PR issues #179#182.
e2e/test.rs A dummy scratch file containing empty test stubs (fn foo() {}). Not referenced in e2e/Cargo.toml or used in any tests.
src/errors/sorobanParser.ts A stray file inside an accidental top-level src/ folder containing only a placeholder comment. Actual SDK error definitions live in sdk/src/errors.ts.
cli/dist/, indexer/dist/, react/dist/ Compiled JavaScript and TypeScript declaration files untracked from Git; .gitignore updated to globally ignore dist/.

Verification

  • Verified that none of the removed files are imported or referenced across the codebase.
  • Ran cargo check --workspace — all contracts and integration tests compile cleanly.

@p3ris0n
p3ris0n merged commit 0ffbef0 into main Aug 4, 2026
3 checks passed
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