Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Airways parsing no. 2 #21

Merged
merged 4 commits into from
Apr 18, 2024
Merged

Airways parsing no. 2 #21

merged 4 commits into from
Apr 18, 2024

Conversation

globin
Copy link
Contributor

@globin globin commented Apr 18, 2024

supersedes #16, see discussion there for further.

Code Metrics Report

main (c8a6b0b) #21 (3139a10) +/-
Coverage 76.4% 76.6% +0.2%
Test Execution Time 1m46s 1m51s +5s
Details
  |                     | main (c8a6b0b) | #21 (3139a10) |  +/-  |
  |---------------------|----------------|---------------|-------|
+ | Coverage            |          76.4% |         76.6% | +0.2% |
  |   Files             |             11 |            11 |     0 |
  |   Lines             |            928 |           961 |   +33 |
+ |   Covered           |            709 |           736 |   +27 |
- | Test Execution Time |          1m46s |         1m51s |   +5s |

Code coverage of files in pull request scope (60.2% → 61.9%)

Files Coverage +/-
src/lib.rs 70.0% +1.6%
src/sct.rs 88.3% -1.4%
src/topsky/map/mod.rs 34.8% 0.0%

Reported by octocov

philsjh and others added 4 commits April 8, 2024 01:56
Updated tests in src/sct.rs to reflect changes
This simplifies the parsing logic but removes the validation,
this can be added optionally later on.
@globin globin merged commit f6406a8 into main Apr 18, 2024
3 checks passed
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.

2 participants