Skip to content

Conversation

lixiname
Copy link

  • Replace legacy setup.py with standardized pyproject.toml
  • Follow PEP 517 (build system) and PEP 621 (metadata)
  • Use setuptools as build backend with modern configuration
  • add pyproject.toml file、test_setuptools.py file (a test scripts of toml)、Modify init.py file
    This improves dependency management, IDE support, and aligns with current Python packaging standards.

Copy link
Contributor

@rgaiacs rgaiacs left a comment

Choose a reason for hiding this comment

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

This looks a great start. All the tests are failing, probably because of the new test. Can change in the GitHub actions configuration be change to increase the version of Python to be used?

[project]
name = "jupyter-repo2docker"
dynamic = ["version"]
description = "Repo2docker: Turn code repositories into Jupyter enabled Docker Images"
Copy link
Contributor

Choose a reason for hiding this comment

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

Replace Repo2docker with repo2docker



[tool.setuptools_scm]
# 这个配置会让 setuptools-scm 自动创建一个 my_awesome_project/_version.py 文件,
Copy link
Contributor

Choose a reason for hiding this comment

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

Remove comments

@@ -0,0 +1,18 @@
import tomllib
Copy link
Contributor

Choose a reason for hiding this comment

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

tomllib requires Python 3.11. This might be the reason that all tests are failing.

try:
with open("pyproject.toml", "rb") as f:
tomllib.load(f)
print("✅ TOML语法正确")
Copy link
Contributor

Choose a reason for hiding this comment

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

Can the Chinese text be translated into English?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

2 participants