-
Notifications
You must be signed in to change notification settings - Fork 290
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 missing types #1168
Add missing types #1168
Conversation
types/defines/hyperdrive.d.ts
Outdated
/* | ||
* The randomly generated password to use when authenticating to your | ||
* database via Hyperdrive. Like the host field, this password is only valid | ||
* within the context of the currently running worker instance from which |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
* within the context of the currently running worker instance from which | |
* within the context of the currently running Worker instance from which |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
types/defines/hyperdrive.d.ts
Outdated
|
||
/* | ||
* A randomly generated hostname that is only valid within the context of the | ||
* currently running worker which, when passed into `connect()` function from |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
* currently running worker which, when passed into `connect()` function from | |
* currently running Worker which, when passed into `connect()` function from |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
829277e
to
2fe02dd
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
since I haven't been able to successfully
bazel build //types:types
on either my linux or mac boxes (even without this change).
Would you be able to DM me the error you're seeing internally? 🙂
types/defines/hyperdrive.d.ts
Outdated
// Copyright (c) 2023 Cloudflare, Inc. | ||
// Licensed under the Apache 2.0 license found in the LICENSE file or at: | ||
// https://opensource.org/licenses/Apache-2.0 | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The contents of files in the defines
directory are concatenated as is to the types output. This means this copyright header will be interested mid-way through the file. I'd probably remove it. 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure, I'll remove it. I just copied it from the other files, all but one of which have a similar header.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also opened up #1176 to remove the other headers in case that's something you think we ought to do.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
2fe02dd
to
7b24b2a
Compare
As suggested by mrbbot on #1168
As suggested by mrbbot on #1168
Although FWIW I haven't really verified that what this generates looks ok, since I haven't been able to successfully
bazel build //types:types
on either my linux or mac boxes (even without this change).