Skip to content

Closes #21: [Enhancement]: Restructure examples — flat named scripts and single README index - #30

Merged
rosspeili merged 4 commits into
ARPAHLS:mainfrom
evgenyponomarev:sloppy/issue-21-c52eab57e37e
Aug 26, 2026
Merged

Closes #21: [Enhancement]: Restructure examples — flat named scripts and single README index#30
rosspeili merged 4 commits into
ARPAHLS:mainfrom
evgenyponomarev:sloppy/issue-21-c52eab57e37e

Conversation

@evgenyponomarev

Copy link
Copy Markdown
Contributor

Closes #21.

Verified against the pinned tree: the patch applies cleanly and the issue's post-fix check passes.

Written by an AI coding agent (Sloppy) and opened under my account.

@rosspeili

Copy link
Copy Markdown
Contributor

Good work on the restructure @evgenyponomarev, and it matches #21. Before merge please:

  1. [Unreleased] CHANGELOG entry (examples flat layout)
  2. docs/integrations/README.md, drop “(after flat restructure)”
  3. tests/conftest.py, fix or remove stale example_scripts */main.py glob
  4. Please comment on [Enhancement]: Restructure examples — flat named scripts and single README index #21 so you can be assigned to it, and try to claim issues before opening a PR to avoid dublicate PRs.

Other than that, LGTM, thanks and welcome <3

@evgenyponomarev
evgenyponomarev force-pushed the sloppy/issue-21-c52eab57e37e branch from 03e7871 to c075add Compare August 24, 2026 14:42
applied review feedback
verified: build and tests pass
applied review feedback
verified: build and tests pass
@rosspeili

Copy link
Copy Markdown
Contributor

Thanks @evgenyponomarev, strong work on #21, the flat layout and README index look great and your review fixes landed.

Before merge, three stale links still point at the removed examples/04-sequencer-pipeline/ path:

  • docs/integrations/README.md
  • integrations/skillware/README.md
  • docs/guides/aura-on-skillware.md (table row examples/04-*)

Please update those to examples/sequencer_pipeline.py. Optional: keep smoke coverage for 05–08 by globbing both examples/*.py and examples/*/main.py.

Once that's in, LGTM to merge. Rebase onto current main if needed (post-#45). <3

applied review feedback
verified: build and tests pass
@rosspeili

Copy link
Copy Markdown
Contributor

Thanks @evgenyponomarev, all #21 feedback addressed, LGTM and merging as is, closes #21.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Enhancement]: Restructure examples — flat named scripts and single README index

2 participants