Skip to content

Commit

Permalink
Add a resolve to ArtifactPath
Browse files Browse the repository at this point in the history
When using artifact path for template configuration, the CDK would get upset.
This just adds a resolve that falls back to that location.

Fixes aws#570
  • Loading branch information
mindstorms6 committed Aug 15, 2018
1 parent 389e9bc commit b9321c6
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 1 deletion.
4 changes: 4 additions & 0 deletions packages/@aws-cdk/aws-codepipeline-api/lib/artifact.ts
Original file line number Diff line number Diff line change
Expand Up @@ -87,4 +87,8 @@ export class ArtifactPath {
get location() {
return `${this.artifact.name}::${this.fileName}`;
}

public resolve() {
return this.location;
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -63,6 +63,7 @@ export = {
changeSetName,
role: changeSetExecRole,
templatePath: new ArtifactPath(buildAction.artifact!, 'template.yaml'),
templateConfiguration: new ArtifactPath(buildAction.artifact!, 'templateConfig.json')
});

new ExecuteChangeSet(stack, 'ExecuteChangeSetProd', {
Expand Down Expand Up @@ -160,7 +161,8 @@ export = {
]
},
"StackName": "BrelandsStack",
"TemplatePath": "OutputYo::template.yaml"
"TemplatePath": "OutputYo::template.yaml",
"TemplateConfiguration": "OutputYo::templateConfig.json"
},
"InputArtifacts": [{"Name": "OutputYo"}],
"Name": "BuildChangeSetProd",
Expand Down

0 comments on commit b9321c6

Please sign in to comment.