Skip to content

refactor: Replaces direct DB session usage with context managers#20569

Merged
laipz8200 merged 1 commit intomainfrom
refactor/remove-directly-dbsession-use-in-workflow
Jun 3, 2025
Merged

refactor: Replaces direct DB session usage with context managers#20569
laipz8200 merged 1 commit intomainfrom
refactor/remove-directly-dbsession-use-in-workflow

Conversation

@laipz8200
Copy link
Copy Markdown
Member

Important

  1. Make sure you have read our contribution guidelines
  2. Ensure there is an associated issue and you have been assigned to it
  3. Use the correct syntax to link this PR: Fixes #<issue number>.

Summary

part of #20449

Screenshots

Before After
... ...

Checklist

  • This change requires a documentation update, included: Dify Document
  • I understand that this PR may be closed in case there was no previous discussion or issues. (This doesn't apply to typos!)
  • I've added a test for each change that was introduced, and I tried as much as possible to make a single atomic change.
  • I've updated the documentation accordingly.
  • I ran dev/reformat(backend) and cd web && npx lint-staged(frontend) to appease the lint gods

Signed-off-by: -LAN- <laipz8200@outlook.com>
@dosubot dosubot bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Jun 3, 2025
@laipz8200 laipz8200 marked this pull request as draft June 3, 2025 06:08
@dosubot dosubot bot added the 💪 enhancement New feature or request label Jun 3, 2025
@laipz8200 laipz8200 marked this pull request as ready for review June 3, 2025 06:17
@laipz8200 laipz8200 requested a review from crazywoola June 3, 2025 06:21
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Jun 3, 2025
@laipz8200 laipz8200 merged commit 72fdafc into main Jun 3, 2025
10 checks passed
@laipz8200 laipz8200 deleted the refactor/remove-directly-dbsession-use-in-workflow branch June 3, 2025 08:16
jsincorporated pushed a commit to jsincorporated/asaAi that referenced this pull request Jul 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

💪 enhancement New feature or request lgtm This PR has been approved by a maintainer size:M This PR changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants