-
Notifications
You must be signed in to change notification settings - Fork 79
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
Conversation
162315a
to
49ce2ce
Compare
@iloveeclipse any remarks from you? |
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. |
Ok, no need to rush from my side. Just want this to be forgotten.
That's right, but is there a lot of maintenance work happening? Currently the |
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? |
Yes |
6b3b132
to
156c4cd
Compare
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.
156c4cd
to
b834fe4
Compare
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?