Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

repair deps #7431

Merged
merged 3 commits into from
Nov 23, 2024
Merged

repair deps #7431

merged 3 commits into from
Nov 23, 2024

Conversation

KTibow
Copy link
Contributor

@KTibow KTibow commented Nov 19, 2024

Description

python dependencies were really weird: dev dependencies were duplicated, and the first one had a definition of click that was invalid syntax. this pr merges them together.

Double check

  • I have tested my parser changes locally with poetry run test_parser "zone_key"
  • I have run pnpx prettier@2 --write . and poetry run format in the top level directory to format my changes.

@github-actions github-actions bot added the dependencies Pull requests that update a dependency file label Nov 19, 2024
@VIKTORVAV99 VIKTORVAV99 self-requested a review November 19, 2024 08:54
Copy link
Member

@VIKTORVAV99 VIKTORVAV99 left a comment

Choose a reason for hiding this comment

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

LGTM! 🎉

Thanks for cleaning this up!

@VIKTORVAV99 VIKTORVAV99 enabled auto-merge (squash) November 23, 2024 10:45
@VIKTORVAV99 VIKTORVAV99 merged commit a7dfc90 into electricitymaps:master Nov 23, 2024
21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants