Skip to content

bump dependencies and update authors#22

Closed
tulioz wants to merge 1 commit intoevent-watcher-lightweight-cachefrom
update-deps
Closed

bump dependencies and update authors#22
tulioz wants to merge 1 commit intoevent-watcher-lightweight-cachefrom
update-deps

Conversation

@tulioz
Copy link
Copy Markdown
Contributor

@tulioz tulioz commented Apr 7, 2026

No description provided.

@aviator-app
Copy link
Copy Markdown

aviator-app bot commented Apr 7, 2026

Current Aviator status

Aviator will automatically update this comment as the status of the PR changes.
Comment /aviator refresh to force Aviator to re-examine your PR (or learn about other /aviator commands).

This PR was merged using Aviator.

Stack

  1. #20 add reconnection logic to event watcher thread (merged)
  2. #21 replace celery State with lightweight uuid to name cache (merged)
  3. 👉 #22 bump dependencies and update authors 👈 (this pr)

See the real-time status of this PR on the Aviator webapp.
Use the Aviator Chrome Extension to see the status of your PR within GitHub.

Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request adds a new author and updates project dependencies. However, several dependency versions specified in pyproject.toml and uv.lock do not exist on the public PyPI registry, posing a security risk and ensuring installation failures. Additionally, the uv.lock file contains non-standard metadata and incorrect dependency mappings for celery, suggesting the file is invalid and needs to be regenerated.

Comment thread pyproject.toml
Comment thread pyproject.toml
Comment thread uv.lock
Comment thread uv.lock
@tulioz tulioz force-pushed the update-deps branch 5 times, most recently from 012ac54 to 28106ca Compare April 7, 2026 23:05
@github-advanced-security
Copy link
Copy Markdown

You are seeing this message because GitHub Code Scanning has recently been set up for this repository, or this pull request contains the workflow file for the Code Scanning tool.

What Enabling Code Scanning Means:

  • The 'Security' tab will display more code scanning analysis results (e.g., for the default branch).
  • Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results.
  • You will be able to see the analysis results for the pull request's branch on this overview once the scans have completed and the checks have passed.

For more information about GitHub Code Scanning, check out the documentation.

Comment thread .github/workflows/ci.yml Fixed
Comment thread .github/workflows/ci.yml Fixed
Comment thread .github/workflows/image-build.yaml Fixed
@tulioz tulioz force-pushed the event-watcher-lightweight-cache branch from 9815901 to 1e30667 Compare April 7, 2026 23:11
@tulioz
Copy link
Copy Markdown
Contributor Author

tulioz commented Apr 7, 2026

/gemini review

Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates project dependencies, adds a new author to the configuration, and cleans up type-checking annotations across several modules. It also introduces a new configuration for GitHub Actions security linting via zizmor. Feedback was provided to use f-strings for string interpolation in redis_watcher.py to improve readability and follow modern Python idioms.

Comment thread celerymon/redis_watcher.py Outdated
@tulioz tulioz force-pushed the event-watcher-lightweight-cache branch from 1e30667 to f248e8a Compare April 7, 2026 23:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants