-
Notifications
You must be signed in to change notification settings - Fork 13
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
Add containerapps-albumapi-javascript
test project & cases
#696
Conversation
mslearn-js
test project/casesmslearn-js
test project & cases
mslearn-js
test project & casescontainerapps-albumapi-javascript
test project & cases
…ode-azurecontainerapps into mwf/albumapi-js-test
…ode-azurecontainerapps into mwf/albumapi-js-test
…inerapps into mwf/nightly-test-prep
…vscode-azurecontainerapps into mwf/add-monorepo-test-project
…ode-azurecontainerapps into mwf/albumapi-js-test
@@ -0,0 +1,10 @@ | |||
FROM node:lts-alpine |
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.
Have you verified that making this Dockerfile invalid breaks the tests?
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.
Thinking about this more, it might be good to explore making a test case with a broken Dockerfile on purpose so we have a consistent and tested experience for that.
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.
Great suggestion, I'm working on an update to address this here!
#708
16504ba
to
8fc527b
Compare
Overall test todos outlined here: #697
Closes #699