Skip to content

Conversation

tom139
Copy link
Contributor

@tom139 tom139 commented Jan 13, 2020

The YAML with authorizers example was malformed.

Issue #, if available: #1378

Description of changes:

I splitted the example YAML in the API Auth Object section between the main Auth: section and the Authorizers: subsection.

Description of how you validated changes:

Manually checked it with a markdown viewer

Checklist:

  • Write/update tests
  • make pr passes
  • Update documentation
  • Verify transformed template deploys and application functions as expected
  • Add/update example to examples/2016-10-31

All above are irrelevant

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

formatting.
YAML with `authorizers` example was malformed (opened twice).
@codecov-io
Copy link

Codecov Report

Merging #1379 into develop will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff            @@
##           develop    #1379   +/-   ##
========================================
  Coverage    94.46%   94.46%           
========================================
  Files           78       78           
  Lines         4555     4555           
  Branches       912      912           
========================================
  Hits          4303     4303           
  Misses         118      118           
  Partials       134      134

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 eabe27d...a75ffd0. Read the comment docs.

Copy link
Contributor

@praneetap praneetap left a comment

Choose a reason for hiding this comment

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

Thanks for your contribution!

@praneetap praneetap assigned keetonian and unassigned praneetap Jan 20, 2020
@keetonian keetonian merged commit dc01997 into aws:develop Jan 23, 2020
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.

4 participants