Skip to content
This repository has been archived by the owner on Mar 15, 2023. It is now read-only.

added debugged Dockerfile #553

Draft
wants to merge 8 commits into
base: main
Choose a base branch
from
Draft

Conversation

CodyCBakerPhD
Copy link
Member

WIP

Builds and runs fine locally, image can currently be pulled from https://hub.docker.com/r/codycbakerphd/test_nwbct

Some To-Do's:

  • make CatalystNeuro namespace on DockerHub for these kinds of things
  • Use https://github.com/eWaterCycle/setup-singularity to setup CI tests of the pulled image; maybe these should only trigger daily?
  • Maybe add tests for if code changes affect the ability to build the image (test on push to PR)
  • Maybe also add auto-building and auto-pushing synchronized with official PyPI releases of the package

@CodyCBakerPhD CodyCBakerPhD self-assigned this Jun 20, 2022
@CodyCBakerPhD CodyCBakerPhD added the enhancement New feature or request label Jun 20, 2022
Base automatically changed from click_yaml_conversion_specification to main June 28, 2022 18:29
@codecov
Copy link

codecov bot commented Jul 18, 2022

Codecov Report

Merging #553 (f5fd4fe) into main (72a56b0) will increase coverage by 0.77%.
The diff coverage is n/a.

❗ Current head f5fd4fe differs from pull request most recent head af86708. Consider uploading reports for the commit af86708 to get more accurate results

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #553      +/-   ##
==========================================
+ Coverage   87.56%   88.34%   +0.77%     
==========================================
  Files          59       59              
  Lines        3218     3218              
==========================================
+ Hits         2818     2843      +25     
+ Misses        400      375      -25     
Flag Coverage Δ
unittests 88.34% <ø> (+0.77%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/nwb_conversion_tools/tools/data_transfers.py 91.54% <0.00%> (+35.21%) ⬆️

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

Successfully merging this pull request may close these issues.

1 participant