Skip to content

Add data getter to Expect - #31

Merged
matklad merged 2 commits into
rust-analyzer:masterfrom
Veetaha:master
May 25, 2022
Merged

Add data getter to Expect#31
matklad merged 2 commits into
rust-analyzer:masterfrom
Veetaha:master

Conversation

@Veetaha

@Veetaha Veetaha commented May 24, 2022

Copy link
Copy Markdown
Contributor

Closes #14.

The getter doesn't return an &str with 'static lifetime intentionally, because it makes the API more conservative, and gives more freedom to the implementation.

@Veetaha

Veetaha commented May 24, 2022

Copy link
Copy Markdown
Contributor Author

Also, it would be cool to get this released in a new version together with #27 and other cool improvements

@matklad

matklad commented May 25, 2022

Copy link
Copy Markdown
Contributor

Also, it would be cool to get this released in a new version together with #27 and other cool improvements

Could you bump the version in Cargo.toml? IIRC this crate is auto-published by CI.

@matklad

matklad commented May 25, 2022

Copy link
Copy Markdown
Contributor

bors d+

@Veetaha

Veetaha commented May 25, 2022

Copy link
Copy Markdown
Contributor Author

bors try

@Veetaha

Veetaha commented May 25, 2022

Copy link
Copy Markdown
Contributor Author

bors ping

@Veetaha

Veetaha commented May 25, 2022

Copy link
Copy Markdown
Contributor Author

@matklad bors doesn't seem to be healthy.

Anyway, I've bumped the minor version of the crate and added some missing entries to the changelog including things missed in 1.2.2 version

@matklad
matklad merged commit 21f04f3 into rust-analyzer:master May 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

Expose data stored in Expect as true public API

2 participants