chore: remove dead code fossils (J8) - #68
Merged
Merged
Conversation
…ts, okta stub) - Drop unused mongoengine dependency (never imported anywhere). - Delete bin/load_home.py (dead duplicate; the live load_home is in views/home_view.py, and the dead file had a broken import). - Remove the commented-out legacy blueprint imports in register_blueprints. - Remove the empty okta_verification.main() stub and its __main__ guard; verify_new_webhook (the real function) is retained. No behavior change. App boots, ruff clean, full harness green.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Removes four confirmed dead-code fossils (internal ref J8). Deletion-only, no behavior change — app boots, ruff clean, full harness green (93 passed).
Removed
mongoenginedependency fromrequirements.txt— never imported anywhere (residue of an abandoned datastore evaluation).bin/load_home.py— a dead duplicate; the liveload_homelives inviews/home_view.py, and the dead file even had a broken import.app.py'sregister_blueprints()(the pre-unified-automations views).okta_verification.main()— an emptypassstub plus its__main__guard.verify_new_webhook(the real, in-use function) is retained.Targets
developfor the batched v3.2 release.