Skip to content

Conversation

@Youssefhajjaoui
Copy link
Contributor

No description provided.

@pedrodesu
Copy link
Contributor

Hello @Youssefhajjaoui!

We appreciate your PR. However, as one of the many additions of the overhaul the Rust piscine is undergoing at the moment, this exercise and its tests are improved. Among adding the required std::rc::Rc, which in fact was not present before, some other more notorious changes will be present in this exercise, and the entire quest in fact - As such, this request is unnecessary at the moment and will be closed. You should be able to notice the new changes for this quest in a matter of days. Please bear with us for a moment as we finish our work on it and contact your campus administration if you and your peers feel like the new changes to the piscine should be pushed with more urgency.

Thank you for your patience and please let me know if you have any more questions!

@pedrodesu pedrodesu closed this Jul 2, 2025
@pedrodesu pedrodesu changed the title import Rc in main test [EXTERNAL] Import Rc in main test Jul 2, 2025
@pedrodesu pedrodesu self-assigned this Jul 2, 2025
@pedrodesu pedrodesu added enhancement New feature or request wontfix This will not be worked on labels Jul 2, 2025
@Youssefhajjaoui
Copy link
Contributor Author

Hello @pedrodesu ,
Thank you for taking the time to review my PR I really appreciate it!
With patience and curiosity, I put together this contribution. I just had a quick question: would it be alright if I suggest some test cases for quests, purely out of curiosity and interest?

@pedrodesu
Copy link
Contributor

Hello @Youssefhajjaoui!

Of course, any suggestions are very much welcome. We strive to improve our content continuously and there's no better way do so than with the feedback of our own students. Feel free to give any suggestions or feedback through a new issue/PR, and follow the expected template when doing so. :)

Thank you!

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

Labels

enhancement New feature or request wontfix This will not be worked on

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants