Skip to content

Revert split CI changes#1639

Merged
tsegismont merged 2 commits intoeclipse-vertx:masterfrom
tsegismont:revert-split-ci
Mar 24, 2026
Merged

Revert split CI changes#1639
tsegismont merged 2 commits intoeclipse-vertx:masterfrom
tsegismont:revert-split-ci

Conversation

@tsegismont
Copy link
Member

See vert-x3/issues#665

Here's some feedback from the experiment in vertx-sql-client.

  • splitting works as expected; it's important to name the scheduled worfklow differently for clarity in the GH Actions view
  • since the scheduled worfklow is a separate file, you must browse to the all workflows view to get an aggregated view, but you can't select both the scheduled and push event in filters
  • as a consequence of the previous point, you can't get a build badge that is green if the latest scheduled workflow or the latest push worfklow is green, otherwise red

Because of these GitHub limitations, it's not possible to solve the problem (low traffic projects workflow deactived) without creating another (impossible to get a correct build badge).

So, this PR reverts changes made in Vert.x SQL Clients

@tsegismont tsegismont merged commit 72ed6a1 into eclipse-vertx:master Mar 24, 2026
31 of 35 checks passed
@tsegismont tsegismont deleted the revert-split-ci branch March 24, 2026 15:47
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.

1 participant