Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Packages/bundles that are already imported/added are shown #1195

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

lathapatil
Copy link
Contributor

Packages/bundles that are already imported/added are shown followed by [already imported/added]

Fixes #146

@gireeshpunathil
Copy link
Contributor

@lathapatil - did change set from #1194 carry over into this PR?

@lathapatil
Copy link
Contributor Author

@lathapatil - did change set from #1194 carry over into this PR?

No, I am trying to figure out what went wrong. Looks like #1194 has not reached here along with its branch !

Copy link

github-actions bot commented Mar 13, 2024

Test Results

   291 files  ±0     291 suites  ±0   1h 0m 16s ⏱️ + 3m 39s
 3 526 tests ±0   3 468 ✅ ±0   58 💤 ±0  0 ❌ ±0 
10 875 runs  ±0  10 698 ✅ ±0  177 💤 ±0  0 ❌ ±0 

Results for commit 21e5af6. ± Comparison against base commit a2762f9.

♻️ This comment has been updated with latest results.

@gireeshpunathil
Copy link
Contributor

gireeshpunathil commented Mar 13, 2024

it is probably because you branched out for this PR from the previous branch, so the tip of the branch got carried over.

nothing to worry: I will help you fix that. if you are using command line then do this:

  • git rebase -I HEAD~2
  • this opens up the commit editor in vim editor
  • you will see 2 entries for the two commits.
  • remove the line that has 214071f in the commit hash
  • save and close
  • git push -f <origin branch> < this pr branch>

@lathapatil lathapatil force-pushed the Issues/146_Show_imported_exported_required_bundles branch from 6d51a60 to f0da0ed Compare March 13, 2024 12:56
@lathapatil
Copy link
Contributor Author

it is probably because you branched out for this PR from the previous branch, so the tip of the branch got carried over.

Yes. Thanks for the support .
I am using EGit and working in windows OS for PDE . I was able to fix this issue using EGit interactive rebase.

@lathapatil
Copy link
Contributor Author

@laeubi Could you please test the PR if it meets the expectations and possibly review the code ?

@laeubi
Copy link
Contributor

laeubi commented Mar 20, 2024

@lathapatil do I need to consider something special?

I fetched this PR then started an embedded workbench and tried the following:
grafik

but package aQute.bnd.build is not shown.

I also think it would be good to have a checkbox (enabled by default) "Show already imported packages", at best the user choice is persisted to survive a restart.

@lathapatil
Copy link
Contributor Author

lathapatil commented Mar 20, 2024

I fetched this PR then started an embedded workbench and tried the following:

After checking out the pull request (PR) while in PDE workspace , switch to the PR branch and launch the Eclipse application. Then, create a plugin project and inspect the manifest file for imported packages, required bundles, and exported packages.

This is what I have attempted, and the changes are visible.

What do you mean by embedded workbench ?

@lathapatil
Copy link
Contributor Author

I also think it would be good to have a checkbox (enabled by default) "Show already imported packages"

This is already part of the PR. But I observed there is some delay in loading all packages

@lathapatil
Copy link
Contributor Author

Please find attached video for working of the feature from the branch https://github.com/lathapatil/eclipse.pde/tree/Issues/146_Show_imported_exported_required_bundles

2024-04-09_15h43_11.mp4

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.

Show of packages / bundles that are already imported.
3 participants