[PR #35] [MERGED] Setup Travis correctly for phpspec (+ cache improvements) #352

Closed
opened 2026-03-01 17:49:22 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/antonioribeiro/google2fa/pull/35
Author: @barryvdh
Created: 7/15/2016
Status: Merged
Merged: 7/17/2016
Merged by: @antonioribeiro

Base: masterHead: patch-2


📝 Commits (1)

📊 Changes

1 file changed (+13 additions, -4 deletions)

View changed files

📝 .travis.yml (+13 -4)

📄 Description

Currently, Travis is not used the run the test. There are tests, so might as well run them :)

  • Use phpspec instead of phpunit
  • Add php 5.6/7.0 to build matrix
  • No need anymore to install the composer phar
  • use composer retry to install the composer dependencies
  • Prefer dist + use the cache/sudo settings to cache the downloads from composer for faster builds.

After this, Travis still needs to be setup as hook by the author.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/antonioribeiro/google2fa/pull/35 **Author:** [@barryvdh](https://github.com/barryvdh) **Created:** 7/15/2016 **Status:** ✅ Merged **Merged:** 7/17/2016 **Merged by:** [@antonioribeiro](https://github.com/antonioribeiro) **Base:** `master` ← **Head:** `patch-2` --- ### 📝 Commits (1) - [`9da969b`](https://github.com/antonioribeiro/google2fa/commit/9da969b1c39c65d4f430dc454d08a3f6e78b2451) Update travis build ### 📊 Changes **1 file changed** (+13 additions, -4 deletions) <details> <summary>View changed files</summary> 📝 `.travis.yml` (+13 -4) </details> ### 📄 Description Currently, Travis is not used the run the test. There are tests, so might as well run them :) - Use phpspec instead of phpunit - Add php 5.6/7.0 to build matrix - No need anymore to install the composer phar - use composer retry to install the composer dependencies - Prefer dist + use the cache/sudo settings to cache the downloads from composer for faster builds. After this, Travis still needs to be setup as hook by the author. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-03-01 17:49:22 +03:00
Sign in to join this conversation.
No labels
bug
pull-request
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/google2fa#352
No description provided.