Skip to content

Conversation

@csc-felipe
Copy link
Contributor

@csc-felipe csc-felipe commented Feb 22, 2023

Description

Using pyproject.toml seems to be the recommended way for building and installing python projects.

Nothing else needs to change, the project still installs as usual with pip.

The build system is now hatch, as it was one of the recommended tools, and is an official project. It has quite many features, but we don't even need to install it.

Related issues

Type of change

  • New feature (non-breaking change which adds functionality)

Changes Made

  • Migrate from setup.py to pyproject.toml
  • Added the frontend dist folder, otherwise the build tries to resolve it as a local directory, and fails to install when it doesn't exist
  • Update cryptography, as dependency review flagged the current version as vulnerable

Testing

  • Tests do not apply

Mentions

I was wondering if we should remove the requirements file, as it is not necessarily needed. Any opinions?

@csc-felipe csc-felipe self-assigned this Feb 22, 2023
@csc-felipe csc-felipe marked this pull request as draft February 22, 2023 08:59
@csc-felipe csc-felipe marked this pull request as ready for review February 22, 2023 09:38
Copy link
Contributor

@blankdots blankdots left a comment

Choose a reason for hiding this comment

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

i think this could can be merged, as it does not affect that much the current development

@csc-felipe csc-felipe merged commit 2bbb616 into devel Feb 27, 2023
@csc-felipe csc-felipe deleted the feature/pyproject branch February 27, 2023 09:04
blankdots pushed a commit that referenced this pull request Jun 9, 2023
UploadNotification closed by user

Closes #1035 and #984

See merge request sds-dev/sd-connect/swift-browser-ui!59
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