Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Configurable REPL prompt #460

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Configurable REPL prompt #460

wants to merge 1 commit into from

Conversation

coke
Copy link
Contributor

@coke coke commented Jan 10, 2025

(when merging, please squash into a single commit)

@coke coke marked this pull request as ready for review January 10, 2025 23:09
@coke coke requested a review from codesections as a code owner January 10, 2025 23:09
@patrickbkr
Copy link
Member

I wonder, how does this relate to the recent let's-make-a-new-REPL work?

@lizmat
Copy link
Collaborator

lizmat commented Jan 17, 2025

@patrickbkr I've incorporated this proposal in the Prompt module: lizmat/Prompt@408130e

Working on the documentation now, which I will steal from @coke :-)

@lizmat
Copy link
Collaborator

lizmat commented Jan 17, 2025

Looking at this more in depth, I think the \t for date time information is wrong, as it clashes syntactically with "\t" for tab. Propose to move \t to \d

@lizmat
Copy link
Collaborator

lizmat commented Jan 17, 2025

The REPL distribution now has support for the RAKUDO_REPL_PROMPT environment variable, and support for expansion of various escape sequences.

I've taken the list here, changed \t to \d, added \t for tab, and added support for some more strftime formats

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.

4 participants