Skip to content

Test Intergration with a console app who use an ApiReactive#91

Merged
nicolaserny merged 53 commits intoxcomponent:masterfrom
fle-bach:master
Feb 14, 2019
Merged

Test Intergration with a console app who use an ApiReactive#91
nicolaserny merged 53 commits intoxcomponent:masterfrom
fle-bach:master

Conversation

@fle-bach
Copy link
Copy Markdown
Contributor

  • circleci migration

Copy link
Copy Markdown

@nicolaserny nicolaserny left a comment

Choose a reason for hiding this comment

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

First review (only cake parts)

Comment thread README.md Outdated
Comment thread build.cake Outdated
Comment thread build.cake Outdated
Comment thread build.cake Outdated
Comment thread build.cake
Comment thread build.cake Outdated
Comment thread cake.scripts/utilities.cake Outdated
Copy link
Copy Markdown

@nicolaserny nicolaserny left a comment

Choose a reason for hiding this comment

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

Remove tools and lib folder + update gitignore

Comment thread docker/integration_tests/XCProjects/HelloWorldV5/Scripts/build.bat Outdated
Comment thread docker/integration_tests/dockerScripts/XCContainer/Dockerfile
Comment thread docker/integration_tests/dockerScripts/XCContainer/wait_for_it.sh
Comment thread .circleci/config.yml Outdated
Comment thread build.cake Outdated
@maurelio1234
Copy link
Copy Markdown
Member

In general looks fine for me, added a couple of comments, but they are optional.

Do not forget to squash the PR instead of simply merging

@nicolaserny nicolaserny merged commit a6ff724 into xcomponent:master Feb 14, 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.

3 participants