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

fix test scripts and fix compilation errors for certain feature combinations #771

Merged
merged 2 commits into from May 1, 2021

Conversation

BurntSushi
Copy link
Member

@BurntSushi BurntSushi commented May 1, 2021

While the test scripts are running in CI, if any of their commands
fail, they don't actually fail the build.

We also fix the underlying errors too.

Fixes #769, Fixes #770

While these test scripts are running in CI, if any of their commands
fail, they don't actually fail the build.
When only the unicode-perl feature is enabled, regex-syntax would fail
to build. It turns out that 'cargo fix' doesn't actually fix all
imports. It looks like it only fixes things that it can build in the
current configuration.

Fixes #769, Fixes #770
@BurntSushi BurntSushi merged commit a8554b3 into master May 1, 2021
@BurntSushi BurntSushi deleted the ag/fix-feature-builds branch May 1, 2021 22:52
@BurntSushi
Copy link
Member Author

This PR is in regex 1.5.3 on crates.io.

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.

regex-syntax fails to compile with unicode-perl feature Some combinations of features fail to build
1 participant