Skip to content

Clean old files after migration to repo-config#97

Merged
llucax merged 3 commits intofrequenz-floss:v0.x.xfrom
llucax:clean-old-files
Aug 9, 2023
Merged

Clean old files after migration to repo-config#97
llucax merged 3 commits intofrequenz-floss:v0.x.xfrom
llucax:clean-old-files

Conversation

@llucax
Copy link
Copy Markdown
Contributor

@llucax llucax commented Aug 9, 2023

These files are not needed anymore.

  • Remove minimum-requirements.txt
  • Remove tox.ini
  • Remove setup.py

llucax added 3 commits August 9, 2023 12:15
@llucax
Remove minimum-requirements.txt
d00fece 
Now `nox` will get the minimum dependencies versions automatically from
`pyproject.toml`.

Signed-off-by: Leandro Lucarella <luca-frequenz@llucax.com>
@llucax
Remove tox.ini
e3d9dab 
Now we are using `nox` instead of `tox`.

Signed-off-by: Leandro Lucarella <luca-frequenz@llucax.com>
@llucax
Remove setup.py
3114bff 
Now the compilation of proto files is done by a setuptools extension in
`repo-config`, so we don't need to do it manually anymore.

Signed-off-by: Leandro Lucarella <luca-frequenz@llucax.com>
@llucax llucax requested a review from a team as a code owner August 9, 2023 10:17
@llucax llucax requested a review from TalweSingh August 9, 2023 10:17
@llucax llucax self-assigned this Aug 9, 2023
@github-actions github-actions Bot added part:tooling Affects the development tooling (CI, deployment, dependency management, etc.) part:python Affects the Python bindings labels Aug 9, 2023
@llucax llucax added this to the v0.16.0 milestone Aug 9, 2023
@llucax llucax added the type:tech-debt Improves the project without visible changes for users label Aug 9, 2023
@llucax llucax enabled auto-merge August 9, 2023 10:18
@llucax llucax mentioned this pull request Aug 9, 2023
Closed
@llucax llucax added this pull request to the merge queue Aug 9, 2023
Merged via the queue into frequenz-floss:v0.x.x with commit 69a2089 Aug 9, 2023
@llucax llucax deleted the clean-old-files branch August 9, 2023 10:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@tiyash-basu-frequenz tiyash-basu-frequenz tiyash-basu-frequenz approved these changes

@TalweSingh TalweSingh Awaiting requested review from TalweSingh TalweSingh was automatically assigned from frequenz-floss/api-team

Assignees

@llucax llucax

Labels

part:python Affects the Python bindings part:tooling Affects the development tooling (CI, deployment, dependency management, etc.) type:tech-debt Improves the project without visible changes for users

Projects

None yet

Milestone

v0.16.0

Development

Successfully merging this pull request may close these issues.

2 participants

@llucax @tiyash-basu-frequenz