Skip to content
This repository has been archived by the owner on Jan 21, 2020. It is now read-only.

Update Travis CI config #37

Merged
merged 4 commits into from
Aug 11, 2017
Merged

Update Travis CI config #37

merged 4 commits into from
Aug 11, 2017

Conversation

geerteltink
Copy link
Member

@geerteltink geerteltink commented Jul 13, 2017

  • Removes irc notifications
  • Removes slack notifications
  • Removes hhvm
  • Adds nightly PHP 7.2 builds
  • Update to latest guidelines

@froschdesign
Copy link
Member

froschdesign commented Jul 13, 2017

@xtreamwayz
All these changes maybe fine, but no one can review your PR's, because where is the reference for this? Where can I read which changes are important, which changes should include or not and which file structure is preferred?
I do not mean the Travis documentation, I mean the requirements for all ZF repositories.

It can not be the goal, that only two or three people knows these requirements. Therefore we need a boilerplate for the Travis configuration file and the Composer file.

/cc @weierophinney

@geerteltink
Copy link
Member Author

@froschdesign I agree. I've asked for sort of a template as well in the slack contributors channel yesterday.

For notifications: https://zendframework.slackarchive.io/contributors/page-4/ts-1499810981101710
hhmv drop was mentioned in a blog post I think.

@froschdesign
Copy link
Member

@xtreamwayz
I know the blog post and I know the discussion on Slack, but this can not be a place for references or informations to contributors and maintainers.

  • The blog is good to inform the "outside world".
  • Slack and the forum is good for discussions, questions, etc.

But these channels are too fragmented and searching ends in a mess.

We already have a place for this kind of information:

Contributors and maintainers information for Zend Framework (https://github.com/zendframework/maintainers)

Now, we just have to use them!

@geerteltink
Copy link
Member Author

For completeness:

@geerteltink
Copy link
Member Author

@froschdesign Are these PR's good to merge now?

@geerteltink geerteltink changed the base branch from master to develop August 11, 2017 18:21
@geerteltink geerteltink changed the base branch from develop to master August 11, 2017 18:22
geerteltink added a commit that referenced this pull request Aug 11, 2017
geerteltink added a commit that referenced this pull request Aug 11, 2017
@geerteltink geerteltink added this to the 2.1.0 milestone Aug 11, 2017
@geerteltink geerteltink merged commit 380a005 into zendframework:master Aug 11, 2017
@geerteltink geerteltink deleted the hotfix/travis branch August 11, 2017 19:02
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants