[PR #1137] [CLOSED] refactor(DrizzleAdapter): minor code improvements #3650

Closed
opened 2026-03-13 11:03:11 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/better-auth/better-auth/pull/1137
Author: @ping-maxwell
Created: 1/4/2025
Status: Closed

Base: mainHead: refactor/drizzle-adapter/minor-improvements


📝 Commits (1)

  • e5b8b54 refactor(DrizzleAdapter): minor code improvements

📊 Changes

1 file changed (+2 additions, -2 deletions)

View changed files

📝 packages/better-auth/src/adapters/drizzle-adapter/drizzle-adapter.ts (+2 -2)

📄 Description

  • made getSchema have a return type for a more strict type safety.
  • removed getModelName from within the getSchema call in line 189 since the getSchema fn already uses getModleName, it’s redundant.

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/better-auth/better-auth/pull/1137 **Author:** [@ping-maxwell](https://github.com/ping-maxwell) **Created:** 1/4/2025 **Status:** ❌ Closed **Base:** `main` ← **Head:** `refactor/drizzle-adapter/minor-improvements` --- ### 📝 Commits (1) - [`e5b8b54`](https://github.com/better-auth/better-auth/commit/e5b8b54356719cf10fa744277f9a9bc1c89064e3) refactor(DrizzleAdapter): minor code improvements ### 📊 Changes **1 file changed** (+2 additions, -2 deletions) <details> <summary>View changed files</summary> 📝 `packages/better-auth/src/adapters/drizzle-adapter/drizzle-adapter.ts` (+2 -2) </details> ### 📄 Description * made `getSchema` have a return type for a more strict type safety. * removed `getModelName` from within the `getSchema` call in line 189 since the `getSchema` fn already uses `getModleName`, it’s redundant. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
GiteaMirror added the pull-request label 2026-03-13 11:03:11 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/better-auth#3650