Skip to content

Commit

Permalink
README numbering fix and resolve a security alert (#64)
Browse files Browse the repository at this point in the history
- Fixes the main README formatting which was resulting in incorrect numbering on the steps
- Resolve one of the security alerts in the repo
  • Loading branch information
wilyle authored Oct 18, 2023
1 parent ba72ce9 commit 4f10607
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 9 deletions.
12 changes: 6 additions & 6 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -26,13 +26,13 @@ This guide uses `apt` as the package manager in the examples. You may need to su

1. Install git and rust:

```shell
sudo apt update
sudo apt install -y git snapd
sudo snap install rustup --classic
```
```shell
sudo apt update
sudo apt install -y git snapd
sudo snap install rustup --classic
```

The rust toolchain version is managed by the `rust-toolchain.toml` file, so once you install `rustup` there is no need to manually install a toolchain or set a default.
The rust toolchain version is managed by the `rust-toolchain.toml` file, so once you install `rustup` there is no need to manually install a toolchain or set a default.

1. Clone this repository with `git clone`

Expand Down
3 changes: 0 additions & 3 deletions freyja/src/cartographer.rs
Original file line number Diff line number Diff line change
Expand Up @@ -95,9 +95,6 @@ impl<
if mapping_client_result.unwrap().has_work {
info!("Cartographer detected mapping work");

// TODO: will this notion of checking and sending inventory exist?
// self.mapping_client.send_inventory(SendInventoryRequest { inventory: self.known_providers.clone() }).await?;

let patches_result = self.get_mapping_as_signal_patches().await;
if patches_result.is_err() {
let error = patches_result.err().unwrap();
Expand Down

0 comments on commit 4f10607

Please sign in to comment.