Skip to content

Migrate from setup.py to pyproject.toml#51

Closed
hadijannat wants to merge 7 commits into
mainfrom
update-pyproject-toml
Closed

Migrate from setup.py to pyproject.toml#51
hadijannat wants to merge 7 commits into
mainfrom
update-pyproject-toml

Conversation

@hadijannat
Copy link
Copy Markdown

@hadijannat hadijannat commented Jul 20, 2024

This pull request updates the build configuration by modifying the pyproject.toml and setup.py files to include development dependencies and refine the project setup. The goal is to improve the development environment setup and ensure consistent dependency management.

Changes:

  1. pyproject.toml: Updated [build-system] requirements to include setuptools>=45, wheel, and setuptools_scm[toml]>=6.2.
    Added dev optional dependencies section to include pytest, flake8, coverage, and other development tools. Refined project metadata such as version, authors, and dependencies. Improved package data and exclusion configuration.
  2. setup.py: Added extras_require section to define dev dependencies.
    Ensured compatibility with the changes in pyproject.toml.

Procedure:
CI Configuration:
The ci.yml file has been updated to use the dev dependencies during the CI pipeline runs. This ensures that all necessary development tools are installed for testing and code quality checks.
Development Setup:
To set up the development environment locally, run:
pip install .[dev]
This will install the package along with all development dependencies specified in the pyproject.toml and setup.py files.
Impact:
These changes streamline the setup process for new developers and CI environments.
Ensures that the development and testing environments are consistent across different setups.

- Remove reference to aas-compliance-check from pyproject.toml
- Add newline at end of pyproject.toml
- Remove entry point for aas-compliance-check from setup.py
@hadijannat
Copy link
Copy Markdown
Author

This pull request updates the build configuration by modifying the pyproject.toml and setup.py files to include development dependencies and refine the project setup. The goal is to improve the development environment setup and ensure consistent dependency management.

Changes:

  1. pyproject.toml: Updated [build-system] requirements to include setuptools>=45, wheel, and setuptools_scm[toml]>=6.2.
    Added dev optional dependencies section to include pytest, flake8, coverage, and other development tools. Refined project metadata such as version, authors, and dependencies. Improved package data and exclusion configuration.
  2. setup.py: Added extras_require section to define dev dependencies.
    Ensured compatibility with the changes in pyproject.toml.

Procedure:
CI Configuration:
The ci.yml file has been updated to use the dev dependencies during the CI pipeline runs. This ensures that all necessary development tools are installed for testing and code quality checks.
Development Setup:
To set up the development environment locally, run:
pip install .[dev]
This will install the package along with all development dependencies specified in the pyproject.toml and setup.py files.
Impact:
These changes streamline the setup process for new developers and CI environments.
Ensures that the development and testing environments are consistent across different setups.

@hadijannat hadijannat self-assigned this Jul 21, 2024
@hadijannat hadijannat mentioned this pull request Jul 24, 2024
@s-heppner s-heppner changed the title Update pyproject toml Migrate from setup.py to pyproject.toml Aug 16, 2024
Copy link
Copy Markdown
Member

@s-heppner s-heppner left a comment

Choose a reason for hiding this comment

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

As the Python ecosystem moves toward pyproject.toml, it's often recommended to fully migrate from setup.py to pyproject.toml.
As long as there is not concrete reason not to migrate fully, I would suggest to delete the setup.py.

Comment thread pyproject.toml Outdated
Comment thread pyproject.toml Outdated
Comment thread .github/workflows/ci.yml
Comment thread .github/workflows/ci.yml Outdated
Comment thread pyproject.toml Outdated
@s-heppner
Copy link
Copy Markdown
Member

Can you please point this PR to eclipse-basyx/main rather than rwth-iat/main?

@s-heppner
Copy link
Copy Markdown
Member

Can you also please update your branch, now that eclipse-basyx/basyx-python-sdk#238 has been merged?

@s-heppner
Copy link
Copy Markdown
Member

Do we still need this PR or can it be closed with respect to the new eclipse-basyx/basyx-python-sdk#290?

@Frosty2500
Copy link
Copy Markdown

I think this PR can be closed.

@Frosty2500 Frosty2500 closed this Oct 8, 2024
@s-heppner s-heppner deleted the update-pyproject-toml branch October 10, 2024 07:00
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