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

Importing CLI utilities from gerbil-utils #1027

Merged
merged 0 commits into from
Nov 28, 2023
Merged

Importing CLI utilities from gerbil-utils #1027

merged 0 commits into from
Nov 28, 2023

Conversation

fare
Copy link
Collaborator

@fare fare commented Oct 27, 2023

No description provided.

@netlify
Copy link

netlify bot commented Oct 27, 2023

Deploy Preview for elastic-ritchie-8f47f9 ready!

Name Link
🔨 Latest commit cb821d7
🔍 Latest deploy log https://app.netlify.com/sites/elastic-ritchie-8f47f9/deploys/6545c77113d1cf0008f5c68d
😎 Deploy Preview https://deploy-preview-1027--elastic-ritchie-8f47f9.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@vyzo
Copy link
Collaborator

vyzo commented Oct 27, 2023

netlify build fails

@fare fare changed the title [WIP] Importing CLI utilities from gerbil-utils Importing CLI utilities from gerbil-utils Oct 28, 2023
@fare
Copy link
Collaborator Author

fare commented Oct 28, 2023

PTAL

@fare fare force-pushed the cli branch 3 times, most recently from 5f12f13 to a8ccf80 Compare October 28, 2023 20:23
@fare fare force-pushed the cli branch 3 times, most recently from ee12990 to cdf7c5b Compare November 3, 2023 22:43
@fare fare changed the base branch from master to gxpkg-c3 November 3, 2023 22:44
@fare fare changed the base branch from gxpkg-c3 to master November 3, 2023 22:45
@fare fare changed the base branch from master to gxpkg-c3 November 4, 2023 04:24
doc/.vuepress/config.js Outdated Show resolved Hide resolved
doc/reference/std/cli/shell.md Outdated Show resolved Hide resolved
stack trace with `dump-stack-trace!`
if `(dump-stack-trace?)` is true (the default).

### dump-stack-trace?
Copy link
Collaborator

Choose a reason for hiding this comment

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

I don't like the existence of this, but I can be convinced otherwise.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

In tests, in UIs, etc., I want to be able to show the message without the backtrace. Even when I do show the backtrace, I want to repeat the non-backtrace message after the backtrace.

doc/reference/std/getopt.md Outdated Show resolved Hide resolved
doc/reference/std/getopt.md Outdated Show resolved Hide resolved
@fare fare force-pushed the cli branch 2 times, most recently from b268898 to 1afcb89 Compare November 4, 2023 23:22
@fare
Copy link
Collaborator Author

fare commented Nov 5, 2023

PTAL

Copy link
Collaborator

@vyzo vyzo left a comment

Choose a reason for hiding this comment

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

this is fine, let's just remove the word "deprecated" from anything related to the getopt move.

We might also want to to update the gxpkg new template to use the new library name.

doc/reference/std/cli/getopt.md Outdated Show resolved Hide resolved
doc/reference/std/getopt.md Outdated Show resolved Hide resolved
src/std/cli/multicall.ss Outdated Show resolved Hide resolved
@fare
Copy link
Collaborator Author

fare commented Nov 8, 2023

PTAL

@fare fare force-pushed the cli branch 3 times, most recently from 81d6bc8 to 79f0413 Compare November 27, 2023 21:17
@fare fare force-pushed the cli branch 2 times, most recently from 8e2bffa to a9ce89d Compare November 28, 2023 12:03
@fare fare merged commit a9ce89d into gxpkg-c3 Nov 28, 2023
4 of 8 checks passed
@fare fare deleted the cli branch November 28, 2023 12:15
@fare fare restored the cli branch November 28, 2023 12:16
@fare
Copy link
Collaborator Author

fare commented Nov 28, 2023

This got merged automatically because of the "wrong" notion of base branch after I reorganized the branches. Reopening as #1057.

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