Skip to content

A new easyblock for Scipion that handles version 2 and newer.#1847

Merged
smoors merged 9 commits intoeasybuilders:developfrom
akesandgren:scipion
Jun 26, 2020
Merged

A new easyblock for Scipion that handles version 2 and newer.#1847
smoors merged 9 commits intoeasybuilders:developfrom
akesandgren:scipion

Conversation

@akesandgren
Copy link
Copy Markdown
Contributor

No support for older versions.

@akesandgren akesandgren added this to the 4.x milestone Nov 11, 2019
Comment thread easybuild/easyblocks/s/scipion.py Outdated
Comment thread easybuild/easyblocks/s/scipion.py Outdated
boegel
boegel previously requested changes Nov 15, 2019
Copy link
Copy Markdown
Member

@boegel boegel left a comment

Choose a reason for hiding this comment

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

mostly tackled by akesandgren#10

Comment thread easybuild/easyblocks/s/scipion.py Outdated
Comment thread easybuild/easyblocks/s/scipion.py Outdated
Comment thread easybuild/easyblocks/s/scipion.py Outdated
Comment thread easybuild/easyblocks/s/scipion.py Outdated
@akesandgren
Copy link
Copy Markdown
Contributor Author

@boegel Anything else I should fix in this PR?

@akesandgren
Copy link
Copy Markdown
Contributor Author

Also, this superseedes #846

@akesandgren
Copy link
Copy Markdown
Contributor Author

@boegel any more comments on this one??

smoors
smoors previously requested changes Apr 14, 2020
Comment thread easybuild/easyblocks/s/scipion.py Outdated
Comment thread easybuild/easyblocks/s/scipion.py Outdated
Comment thread easybuild/easyblocks/s/scipion.py Outdated
@hound hound bot deleted a comment from akesandgren May 13, 2020
@hound hound bot deleted a comment from ocaisa May 13, 2020
@akesandgren akesandgren dismissed stale reviews from smoors and boegel June 26, 2020 08:20

Suggestions incorporated

Copy link
Copy Markdown
Contributor

@smoors smoors left a comment

Choose a reason for hiding this comment

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

lgtm

@smoors
Copy link
Copy Markdown
Contributor

smoors commented Jun 26, 2020

closing-reopening

@smoors smoors closed this Jun 26, 2020
@smoors smoors reopened this Jun 26, 2020
@smoors smoors modified the milestones: 4.x, next release (4.2.2?) Jun 26, 2020
@smoors
Copy link
Copy Markdown
Contributor

smoors commented Jun 26, 2020

Going in, thanks @akesandgren!

@smoors smoors merged commit b7927a1 into easybuilders:develop Jun 26, 2020
@akesandgren akesandgren deleted the scipion branch June 27, 2020 06:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants