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.

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

update piwik-icons submodule #12389

Merged
merged 5 commits into from Dec 29, 2017
Merged

update piwik-icons submodule #12389

merged 5 commits into from Dec 29, 2017

Conversation

Findus23
Copy link
Member

As I was in the middle of a restructure, the last Piwik release didn't include an update of the icons.
Because it's finished now Piwik 3.3 will have many icon changes (289 files in 37 commits):
matomo-org/matomo-icons@ac9c01d...48d5d33

Thanks again to @sogen for also contributing icons.

@Findus23 Findus23 added not-in-changelog For issues or pull requests that should not be included in our release changelog on matomo.org. Task Indicates an issue is neither a feature nor a bug and it's purely a "technical" change. labels Dec 28, 2017
@Findus23 Findus23 added this to the 3.3.0 milestone Dec 28, 2017
@Findus23
Copy link
Member Author

@sgiehl
Copy link
Member

sgiehl commented Dec 28, 2017

It's failing often in the last days... maybe we need to increase the wait times. will check that when I have some time

@Findus23
Copy link
Member Author

Now all (working) tests should be fixed

@sgiehl
Copy link
Member

sgiehl commented Dec 29, 2017

Not sure why some of the tests nearly always fail for the build on branches, but work for PR builds against master... You can update the one remaining here: https://builds-artifacts.piwik.org/piwik/piwik/3.x-dev/25440/Live_visitor_log.png

@Findus23
Copy link
Member Author

Done in a95699e

@sgiehl
Copy link
Member

sgiehl commented Dec 29, 2017

Will have another look as soon as the builds finished. Guess we can merge it then :)

@sgiehl sgiehl merged commit 889af9b into 3.x-dev Dec 29, 2017
@sgiehl sgiehl deleted the update-piwik-icons-again branch December 29, 2017 11:23
@Findus23 Findus23 restored the update-piwik-icons-again branch October 4, 2019 09:48
@sgiehl sgiehl deleted the update-piwik-icons-again branch February 10, 2020 13:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
not-in-changelog For issues or pull requests that should not be included in our release changelog on matomo.org. Task Indicates an issue is neither a feature nor a bug and it's purely a "technical" change.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants