Conversation
Covers the OOP features merged in radon PRs #219 (inheritance, super(), C3 MRO) and #220 (enforced public/private/protected access modifiers, abstract classes/methods) -- new sections in classes.md, plus a landing page feature-list mention. Every code example was run against the actual interpreter to verify its claimed output.
Covers async fun/await, spawn()/sleep()/gather() from radon PR #222 (async/await support, currently in review -- not yet merged to radon's master): short intros in functions.md and built-in-functions.md, and a full guide on a new async.md page (nav entry added to mkdocs.yml) covering the closure-capture pitfall in loops and a worked sequential-vs-concurrent timing example. Every code example was run against the actual interpreter to verify its claimed output.
…keys
Found while auditing recent merged PRs against the docs (nothing else
was missing -- __destructor__, del, HashMap/array unpacking, static
methods, and the objective-method-syntax migration were all already
covered accurately):
- del on a hashmap key (del hm["key"]) works but wasn't shown, only the
array-index case was.
- Two existing examples claimed hashmap print output with double-quoted
keys ({"color": "red", ...}); the interpreter actually prints
single-quoted keys ({'color': "red", ...}), verified by running both
snippets directly.
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
1 participant
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.
Summary
Three commits, grouped by feature:
classes.md:super(), multiple/multilevel/hybrid inheritance and a note on C3-linearized MRO, enforcedpublic/private/protected(including theRuntimeErrorthey raise), andabstract class/abstract methods.master. Short intros infunctions.md/built-in-functions.md, plus a full newasync.mdpage (added to nav) coveringspawn()/sleep()/gather(), a worked sequential-vs-concurrent timing example, and the closure-capture pitfall you can hit building tasks inside aforloop.delon a hashmap key wasn't documented (only the array case was), and two existing examples claimed the wrong hashmapprint()quoting.Verification
Every code snippet across all three commits was run against the actual
radoninterpreter to confirm its claimed output before being committed — including the two inaccuracies in commit 3, which were caught this way. Site builds clean undermkdocs build --strict.One thing to flag before merging
Commit 2 (async/await) documents radon#222, which is still open. If this PR merges before that one, the live docs site will describe a language feature that isn't in a released Radon build yet. Options: