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

Default destroy on stale is No #485

Merged
merged 1 commit into from Sep 2, 2019
Merged

Default destroy on stale is No #485

merged 1 commit into from Sep 2, 2019

Conversation

@EtienneM
Copy link
Member

@EtienneM EtienneM commented Sep 2, 2019

╰─$ scalingo --app test-etienne integration-link-create https://framagit.org/Titizebioutifoul/sample-go-martini
? Branch to auto-deploy (empty to disable):
? Automatically deploy review apps: Yes
? Automatically destroy review apps when the pull/merge request is closed: Yes
? Hours before automatically destroying the review apps: 0
? Automatically destroy review apps after some time without deploy/commits: No
-----> Your app 'test-etienne' is linked to the repository https://framagit.org/Titizebioutifoul/sample-go-martini.
╭─emichon@biniou ~/Documents/Scalingo/golang/src/github.com/Scalingo/cli 2.4.2‹fix/484/default_destroy_stale*›
╰─$ scalingo --app test-etienne integration-link
Application: test-etienne (5d6d0d2bb4553d00019d22b6)
Integration: GitLab self-hosted (1ce487c9-3633-48be-b674-c87f2826f74f)

Repository: Titizebioutifoul/sample-go-martini
Auto Deploy: ✘
Review Apps Deploy: ✔
        Destroy on Close: instantly
        Destroy on Stale: ✘

Fix #484

@EtienneM EtienneM marked this pull request as ready for review Sep 2, 2019
@EtienneM EtienneM requested a review from Soulou Sep 2, 2019
@Soulou
Copy link
Member

@Soulou Soulou commented Sep 2, 2019

LGTM

Loading

@Soulou Soulou merged commit c634384 into master Sep 2, 2019
1 check passed
Loading
EtienneM added a commit that referenced this issue Sep 2, 2019
@EtienneM EtienneM deleted the fix/484/default_destroy_stale branch Sep 2, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

2 participants