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

add: auto_inactive:false to createDeploymentStatus #726

Open
wants to merge 4 commits into
base: develop
Choose a base branch
from

Conversation

nojvek
Copy link
Contributor

@nojvek nojvek commented Feb 9, 2022

Github automatically makes any previous environments stale, if there is a new deployment status. With Netlify, the urls still remain valid even if there is a new status.

In our case, we use netlify for PR builds i.e every commit gets a unique netlify link. The issue is Github UI loses that link when anyone else adds a new commit.

By making auto_inactive: false, the links would preserve and Github's UI would be useful.

image

@nojvek
Copy link
Contributor Author

nojvek commented Feb 9, 2022

fyi @nwtgck ^

@nwtgck
Copy link
Owner

nwtgck commented Feb 9, 2022

@nojvek
Thanks. I have not tried auto_inactive: false yet.

What about clicking the following ✅ and go Netlify? Each commit has ✅.
image

@nojvek
Copy link
Contributor Author

nojvek commented Feb 17, 2022

That works. Although the deployment that shows at the bottom near the review section goes stale when any new deployment with same environment is made.

This would save 2 clicks, and be less confusing as the deploy link would always be there.

@nojvek
Copy link
Contributor Author

nojvek commented Mar 5, 2022

@nwtgck - any blockers on merging this?

@nwtgck
Copy link
Owner

nwtgck commented Mar 5, 2022

No blocking. I've not tryied auto_inactive:false yet.

@nojvek
Copy link
Contributor Author

nojvek commented Apr 12, 2022

Kind ping on this @nwtgck ^

I know can clone this repo to unblock myself, but would be nice to land this one line change and keep it in orig repo.

@nojvek
Copy link
Contributor Author

nojvek commented May 11, 2022

Another kind ping on this @nwtgck plz.

@nojvek
Copy link
Contributor Author

nojvek commented Jun 4, 2022

Just following up. Anything blocking?

@nwtgck
Copy link
Owner

nwtgck commented Jun 11, 2022

I apologize for the very long wait. I finally tried this feature on nwtgck/use-actions-netlify#25. But unfortunately, I'm not sure the difference between auto_inactive: false and without it.

image

@andersonba
Copy link

ping

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.

None yet

3 participants