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

Revert "Workaround for missing 4.31 SWT API descriptions" #1397

Merged
merged 1 commit into from
Sep 25, 2024

Conversation

HannesWell
Copy link
Member

This reverts commit 2fd4268 and commit a6824f7, introduced with #1191 and #1192

These workarounds for missing API description files are not necessary anymore because they are now correctly included again for some time.

@iloveeclipse or do you see a reason why this should be kept?

Copy link

github-actions bot commented Sep 8, 2024

Test Results

   285 files  ±0     285 suites  ±0   51m 40s ⏱️ +50s
 3 581 tests ±0   3 505 ✅ ±0   76 💤 ±0  0 ❌ ±0 
10 935 runs  ±0  10 704 ✅ ±0  231 💤 ±0  0 ❌ ±0 

Results for commit b834fe4. ± Comparison against base commit c862d44.

♻️ This comment has been updated with latest results.

@HannesWell
Copy link
Member Author

@iloveeclipse any remarks from you?
If there is no objections I'll submit this latest tomorrow (Tuesday) evening.

@iloveeclipse
Copy link
Member

I had no time to review that in the IDE, and tomorrow I will be in meetings whole day, so I can't do a proper review.

Beside this, the workaround was for API checks in 4.33 IDE / 4.32 target only, which shouldn't be interested on master branch. Whoever develops on 4.32 maintenance branches wuld need to stay with 4.33 IDE.

@HannesWell
Copy link
Member Author

Ok, no need to rush from my side. Just want this to be forgotten.

Whoever develops on 4.32 maintenance branches wuld need to stay with 4.33 IDE.

That's right, but is there a lot of maintenance work happening? Currently the R4_32_maintenance branch is still in its initial state after the release and AFAICT usually if at all only selected changes are backported. In that case I would say it's acceptable that the API tools are not available then.

@iloveeclipse
Copy link
Member

is there a lot of maintenance work happening?

I don't know that and I personally don't care simply because a workaround is possible. I guess some bigger companies might have in house maintenance branches, but if they use API tooling I don't know. At least at Advantest we don't care about API checks on maintenance branch we have.

@HannesWell
Copy link
Member Author

is there a lot of maintenance work happening?

I don't know that and I personally don't care simply because a workaround is possible. I guess some bigger companies might have in house maintenance branches, but if they use API tooling I don't know. At least at Advantest we don't care about API checks on maintenance branch we have.

The only company I know of at all that cares about maintenance is IBM and AFAIK they play all maintenance work back to the official eclipse repos and only do back-ports of bug fixes or critical dependency updates.

So if I understand you correctly you are fine with this, aren't you?

@iloveeclipse
Copy link
Member

Yes

@HannesWell HannesWell force-pushed the revert-1191 branch 4 times, most recently from 6b3b132 to 156c4cd Compare September 25, 2024 17:11
This reverts commit 2fd4268
and commit a6824f7,
introduced with eclipse-pde#1191
and eclipse-pde#1192

These workarounds for missing API description files are not necessary
anymore because they are now correctly included again for some time.
@HannesWell HannesWell merged commit c9666d1 into eclipse-pde:master Sep 25, 2024
18 checks passed
@HannesWell HannesWell deleted the revert-1191 branch September 25, 2024 20:08
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.

2 participants