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

CompatHelper: bump compat for SymbolicUtils to 1, (keep existing compat) #104

Merged

Conversation

github-actions[bot]
Copy link
Contributor

This pull request changes the compat entry for the SymbolicUtils package from 0.17, 0.18, 0.19 to 0.17, 0.18, 0.19, 1.
This keeps the compat entries for earlier versions.

Note: I have not tested your package with this new compat entry.
It is your responsibility to make sure that your package tests pass before you merge this pull request.

@ChrisRackauckas ChrisRackauckas force-pushed the compathelper/new_version/2023-01-14-00-07-48-714-00114732458 branch from 0b38b80 to 9e367f0 Compare January 14, 2023 00:07
@codecov
Copy link

codecov bot commented Jan 14, 2023

Codecov Report

Merging #104 (c45037d) into main (c35cd9e) will decrease coverage by 5.77%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #104      +/-   ##
==========================================
- Coverage   94.23%   88.47%   -5.77%     
==========================================
  Files           6        6              
  Lines         295      295              
==========================================
- Hits          278      261      -17     
- Misses         17       34      +17     
Impacted Files Coverage Δ
src/systems.jl 83.33% <0.00%> (-8.98%) ⬇️
src/events.jl 88.46% <0.00%> (-7.70%) ⬇️
src/utils.jl 88.88% <0.00%> (-5.56%) ⬇️
src/reactions.jl 94.33% <0.00%> (-4.72%) ⬇️
src/rules.jl 92.68% <0.00%> (-2.44%) ⬇️

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

@paulflang
Copy link
Member

@ChrisRackauckas : I am a bit puzzled why changing the compat reduces code coverage that much. Otherwise, I guess this can be merged. Also, I see perhaps we should no longer support/test Julia 1. What do you think?

@isaacsas
Copy link
Member

Probably better to hold off on this till ModelingToolkit and Catalyst get updated first. You'll want to bump compat to require their new versions at that point anyways.

@pogudingleb
Copy link

It seems that Catalyst and ModelingToolkit have been updated. Can this PR be merged now?

@paulflang paulflang merged commit 1233e88 into main Feb 25, 2023
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