Skip to content

fix: support str type from faas handlers#39

Merged
jyecusch merged 1 commit into
mainfrom
feature/str-response
Sep 1, 2021
Merged

fix: support str type from faas handlers#39
jyecusch merged 1 commit into
mainfrom
feature/str-response

Conversation

@jyecusch
Copy link
Copy Markdown
Member

@jyecusch jyecusch commented Sep 1, 2021

No description provided.

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Sep 1, 2021

Codecov Report

Merging #39 (0346117) into main (439c13e) will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##             main      #39   +/-   ##
=======================================
  Coverage   95.91%   95.91%           
=======================================
  Files          16       16           
  Lines         710      710           
=======================================
  Hits          681      681           
  Misses         29       29           
Impacted Files Coverage Δ
nitric/faas/faas.py 85.96% <100.00%> (ø)

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 439c13e...0346117. Read the comment docs.

@jyecusch jyecusch force-pushed the feature/str-response branch from e36a936 to 0346117 Compare September 1, 2021 05:18
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Sep 1, 2021

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@jyecusch jyecusch merged commit 72b8d91 into main Sep 1, 2021
@jyecusch jyecusch deleted the feature/str-response branch April 16, 2025 08:18
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