-
Notifications
You must be signed in to change notification settings - Fork 105
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
Review java documentation #703
Conversation
@BraunMatthias I have resolved the merge conflicts that I caused with my parallel refactoring of Please pull this PR + the internal repo and you should have a consistent state. |
Hm.. looking at it.. still too many diffs not caused by you. I guess I need to go back to square one and do the merge from main again. Sorry, stay tuned.. |
ace7fe4
to
cf60467
Compare
Alright, this should now be your intended state, i.e. only your changes. Now, for all file moves and renames, this is quite incompatible to the current URLs. You need to at least add Still, all links to the old URLs need to be adjusted as these need to use the new URL pattern. We can also discuss how to handle this... |
Yes, redirect links have to be added in a step afterwards. But I wanted just to see and discuss the result in a preview. |
4b952ad
to
ef52090
Compare
Peer PR: https://github.tools.sap/cap/docs/pull/609