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

SqlCommand #73

Merged
merged 3 commits into from
Jan 10, 2024
Merged

SqlCommand #73

merged 3 commits into from
Jan 10, 2024

Conversation

asgerhallas
Copy link
Owner

No description provided.

@asgerhallas asgerhallas merged commit 4eaf8b7 into master Jan 10, 2024
1 of 2 checks passed
@asgerhallas asgerhallas deleted the sql-command branch January 10, 2024 09:57
Copy link

what-the-diff bot commented Jan 10, 2024

PR Summary

  • Addition of Unit Tests for 'SqlCommand' and 'SqlBuilder' classes
    The team added two new files named SqlCommandTests.cs and SqlBuilderTests.cs. These files contain the unit tests for SqlCommand and SqlBuilder classes respectively. This ensures the features they offer are working as desired and is an essential part of maintaining the robustness and dependability of our software.

  • Implementation of 'SqlCommand' class
    Another new file named SqlCommand.cs was introduced. This is the implementation of the SqlCommand class. This class will facilitate executing SQL commands against a database, which is a crucial operation for any data-driven software.

  • Update in Configuration Settings
    Changes were made in the Configuration.cs file. A case was included for the execution of SqlCommand in CommandExecutor. This means that our CommandExecutor will now be able to handle and properly dispatch SQL Commands, expanding its capabilities.

  • Code Cleanup and Logic Improvement
    In the DocumentStoreEx.cs and the SqlBuilder.cs files, some unused code was removed (termed as "using statements"). This will improve efficiency within the code. In addition, changes were made to improve the logic of the Append method inside the SqlBuilder class, which could result in better and more efficient SQL command generation. The cleanup makes the code easier to read which aids future development and maintenance tasks.

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.

1 participant