-
Notifications
You must be signed in to change notification settings - Fork 582
Make type aliases stricter #1066
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
Conversation
Let me elaborate a bit on that last point. The However, I found out that So, the dilemma I have is whether we should update those types to be more restrictive and therefore more useful, but at the same time break compatibility? Maybe compatibility is overrated in this particular case? |
Hello @pegasd |
Hey there, @florindragos !
in reality it also matches a multiline string like
|
By I guess I could update those as well, but as I said, they're very much legacy and things may break for users in unexpected ways. What do you propose? I can take a stab at |
Anything I can do to make this PR move along? |
Hi @pegasd, |
This PR makes the following changes:
\A
and\z
instead of^
and$
)Stdlib::MAC
type that were absentStdlib::Compat::*
types intact for compatibility purposes (is this correct behavior? or should I update those as well?)