Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Move config.ini.travis.php to this repo from Piwik core #15

Closed
diosmosis opened this issue Nov 1, 2015 · 1 comment · Fixed by #20
Closed

Move config.ini.travis.php to this repo from Piwik core #15

diosmosis opened this issue Nov 1, 2015 · 1 comment · Fixed by #20
Labels
bug

Comments

@diosmosis
Copy link
Member

Since the contents of this file are only for travis and not for running tests locally, they should be included in this repo. Not doing so can cause problems when it is changed to work w/ the latest travis-scripts, but testing against an older Piwik results in using an unchanged version.
@diosmosis diosmosis added the bug label Nov 1, 2015
diosmosis pushed a commit that referenced this issue Nov 1, 2015
Refs #15, quick temporary fix for now.
b166a8c
@diosmosis
Copy link
Member Author

TODO:

  • Remove change in this commit: 7891fd0

@mattab mattab added this to the Current sprint milestone Nov 2, 2015
@diosmosis diosmosis mentioned this issue Dec 28, 2015
Merged
diosmosis added a commit that referenced this issue Dec 28, 2015
@diosmosis
Merge pull request #20 from piwik/config_ini_travis
c9c4d7a 
Fixes #15, move config.ini.travis.php to this repo.
@innocraft-automation innocraft-automation removed this from the Current sprint milestone Jan 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Move config.ini.travis.php to this repo. matomo-org/travis-scripts
3 participants
@diosmosis @mattab @innocraft-automation