Skip to content

fix(deps): Bump dependencies and fix some of the dev tooling #354

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

Merged
merged 12 commits into from
Mar 22, 2021

Conversation

michaelbrewer
Copy link
Contributor

@michaelbrewer michaelbrewer commented Mar 21, 2021

Issue #, if available:

Description of changes:

Changes:

  • Manually update to latest dev dependencies
  • Run poetry update
  • Run make pr which reformatted some code
  • tests(lazy-loader): Add missing __dir__ test
  • Configure pytest via toml
  • chore: Fix some typos and typing

Manual update:

  • black 20.8b1 - Currently reformats some existing code, but overall better
  • isort 5.8.0
  • flake8-isort 4.0.0
  • mkdocs-material 7.0.6
  • pytest 6.2.2 - Move pytest config to toml
  • pytest-cov - Add --cov=aws_lambda_powertools

Checklist

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Michael Brewer added 3 commits March 20, 2021 22:47
@codecov-io
Copy link

codecov-io commented Mar 21, 2021

Codecov Report

Merging #354 (5122a07) into develop (961b25e) will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff            @@
##           develop     #354   +/-   ##
========================================
  Coverage    99.88%   99.88%           
========================================
  Files           95       95           
  Lines         3529     3529           
  Branches       173      173           
========================================
  Hits          3525     3525           
  Misses           2        2           
  Partials         2        2           
Impacted Files Coverage Δ
aws_lambda_powertools/shared/functions.py 100.00% <ø> (ø)
.../utilities/data_classes/api_gateway_proxy_event.py 100.00% <ø> (ø)
.../utilities/data_classes/cognito_user_pool_event.py 100.00% <ø> (ø)
..._powertools/utilities/parser/envelopes/dynamodb.py 100.00% <ø> (ø)
...ambda_powertools/utilities/parser/envelopes/sns.py 100.00% <ø> (ø)
...ools/utilities/idempotency/persistence/dynamodb.py 100.00% <100.00%> (ø)
...ambda_powertools/utilities/parameters/appconfig.py 100.00% <100.00%> (ø)
aws_lambda_powertools/utilities/parameters/base.py 100.00% <100.00%> (ø)
aws_lambda_powertools/utilities/parameters/ssm.py 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 961b25e...5122a07. Read the comment docs.

@michaelbrewer michaelbrewer changed the title fix(deps): Bump dependencies fix(deps): Bump dependencies and fix some of the dev tooling Mar 21, 2021
@heitorlessa heitorlessa added the internal Maintenance changes label Mar 22, 2021
@heitorlessa
Copy link
Contributor

Awesome @michaelbrewer - Like we discussed in the office hours, could you run a docs build (API and MKDocs) including the docker image to test it?

I don't foresee any issue but want to be triple sure before I merge it. Also happy to do it myself later this week

@michaelbrewer
Copy link
Contributor Author

Awesome @michaelbrewer - Like we discussed in the office hours, could you run a docs build (API and MKDocs) including the docker image to test it?

I don't foresee any issue but want to be triple sure before I merge it. Also happy to do it myself later this week

@heitorlessa no obvious issues found so far. Note, I do see warnings for the API Docs build process, but they are the same as before :)

@heitorlessa heitorlessa merged commit 8c20b7a into aws-powertools:develop Mar 22, 2021
@michaelbrewer michaelbrewer deleted the fix-dep branch March 22, 2021 22:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
internal Maintenance changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants