Skip to content

suggested changes to LC 2677 chunk array#1

Open
danielzayas wants to merge 2 commits intomainfrom
dz-help
Open

suggested changes to LC 2677 chunk array#1
danielzayas wants to merge 2 commits intomainfrom
dz-help

Conversation

@danielzayas
Copy link
Copy Markdown
Collaborator

Feedback and suggested changes

// LC 2677 - Chunk Array -- S: O(), T: O()

/*
Nice work so far!
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

@elliotvhill I've suggested changes to your LC 2677 solution!

I would encouraged you to:

  1. create a new lc-2677 branch on your laptop
  2. make the changes to this file and then git push
  3. create a new pull request add the danielzayas GitHub user as a reviewer
  4. I will comment and suggest changes
  5. After it’s passing all the leetcode tests, we will merge it to the main branch

@danielzayas danielzayas requested a review from elliotvhill April 29, 2024 16:51
@elliotvhill elliotvhill self-assigned this Apr 30, 2024
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.

2 participants