Skip to content

Conversation

crazy-max
Copy link
Contributor

Signed-off-by: CrazyMax <crazy-max@users.noreply.github.com>
@milosgajdos milosgajdos requested a review from thaJeztah July 20, 2022 07:44
Copy link
Member

@thaJeztah thaJeztah left a comment

Choose a reason for hiding this comment

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

LGTM

@crazy-max do you know if there will be a v1.1.2 with my fix for macOS? tonistiigi/xx@v1.1.1...master

@crazy-max
Copy link
Contributor Author

LGTM

@crazy-max do you know if there will be a v1.1.2 with my fix for macOS? tonistiigi/xx@v1.1.1...master

Ah good point, forgot to trigger a release for it. Will do!

@milosgajdos
Copy link
Member

@crazy-max should I wait for the new release instead of merging this?

@crazy-max
Copy link
Contributor Author

@milosgajdos No good to go, it's unrelated with this repo.

@thaJeztah
Copy link
Member

Ah good point, forgot to trigger a release for it. Will do!

thanks! didn't want to push too hard for it, as it was fixing a bit of a corner-case, but thought I'd check 😅

@milosgajdos milosgajdos merged commit 8794122 into distribution:main Jul 20, 2022
@crazy-max crazy-max deleted the update-xx branch July 20, 2022 11:02
@crazy-max
Copy link
Contributor Author

@thaJeztah Btw #3644 LGTY or does it need some changes?

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