#4504 "rebased onto" on PR pages seems to be wrong Git terminology
Opened 3 months ago by scfc. Modified 15 days ago

When I upload another version of a PR, Pagure notes that event on the web page as (for example) "rebased onto 19be446" where 19be446 is my updated commit. (The text was apparently introduced by 07aa8cb.)

This feels wrong with regard to Git terminology: https://git-scm.com/docs/git-rebase suggests that 19be446 in "rebased onto 19be446" would refer to a repository's branch head that has been updated since the PR was originally published, and my PR was rebased onto that.

I'm not a native English speaker, but I believe something like "rebased as 19be446" would be more fitting.


Indeed, we need to adjust this.

Metadata Update from @pingou:
- Issue tagged with: bug

2 months ago

Metadata Update from @jlanda:
- Issue assigned to jlanda

2 months ago

Metadata Update from @jlanda:
- Issue set to the milestone: 5.7

2 months ago

Metadata Update from @jlanda:
- Issue set to the milestone: None (was: 5.7)

2 months ago

"Rebased as" could also be misinterpreted as rebasing refers to the branch your branch is forked from. On GH the PR notes will show a message similar to this when a force push or rebase is done.

@blackknight36 blackknight36 force-pushed the blackknight36:add_mod_ldap branch from 0405a7f to 039c77d 9 days ago

Yeah, the rebased <from_sha> to <to_sha> would be nice.

Login to comment on this ticket.

Metadata
Related Pull Requests
  • #4520 Closed 2 months ago