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

Add feature gated Cow module paths #900

Merged
merged 1 commit into from
Jul 24, 2023
Merged

Conversation

veeenu
Copy link
Contributor

@veeenu veeenu commented Jul 23, 2023

The generator currently wraps tags in an instance of alloc::borrow::Cow. This PR changes that to std::borrow::Cow conditional on the std feature flag being active.

Closes #899.

@veeenu veeenu requested a review from a team as a code owner July 23, 2023 14:01
@veeenu veeenu requested review from tomtau and removed request for a team July 23, 2023 14:01
Copy link
Contributor

@tomtau tomtau left a comment

Choose a reason for hiding this comment

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

thanks!

@tomtau tomtau merged commit da0feb0 into pest-parser:master Jul 24, 2023
9 checks passed
flyingsilverfin pushed a commit to typedb/dependencies that referenced this pull request Oct 2, 2023
## What is the goal of this PR?

We update to pest and pest-derive v2.7.4, which among other things
purports to fix the error where [deriving Parser fails on "undeclared
crate or module
`alloc`"](pest-parser/pest#899)
(pest-parser/pest#900).
flyingsilverfin pushed a commit to typedb/typeql that referenced this pull request Oct 2, 2023
## What is the goal of this PR?

We update to pest and pest-derive v2.7.4, which among other things purports to fix the error where [deriving Parser fails on "undeclared crate or module `alloc`"](pest-parser/pest#899) (pest-parser/pest#900).

## What are the changes implemented in this PR?

We also clean up some compile warnings (unused imports and hashes).
flyingsilverfin pushed a commit to typedb/typedb-driver that referenced this pull request Oct 2, 2023
## What is the goal of this PR?

We update to pest and pest-derive v2.7.4, which among other things purports to fix the error where [deriving Parser fails on "undeclared crate or module `alloc`"](pest-parser/pest#899) (pest-parser/pest#900).

## What are the changes implemented in this PR?

We also update README URLs and project descriptions.
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.

Expansion error in pest-derive with tags
2 participants