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

Drop "credentials" input and prep for 1.0 release #279

Merged
merged 1 commit into from
Nov 17, 2022
Merged

Conversation

sethvargo
Copy link
Member

@sethvargo sethvargo commented Nov 17, 2022

This drops the long-deprecated "credentials" input and pulls many of the expensive integration tests into unit tests which assert the same behavior. It also switches to using the JSON output from the "gcloud app deploy" command instead of parsing the text output, since that has proven to be unstable.

Closes #166
Closes #255
Closes #268
Closes #277
Closes #278

@sethvargo sethvargo requested a review from verbanicm November 17, 2022 18:55
@sethvargo sethvargo requested a review from a team as a code owner November 17, 2022 18:55
src/main.ts Show resolved Hide resolved
src/main.ts Outdated Show resolved Hide resolved
@sethvargo sethvargo force-pushed the sethvargo/one branch 2 times, most recently from 61715b1 to e3ea979 Compare November 17, 2022 19:34
This drops the long-deprecated "credentials" input and pulls many of the expensive integration tests into unit tests which assert the same behavior. It also switches to using the JSON output from the "gcloud app deploy" command instead of parsing the text output, since that has proven to be unstable.
@sethvargo sethvargo merged commit 3177e3f into main Nov 17, 2022
@sethvargo sethvargo deleted the sethvargo/one branch November 17, 2022 19:59
sethvargo pushed a commit that referenced this pull request Nov 17, 2022
## What's Changed
* Drop "credentials" input and prep for 1.0 release by @sethvargo in
#279


**Full Changelog**:
v0.8.2...3177e3f
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
2 participants