add bump-recipe command #188
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
schuylermartin45
requested changes
Oct 18, 2024
Co-authored-by: Schuyler Martin <schuylermartin45@gmail.com>
schuylermartin45
requested changes
Nov 5, 2024
Co-authored-by: Schuyler Martin <schuylermartin45@gmail.com>
Co-authored-by: Schuyler Martin <schuylermartin45@gmail.com>
Co-authored-by: Schuyler Martin <schuylermartin45@gmail.com>
Co-authored-by: Schuyler Martin <schuylermartin45@gmail.com>
Co-authored-by: Schuyler Martin <schuylermartin45@gmail.com>
* Add tilde to _JINJA_VAR_FUNCTION_PATTERN * Create V0_FORBIDDEN_JINJA constant and check for strings containing them when converting recipes * Add exception for when Jinja syntax that is too complex to convert is detected * Make the linter happy * Change name of V0_UNSUPPORTED_JINJA constant * Log 'complex Jinja' warning with the MessageTable instance instead of throwing an exception * Add more unsupported Jinja to regression_jinja_sub.yaml test recipe file to catch new warning in test * Simplify warning message and update related test
* Initial blueprint for dependency scanning module * Starts to adapt work from abandonned dependency CLI branch * Finishes initial work on pulling module names as dependencies * Adds local module filtering * Adds dependency type support to PythonDependencyScanner * Adds MessageTable instance to base dependency class * Adds missing docs * Minor fixes * Adds note about scanning performance * Addresses linter and analyzer issues * Adds initial dependency scanning unit test * Adds ignore line to dummy project file * Improves unit test coverage * Updates recipe file to ignore test data directory * Adds support for multiple imports on one line
* Renames RecipeParserDeps -> RecipeReaderDeps * Renames RecipeParserDeps -> RecipeReaderDeps * Fleshes-out initial utility functions in the new RecipeParserDeps class * Improves SelectorParser capabilities * Adds selector management to the add_dependency() function * Adds unit test for selector rendering * Added SelectorParser type variant to test_add_selector() * Adds initial RecipeParserDeps unit tests * Adds initial multi-output add_dependency() test variants * Adds unit tests for remove_dependency() * Adds invalid path tests to test_add_dependency() * Adds alternative DependencyMode unit tests to test_add_dependency() * Adds missing invalid path checks for test_add_dependency() * Adds more edge-case unit tests to test_add_dependency() * Fixes patch-add append edge case * Adds missing unit test for adding a dependency to a non-existent dependency section * Adds another missing unit test for adding a dependency to a non-existent dependency section
* Fixes resilience issue with dependency_data_from_str() function also makes function names more consistent * Adds missing unit tests for dependency module * Removes outdated TODO comment * Adds regression testing for the numpy dependency parsing issue
…e` (#227) * Ignores another pytest deprecation * Simplifications and deprecations for the Makefile. The preferred method for installing CRM is through conda/conda-forge * Overhauls existing README documentation * Updates README TOC * More README doc work * More README improvements * Filling more doc gaps * Simplifies and automates the API documentaiton process * Update README.md Co-authored-by: Mahe Iram Khan <65779580+ForgottenProgramme@users.noreply.github.com> * Update README.md Co-authored-by: Mahe Iram Khan <65779580+ForgottenProgramme@users.noreply.github.com> * Update README.md Co-authored-by: Mahe Iram Khan <65779580+ForgottenProgramme@users.noreply.github.com> * Adds a contribution file * Update CONTRIBUTING.md --------- Co-authored-by: Mahe Iram Khan <65779580+ForgottenProgramme@users.noreply.github.com>
schuylermartin45
approved these changes
Nov 6, 2024
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.
Description
resolves #129
adds a
bump-recipecommand with--build-numflagTo do: