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

Get rid of "retainLines" compiler option (closes #1267) #1369

Merged
merged 4 commits into from
Apr 3, 2017

Conversation

georgiy-abbasov
Copy link
Contributor

\cc @inikulin, it's just an example, how can we solve the issue by little changing callsite-record module here, here and here

@testcafe-build-bot
Copy link
Collaborator

❌ Tests for the commit c94fcde have failed. See details:

2 similar comments
@testcafe-build-bot
Copy link
Collaborator

❌ Tests for the commit c94fcde have failed. See details:

@testcafe-build-bot
Copy link
Collaborator

❌ Tests for the commit c94fcde have failed. See details:

@testcafe-build-bot
Copy link
Collaborator

❌ Tests for the commit d21548c have failed. See details:

@testcafe-build-bot
Copy link
Collaborator

❌ Tests for the commit 03c3efc have failed. See details:

@georgiy-abbasov georgiy-abbasov changed the title [WIP] Get rid of "retainLines" compiler option (closes #1267) Get rid of "retainLines" compiler option (closes #1267) Mar 31, 2017
@georgiy-abbasov
Copy link
Contributor Author

FPR

Copy link
Contributor

@inikulin inikulin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@inikulin inikulin merged commit 8e449c8 into DevExpress:master Apr 3, 2017
kirovboris pushed a commit to kirovboris/testcafe-phoenix that referenced this pull request Dec 18, 2019
…vExpress#1369)

* Get rid of "retainLines" compiler option (closes DevExpress#1267)

* Update custom "callsite-record" module

* update callsite-record to 4.0.0

* comment added
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.

3 participants