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

Skip to content

(snippet) check for input on standard in when deciding to create a snippet with lab snippet#102

Merged
zaquestion merged 2 commits intomasterfrom
snippet-root-stdin
Jan 17, 2018
Merged

(snippet) check for input on standard in when deciding to create a snippet with lab snippet#102
zaquestion merged 2 commits intomasterfrom
snippet-root-stdin

Conversation

@zaquestion
Copy link
Owner

This is better than just checking if its a global snippet since it works for project and personal snippets. It also forces snippets created from scratch (editor workflow) to explicitly use lab snippet create

…ippet with `lab snippet`

This is better than just checking if its a global snippet since it works for project and personal snippets. It also forces snippets created from scratch (editor workflow) to explicitly use `lab snippet create`
@codecov
Copy link

codecov bot commented Jan 17, 2018

Codecov Report

Merging #102 into master will increase coverage by 0.45%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #102      +/-   ##
==========================================
+ Coverage    62.6%   63.05%   +0.45%     
==========================================
  Files          32       32              
  Lines        1230     1229       -1     
==========================================
+ Hits          770      775       +5     
+ Misses        301      296       -5     
+ Partials      159      158       -1
Impacted Files Coverage Δ
internal/git/edit.go 78.78% <ø> (-0.32%) ⬇️
cmd/snippet.go 86.95% <100%> (+17.39%) ⬆️
cmd/snippetCreate.go 65.26% <100%> (+2.1%) ⬆️

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 e275c7a...509f107. Read the comment docs.

@zaquestion zaquestion merged commit c77b3bd into master Jan 17, 2018
@zaquestion zaquestion added this to the 0.9.2 milestone Jan 18, 2018
@zaquestion zaquestion deleted the snippet-root-stdin branch January 19, 2018 10:53
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.

1 participant

Comments