-
Notifications
You must be signed in to change notification settings - Fork 683
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
subscriber: don't gate with_ansi()
on the "ansi" feature
#3020
base: v0.1.x
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for catching this!
Github actions aren't starting. I'm guessing that's because this PR is more than 30 days old. I'll try a no-op push to the branch to see if it gets things moving. |
Head branch was pushed to by a user without write access
be13955
to
8544143
Compare
Ah, now it's stopped by "This workflow requires approval from a maintainer." @hawkw can you help? Also, auto-merge was disabled because I pushed to the branch. (No changes were made.) |
Ping? @hawkw you approved this PR but CI won't run until a maintainer intervenes. |
My bad! Running the workflow now. |
Clippy lint warnings appear to be unrelated; CI runs latest stable and at least some of the lints are things I recognize as being new in 1.80. |
The commit 1cb523b removed this cfg gate on master. However, when the change was backported in 049ad73 the docs were updated but the
cfg
change was omitted. Some more detail and links are in issue #2996.This made the docs misleading, since they say "This method itself is still available without the feature flag."
Fixes #2996.