Skip to content
This repository has been archived by the owner on Apr 4, 2019. It is now read-only.

Deployment API #45

Closed
Baggz opened this issue Dec 10, 2014 · 15 comments
Closed

Deployment API #45

Baggz opened this issue Dec 10, 2014 · 15 comments

Comments

@Baggz
Copy link
Contributor

Baggz commented Dec 10, 2014

Hi,

Would it be possible to use the Deployment API to mark a PR as deployed?

image

(I know you can open Trello and see the respective card.)

@Almad
Copy link
Contributor

Almad commented Dec 10, 2014

That...is a great idea!

@Almad
Copy link
Contributor

Almad commented Dec 10, 2014

@Baggz although, is there anything for which this is an enabler? Trello is the primary place to review...

@abtris
Copy link
Contributor

abtris commented Jan 13, 2015

👍

@honzajavorek
Copy link
Contributor

@Almad I think multiple sources of the same truth are a good thing in case something blows up (aka platform support) and you need to quickly find a culprit PR/code/whatever. Switching back and forth between GitHub and Trello is... I am going to use "not convenient".

@Almad
Copy link
Contributor

Almad commented Jan 13, 2015

@honzajavorek Source of truth should be only one, but I agree that propagating it everywhere is a good idea.

Primary source of truth is obviously heroku, primary source of review is Trello (because it retains whether it's just deployed, or verified as well).

@honzajavorek
Copy link
Contributor

That's probably what I meant by "multiple sources of the same truth" and wrote it in a bad way :) We're on the same page.

@Almad
Copy link
Contributor

Almad commented Mar 23, 2015

@Baggz Is it fixed by #50 ?

@Almad
Copy link
Contributor

Almad commented Mar 23, 2015

(released as 0.11.4)

@XVincentX
Copy link
Contributor

Looks like it is not working. I can't see the deploy informations in closed pull requests.

@Almad
Copy link
Contributor

Almad commented Apr 1, 2015

@XVincentX do you use 0.11.4?

@XVincentX
Copy link
Contributor

Yes I do!
image

@Baggz
Copy link
Contributor Author

Baggz commented Jun 25, 2015

👍

@Almad
Copy link
Contributor

Almad commented Jun 26, 2015

@Baggz that is +1 for fixing or that is works? ;)

@Baggz
Copy link
Contributor Author

Baggz commented Jun 26, 2015

@Almad

Ah, sorry... It's ‘same here’ (= it's not working properly).

@abtris
Copy link
Contributor

abtris commented Sep 12, 2018

Migrated to apiaryio/bb#41

@abtris abtris closed this as completed Sep 12, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

5 participants