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

publications using NEURON; progress toward #1728 #1819

Merged
merged 2 commits into from
May 12, 2022
Merged

Conversation

ramcdougal
Copy link
Member

@ramcdougal ramcdougal commented May 12, 2022

Content converted from https://www.neuron.yale.edu/neuron/publications/neuron-bibliography

It ends up looking like:

image

@codecov-commenter
Copy link

codecov-commenter commented May 12, 2022

Codecov Report

Merging #1819 (a886b88) into master (7bb6808) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1819   +/-   ##
=======================================
  Coverage   45.45%   45.45%           
=======================================
  Files         550      550           
  Lines      113130   113130           
=======================================
  Hits        51423    51423           
  Misses      61707    61707           

📣 Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

Copy link
Member

@pramodk pramodk left a comment

Choose a reason for hiding this comment

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

LGTM!

Quick question for clarity: in the future, we will update this rst file or website?

@ramcdougal
Copy link
Member Author

My position is that the website should be decommissioned in favor of running everything through readthedocs.

I think Ted's existing script generates HTML which is why I included the pandoc command as a comment, but it was pretty seamless to go from HTML to rst.

Copy link
Member

@alexsavulescu alexsavulescu left a comment

Choose a reason for hiding this comment

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

LGTM

@ramcdougal ramcdougal merged commit fcd609e into master May 12, 2022
@ramcdougal ramcdougal deleted the pubs-update branch May 12, 2022 20:30
@ramcdougal ramcdougal mentioned this pull request May 12, 2022
41 tasks
@alexsavulescu alexsavulescu mentioned this pull request Jun 29, 2022
19 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants