Skip to content

Develop#145

Merged
LuernOutOfOrder merged 57 commits intomainfrom
develop
Jun 7, 2025
Merged

Develop#145
LuernOutOfOrder merged 57 commits intomainfrom
develop

Conversation

@LuernOutOfOrder
Copy link
Owner

This pull request introduces several improvements to the nyx CLI tool, focusing on code quality enhancements, usability improvements, and new functionality. The most significant changes include the addition of command aliases in the Cargo configuration, the introduction of a new health module, and the removal of redundant helper functions for logging. Additionally, multiple refactorings were performed to improve code readability and efficiency.

Usability Improvements:

  • .cargo/config.toml: Added command aliases for common cargo operations such as build, test, run, and clippy, making commands shorter and easier to use. Also added a future-incompat-report configuration to display warnings about future incompatibilities.
  • Cargo.toml: Updated the version of the nyx package from 2.8.0 to 2.10.2.

New Functionality:

  • src/health/mod.rs: Introduced a new health module to check user environment health based on configuration files, with support for system and network health checks.

Code Simplification and Refactoring:

  • src/cleanup/mod.rs, src/git/mod.rs, src/hello/mod.rs, src/doctor/mod.rs: Refactored multiple functions to use &str instead of String where possible, replaced String::from_utf8_lossy with str::from_utf8 for more efficient string handling, and removed unnecessary .to_string() calls. [1] [2] [3]
  • src/doctor/helper.rs: Removed redundant helper functions (installed, not_installed, warning) and replaced their usage with centralized logging functions. [1] [2]

Code Quality Improvements:

  • .github/workflows/rust.yml: Added a linting step using cargo clippy to enforce coding standards and prevent warnings.
  • src/cleanup/mod.rs: Improved error messages and logging by replacing .to_string() with .to_owned() for better performance and clarity. [1] [2]

These changes collectively enhance the usability, maintainability, and functionality of the nyx CLI tool.

LuernOutOfOrder and others added 30 commits May 10, 2025 00:28
…d-to-run-in-config

120 add a list of command to run in config
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…o-nyx-usage-message

Add missing usage for update command
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…fig-option-enabled

136 add log files if config option enabled
LuernOutOfOrder and others added 23 commits June 2, 2025 15:01
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…hub.com:LuernOutOfOrder/NYX into 140-check-all-codebase-to-optimize-memory-usage
…o-optimize-memory-usage

Optimize memory usage
@LuernOutOfOrder LuernOutOfOrder added this to the v3.0.0 milestone Jun 7, 2025
@LuernOutOfOrder LuernOutOfOrder self-assigned this Jun 7, 2025
@LuernOutOfOrder LuernOutOfOrder added the enhancement New feature or request label Jun 7, 2025
@LuernOutOfOrder LuernOutOfOrder merged commit 9fecbea into main Jun 7, 2025
0 of 2 checks passed
@LuernOutOfOrder LuernOutOfOrder deleted the develop branch June 7, 2025 20:27
@LuernOutOfOrder LuernOutOfOrder restored the develop branch June 7, 2025 20:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant