Skip to content

Remove script metadata comments#58

Merged
charlesbmi merged 2 commits intomainfrom
fix/script-deps
Dec 24, 2025
Merged

Remove script metadata comments#58
charlesbmi merged 2 commits intomainfrom
fix/script-deps

Conversation

@charlesbmi
Copy link
Collaborator

Removed script metadata comments from doppler.py. dependencies managed in pyproject.toml

Review checklist

  • All existing tests and checks pass
  • Unit tests covering the new feature or bugfix have been added
  • The documentation has been updated if necessary

Removed script metadata comments from doppler.py. dependencies managed in pyproject.toml
@qodo-code-review
Copy link

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ Recommended focus areas for review

Compatibility

Verify that removing the inline script metadata does not break any tooling or execution paths that previously relied on those comments (e.g., running the file as a standalone script, packaging/automation that parsed these headers, or runtime assumptions about requires-python / dependencies).

import marimo

__generated_with = "0.18.4"
app = marimo.App()
Dependency Source

Ensure the dependencies previously declared in the script metadata are correctly represented in pyproject.toml (including any optional/extra groups if applicable), and that pyzmq is still included where needed for this script to run.

import marimo

__generated_with = "0.18.4"
app = marimo.App()

@qodo-code-review
Copy link

qodo-code-review bot commented Dec 23, 2025

PR Code Suggestions ✨

No code suggestions found for the PR.

@charlesbmi charlesbmi merged commit 5670686 into main Dec 24, 2025
9 checks passed
@charlesbmi charlesbmi deleted the fix/script-deps branch December 24, 2025 00:37
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.

1 participant