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

Check getBundle returns non-null after PDE Manifest-editor opened. #1376

Conversation

peddaiahjuturu
Copy link
Contributor

Bug description

(Regression introduced by #949 and reported in #864)
After opening MANIFEST.MF file in PDE Manifest-editor and delete the project, ll get NPE.

@peddaiahjuturu
Copy link
Contributor Author

@HannesWell , @laeubi , @alshamams , i can`t add reviewers for this PR. can you guys review it.
Note: legal agreement check is failing with "Author Eclipse account cannot be found" issue.

@merks
Copy link
Contributor

merks commented Aug 14, 2024

Will will need to get an ECA in place:

https://github.com/eclipse-platform/.github/blob/main/CONTRIBUTING.md#setting-up-your-eclipse-and-github-account

Copy link

Test Results

   275 files   -  10     275 suites   - 10   44m 15s ⏱️ - 7m 11s
 3 580 tests ±  0   3 504 ✅ ±  0   76 💤 ± 0  0 ❌ ±0 
10 579 runs   - 353  10 401 ✅  - 300  178 💤  - 53  0 ❌ ±0 

Results for commit 2f9f695. ± Comparison against base commit 9b87c23.

@peddaiahjuturu
Copy link
Contributor Author

I`m closing this PR and create new PR with primary email id.

@peddaiahjuturu peddaiahjuturu deleted the bugs/1374_after_opening_MANIFEST_file_geting_NPE_on_project_delete branch August 14, 2024 11:09
@HannesWell
Copy link
Member

I`m closing this PR and create new PR with primary email id.

You can always force-push amended commits to the branch of a Pull-Request. And while amending a commit you can also change the author/commiter mail.
So there is no need to close a PR and create a new one, just to fix an email, message or alike. :)

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