-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Buildpacks support Auto sync #4079
Buildpacks support Auto sync #4079
Conversation
d9a5bbe
to
f1c1ac8
Compare
Codecov Report
|
@@ -219,7 +219,7 @@ func setDefaultSync(a *latest.Artifact) { | |||
if a.Sync != nil { | |||
if len(a.Sync.Manual) == 0 && len(a.Sync.Infer) == 0 && a.Sync.Auto == nil { | |||
switch { | |||
case a.JibArtifact != nil: | |||
case a.JibArtifact != nil || a.BuildpackArtifact != 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.
Do you think we should check that the builder is our builder? Or warn (in case someone builds a builder that includes our devmode buildpacks)?
@@ -661,7 +661,7 @@ func TestNewSyncItem(t *testing.T) { | |||
}, | |||
}, | |||
evt: filemon.Events{ | |||
Added: []string{"file.go", "ignored.txt"}, |
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.
The devmode.sync in this example only identifies *.go
files, so this shouldn't change with ignored.txt
, should it? Oh, I think you just wanted to test this separately.
Please visit http://34.94.185.45:1313 to view changes to the docs. |
Signed-off-by: David Gageot <david@gageot.net>
6b8407f
to
0076526
Compare
Signed-off-by: David Gageot david@gageot.net