Skip to content

Modernize repo#9

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

Modernize repo#9
wtgee merged 8 commits intomainfrom
modernize-repo

Conversation

@wtgee
Copy link
Member

@wtgee wtgee commented Oct 20, 2025

  • Move build settings over to pyproject.toml.
  • Remove unused files.
  • Remove ImagePathInfo (there is one in panoptes-utils.
  • Move the ImageStatus and ObservationStatus enums into the settings file.
  • Update docs

Closes #8

@wtgee wtgee requested a review from Copilot October 20, 2025 21:03
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 project's build and configuration infrastructure by migrating from setuptools to hatchling, consolidating settings, and updating documentation.

Key changes:

  • Migrated build configuration from setup.py/setup.cfg to pyproject.toml using hatchling
  • Consolidated enums (ImageStatus, ObservationStatus) into settings.py and removed ImagePathInfo class
  • Updated documentation build process and added comprehensive build instructions

Reviewed Changes

Copilot reviewed 20 out of 20 changed files in this pull request and generated no comments.

Show a summary per file
File Description
pyproject.toml Replaced setuptools configuration with hatchling build backend and consolidated all project metadata
setup.py Removed legacy setuptools entry point
setup.cfg Removed legacy configuration file
tox.ini Removed tox configuration
environment.yaml Removed conda environment file
.isort.cfg Removed isort configuration (likely moved to pyproject.toml or replaced by ruff)
src/panoptes/data/settings.py Added ImageStatus and ObservationStatus enums, removed PATH_MATCHER regex
src/panoptes/data/images.py Removed entire file containing ImagePathInfo and ImageStatus classes
src/panoptes/data/observations.py Removed ObservationStatus enum (moved to settings.py) and reordered imports
src/panoptes/data/search.py Reordered imports, removed unused astropy import, simplified f-string formatting
src/panoptes/data/utils/cli/main.py Updated type hints to use modern Python 3.12 syntax, removed unused exception variable
tests/conftest.py Removed placeholder test configuration file
tests/test_skeleton.py Removed skeleton test file
docs/conf.py Updated copyright year, reordered imports, added autodoc_mock_imports, modernized f-string
docs/index.md Added user documentation table of contents
docs/readme.md Added comprehensive documentation build instructions
docs/requirements-full.txt Added full documentation requirements file
.readthedocs.yaml Updated to use requirements-full.txt for documentation builds
.github/workflows/docs.yml Added new GitHub Actions workflow for building and deploying documentation
README.md Reformatted code examples from doctest style to plain Python

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

@wtgee wtgee requested a review from Copilot October 20, 2025 21:19
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

Copilot reviewed 23 out of 23 changed files in this pull request and generated 2 comments.


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

@wtgee wtgee merged commit 8302c41 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.

Update project build

2 participants