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.

Sign up for GitHub

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

Jump to bottom

[BundleContextExtension] More informative error on null BundleContext #500

Merged
merged 1 commit into from
Mar 28, 2022
Merged

[BundleContextExtension] More informative error on null BundleContext #500

merged 1 commit into from
Mar 28, 2022

Conversation

kriegfrj
Copy link
Contributor

Fixes #302.

@kriegfrj
[BundleContextExtension] More informative error on null BundleContext
dec6328 
Fixes eclipse-osgi-technology#302.

Signed-off-by: Fr Jeremy Krieg <fr.jkrieg@greekwelfaresa.org.au>
@kriegfrj kriegfrj requested a review from rotty3000 March 28, 2022 12:58
@kriegfrj kriegfrj mentioned this pull request Mar 28, 2022
Closed
rotty3000
rotty3000 approved these changes Mar 28, 2022
View reviewed changes
Copy link
Contributor

@rotty3000 rotty3000 left a comment

Choose a reason for hiding this comment

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

LGTM

@kriegfrj kriegfrj merged commit 0426c0b into eclipse-osgi-technology:main Mar 28, 2022
@kriegfrj kriegfrj deleted the 302-bundlecontext-null branch March 28, 2022 13:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rotty3000 rotty3000 rotty3000 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

BundleContextExtension does not check for null
2 participants
@kriegfrj @rotty3000