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

chore: Add csharpier #11

Merged
merged 1 commit into from
Nov 26, 2024
Merged

chore: Add csharpier #11

merged 1 commit into from
Nov 26, 2024

Conversation

bilby91
Copy link
Member

@bilby91 bilby91 commented Nov 26, 2024

This pull request includes several changes aimed at improving code formatting, updating the CI workflow, and refactoring test methods for better readability. The most important changes are:

CI Workflow Improvements:

  • Updated .github/workflows/ci.yaml to include a new linting job using csharpier for code formatting checks. This change ensures code consistency and quality by running dotnet csharpier as part of the CI process.

Code Formatting:

  • Added csharpier tool configuration to .config/dotnet-tools.json to standardize code formatting across the project.
  • Refactored TodoApi/Program.cs and TodoApi/Data/TodoContext.cs to improve code readability by using more concise syntax. [1] [2]

Test Code Refactoring:

  • Improved readability of test methods in TodoApi.Tests/Controllers/TodoListsControllerTests.cs by reformatting Assert.Equal statements and method calls. These changes make the tests easier to read and maintain. [1] [2] [3] [4] [5] [6]

Migration Script Formatting:

  • Reformatted the migration script in TodoApi/Migrations/20241126134649_CreateTodoLists.cs to improve readability by adjusting the indentation and line breaks.

@bilby91 bilby91 merged commit 125dde1 into main Nov 26, 2024
2 checks passed
@bilby91 bilby91 deleted the chore/add-csharpier branch November 26, 2024 14:14
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