Skip to content
This repository has been archived by the owner on Feb 26, 2025. It is now read-only.

revise documentation #917

Merged
merged 6 commits into from
May 26, 2021
Merged

revise documentation #917

merged 6 commits into from
May 26, 2021

Conversation

asanin-epfl
Copy link
Contributor

@asanin-epfl asanin-epfl commented May 11, 2021

Almost all done except definitions and file_formats. They need to be moved to MorphIO. done

Can be merged only after BlueBrain/MorphIO#313 is merged and new MorphIO released.

In order to merge the bulk of this PR, the part that depends on MorphIO was moved to #925 .

@asanin-epfl asanin-epfl self-assigned this May 11, 2021
@codecov-commenter
Copy link

codecov-commenter commented May 11, 2021

Codecov Report

Merging #917 (8a9816d) into master (bea16be) will not change coverage.
The diff coverage is n/a.

❗ Current head 8a9816d differs from pull request most recent head 03a24ae. Consider uploading reports for the commit 03a24ae to get more accurate results

@@            Coverage Diff            @@
##            master      #917   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           37        37           
  Lines         2116      2116           
=========================================
  Hits          2116      2116           

@asanin-epfl asanin-epfl marked this pull request as draft May 12, 2021 10:20
Almost all done except definitions and file_formats. They need to be moved to MorphIO
@asanin-epfl asanin-epfl marked this pull request as ready for review May 25, 2021 15:04
Copy link
Contributor

@arnaudon arnaudon left a comment

Choose a reason for hiding this comment

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

LGTM!

Co-authored-by: Adrien Berchet <[email protected]>
@asanin-epfl asanin-epfl merged commit 19bfdf3 into master May 26, 2021
@asanin-epfl asanin-epfl deleted the documentation branch May 26, 2021 08:46
@asanin-epfl
Copy link
Contributor Author

Thank you!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants