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

Failing helper overload spec #1655

Closed
wants to merge 2 commits into from

Conversation

baelter
Copy link
Contributor

@baelter baelter commented Oct 24, 2020

No description provided.

@jkowens
Copy link
Member

jkowens commented Oct 25, 2020

It doesn't look like this spec is failing, can you double check this? Thank you for working on this, a failing spec will be very helpful.

@baelter
Copy link
Contributor Author

baelter commented Oct 25, 2020

File had wrong name...

@jkowens
Copy link
Member

jkowens commented Oct 26, 2020

@namusyaka do you have any thoughts on this? Should we plan on reverting the change in #1213?

@namusyaka
Copy link
Member

@jkowens Yes, we should revert the change, it shouldn't be merged into v2.1.0 (probably v3.0.0 seems good).

namusyaka added a commit that referenced this pull request Nov 4, 2020
@namusyaka
Copy link
Member

Merged in #1662 , thanks.

@namusyaka namusyaka closed this Nov 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants