Skip to content

Standard integration tests, PEP8, docker adjustments #23

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.

Sign up for GitHub

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

Merged
merged 22 commits into from
Jul 12, 2022
Merged

Standard integration tests, PEP8, docker adjustments #23

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
22 commits
Select commit Hold shift + click to select a range
dcb062a
WIP PEP8
CBroz1 May 2, 2022
2f842b2
WIP Draft integration tests
CBroz1 May 4, 2022
bbfe140
WIP standard pytests. missing element-event. passed CLI, need docker …
CBroz1 May 4, 2022
c143548
WIP docker edits
CBroz1 May 5, 2022
7d4f879
minor docker changes
CBroz1 May 6, 2022
0cfdcf3
Revised docker from @iamamutt
CBroz1 May 13, 2022
90d0d74
Apply suggestions from code review
CBroz1 May 16, 2022
c77ac0c
WIP. Replace with dj image. TLS config statement
CBroz1 May 17, 2022
39d252d
Docker success *if detatched*
CBroz1 May 23, 2022
6a66ec4
Add contact info to Code of Conduct
CBroz1 Jun 17, 2022
a875dca
Remove order constraints in test_pipeline_generation
CBroz1 Jun 23, 2022
a777f34
WIP: Apply suggestions 1
CBroz1 Jul 8, 2022
c34d325
WIP: Apply suggestions 2
CBroz1 Jul 8, 2022
7e41a3a
WIP: Apply suggestions 3. All except Docker
CBroz1 Jul 8, 2022
217a84d
Revert dockerfile arg; adjust 05-explore
CBroz1 Jul 8, 2022
d7165ac
Apply suggestions from code review
CBroz1 Jul 12, 2022
8435df2
WIP. See details. Jupysync, verbose context, abc.Seq, etc.
CBroz1 Jul 12, 2022
b1e45a2
WIP: verbosity if to oneliner
CBroz1 Jul 12, 2022
68b04ed
Version bump
CBroz1 Jul 12, 2022
6fc72ee
Notebook fix black'd issues. Docker compose vol map
CBroz1 Jul 12, 2022
fd89eba
Revise notebook 02 - formatting issues
CBroz1 Jul 12, 2022
50fcc20
Revert nb 05
CBroz1 Jul 12, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Apply suggestions from code review 
Co-authored-by: Kabilar Gunalan <[email protected]>
  • Loading branch information
@CBroz1 @kabilar
CBroz1 and kabilar authored May 16, 2022
commit 90d0d746e8a5fefccee6d5ca5e8b340373fcd319
2 changes: 1 addition & 1 deletion docker/Dockerfile.test
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
FROM datajoint/djbase:py3.9-debian-8eb1715

ARG GITHUB_USERNAME=cbroz1
ARG GITHUB_USERNAME=datajoint
USER anaconda:anaconda

COPY ./docker/apt_requirements.txt /tmp/
Expand Down
2 changes: 1 addition & 1 deletion docker/docker-compose-test.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ services:
context: ../.
dockerfile: ./docker/Dockerfile.test
args:
- GITHUB_USERNAME=cbroz1
- GITHUB_USERNAME=datajoint
image: workflow-miniscope-test:0.1.0
container_name: workflow-miniscope-test
environment:
Expand Down