-
Notifications
You must be signed in to change notification settings - Fork 0
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
fix: update configuration messages #2
Merged
Conversation
This file contains 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
This commit updates the display messages related to the auto-creation of the configuration file for clarity and consistency. The warning message is replaced with an info message in the configuration file, and the test is adjusted accordingly to reflect this change.
These changes refactor the test structure by removing the unused 'ai_commit' fixture and updating several test functions to improve clarity and maintainability. The changes also involve adding assertions to ensure proper configuration handling and removing commented-out code to enhance readability.
This change modifies the way configuration paths are handled by creating the XDG configuration directory if it does not exist, ensuring that the application can properly utilize the XDG configuration path.
These changes update the test cases to call the run() method on the AiCommit instance, ensuring that the tests accurately reflect the current implementation of the AiCommit class.
This change comments out the mock_argv fixture, which may no longer be needed for the tests.
These changes modify several test functions to include mock_home_dir, ensuring that the existence of the config file is validated as part of the tests. This enhances test reliability and covers more scenarios related to configuration file checks.
These changes enhance the mock home directory setup and add assertions to verify the output of the system under test, ensuring that the tests accurately reflect the expected behavior.
This change updates the .gitignore file to ensure that the tmp directory is ignored by Git, preventing temporary files from being tracked.
This change modifies the pytest options in the pyproject.toml file to include parameters for cache management and temporary directory configuration, improving test execution.
This change refactors the test files by removing unnecessary whitespace and ensuring consistent formatting, which enhances code readability and maintainability.
This commit groups changes that comment out existing test functions in the test_system.py file. The changes may be part of a refactoring effort or to temporarily disable tests while further development is conducted.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
This commit updates the display messages related to the auto-creation of the configuration file for clarity and consistency. The warning message is replaced with an info message in the configuration file, and the test is adjusted accordingly to reflect this change.