Skip to content

refactor(config): remove unused config files#6566

Merged
kuny0707 merged 1 commit intotronprotocol:developfrom
halibobo1205:feat/remove_unused_config
Mar 3, 2026
Merged

refactor(config): remove unused config files#6566
kuny0707 merged 1 commit intotronprotocol:developfrom
halibobo1205:feat/remove_unused_config

Conversation

@halibobo1205
Copy link
Contributor

@halibobo1205 halibobo1205 commented Mar 2, 2026

Summary

  • Removed 4 unused config files from framework/src/main/resources/:
    • config-backup.conf — old backup with no references in the codebase
    • config-beta.conf — no references in the codebase
    • config-test-net.conf — no references in the codebase
    • config-localtest.conf — only used in tests; src/test/resources/config-localtest.conf takes precedence on the test classpath, making this copy dead code

Test plan

  • Verify existing tests still pass (they load config-localtest.conf from src/test/resources)
  • Verify no runtime references to removed files

@halibobo1205
Copy link
Contributor Author

Maybe should adding a CONFIG_MIGRATION.md to indicate that the these deleted files have been moved to https://github.com/tronprotocol/tron-deployment?

I think this will help users find the correct configs after this cleanup.

I don't see the need for this. These configurations were originally set up for unit tests — they serve no purpose in the production environment and are not being utilized in the current test environment either.

@vividctrlalt
Copy link
Contributor

Great cleanup! These files should have been removed a long time ago.

Removed 4 config files from framework/src/main/resources that were
not referenced anywhere in the codebase:
- config-backup.conf: no references
- config-beta.conf: no references
- config-test-net.conf: no references
- config-localtest.conf: only used in tests, shadowed by
  src/test/resources/config-localtest.conf on the test classpath
@halibobo1205 halibobo1205 force-pushed the feat/remove_unused_config branch from 213a105 to b302b5f Compare March 3, 2026 07:30
@kuny0707 kuny0707 merged commit ba73305 into tronprotocol:develop Mar 3, 2026
11 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.

3 participants