-
Notifications
You must be signed in to change notification settings - Fork 121
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
[Guide] Search #473
Comments
Hi, I want to work on this issue |
@sayuree go for it :) |
@nhtruong I already had a guide ready, can I do a PR? It's not very right of me, but I didn't work for nothing either :[ |
@Nicksqain Well, I assume that according to the code of conduct, you are not allowed to work on already assigned issues. I admit that I did not submit the changes/commits right away, but still, it does not give an opportunity for others to take up the issue. There are still a bunch of issues you could work on, instead of attaching PR on already assigned ones. Please consider the code of conduct in the future. |
@nhtruong Yes, I apologize for my actions. It won't happen again. If suddenly the appointed person for some reason will not be able to work, then please accept my candidacy. I'm not trying to take away the time spent. It's just that I already did, but couldn't ask to be assigned to me. I apologize again. |
@Nicksqain It's all good. I appreciate your candor and passion for this project. Since @sayuree has already been assigned to this issue, they have the priority on this issue. Feel free to pick up any other unclaimed issues and keep up the great work :D |
Create
guides/search.md
similar to the Ruby guideYou must assure that:
try/catch
and print out the expected error.```ruby
)The text was updated successfully, but these errors were encountered: