Skip to content

Conversation

@opportunity356
Copy link
Contributor

@opportunity356 opportunity356 commented Dec 13, 2024

  1. Removed setup.py due to build configuration in pyproject.toml. Command python setup.py install will not work. Use these alternatives to install package from source: https://packaging.python.org/en/latest/discussions/setup-py-deprecated/#what-commands-should-be-used-instead)
  2. Enchancements to CI and dev tools:
    • added pre-commit hooks
    • removed requirements*.txt files (use dependency-groups from pyproject.toml)
    • tox.ini configuration was moved to pyproject.toml
    • using uv run in Makefile
    • added uv.lock file
    • refactored workflow with checks and tests triggered by pull_request

@opportunity356 opportunity356 force-pushed the better-pyproject branch 22 times, most recently from 94cd53c to 03efac5 Compare December 13, 2024 22:44
@opportunity356 opportunity356 force-pushed the better-pyproject branch 7 times, most recently from be6b47c to 5f9440d Compare December 14, 2024 09:43
@opportunity356 opportunity356 force-pushed the better-pyproject branch 5 times, most recently from 1706256 to 7205e4b Compare December 14, 2024 10:01
Copy link
Contributor Author

Choose a reason for hiding this comment

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

autogenerated, no need to review

"isort>=5.13.2",
]
genproto = [
"grpcio-tools==1.59.3",
Copy link
Contributor Author

@opportunity356 opportunity356 Dec 16, 2024

Choose a reason for hiding this comment

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

Strictly pinned versions for some packages used in lint and format checks, because newer versions produce new errors. Will relax these restrictions in next PR, where also will fix linting and formating errors

pyproject.toml Outdated

[project]
name = "yandexcloud"
version = "0.328.0"
Copy link
Contributor

Choose a reason for hiding this comment

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

а теперь эту версию руками менеджерить надо?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

если правильно настроил semversion, то нет, он сам будет обновлять как раньше

pyproject.toml Outdated
version = "0.328.0"
authors = [{name = "Yandex LLC", email = "[email protected]"}]
license = {text = "MIT"}
description = "The Yandex.Cloud official SDK"
Copy link
Contributor

Choose a reason for hiding this comment

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

а почему мы Yandex.Cloud пишем в python sdk всегда через точку?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Исторически так было. Поправил

@opportunity356 opportunity356 merged commit 3c7c5e4 into master Dec 16, 2024
7 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.

3 participants