Skip to content

Conversation

@Levev
Copy link
Collaborator

@Levev Levev commented Nov 7, 2025

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.

chore: Revert runtime-based container names
Copy link
Collaborator

@waffles-dev waffles-dev left a 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

@Levev Levev mentioned this pull request Nov 8, 2025
Levev and others added 2 commits November 8, 2025 21:39
chore: Add color rule for Creatd container state

chore: Remove type assertion and @ts-expect-error using Reflect.set
@KillerBOSS2019
Copy link
Contributor

4c02960#r170109134

PR #413 forgot about adding cmd: arg back to launch command. This is required for launching UWP app.

multiMonitor: 0,
rdpArgs: [],
disableAnimations: false,
containerRuntime: ContainerRuntimes.DOCKER, // TODO: Ideally should be podman once we flesh out everything
Copy link
Collaborator

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

Suggested change
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) => {
Copy link
Collaborator

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) => {
Copy link
Collaborator

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"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants