Fix prompt artifacts, README link, and requires-python - #804
Merged
stephengruppetta merged 1 commit intoAug 11, 2026
Merged
Conversation
- prompts.md: remove 5 stray `:contentReference[oaicite:N]{index=N}`
citation artifacts left over from pasted assistant output
- README.md: point at the real article URL (CMS post 2234, slug
`cursor-vs-copilot`) instead of `cursor-vs-github-copilot-python`,
which would 404, and match the published title
- pyproject.toml (both packages): requires-python >=3.9 -> >=3.10.
The pinned pytest==9.0.3 needs >=3.10, so dependency resolution
failed outright on the declared floor
Verified: both suites pass (16 + 13), ruff 0.14.1 format/check clean.
stephengruppetta
merged commit Aug 11, 2026
fc70d8f
into
realpython:cursor-vs-copilot-python
1 check passed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Hi @brianMutea — like #803, this targets your
cursor-vs-copilot-pythonbranch rather thanmaster, so merging it folds the fixes straight into #802. Four files, docs and metadata only — no changes to the sample code.1. Stray citation artifacts in
prompts.mdFive occurrences of
:contentReference[oaicite:N]{index=N}(lines 21, 40, 51, 65, 75), left over from pasted assistant output. They'd render as visible junk. Copilot's reviewer flagged these too.2. README pointed at a URL that would 404
The live CMS post is 2234, slug
cursor-vs-copilot— which also matches the Slug custom field on the Trello card. Title updated to the published one ("Cursor vs Copilot: Which AI Editor Is Better for Python?") at the same time.3.
requires-pythonwas unsatisfiableBoth packages declared
>=3.9while pinningpytest==9.0.3, which requires>=3.10. That isn't cosmetic — resolution fails outright:So anyone following the README hit a wall on first install. Raised the floor to
>=3.10;uv run pytestnow resolves and runs in both packages. (The source itself is genuinely 3.9-compatible — it was only the test pin that broke the claim.)Verified after the change
cursor/: 16 passed ·github-copilot/: 13 passedruff format --checkandruff checkclean under the pinned 0.14.1Not included — one for @stephengruppetta to decide
The folder is
cursor-vs-copilot-python/while the article slug iscursor-vs-copilot. The repo README says subfolders are "named after the article slug", and 6 of 8 folders I sampled match theirs exactly. But naming isn't consistent everywhere —tutorial-draftsusesarticles/cursor-vs-windsurfwhile that article's slug iscursor-vs-windsurf-python— so I've left it alone rather than rename unilaterally. It's a one-linegit mvif wanted.🤖 Prepared by RPBot on behalf of @stephengruppetta