fix: Add pylings to the Rustlings third-party exercises list - #35
fix: Add pylings to the Rustlings third-party exercises list#35webbrain-one wants to merge 1 commit into
Conversation
📝 WalkthroughWalkthroughThe changelog now includes an ChangesChangelog update
Estimated code review effort: 1 (Trivial) | ~2 minutes Possibly related issues
Suggested reviewers: Poem
🚥 Pre-merge checks | ✅ 3 | ❌ 2❌ Failed checks (2 warnings)
✅ Passed checks (3 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@CHANGELOG.md`:
- Line 10: Remove the Rustlings third-party exercises listing claim from
CHANGELOG.md until the upstream documentation change is merged. Once confirmed,
add it back only with the current project name, Pythonlings repository link, and
associated upstream issue or PR.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
|
|
||
| ### Added | ||
|
|
||
| - Listed in the Rustlings third-party exercises community list. |
There was a problem hiding this comment.
🎯 Functional Correctness | 🟡 Minor | ⚡ Quick win
🧩 Analysis chain
🏁 Script executed:
#!/bin/bash
set -euo pipefail
printf 'Tracked files matching CHANGELOG or docs:\n'
git ls-files | rg -i '(^|/)(CHANGELOG\.md|README\.md|.*docs.*|.*community.*)' || true
printf '\nCHANGELOG excerpt:\n'
sed -n '1,25p' CHANGELOG.md 2>/dev/null || true
printf '\nREADME / repo top-level name/project mentions:\n'
fd -a 'README\.md|README.*|.*rs|.*py|.*toml|.*json|.*yaml|.*yml$' . | sed 's#^\./##' | head -80
printf '\nProject names in top files:\n'
rg -n "pythonlings|pylings|python" README.md pyproject.toml Cargo.toml package.json 2>/dev/null | head -80 || true
printf '\nReferences to Rustlings community list (`#10`, rustlings, community exercises):\n'
rg -n "`#10`|community(-|_)?exercises|rustlings|communities?|issue|pull request|pr" . -S --glob '!/.git/**' --glob '!dist/**' --glob '!build/**' | head -200 || trueRepository: abhiksark/pythonlings
Length of output: 8455
🌐 Web query:
Rustlings community-exercises add project issue or pull request pythonlings
💡 Result:
Rustlings supports community-created exercises, which users can add by following the official guide and submitting a pull request to the Rustlings repository [1]. Pythonlings is an independent, Rustlings-inspired project for learning Python by fixing broken programs [2][3]. The Pythonlings project (originally named Pylings) maintains its own repository and issue tracker [2][4]. The maintainer of Pythonlings has created an issue within their own repository to track the goal of adding Pythonlings to the Rustlings list of third-party exercises [5]. To contribute to or interact with Pythonlings, you should use the issue tracker and pull request system on the Pythonlings GitHub repository [6][7]. For official Rustlings community exercise support, you should refer to the Rustlings repository documentation [1].
Citations:
- 1: https://github.com/rust-lang/rustlings/blob/4bab5966/website/content/community-exercises/index.md
- 2: https://github.com/abhiksark/pythonlings
- 3: https://pypi.org/project/pythonlings/
- 4: feat!: rename project from pylings to pythonlings #22
- 5: Add pylings to the Rustlings third-party exercises list #10
- 6: https://github.com/abhiksark/pylings
- 7: https://github.com/abhiksark/pythonlings/pulls
🌐 Web query:
site:rustlings.rust-lang.org/community-exercises pythonlings pylings
💡 Result:
The Rustlings Community Exercises page does not currently list Pythonlings or Pylings. It lists Japanese, Simplified Chinese, Ukrainian, and Korean Rustlings translations. (rustlings.rust-lang.org)
Citations:
Do not claim the Rustlings listing until it is merged.
The Rustlings community exercises page does not currently list Pythonlings. Land or link the upstream documentation change using the current project name, the Pythonlings repository link, and the associated upstream issue or PR.
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@CHANGELOG.md` at line 10, Remove the Rustlings third-party exercises listing
claim from CHANGELOG.md until the upstream documentation change is merged. Once
confirmed, add it back only with the current project name, Pythonlings
repository link, and associated upstream issue or PR.
|
Hi @webbrain-one |
Closes #10
Summary by CodeRabbit