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

[Do not merge] Demo PR for G-Search #230

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

[Do not merge] Demo PR for G-Search #230

wants to merge 4 commits into from

Conversation

jjsilva4
Copy link
Contributor

No description provided.

@jjsilva4 jjsilva4 self-assigned this Nov 22, 2024
@jjsilva4 jjsilva4 added bug Something isn't working documentation Improvements or additions to documentation labels Nov 22, 2024
@jjsilva4
Copy link
Contributor Author

Hey @jjsilva4, while reviewing your PR, I'd suggest the following code changes:

👉 Hey Trevor! I have a Code Suggestion for #230

#230

You can also review and apply these suggestions locally on your machine.

Learn more about GitKraken Code Suggest

Code Suggest liberates your code reviews from GitHub's restrictive, comment-only feedback style. As simple as suggesting changes in a Google-doc, provide real code suggestions from where you code, e.g. your IDE, and on anything in your project — not just on the lines of code changed in the PR.

Join your team on GitKraken to speed up PR review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@trevor-polidore trevor-polidore Awaiting requested review from trevor-polidore

Assignees

@jjsilva4 jjsilva4

Labels
bug Something isn't working documentation Improvements or additions to documentation
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jjsilva4 @trevor-polidore