Skip to content

Conversation

@rakibulhaq
Copy link
Owner

No description provided.

Copilot AI review requested due to automatic review settings June 29, 2025 16:00
@rakibulhaq rakibulhaq linked an issue Jun 29, 2025 that may be closed by this pull request
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Adds a new command‐line interface for running PySentinel in both synchronous and asynchronous modes, along with tests and documentation updates.

  • Introduce pysentinel script entry point and CLI module (pysentinel/cli/cli.py).
  • Add comprehensive pytest coverage for CLI behavior.
  • Bump version in pyproject.toml and update README.md with installation & usage instructions.

Reviewed Changes

Copilot reviewed 23 out of 26 changed files in this pull request and generated no comments.

Show a summary per file
File Description
pysentinel/cli/cli.py New CLI with sync/async startup, config validation
tests/pysentinel/cli/test_cli.py Test suite for CLI commands and error conditions
pyproject.toml Version bump to 0.1.4 and added pysentinel script
README.md Extended with CLI installation and usage sections
* Other files: consistent quote style and trailing commas
Files not reviewed (1)
  • .idea/workspace.xml: Language not supported
Comments suppressed due to low confidence (2)

README.md:156

  • The code fence under "Install from PyPI (Recommended)" is opened but not closed, and an extra empty fence follows. Collapse to a single properly fenced block containing the install commands.
```bash

README.md:159

  • Remove this redundant opening fence and ensure the block is closed with ``` on its own line after the install commands.
```bash

@rakibulhaq rakibulhaq merged commit 5cf1ac5 into main Jun 29, 2025
2 checks passed
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.

Add CLI functionality

2 participants