sapling icon indicating copy to clipboard operation
sapling copied to clipboard

The github extension breaks some commands for non-github repos

Open rbehrends opened this issue 3 years ago • 2 comments

This affects especially the smartlog and pull subcommands. sl pull is further affected in that disabling the github extension also results in an unknown command 'debugprmarker' error, unless you also disable the post-pull.prmarker hook.

Steps to reproduce:

$ sl clone https://gitlab.com/gitlab-examples/docker
$ cd docker
$ sl sl
abort: Either gitlab.com is not a GitHub Enterprise hostname or you are not logged in.
Authenticate using the GitHub CLI: `gh auth login --git-protocol https --hostname gitlab.com`
$ sl pull
pulling from https://gitlab.com/gitlab-examples/docker
abort: Either gitlab.com is not a GitHub Enterprise hostname or you are not logged in.
Authenticate using the GitHub CLI: `gh auth login --git-protocol https --hostname gitlab.com`
warning: post-pull.prmarker hook exited with status 255
$ printf '[extensions]\ngithub=!\n' >>$(sl configfile -l)
$ sl sl
@  29b7d9317  2016-04-15 11:25  ayufan  remote/master
│  initial commit
~
$ sl pull
pulling from https://gitlab.com/gitlab-examples/docker
unknown command 'debugprmarker'
(use 'sl help' to get help)
warning: post-pull.prmarker hook exited with status 255
sl --config hooks.post-pull.prmarker= pull
pulling from https://gitlab.com/gitlab-examples/docker

(This is based on the most recent sapling release, I think some earlier releases did not exhibit this behavior.)

rbehrends avatar Nov 20 '22 18:11 rbehrends

@rbehrends thanks for reporting: this is a hi-pri issue!

bolinfest avatar Nov 21 '22 15:11 bolinfest

@rbehrends thanks for reporting. Currently, debugprmarker is part of 'github' extension for automatically hiding merged PRs. You can disable post-pull.prmarker hook as a temporary fix. I am working on this issue.

$ sl config -e

# add below lines to disable `prmarker`
[hooks]
post-pull.prmarker=

zzl0 avatar Nov 21 '22 18:11 zzl0

For anyone who met this issue, thanks for your interest in Sapling. we have a fix for this bug and it will be included in next release. While waiting for the new release, you can unblock yourself by disable github and prmarker extension/hook.

$ sl config -e

# to disable `github` extension
[extensions]
github=!

# to disable `prmarker` hook
[hooks]
post-pull.prmarker=

zzl0 avatar Nov 28 '22 03:11 zzl0

fixed in the new release.

zzl0 avatar Dec 03 '22 00:12 zzl0