Skip to content

Ci and lint update 20260227#13

Merged
AmandaBirmingham merged 3 commits intomainfrom
ci_and_lint_update_20260227
Feb 28, 2026
Merged

Ci and lint update 20260227#13
AmandaBirmingham merged 3 commits intomainfrom
ci_and_lint_update_20260227

Conversation

@AmandaBirmingham
Copy link
Copy Markdown
Collaborator

No description provided.

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the project’s CI and linting setup by removing legacy flake8/pep8 dependencies/config and introducing ruff, along with some formatting-only reflows to satisfy stricter lint rules.

Changes:

  • Switch CI linting from flake8 to ruff and add ruff configuration in pyproject.toml.
  • Remove flake8/pep8 from packaging and conda environment dependencies.
  • Reformat a number of source/test lines (line wrapping) and add .DS_Store to .gitignore.

Reviewed changes

Copilot reviewed 11 out of 12 changed files in this pull request and generated 5 comments.

Show a summary per file
File Description
setup.py Drops flake8/pep8 from install_requires (but still includes nose).
setup.cfg Removes the old [flake8] configuration block.
pyproject.toml Adds initial ruff lint rule selection.
environment.yml Replaces flake8/pep8 with ruff in the conda environment.
.github/workflows/main.yaml Updates CI python matrix and switches lint step to ruff check.
.gitignore Ignores macOS .DS_Store.
pysyndna/src/quant_orfs.py Formatting-only line wrap.
pysyndna/src/calc_cell_counts.py Formatting-only line wraps.
pysyndna/tests/test_util.py Formatting-only line wrap.
pysyndna/tests/test_quant_orfs.py Formatting-only line wraps (comprehensions/DF construction).
pysyndna/tests/test_fit_syndna_models.py Formatting-only line wraps; introduces a couple readability issues in comments/continuations.
pysyndna/tests/test_calc_cell_counts.py Formatting-only line wraps; adds # noqa: E501 for long test names.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@AmandaBirmingham AmandaBirmingham merged commit a64687d into main Feb 28, 2026
2 checks passed
@AmandaBirmingham AmandaBirmingham deleted the ci_and_lint_update_20260227 branch February 28, 2026 01:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants