Skip to content

Remove RESET, MATCH and FORALL#2004

Merged
pramodk merged 2 commits into
masterfrom
remove_match_reset_forall
Oct 2, 2022
Merged

Remove RESET, MATCH and FORALL#2004
pramodk merged 2 commits into
masterfrom
remove_match_reset_forall

Conversation

@alkino

@alkino alkino commented Oct 1, 2022

Copy link
Copy Markdown
Member

see #1957

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

Merging #2004 (a4c558b) into master (06615ec) will increase coverage by 0.04%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #2004      +/-   ##
==========================================
+ Coverage   46.59%   46.64%   +0.04%     
==========================================
  Files         525      525              
  Lines      118869   118740     -129     
==========================================
- Hits        55387    55384       -3     
+ Misses      63482    63356     -126     
Impacted Files Coverage Δ
src/modlunit/init.cpp 100.00% <ø> (ø)
src/nmodl/deriv.cpp 72.91% <ø> (+17.60%) ⬆️
src/nmodl/init.cpp 100.00% <ø> (ø)
src/nmodl/noccout.cpp 94.70% <ø> (+0.47%) ⬆️

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

@azure-pipelines

Copy link
Copy Markdown

✔️ a4c558b -> Azure artifacts URL

Comment thread src/modlunit/parse1.ypp Outdated
@azure-pipelines

Copy link
Copy Markdown

✔️ 2c6e109b5f4c14e514b04b19e71ef1f8eec0d1ae -> Azure artifacts URL

@alkino alkino force-pushed the remove_match_reset_forall branch from 2c6e109 to 9a9687f Compare October 1, 2022 18:16
@azure-pipelines

Copy link
Copy Markdown

✔️ 9a9687f -> Azure artifacts URL

@pramodk pramodk left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@pramodk pramodk merged commit f22476b into master Oct 2, 2022
@pramodk pramodk deleted the remove_match_reset_forall branch October 2, 2022 07:40
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.

4 participants