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

feat: refactor db tasks for simplicity #1431

Merged
merged 1 commit into from
Aug 19, 2023
Merged

feat: refactor db tasks for simplicity #1431

merged 1 commit into from
Aug 19, 2023

Conversation

lgrossi
Copy link
Contributor

@lgrossi lgrossi commented Aug 17, 2023 •
edited
Loading

Description

Small refactor in the database tasks to simplify code e separate responsibilities (execution vs store).

@lgrossi lgrossi force-pushed the lucas/poc-remove-db-task branch 2 times, most recently from c06e74e to c2c115c Compare August 17, 2023 20:06
@lgrossi lgrossi changed the base branch from main to lucas/thread-pool-2 August 17, 2023 20:06
@lgrossi lgrossi force-pushed the lucas/poc-remove-db-task branch 3 times, most recently from 083879e to 3a175de Compare August 17, 2023 21:34
@lgrossi lgrossi marked this pull request as ready for review August 17, 2023 22:34
@lgrossi lgrossi force-pushed the lucas/poc-remove-db-task branch from 3a175de to 578818b Compare August 17, 2023 22:36
Base automatically changed from lucas/thread-pool-2 to main August 18, 2023 07:46
@lgrossi lgrossi changed the title feat: Remove db task from ban feat: Refactor db tasks for simplicity Aug 18, 2023
@lgrossi lgrossi force-pushed the lucas/poc-remove-db-task branch 2 times, most recently from d2ff113 to 0d93c80 Compare August 19, 2023 19:48
@lgrossi lgrossi force-pushed the lucas/poc-remove-db-task branch from 0d93c80 to 4723f13 Compare August 19, 2023 19:51
@dudantas dudantas changed the title feat: Refactor db tasks for simplicity feat: refactor db tasks for simplicity Aug 19, 2023
@sonarqubecloud
Copy link

Please retry analysis of this Pull-Request directly on SonarCloud.

@lgrossi lgrossi merged commit 6c7c863 into main Aug 19, 2023
@lgrossi lgrossi deleted the lucas/poc-remove-db-task branch August 20, 2023 08:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dudantas dudantas dudantas approved these changes

Assignees
No one assigned
Labels
Area: Source
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@lgrossi @dudantas