search

kevmoens / Ellabit

The interactive C# coding app to learn/practice coding.
MIT License
55 stars 21 forks source link

Create tests for linq queries #4

Open kevmoens opened 2 years ago

10xJosh commented 1 year ago

Is this still up for grabs?

kevmoens commented 1 year ago

YesSent from my iPhoneOn Nov 19, 2023, at 12:56 PM, Josh Davis @.***> wrote: Is this still up for grabs?

—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

10xJosh commented 1 year ago

Alright awesome! I'd love to take this one on! I'm new to contributing to open source so if you have any feedback to give me at all let me know! I really want to improve and get better!

10xJosh commented 1 year ago

Hey Kevin!

I took a look at the code and noticed that under the "Needs Review" folder within the "Challenges" folder, there are Test methods that have

in the return statements of the test methods. (Example: https://github.com/kevmoens/Ellabit/blob/b82f5ed47c19a06f250b013d14aaa19c4bedaa57/Ellabit/Challenges/NeedsReview/DescriptionReview/Challenge375Fareysequence.cs#L60C30-L60C48)

I wanted to confirm that its those challenges that have the tag in the code that are the ones that need to be created, correct?