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.

Sign up for GitHub

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

Jump to bottom

Add SQL Server support #10324

Closed
wants to merge 58 commits into from
Closed

Add SQL Server support #10324

wants to merge 58 commits into from

Conversation

AdRiley
Copy link
Member

@AdRiley AdRiley commented Jun 20, 2024 •
edited
Loading

Pull Request Description

image

Important Notes

Checklist

Please ensure that the following checklist has been satisfied before submitting the PR:

  • The documentation has been updated, if necessary.
  • Screenshots/screencasts have been attached, if there are any visual changes. For interactive or animated visual changes, a screencast is preferred.
  • All code follows the
    Scala,
    Java,
    TypeScript,
    and
    Rust
    style guides. In case you are using a language not listed above, follow the Rust style guide.
  • Unit tests have been written where possible.

@hubertp hubertp added the CI: Clean build required CI runners will be cleaned before and after this PR is built. label Jun 20, 2024
@AdRiley AdRiley force-pushed the wip/adr/add-sql-server branch 2 times, most recently from 040196e to 49f6ed2 Compare June 28, 2024 13:28
@AdRiley AdRiley force-pushed the wip/adr/add-sql-server branch 2 times, most recently from 0f17292 to 0cd34dd Compare July 11, 2024 12:41
@AdRiley AdRiley closed this Aug 8, 2024
@AdRiley AdRiley deleted the wip/adr/add-sql-server branch August 8, 2024 10:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jdunkerley jdunkerley Awaiting requested review from jdunkerley jdunkerley will be requested when the pull request is marked ready for review jdunkerley is a code owner

@radeusgd radeusgd Awaiting requested review from radeusgd radeusgd will be requested when the pull request is marked ready for review radeusgd is a code owner

@GregoryTravis GregoryTravis Awaiting requested review from GregoryTravis GregoryTravis will be requested when the pull request is marked ready for review GregoryTravis is a code owner

@marthasharkey marthasharkey Awaiting requested review from marthasharkey marthasharkey will be requested when the pull request is marked ready for review marthasharkey is a code owner

@4e6 4e6 Awaiting requested review from 4e6 4e6 will be requested when the pull request is marked ready for review 4e6 is a code owner

@JaroslavTulach JaroslavTulach Awaiting requested review from JaroslavTulach JaroslavTulach will be requested when the pull request is marked ready for review JaroslavTulach is a code owner

@hubertp hubertp Awaiting requested review from hubertp hubertp will be requested when the pull request is marked ready for review hubertp is a code owner

@Akirathan Akirathan Awaiting requested review from Akirathan Akirathan will be requested when the pull request is marked ready for review Akirathan is a code owner

@Frizi Frizi Awaiting requested review from Frizi Frizi will be requested when the pull request is marked ready for review Frizi is a code owner

Assignees
No one assigned
Labels
CI: Clean build required CI runners will be cleaned before and after this PR is built.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@AdRiley @hubertp