Skip to content

Feature/444 rush add bazel builds as a target - #56

Merged
troels-im merged 195 commits into
mainfrom
feature/444--rush-add-bazel-builds-as-a-target
Feb 27, 2026
Merged

Feature/444 rush add bazel builds as a target#56
troels-im merged 195 commits into
mainfrom
feature/444--rush-add-bazel-builds-as-a-target

Conversation

@troels-im

Copy link
Copy Markdown
Contributor

No description provided.

troels-im and others added 30 commits May 7, 2025 22:54
…nality

- Redesigned the project structure for improved modularization and clarity.
- Added a new `core` module to encapsulate dotenv loading and saving functionality, replacing the previous `dotenv_utils`.
- Removed obsolete files: `dotenv_utils.rs` and `git.rs`, migrating their logic to the new structure.
- Updated `main.rs`, `public_env_defs.rs`, and other relevant modules to use the new core utilities.
- Enhanced organization of command implementations in `cli` by modularizing into specific command files.
- Improved handling of environment variables to include support for multi-line and quoted values in `.env` files.
- Adjusted Cargo.toml dependencies to align with the new project organization and structure.

This refactor aims to make the Rush CLI more maintainable, testable, and extensible, improving overall code quality and project navigation.
- Introduced a new module for path matching, separating concerns and making it more maintainable.
- Implemented functions to find and parse .gitignore files, enhancing the path matching capabilities.
- Updated existing code to utilize the newly refactored path matcher structure.
- Added detailed tests to ensure the functionality of the path matcher and .gitignore parsing.
- Restructured directories within the project for better organization and clarity.
- Removed legacy path matching code from previous structure.

This refactoring aims to improve the maintainability, testability, and functionality of the Rush CLI codebase.
@troels-im
troels-im merged commit 5a17ef7 into main Feb 27, 2026
1 check failed
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