-
Notifications
You must be signed in to change notification settings - Fork 507
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
Cpp Solution: Connecting Two Barns #4990
Conversation
for more information, see https://pre-commit.ci
I thought these were allowed under here but I can fix them |
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thank you for your contribution!
Resolve comments
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
some minor changes
gj
Co-authored-by: envyaims <[email protected]>
Co-authored-by: envyaims <[email protected]>
Change cnt to num_components
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok i made some changes if you could test it again that'd be great
for more information, see https://pre-commit.ci
Not sure what testing means but it still passes the problem submission. Should I be the one updating the branch? |
uh no i just made some changes that would've been easier to make myself |
Place an "x" in the corresponding checkbox if it is done or does not apply to this pull request.