-
Notifications
You must be signed in to change notification settings - Fork 0
Comparing changes
Open a pull request
base repository: TransactionProcessing/CallbackHandler
base: deepsource-autofix-e0c9e98b
head repository: TransactionProcessing/CallbackHandler
compare: main
- 12 commits
- 9 files changed
- 3 contributors
Commits on Sep 18, 2025
-
Merge pull request #133 from TransactionProcessing/deepsource-autofix…
…-e0c9e98b refactor: use file scoped namespaces
Configuration menu - View commit details
-
Copy full SHA for 66fdec1 - Browse repository at this point
Copy the full SHA 66fdec1View commit details
Commits on Sep 19, 2025
-
refactor: use file scoped namespaces
This PR modernizes the C# codebase by converting all traditional block-style namespace declarations to file-scoped namespace syntax. This change flattens the file structure, removes unnecessary indentation levels, and aligns the code with the latest C# conventions. - Use File Scoped `namespace`s instead of typical `namespace`s: Many source files originally declared block-scoped namespaces with braces, causing extra nesting and indentation. Each file has been updated to use the file-scoped namespace syntax (ending with a semicolon), and the `using` directives and class/record declarations have been moved to the top of the file. This refactor improves readability and consistency across the project. > This Autofix was generated by AI. Please review the change before merging.
Configuration menu - View commit details
-
Copy full SHA for cda200b - Browse repository at this point
Copy the full SHA cda200bView commit details -
Configuration menu - View commit details
-
Copy full SHA for 4a3a5f1 - Browse repository at this point
Copy the full SHA 4a3a5f1View commit details -
refactor: simplify lambda when its body has a single statement
This PR refactors lambda expressions that contain only a single statement to use more concise, expression-bodied forms and improves readability across chained calls. - Consider simplifying lambda when its body has a single statement: The original code used a block lambda for an async call and a tightly coupled Setup expression. In the first patch, the `Setup(a => a.GetLatestVersion(...))` call is reformatted across multiple lines for clarity while retaining its expression-bodied form. In the second patch, the multi-line block inside `Should.NotThrow(async () => { await ... })` is converted into an inline, expression-bodied async lambda, removing unnecessary braces and making the code more succinct. > This Autofix was generated by AI. Please review the change before merging.
Configuration menu - View commit details
-
Copy full SHA for 5d6a824 - Browse repository at this point
Copy the full SHA 5d6a824View commit details -
Merge pull request #134 from TransactionProcessing/deepsource-autofix…
…-9ce2dbf5 refactor: use file scoped namespaces
Configuration menu - View commit details
-
Copy full SHA for 332f413 - Browse repository at this point
Copy the full SHA 332f413View commit details -
refactor: rename DockerHelper subclass to CallbackHandlerDockerHelper
This PR renames the child class to prevent name collisions with its parent class. - Child and parent classes cannot share the same name: The issue arose because the subclass "DockerHelper" shared the exact name with its base class, leading to ambiguity and compilation errors. We addressed this by renaming the subclass to "CallbackHandlerDockerHelper", clearly distinguishing it from the parent. > This Autofix was generated by AI. Please review the change before merging.
Configuration menu - View commit details
-
Copy full SHA for 1c2dfd4 - Browse repository at this point
Copy the full SHA 1c2dfd4View commit details -
Configuration menu - View commit details
-
Copy full SHA for 585b643 - Browse repository at this point
Copy the full SHA 585b643View commit details -
Merge pull request #135 from TransactionProcessing/deepsource-autofix…
…-4a8f08f9 refactor: rename DockerHelper subclass to CallbackHandlerDockerHelper
Configuration menu - View commit details
-
Copy full SHA for e9bc017 - Browse repository at this point
Copy the full SHA e9bc017View commit details -
refactor: enforce one public class per namespace
This PR refactors the code to comply with the guideline of having a single public class per namespace. It removes an unintended BOM character, standardizes brace placement, and adds missing closing braces to properly scope classes within their namespaces. - Consider having 1 `public` class per namespace: DeepSource flagged that having multiple public classes in a single namespace can lead to tangled code organization and lower maintainability. The patch removes a hidden BOM character in the using directive that was causing inconsistent file encoding. It also standardizes opening brace placement and adds missing closing braces to ensure each public class is scoped within its own namespace. > This Autofix was generated by AI. Please review the change before merging.
Configuration menu - View commit details
-
Copy full SHA for eff51be - Browse repository at this point
Copy the full SHA eff51beView commit details -
refactor: declare classes and structs within a namespace
This PR reorganizes the code by encapsulating all classes within the `CallbackHandler.IntegrationTests.Common` namespace, aligning with C# best practices and improving code maintainability and scope management. - Classes and structs should be declared within a `namespace`: DeepSource flagged that several classes (`GenericSteps`, `TestingContext`, `Setup`) were declared at the file root without a namespace. We introduced the `CallbackHandler.IntegrationTests.Common` namespace and wrapped these class definitions inside it, updating indentation and braces to ensure proper encapsulation and readability. > This Autofix was generated by AI. Please review the change before merging.
Configuration menu - View commit details
-
Copy full SHA for ccb78d8 - Browse repository at this point
Copy the full SHA ccb78d8View commit details -
Configuration menu - View commit details
-
Copy full SHA for da9f6f7 - Browse repository at this point
Copy the full SHA da9f6f7View commit details -
Merge pull request #136 from TransactionProcessing/deepsource-autofix…
…-cd47f680 refactor: enforce one public class per namespace
Configuration menu - View commit details
-
Copy full SHA for 492f425 - Browse repository at this point
Copy the full SHA 492f425View commit details
This comparison is taking too long to generate.
Unfortunately it looks like we can’t render this comparison for you right now. It might be too big, or there might be something weird with your repository.
You can try running this command locally to see the comparison on your machine:
git diff deepsource-autofix-e0c9e98b...main