Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

FEATURE: Module Monitor #22

Merged
merged 17 commits into from
Aug 16, 2024
Merged

FEATURE: Module Monitor #22

merged 17 commits into from
Aug 16, 2024

Conversation

trentonyo
Copy link
Contributor

@trentonyo trentonyo commented Aug 8, 2024

Module Monitor

ASSOCIATED ISSUES:

Description

👁️💻👁️ Module Monitor: A little mini terminal will scroll while Modules' init scripts are run, piping that subprocess' output. (Open a path on not Windows to see)

⚡🐍 Python Minimum Version: Opened up the minimum Python version and expanded the test matrix to cover them.

Definition of Done

  • Changes are approved
  • Contains no breaking changes
  • Changes are implemented in all components
  • Each component associated with issue is updated (no stubs)
  • Each new feature has unit tests written and confirmed
  • All tests pass, no regressions
  • All documentation is updated, including release notes

Bump the version to 0.2.0 and update authors in pyproject.toml. Introduce starterfile functionality with a new CLI command and corresponding tests for managing starterfile setup.
…steps.md.

Implemented functionality to replace environment variables in the Startersteps.md file when present. Introduced tests to verify this new behavior and added necessary environment configuration files for testing.
Updated `starterfile_up_only` to print an error message and exit if the starterfile doesn't exist. Also, added a test to ensure this validation works as expected by including a non-existent starterfile path.
Updated Black to version 24.8.0 and Coverage to version 7.6.1 in the poetry.lock file. This ensures compatibility with the latest features and bug fixes for these dependencies.
Modified the GitHub Actions workflow to test on Ubuntu, Windows, and macOS. This ensures broader compatibility and catches platform-specific issues.
Refactor module initialization to directly invoke `get_script` for "init" and "destroy" scripts without raising exceptions. Updated unit tests to ensure missing scripts correctly trigger failures, improving code clarity and robustness.
…unners

Updated the GitHub Actions YAML to specify the default shell as bash and ensured the virtual environment is sourced before running pytest. This improves consistency in the test environment setup.
Introduce monitored_subprocess to display subprocess output with rich live updates. Enhanced Module methods to support real-time logging and better error handling by integrating console and log_path arguments.
Upgrade Poetry to 1.8.3, include new dependencies (`exceptiongroup` and `tomli`), and upgrade `pyyaml` to 6.0.2. This ensures compatibility with newer Python versions and improves the package management.
Updated module schema to use `is_yaml_loadable_type` for `type` and `default` validation. Added new end-to-end tests for complex starter workflow, including a test complex YAML file and environment file. Incremented version to 0.3.3.
@trentonyo trentonyo marked this pull request as ready for review August 10, 2024 07:56
@trentonyo trentonyo merged commit 1e14e02 into dev Aug 16, 2024
16 of 17 checks passed
@trentonyo trentonyo deleted the feature/cli-module-pipe branch August 16, 2024 05:37
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.

Set version requirements
1 participant