make file auto clone

[abdelaziz-mahdy]

the make file will auto clone repo, before generating now since i saw you working on #299

[codecov-commenter]

⚠️ Please install the to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 92.03%. Comparing base (0f1f788) to head (bc98fd4).

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #300   +/-   ##
=======================================
  Coverage   92.03%   92.03%           
=======================================
  Files          49       49           
  Lines        9129     9129           
=======================================
  Hits         8402     8402           
  Misses        727      727           

Flag	Coverage Δ
unittests	92.03% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[rainyl]

the make file will auto clone repo, before generating now since i saw you working on #299

Yes I have implemented the dart code, but havn't finished the tests, you can work on it if you are willing to. :)

And for the makefile, it seems you are cloning the rainyl/dartcv.git and moving it to packages/dartcv/src, why don't you use the submodule commands? I mean

# clone the repo and submodule
git clone https://github.com/rainyl/opencv_dart.git --recursive

# if you forgot --recursive during the cloning
git submodule update --init --recursive

# update to the latest commit of the submodule
git submodule update --remote

Therefore, I think if we want to simplify the above steps, just let make submodule_update to run git submodule update --init --recursive if the directory packages/dartcv/src is empty, else run git submodule update --remote, what do you think?

[abdelaziz-mahdy]

Well, I didn't notice at first that these were submodules, so I updated the Makefile to make it work as you mentioned. Let me know what you think. (I think this is safer since anyone who wants to contribute can just run the make command.)

[rainyl]

Let me know what you think. (I think this is safer since anyone who wants to contribute can just run the make command.)

Sorry for my misleading 😢 . Agree it is safer, just do it~

[abdelaziz-mahdy]

Let me know what you think. (I think this is safer since anyone who wants to contribute can just run the make command.)

Sorry for my misleading 😢 . Agree it is safer, just do it~

I already did 😅, and looks like it works correctly

[rainyl]

@abdelaziz-mahdy Merged, thanks～