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

Pin clap dependency in Cargo.toml #1232

Merged
merged 3 commits into from
Jun 14, 2022
Merged

Pin clap dependency in Cargo.toml #1232

merged 3 commits into from
Jun 14, 2022

Conversation

o0Ignition0o
Copy link
Contributor

Workaround against #1231

A minor release of Clap occured yesterday; which introduced a breaking change.

This might lead cargo scaffold users to hit a panic a runtime when the router tries to parse env variables and arguments.

This patch Pins the clap dependency to the version that was available before the release, until the root cause is found and fixed.

Workaround against #1231

A minor release of Clap occured yesterday; which introduced a breaking change.

This might lead cargo scaffold users to hit a panic a runtime when the router tries to parse env variables and arguments.

This patch Pins the clap dependency to the version that was available before the release, until the root cause is found and fixed.
@netlify
Copy link

netlify bot commented Jun 14, 2022

Deploy Preview for apollo-router-docs canceled.

Name Link
🔨 Latest commit ab3cd3a
🔍 Latest deploy log https://app.netlify.com/sites/apollo-router-docs/deploys/62a855278b84800009091894

@github-actions

This comment has been minimized.

@o0Ignition0o o0Ignition0o marked this pull request as ready for review June 14, 2022 09:31
@o0Ignition0o o0Ignition0o merged commit 736c00c into main Jun 14, 2022
@o0Ignition0o o0Ignition0o deleted the igni/pin_clap branch June 14, 2022 10:01
This was referenced Jun 14, 2022
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.

3 participants