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

Convert more constructors to primary constructors #225

Merged
merged 2 commits into from
Nov 21, 2023
Merged

Convert more constructors to primary constructors #225

merged 2 commits into from
Nov 21, 2023

Conversation

tjementum
Copy link
Member

@tjementum tjementum commented Nov 21, 2023 •
edited
Loading

Summary & Motivation

Refactoring the remaining parts of the codebase to make use of primary constructors wherever possible.

Checklist

  • I have added a Label to the pull-request
  • I have added tests, and done manual regression tests
  • I have updated the documentation, if necessary

@tjementum tjementum self-assigned this Nov 21, 2023
@tjementum tjementum added the Enhancement New feature or request label Nov 21, 2023
@tjementum tjementum linked an issue Nov 21, 2023 that may be closed by this pull request
Convert code to use to primary constructors #216
Closed
@tjementum tjementum changed the title Refactor user command and queries to use primary constructors Convert more constructors to primary constructors Nov 21, 2023
@sonarqubecloud SonarQubeCloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 3 Code Smells

100.0% 100.0% Coverage
0.0% 0.0% Duplication

@tjementum tjementum merged commit 75f545b into main Nov 21, 2023
7 checks passed
@tjementum tjementum deleted the add-primary-constructors-to-user-commands branch November 21, 2023 12:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@tjementum tjementum

Labels
Enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Convert code to use to primary constructors
1 participant
@tjementum