Skip to content

Add enhanced clippy - #26

Open
ThomasSaunders wants to merge 41 commits into
devfrom
clippy
Open

ThomasSaunders wants to merge 41 commits into
devfrom
clippy

Conversation

@ThomasSaunders

Copy link
Copy Markdown
Contributor

Make Clippy requirements inline with AirspaceHub. This PR adds the config and fixes the findings.

@ThomasSaunders ThomasSaunders self-assigned this Oct 18, 2026
Comment thread src/airport.rs Fixed
Comment thread src/airport.rs
let path = if !future_data {
AIRPORT_DATA_LOCATION
} else {
pub fn read_airports(future_data: bool) -> Result<Vec<Airport>, Box<dyn std::error::Error>> {
Comment thread src/airport.rs Fixed
Comment thread src/airport.rs
}

pub fn generate_airport_changes() {
pub fn generate_airport_changes() -> Result<(), Box<dyn std::error::Error>> {
Comment thread src/airport.rs Fixed
Comment thread src/airport.rs Fixed
Comment thread src/preferential_route.rs Fixed
Comment thread src/preferential_route.rs
}

pub fn generate_tec_route_changes() {
pub fn generate_tec_route_changes() -> Result<(), Box<dyn std::error::Error>> {
Comment thread src/preferential_route.rs Fixed
Comment thread src/preferential_route.rs Fixed
@codecov-commenter

codecov-commenter commented Oct 18, 2026 •

Copy link
Copy Markdown

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

❌ Patch coverage is 0% with 272 lines in your changes missing coverage. Please review.
✅ Project coverage is 0.00%. Comparing base (6f0fc36) to head (4777879).

Files with missing lines Patch % Lines
src/airport.rs 0.00% 132 Missing ⚠️
src/preferential_route.rs 0.00% 130 Missing ⚠️
src/util.rs 0.00% 6 Missing ⚠️
src/main.rs 0.00% 4 Missing ⚠️
❗ Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@          Coverage Diff          @@
##             dev     #26   +/-   ##
=====================================
  Coverage   0.00%   0.00%           
=====================================
  Files          3       4    +1     
  Lines        543     540    -3     
=====================================
+ Misses       543     540    -3     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Comment thread src/preferential_route.rs Fixed
Comment thread src/preferential_route.rs Fixed
Comment thread src/preferential_route.rs Fixed
Comment thread src/preferential_route.rs Fixed
Comment thread src/preferential_route.rs Fixed
Comment thread src/airport.rs Fixed
Comment thread src/airport.rs Fixed
Comment thread src/airport.rs Fixed
@ThomasSaunders
ThomasSaunders marked this pull request as ready for review October 18, 2026 20:48
@ThomasSaunders
ThomasSaunders marked this pull request as ready for review October 18, 2026 20:48
Comment thread src/preferential_route.rs
Ok(route_list)
}

fn process_tec_route(raw_data: Vec<&str>) -> Result<Vec<PreferentialRoute>, Box<dyn std::error::Error>> {
Comment thread src/preferential_route.rs
Ok(route_list)
}

fn process_tec_route(raw_data: Vec<&str>) -> Result<Vec<PreferentialRoute>, Box<dyn std::error::Error>> {
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants