Skip to content

Remove non-runtime dependencies from requirements.txt#212

Merged
jellybean2004 merged 1 commit into
SasView:release_0.12.0from
llimeht:tmp/requirements
Jun 10, 2026
Merged

Remove non-runtime dependencies from requirements.txt#212
jellybean2004 merged 1 commit into
SasView:release_0.12.0from
llimeht:tmp/requirements

Conversation

@llimeht

@llimeht llimeht commented Jun 3, 2026

Copy link
Copy Markdown
Contributor

Move development-time dependencies into requirements-dev.txt, to ensure that the sasdata wheel does not end up depending on testing tools.

Closes: #211

@codescene-delta-analysis codescene-delta-analysis Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Our agent can fix these. Install it.

No application code in the PR — skipped Code Health checks.

Quality Gate Profile: Custom Configuration
Install CodeScene MCP: safeguard and uplift AI-generated code. Catch issues early with our IDE extension and CLI tool.

@krzywon krzywon left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This looks correct. I will let @jellybean2004, the release manager, decide whether or not this belongs in the next release.

@jellybean2004

Copy link
Copy Markdown
Member

As @llimeht has marked the issue as a blocker for the release, I will trust his judgment and add this to the next pre-release. If any major issues arise, I may have to omit it from the release.

@jellybean2004 jellybean2004 merged commit 79e5b7a into SasView:release_0.12.0 Jun 10, 2026
6 checks passed
@llimeht llimeht deleted the tmp/requirements branch June 10, 2026 10:56
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.

3 participants