Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
General documentation of the hooks feature in `signac-flow`. * Added some sections about hooks. * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * Added information about project level hooks. * Typo. * added hooks to index. * Formatting and wording. * Indent note text. * Remove redundant information Co-authored-by: Brandon Butler <butlerbr@umich.edu> * More specific/descriptive wording about hooks. Co-authored-by: Brandon Butler <butlerbr@umich.edu> * Apply suggestions from code review Co-authored-by: Brandon Butler <butlerbr@umich.edu> * Updated decorator names. * Wrong reference. * Update docs/source/hooks.rst Co-authored-by: Brandon Butler <butlerbr@umich.edu> * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * Update docs/source/hooks.rst Co-authored-by: Alyssa Travitz <31974495+atravitz@users.noreply.github.com> * Apply suggestions from code review Co-authored-by: Alyssa Travitz <31974495+atravitz@users.noreply.github.com> * Applied suggestions from Alyssa and Corwin. * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * Update docs/source/hooks.rst Co-authored-by: Corwin Kerr <cbkerr@umich.edu> * Use install_hooks method. * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * Added also for continuity. * Clarify a few words Co-authored-by: Kelly Wang <47036428+klywang@users.noreply.github.com> * Fix format of signac - use bold * Clarify triggers and naming * Move similar sentences together * Improve wording of example uses of hooks * Fix syntax * Remove duplicate sentence * Condense examples * Update names of hooks * Move trigger list to separate section * Remove old indexing section from toc * Reflow text * Clarify success log example * Fix list * Link to executable directive * Specify how flow is run * Make demo operation more descriptive * Syntax fix * Move duplicate info together for discussion * Add draft text to hook trigger section * Update project level hooks example * Update project level hooks example description * Update intro for custom install_hooks method * Switch to on_exception * Update operation hooks example code * There is a rat in separate * Fix install_hooks example * Replace add_hook with operation_hooks * Improve line wrapping of comment * Replace failure language with exception * Remove redundant explanation of arguments * Use dashes in sphinx section labels * Remove redundant description * Small clarifications * Add more description to project-level hooks example * Improve wording * Apply settled suggestions from code review Co-authored-by: Bradley Dice <bdice@bradleydice.com> * Update docs/source/hooks.rst Co-authored-by: Bradley Dice <bdice@bradleydice.com> * Update docs/source/hooks.rst Co-authored-by: Bradley Dice <bdice@bradleydice.com> * Assign hook to list rather than appending Co-authored-by: Bradley Dice <bdice@bradleydice.com> * Make it clear that install hooks returns project Co-authored-by: Bradley Dice <bdice@bradleydice.com> * Use same form in example hooks * Apply suggestions from code review Co-authored-by: Bradley Dice <bdice@bradleydice.com> * shorten time logging example Co-authored-by: Bradley Dice <bdice@bradleydice.com> * Make custom hook install take project as arg, not constructor Co-authored-by: Bradley Dice <bdice@bradleydice.com> * Update docs/source/hooks.rst Co-authored-by: Bradley Dice <bdice@bradleydice.com> * Remove superfluous comments Co-authored-by: Brandon Butler <butlerbr@umich.edu> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com> Co-authored-by: Brandon Butler <butlerbr@umich.edu> Co-authored-by: Alyssa Travitz <31974495+atravitz@users.noreply.github.com> Co-authored-by: Corwin Kerr <cbkerr@umich.edu> Co-authored-by: Bradley Dice <bdice@bradleydice.com>
- Loading branch information