Skip to content

Conversation

mndeveci
Copy link
Contributor

#1861

Description of changes:
Update documentation related to MSK event source object

Checklist:

  • Write/update tests
  • make pr passes
  • Update documentation
  • Verify transformed template deploys and application functions as expected

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@mndeveci mndeveci linked an issue Dec 15, 2020 that may be closed by this pull request
@codecov-io
Copy link

codecov-io commented Dec 16, 2020

Codecov Report

Merging #1862 (97021d8) into master (101fa43) will increase coverage by 0.05%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1862      +/-   ##
==========================================
+ Coverage   93.88%   93.93%   +0.05%     
==========================================
  Files          88       88              
  Lines        5739     5739              
  Branches     1166     1166              
==========================================
+ Hits         5388     5391       +3     
+ Misses        162      160       -2     
+ Partials      189      188       -1     
Impacted Files Coverage Δ
samtranslator/translator/logical_id_generator.py 100.00% <0.00%> (+9.09%) ⬆️

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 101fa43...97021d8. Read the comment docs.

@mndeveci mndeveci merged commit c36fdf6 into master Dec 17, 2020
mndeveci added a commit to mndeveci/serverless-application-model that referenced this pull request Dec 17, 2020
mgrandis pushed a commit to mgrandis/serverless-application-model that referenced this pull request Mar 2, 2021
@jfuss jfuss deleted the mndeveci-patch-3 branch October 8, 2021 20:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Typo in serverless-application-model documentation?
3 participants