search

k06a / boolinq

Simplest C++ header-only LINQ template library
MIT License
628 stars 79 forks source link

Configure one of the tests to use C++14 #54

Closed k06a closed 2 years ago

k06a commented 2 years ago

This select lambda argument type in this test looks insane:

https://github.com/k06a/boolinq/blob/57ece8eff2bf9c9b2076d07f9724a318c37d3d8c/test/GroupByTest.cpp#L172-L180

  1. Extract this test to separate target test14 (create it) or something and configure it to use C++14 and use auto instead of this long type.
  2. Add this example to README, specify that C++14 is required to use auto here.
gitcoinbot commented 2 years ago

Issue Status: 1. Open 2. Started 3. Submitted 4. Done

This issue now has a funding of 0.03 ETH (113.79 USD @ $3792.85/ETH) attached to it.

gitcoinbot commented 2 years ago

Issue Status: 1. Open 2. Started 3. Submitted 4. Done

Work has been started.

These users each claimed they can complete the work by 264 years, 10 months from now. Please review their action plans below:

1) nargeslein has been approved to start work.

Hi k06a, I am happy to work on this. I am a general software developer with many years (14 years+) of experience in C#.NET and have also worked in Java/Rust/C++/SQL.

I would start by making myself familiar with the project and then follow this action plan (according to your request)

In addition I would suggest recommendations if I spot any and rediscuss with you.

Best regards

Learn more on the Gitcoin Issue Details page.

gitcoinbot commented 2 years ago

Issue Status: 1. Open 2. Started 3. Submitted 4. Done

Work for 0.03 ETH (112.16 USD @ $3738.66/ETH) has been submitted by:

  1. @nargeslein

@k06a please take a look at the submitted work:

gitcoinbot commented 2 years ago

Issue Status: 1. Open 2. Started 3. Submitted 4. Done

The funding of 0.03 ETH (110.54 USD @ $3684.59/ETH) attached to this issue has been approved & issued to @nargeslein.