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

API errors not reported for PR's #1003

Closed
iloveeclipse opened this issue Jan 25, 2024 · 4 comments · Fixed by #1011
Closed

API errors not reported for PR's #1003

iloveeclipse opened this issue Jan 25, 2024 · 4 comments · Fixed by #1011
Labels
bug Something isn't working regression Something that used to work

Comments

@iloveeclipse
Copy link
Member

See #1002.

All PR validation checks are green, but the PR has API errors.

image

It must be recent regression with the build validation, something like #988 could be related, but I haven't checked when it started to fail.

We must fix that ASAP, otherwise we might blindly merge some broken API...

@iloveeclipse iloveeclipse added bug Something isn't working regression Something that used to work labels Jan 25, 2024
@iloveeclipse
Copy link
Member Author

@HannesWell : could you please check?

@laeubi
Copy link
Contributor

laeubi commented Jan 25, 2024

It must be recent regression with the build validation, something like #988 could be related, but I haven't checked when it started to fail.

As far as I know SWT was never API checked (@akurtakov might can confirm) if you look at the build scripts as the -Papi-check profile is not active on Jenkins pipeline. Even if I go back 2 years I can't find any sign for this profile to being active any time: https://github.com/eclipse-platform/eclipse.platform.swt/blob/4fc9d68c6f54aba4cb0899a10723e4d01bea70c5/Jenkinsfile

@akurtakov
Copy link
Member

API checks for swt are simply too hard to setup for console build . We have always relied on committers checking it in the workspace as there are even 3 different impl. for pretty much everything and one can break the API of only one of them.

@HannesWell
Copy link
Member

API checks for swt are simply too hard to setup for console build . We have always relied on committers checking it in the workspace as there are even 3 different impl. for pretty much everything and one can break the API of only one of them.

With the recent build rework, i.e. that El all bundles are built together, I can try to simply API check each fragment individually.

HannesWell added a commit to HannesWell/eclipse.platform.swt that referenced this issue Jan 28, 2024
HannesWell added a commit to HannesWell/eclipse.platform.swt that referenced this issue Jan 28, 2024
HannesWell added a commit to HannesWell/eclipse.platform.swt that referenced this issue Feb 2, 2024
HannesWell added a commit to HannesWell/eclipse.platform.swt that referenced this issue Feb 3, 2024
HannesWell added a commit to HannesWell/eclipse.platform.swt that referenced this issue Feb 4, 2024
HannesWell added a commit to HannesWell/eclipse.platform.swt that referenced this issue Feb 5, 2024
HannesWell added a commit to HannesWell/eclipse.platform.swt that referenced this issue Feb 5, 2024
HannesWell added a commit to HannesWell/eclipse.platform.swt that referenced this issue Feb 6, 2024
HannesWell added a commit to HannesWell/eclipse.platform.swt that referenced this issue Feb 6, 2024
HannesWell added a commit to HannesWell/eclipse.platform.swt that referenced this issue Feb 7, 2024
HannesWell added a commit to HannesWell/eclipse.platform.swt that referenced this issue Feb 8, 2024
laeubi pushed a commit to HannesWell/eclipse.platform.swt that referenced this issue Mar 11, 2024
iloveeclipse pushed a commit to HannesWell/eclipse.platform.swt that referenced this issue Mar 11, 2024
HannesWell added a commit to HannesWell/eclipse.platform.swt that referenced this issue Mar 12, 2024
HannesWell added a commit to HannesWell/eclipse.platform.swt that referenced this issue Mar 12, 2024
HannesWell added a commit to HannesWell/eclipse.platform.swt that referenced this issue Mar 12, 2024
HannesWell added a commit to HannesWell/eclipse.platform.swt that referenced this issue Mar 12, 2024
HannesWell added a commit to HannesWell/eclipse.platform.swt that referenced this issue Mar 12, 2024
HannesWell added a commit that referenced this issue Mar 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working regression Something that used to work
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants