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

Small cleanup refactoring #903

Merged
merged 1 commit into from
Jan 8, 2015
Merged

Small cleanup refactoring #903

merged 1 commit into from
Jan 8, 2015

Conversation

Vinai
Copy link
Contributor

@Vinai Vinai commented Jan 5, 2015

This PR against the develop branch replaces #750.

It contains the following refactorings:

  • Fix typo in method name
  • Change interpret method return value to conform with Layout\ReaderInterface

@Vinai
Copy link
Contributor Author

Vinai commented Jan 5, 2015

The integration test failure is because of the issue fixed in #913, not because of this PR.

@vpelipenko
Copy link
Contributor

@Vinai, #913 is merged, please, update this pull request for green tests.

@vpelipenko
Copy link
Contributor

@Vinai, could you remove the last empty commit from this pull request? To re-run tests you just need to click "Restart Job" button on job page (look at https://travis-ci.org/magento/magento2/jobs/45948063 for example). We would like to prevent empty commits in our development history. The other way to run tests on updated code is syncing your branch with latest changes from upstream repository. For more information see - https://help.github.com/articles/syncing-a-fork/

@vpelipenko vpelipenko self-assigned this Jan 8, 2015
* Fix typo in method name
* Change interpret  method return value to conform with Layout\ReaderInterface
@Vinai
Copy link
Contributor Author

Vinai commented Jan 8, 2015

Sure, I updated the PR with a squashed commit.

@vpelipenko
Copy link
Contributor

CR: passed
Builds: green
Resolution: OK to merge

@vpelipenko vpelipenko added CS and removed bug report labels Jan 8, 2015
vpelipenko added a commit that referenced this pull request Jan 8, 2015
Small cleanup refactoring (MAGETWO-32404)
@vpelipenko vpelipenko merged commit 7593a65 into magento:develop Jan 8, 2015
@Vinai Vinai deleted the view-page-config-reader-body branch January 8, 2015 15:57
magento-team pushed a commit that referenced this pull request Mar 14, 2017
[Epam] Extend functional test sprint 3

- MTO-102: [Test] Captcha on register customer
- MTO-136: [Test] [Test] Captcha on Contact Us form
- MTO-144: [Test] Captcha and lockout for Storefront login page
- MTO-121: [Variation] Move Anchored Category with Products (cron is ON, "Update on Save")
- MTO-134: [Variation] Add Simple Product with required Custom Options to Cart
okorshenko pushed a commit that referenced this pull request Nov 27, 2017
 - Merge Pull Request magento-engcom/magento2ce#903 from RomaKis/magento2:9515
 - Merged commits:
   1. 97d449c
   2. 5c3d295
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.

3 participants