Skip to content

format: format the invoker's Dockerfile#5037

Merged
rabbah merged 1 commit into
apache:masterfrom
ZinuoCai:master
Feb 1, 2021
Merged

format: format the invoker's Dockerfile#5037
rabbah merged 1 commit into
apache:masterfrom
ZinuoCai:master

Conversation

@ZinuoCai

@ZinuoCai ZinuoCai commented Dec 22, 2020

Copy link
Copy Markdown

format: format the invoker's Dockerfile

Description

Related issue and scope

  • I opened an issue to propose and discuss this change (#????)

My changes affect the following components

  • API
  • Controller
  • Message Bus (e.g., Kafka)
  • Loadbalancer
  • Invoker
  • Intrinsic actions (e.g., sequences, conductors)
  • Data stores (e.g., CouchDB)
  • Tests
  • Deployment
  • CLI
  • General tooling
  • Documentation

Types of changes

  • Bug fix (generally a non-breaking change which closes an issue).
  • Enhancement or new feature (adds new functionality).
  • Breaking change (a bug fix or enhancement which changes existing behavior).

Checklist:

  • I signed an Apache CLA.
  • I reviewed the style guides and followed the recommendations (Travis CI will check :).
  • I added tests to cover my changes.
  • My changes require further changes to the documentation.
  • I updated the documentation where necessary.

@codecov-io

codecov-io commented Dec 22, 2020 •
edited
Loading

Copy link
Copy Markdown

Codecov Report

Merging #5037 (01c4e79) into master (4babe39) will decrease coverage by 47.90%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##           master    #5037       +/-   ##
===========================================
- Coverage   83.81%   35.91%   -47.91%     
===========================================
  Files         202      202               
  Lines        9818     9818               
  Branches      416      416               
===========================================
- Hits         8229     3526     -4703     
- Misses       1589     6292     +4703
Impacted Files Coverage Δ
...a/org/apache/openwhisk/common/ConfigMapValue.scala 0.00% <0.00%> (-100.00%) ⬇️
.../apache/openwhisk/core/controller/Namespaces.scala 0.00% <0.00%> (-100.00%) ⬇️
...pache/openwhisk/core/controller/CorsSettings.scala 0.00% <0.00%> (-100.00%) ⬇️
...che/openwhisk/core/entitlement/RateThrottler.scala 0.00% <0.00%> (-100.00%) ⬇️
...he/openwhisk/core/entitlement/KindRestrictor.scala 0.00% <0.00%> (-100.00%) ⬇️
...penwhisk/core/database/cosmosdb/CosmosDBUtil.scala 0.00% <0.00%> (-100.00%) ⬇️
.../openwhisk/core/entitlement/ActionCollection.scala 0.00% <0.00%> (-100.00%) ⬇️
.../openwhisk/core/entitlement/LocalEntitlement.scala 0.00% <0.00%> (-100.00%) ⬇️
...nwhisk/core/database/cosmosdb/CosmosDBConfig.scala 0.00% <0.00%> (-100.00%) ⬇️
...openwhisk/core/entitlement/PackageCollection.scala 0.00% <0.00%> (-100.00%) ⬇️
... and 133 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 4babe39...01c4e79. Read the comment docs.

rabbah
rabbah approved these changes Dec 22, 2020
View reviewed changes

@rabbah rabbah left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks @ZinuoCai

@rabbah rabbah merged commit faae555 into apache:master Feb 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@rabbah rabbah rabbah approved these changes
@upgle upgle upgle approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@ZinuoCai @codecov-io @rabbah @upgle