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

Changes for frozen string literals #105

Merged
merged 2 commits into from
Jul 1, 2016

Conversation

twalpole
Copy link
Contributor

@twalpole twalpole commented Mar 1, 2016

Minor updates for compatibility with frozen string literals

@mattwynne
Copy link
Member

I don't think the build failures are anything you've done. More likely something to do with #103.

@brasmusson
Copy link
Contributor

Maybe it would have been better not to merge #103 as it did not updated the tests it broke

@mattwynne
Copy link
Member

Agreed - my bad. I didn't notice it had broken anything when I merged it.

@aslakhellesoy
Copy link
Contributor

How can we improve the CI notification so you're less likely to miss out on build errors @mattwynne?

@mattwynne
Copy link
Member

I'm puzzled why I didn't see the Travis check in the PR. There was something weird about that PR. NM - it's not a big deal to fix.

@mattwynne mattwynne merged commit 77021f2 into cucumber:master Jul 1, 2016
@mattwynne
Copy link
Member

Thanks @twalpole - immutability FTW! 👍

mattwynne added a commit that referenced this pull request Jul 1, 2016
mattwynne added a commit to cucumber/cucumber-ruby that referenced this pull request Jul 2, 2016
CyrilPatrice pushed a commit to PagesjaunesMobile/cucumber-ruby-core that referenced this pull request Feb 1, 2017
@lock
Copy link

lock bot commented Oct 24, 2018

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked as resolved and limited conversation to collaborators Oct 24, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants