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

chore: fix typos, docstrings and type hints #154

Merged
merged 1 commit into from
Sep 2, 2020

Conversation

michaelbrewer
Copy link
Contributor

Issue #, if available:

Description of changes:

Changes:

  • Fix some of the type hints
  • Add messing docstrings for parameters
  • Fix typos

Checklist

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

Changes:
* Fix some of the type hints
* Add messing docstrings for parameters
* Fix typos
@codecov-commenter
Copy link

codecov-commenter commented Sep 1, 2020

Codecov Report

Merging #154 into develop will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##           develop      #154   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           28        28           
  Lines          793       793           
  Branches        72        72           
=========================================
  Hits           793       793           
Impacted Files Coverage Δ
aws_lambda_powertools/metrics/base.py 100.00% <100.00%> (ø)
aws_lambda_powertools/utilities/batch/sqs.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 b28c47e...ef4c3ed. Read the comment docs.

@to-mc to-mc self-requested a review September 2, 2020 07:11
@to-mc to-mc merged commit 1951574 into aws-powertools:develop Sep 2, 2020
@heitorlessa
Copy link
Contributor

Thank you @michaelbrewer for improving the API docs and addressing those typos too

@heitorlessa heitorlessa added the documentation Improvements or additions to documentation label Sep 2, 2020
@michaelbrewer michaelbrewer deleted the chore-housekeeping branch September 2, 2020 13:42
heitorlessa referenced this pull request in heitorlessa/aws-lambda-powertools-python Sep 11, 2020
* develop: (57 commits)
  chore: bump version to 1.5.0 (#158)
  chore(batch): Housekeeping for recent changes (#157)
  docs: address readability feedbacks
  chore: add debug logging for sqs batch processing
  chore: remove middlewares module, moving decorator functionality to base and sqs
  docs: add detail to batch processing
  fix: throw exception by default if messages processing fails
  chore: add test for sqs_batch_processor interface
  fix: add sqs_batch_processor as its own method
  docs: simplify documentation more SQS specific focus Update for sqs_batch_processor interface
  chore: add sqs_batch_processor decorator to simplify interface
  feat(parameters): transform = "auto" (#133)
  chore: fix typos, docstrings and type hints (#154)
  chore: tiny changes for readability
  fix: ensure debug log event has latest ctx
  docs: rephrase the wording to make it more clear
  docs: refactor example; improve docs about creating your own processor
  refactor: remove references to BaseProcessor. Left BasePartialProcessor
  docs: add newly created Slack Channel
  fix: update image with correct sample
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants