-
Notifications
You must be signed in to change notification settings - Fork 4
Open
Description
| new File(this, "Default", { filePath: "LICENSE", content: existingLicense?.toString() ?? this.content }); |
This part here uses existing file contents first, which means if the file exists, then the contents will never be updated.
content: existingLicense?.toString() ?? this.content
Maybe this is desirable for licenses not to change, but I am not sure.
However, from the technical side, it does go against the spirit of the projen approach, where it always clobbers the contents.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels