Skip to content

Added all existing analyze options as arguments#93

Merged
sethblack merged 1 commit intosethblack:masterfrom
TBG-FR:new-cli-params
Feb 14, 2023
Merged

Added all existing analyze options as arguments#93
sethblack merged 1 commit intosethblack:masterfrom
TBG-FR:new-cli-params

Conversation

@TBG-FR
Copy link
Copy Markdown
Contributor

@TBG-FR TBG-FR commented Feb 13, 2023

Using the README and code informations, I added the available analyze() options/arguments as arguments for the tool/cli

Now what we could achieve by code only (output = analyze(site, sitemap, follow_links=False) or output = analyze(site, sitemap, analyze_headings=True, analyze_extra_tags=True), or both) is available from the cli

Argument Default Description
--analyze-headings False Analyze heading tags (h1-h6).
--analyze-extra-tags False Analyze other extra additional tags.
--no-follow-links True Analyze all the existing inner links as well (might be time consuming).

Examples

# original command with defaults (no headings, no extra tags, and follow links)
docker run sethblack/python-seo-analyzer http://host.docker.internal:8080/ --output-format html  > results.html

# analyze command with headings, extra tags, and follow links
docker run sethblack/python-seo-analyzer http://host.docker.internal:8080/ --analyze-headings --analyze-extra-tags --output-format html  > results.html

# analyze with headings, extra tags, but without follow links
docker run sethblack/python-seo-analyzer http://host.docker.internal:8080/ --analyze-headings --analyze-extra-tags --no-follow-links --output-format html  > results.html

# analyze without headings, without extra tags, and without follow links)
docker run sethblack/python-seo-analyzer http://host.docker.internal:8080/ --no-follow-links --output-format html  > results.html

@sethblack sethblack merged commit 3734a02 into sethblack:master Feb 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants