8000 modify wf actions `.yml` by yibeichan · Pull Request #35 · nipype/pydra-tutorial · GitHub
[go: up one dir, main page]
Skip to content

modify wf actions .yml #35

New 8000 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
Closed
wants to merge 27 commits into from
Closed

modify wf actions .yml #35

wants to merge 27 commits into from

Conversation

yibeichan
Copy link
Collaborator
  1. switch from setup-python to setup-miniconda

datalad needs git-annex, which can only be installed through conda not pip

  1. add etc/environment.yml for conda env pydra-tutorial
  2. remove requirements.txt

yibeichan added 24 commits July 27, 2022 16:35
@yibeichan
rename the sixth tutorial
57b27ed
@yibeichan
update 6th tutorial title
02a852d
@yibeichan
create 7th tutorial
0f6652c
@yibeichan
add datalad
4b4ed06
@yibeichan
add 7th tutorial
4ca4477
@yibeichan
add 7th tutorial
435d771
@yibeichan
fix typo
45b849f
@yibeichan
add test notebook
bf37cd3
@yibeichan
testing
b6f9fb0
@yibeichan
testing
e6c8db7
@yibeichan
test workflow
2c7e5fc
@yibeichan
update tests
58417ba
@yibeichan
add correct notebook
3e5b0de
@yibeichan
add correct notebook
9a2e9af
@yibeichan
add correct notebook
7363d70
@yibeichan
improve the workflow
8d0ef32
@yibeichan
fix minor errors
3c142c1
@yibeichan
fix subj_id index issue
43f31f0
@yibeichan
update notebooks.yml
412f311 
1. change setup-python to setup-miniconda
2. set up conda env from `.yml`
@yibeichan
update gh-pages.yml
02a90bf 
1. change setup-python to setup-miniconda
2. setup conda env pydra-tutorial
@yibeichan
Merge branch 'git-wf-actions' of github.com:Yibeichan/pydra-tutorial …
bc67a9e 
…into git-wf-actions
@yibeichan
modify wf actions
a87403f
@yibeichan
fix incompatible packages
ef0ef26
@yibeichan
fix minor errors
dc36fa2
@yibeichan
Copy link
Collaborator Author

@effigies sorry I didn't know that even if I switch to a new branch, all commits I have from different branches will still appear in one place. Would you like to review the two .yml files I modified?

@djarecka
Copy link
Contributor
djarecka commented Sep 2, 2022 •
edited
Loading

@yibeichan - this PR contain all the changes rom #34, am I right?

not sure why it doesn't run GA for your last commit...

@yibeichan
Copy link
Collaborator Author

it looks so. Here it shows 24 commits, but no testing is going on. (I'm a bit confused about how different branches work here

effigies
effigies reviewed Sep 2, 2022
View reviewed changes
Copy link
Contributor
@effigies effigies left a comment

Choose a reason for hiding this comment

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

A couple syntax errors.

effigies
effigies reviewed Sep 2, 2022
View reviewed changes
@effigies effigies mentioned this pull request Sep 2, 2022
Merged
@yibeichan
Copy link
Collaborator Author

This one was fixed in #36 so we can close it

@yibeichan yibeichan closed this Sep 9, 2022
@yibeichan yibeichan deleted the git-wf-actions branch October 12, 2022 01:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@effigies effigies effigies left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@yibeichan @djarecka @effigies
0