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.

Sign up for GitHub

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

Jump to bottom

v6: fix panic on new stable #73

Merged

leshow merged 1 commit into master from fix_v6_opts

Jun 6, 2024

Collaborator

leshow commented Jun 6, 2024

missing a check to make sure the next comparison doesn't run off the end of the array


          v6: fix panic on new stable

1b1b6ec

Collaborator

coveralls commented Jun 6, 2024 •

edited

Loading

Pull Request Test Coverage Report for Build 9407927273

Details

5 of 5 (100.0%) changed or added relevant lines in 1 file are covered.
3 unchanged lines in 3 files lost coverage.
Overall coverage remained the same at 52.626%

Files with Coverage Reduction	New Missed Lines	%
src/v4/relay.rs	1	58.87%
src/encoder.rs	1	81.05%
src/v6/options.rs	1	54.94%

Totals
Change from base Build 7548501112:	0.0%
Covered Lines:	1603
Relevant Lines:	3046

💛 - Coveralls

leshow force-pushed the fix_v6_opts branch from 27bac0e to 1b1b6ec Compare

June 6, 2024 21:20

Collaborator

coveralls commented Jun 6, 2024 •

edited

Loading

Pull Request Test Coverage Report for Build 9407993763

Details

5 of 5 (100.0%) changed or added relevant lines in 1 file are covered.
3 unchanged lines in 3 files lost coverage.
Overall coverage remained the same at 52.626%

Files with Coverage Reduction	New Missed Lines	%
src/v4/relay.rs	1	58.87%
src/encoder.rs	1	81.05%
src/v6/options.rs	1	54.94%

Totals
Change from base Build 7548501112:	0.0%
Covered Lines:	1603
Relevant Lines:	3046

💛 - Coveralls

leshow merged commit 8af42b3 into master

18 checks passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet