Skip to content

Space: recursive get_children#72

Merged
PenguinBoi12 merged 2 commits into
mainfrom
space/recursive-get_children
Jul 2, 2026
Merged

Space: recursive get_children#72
PenguinBoi12 merged 2 commits into
mainfrom
space/recursive-get_children

Conversation

@PenguinBoi12

@PenguinBoi12 PenguinBoi12 commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

Adds a depth parameter to Space.get_children() to recursively collect children of sub-spaces. Defaults to 1 for backwards compatibility.

Example

space = bot.get_space("!space123:matrix.org")

for child in space.get_children():
    print(child.name)

for child in space.get_children(depth=3):
    print(child.name)

@PenguinBoi12 PenguinBoi12 self-assigned this Jul 1, 2026
@PenguinBoi12 PenguinBoi12 force-pushed the space/recursive-get_children branch from 021140b to 5979897 Compare July 1, 2026 05:01
@PenguinBoi12 PenguinBoi12 marked this pull request as ready for review July 1, 2026 16:08
@PenguinBoi12 PenguinBoi12 requested a review from chrisdedman July 1, 2026 16:08
@PenguinBoi12 PenguinBoi12 added refactor feature A new feature and removed refactor labels Jul 2, 2026
chrisdedman
chrisdedman previously approved these changes Jul 2, 2026

@chrisdedman chrisdedman left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM.

@PenguinBoi12 PenguinBoi12 force-pushed the space/recursive-get_children branch from 66eb47e to b383f74 Compare July 2, 2026 21:43
@PenguinBoi12 PenguinBoi12 requested a review from chrisdedman July 2, 2026 21:45

@chrisdedman chrisdedman left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@PenguinBoi12 PenguinBoi12 merged commit ad6a501 into main Jul 2, 2026
4 checks passed
@PenguinBoi12 PenguinBoi12 deleted the space/recursive-get_children branch July 2, 2026 21:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature A new feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants