Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@SietsmaRJ
Copy link
Contributor

SOP

Changed

  • Updated the capice project to use pyproject.toml instead of setup.py which will become deprecated.

Important notes

  • Poetry is chosen because of the default integration with include, which is something that setuptools can not function with yet. This means that non-python project files will not be included in the final build (either wheel or sdist).
  • For the reviewer: please check carefully if all non-python project files (within src) are included in a wheel / sdist after building.

Before merge:

  • Functionality works & meets specs
  • No Travis issues
  • Code reviewed
  • Documentation was updated

After merge:

  • Added feature/fix to draft release notes
  • Removed merged branches

- Changed setup of CAPICE from setup.py to pyproject.toml, including configuration for different tools
Need a better way to patch the CAPICE version now
# Conflicts:
#	setup.py
#	src/molgenis/capice/__init__.py
- Updated the capice project to use pyproject.toml instead of setup.py which will become deprecated.
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

83.3% 83.3% Coverage
0.0% 0.0% Duplication

@svandenhoek svandenhoek marked this pull request as draft March 16, 2023 13:35
"Molgenis <[email protected]>"
]
readme = "README.md"
license = "LICENSE"
Copy link
Contributor

Choose a reason for hiding this comment

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

]
build-backend = "poetry.core.masonry.api"

[tool.poetry]
Copy link
Contributor

Choose a reason for hiding this comment

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

[build-system]
requires = [
"wheel",
"poetry-core"
Copy link
Contributor

Choose a reason for hiding this comment

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

Not sure if new releases pushed to Pypi through Travis works well with Poetry by default with the deploy step. We gotta make sure we don't break this (f.e. due to a manifest file not being interpreted in case of capice-resources repo).

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