Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Tink CLI Template create should use --file instead of -p #446

Merged
merged 1 commit into from
Mar 3, 2021

Conversation

gianarb
Copy link
Contributor

@gianarb gianarb commented Mar 3, 2021

A few weeks ago we decided to use --file for template creation:

#385

But a few days ago probably during a rebase, I broke that
behavior and left the template to create command in a broken state

76793ee#diff-f5b9b4afa42ee036d5b6836337e9d6273de8c341267f7ea75eddc561f0ad6441

As detected by Dan #445

This commit should fix the issue, and it will require a documentation
update as soon as we release a new version of the sandbox.

Fixed #445

@gianarb gianarb requested a review from thebsdbox March 3, 2021 12:21
Few weeks ago we decided to use --file for template creation:

#385

But a few days days ago probably during a rebase I broke that
behavior and left the template create command in a broken state

76793ee#diff-f5b9b4afa42ee036d5b6836337e9d6273de8c341267f7ea75eddc561f0ad6441

As detected by Dan #445

This commit should fix the issue, and it will require a documentation
update as soon as we release a new version of sandbox.

Signed-off-by: Gianluca Arbezzano <[email protected]>
@gianarb gianarb force-pushed the fix/template_create_is_broken branch from 3601f74 to f08f711 Compare March 3, 2021 12:21
@codecov
Copy link

codecov bot commented Mar 3, 2021

Codecov Report

Merging #446 (ad1c7a5) into master (b4aa3b5) will not change coverage.
The diff coverage is 0.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #446   +/-   ##
=======================================
  Coverage   35.14%   35.14%           
=======================================
  Files          47       47           
  Lines        2888     2888           
=======================================
  Hits         1015     1015           
  Misses       1781     1781           
  Partials       92       92           
Impacted Files Coverage Δ
cmd/tink-cli/cmd/template/create.go 0.00% <0.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 b4aa3b5...f08f711. Read the comment docs.

Copy link
Contributor

@thebsdbox thebsdbox left a comment

Choose a reason for hiding this comment

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

Looks fine to me.

@gianarb gianarb added the ready-to-merge Signal to Mergify to merge the PR. label Mar 3, 2021
@mergify mergify bot merged commit db0edc3 into master Mar 3, 2021
@mergify mergify bot deleted the fix/template_create_is_broken branch March 3, 2021 13:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-to-merge Signal to Mergify to merge the PR.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Template create function appears broken
2 participants