proposal: save local config to .git/config

[maxice8]

Description
I set host of a repo to gitlab.alpinelinux.org and glab wrote .glab-cli/config/config.yaml, to hide it wrote to .gitignore.

Expected Behavior vs Actual Behavior
It to write somewhere where it doesn't modify my repository like .git/config in a [glab] section

Steps to Reproduce

1. glab config set editor nvim

Your Environment

• Version used (Run glab --version): glab version 1.12.1-82-g0c3ff7e (2020-12-21)
• Operating System and version: Fedora SilverBlue 33

[profclems]

To avoid this modifying the .gitignore, use the --global flag.

I agree with your proposal though. This is something I have thought about and I knew it will be brought.
We thought of making glab write to the global config by default but halted for backward compatibility reasons.

[profclems]

I'm changing the issue title to "save local config to .git/config" and mark it as a proposal since it is not really a bug.
We already know it changes the .gitignore when a local config is created

[maxice8]

Another thing, will it be possible to save configuration like defaults for --when-pipeline-succeeds ?

[profclems]

I haven't thought about this but we can add it up. It is not currently a respected setting.

[maxice8]

At the moment I have a shellscript that does what I need:

#!/bin/sh
opts=""
mrs=""
for arg; do
    case "$arg" in
        ''|*[!0-9]*) opts="$opts $arg" ;;
        *) mrs="$mrs $arg" ;;
    esac
done

[ -n "$mrs" ] || exit 1

if [ -n "$AMR_DRY_RUN" ]; then
    printf -- "Merge Requests: !%s\n" $(printf -- "%s" "$mrs")
    printf -- "Opts: %s\n" $(printf -- "%s" "$opts")
    exit 0
fi

for mr in $mrs; do
    glab mr rebase "$mr" && glab mr merge "$mr" --when-pipeline-succeeds=false $opts
done

[profclems]

Nice. This has brought up a nice-to-have feature for glab: merging multiple merge requests with comma-separated ids as arg

$ glab mr merge 1,2,3,4
#=> merges MRs with ids 1, 2, 3, and 4 💬

[maxice8]

Wouldn't it be better to just accept an arbitrary number of arguments, each one being an id ?

There are no positional arguments beside the merge request ids, all options are flags.

[profclems]

This will be much easier to work on if we move the local config .glab-cli to .git/glab-cli.

With the migration, at first run, we check if the there's a local config file and move it to .git/glab-cli

[clemsbot]

This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 5 days.