Skip to content
New issue

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

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

Already on GitHub? # to your account

Update dependencies #61

Merged
merged 10 commits into from
May 8, 2023
Merged

Update dependencies #61

merged 10 commits into from
May 8, 2023

Conversation

legokichi
Copy link
Contributor

  • bump rust_toolchain version
  • clippy
  • udate outdated crates

Copy link
Contributor

@eldesh eldesh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

commented


use gatekeeper as gk;

#[derive(StructOpt, Debug)]
#[structopt(name = "gatekeeper")]
#[derive(clap::Parser, Debug)]
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

-V option seems to be deleted.

gatekeeper$ cargo run -- -V
   Compiling gatekeeper v2.2.0 (/home/eldesh/work/gatekeeper)
    Finished dev [unoptimized + debuginfo] target(s) in 3.27s
     Running `target/debug/gatekeeperd -V`
gatekeeperd
error: unexpected argument '-V' found

Usage: gatekeeperd [OPTIONS]

For more information, try '--help'.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

21d64f3
fixed

@legokichi legokichi requested a review from eldesh March 8, 2023 06:07
eldesh
eldesh previously approved these changes Mar 9, 2023
Copy link
Contributor

@eldesh eldesh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good 👍

@eldesh
Copy link
Contributor

eldesh commented Mar 9, 2023

Why statuses are not pass…?

Copy link
Contributor

@eldesh eldesh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@legokichi
Copy link
Contributor Author

legokichi commented Apr 28, 2023

@legokichi legokichi requested a review from eldesh May 7, 2023 05:30
Copy link
Contributor

@eldesh eldesh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please resolve conflicts

@legokichi legokichi force-pushed the update branch 3 times, most recently from cb35cd4 to 15ff94f Compare May 8, 2023 06:37
Copy link
Contributor

@eldesh eldesh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🆗

@legokichi legokichi merged commit be32b01 into Idein:master May 8, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants