-
Notifications
You must be signed in to change notification settings - Fork 286
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
Pass mountPaths.appDirName to lifecycle as -app arg #1146
Conversation
Signed-off-by: Joe Kutner <[email protected]>
Signed-off-by: Joe Kutner <[email protected]>
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.
I should've thought of this when I merged in the other PR. Good catch!
@@ -193,7 +193,7 @@ func (l *LifecycleExecution) Create(ctx context.Context, publish bool, dockerHos | |||
|
|||
opts := []PhaseConfigProviderOperation{ | |||
WithFlags(l.withLogLevel(flags...)...), | |||
WithArgs(repoName), | |||
WithArgs("-app", l.mountPaths.appDirName(), repoName), |
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.
Does lifecycle expect it in the format \appdir
, or in c:\appdir
? If the later, we can use mountPaths.appDir()
instead
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.
oh, i might need to revert 51887a3 in favor of using mountPaths.appDir()
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.
fixed in 9cc6613
Signed-off-by: Joe Kutner <[email protected]>
Codecov Report
@@ Coverage Diff @@
## main #1146 +/- ##
=======================================
Coverage 80.80% 80.80%
=======================================
Files 136 136
Lines 8299 8299
=======================================
Hits 6705 6705
Misses 1164 1164
Partials 430 430
Flags with carried forward coverage won't be shown. Click here to find out more. |
Closing this in favor of #1149 |
Summary
Fast follow fix for #1141
Output
Before
app dir was mounted, but lifecycle was not
cd
-ing into it.After
The provide workspace becomes the
CNB_APP_DIR
Documentation