-
Notifications
You must be signed in to change notification settings - Fork 3.8k
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
Destination SFTP JSON: Migrate to base image + poetry #40760
base: master
Are you sure you want to change the base?
Conversation
The latest updates on your projects. Learn more about Vercel for Git βοΈ 1 Skipped Deployment
|
@@ -1,118 +1,91 @@ | |||
# Sftp Json Destination | |||
# Sftp-Json source connector |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hmmmm, I think you'd have to check out the old readme and manually change it. I think it also could be that we messed up other connectors like that too.
python main.py spec | ||
python main.py check --config secrets/config.json | ||
python main.py discover --config secrets/config.json | ||
python main.py read --config secrets/config.json --catalog integration_tests/configured_catalog.json |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lol, both the old and the new one are wrong.
destination-sftp-json = "destination_sftp_json.run:run" | ||
|
||
[tool.poetry.group.dev.dependencies] | ||
pytest = "^6.1" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
pytest = "^6.1" | |
pytest = "^8" |
|
||
[tool.poetry.group.dev.dependencies] | ||
pytest = "^6.1" | ||
docker = "==5.0.3" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The new docker library is 7+, but there are cross-dependencies. It could be that it's better to migrate first and upgrade separately, this is rather high usage.
What
Migrates destination SFTP JSON to our base image and poetry
User Impact
No impact
Can this PR be safely reverted and rolled back?