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

WIP: v3.0.0 #14

Merged
merged 45 commits into from
Jun 4, 2020
Merged

WIP: v3.0.0 #14

merged 45 commits into from
Jun 4, 2020

Conversation

charlieparkes
Copy link
Collaborator

No description provided.

charlieparkes and others added 30 commits May 15, 2020 15:38
@charlieparkes
Add project_urls to setup()
ace5eee
@charlieparkes
Update README.md
f68b629
@charlieparkes
Update README.md
ddd4f43
@charlieparkes
Run unit and integration tests at the same time to fix codecov
077e9ff
@charlieparkes
Split return value handling into separate function return_handler()
8101a64
@charlieparkes
Fix parameters required by return_handler to split that code apart
23581f5
@charlieparkes
Split pipeline into smaller functions. Move interfaces with thirdpart…
c9c08a1 
…ies to contrib/. Move dict tools to contrib/mindictive
@charlieparkes
Make utils.repr() render a string which is more likely to be usable t…
009d9da 
…o create the same object again
@charlieparkes
Document include_all_params argument for Action class
23a8f40
@charlieparkes
Fix import of utils.repr in lpipe.logging
94c9499
@charlieparkes
Add type hints everywhere
70c17ff
@charlieparkes
Fix tabs -> spaces
3431204
@charlieparkes
Merge pull request #4 from mintel/reduce_complexity
9b4f5a0 
Reduce complexity, add docs, move integrations to contrib
@charlieparkes
Move boto3 proxy to contrib (#6)
582fd9f 
* Move _boto3 module to contrib.boto3

* Fix import for emit_logs in testing.awslambda

* Change travisci tests command to just 'make test'

* Fix imports to various utils in testing.awslambda
@charlieparkes
Remove default import and usage of sentry and allow user to pass any …
5a8c8a4 
…exception handler function in it's place
@charlieparkes
Don't require sentry in lpipe or test it in main tests
7453b6b
@charlieparkes
Add sentry-sdk to dev dependencies
91c96ca
@awichmann-mintel
fix sqs.delete_message_batch (#8)
a7278cd
@charlieparkes
Merge pull request #7 from mintel/use_any_exception_handler
0617ec3 
Use any exception handler
@charlieparkes
Don't require sentry in lpipe or test it in main tests
66503e1
@charlieparkes
Disable integration tests since they aren't checking anything useful …
2b7f423 
…to lpipe. Testing the test fixtures themselves will move to a different library.
@charlieparkes
Improve test coverage of contrib.boto3
783c7d7
@charlieparkes
Improve coverage of pipeline and signature modules
f4d57d4
@charlieparkes
Move major classes to their own modules, import all major classes at …
2a3addb 
…top level module
@charlieparkes
fmt
0e1b73d
@charlieparkes
Update lpipe.__init__ docstring to match github repo description
4fb75b2
@charlieparkes
fmt
b514bff
@charlieparkes
lock env
7e9e0c7
@charlieparkes
Fix linting errors
ca2cb6b
@charlieparkes
Merge pull request #9 from mintel/raise_coverage
eacf316 
Raise coverage
@codecov
Copy link

codecov bot commented May 21, 2020 •
edited
Loading

Codecov Report

Merging #14 into master will increase coverage by 44.32%.
The diff coverage is 91.94%.

Impacted file tree graph

@@             Coverage Diff             @@
##           master      #14       +/-   ##
===========================================
+ Coverage   46.84%   91.16%   +44.32%     
===========================================
  Files          14       21        +7     
  Lines         807      826       +19     
===========================================
+ Hits          378      753      +375     
+ Misses        429       73      -356
Impacted Files Coverage Δ
lpipe/testing/awslambda.py 29.72% <50.00%> (-59.16%) ⬇️
lpipe/contrib/sentry.py 95.12% <60.00%> (ø)
lpipe/pipeline.py 86.60% <83.56%> (+70.93%) ⬆️
lpipe/logging.py 97.50% <85.71%> (+62.71%) ⬆️
lpipe/contrib/sqs.py 97.77% <90.00%> (ø)
lpipe/normalize.py 92.30% <92.30%> (ø)
lpipe/action.py 100.00% <100.00%> (ø)
lpipe/contrib/boto3.py 80.76% <100.00%> (ø)
lpipe/contrib/kinesis.py 96.77% <100.00%> (ø)
lpipe/contrib/mindictive.py 100.00% <100.00%> (ø)
... and 22 more

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 acc2c18...6ba464c. Read the comment docs.

@charlieparkes charlieparkes merged commit 6ba464c into master Jun 4, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
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
@charlieparkes @awichmann-mintel