-
Notifications
You must be signed in to change notification settings - Fork 177
Fix docker-app save
on single-file application
#352
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Minor typo
internal/packager/registry.go
Outdated
df := filepath.Join(app.Path, "__Dockerfile-docker-app__") | ||
dir, err := ioutil.TempDir("", app.Name) | ||
if err != nil { | ||
return "", errors.Wrap(err, "cannot create temporary directoryr") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
directoryr -> directory
a9f780e
to
2bd75af
Compare
types/types.go
Outdated
@@ -39,6 +39,19 @@ func (a *App) Metadata() []byte { | |||
return a.metadataContent | |||
} | |||
|
|||
func (a *App) Extract(path string) error { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lint error, missing method doc.
types/types.go
Outdated
@@ -39,6 +39,19 @@ func (a *App) Metadata() []byte { | |||
return a.metadataContent | |||
} | |||
|
|||
func (a *App) Extract(path string) error { | |||
if err := ioutil.WriteFile(filepath.Join(path, internal.MetadataFileName), a.Metadata(), 0755); err != nil { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: mode should rather be 0644, no need for exec bit
fe4e1c6
to
0dac7a6
Compare
0dac7a6
to
f96d6c8
Compare
Signed-off-by: Vincent Demeester <vincent@sbr.pm>
f96d6c8
to
a8a9779
Compare
Codecov Report
@@ Coverage Diff @@
## master #352 +/- ##
==========================================
- Coverage 58.6% 58.42% -0.18%
==========================================
Files 56 56
Lines 2802 2819 +17
==========================================
+ Hits 1642 1647 +5
- Misses 939 945 +6
- Partials 221 227 +6
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Fixes #351
0.4.1
release 👼Signed-off-by: Vincent Demeester vincent@sbr.pm