Thanks to visit codestin.com
Credit goes to github.com

Skip to content
This repository was archived by the owner on Dec 9, 2024. It is now read-only.

Fixed alarm annotations of trigger can not be saved #168

Merged
merged 1 commit into from
May 21, 2019

Conversation

KeonHee
Copy link
Contributor

@KeonHee KeonHee commented May 16, 2019

#167

If the feed exists, I changed it to match the ow.trigger.create()'s parameter format. I have also added tests for this

@codecov
Copy link

codecov bot commented May 16, 2019

Codecov Report

Merging #168 into master will increase coverage by 0.24%.
The diff coverage is 87.5%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #168      +/-   ##
==========================================
+ Coverage    84.8%   85.04%   +0.24%     
==========================================
  Files          47       47              
  Lines        1829     1826       -3     
  Branches      378      378              
==========================================
+ Hits         1551     1553       +2     
+ Misses        278      273       -5
Impacted Files Coverage Δ
deploy/lib/deployTriggers.js 73.91% <87.5%> (+16.22%) ⬆️

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 84ca5ac...be5a3a7. Read the comment docs.

@style95
Copy link

style95 commented May 20, 2019

@jthomas Could you have a look at this?
It seems we are unable to create a trigger with feed using the latest serverless framework.
(More precisely, the feed annotation is not added in the trigger information.)

@jthomas
Copy link
Contributor

jthomas commented May 20, 2019

@style95 Let me have a look at this tomorrow.

@style95
Copy link

style95 commented May 21, 2019

@jthomas Thank you so much.

@jthomas jthomas merged commit 7fd9105 into serverless:master May 21, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants