-
Notifications
You must be signed in to change notification settings - Fork 157
Debug without having to package or install #287
Debug without having to package or install #287
Conversation
} | ||
|
||
// tslint:disable-next-line no-empty | ||
public dispose(): void { | ||
} | ||
|
||
private getNewVersion(): string { | ||
const date: Date = new Date(); | ||
return `${this.PREFIX}-${date.getHours()}${date.getMinutes()}${date.getDate()}${date.getMonth()}${date.getFullYear()}`; |
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.
This should be YYYYMMDDHHMM so it can be easily sorted in chronological order 😉
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.
Also, for consistency suggest you ensure the number of digits (12 in total) for each field.
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.
See comments regarding timestamps
Automatically packages and installs a smart contract Also excluded images from the package contributes to #232 Signed-off-by: Caroline Church <caroline.church@uk.ibm.com>
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.
LGTM 👍
Signed-off-by: Caroline Church <caroline.church@uk.ibm.com>
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.
LGTM 👍
Automatically packages and installs a smart contract
Also excluded images from the package
contributes to #232
Signed-off-by: Caroline Church caroline.church@uk.ibm.com