datahike icon indicating copy to clipboard operation
datahike copied to clipboard

clean up project files

Open kordano opened this issue 2 years ago • 3 comments

The namespaces are too complected and need some refactoring. Unused and deprecated functions need to be removed. Additionally, the folder structure needs to be reviewed and a better package structure recommended. Also have a look at the test folder and its strange structure.

kordano avatar Jun 24 '22 11:06 kordano

The db namespace will be refactored with #555

Concerning the test folder, I can see, that

  • the second test subfolder can be removed, i.e. test->datahike->test->api_test.cljc would become test->datahike->api_test.cljc
  • test.cljc can be removed
  • backward_compatibility_test is something different than the rest of the tests and should be moved out of the test folder
  • the js folder probably can be removed as well (there is a js test runner that can be pulled on demand)

What do you think?

jsmassa avatar Aug 31 '22 16:08 jsmassa

I think, we should talk about some of the unused functions. I remember that for some of them I do not know, why they are never used.

jsmassa avatar Aug 31 '22 16:08 jsmassa

Here is another list of refactoring ideas I found:

  • error handling
  • refactoring of README
    • sections into files
    • contribute md -> more, also as developer guide for work with repo
      • i.e. conventional commits, formatting
  • core namespace refactoring -> which functions can be moved?
  • transactor.clj: refactoring for easier understanding

jsmassa avatar Sep 06 '22 15:09 jsmassa