Skip to content
This repository has been archived by the owner on Dec 6, 2023. It is now read-only.

fix OUTPUT option description in spider_plus #713

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

Conversation

CameronTStark
Copy link

There's an issue with the spider_plus module description incorrectly stating the option name for the output path.

The current description for the spider_plus module shows:
OUTPUT_FOLDER Path of the remote folder where the dump will occur (Default: /tmp/cme_spider_plus)
See code here

However, the code looks only for OUTPUT:
context.log.info(' OUTPUT: {output}'.format(output=self.output_folder))
See code here

This PR fixes the module description to correctly show OUTPUT instead of OUTPUT_FOLDER.

@mpgn mpgn added the all good tested in my lab label Feb 12, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
all good tested in my lab
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants