You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Improve the validation and testing of the preCICE configuration formatter to support the transition of PR #3 from a draft to a fully implemented feature. (The changes there also need to be applied to our own subscript that is not a cli-script with args)
Background
The current pre-commit hook for preCICE configs on precice/precice-pre-commit-hooks#3 needs further refinement/testing to ensure robust and comprehensive formatting of configuration files.
Pull request on original pre-commit hook has been marked as ready for review. All changes requested there also need to be applied to our changed up script. Otherwise i hope this is finished
Another idea i had was to use the formatter to sort alphabetically everything and then call it on both the original and the generated precice config. Then you can just compare line by line.
Enhance and Test preCICE Configuration Formatter
Objective
Improve the validation and testing of the preCICE configuration formatter to support the transition of PR #3 from a draft to a fully implemented feature. (The changes there also need to be applied to our own subscript that is not a cli-script with args)
Background
The current pre-commit hook for preCICE configs on precice/precice-pre-commit-hooks#3 needs further refinement/testing to ensure robust and comprehensive formatting of configuration files.
Related Issues
Expected Outcomes
The text was updated successfully, but these errors were encountered: