Skip to content

Commit

Permalink
Fix Makefile to call the same lints, remove old helper script which d…
Browse files Browse the repository at this point in the history
…oes not compile
  • Loading branch information
r-n-o committed Nov 11, 2024
1 parent 7875b22 commit 0d31941
Show file tree
Hide file tree
Showing 5 changed files with 11 additions and 84 deletions.
4 changes: 2 additions & 2 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -18,11 +18,11 @@ test: out/.common-loaded

.PHONY: lint
lint: out/.common-loaded
$(call run,cargo clippy -- -D warnings)
$(call run,make -C lint)

.PHONY: format
format: out/.common-loaded
$(call run,rustfmt)
$(call run,make -C fmt)

.PHONY: docs
docs: out/.common-loaded
Expand Down
4 changes: 4 additions & 0 deletions src/Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -144,6 +144,10 @@ lint:

.PHONY: clippy
clippy:
cargo clippy -- -D warnings

.PHONY: clippy-fix
clippy-fix:
cargo clippy --fix --allow-dirty

.PHONY: fmt
Expand Down
79 changes: 0 additions & 79 deletions src/integration/src/bin/gen_att_doc.rs

This file was deleted.

2 changes: 2 additions & 0 deletions src/qos_crypto/src/shamir.rs
Original file line number Diff line number Diff line change
@@ -1,3 +1,5 @@
//! Shamir Secret Sharing module. We use the [`vsss-rs`](https://crates.io/crates/vsss-rs)
use rand_core::OsRng;
use vsss_rs::Gf256;

Expand Down
6 changes: 3 additions & 3 deletions src/qos_net/src/proxy_stream.rs
Original file line number Diff line number Diff line change
Expand Up @@ -31,10 +31,10 @@ impl ProxyStream {
/// # Arguments
///
/// * `addr` - the USOCK or VSOCK to connect to (this socket should be bound
/// to a qos_net proxy) `timeout` is the timeout applied to the socket
/// to a qos_net proxy) `timeout` is the timeout applied to the socket
/// * `timeout` - the timeout to connect with
/// * `hostname` - the hostname to connect to (the remote qos_net proxy will
/// resolve DNS)
/// resolve DNS)
/// * `port` - the port the remote qos_net proxy should connect to
/// (typically: 80 or 443 for http/https)
/// * `dns_resolvers` - array of resolvers to use to resolve `hostname`
Expand Down Expand Up @@ -80,7 +80,7 @@ impl ProxyStream {
///
/// # Arguments
/// * `addr` - the USOCK or VSOCK to connect to (this socket should be bound
/// to a qos_net proxy) `timeout` is the timeout applied to the socket
/// to a qos_net proxy) `timeout` is the timeout applied to the socket
/// * `timeout` - the timeout to connect with
/// * `ip` - the IP the remote qos_net proxy should connect to
/// * `port` - the port the remote qos_net proxy should connect to
Expand Down

0 comments on commit 0d31941

Please sign in to comment.