Skip to content

Conversation

@tigalab
Copy link

@tigalab tigalab commented Mar 4, 2020

This pull request contains the improvements made by the Tiga for Wirecloud. These improvements are part of the fiQare project, which is based on ISO 25010 to improve software quality. More info: https://fiqare.eu/

@coveralls
Copy link

Coverage Status

Coverage decreased (-28.5%) to 53.372% when pulling 0b9ec23 on tigalab:tigalab into 1ddcf1b on Wirecloud:develop.

@aarranz
Copy link
Member

aarranz commented Mar 10, 2020

Hi @tigalab,

Thank you very much for your contributing into WireCloud.

However, I've to say that the PR is very large and I don't know what are the advantages of applying it. Can you elaborate it?. Also, it may be a good idea to split it into several PRs.

Best Regards,
Álvaro

@tigalab
Copy link
Author

tigalab commented Apr 16, 2020

Hi @aarranz ,

I am grateful for your review, thank you very much,

This PR contain some minor code improvements which is made by a tool.
This why this PR seems very large however i can not say it is a big problem these are just small code corrections but i am going to seperate these to 3 part.

It also might be a little big but i think it will be much better for you.

Our best compliments,
Tigalab.

@tigalab tigalab closed this Jun 5, 2020
@tigalab tigalab reopened this Jun 5, 2020
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

Successfully merging this pull request may close these issues.

3 participants