Skip to content

Commit 9a2a152

Browse files
committed
Merge branch 'feature/58' into develop
Close #58 Fixes #57
2 parents 3a0b7ff + 5cedb1f commit 9a2a152

20 files changed

+976
-496
lines changed

.gitattributes

Lines changed: 9 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,9 @@
1-
/test export-ignore
2-
.coveralls.yml export-ignore
3-
.gitattributes export-ignore
4-
.gitignore export-ignore
5-
.travis.yml export-ignore
6-
phpcs.xml export-ignore
7-
phpunit.xml.dist export-ignore
1+
/.coveralls.yml export-ignore
2+
/.gitattributes export-ignore
3+
/.gitignore export-ignore
4+
/.travis.yml export-ignore
5+
/composer.lock export-ignore
6+
/docs/ export-ignore
7+
/phpcs.xml export-ignore
8+
/phpunit.xml.dist export-ignore
9+
/test/ export-ignore

.gitignore

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
vendor/
2-
phpunit.xml
3-
clover.xml
4-
coveralls-upload.json
1+
/clover.xml
2+
/coveralls-upload.json
3+
/phpunit.xml
4+
/vendor/

.travis.yml

Lines changed: 14 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -5,13 +5,11 @@ language: php
55
cache:
66
directories:
77
- $HOME/.composer/cache
8-
- vendor
98

109
env:
1110
global:
1211
- COMPOSER_ARGS="--no-interaction"
13-
- COVERAGE_DEPS="satooshi/php-coveralls"
14-
- LEGACY_DEPS="phpunit/phpunit doctrine/instantiator"
12+
- COVERAGE_DEPS="php-coveralls/php-coveralls"
1513

1614
matrix:
1715
include:
@@ -21,7 +19,7 @@ matrix:
2119
- php: 5.6
2220
env:
2321
- DEPS=locked
24-
- TEST_COVERAGE=true
22+
- LEGACY_DEPS="phpunit/phpunit"
2523
- php: 5.6
2624
env:
2725
- DEPS=latest
@@ -31,7 +29,7 @@ matrix:
3129
- php: 7
3230
env:
3331
- DEPS=locked
34-
- CS_CHECK=true
32+
- LEGACY_DEPS="phpunit/phpunit"
3533
- php: 7
3634
env:
3735
- DEPS=latest
@@ -41,40 +39,38 @@ matrix:
4139
- php: 7.1
4240
env:
4341
- DEPS=locked
42+
- CS_CHECK=true
43+
- TEST_COVERAGE=true
4444
- php: 7.1
4545
env:
4646
- DEPS=latest
47-
- php: hhvm
47+
- php: 7.2
4848
env:
4949
- DEPS=lowest
50-
- php: hhvm
50+
- php: 7.2
5151
env:
5252
- DEPS=locked
53-
- php: hhvm
53+
- php: 7.2
5454
env:
5555
- DEPS=latest
56-
allow_failures:
57-
- php: hhvm
58-
59-
notifications:
60-
irc: "irc.freenode.org#apigility-dev"
61-
email: false
6256

6357
before_install:
6458
- if [[ $TEST_COVERAGE != 'true' ]]; then phpenv config-rm xdebug.ini || return 0 ; fi
65-
- travis_retry composer self-update
6659

6760
install:
6861
- travis_retry composer install $COMPOSER_ARGS --ignore-platform-reqs
69-
- if [[ $TRAVIS_PHP_VERSION =~ ^5.6 ]]; then travis_retry composer update $COMPOSER_ARGS --with-dependencies $LEGACY_DEPS ; fi
62+
- if [[ $LEGACY_DEPS != '' ]]; then travis_retry composer update $COMPOSER_ARGS --with-dependencies $LEGACY_DEPS ; fi
7063
- if [[ $DEPS == 'latest' ]]; then travis_retry composer update $COMPOSER_ARGS ; fi
7164
- if [[ $DEPS == 'lowest' ]]; then travis_retry composer update --prefer-lowest --prefer-stable $COMPOSER_ARGS ; fi
7265
- if [[ $TEST_COVERAGE == 'true' ]]; then travis_retry composer require --dev $COMPOSER_ARGS $COVERAGE_DEPS ; fi
73-
- composer show
66+
- stty cols 120 && composer show
7467

7568
script:
7669
- if [[ $TEST_COVERAGE == 'true' ]]; then composer test-coverage ; else composer test ; fi
7770
- if [[ $CS_CHECK == 'true' ]]; then composer cs-check ; fi
7871

7972
after_script:
80-
- if [[ $TEST_COVERAGE == 'true' ]]; then composer upload-coverage ; fi
73+
- if [[ $TEST_COVERAGE == 'true' ]]; then travis_retry php vendor/bin/php-coveralls -v ; fi
74+
75+
notifications:
76+
email: false

CHANGELOG.md

Lines changed: 4 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -2,27 +2,13 @@
22

33
All notable changes to this project will be documented in this file, in reverse chronological order by release.
44

5-
## 1.3.0 - TBD
5+
## 1.3.0 - 2018-05-08
66

