Skip to content
This repository has been archived by the owner on Sep 24, 2019. It is now read-only.

(maint) Add execute_manifest_on for local runner #23

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Iristyle
Copy link
Contributor

@Iristyle Iristyle commented Mar 8, 2018

For API parity this should probably be added as well.

(maint) Add execute_manifest_on for local runner …

  • Support full API parity, even for a local runner where it doesn't
    make a lot of sense.

    Without this support it makes it difficult to work on test suites
    that require the use of execute_manifest_on.

    Emit a warning that the "hosts" parameter is ignored given there
    is only one host and it doesn't make sense.

 - Support full API parity, even for a local runner where it doesn't
   make a lot of sense.

   Without this support it makes it difficult to work on test suites
   that require the use of execute_manifest_on.

   Emit a warning that the "hosts" parameter is ignored given there
   is only one host and it doesn't make sense.
@Iristyle Iristyle force-pushed the maint-fix-local-runner branch from faf6844 to 5a0ab45 Compare March 8, 2018 17:23
@glennsarti
Copy link
Contributor

I have rubocop fixes in PR #25

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants