fix: include_remote: false had no effect on revwalk scope

[stumash]

When no refspecs were provided, configure_revwalk always called push_glob("*"), walking all refs regardless of Settings::include_remote. This meant Builder::with_max_count would count commits from the full graph even when the caller requested local-only traversal.

Fix: pass include_remote into configure_revwalk and use push_glob("refs/heads/*") when false.

[stumash]

@peso this fix is so that when using git-graph, the --local and -n flags work well together. right now, if I do

git-graph --local -n 10

it won't show me 10 commits from the --local subgraph. instead it will show me probably less than 10 commits because first it gets 10 commits and then it filters for what's local.

[stumash]

my workaround for now using is to pass git-graph all the local branch names manually, which does work, but defeats the purpose of the --local flag

[peso]

Looks good, but you need to run cargo fmt before it can pass CI.

[stumash]

I didn't want to do formatting on files that were already failing in the master branch and introduce unrelated changes
edit: I didn't realize cargo fmt only applies to my changes

I'll do it now though one sec

[stumash]

alright @peso I ran cargo fmt on the files I modified. Any linting issues in CI that remain are the same as those on the master branch

[stumash]

@peso I applied linter fixes to all the other files in the repo to pass CI. ready to merge whenever

[peso]

Sorry that you had to fix formatting outside your PR. I forgot the state the code is still in.
But thank you for that upgrade as well.

[stumash]

huge fan, love git-graph. truly it's the only thing on the internet that will dump a human readable recent git history into my terminal. happy to contribute a fix.