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

Bypass API analysis for classes from a test classpath entry #1313

Merged
merged 1 commit into from
Jun 26, 2024

Conversation

merks
Copy link
Contributor

@merks merks commented Jun 25, 2024

Copy link

Test Results

   291 files  +  2     291 suites  +2   55m 17s ⏱️ - 7m 25s
 3 578 tests ±  0   3 502 ✅ +  1   76 💤  -  1  0 ❌ ±0 
11 031 runs  +166  10 800 ✅ +138  231 💤 +28  0 ❌ ±0 

Results for commit a69e822. ± Comparison against base commit a5391bd.

Copy link
Contributor

@laeubi laeubi left a comment

Choose a reason for hiding this comment

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

Looks good!

@laeubi laeubi merged commit 40fe485 into eclipse-pde:master Jun 26, 2024
17 checks passed
@HannesWell
Copy link
Member

In a (private) chat with Ed I understood this as a 'last' resort solution but since we got eclipse-platform/eclipse.platform.swt#1305 this is not necessary anymore to fix the underlying issue.
So should we still keep this? If yes, maybe we should move the check to a higher level and let the analyzer just skip test source folders. Then it does not have to be checked again and again when creating problems (which are not always errors).

@merks
Copy link
Contributor Author

merks commented Jun 27, 2024

I didn’t merge it. I’m not sure we really need this given swt solved the problem without this.

@laeubi
Copy link
Contributor

laeubi commented Jun 27, 2024

I’m not sure we really need this given swt solved the problem without this.

Well SWT not really "solved" this, they installed a workaround (making thing package private) wile this works in this case I still believe test code should not be API checked, if it can happen on a higher level why not.

@merks merks deleted the pr-skip-api-analysis-for-tests branch June 27, 2024 07:34
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