Skip to content

Downgrade goterm to 1.0.1#9080

Merged
ndeloof merged 1 commit into
docker:v2from
ulyssessouza:downgrade-goterm-1.0.1
Jan 4, 2022
Merged

Downgrade goterm to 1.0.1#9080
ndeloof merged 1 commit into
docker:v2from
ulyssessouza:downgrade-goterm-1.0.1

Conversation

@ulyssessouza

Copy link
Copy Markdown
Contributor

What I did

This is to avoid a release error:
https://github.com/docker/compose/runs/4701103662?check_suite_focus=true

This is the last version before the introduction of the line triggering the error

This is to avoid a release error:
https://github.com/docker/compose/runs/4701103662?check_suite_focus=true

Signed-off-by: Ulysses Souza <ulyssessouza@gmail.com>
@ulyssessouza
ulyssessouza requested a review from ndeloof January 4, 2022 12:13
@thaJeztah

Copy link
Copy Markdown
Member

This related to buger/goterm#39 / buger/goterm#41 ?

@ulyssessouza

Copy link
Copy Markdown
Contributor Author

@thaJeztah Yes. I can transform that into a bump to 1.0.4 if it comes in time.

@thaJeztah thaJeztah 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.

LGTM

@thaJeztah

Copy link
Copy Markdown
Member

(I pinged the maintainer on that PR 🤞 hopefully we'll be able to bump to v1.0.4 soon then)

Curious; do you know why cross build is disabled in CI? I guess that could've caught this problem?

@ulyssessouza

Copy link
Copy Markdown
Contributor Author

@thaJeztah The cross build is called in the release

@ndeloof

ndeloof commented Jan 4, 2022

Copy link
Copy Markdown
Contributor

IIRC it was disabled as it had significant impact on build time for PRs, but we could use parallel GHA jobs

@ndeloof
ndeloof merged commit 6dc6bed into docker:v2 Jan 4, 2022
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