Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Evaluates the complexity and maintainability of code #35

Closed
eduardolundgren opened this issue Sep 26, 2014 · 1 comment
Closed

Evaluates the complexity and maintainability of code #35

eduardolundgren opened this issue Sep 26, 2014 · 1 comment

Comments

@eduardolundgren
Copy link
Contributor

Source code complexity evaluation is now part of test task. If it's too complex tests are going to fail. And they are failing now, specially because Trie and EventEmitter.
Let's go through each of our modules reducing the complexity of them.
/cc @mairatma @ipeychev

@mairatma
Copy link
Contributor

Makes sense, these are functions that are more complex, we can separate
them into simpler function calls. I can do this quickly today.

On Fri, Sep 26, 2014 at 7:44 PM, Eduardo Lundgren [email protected]
wrote:

Source code complexity evaluation is now part of test task. If it's too
complex tests are going to fail. And they are failing now, specially
because Trie and EventEmitter.
Let's go through each of our modules reducing the complexity of them.
/cc @mairatma https://github.com/mairatma @ipeychev
https://github.com/ipeychev

https://camo.githubusercontent.com/c83114ea062ede5206e992abf8fc0784ebdff53c/687474703a2f2f6564756172646f2e696f2f696d6167652f33393358336c3172323630742f53637265656e25323053686f74253230323031342d30392d3237253230617425323031322e33372e3237253230414d2e706e67


Reply to this email directly or view it on GitHub
#35.

Maira Bello
Software Engineer

Liferay Inc.
Enterprise. Open Source. For Life
www.liferay.com

@mairatma mairatma self-assigned this Sep 29, 2014
clovisdasilvaneto pushed a commit to clovisdasilvaneto/metal.js that referenced this issue Jun 7, 2017
Adds a link to the metal slack chat.
jbalsas pushed a commit that referenced this issue Jan 18, 2018
Adds a link to the metal slack chat.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants