search

codingteam / loglist

Reincarnation of the famous service
https://loglist.xyz/
MIT License
7 stars 4 forks source link

Refactor out helpers package #205

Open rexim opened 7 years ago

rexim commented 7 years ago

Derived from https://github.com/codingteam/loglist/pull/204#discussion_r115435058

ForNeVeR commented 7 years ago

For the record (GitHub didn't properly show the linked comment), the comment text is:

"Helpers" and "Utilities" are instances of coincidental cohesion which is the worst possible cohesion according to Larry Constantine https://en.wikipedia.org/wiki/Cohesion_(computer_science)#Types_of_cohesion

I will create an issue to eliminate this package, but for I now I strongly recommend to create cors package and move everything related to CORS there.