-
-
Notifications
You must be signed in to change notification settings - Fork 384
feat: Fix issues before release 0.9.0 #463
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
base: main
Are you sure you want to change the base?
Conversation
chore: Revert runtime-based container names
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.
the rename changes LGTM - tested that my old container is now working again
Co-authored-by: waffles-dev <[email protected]>
chore: Add color rule for Creatd container state chore: Remove type assertion and @ts-expect-error using Reflect.set
|
PR #413 forgot about adding |
| multiMonitor: 0, | ||
| rdpArgs: [], | ||
| disableAnimations: false, | ||
| containerRuntime: ContainerRuntimes.DOCKER, // TODO: Ideally should be podman once we flesh out everything |
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.
Put comment on line above
| containerRuntime: ContainerRuntimes.DOCKER, // TODO: Ideally should be podman once we flesh out everything | |
| // TODO: Ideally should be podman once we flesh out everything | |
| containerRuntime: ContainerRuntimes.DOCKER, |
| logger.info("Performing migrations for 0.9.0"); | ||
|
|
||
| // Compose migration | ||
| this.containerMgr!.exists().then(async (value) => { |
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.
getting warnings for async within constructor
| logger.info("Performing migrations for 0.9.0"); | ||
|
|
||
| // Compose migration | ||
| this.containerMgr!.exists().then(async (value) => { |
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.
In regards to the !:
"This assertion is unnecessary since it does not change the type of the expression"
This PR aims to fix some issues in the current HEAD of WinBoat before the next release.
I'll be going through and testing migrations, features, etc. and pushing the fixes here.