Skip to content

Add counter to update classpath message#2318

Open
fedejeanne wants to merge 1 commit into
eclipse-pde:masterfrom
fedejeanne:codex/add_plugin_count_update_classpath_job
Open

Add counter to update classpath message#2318
fedejeanne wants to merge 1 commit into
eclipse-pde:masterfrom
fedejeanne:codex/add_plugin_count_update_classpath_job

Conversation

@fedejeanne
Copy link
Copy Markdown
Contributor

When updating the classpath of the plugins, the message now includes additional information about how many out of how many plugins have been processed (e.g. 10/321), useful for large workspaces.

Screenshots

Here's the progress monitor for 2 plugins: plugin1 and plugin2

image
image
image

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 28, 2026

Test Results

  126 files  ±0    126 suites  ±0   35m 48s ⏱️ - 4m 5s
3 502 tests ±0  3 379 ✅  - 69   54 💤 ±0  69 ❌ +69 
9 321 runs  ±0  9 117 ✅  - 74  130 💤 ±0  74 ❌ +74 

For more details on these failures, see this check.

Results for commit 7535c85. ± Comparison against base commit 604557a.

♻️ This comment has been updated with latest results.

@vogella
Copy link
Copy Markdown
Contributor

vogella commented Apr 28, 2026

LGTM

@fedejeanne
Copy link
Copy Markdown
Contributor Author

@HannesWell ?

Copy link
Copy Markdown
Member

@HannesWell HannesWell left a comment

Choose a reason for hiding this comment

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

I really like the idea, but I think the exact label is not yet ideal.

@fedejeanne fedejeanne force-pushed the codex/add_plugin_count_update_classpath_job branch from 116c3c1 to 174bf0d Compare May 11, 2026 07:49
When updating the classpath of the plugins, the message now includes
additional information about how many out of how many plugins have been
processed (e.g. 10/321), useful for large workspaces.
@fedejeanne fedejeanne force-pushed the codex/add_plugin_count_update_classpath_job branch from 174bf0d to 7535c85 Compare May 11, 2026 07:51
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