Skip to content

New daemon CLI with circus#1213

Closed
DropD wants to merge 6 commits into
aiidateam:workflowsfrom
DropD:workflows-daemon
Closed

New daemon CLI with circus#1213
DropD wants to merge 6 commits into
aiidateam:workflowsfrom
DropD:workflows-daemon

Conversation

@DropD
Copy link
Copy Markdown
Contributor

@DropD DropD commented Mar 2, 2018

Fix #1055

What is not done:

  • Automated tests for the cli
  • The new cli is not used in the daemon test
  • I did not test actually submitting to daemons for multiple profiles simultaneously

Reviewers (@muhrin, @sphuber): please advise on whether to make tests into a separate issue & PR or to push into this one.

PS: It is really a shame that Aida has no deamon characters (or at least I don't remember any).

@sphuber
Copy link
Copy Markdown
Contributor

sphuber commented Mar 7, 2018

Closed in favor of PR #1217

@sphuber sphuber closed this Mar 7, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants