Skip to content

Conversation

@ClausKlein
Copy link
Collaborator

@ClausKlein ClausKlein commented Jul 9, 2025

I will not longer fight against windmills!

@coveralls
Copy link

coveralls commented Jul 9, 2025

Coverage Status

coverage: 92.578% (+0.4%) from 92.179%
when pulling fe6ab19 on ClausKlein:feature/finish-my-contribution
into 73880e9 on bemanproject:main.

@ClausKlein
Copy link
Collaborator Author

ClausKlein commented Jul 9, 2025

@ednolan how is it possible the CI testcoverage changed without any code modification?

@dietmarkuehl random wait conditon affects the code coverage!

@ednolan
Copy link
Member

ednolan commented Jul 9, 2025

I've disabled the setting that causes coverage decreases to cause CI failures. Coveralls will still leave comments on pull requests but will not fail the build. Unfortunately I think you need to resync the branch for this setting to apply.

@ClausKlein
Copy link
Collaborator Author

Unfortunately I think you need to resync the branch for this setting to apply.

Yes, no idea how?

@ednolan
Copy link
Member

ednolan commented Jul 9, 2025

Just push an additional commit of any kind

@ClausKlein
Copy link
Collaborator Author

@ednolan now, it takes hours to build on ci?

@ednolan
Copy link
Member

ednolan commented Jul 9, 2025

Not hours, but about a half an hour. It looks like the issue is that we're running into the concurrency limits of GitHub. If you check each individual job, they take less than 3 minutes to run, but because of the concurrency limits, many of them are running serially instead of in parallel.

@ClausKlein
Copy link
Collaborator Author

ClausKlein commented Jul 9, 2025

Not hours, but about a half an hour. It looks like the issue is that we're running into the concurrency limits of GitHub. If you check each individual job, they take less than 3 minutes to run, but because of the concurrency limits, many of them are running serially instead of in parallel.

3 pushes in 10 min. -> more than an hour to wait ...

Who pays for the time I am waiting for a review and than the rebase with push and again ....?

@dietmarkuehl dietmarkuehl merged commit d4236a1 into bemanproject:main Jul 10, 2025
41 checks passed
@ClausKlein ClausKlein deleted the feature/finish-my-contribution branch July 10, 2025 06:12
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