Age | Commit message (Collapse) | Author |
|
Ideas for getting diff colours from the Git config and parsing them
using 'colorparse'.
Leaving this aside and planning to add diff colours by executing
`git-diff` instead. Shelling out to `git diff` would allow us to accept
diff command line options like `--word-diff` and have them work
transparently.
|
|
|
|
This is subsumed into `Cargo.lock` and doesn't do anything any more.
|
|
Probably not going to do this one as it's not always possible to get the
commenter's name and email address, and even then it might not be the
same as what they use in the repo.
|
|
|
|
This was replaced by `RELEASE_PRODUCTS` so that I could make
`DIST_PRODUCTS` based on `PRODUCTS`.
|
|
|
|
|
|
Add recipes to build release targets and copy the binaries and man pages
into the `dist/` folder. We'll use this to package a release archive.
|
|
Got an error:
error: failed to open: /usr/local/.crates.toml
when I tried the command with the `--root /usr/local` argument, so
decided to remove it.
Need to use the root crate name in the `cargo uninstall` command.
|
|
Since this is a Git command, this makes more sense to me.
|
|
Found a suggestion on the `gh` GitHub CLI project page from 'mislav'
(https://github.com/mislav) and 'probablycorey'
(https://github.com/probablycorey) to use as an example.
Haven't tested the install/uninstall instructions yet (copied these from
another one of my projects).
|
|
I hadn't tested with HTTPS remote URLs, so assumed the code I copied
from `github_suggestion::url` would work. It worked for suggestion
comment URLs, but it doesn't work for remote URLs because the latter end
in ".git". Remove this suffix to fix the bug.
|
|
Forgot to include this so we didn't get extra information about the
cause of the error.
|
|
|
|
|
|
Copied from `git-sugpatch.1`, since the general elements are basically
the same. Modified to describe the specifics of the `git-sugapply`
command.
|
|
Make it clear that these options require argument values.
|
|
To make it clear which ID needs to be used when using the short form.
|
|
|
|
|
|
|
|
Allow getting the program version from the command line.
|
|
|
|
We have multiple binaries in `src/bin/`, so this file is no longer
needed.
|
|
Light documentation for our various functions and types.
|
|
|
|
|
|
Since this check for suggestion arguments is in both binaries, we can
move it to the common preflight function.
Clean up other areas now that we moved the function.
|
|
Use the new logic from `git-sugpatch` to build a `Config` and run
`apply()` on all suggestion input arguments.
|
|
|
|
|
|
Print usage on `-h` and `--help`.
Store the usage brief on `Config` in order to be able to print it from
multiple places.
|
|
Add `Options` to `Config` to allow us to get a usage string from a
`Config` instance, which is where we create `Options`.
Needed to remove the `Debug` derive on `Config` in order to add the
field as `Options` doesn't implement `Debug`.
|
|
This was a test, before I worked out the API for this function.
|
|
|
|
Replace my test exit code with a real one.
|
|
|
|
|
|
Print the error and exit.
|
|
Turns out this didn't need to be public at all.
|
|
Split this out into a separate function because I want to reuse it in
`git-sugapply`.
|
|
Forgot to change this line when I set up the loop over suggestion
arguments. We want to parse the current argument instead of hard-coding
the first one.
|
|
|
|
Made this `pub` for one of the test tries in
38a871f28bad90e238021d8cc46b9fa926f9df75, but in the end it wasn't
necessary. Restore this to the prior visibility.
|
|
The source error's message doesn't include the name. Ensure we print the
name of the variable we're trying to get so that users know which one is
missing.
|
|
Ensure the source error is surfaced.
|
|
Remove unused code.
|
|
We want to allow not having a remote when URL arguments are given, but
require it when a suggestion ID argument is given (otherwise we wouldn't
have an owner-repo pair to make the GitHub request).
Had some trouble with the `OwnerRepo.o_r` value. It was being moved into
the closure, so tried a loop. There was a similar problem with the loop.
However, by returning, I was able to get a reference to the `Result`
instead of having it be moved.
|
|
Add `exitcode` to exit with an appropriate code.
Add the `gseprintln` macro to wrap `eprintln!()`, prefixing the output
with "error: ".
|