Skip to content
This repository has been archived by the owner on Jan 30, 2020. It is now read-only.

Prepare for 3.0 release #21

Merged
merged 4 commits into from
Apr 1, 2016

Conversation

weierophinney
Copy link
Member

This pull request incorporates and builds on #20, with the following additions:

  • Removes the xml2json functionality, which has been extracted to zend-xml2json.
  • Per Feature/split zend json server #20, removes the Zend\Json\Server subcomponent, which has been extracted to zend-json-server.
  • Performs a thorough code audit. The code in this component dates back to some of the earliest in Zend Framework, and has received minimal attention over the years. A number of refactors, primarily extract methods, were performed, as well as simplification of a number of loops and conditionals. The test/JsonTest.php file was audited to extract a number of extra classes into the test/TestAsset/ folder, and several data providers and a new assertion written.
  • Switches from php-cs-fixer to phpcs.
  • Adds QA scripts to the composer.json.
  • Reduces the dependencies listed in composer.json based on the subcomponent extractions performed.
  • Edits and re-structures the documentation for publication.

webit4me and others added 4 commits March 25, 2016 08:23
- Updated all license docblocks.
- Removed test assets related to zend-json-server.
- Updated dependency list to reflect what's actually used.
- Updated test harness to use composer autoloader for bootstrapping.
- Switched to phpcs for coding standards fixes.
- Added scripts in composer.json for testing, CS checks.
- Reviewed all code for internal consistency with newer code, and to extract
  methods where complexity was observed.
- Extracted test asset classes into discrete files.
- Audited tests to ensure they detail behavior being tested.
The functionality is now part of a new package,
[zend-xml2json](https://zendframework.github.io/zend-xml2json/)
- Renamed files
- Edited all files
- Added migration guide
- Removed bookdown artifacts
- Added mkdocs configuration
- Added documentation build automation
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants