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

Semantic: check there is at most one DERIVATIVE block #1099

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

pramodk
Copy link
Contributor

@pramodk pramodk commented Nov 21, 2023

This is not forbidden to have more, but due to a bug this fail in nrn.

Just to re-trigger CI testing for #1098

Nicolas Cornu and others added 2 commits November 21, 2023 11:47
This is not forbidden to have more, but due to a bug this fail in nrn.
Copy link

codecov bot commented Nov 21, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (f57d0d7) 88.68% compared to head (ef4d3c3) 88.67%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1099      +/-   ##
==========================================
- Coverage   88.68%   88.67%   -0.01%     
==========================================
  Files         170      170              
  Lines       12548    12540       -8     
==========================================
- Hits        11128    11120       -8     
  Misses       1420     1420              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@bbpbuildbot

This comment has been minimized.

@bbpbuildbot

This comment has been minimized.

@bbpbuildbot
Copy link
Collaborator

Logfiles from GitLab pipeline #172193 (:white_check_mark:) have been uploaded here!

Status and direct links:

@1uc
Copy link
Collaborator

1uc commented May 10, 2024

@pramodk this looks useful. It contains unrelated code which is now outdated. Should this be revived and merged? Alternatively we can close it as too old.

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