Skip to content

Extension example parsing - #70

Merged
jadeddelta merged 4 commits into
autodocfrom
extension-example-parsing
Jul 9, 2026
Merged

Extension example parsing#70
jadeddelta merged 4 commits into
autodocfrom
extension-example-parsing

Conversation

@becky-gilbert

Copy link
Copy Markdown

Fixes #60

@jadeddelta this is my attempt at doing what you suggested here, namely:

first finding all extension mentions, and then doing a catchall for the plugin regex (with deduping)

Any feedback is welcome, but in particular - I'm not entirely sure that I got the intended behavior right. Maybe the first thing to do is have a look at my test example files and the test assertions on each, to see if that's how you think the parser should handle each case? Happy to make changes!

…nsions property, detect extensions in initJsPsych and produce error when extension is registered but never used in a trial
@changeset-bot

changeset-bot Bot commented Jul 7, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 5dd528a

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@becky-gilbert
becky-gilbert requested a review from jadeddelta July 7, 2026 00:01
@becky-gilbert becky-gilbert linked an issue Jul 7, 2026 that may be closed by this pull request

@jadeddelta jadeddelta left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

sorry for the delay, this looks great! i think you basically covered everything i asked for, just broadening the auto-infer process to include objects that mention the extension type. i'll get this merged up, thank you so much again!

@jadeddelta
jadeddelta merged commit 8479e6a into autodoc Jul 9, 2026
6 checks passed
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.

autodoc example parsing

2 participants