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

Invoke newt to build devcontainers with cloudbuild #1

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

aaronlehmann
Copy link
Owner

cc @coryb

@coryb
Copy link

coryb commented Nov 23, 2022

LGTM

@aaronlehmann aaronlehmann force-pushed the build-with-cloudbuild branch 4 times, most recently from c9c7e07 to 9da8bb6 Compare November 25, 2022 20:13
chrmarti and others added 8 commits November 28, 2022 15:50
…iners#298)

* requestResolveHeaders() also returns responseBody

* print responseBody for "postUploadSessionId" on error

* missing semicolon

* one more semicolon
…s#293)

* validate input of getRef()

* update tests using getRef()

* add unit tests

* Refactor getRef()

* add more test cases

* dregex match entire string
* fix request headers

* remove debug line

* v0.25.1
…evcontainers#303)

* create getCollectionRef() to centralize validating Collection identifier

* missing a quote

* another quote
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.

4 participants