Skip to content
This repository has been archived by the owner on Dec 20, 2024. It is now read-only.

Release 14/11/24 #135

Merged
merged 8 commits into from
Nov 15, 2024
Merged

Release 14/11/24 #135

merged 8 commits into from
Nov 15, 2024

Conversation

bram-vdberg
Copy link
Collaborator

  • Update SQLAlchemy version from <2.0 to >2.0&<3.0
  • Update setup.py to install SQL files

harisang and others added 6 commits October 30, 2024 17:50
This PR cleans up some leftover references to participation rewards
This PR removes the changes introduced by #122, #117, #115, and #116 and
removes Prefect from the dune-sync repo.
This PR adds a setup.py file so that the dune-sync repo can be installed
with pip using `pip install
git+https://github.com/cowprotocol/dune-sync.git@<branch-name>`. This
can be tested with: `pip install
git+https://github.com/cowprotocol/dune-sync.git@setup_pip`.
This PR updates the sqlalchemy version of the dune-sync repo to <3.0.
This is needed for compatibility with Prefect. It also updates the
setup.py to include the .sql files.
@bram-vdberg bram-vdberg requested a review from harisang November 14, 2024 10:53
Copy link

socket-security bot commented Nov 14, 2024

New and removed dependencies detected. Learn more about Socket for GitHub ↗︎

Package New capabilities Transitives Size Publisher
pypi/[email protected] environment, eval, filesystem, network, shell, unsafe +160 1.73 GB CaselIT

🚮 Removed packages: pypi/[email protected]

View full report↗︎

Rebasing the dev branch agains main accidentally introduced formatting
changes to the setup.py that causes the tests to fail. This PR fixes
this.
@bram-vdberg bram-vdberg marked this pull request as ready for review November 14, 2024 11:02
@bram-vdberg
Copy link
Collaborator Author

The tests failed the first time due to a timeout error. Re-running the tests seems to have fixed it.

manifest.in Outdated Show resolved Hide resolved
Copy link
Contributor

@harisang harisang left a comment

Choose a reason for hiding this comment

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

added small comment

Co-authored-by: Haris Angelidakis <[email protected]>
@bram-vdberg bram-vdberg merged commit 978fabd into main Nov 15, 2024
6 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants