Skip to content

Ensure checksums beside sources in dumped easyconfigs#2661

Merged
boegel merged 5 commits intodevelopfrom
ocaisa-patch-1
Dec 7, 2018
Merged

Ensure checksums beside sources in dumped easyconfigs#2661
boegel merged 5 commits intodevelopfrom
ocaisa-patch-1

Conversation

@ocaisa
Copy link
Copy Markdown
Member

@ocaisa ocaisa commented Nov 15, 2018

No description provided.

Comment thread easybuild/framework/easyconfig/format/format.py
@boegel
Copy link
Copy Markdown
Member

boegel commented Nov 15, 2018

@ocaisa Can we come up with a test to check the fix?

@boegel boegel added the bug fix label Nov 15, 2018
@boegel boegel added this to the 3.8.0 milestone Nov 15, 2018
Comment thread test/framework/easyconfig.py Outdated
@boegel
Copy link
Copy Markdown
Member

boegel commented Dec 7, 2018

@ocaisa I've pushed in a test for this, please take a look? If you're OK with it and Travis gives green light, this is good to go.

@ocaisa
Copy link
Copy Markdown
Member Author

ocaisa commented Dec 7, 2018

@boegel LGTM

@boegel
Copy link
Copy Markdown
Member

boegel commented Dec 7, 2018

Thanks @ocaisa!

@boegel boegel merged commit 895b84a into develop Dec 7, 2018
@boegel boegel deleted the ocaisa-patch-1 branch December 7, 2018 14:02
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.

3 participants