Skip to content

Replace help syntax form with ,doc REPL command.#509

Merged
toddjonker merged 2 commits intoion-fusion:mainfrom
toddjonker:cli/doc
Mar 31, 2026
Merged

Replace help syntax form with ,doc REPL command.#509
toddjonker merged 2 commits intoion-fusion:mainfrom
toddjonker:cli/doc

Conversation

@toddjonker
Copy link
Copy Markdown
Member

Known issue: Documentation isn't located for bindings defined in the current namespace, and I'm not sure why the current code doesn't work. However, this may not be a meaningful issue, and it has not worked for at least as long as we've had Gradle builds.


By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Known issue: Documentation isn't located for bindings `define`d in the current namespace, and I'm not sure why the current code doesn't work. However, this may not be a meaningful issue, and it has not worked for at least as long as we've had Gradle builds.
@SharkBaitDLS
Copy link
Copy Markdown
Contributor

Documentation isn't located for bindings defined in the current namespace

Worth adding a disabled unit test for this case now while we're here, or will that come alongside the fix?

@toddjonker
Copy link
Copy Markdown
Member Author

Documentation isn't located for bindings defined in the current namespace

Worth adding a disabled unit test for this case now while we're here, or will that come alongside the fix?

I added a test and cut issue #510 about the problem.

@toddjonker
Copy link
Copy Markdown
Member Author

To be clear, I don't have any intention of spending time on that bug, it's nice-to-have at best and I can't think of on important use case for it.

@toddjonker toddjonker merged commit 2e779ef into ion-fusion:main Mar 31, 2026
3 of 4 checks passed
@toddjonker toddjonker deleted the cli/doc branch March 31, 2026 21:04
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.

2 participants