Skip to content

OSGi Repoviewer: sync features with bndtools#1758

Merged
laeubi merged 1 commit intoeclipse-pde:masterfrom
chrisrueger:repoviewer-sync-features-with-bndtools
May 6, 2025
Merged

OSGi Repoviewer: sync features with bndtools#1758
laeubi merged 1 commit intoeclipse-pde:masterfrom
chrisrueger:repoviewer-sync-features-with-bndtools

Conversation

@chrisrueger
Copy link
Contributor

@chrisrueger chrisrueger commented May 6, 2025

  • adds features which were added in bndtools RepositoriesView after initial migration to PDE, e.g.
    • fix memory leak in RepositoryTreeContentProvider
    • right click menu for "Showing Bundles with substitution packages (self-imports)"
    • remember expansion state when clearing filter
    • now also shows the bnd WorkspaceRepository (screenshot) -> also icon is now fixed
image

@chrisrueger chrisrueger force-pushed the repoviewer-sync-features-with-bndtools branch 2 times, most recently from 7d9177a to 13b94b7 Compare May 6, 2025 08:39
@chrisrueger
Copy link
Contributor Author

I am just fighting with JGit cleaning up and creating other PRs. Hold on for a few minutes.

@chrisrueger chrisrueger force-pushed the repoviewer-sync-features-with-bndtools branch 3 times, most recently from 0f25e75 to 74eba17 Compare May 6, 2025 10:44
@chrisrueger chrisrueger marked this pull request as ready for review May 6, 2025 10:45
@chrisrueger chrisrueger force-pushed the repoviewer-sync-features-with-bndtools branch 3 times, most recently from b9cec36 to 258769c Compare May 6, 2025 10:51
@chrisrueger
Copy link
Contributor Author

chrisrueger commented May 6, 2025

@laeubi I think this one is ready. Icon is also fixed
image

@github-actions
Copy link

github-actions bot commented May 6, 2025

Test Results

   285 files  ±0     285 suites  ±0   51m 46s ⏱️ - 3m 42s
 3 611 tests ±0   3 535 ✅ ±0   76 💤 ±0  0 ❌ ±0 
11 025 runs  ±0  10 794 ✅ ±0  231 💤 ±0  0 ❌ ±0 

Results for commit 7e84701. ± Comparison against base commit c00ecd3.

♻️ This comment has been updated with latest results.

@laeubi
Copy link
Contributor

laeubi commented May 6, 2025

- add features added in bndtools RepositoriesView since pde migration

Repo View: Show workspace-repo if bnd workspace

- if there is a cnf project (which is the marker for a bnd workspace)
then we now show the WorkspaceRepository which contains all projects

better detect BND workspace

add icon for workspacerepo

fix compiler warnings
@chrisrueger chrisrueger force-pushed the repoviewer-sync-features-with-bndtools branch from 258769c to 7e84701 Compare May 6, 2025 12:20
@chrisrueger
Copy link
Contributor Author

@chrisrueger can you look at the compiler warnings? https://github.com/eclipse-pde/eclipse.pde/pull/1758/checks?check_run_id=41716536640

It was 3 unused imports. Sorry for that. I am so used to Save-actions -> Organize Import, that I didn't check that. But it was disabled in this new Eclipse instance.

@laeubi laeubi merged commit 957cb89 into eclipse-pde:master May 6, 2025
19 checks passed
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.

2 participants