| Summary: | Yellow 'ALREADY_UPDATE' status message when merging nothing new to the active branch | ||
|---|---|---|---|
| Product: | [ECD] Orion | Reporter: | Remy Suen <remy.suen> |
| Component: | Git | Assignee: | Remy Suen <remy.suen> |
| Status: | RESOLVED FIXED | QA Contact: | |
| Severity: | normal | ||
| Priority: | P3 | CC: | Michael_Rennie |
| Version: | 13.0 | ||
| Target Milestone: | 14.0 | ||
| Hardware: | PC | ||
| OS: | Windows NT | ||
| See Also: |
https://git.eclipse.org/r/88245 https://git.eclipse.org/c/orion/org.eclipse.orion.client.git/commit/?id=69722c699faa7a3fc4f2dc62e55e665a1bb8837f |
||
| Whiteboard: | |||
|
Description
Remy Suen
If you look at the gitCommands.js file, you will see that we are expecting an "ALREADY_UP_TO_DATE" string. https://github.com/eclipse/orion.client/blob/a97156cb8b1c4e57673308a896c2f37d50c25917/bundles/org.eclipse.orion.client.git/web/orion/git/gitCommands.js#L745 That's a constant defined in JGit. http://download.eclipse.org/jgit/site/4.6.0.201612231935-r/apidocs/org/eclipse/jgit/api/MergeResult.MergeStatus.html#ALREADY_UP_TO_DATE The fix is to return the same string in commit.js if we detect the same scenario after a merge. New Gerrit change created: https://git.eclipse.org/r/88245 Gerrit change https://git.eclipse.org/r/88245 was merged to [master]. Commit: http://git.eclipse.org/c/orion/org.eclipse.orion.client.git/commit/?id=69722c699faa7a3fc4f2dc62e55e665a1bb8837f Merged, thanks Remy! |