Skip to content

[scala/http4s] Marked as broken#10693

Closed
joanhey wants to merge 1 commit intoTechEmpower:masterfrom
joanhey:http4s-broken
Closed

[scala/http4s] Marked as broken#10693
joanhey wants to merge 1 commit intoTechEmpower:masterfrom
joanhey:http4s-broken

Conversation

@joanhey
Copy link
Contributor

@joanhey joanhey commented Feb 6, 2026

Fail in the last runs.

@cptwunderlich @alphaho @gcnyin @limansky @AL333Z @RafalSumislawski ... Please fix.

@limansky
Copy link
Contributor

limansky commented Feb 6, 2026

Could you please provide more details? Are there any logs?

@joanhey
Copy link
Contributor Author

joanhey commented Feb 6, 2026

Yes, but I'm not a guardian.
Anybody need to be responsible of their frameworks !!

Log:
https://tfb-status.techempower.com/unzip/results.2026-01-26-17-36-11-536.zip/results/20260118211352/http4s/build/http4s.log

@joanhey
Copy link
Contributor Author

joanhey commented Feb 6, 2026

Really when a framework fail for more than 4 times consecutive, need to be marked here automatically.

PD: also in each run need to be informed when fail more than 4 times (had problems starting or stopping) and the last time that was working OK (which run and the github commit hash).

@RafalSumislawski
Copy link
Contributor

Fixing here: #10703

@joanhey
Copy link
Contributor Author

joanhey commented Feb 7, 2026

Fixed with #10703

@joanhey joanhey closed this Feb 7, 2026
@joanhey
Copy link
Contributor Author

joanhey commented Feb 11, 2026

Pull again for not use the time in the runs !!

@joanhey joanhey reopened this Feb 11, 2026
@joanhey
Copy link
Contributor Author

joanhey commented Feb 11, 2026

@msmith-techempower

@msmith-techempower
Copy link
Member

I'm confused what I'm being flagged for - is this ready for review?

@joanhey
Copy link
Contributor Author

joanhey commented Feb 11, 2026

Me to confussed, but #10703

Not merged, and we need time for the runs !!

@joanhey
Copy link
Contributor Author

joanhey commented Feb 11, 2026

I never will close a PR again, if the same dev fix it, first the problem.
When the dev fix it, open the PR and change it.

@msmith-techempower
Copy link
Member

Resolved in #10703

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.

4 participants