Skip to content

Modernize repo hotfix 2#11

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

Modernize repo hotfix 2#11
wtgee merged 2 commits intomainfrom
modernize-repo-hotfix-2

Conversation

@wtgee
Copy link
Member

@wtgee wtgee commented Oct 20, 2025

Udpating testing and build to use hatch.

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

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

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.

]
[tool.hatch.envs.tests.scripts]
test = "pytest"
test-cov = "pytest --cov=panoptes.data --cov-report term-missing --verbose"
Copy link

Copilot AI Oct 20, 2025

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