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.

Sign up for GitHub

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

Jump to bottom

Dependency update #1086

Merged
merged 24 commits into from
Sep 23, 2022
Merged

Dependency update #1086

merged 24 commits into from
Sep 23, 2022

Conversation

rashidakanchwala
Copy link
Contributor

@rashidakanchwala rashidakanchwala commented Sep 16, 2022 •
edited
Loading

Description

Development notes

Most dependencies didn't break anything. A few did -- hence below are the changes made

  • on circleCI, we have pinned the python 3.10 to python 3.10.6 because it's failing with mypy. A fix will be released soon and then we will revert it back to python 3.10 (Positional-only parameters with python 3.10.7 (and numpy) (fixed in 0.981) python/mypy#13627)
  • pylint & flake8 had some new linting requirements such as
    • any sort of 'with open' file should have an encoding set (multiple places)
    • there should be a timeout specified for responses (jupyter.py - line 76)
  • fastAPI now requires us to return None if there's no value in dataclass properties. So we are doing that (flowchart.py), Also we must return all properties specified in responses.

QA notes

Checklist

  • Read the contributing guidelines
  • Opened this PR as a 'Draft Pull Request' if it is work-in-progress
  • Updated the documentation to reflect the code changes
  • Added new entries to the RELEASE.md file
  • Added tests to cover my changes

