jj git push: remove the NotFastForward error

Now that we always force push, it should not occur in practice.

cli/src/commands/git.rs

@@ -1015,11 +1015,6 @@ fn cmd_git_push(
     })
     .map_err(|err| match err {
         GitPushError::InternalGitError(err) => map_git_error(err),
-        GitPushError::NotFastForward => user_error_with_hint(
-            "The push conflicts with changes made on the remote (it is not fast-forwardable).",
-            "Try fetching from the remote, then make the branch point to where you want it to be, \
-             and push again.",
-        ),
         GitPushError::RefInUnexpectedLocation(refs) => user_error_with_hint(
             format!(
                 "Refusing to push a branch that unexpectedly moved on the remote. Affected refs: \

lib/src/git.rs

@@ -1209,9 +1209,6 @@ pub enum GitPushError {
         name = REMOTE_NAME_FOR_LOCAL_GIT_REPO
     )]
     RemoteReservedForLocalGitRepo,
-    // TODO: Delete this? It should never trigger
-    #[error("Push is not fast-forwardable")]
-    NotFastForward,
     #[error("Refs in unexpected location: {0:?}")]
     RefInUnexpectedLocation(Vec<String>),
     #[error("Remote rejected the update of some refs (do you have permission to push to {0:?}?)")]
@@ -1424,12 +1421,7 @@ fn push_refs(
         push_options.remote_callbacks(callbacks);
         remote
             .push(refspecs, Some(&mut push_options))
-            .map_err(|err| match (err.class(), err.code()) {
-                (git2::ErrorClass::Reference, git2::ErrorCode::NotFastForward) => {
-                    GitPushError::NotFastForward
-                }
-                _ => GitPushError::InternalGitError(err),
-            })
+            .map_err(GitPushError::InternalGitError)
     };
     if !failed_push_negotiations.is_empty() {
         // If the push negotiation returned an error, `remote.push` would not