Skip to content

Conversation

@dom96
Copy link
Contributor

@dom96 dom96 commented Jul 8, 2025

Summary

Updated the Python docs to refer to cf-requirements.txt instead of requirements.txt and added a new section about pywrangler to point people to it instead of the old Built-in package pathway.

Documentation checklist

  • The documentation style guide has been adhered to.
  • If a larger change - such as adding a new page- an issue has been opened in relation to any incorrect or out of date information that this PR fixes.
  • Files which have changed name or location have been allocated redirects.

@hyperlint-ai
Copy link
Contributor

hyperlint-ai bot commented Jul 8, 2025

Howdy and thanks for contributing to our repo. The Cloudflare team reviews new, external PRs within two (2) weeks. If it's been two weeks or longer without any movement, please tag the PR Assignees in a comment.

We review internal PRs within 1 week. If it's something urgent or has been sitting without a comment, start a thread in the Developer Docs space internally.


PR Change Summary

Renamed requirements file and introduced a new section on pywrangler for managing Python packages in Workers.

  • Updated documentation to refer to cf-requirements.txt instead of requirements.txt
  • Added a new section on pywrangler for package management
  • Deprecated the use of explicitly defined built-in packages in favor of pywrangler

Modified Files

  • src/content/docs/workers/languages/python/how-python-workers-work.mdx
  • src/content/docs/workers/languages/python/packages/index.mdx

How can I customize these reviews?

Check out the Hyperlint AI Reviewer docs for more information on how to customize the review.

If you just want to ignore it on this PR, you can add the hyperlint-ignore label to the PR. Future changes won't trigger a Hyperlint review.

Note specifically for link checks, we only check the first 30 links in a file and we cache the results for several hours (for instance, if you just added a page, you might experience this). Our recommendation is to add hyperlint-ignore to the PR to ignore the link check for this PR.

@github-actions
Copy link
Contributor

github-actions bot commented Jul 8, 2025

This pull request requires reviews from CODEOWNERS as it changes files that match the following patterns:

Pattern Owners
/src/content/docs/workers/ @cloudflare/workers-docs, @GregBrimble, @irvinebroque, @mikenomitch, @korinne, @WalshyDev, @cloudflare/deploy-config, @cloudflare/pcx-technical-writing, @kodster28, @cloudflare/wrangler, @cloudflare/workers-runtime-1, @cloudflare/wrangler

@dom96 dom96 force-pushed the dominik/pywrangler-docs branch from 9f68aaf to c06e0b2 Compare July 9, 2025 10:13
@dom96 dom96 requested review from a team and removed request for a team July 9, 2025 10:15
1. Wrangler uploads your Python code and your `requirements.txt` file to the Workers API.
2. Cloudflare sends your Python code, and your `requirements.txt` file to the Workers runtime to be validated.
3. Cloudflare creates a new v8 isolate for your Worker, and automatically injects Pyodide plus any packages you’ve specified in your `requirements.txt` file.
1. Wrangler uploads your Python code and your `cf-requirements.txt` file to the Workers API.
Copy link
Contributor

Choose a reason for hiding this comment

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

Do we want the cf-requirements references without any other context for them? Prob not

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I removed references to it, though this section is specifically about how built-in packages are handled. Should I just remove this section, for now at least?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'll keep it in for now, we can remove it later in a separate PR if needed.

@dom96 dom96 force-pushed the dominik/pywrangler-docs branch from c06e0b2 to cc9c703 Compare July 17, 2025 15:54
@dom96 dom96 merged commit 964c17d into production Jul 24, 2025
11 checks passed
@dom96 dom96 deleted the dominik/pywrangler-docs branch July 24, 2025 10:47
sdnts pushed a commit to sdnts/cloudflare-docs that referenced this pull request Jul 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

product:workers Related to Workers product size/s

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants