Skip to content

Update core version to 3.2.0#9

Merged
vmatyus merged 1 commit intomasterfrom
188_core_update
Sep 22, 2020
Merged

Update core version to 3.2.0#9
vmatyus merged 1 commit intomasterfrom
188_core_update

Conversation

@vmatyus
Copy link
Copy Markdown
Contributor

@vmatyus vmatyus commented Sep 22, 2020

PR summary

Note: An existing issue is required before opening a PR.

PR Checklist

Please make sure that your PR fulfills the following requirements:

  • The commit message follows the
    Angular Commit Message Guidelines.
  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)

PR Type

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • New tests
  • Build/CI related changes
  • Documentation content changes
  • Other (please describe)

What is the current behavior?

What is the new behavior?

  • resolve path params by name, not by parameter ordering
  • IAM Authenticator: add support for optional 'scope' property

Does this PR introduce a breaking change?

  • Yes
  • No

Other information

@codecov
Copy link
Copy Markdown

codecov Bot commented Sep 22, 2020

Codecov Report

Merging #9 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master       #9   +/-   ##
=======================================
  Coverage   94.61%   94.61%           
=======================================
  Files           8        8           
  Lines        6373     6373           
=======================================
  Hits         6030     6030           
  Misses        343      343           

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 a0f6e43...9f5d705. Read the comment docs.

@vmatyus vmatyus self-assigned this Sep 22, 2020
@vmatyus
Copy link
Copy Markdown
Contributor Author

vmatyus commented Sep 22, 2020

Thank you for the review @ricellis and @bessbd ! I am about to merge this PR!

@vmatyus vmatyus merged commit 53b6b83 into master Sep 22, 2020
@vmatyus vmatyus deleted the 188_core_update branch September 22, 2020 11:43
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.

3 participants