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
Deprecate obsolete methods #1295
Conversation
Codecov Report
@@ Coverage Diff @@
## main #1295 +/- ##
==========================================
- Coverage 67.26% 67.18% -0.08%
==========================================
Files 314 314
Lines 41887 41904 +17
Branches 16852 16863 +11
==========================================
- Hits 28174 28155 -19
- Misses 11483 11518 +35
- Partials 2230 2231 +1
📣 Codecov can now indicate which changes are the most critical in Pull Requests. Learn more |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks fine to me. I just found two typos -- the one below, and one in the commit message mentioning "ubuntu-18.05"
Reaction::check (renamed from no-argument Reaction::validate) does not require a Kinetics object; nomenclature reflects ReactionRate::check and ReactionRate::validate.
@speth ... thanks for the prompt review! Things should be ready to go ... |
Changes proposed in this pull request
setup_cantera
scriptsIf applicable, fill in the issue number this pull request is fixing
Closes Cantera/enhancements#135, closes #1280
If applicable, provide an example illustrating new features this pull request is introducing
Checklist
scons build
&scons test
) and unit tests address code coverage