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
feat(lib): make asset paths change if asset content changes #769
Conversation
a6c5aa6
to
92a7855
Compare
@@ -3,10 +3,10 @@ | |||
exports[`full integration test deploy in Terraform Cloud 1`] = ` | |||
"Deploying Stack: hello-terra | |||
Resources | |||
✔ NULL_RESOURCE test null_resource.test | |||
~ NULL_RESOURCE test null_resource.test |
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.
Is this right?
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.
No, I hoped that it was fixed with the JSON parsing
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'll try a rebase, unsure if it has been lately
cf896a6
to
6585e3f
Compare
0b02972
to
21d32ad
Compare
4269fca
to
174f26a
Compare
14979ba
to
b3830ea
Compare
b3830ea
to
1bb9fca
Compare
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.
Are there plans to deal with stale assets as well?
@skorfmann I added this now, I don't think it's necessary right now as we clean the entire directory in the beginning, but now we can be sure we don't break anything once we change this behavior |
this will be necessary once we stop removing the entire out folder on synth
fa4959c
to
a9a3bc2
Compare
I'm going to lock this pull request because it has been closed for 30 days. This helps our maintainers find and focus on the active issues. If you've found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further. |
Closes #762