Skip to content

Comments

test-4-cli#49

Merged
yuvalyacoby merged 2 commits intobaz-scm:mainfrom
baz-test:test-for-cli
Dec 24, 2025
Merged

test-4-cli#49
yuvalyacoby merged 2 commits intobaz-scm:mainfrom
baz-test:test-for-cli

Conversation

@yuvalyacoby
Copy link
Contributor

@yuvalyacoby yuvalyacoby commented Dec 15, 2025

Generated description

Below is a concise technical summary of the changes proposed in this PR:
Correct the quadratic equation solver in python/edits.py by replacing b2 with b in the solution calculations and introducing a redundant sol23 variable.

Latest Contributors(0)
UserCommitDate
This pull request is reviewed by Baz. Review like a pro on (Baz).

sol2 = (-b+cmath.sqrt(d))/(2*a)
sol23 = (-b+cmath.sqrt(d))/(2*a)

print('The solution are {0} and {1}'.format(sol1, sol2))
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is the print on purpose ?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

x

@baz-reviewer baz-reviewer bot added the baz approved Baz-reviewer has approved this PR label Dec 15, 2025
@yuvalyacoby yuvalyacoby merged commit 36f7f59 into baz-scm:main Dec 24, 2025
yuvalyacoby added a commit that referenced this pull request Dec 24, 2025
This reverts commit 36f7f59.
@yuvalyacoby yuvalyacoby mentioned this pull request Dec 24, 2025
yuvalyacoby added a commit that referenced this pull request Dec 24, 2025
This reverts commit 36f7f59.
yuvalyacoby added a commit that referenced this pull request Dec 24, 2025
yuvalyacoby added a commit that referenced this pull request Dec 24, 2025
yuvalyacoby added a commit that referenced this pull request Dec 24, 2025
yuvalyacoby added a commit that referenced this pull request Dec 24, 2025
yuvalyacoby added a commit that referenced this pull request Dec 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

baz approved Baz-reviewer has approved this PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant