CI remove test_docstrings (duplicate with ruff, which runs pydocstyle) #196
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
BorisMuzellec
requested review from
maikia,
arthurPignetOwkin and
mandreux-owkin
as code owners
BorisMuzellec
added a commit
that referenced
this pull request
Oct 26, 2023
BorisMuzellec
added a commit
to adamgayoso/PyDESeq2
that referenced
this pull request
Oct 27, 2023
BorisMuzellec
added a commit
that referenced
this pull request
Nov 24, 2023
* Create inference.py * Create default_inference.py * Update dds.py * Update ds.py * Update plot_minimal_pydeseq2_pipeline.py * Update plot_pandas_io_example.py * Update plot_step_by_step.py * don't process non-zeros in utils * Update dds.py * formatting * formatting * ignore typing * ignore typing * docstrings * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * cleanup * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * noqa * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * docstrings * CI remove test_docstrings (duplicate with ruff, which runs pydocstyle) (#196) * docs: fix docs, incorporate new classes * refactor: remove pass statements from abstractmethods with docstrings * style: rename MLE_lfc variable to lowercase mle_lfc --------- Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com> Co-authored-by: Boris Muzellec <BorisMuzellec@users.noreply.github.com> Co-authored-by: Boris MUZELLEC <boris.muzellec@owkin.com>
BorisMuzellec
added a commit
that referenced
this pull request
Nov 24, 2023
* split vst functions into fit & transform * fix leakage of size factors * handle exceptions and document specific cases and functions * update tests * fix docstring * CI: Add ruff pre-commit hook (#192) * CI: add ruff * ci: fix ruff line length argument * ci: update ruff configuration, remove flake8 and isort * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * CI: integrate isort options to ruff, remove nbqa * CI: fix force-single-line typo * CI: add ruff exceptions for sphinx gallery examples * chore: fix linting * chore: linting * ci: add exception to linting for sphinx examples * docs: renove extra period in sphinx examples --------- Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com> * CI Remove precommit from workflow (#195) * CI remove test_docstrings (duplicate with ruff, which runs pydocstyle) (#196) * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * refactor: minor docstring and line comment edits * refactor: minor docstring and line comment edits --------- Co-authored-by: SimonGrouard <simon.grouard@owkin.com> Co-authored-by: Boris Muzellec <BorisMuzellec@users.noreply.github.com> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com> Co-authored-by: Adam Gayoso <adamgayoso@users.noreply.github.com> Co-authored-by: Boris MUZELLEC <boris.muzellec@owkin.com>
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.
Reference Issue or PRs
#191
What does your PR implement? Be specific.
test_docstringsrunsnumpydoc.validate, but now that the CI runs pydocstyle via ruff, it's no longer necessary.