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

Role definition changed #4307

Merged
merged 2 commits into from
Oct 2, 2019
Merged

Conversation

helen-dikareva
Copy link
Collaborator

@helen-dikareva helen-dikareva commented Sep 20, 2019

UPD by @AndreyBelym:

Actual changes to definitions are made by @helen-dikareva.

I've split our huge definition file into sub-definitions and added assembly system that relies on Mustache.

Tests are copied (with some modifications) from test/server/data/test-suites/typescript-defs.

@testcafe-build-bot
Copy link
Collaborator

❌ Tests for the commit 8ba9c00 have failed. See details:

@testcafe-build-bot
Copy link
Collaborator

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

@testcafe-build-bot
Copy link
Collaborator

@testcafe-build-bot
Copy link
Collaborator

Copy link
Collaborator

@kirovboris kirovboris left a comment

Choose a reason for hiding this comment

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

Discussed in private

@testcafe-build-bot
Copy link
Collaborator

aleks-pro
aleks-pro previously approved these changes Oct 1, 2019
Copy link
Collaborator Author

@helen-dikareva helen-dikareva left a comment

Choose a reason for hiding this comment

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

LGTM

@testcafe-build-bot
Copy link
Collaborator

✅ Tests for the commit 629787a have passed. See details:

@AndreyBelym AndreyBelym requested a review from kirovboris October 2, 2019 12:58
@AndreyBelym AndreyBelym merged commit bfd8985 into DevExpress:master Oct 2, 2019
kirovboris pushed a commit to kirovboris/testcafe-phoenix that referenced this pull request Dec 18, 2019
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.

5 participants