Skip to content

README: Fix AppVeyor branch and add link to LICENSE #4763

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

Closed
wants to merge 1 commit into from

Conversation

gerardroche
Copy link
Contributor

  • Point AppVeyor status badge to master

* Point AppVeyor status badge to master
@gerardroche
Copy link
Contributor Author

We could add a table with the status of all minor branches. There's lots of possibilities. This could be on the readme, or a dedicated status on the php.net?

Branch AppVeyor Travis CI Azure Pipelines
master Build status Build status Build Status
7.4 Build status Build status Build Status
7.3 Build status Build status n/a
7.2 Build status Build status n/a
7.1 Build status Build status n/a
Branch AppVeyor Travis CI Azure Pipelines Bug Fixes Until Security Fixes Until
master Build status Build status Build Status
7.4 Build status Build status Build Status
7.3 Build status Build status 6 Dec 2020 6 Dec 2021
7.2 Build status Build status 30 Nov 2019 30 Nov 2020
7.1 Build status Build status 1 Dec 2018 1 Dec 2019
Branch AppVeyor Travis CI Azure Pipelines
master Build status Build status Build Status
7.4 Build status Build status Build Status
7.3 Build status Build status
7.2 Build status Build status
7.1 Build status Build status
7.0 Build status Build status
5.6 Build status
5.5 Build status
5.4 Build status
5.3 Build status
Branch AppVeyor Travis CI Azure Pipelines Bug Fixes Security Fixes
master Build status Build status Build Status
7.4 Build status Build status Build Status
7.3 Build status Build status 6 Dec 2020 6 Dec 2021
7.2 Build status Build status 30 Nov 2019 30 Nov 2020
7.1 Build status Build status 1 Dec 2018 1 Dec 2019
7.0 Build status Build status
5.6 Build status
5.5 Build status
5.4 Build status
5.3 Build status

@php-pulls php-pulls closed this in 1afcf28 Oct 1, 2019
@nikic
Copy link
Member

nikic commented Oct 1, 2019

Regarding adding a larger matrix of build badges, I'm neutral on that -- I can't say I ever looked at them ^^ Maybe @petk @carusogabriel @cmb69 have thoughts about this?

@carusogabriel
Copy link
Contributor

I'm neutral, as well. I rarely look into them, I just strait to the respective websites.

Having on the master is interesting, as we need to ensure that this build in specific is always green.

@cmb69
Copy link
Member

cmb69 commented Oct 1, 2019

What would be the benefit of having such a CI badge matrix?

@nikic
Copy link
Member

nikic commented Oct 2, 2019

What might be useful would be links to the last scheduled builds on the 7.4/master branch, as those contain our full test matrixes and are not easily accessible right now. Though I'm not sure if there is any way to get the last scheduled build specifically...

@gerardroche
Copy link
Contributor Author

Having thought about it, there isn't a lot of value of a matrix of build badges. I guess the idea is to try bring more awareness to supported versions, but a matrix of badges on the readme isn't going to achieve that.

I like the supported versions page. I only became aware of it recently. My understanding of supported versions before that was fuzzy. I also think the recent cleanup of the readme and other docs has helped.

What might be useful would be links to the last scheduled builds on the 7.4/master branch, as those contain our full test matrixes and are not easily accessible right now. Though I'm not sure if there is any way to get the last scheduled build specifically...

I looked into it and I can't see a way to do it. CI service APIs generally only allow you to get the status of a branch. Most don't even allow you to get the status of a tag. I'm surprised the Azure UI doesn't have a filter for scheduled builds.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants