Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Activity
  • Dec 12 20:36
    stappersg commented #144
  • Dec 11 12:53
    JohnTitor closed #144
  • Dec 11 12:53
    JohnTitor commented #144
  • Dec 10 13:17
    killercup transferred #154
  • Dec 08 13:06
    spacekookie synchronize #58
  • Dec 08 13:06

    spacekookie on master

    Clarify source position Make i… (compare)

  • Dec 08 13:06
    spacekookie closed #159
  • Dec 07 00:50
    Dylan-DPC added as member
  • Dec 07 00:50
    spacekookie added as member
  • Dec 07 00:50
    yoshuawuyts added as member
  • Dec 07 00:50
    killercup added as member
  • Dec 07 00:50
    Dylan-DPC removed as member
  • Dec 07 00:50
    spacekookie removed as member
  • Dec 07 00:50
    yoshuawuyts removed as member
  • Dec 07 00:50
    killercup removed as member
  • Dec 07 00:49
    spacekookie edited as member
  • Dec 06 23:52
    spacekookie commented #159
  • Dec 06 23:49
    spacekookie commented #144
  • Dec 06 21:03
    trevordmiller commented #161
  • Dec 05 14:25
    epage commented #161
Ed Page
@epage
Ah, it assumes v as well
Jacob Finkelman
@Eh2406
does that mean that the name needs to start with a v for install to work
Ed Page
@epage
Yup
Jacob Finkelman
@Eh2406
ok closing my pr.
Ed Page
@epage
It has to know what tags are relevant
Jacob Finkelman
@Eh2406
I closed my pr to cargo sweep holmgr/cargo-sweep#21
Jacob Finkelman
@Eh2406
I don't have perms, so hopefully @holmgr can retag things this weekend, and we will see if it is fully working.
Jacob Finkelman
@Eh2406
set up travis on my fork, and pushed a tag: https://github.com/Eh2406/cargo-sweep/releases/tag/v0.4.0
Jacob Finkelman
@Eh2406
It picked up the tag and tried to deploy but it has his secrets, so it did not work.
https://travis-ci.org/Eh2406/cargo-sweep/jobs/484559290#L267
https://travis-ci.org/Eh2406/cargo-sweep/jobs/484559285#L647
So it looks like changing the tags will work.
Thanks for the help!
Ed Page
@epage
You're welcome
Ed Page
@epage
RE assert_cmd 1.0
This is in the works.
I've been letting it sit to gather feedback before bumping to 1.0. This led to assert-rs/assert_cmd#69 and am in that same holding pattern; waiting for people to upgrade, try it out, and report back problems

assert_fs is in a similar boat but less controversial in my mind; I'd be willing to bump that any time.

Should they both go in the same announcement post?

Pascal Hertleif
@killercup
@epage morning! yeah, wanted to ping you about that. at the all hands cli meeting yesterday we talked about assert-rs for a bit -- it's probably the most complete of all the items we had on the WG todo list from last year!
Pascal Hertleif
@killercup
what are your immediate plans for these crates? i haven't been following in detail, but _cmd and _fs both look like there are no breaking changes happening soon, right? if we can squish the last known bugs, pushing them to 1.0 sounds like a good idea.
i'd love to get one or more blog posts out explaining to people how these crates fit together and how amazing predicates is. i can totally (help) write this
this might also be the right time to make assert_cli re-export the 1.0 crates, as well as helpers like tempfile
Ed Page
@epage
I would put them at release candidate stage (and have been for a while). I've taken care of any 1.0 blocking bugs but I consider the recent API breakers to be uncertain enough that I was planning on waiting a month or so to see what feedback trickles in.
So the current plan for assert_cli is to just aggregate the various CLI-related testing crates? At one point you were talking about a higher level, abstraction like in the waltz tests
Oh and two things I'd like to see but not blocking 1.0
  • Directory diffing. Still been mulling over what'd be the appropriate API
  • Snapshot testing as you suggested. I'm waiting to see how the other snapshot testing crates mature
Katharina Fey
@spacekookie
:wave:
waves at Pascal from 1 metre away
Jeremy Wright
@jmwright

