Push to gerrit is pushing to branch instead of refs/for/<branch>
See original GitHub issueWe have a strange behavior here, the plugin pushed a commit as a normal push instead of as a review.
Situation:
editing a review that depends on 2 other reviews those reviews target a branch other than master committing and choosing “commit and push” in the push dialogs:
- the 3 review are visible,
- push to gerrit check box is pre-checked (didn’t touch this box)
- the destination branch is preset to features/xyz from a previous push to gerrit which i don’t change and is correct
- hashtags are preset and not changed I click on “push” and all 3 review are now marked as submitted in gerrit
In the version control logs on intellij i find:
create mode 100644 view/src/main/angular/tsconfig.app.json
create mode 100644 view/src/main/angular/tsconfig.json
create mode 100644 view/src/main/angular/tsconfig.spec.json
17:24:14.361: [management-console] git -c credential.helper= -c core.quotepath=false -c log.showSignature=false push --progress --porcelain origin refs/heads/review/david_delbecq/1852:features/data-pricing-console
Enumerating objects: 146, done.
Delta compression using up to 8 threads
Total 144 (delta 33), reused 79 (delta 24)
remote:
remote: Processing changes: refs: 1, closed: 1 (\)
remote: Processing changes: refs: 1, closed: 1 (|)
remote: Processing changes: refs: 1, closed: 1 (/)
remote: Processing changes: refs: 1, closed: 1 (-)
remote: Processing changes: refs: 1, closed: 1 (\)
remote: Processing changes: refs: 1, closed: 1 (\)
remote: Processing changes: refs: 1, closed: 1 (\)
remote: Processing changes: refs: 1, closed: 1, done
To ssh://git.obfuscated/project/name
refs/heads/review/david_delbecq/1852:refs/heads/features/xyz cbc2cd9..2ae6c59
This behavior never happened before and happened twice this week. This might be related to the fact it’s the first time we do both branch targeting, hashtags and chained reviews on the same review.
It looks like the checkbox “submit to gerrit” was prechecked by intellij in both occurences, but the model didn’t match this value. Most of the time it’s not prechecked and we have to manually check it.
intellij version: 2019.2.3 community edition gerrit plugin version: 1.2.2-146 gerrit version: 3.0.1
Issue Analytics
- State:
- Created 3 years ago
- Comments:7 (3 by maintainers)
Top GitHub Comments
@uwolfer The issue is fixed by IntelliJ in Version 2021.3.3
@benno-mueller-saxess: The referenced issue should be available in the next release of IntelliJ - please report back if that resolves your issue.