Skip to content

Conversation

@tobias-trozowski
Copy link
Contributor

Description

This PR adds automated testing and release-deployment (github releases), coverage generation, zpk generation and deployment and others. For detailed list of changes see below.

THIS PR REQUIRES REPOSITORY OWNER ACTION

Actions required by repository owner

  • Enable this repository in travis-ci.org.
  • Enable this repository in coveralls.io
  • replace {ENCRYPTED GITHUB OAUTH TOKEN} in .travis.yml with your encryped personal access token

Generate Access Token

Create a new personal access token and grant access to:

  • repo:status
  • repo_deployment
  • public_repo

Encrypt Access Token

Install the TravisCI command line tool
Run travis encrypt {YOUR PERSONAL GITHUB OAUTH TOKEN}

Changelog

Added

  • automated testing for php versions 5.3, 5.4, 5.5, 5.6, 7.0, and 7.1
  • automated coverage generation and upload (coveralls.io)
  • automated ZPK generation
  • automated ZPK deployment on releases
  • build.xml (phing) to build/generate the ZPK
  • added namespaces for tests

Deprecated

  • Nothing.

Removed

  • development files from deployment.properties
  • TestSuite classes as they are defined in phpunit.xml.dist now

Fixed

  • fixed file paths to match PSR-4 standard
  • fixed unit tests where $log->info() was expected but never called (this leads to an error in __destruct in Zend\Logger)
  • fixed $deployment->getCurrentAction when there is no file in trace item

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

added phpunit config
added coveralls config
# Conflicts:
#	composer.json
#	phpunit.xml.dist
…d $log->info was not called

fixed doc-blocks
replaced TestSuite classes with proper xml config
added badges to readme
changed update-url to update.json
fixed composer warning "deprecated option "installed""
disabled lowest/latest DEPS environtment as we are using a fixed version of zendframework
composer now requires specific components of zendframework instead of all
@janatzend janatzend merged commit 0d6774b into zend-patterns:master Jul 17, 2017
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