Skip to content

EQPS-240: magento2-infrastructure SVC Extraction Sync #15

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

Merged
merged 7 commits into from
Nov 13, 2019

Conversation

roribio
Copy link
Contributor

@roribio roribio commented Nov 5, 2019

Scope

Story

  • EQPS-240: Incorporate latest magento2-infrastructure SVC code to public SVC repo.

The goal of this effort is to break out the SVC tool from the “magento2-infrastructure” GitHub repository into its own project so it can be open-sourced and shared with the Magento community at large.

Learn more on the project’s wiki page, here.

Changes

  • Incorporated changes from magento2-infrastructure SVC tool since after Sep. 27th, 2019.
  • Updated namespaces to reflect directory structure of independent tool. Code is now namespaced as 'Magento/SemanticVersionChecker'
  • Updated composer dependencies to include required libraries to allow independent operation. This includes wikimedia/less.php, phpstan/phpdoc-parser and symfony/console.
  • Updated composer PS4 autoload to include "Magento\\SemanticVersionChecker\\Test\\": "tests/" to allow unit tests to conform to correct directory structure.
  • Updated Unit Tests, including bootstrap and PHPUnit files.
  • Changed composer to include autoloading test namespace.

Notes

Manual Testing Scenarios

See Manual Testing Scenarios section at https://github.com/magento/magento2-infrastructure/pull/974

Related Pull Requests

@magterskine magterskine merged commit c236238 into master Nov 13, 2019
mmansoor-magento pushed a commit that referenced this pull request Nov 5, 2020
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.

2 participants