Skip to content

Update quickstart to support Opt::parse_args#433

Closed
gth828r wants to merge 1 commit into
cloudflare:mainfrom
gth828r:432.fix-quickstart-parse-args
Closed

Update quickstart to support Opt::parse_args#433
gth828r wants to merge 1 commit into
cloudflare:mainfrom
gth828r:432.fix-quickstart-parse-args

Conversation

@gth828r
Copy link
Copy Markdown
Contributor

@gth828r gth828r commented Oct 16, 2024

The current quickstart document suggests to use pingora v0.1, but it also suggests using the Opt::parse_args interface, which was introduced in release v0.3.0. Update the quickstart document to suggest using v0.3 when setting up the cargo.toml file.

Addresses #432

The current quickstart document suggests to use pingora v0.1, but it
also suggests using the Opt::parse_args interface, which was introduced
in release v0.3.0. Update the quickstart document to suggest using v0.3
when setting up the cargo.toml file.

Addresses cloudflare#432
@gumpt gumpt requested a review from johnhurt October 18, 2024 16:04
@johnhurt johnhurt added the Accepted This change is accepted by us and merged to our internal repo label Oct 18, 2024
johnhurt pushed a commit that referenced this pull request Oct 28, 2024
The current quickstart document suggests to use pingora v0.1, but it
also suggests using the Opt::parse_args interface, which was introduced
in release v0.3.0. Update the quickstart document to suggest using v0.3
when setting up the cargo.toml file.

Addresses #432

Includes-commit: 9153a8a
Replicated-from: #433
johnhurt pushed a commit that referenced this pull request Oct 28, 2024
The current quickstart document suggests to use pingora v0.1, but it
also suggests using the Opt::parse_args interface, which was introduced
in release v0.3.0. Update the quickstart document to suggest using v0.3
when setting up the cargo.toml file.

Addresses #432

Includes-commit: 9153a8a
Replicated-from: #433
@johnhurt
Copy link
Copy Markdown
Contributor

This has been merged as part of the weekly sync. Thank you for your contributions! 🎉

@johnhurt johnhurt closed this Oct 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Accepted This change is accepted by us and merged to our internal repo

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants