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

docs: Add release notes for v1.19.0 #2406

Merged
merged 4 commits into from
Jul 6, 2023
Merged

docs: Add release notes for v1.19.0 #2406

merged 4 commits into from
Jul 6, 2023

Conversation

kyleconroy
Copy link
Collaborator

No description provided.

@kyleconroy kyleconroy force-pushed the kyle/docs-1-19 branch 2 times, most recently from e0792fa to bc37e81 Compare July 6, 2023 01:33
@kyleconroy kyleconroy marked this pull request as ready for review July 6, 2023 01:33

The `database` mapping supports the following keys:

- `url`:
Copy link
Collaborator

@andrewmbenton andrewmbenton Jul 6, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I know this isn't the ideal time to offer naming suggestions, but we can't really change this after release so it's worth a brief discussion. uri is better than url in this context I think, if we want to indicate that we are only supporting the URI-style connection string format for various drivers (and not other formats e.g. key/value pairs).

I took a quick read of the postgres and mysql and sqlite docs on this and they all mention "URI."

If we want to be more flexible or generic we could use something like conn or conn_string or connection_string.

```

We also encourage running [`sqlc vet`](../reference/cli.html#vet). To get the most value out of `vet`,
you'll want to set up a running database. See the [vet] documentation for a
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[vet] is missing a link url here, but would probably be redundant given the link in the previous sentence so probably just remove square brackets.

@kyleconroy kyleconroy merged commit 7ca892f into main Jul 6, 2023
@kyleconroy kyleconroy deleted the kyle/docs-1-19 branch July 6, 2023 16:27
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.

2 participants