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

Tigalab #474

Open
wants to merge 4 commits into
base: develop
Choose a base branch
from
Open

Tigalab #474

wants to merge 4 commits into from

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