Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

tests: don't have cargo install test tools #216

Closed
wants to merge 2 commits into from
Closed

tests: don't have cargo install test tools #216

wants to merge 2 commits into from

Commits on Sep 22, 2022

  1. testing: don't have cargo install test tools 

    I noticed that `cargo install --path .` will install the new
`fake-editor` and `fake-diff-editor` binaries, which we don't want. It
doesn't seem to work to put them in a separate crate and have a
dev-dependency on it because `cargo` doesn't like dev-dependencies to
binary targets. However, it seems that we can put them in `examples/`
to be able to access them without having `cargo install` install them.
    @martinvonz
    martinvonz committed Sep 22, 2022
    Configuration menu
    Copy the full SHA
    e48334d View commit details
    Browse the repository at this point in the history

  2. readme: remove now-unnecessary --bin jj argument to cargo install

    @martinvonz
    martinvonz committed Sep 22, 2022
    Configuration menu
    Copy the full SHA
    918da28 View commit details
    Browse the repository at this point in the history