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

Deployments: use proper media-type for in_progress/queued, inactive state #979

Merged
merged 3 commits into from
Mar 25, 2021

Conversation

staabm
Copy link
Contributor

@staabm staabm commented Mar 25, 2021

closes #978

@staabm staabm changed the title Deployments: use correct media-type for in_progress/queued, inactive Deployments: use proper media-type for in_progress/queued, inactive state Mar 25, 2021
@acrobat acrobat merged commit 533a798 into KnpLabs:master Mar 25, 2021
@acrobat
Copy link
Collaborator

acrobat commented Mar 25, 2021

Thank you @staabm

@staabm staabm deleted the patch-2 branch March 25, 2021 18:04
@staabm
Copy link
Contributor Author

staabm commented Mar 25, 2021

Thx for merging @acrobat .

Could this change also be merged into the 2.x branch?
Alternatively would you accept a Similar PR to the 2.x branch?

@acrobat
Copy link
Collaborator

acrobat commented Mar 25, 2021

Yes, this is a bugfix that could be backported! Can you create a PR for 2.x? Thanks!

clxmstaab pushed a commit to clxmstaab/php-github-api that referenced this pull request Mar 26, 2021
…eued, inactive state (staabm)

This PR was squashed before being merged into the 3.1.x-dev branch.

Discussion
----------

closes KnpLabs#978

Commits
-------

5ffefdc Deployments: use correct media-type for in_progress/queued, inactive
bf3eb1f Update Deployment.php
ca128f6 cs
@clxmstaab
Copy link
Contributor

Yes, this is a bugfix that could be backported! Can you create a PR for 2.x? Thanks!

here we go #981

acrobat added a commit that referenced this pull request Mar 27, 2021
This PR was merged into the 2.x branch.

Discussion
----------

Deployments: use proper media-type for in_progress/queued, inactive state

as discussed in #979 (comment)

Commits
-------

0c2815f bug #979 Deployments: use proper media-type for in_progress/queued, inactive state (staabm)
acrobat added a commit that referenced this pull request Mar 27, 2021
* 2.x:
  bug #979 Deployments: use proper media-type for in_progress/queued, inactive state (staabm)
  Update changelog for 2.19.1 release
acrobat added a commit to amacrobert-meq/php-github-api that referenced this pull request Mar 28, 2021
…integration-doc-update

* upstream/2.x:
  Correctly link to github actions docs and fix backlinks
  Improved bc check
  bug KnpLabs#979 Deployments: use proper media-type for in_progress/queued, inactive state (staabm)
@clxmstaab
Copy link
Contributor

@acrobat could we also get a new 3.x release? some bugfixes - like the one in this PR - are not yet part of a 3.x release, but are fixed on the latest 2.x releases

@acrobat
Copy link
Collaborator

acrobat commented Apr 28, 2021

@clxmstaab I will tag a new 3.x release today!

@acrobat
Copy link
Collaborator

acrobat commented Apr 28, 2021

@clxmstaab 3.2.0 is released!

@clxmstaab
Copy link
Contributor

clxmstaab commented Apr 28, 2021

thank you so much <3

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.

create deployment status in_progress or queued require different media-type
3 participants