Skip to content
This repository was archived by the owner on Apr 9, 2025. It is now read-only.

Conversation

@jilu1
Copy link
Collaborator

@jilu1 jilu1 commented Jun 24, 2020

Description (*)

Fixed Issues (if relevant)

  1. magento/magento-semver#<issue_number>: Issue title

Manual testing scenarios (*)

  1. ...

  2. ...

Questions or comments

Contribution checklist (*)

  • Pull request has a meaningful description of its purpose
  • All commits are accompanied by meaningful commit messages
  • All new or changed code is covered with unit/integration tests (if applicable)
  • All automated tests passed successfully (all builds are green)

roribio and others added 15 commits December 20, 2019 16:30
* readme:
  Update README.md: Add information about releases and versioning
* EQPS-477: Dynamically set the correct path for autoload.php based on the location of the ‘vendor’ folder

* EQPS-477: Make zend-stdlib a required dependency
Co-authored-by: Peter Dohogne <pdohogne@adobe.com>
- Added nikic/php-parser as hard dependency in composer
- Improved performance of DependencyMap creation.
- Remove stms when adding ClassMethod nodes to dependency tree
- Added better doc comments
- Addeed more aggresive AbstractApiVisitor traversal termination
- Refactored code to use switch statements
* MC-33156: Update magento-semver to be compatible with PHP7.4

* MC-33156: Update magento-semver to be compatible with PHP7.4

* MC-33156: Update magento-semver to be compatible with PHP7.4

* MC-33156: Update magento-semver to be compatible with PHP7.4
magento#38)

- Fixed Versioning level
- Updated Tests to comply with versioning change
# Conflicts:
#	composer.json
#	composer.lock
#	src/ClassHierarchy/Entity.php
@jilu1 jilu1 changed the title Mqe 2195 MQE-2195: Update WIP SVC Branch Jun 24, 2020
Copy link

@soumyau soumyau left a comment

Choose a reason for hiding this comment

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

left a few svc policy related questions and minor comment.

… repo

- Added ation group ref change case
- Added adding/removing/changing <remove> action case
Copy link

@soumyau soumyau left a comment

Choose a reason for hiding this comment

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

It'll be good to have a team discussion implementation of <remove action>. Changes if needed to be addressed in the follow-up ticket.

'changed-files.log'
),
new InputOption(
'mftf',
Copy link

@okolesnyk okolesnyk Aug 5, 2020

Choose a reason for hiding this comment

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

bin/semver compare sourcePath targetPath - runs as it is right now (without MFTF)
bin/semver compare sourcePath targetPath --analyzer=mftf - runs only MFTF
bin/semver compare sourcePath targetPath --analyzer=mftf --analyzer=api - runs only MFTF and API analyzers

soumyau and others added 8 commits September 8, 2020 17:20
1. PHP warning on ActionGroupAnalyzer.php fixed
2. implemented action sequence use case
…ases in SVC tool

added <section> entity use cases
…ases in SVC tool

Added use cases for
1. operation changed
2. operation child element added
MQE-2245: Implement additional <test> entity use cases in SVC tool
# Conflicts:
#	composer.json
#	composer.lock
soumyau and others added 21 commits September 16, 2020 13:01
…ases in SVC tool

Added use case for parameterized attribute added
MQE-2247: Implement additional <section> and <operation> entity use cases in SVC tool
Mqe 2286: Update pangolin SVC branch with Magento SVC change
MQE-2246: Implement <suite> entity use cases in MFTF SVC tool
MQE-2307: Add use cases for <remove> action in SVC
MQE-2308: MFTF SVC report does not capture per item change details
MQE-2311: Changed files list for MFTF SVC to only include MFTF files
MQE-2316: MFTF SVC does not check newly added modules
MQE-2305: Implement `-report-type=mftf` in magento-semver to launch MFTF SVC
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants