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

Update result_collection.jl #389

Merged
merged 2 commits into from
May 8, 2023
Merged

Conversation

yp51md
Copy link
Contributor

@yp51md yp51md commented May 8, 2023

add verbose && before all the @info for a clean print.

add verbose && before all the @info
@Datseris
Copy link
Member

Datseris commented May 8, 2023

great, also bump patch version (3rd number) in the version in Project.toml file

@codecov
Copy link

codecov bot commented May 8, 2023

Codecov Report

Merging #389 (b42ed9e) into main (a9a2383) will not change coverage.
The diff coverage is 100.00%.

❗ Current head b42ed9e differs from pull request most recent head e1c06b0. Consider uploading reports for the commit e1c06b0 to get more accurate results

@@           Coverage Diff           @@
##             main     #389   +/-   ##
=======================================
  Coverage   90.69%   90.69%           
=======================================
  Files           8        8           
  Lines         752      752           
=======================================
  Hits          682      682           
  Misses         70       70           
Impacted Files Coverage Δ
src/result_collection.jl 98.18% <100.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@Datseris Datseris merged commit ab87902 into JuliaDynamics:main May 8, 2023
3 of 4 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.

None yet

2 participants