Skip to content

Modernize repo hotfix 2#11

Merged
wtgee merged 2 commits into
mainfrom
modernize-repo-hotfix-2
Oct 20, 2025
Merged

Modernize repo hotfix 2#11
wtgee merged 2 commits into
mainfrom
modernize-repo-hotfix-2

Conversation

@wtgee

@wtgee wtgee commented Oct 20, 2025

Copy link
Copy Markdown
Member

Udpating testing and build to use hatch.

@wtgee wtgee requested a review from Copilot October 20, 2025 22:18

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull Request Overview

This PR modernizes the repository's build and testing infrastructure by migrating from tox to hatch for package management and testing.

Key Changes:

  • Migrated testing dependencies from project.optional-dependencies to tool.hatch.envs.tests
  • Updated GitHub Actions workflow to use hatch instead of tox for building and publishing
  • Incremented version number in CHANGELOG to 0.2.3

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
pyproject.toml Removed project.optional-dependencies.testing section and pytest configuration; added hatch test environment with dependencies and test scripts
CHANGELOG.md Updated version number from 0.2.2 to 0.2.3
.github/workflows/create-release.yml Replaced tox commands with hatch equivalents for building the package

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread pyproject.toml
]
[tool.hatch.envs.tests.scripts]
test = "pytest"
test-cov = "pytest --cov=panoptes.data --cov-report term-missing --verbose"

Copilot AI Oct 20, 2025

Copy link

Choose a reason for hiding this comment

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

The pytest configuration options previously defined in [tool.pytest.ini_options] (including norecursedirs and testpaths) have been removed. Consider adding a [tool.pytest.ini_options] section back with norecursedirs = ['dist', 'build'] and testpaths = ['tests'] to maintain the same test discovery behavior.

Suggested change
test-cov = "pytest --cov=panoptes.data --cov-report term-missing --verbose"
test-cov = "pytest --cov=panoptes.data --cov-report term-missing --verbose"
[tool.pytest.ini_options]
norecursedirs = ["dist", "build"]
testpaths = ["tests"]

Copilot uses AI. Check for mistakes.
@wtgee wtgee merged commit fa301ea into main Oct 20, 2025
3 checks passed
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