Hello everyone. Is there a standard, cross-platform way to add tab completion to Rust CLIs? The solutions I've found in my searches look like they probably won't be cross-platform. Here are a couple of links I've found.
rust-lang/rustup.rs#278

https://www.reddit.com/r/rust/comments/4qvltw/claprs_can_now_generate_bash_tabcompletion/

I expect to use a combination of static (clap-rs generated) and dynamic tab completion (think path completion on the command line). I figured this group would be the most well connected with the best practices for this currently.

Pascal Hertleif
@killercup
@epage sounds good! i meant that once we have cmd and fs at 1.0, we should start building up assert-cli again from that, and iterate on its design from that
and yeah, i agree, dir diff and snapshots would be great to have. have you seen the progress on the insta crate?
Ed Page
@epage
been watching it.
Daniel Sockwell
@codesections
@jmwright What sort of cross-platform support are you looking for beyond what Clap provides? It does Bash, Powershell, Zsh, Fish, and Elvish, which seems to have most platforms covered
Jeremy Wright
@jmwright
@codesections Thanks for the reply. The only reference I found was to a bash autocomplete script, so I thought that only bash was supported. Looks like I was wrong.
Katharina Fey
@spacekookie
Just FYI, we renamed the github organisation to rust-cli to be more in-line with the rest of the working groups
Daniel Sockwell
@codesections
That makes sense. The old name was pretty fun, but I can see the logic
Gargi Sharma
@gs0510
Hi all! I am looking for repositories with "good-first-issue" to contribute for this working group. I am looking at the repos under "https://github.com/rust-cli". Am I looking at the right place? Thanks!
Ed Page
@epage
That is one of the places

Two other related repos

At the moment, neither of those have good-first-issues but liquid is a dependency of some of those and it has some

Gargi Sharma
@gs0510
Amazing! Thanks @epage :)
Dylan DPC
@Dylan-DPC
Hey :) i was trying to get clap-derive to build with latest changes of clap, but it failed. It keeps complaining that i have a get_args_from_safe() method on App which doesn't exist even though i replaced all occurrences of that function. If anyone wants to look you can check out https://github.com/clap-rs/clap_derive/tree/fix/branch
Ricky
@deg4uss3r
@Dylan-DPC I was able to build it off that branch on MacOS with Rust 1.33
Dylan DPC
@Dylan-DPC
strange
Dylan DPC
@Dylan-DPC
@deg4uss3r build works for me as well, can you try cargo test?
Ricky
@deg4uss3r
Sorry was in a meeting :P trying cargo test now
yeah seeing the same no method named get_matches_from_safe errors
Dylan DPC
@Dylan-DPC
@deg4uss3r thanks.
Mendelt Siebenga
@Mendelt
Hi everyone, I've been working on a crate that might be of interest to the CLI working group and it's slowly getting into a usable state. It's a library and a set of macro's to help build interactive command-shell like applications. Although this might cross the line too much into TUI territory. You can find it at https://crates.io/crates/cmdr
If you think this fits into the rust cli ecosystem I'm very interested in feedback
Bruce Mitchener
@waywardmonkeys
@Mendelt I was working on something similar in a crate called commands … I’d like to talk more about this sort of thing sometime! I’m basing it on things like TOPS-20 and CLIM command systems.
Mendelt Siebenga
@Mendelt
@waywardmonkeys Great! I'll have a look at commands. I'm not familiar with TOPS-20 and CLIM. I worked with Cmd in Python a lot and wanted to build something similar for rust.
Bruce Mitchener
@waywardmonkeys
@Mendelt It is mid-afternoon here, so I’m going out shortly to pick up my daughter from school and stuff … what timezone are you in?
Mendelt Siebenga
@Mendelt
I'm in Europe, near rotterdam. It's 9 AM over here. But I'm at work right now so my availability is limited.
Dylan DPC
@Dylan-DPC
This message was deleted
Pascal Hertleif
@killercup
While you can still us this channel, please note that

the official chat for the CLI WG is now #wg-cli on the rust-lang discord server

Ricky
@deg4uss3r
lol I was just going to ask if this is still the main room or not