77
### Added
88

9-
- Nothing.
10-
11-
### Deprecated
12-
13-
- Nothing.
14-
15-
### Removed
16-
17-
- Nothing.
18-
19-
### Fixed
20-
21-
- Nothing.
22-
23-
## 1.2.4 - TBD
9+
- [#58](https://github.com/zfcampus/zf-api-problem/pull/58) adds support for PHP 7.2.
2410

25-
### Added
11+
### Changed
2612

2713
- Nothing.
2814

@@ -32,7 +18,7 @@ All notable changes to this project will be documented in this file, in reverse
3218

3319
### Removed
3420

35-
- Nothing.
21+
- [#58](https://github.com/zfcampus/zf-api-problem/pull/58) removes support for HHVM.
3622

3723
### Fixed
3824

CONDUCT.md renamed to CODE_OF_CONDUCT.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
# Contributor Code of Conduct
22

3-
The Zend Framework project adheres to [The Code Manifesto](http://codemanifesto.com)
3+
This project adheres to [The Code Manifesto](http://codemanifesto.com)
44
as its guidelines for contributor interactions.
55

66
## The Code Manifesto

CONTRIBUTING.md

Lines changed: 155 additions & 46 deletions
Original file line numberDiff line numberDiff line change
@@ -1,80 +1,189 @@
11
# CONTRIBUTING
22

3-
Apigility and related modules (of which this is one) are open source and licensed
4-
as [BSD-3-Clause](http://opensource.org/licenses/BSD-3-Clause). Contributions
5-
are welcome in the form of issue reports and pull requests.
6-
7-
All pull requests should include unit tests when applicable, and should follow
8-
our coding standards (more on these below); failure to do so may result in
9-
rejection of the pull request. If you need help writing tests, please ask on the
10-
developer mailing list and/or in IRC.
11-
123
## RESOURCES
134

14-
If you wish to contribute to Apigility modules, please be sure to
5+
If you wish to contribute to this project, please be sure to
156
read/subscribe to the following resources:
167

17-
- [Coding Standards](https://github.com/zendframework/zf2/wiki/Coding-Standards)
18-
- [ZF Git Guide](https://github.com/zendframework/zf2/blob/master/README-GIT.md)
19-
- [Apigility developer mailing list](http://bit.ly/apigility-dev)
20-
- Apigility developer IRC channel: #apigility-dev on Freenode.net
8+
- [Coding Standards](https://github.com/zendframework/zend-coding-standard)
9+
- [Forums](https://discourse.zendframework.com/c/contributors)
10+
- [Chat](https://zendframework-slack.herokuapp.com)
11+
- [Code of Conduct](CODE_OF_CONDUCT.md)
12+
13+
If you are working on new features or refactoring
14+
[create a proposal](https://github.com/zfcampus/zf-api-problem/issues/new).
2115

22-
If you are working on new features, refactoring an existing module, or proposing
23-
a new module, please send an email to the developer mailing list.
16+
## RUNNING TESTS
2417

25-
## REPORTING POTENTIAL SECURITY ISSUES
18+
To run tests:
2619

27-
If you have encountered a potential security vulnerability in any Apigility
28-
module, please report it to us at [[email protected]](mailto:[email protected]).
29-
We will work with you to verify the vulnerability and patch it.
20+
- Clone the repository:
3021

31-
When reporting issues, please provide the following information:
22+
```console
23+
$ git clone git://github.com/zfcampus/zf-api-problem.git
24+
$ cd zf-api-problem
25+
```
3226

33-
- Module(s) affected
34-
- A description indicating how to reproduce the issue
35-
- A summary of the security vulnerability and impact
27+
- Install dependencies via composer:
3628

37-
We request that you contact us via the email address above and give the project
38-
contributors a chance to resolve the vulnerability and issue a new release prior
39-
to any public exposure; this helps protect Apigility users, and provides them
40-
with a chance to upgrade and/or update in order to protect their applications.
29+
```console
30+
$ composer install
31+
```
4132

42-
For sensitive email communications, please use
43-
[our PGP key](http://framework.zend.com/zf-security-pgp-key.asc).
33+
If you don't have `composer` installed, please download it from https://getcomposer.org/download/
4434

45-
## RUNNING TESTS
35+
- Run the tests using the "test" command shipped in the `composer.json`:
36+
37+
```console
38+
$ composer test
39+
```
4640

47-
First, use [Composer](https://getcomposer.org) to install all dependencies:
41+
You can turn on conditional tests with the `phpunit.xml` file.
42+
To do so:
43+
44+
- Copy `phpunit.xml.dist` file to `phpunit.xml`
45+
- Edit `phpunit.xml` to enable any specific functionality you
46+
want to test, as well as to provide test values to utilize.
47+
48+
## Running Coding Standards Checks
49+
50+
First, ensure you've installed dependencies via composer, per the previous
51+
section on running tests.
52+
53+
To run CS checks only:
4854

4955
```console
50-
$ composer install
56+
$ composer cs-check
5157
```
5258

53-
To run tests, use the PHPUnit executable installed by Composer:
59+
To attempt to automatically fix common CS issues:
5460

5561
```console
56-
$ composer test
62+
$ composer cs-fix
5763
```
5864

59-
## CODING STANDARDS
65+
If the above fixes any CS issues, please re-run the tests to ensure
66+
they pass, and make sure you add and commit the changes after verification.
67+
68+
## Recommended Workflow for Contributions
69+
70+
Your first step is to establish a public repository from which we can
71+
pull your work into the master repository. We recommend using
72+
[GitHub](https://github.com), as that is where the component is already hosted.
73+
74+
1. Setup a [GitHub account](https://github.com/), if you haven't yet
75+
2. Fork the repository (https://github.com/zfcampus/zf-api-problem)
76+
3. Clone the canonical repository locally and enter it.
77+
78+
```console
79+
$ git clone git://github.com/zfcampus/zf-api-problem.git
80+
$ cd zf-api-problem
81+
```
82+
83+
4. Add a remote to your fork; substitute your GitHub username in the command
84+
below.
85+
86+
```console
87+
$ git remote add {username} [email protected]:{username}/zf-api-problem.git
88+
$ git fetch {username}
89+
```
90+
91+
### Keeping Up-to-Date
92+
93+
Periodically, you should update your fork or personal repository to
94+
match the canonical ZF repository. Assuming you have setup your local repository
95+
per the instructions above, you can do the following:
6096

61-
While Apigility uses Zend Framework 2 coding standards, in practice, we check
62-
standards against PSR-1/2. To check for CS issues:
6397

6498
```console
65-
$ composer cs-check
99+
$ git checkout master
100+
$ git fetch origin
101+
$ git rebase origin/master
102+
# OPTIONALLY, to keep your remote up-to-date -
103+
$ git push {username} master:master
104+
```
105+
106+
If you're tracking other branches -- for example, the "develop" branch, where
107+
new feature development occurs -- you'll want to do the same operations for that
108+
branch; simply substitute "develop" for "master".
109+
110+
### Working on a patch
111+
112+
We recommend you do each new feature or bugfix in a new branch. This simplifies
113+
the task of code review as well as the task of merging your changes into the
114+
canonical repository.
115+
116+
A typical workflow will then consist of the following:
117+
118+
1. Create a new local branch based off either your master or develop branch.
119+
2. Switch to your new local branch. (This step can be combined with the
120+
previous step with the use of `git checkout -b`.)
121+
3. Do some work, commit, repeat as necessary.
122+
4. Push the local branch to your remote repository.
123+
5. Send a pull request.
124+
125+
The mechanics of this process are actually quite trivial. Below, we will
126+
create a branch for fixing an issue in the tracker.
127+
128+
```console
129+
$ git checkout -b hotfix/9295
130+
Switched to a new branch 'hotfix/9295'
66131
```
67132

68-
This will report CS issues. You can also attempt to fix many reported errors
69-
automatically:
133+
... do some work ...
134+
70135

71136
```console
72-
$ composer cs-fix
137+
$ git commit
73138
```
74139

75-
If you use `cs-fix` to fix issues, make certain you add and commit any files
76-
changed!
140+
... write your log message ...
141+
142+
143+
```console
144+
$ git push {username} hotfix/9295:hotfix/9295
145+
Counting objects: 38, done.
146+
Delta compression using up to 2 threads.
147+
Compression objects: 100% (18/18), done.
148+
Writing objects: 100% (20/20), 8.19KiB, done.
149+
Total 20 (delta 12), reused 0 (delta 0)
150+
To ssh://[email protected]/{username}/zf-api-problem.git
151+
b5583aa..4f51698 HEAD -> master
152+
```
153+
154+
To send a pull request, you have two options.
155+
156+
If using GitHub, you can do the pull request from there. Navigate to
157+
your repository, select the branch you just created, and then select the
158+
"Pull Request" button in the upper right. Select the user/organization
159+
"zfcampus" (or whatever the upstream organization is) as the recipient.
160+
161+
#### What branch to issue the pull request against?
162+
163+
Which branch should you issue a pull request against?
164+
165+
- For fixes against the stable release, issue the pull request against the
166+
"master" branch.
167+
- For new features, or fixes that introduce new elements to the public API (such
168+
as new public methods or properties), issue the pull request against the
169+
"develop" branch.
170+
171+
### Branch Cleanup
172+
173+
As you might imagine, if you are a frequent contributor, you'll start to
174+
get a ton of branches both locally and on your remote.
175+
176+
Once you know that your changes have been accepted to the master
177+
repository, we suggest doing some cleanup of these branches.
178+
179+
- Local branch cleanup
180+
181+
```console
182+
$ git branch -d <branchname>
183+
```
77184

78-
## Conduct
185+
- Remote branch removal
79186

80-
Please see our [CONDUCT.md](CONDUCT.md) to understand expected behavior when interacting with others in the project.
187+
```console
188+
$ git push {username} :<branchname>
189+
```

0 commit comments

Comments
 (0)