Fix clippy::semicolon_if_nothing_returned

[SchahinRohani]

How Has This Been Tested?

bazel test ...

Checklist

• Updated documentation if needed
• Tests added/amended
• bazel test //... passes locally
• PR is contained in a single commit, using git amend see some docs

---

This change is 

[aaronmondal]

Reviewed 12 of 12 files at r1, all commit messages.
Reviewable status: 1 of 1 LGTMs obtained, and all files reviewed, and 1 discussions need to be resolved

---

nativelink-worker/src/running_actions_manager.rs line 1199 at r1 (raw file):

                    OutputType::DirectorySymlink(output_symlink) => {
                        output_directory_symlinks.push(output_symlink);
                    }

nit

Suggestion:

                    OutputType::FileSymlink(output_symlink) =>
                        output_file_symlinks.push(output_symlink),
                    OutputType::DirectorySymlink(output_symlink) =>
                        output_directory_symlinks.push(output_symlink),

[SchahinRohani]

Reviewable status: 1 of 1 LGTMs obtained, and all files reviewed, and 1 discussions need to be resolved

---

nativelink-worker/src/running_actions_manager.rs line 1199 at r1 (raw file):

Previously, aaronmondal (Aaron Siddhartha Mondal) wrote…

nit

cargo fmt formats it like this:

OutputType::DirectorySymlink(output_symlink) => {
    output_directory_symlinks.push(output_symlink);
}

So i guess the current formatting follows the standard Rust conventions.

[aaronmondal]

Reviewable status: complete! 1 of 1 LGTMs obtained, and all files reviewed

[adam-singer]

Reviewed 12 of 12 files at r1, all commit messages.
Reviewable status: complete! 2 of 1 LGTMs obtained, and all files reviewed