Skip to content
This repository has been archived by the owner on Oct 17, 2023. It is now read-only.

init command #279

Merged
merged 1 commit into from
Feb 27, 2017
Merged

init command #279

merged 1 commit into from
Feb 27, 2017

Conversation

jipperinbham
Copy link
Contributor

@jipperinbham jipperinbham commented Feb 24, 2017

  • CHANGELOG.md updated
  • README.md updated

fixes #254

@codecov
Copy link

codecov bot commented Feb 24, 2017

Codecov Report

Merging #279 into master will increase coverage by 0.23%.
The diff coverage is 50%.

@@            Coverage Diff             @@
##           master     #279      +/-   ##
==========================================
+ Coverage   71.95%   72.19%   +0.23%     
==========================================
  Files          33       33              
  Lines        2236     2273      +37     
==========================================
+ Hits         1609     1641      +32     
- Misses        486      487       +1     
- Partials      141      145       +4
Impacted Files Coverage Δ
pkg/adaptor/mongodb/mongodb.go 76.84% <ø> (ø)
pkg/adaptor/file/file.go 85.18% <ø> (ø)
pkg/adaptor/rethinkdb/rethinkdb.go 80.82% <ø> (ø)
pkg/adaptor/postgres/postgres.go 67.52% <0%> (ø)
pkg/adaptor/elasticsearch/elasticsearch.go 86.72% <60%> (-1.67%)
pkg/state/filestore.go 52.83% <0%> (+3.77%)
pkg/transporter/pipeline.go 66.14% <0%> (+7.89%)

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 e30886a...66f785a. Read the comment docs.

Copy link
Contributor

@trinchan trinchan left a comment

Choose a reason for hiding this comment

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

LGTM. The configs are basic enough to not warrant using template files, but if they ever are, we can revisit

return err
}
defer appFileHandle.Close()
appFileHandle.WriteString("Source({name:\"source\", namespace:\"test./.*/\"}).save({name:\"sink\", namespace:\"test./.*/\"})\n")
Copy link
Contributor

Choose a reason for hiding this comment

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

Escaped quotes could be avoided using raw strings with backticks (`)

init [source] [sink] will create a stubbed out YAML and JS file with the defaults provided from calling SampleConfig of the adaptor

fixes #254
@jipperinbham jipperinbham merged commit b13ef7c into master Feb 27, 2017
@jipperinbham jipperinbham deleted the 254-init-command branch February 27, 2017 19:18
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.

Proposal: transporter init
2 participants