kbase / kb_sdk

Build and test new apps for the KBase platform
http://kbase.github.io/kb_sdk_docs
MIT License
27 stars 32 forks source link

Better perl #356

Closed ialarmedalien closed 4 years ago

ialarmedalien commented 4 years ago

Part I of some improvements I made a while back to the perl sdk app templates. I'm using the fully improved perl sdk app for the Jacobson / Cytoscape app (kb_cytoscape).

Commit 1 is almost entirely whitespace changes from running the code formatter perltidy over the codebase.

Commit 2 has more simple changes, including:

Note: the tests are marked as failed because this is from my own forked repo. They have passed on my repo: https://github.com/ialarmedalien/kb_sdk/tree/better_perl (note little green tick).

MrCreosote commented 4 years ago

Sorry, I don't understand why the tests would fail in your repo but pass here...? Is it due to secrets not working on PRs?

ialarmedalien commented 4 years ago

@MrCreosote yes, exactly.

MrCreosote commented 4 years ago

I'm still working on it. There's just a lot of changes and I've had a lot of meetings the past couple days.