Skip to content

Time sum with literals : forgotten fixes required by review#3505

Closed
guilpier-code wants to merge 1 commit intodevelopfrom
fix/time-sum-for-literals
Closed

Time sum with literals : forgotten fixes required by review#3505
guilpier-code wants to merge 1 commit intodevelopfrom
fix/time-sum-for-literals

Conversation

@guilpier-code
Copy link
Copy Markdown
Contributor

@guilpier-code guilpier-code commented Mar 19, 2026

Apply some non pushed fixes due to remarks made by a reviewer (@flomnes) on closed PR 3464.

In the initial PR, I replied to remarks saying I made the required fixes. But I never pushed them eventually, until now.

@sonarqubecloud
Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
B Maintainability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

@guilpier-code guilpier-code changed the title Time sum with literals : fix required by review Time sum with literals : forgotten fixes required by review Mar 20, 2026
@flomnes
Copy link
Copy Markdown
Member

flomnes commented Mar 24, 2026

I prefer the previous version with explicit constructors. Sorry !

@flomnes flomnes closed this Mar 24, 2026
@guilpier-code guilpier-code deleted the fix/time-sum-for-literals branch April 17, 2026 15:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants