Skip to content

feat: add peat pull --skip-scan flag#63

Open
jgallagheriii wants to merge 1 commit into
sandialabs:mainfrom
jgallagheriii:feat/pull-skip-scan
Open

feat: add peat pull --skip-scan flag#63
jgallagheriii wants to merge 1 commit into
sandialabs:mainfrom
jgallagheriii:feat/pull-skip-scan

Conversation

@jgallagheriii

Copy link
Copy Markdown

Add peat pull --skip-scan flag

Description

  • Add --skip-scan CLI flag to bypass scan phase and pull directly from hosts defined in a config file; removes requirement for -i/-f when a config with a hosts list is supplied
  • Per-host peat_module mappings in the hosts list are respected; a single -d device type can be used as a fallback
  • Add pull_skip_scan setting to Configuration and both example config files
  • Add unit tests for skip-scan logic in pull_api and cli_args

Related Issue

Closes #60

Checklist

Please check the following items as they're completed.
Completion of all checklist items signals to maintainers that a PR is fully ready for review.

  • This PR conforms to the process detailed in the Contributing Guide
  • I have included no proprietary/sensitive information in my code
  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have tested my code
  • I have added a description of my changes to the changelog in CHANGELOG.rst
  • Apply appropriate Tags to this PR, if any (e.g. bugfix, enhancement(feature), documentation, etc.)
  • Removing "Draft" status from the PR (if applicable).

@jgallagheriii jgallagheriii force-pushed the feat/pull-skip-scan branch 2 times, most recently from b6b2d2f to 20a3c82 Compare May 8, 2026 16:29
Comment thread peat/api/pull_api.py Outdated
Comment thread peat/api/pull_api.py Outdated
Comment thread peat/api/pull_api.py
Comment thread peat/cli_args.py
Comment thread peat/cli_main.py
Comment thread peat/cli_main.py
Comment thread peat/settings.py
Comment thread tests/api/test_pull_api.py Outdated
Comment thread tests/api/test_pull_api.py Outdated
@GhostofGoes

Copy link
Copy Markdown
Contributor

Overall, this looks good, and adds a long-overdue feature to the CLI. Good work!

- Add --skip-scan CLI flag to bypass scan phase and pull directly from
  hosts defined in a config file; removes requirement for -i/-f when a
  config with a hosts list is supplied
- Per-host peat_module mappings in the hosts list are respected; a single
  -d device type can be used as a fallback
- Add pull_skip_scan setting to Configuration and both example config files
- Add unit tests for skip-scan logic in pull_api and cli_args

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@jgallagheriii jgallagheriii force-pushed the feat/pull-skip-scan branch from 20a3c82 to 6b7f87e Compare May 15, 2026 20:19
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.

[Feature] Skip scanning for pulls with configs

2 participants