Skip to content

build: fix inclusion of aw-cli command and click dep - #127

Merged
ErikBjare merged 1 commit into
ActivityWatch:masterfrom
carlwgeorge:fix-aw-cli
Feb 5, 2025
Merged

ErikBjare merged 1 commit into
ActivityWatch:masterfrom
carlwgeorge:fix-aw-cli

Conversation

@carlwgeorge

@carlwgeorge carlwgeorge commented Dec 20, 2024

Copy link
Copy Markdown
Contributor

The aw-cli entry point is part of the wheel, but is non-functional without the aw_cli package and the click dependency.


Important

Fix aw-cli command by including aw_cli package and adding click dependency in pyproject.toml.

  • Dependencies:
    • Add click to [tool.poetry.dependencies] in pyproject.toml to ensure aw-cli command functionality.
  • Packages:
    • Include aw_cli in [tool.poetry.packages] in pyproject.toml to make the aw-cli entry point functional.

This description was created by Ellipsis for 0a64f19. It will automatically update as commits are pushed.

The aw-cli entry point is part of the wheel, but is non-functional
without the aw_cli package and the click dependency.

@ellipsis-dev ellipsis-dev Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

👍 Looks good to me! Reviewed everything up to 0a64f19 in 7 seconds

More details
  • Looked at 20 lines of code in 1 files
  • Skipped 0 files when reviewing.
  • Skipped posting 1 drafted comments based on config settings.
1. pyproject.toml:33
  • Draft comment:
    Adding 'click' as a dependency is correct to ensure the functionality of the aw-cli command, as it is required for command-line interface operations.
  • Reason this comment was not posted:
    Confidence changes required: 0%
    The addition of the 'click' dependency is necessary for the functionality of the aw-cli command, as described in the PR. This is a valid change.

Workflow ID: wflow_jvnjKYcYUuFfcZlX


You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.

@ErikBjare

Copy link
Copy Markdown
Member

Oops, thanks!

@ErikBjare ErikBjare changed the title Fix aw-cli command build: fix inclusion of aw-cli command and click dep Feb 5, 2025
@ErikBjare
ErikBjare merged commit 23a0108 into ActivityWatch:master Feb 5, 2025
@carlwgeorge
carlwgeorge deleted the fix-aw-cli branch February 5, 2025 20:21
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