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.

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

routes_it.rs:26-29: Import assert_that!() macro like in... #85

Open
0pdd opened this issue Jul 18, 2024 · 2 comments
Open

routes_it.rs:26-29: Import assert_that!() macro like in... #85

0pdd opened this issue Jul 18, 2024 · 2 comments
Labels
bug Something isn't working pdd

Comments

@0pdd
Copy link
Collaborator

0pdd commented Jul 18, 2024

The puzzle 79-f3acfe10 from #79 has to be resolved:

// @todo #79:25min Import assert_that!() macro like in lib.rs instead of assert_that.
// For some reason we can't use '#[macro_use] extern crate hamcrest;'
// inside this mod. We should figure out why and replace deprecated import
// with required macro.

The puzzle was created by rultor on 18-Jul-24.

Estimate: 25 minutes, role: DEV.

If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

Copy link

@0pdd thanks for the report, here is a feedback:

  • The todo message could be more specific about the issue with importing the macro. It just states that there is an issue, but doesn't provide enough details for someone to understand and fix the problem easily. Context about why the current import method is not working would be helpful.
  • The todo message does not provide any guidance on how to fix the issue. It just describes the problem. Providing some suggestions on how to approach fixing the problem would make this todo more valuable to the codebase contributor.
  • The todo message does not mention any urgency or priority associated with the task. It would be helpful to know if this task is more important than others, so that contributors can prioritize their work accordingly.

Please fix the bug report in order it to get resolved faster.
Analyzed with Phind/Phind-CodeLlama-34B-v2

@0pdd
Copy link
Collaborator Author

0pdd commented Jul 18, 2024

@0pdd thanks for the report, here is a feedback: * The todo message could be more specific about the...

I see you're talking to me, but I can't reply since I'm not a chat bot.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working pdd
Projects
None yet
Development

No branches or pull requests

1 participant