search

dnanexus-archive / dx-cwl

Import and run CWL workflows on DNAnexus (alpha)
Apache License 2.0
13 stars 6 forks source link

can use provided DNANexus instance when compile-tool #19

Open hurchu opened 6 years ago

hurchu commented 6 years ago

I added one optional parameter dnanexus_instance. When run compile-tool, we can provide the instance the user prefers.

geetduggal commented 6 years ago

Hi Junfei, thanks for your PRs! I noticed you were updating the branch, so just let me know when you're ready and I'll review it to incorporate it as a part of the tool. Thanks again

hurchu commented 6 years ago

Hi Geet,

I've done some updates to support:

  1. publish apps
  2. build workflow with apps
  3. add suggestions
  4. try no use compile_inline_js_wf_input to build extra applets

I'm ready to create pull request from my master.

Best,

Junfei

On Wed, Jan 31, 2018 at 5:03 PM, Geet Duggal notifications@github.com wrote:

Hi Junfei, thanks for your PRs! I noticed you were updating the branch, so just let me know when you're ready and I'll review it to incorporate it as a part of the tool. Thanks again

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/dnanexus/dx-cwl/pull/19#issuecomment-362100652, or mute the thread https://github.com/notifications/unsubscribe-auth/AMnCpLXrozdAbp4hmukZ_-K6i6vAp5noks5tQPEsgaJpZM4Ryl0G .

-- Junfei Zhu

hurchu commented 6 years ago

Hi Geet, It's ready for you to review. Thanks~