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

Update pytest on destinations #44524

Closed
natikgadzhi opened this issue Aug 21, 2024 · 0 comments · Fixed by #44530
Closed

Update pytest on destinations #44524

natikgadzhi opened this issue Aug 21, 2024 · 0 comments · Fixed by #44530

Comments

@natikgadzhi
Copy link
Contributor

natikgadzhi commented Aug 21, 2024

A lot of destinations are stuck on pytest 6.2, which is causing a bunch of high severity security alerts on github.

Fixing them should be easy:

  1. Search and replace: within ./airbyte-integrations/connectors/destinations-*/pyproject.toml find pytest = 6\.2\..* (pseudocode regex) and replace with ^8.3.1 or whatever the recent one is.
  2. Bump the version of the affected connectors. The easiest way to do it is to run airbyte-ci connectors --modified bump-version patch "Update test dependencies".
  3. Then, make a pull request with them. A single PR with all of them is probably okay.

This is not very complex work, but it gives us nice security cleanup.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

3 participants