Skip to content

Fixes #38927 - Drop async-ssh#128

Merged
ofedoren merged 1 commit into
theforeman:masterfrom
adamruzicka:no-async-ssh
Nov 28, 2025
Merged

Fixes #38927 - Drop async-ssh#128
ofedoren merged 1 commit into
theforeman:masterfrom
adamruzicka:no-async-ssh

Conversation

@adamruzicka

Copy link
Copy Markdown
Contributor

No description provided.

@ofedoren ofedoren left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems like it's ready, is there something missing to undraft it?

Also, given that there doesn't seem to be an easy way to migrate to a different mode, this will probably require a big red release note.

@adamruzicka adamruzicka marked this pull request as ready for review November 27, 2025 14:19
@adamruzicka

Copy link
Copy Markdown
Contributor Author

is there something missing to undraft it?

Not anymore, I wanted to get the whole set of PRs ready first

@adamruzicka

Copy link
Copy Markdown
Contributor Author

this will probably require a big red release note.

And also a big version bump, maybe even up to 1.0.0

@ofedoren ofedoren left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah, I guess a RM issue linked would be nice to have though. Otherwise ACK.

@adamruzicka adamruzicka changed the title Drop async-ssh Fixes #38927 - Drop async-ssh Nov 27, 2025
@adamruzicka

Copy link
Copy Markdown
Contributor Author

🍏

@ofedoren ofedoren left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, @adamruzicka, let's get this in.

@ofedoren ofedoren merged commit d8d1811 into theforeman:master Nov 28, 2025
11 checks passed
@adamruzicka adamruzicka deleted the no-async-ssh branch November 28, 2025 11:58
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.

2 participants