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

Make BaseApiAnalyzer#getJavaProject protected #786

Merged
merged 1 commit into from
Oct 8, 2023

Conversation

laeubi
Copy link
Contributor

@laeubi laeubi commented Oct 8, 2023

Currently there is no way to pass the project to the analyzer without having an project ProjectComponent, but users of BaseApiAnalyzer might have other means to find the project in question.

This makes the BaseApiAnalyzer#getJavaProject() protected so implementors can override this (e.g. Tycho).

This relates to

Currently there is no way to pass the project to the analyzer without
having an project ProjectComponent, but users of BaseApiAnalyzer might
have other means to find the project in question.

This makes the BaseApiAnalyzer#getJavaProject() protected so
implementors can override this (e.g. Tycho).
@github-actions
Copy link

github-actions bot commented Oct 8, 2023

Test Results

     273 files  +       6       273 suites  +6   1h 12m 30s ⏱️ + 23m 18s
  3 340 tests  -        1    3 309 ✔️  -        1  30 💤 ±  0  0  - 1  1 🔥 +1 
10 317 runs  +2 721  10 226 ✔️ +2 697  90 💤 +24  0  - 1  1 🔥 +1 

For more details on these errors, see this check.

Results for commit 8981f97. ± Comparison against base commit c9cbfbe.

This pull request removes 1 test.
AllPDETests org.eclipse.pde.ui.tests.launcher.AllLauncherTests org.eclipse.pde.ui.tests.launcher.FeatureBasedLaunchTest ‑ Unknown test

@laeubi
Copy link
Contributor Author

laeubi commented Oct 8, 2023

@laeubi laeubi merged commit 540c9ef into eclipse-pde:master Oct 8, 2023
12 of 14 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.

1 participant