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.

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

Jump to bottom

Integrate CLI with unit tests #196

Open
wants to merge 74 commits into
base: main
Choose a base branch
from
Open

Integrate CLI with unit tests #196

wants to merge 74 commits into from

Conversation

soniamitchell
Copy link
Contributor

@soniamitchell soniamitchell commented Nov 28, 2021 •
edited
Loading

Integrate CLI with unit tests

@soniamitchell
Read config data from CLI output
1aff592
@soniamitchell
delete file
6240df2
@soniamitchell
Set up fair CLI
c529db4
@soniamitchell
Update documentation
037a02f
@soniamitchell
Start updating test
190e0c7
@soniamitchell
Move files
b7d66bc
@soniamitchell
Move files
0bbee40
@soniamitchell
Have to keep pushing to keep the registry clean whilst trying to inte…
1815558 
…grate `fair run` with unit tests
@soniamitchell
Update tests
e548361
@soniamitchell
Fix path
9c3b63e
@soniamitchell
save path
81f1efa
@soniamitchell
Update workflow [skip ci]
51cba3f
@soniamitchell
Update tests [skip ci]
171abe0
@soniamitchell
return config file path [skip ci]
ce2a72c
@soniamitchell
update tests [skip ci]
bcc09b6
@soniamitchell
return config file path [skip ci]
eb4f29f
@soniamitchell
update tests [skip ci]
9f54e6c
@soniamitchell
update tests [skip ci]
dad5802
@soniamitchell
update tests [skip ci]
8b6738b
@soniamitchell
fix read writes [skip ci]
a7b0448
@soniamitchell
update tests [skip ci]
4a713da
@soniamitchell
take namespace from config [skip ci]
1e33c53
@soniamitchell
update tests [skip ci]
8f87a19
@soniamitchell
update tests [skip ci]
f6f1603
@soniamitchell
update tests [skip ci]
b9b62cc
@soniamitchell
update tests [skip ci]
f61a2d7
@soniamitchell
update tests [skip ci]
1492d92
@soniamitchell
remove tests until CLI is fixed [skip ci]
b2f27dc
@soniamitchell
update tests [skip ci]
a5726d9
@soniamitchell
update tests [skip ci]
ee7f95f
@soniamitchell soniamitchell changed the title Experimental branch Integrate CLI with unit tests Nov 28, 2021
@soniamitchell
Copy link
Contributor Author

CI errors are related to the local and remote repo being in the same place on GitHub actions. Once the next version of the CLI is released, use fair run --dirty and the tests should all pass.

@soniamitchell
remove examples [skip ci]
df97c61
@soniamitchell
edit documentation [skip ci]
cfdd4a5
@soniamitchell
Only run fair init --ci if .fair directory doesn't exist [skip ci]
ba4f1b0
@soniamitchell
Shouldn't need global variable once CLI is updated -- reference `.fai…
3b377d8 
…r/cli-config.yaml` instead [skip ci]
@soniamitchell
refactor [skip ci]
d403115
@soniamitchell
adding new test -- clean repo to test [skip ci]
2238042
@soniamitchell
clean repo to test [skip ci]
0883ce5
@soniamitchell
add comment [skip ci]
33d668e
@soniamitchell
remove test -- not sure whether the CLI should do this [skip ci]
1add109
@soniamitchell
Remove dependency
a537a9e
@soniamitchell
update print(handle) [skip ci]
ccef365
@soniamitchell
edit messages [skip ci]
71aa909
@soniamitchell
update messages [skip ci]
7e67639
@soniamitchell
update message [skip ci]
7f885af
@soniamitchell
fix badge
b52efd6
@soniamitchell
Fix CI - can't find master
81a2aeb
@soniamitchell
Fix CI - update R version
4110ed2
@soniamitchell
Fix CI - update R dependencies version
e7ff347
@soniamitchell
Fix CI - update versions
8e440dd
@soniamitchell
Fix CI - install R API
149cb3d
@soniamitchell
Fix CI - install remotes
8a7ca41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@soniamitchell