dependabot bot and others added 21 commits September 16, 2022 12:23
@dependabot
Update pytest requirement from ~=6.2.0 to ~=7.1.3 in /package (#1067)
a938492 
Updates the requirements on [pytest](https://github.com/pytest-dev/pytest) to permit the latest version.
- [Release notes](https://github.com/pytest-dev/pytest/releases)
- [Changelog](https://github.com/pytest-dev/pytest/blob/main/CHANGELOG.rst)
- [Commits](pytest-dev/pytest@6.2.0...7.1.3)

---
updated-dependencies:
- dependency-name: pytest
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@dependabot
Bump aiofiles from 0.6.0 to 22.1.0 in /package (#1068)
3dd6bba 
Bumps [aiofiles](https://github.com/Tinche/aiofiles) from 0.6.0 to 22.1.0.
- [Release notes](https://github.com/Tinche/aiofiles/releases)
- [Commits](Tinche/aiofiles@v0.6.0...v22.1.0)

---
updated-dependencies:
- dependency-name: aiofiles
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@dependabot
Bump psutil from 5.8.0 to 5.9.2 in /package (#1072)
106307a 
Bumps [psutil](https://github.com/giampaolo/psutil) from 5.8.0 to 5.9.2.
- [Release notes](https://github.com/giampaolo/psutil/releases)
- [Changelog](https://github.com/giampaolo/psutil/blob/master/HISTORY.rst)
- [Commits](giampaolo/psutil@release-5.8.0...release-5.9.2)

---
updated-dependencies:
- dependency-name: psutil
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@dependabot
Update black requirement from ~=22.0 to ~=22.8 in /package (#1069)
3bc97c1 
Updates the requirements on [black](https://github.com/psf/black) to permit the latest version.
- [Release notes](https://github.com/psf/black/releases)
- [Changelog](https://github.com/psf/black/blob/main/CHANGES.md)
- [Commits](psf/black@22.1.0...22.8.0)

---
updated-dependencies:
- dependency-name: black
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@dependabot
Update isort requirement from ~=5.8.0 to ~=5.10.1 in /package (#1073)
8dff06a 
Updates the requirements on [isort](https://github.com/pycqa/isort) to permit the latest version.
- [Release notes](https://github.com/pycqa/isort/releases)
- [Changelog](https://github.com/PyCQA/isort/blob/main/CHANGELOG.md)
- [Commits](PyCQA/isort@5.8.0...5.10.1)

---
updated-dependencies:
- dependency-name: isort
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@dependabot
Update semver requirement from ~=2.10 to ~=2.13 in /package (#1074)
bb5f253 
Updates the requirements on [semver](https://github.com/python-semver/python-semver) to permit the latest version.
- [Release notes](https://github.com/python-semver/python-semver/releases)
- [Changelog](https://github.com/python-semver/python-semver/blob/master/docs/changelog-semver2.rst)
- [Commits](python-semver/python-semver@2.10.0...2.13.0)

---
updated-dependencies:
- dependency-name: semver
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@dependabot
Update bandit requirement from ~=1.6.2 to ~=1.7.4 in /package (#1075)
4381a36 
Updates the requirements on [bandit](https://github.com/PyCQA/bandit) to permit the latest version.
- [Release notes](https://github.com/PyCQA/bandit/releases)
- [Commits](PyCQA/bandit@1.6.2...1.7.4)

---
updated-dependencies:
- dependency-name: bandit
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@dependabot
Update pytest-mock requirement from ~=3.6.1 to ~=3.8.2 in /package (#…
6294b97 
…1076)

Updates the requirements on [pytest-mock](https://github.com/pytest-dev/pytest-mock) to permit the latest version.
- [Release notes](https://github.com/pytest-dev/pytest-mock/releases)
- [Changelog](https://github.com/pytest-dev/pytest-mock/blob/main/CHANGELOG.rst)
- [Commits](pytest-dev/pytest-mock@v3.6.1...v3.8.2)

---
updated-dependencies:
- dependency-name: pytest-mock
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@dependabot
Update mypy requirement from ~=0.930 to ~=0.971 in /package (#1078)
033042c 
Updates the requirements on [mypy](https://github.com/python/mypy) to permit the latest version.
- [Release notes](https://github.com/python/mypy/releases)
- [Commits](python/mypy@v0.930...v0.971)

---
updated-dependencies:
- dependency-name: mypy
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@dependabot
Update watchgod requirement from ~=0.8.0 to ~=0.8.2 in /package (#1082)
9a32d98 
Updates the requirements on [watchgod](https://github.com/samuelcolvin/watchfiles) to permit the latest version.
- [Release notes](https://github.com/samuelcolvin/watchfiles/releases)
- [Commits](samuelcolvin/watchfiles@v0.8...v0.8.2)

---
updated-dependencies:
- dependency-name: watchgod
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@dependabot
Update pytest-cov requirement from ~=2.11.1 to ~=3.0.0 in /package (#…
67741a2 
…1079)

Updates the requirements on [pytest-cov](https://github.com/pytest-dev/pytest-cov) to permit the latest version.
- [Release notes](https://github.com/pytest-dev/pytest-cov/releases)
- [Changelog](https://github.com/pytest-dev/pytest-cov/blob/master/CHANGELOG.rst)
- [Commits](pytest-dev/pytest-cov@v2.11.1...v3.0.0)

---
updated-dependencies:
- dependency-name: pytest-cov
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@dependabot @rashidakanchwala
Update uvicorn[standard] requirement in /package (#1083)
5bf1c33 
Updates the requirements on [uvicorn[standard]](https://github.com/encode/uvicorn) to permit the latest version.
- [Release notes](https://github.com/encode/uvicorn/releases)
- [Changelog](https://github.com/encode/uvicorn/blob/master/CHANGELOG.md)
- [Commits](encode/uvicorn@0.17.0...0.18.3)

---
updated-dependencies:
- dependency-name: uvicorn[standard]
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: rashidakanchwala <37628668+rashidakanchwala@users.noreply.github.com>
@dependabot
Update trufflehog requirement from ~=2.1.0 to ~=2.2.1 in /package (#1080
3398a51 
)

Updates the requirements on [trufflehog](https://github.com/dxa4481/truffleHog) to permit the latest version.
- [Release notes](https://github.com/dxa4481/truffleHog/releases)
- [Changelog](https://github.com/trufflesecurity/trufflehog/blob/main/.goreleaser.yml)
- [Commits](https://github.com/dxa4481/truffleHog/commits)

---
updated-dependencies:
- dependency-name: trufflehog
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@dependabot
Update pytest-asyncio requirement from ~=0.17.2 to ~=0.19.0 in /packa…
a6142be 
…ge (#1085)

Updates the requirements on [pytest-asyncio](https://github.com/pytest-dev/pytest-asyncio) to permit the latest version.
- [Release notes](https://github.com/pytest-dev/pytest-asyncio/releases)
- [Changelog](https://github.com/pytest-dev/pytest-asyncio/blob/master/CHANGELOG.rst)
- [Commits](pytest-dev/pytest-asyncio@v0.17.2...v0.19.0)

---
updated-dependencies:
- dependency-name: pytest-asyncio
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@dependabot
Update sqlalchemy requirement from ~=1.2 to ~=1.4 in /package (#1081)
01c0a80 
Updates the requirements on [sqlalchemy](https://github.com/sqlalchemy/sqlalchemy) to permit the latest version.
- [Release notes](https://github.com/sqlalchemy/sqlalchemy/releases)
- [Changelog](https://github.com/sqlalchemy/sqlalchemy/blob/main/CHANGES.rst)
- [Commits](https://github.com/sqlalchemy/sqlalchemy/commits)

---
updated-dependencies:
- dependency-name: sqlalchemy
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@dependabot
Update flake8 requirement from ~=3.9.2 to ~=5.0.4 in /package (#1066)
6847ea4 
 Update flake8, pylint and pin python version to 3.10.6 to work with mypy
@dependabot @rashidakanchwala
Update pylint requirement from ~=2.8.2 to ~=2.15.3 in /package (#1089)
143ee52 
Updates the requirements on [pylint](https://github.com/PyCQA/pylint) to permit the latest version.
- [Release notes](https://github.com/PyCQA/pylint/releases)
- [Commits](pylint-dev/pylint@v2.8.2...v2.15.3)

---
updated-dependencies:
- dependency-name: pylint
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: rashidakanchwala <37628668+rashidakanchwala@users.noreply.github.com>
@dependabot @rashidakanchwala
Update ipython requirement from <8.0,>=7.0.0 to >=7.0.0,<9.0 in /pack…
72868ee 
…age (#1077)

* Update ipython requirement from <8.0,>=7.0.0 to >=7.0.0,<9.0 in /package

Updates the requirements on [ipython](https://github.com/ipython/ipython) to permit the latest version.
- [Release notes](https://github.com/ipython/ipython/releases)
- [Commits](ipython/ipython@7.0.0...8.5.0)

---
updated-dependencies:
- dependency-name: ipython
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>

* fix ipython import

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Rashida Kanchwala <rashida_kanchwala@mckinsey.com>
@rashidakanchwala
Merge branch 'main' into dependency-update
d38e3cf
@dependabot @antonymilne
Update matplotlib requirement from ~=3.3.4 to ~=3.6.0 in /package (#1071
4ba125d 
)

* Update matplotlib requirement from ~=3.3.4 to ~=3.6.0 in /package

Updates the requirements on [matplotlib](https://github.com/matplotlib/matplotlib) to permit the latest version.
- [Release notes](https://github.com/matplotlib/matplotlib/releases)
- [Commits](matplotlib/matplotlib@v3.3.4...v3.6.0)

---
updated-dependencies:
- dependency-name: matplotlib
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>

* Update package/test_requirements.txt

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Antony Milne <49395058+AntonyMilneQB@users.noreply.github.com>
@dependabot @antonymilne @rashidakanchwala
Update fastapi requirement from <0.67.0,>=0.63.0 to >=0.63.0,<0.86.0 …
d7f3ead 
…in /package (#1070)

* Update fastapi requirement in /package

Updates the requirements on [fastapi](https://github.com/tiangolo/fastapi) to permit the latest version.
- [Release notes](https://github.com/tiangolo/fastapi/releases)
- [Commits](fastapi/fastapi@0.63.0...0.85.0)

---
updated-dependencies:
- dependency-name: fastapi
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>

* Update fastapi requirement in /package

Updates the requirements on [fastapi](https://github.com/tiangolo/fastapi) to permit the latest version.
- [Release notes](https://github.com/tiangolo/fastapi/releases)
- [Commits](fastapi/fastapi@0.63.0...0.85.0)

---
updated-dependencies:
- dependency-name: fastapi
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>

* Update package/requirements.txt

* removed ignore comment

* lint

* fix tests

* fix lint

* fix lint

* fix lint

* fix tests and lint

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Antony Milne <49395058+AntonyMilneQB@users.noreply.github.com>
Co-authored-by: Rashida Kanchwala <rashida_kanchwala@mckinsey.com>
@rashidakanchwala rashidakanchwala marked this pull request as ready for review September 22, 2022 12:20
@rashidakanchwala rashidakanchwala requested review from antonymilne and tynandebold and removed request for limdauto and tynandebold September 22, 2022 12:20
antonymilne
antonymilne approved these changes Sep 22, 2022
View reviewed changes
Copy link
Contributor

@antonymilne antonymilne left a comment

Choose a reason for hiding this comment

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

Looks great, thank you for all this and the clear write-up! ⭐

In the future I think we might like to consider other configuration options like assignees, reviewers and versioning-strategy. But happy to leave dependabot running for now, see how it works, and modify the configuration as we go along if other options would suit us better.

package/kedro_viz/launchers/jupyter.py
@@ -12,7 +12,7 @@

import IPython
import requests
from IPython.core.display import HTML, display
from IPython.display import HTML, display
Copy link
Contributor

Choose a reason for hiding this comment

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

Why did this change?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

new IPython version update - https://ipython.readthedocs.io/en/stable/api/generated/IPython.display.html

package/test_requirements.txt Outdated Show resolved Hide resolved
@rashidakanchwala @antonymilne
Update package/test_requirements.txt
2d546a5 
Co-authored-by: Antony Milne <49395058+AntonyMilneQB@users.noreply.github.com>
@rashidakanchwala rashidakanchwala merged commit edf45a4 into main Sep 23, 2022
@rashidakanchwala rashidakanchwala deleted the dependency-update branch September 23, 2022 07:39
@antonymilne antonymilne mentioned this pull request Sep 23, 2022
Closed
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@antonymilne antonymilne antonymilne approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@rashidakanchwala @antonymilne