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

.coafile: Set max line length for project pages #583

Closed
jayvdb opened this issue May 23, 2018 · 13 comments
Closed

.coafile: Set max line length for project pages #583

jayvdb opened this issue May 23, 2018 · 13 comments

Comments

@jayvdb
Copy link
Member

jayvdb commented May 23, 2018

Similar to coala/cEPs#134 , the project pages (i.e. pages in https://github.com/coala/projects/tree/master/_projects) should have a reasonable line length to improve the ability to read & code review them.

c.f. coala/cEPs#157

Try length 120 for existing files so it requires less disruptive changes to the existing files.

Then add another section which requires 80 cpl and excludes the existing files.

@JameerBabu
Copy link

Hello everyone, I would like to work on this issue. Please assign me to work on it. @jayvdb please give me a chance.

@Man-Jain
Copy link
Member

@jayvdb Can you please guide me on how can I do this.
More specifically asking how can I exclude the existing files from the 80 cpl length.

@jayvdb
Copy link
Member Author

jayvdb commented Jul 20, 2018

Please read the issue carefully, as it explains how to proceed, and look at the linked pages and understand them.

@Man-Jain
Copy link
Member

Man-Jain commented Jul 20, 2018

Yes @jayvdb I have added a section with 120 line and fixed all the code that was violating the condition.
But what I am not able to under is how should I exclude the previous files. Should I mention all the previous file's names in ignore property or is there any other way.

@jayvdb
Copy link
Member Author

jayvdb commented Jul 20, 2018

Unfortunately no other way.

But lets create one after this .. coala/coala#5646 ;-)

@jayvdb
Copy link
Member Author

jayvdb commented Jul 20, 2018

You can list directories in the ignore. Dunno if that helps much.

Man-Jain added a commit to Man-Jain/projects that referenced this issue Jul 20, 2018
	  character length.
This commits ensures that the files in the _projects folder are having
a maximum line length of 120 characters for existing files whereas 80
characters for new files. This feature helps to improve the ability to
read the code and review it.

Fixes coala#583
Man-Jain added a commit to Man-Jain/projects that referenced this issue Jul 20, 2018
This commit ensures that the files in the _projects folder are having
a maximum line length of 120 characters for existing files whereas 80
characters for new files. This feature helps to improve the ability to
read the code and review it.

Fixes coala#583
Man-Jain added a commit to Man-Jain/projects that referenced this issue Jul 21, 2018
This commit ensures that the files in the _projects folder are having
a maximum line length of 120 characters for existing files whereas 80
characters for new files. This feature helps to improve the ability to
read the code and review it.

Fixes coala#583
Man-Jain added a commit to Man-Jain/projects that referenced this issue Jul 22, 2018
This commit ensures that the files in the _projects folder are having
a maximum line length of 120 characters for existing files whereas 80
characters for new files. This feature helps to improve the ability to
read the code and review it.

Fixes coala#583
Man-Jain added a commit to Man-Jain/projects that referenced this issue Jul 22, 2018
This commit ensures that the files in the _projects folder are having
a maximum line length of 120 characters for existing files whereas 80
characters for new files. This feature helps to improve the ability to
read the code and review it.

Fixes coala#583
Man-Jain added a commit to Man-Jain/projects that referenced this issue Jul 22, 2018
This commit ensures that the files in the _projects folder are having
a maximum line length of 120 characters for existing files whereas 80
characters for new files. This feature helps to improve the ability to
read the code and review it.

Fixes coala#583
Man-Jain added a commit to Man-Jain/projects that referenced this issue Jul 22, 2018
This commit ensures that the files in the _projects folder are having
a maximum line length of 120 characters for existing files whereas 80
characters for new files. This feature helps to improve the ability to
read the code and review it.

Fixes coala#583
@ksdme
Copy link
Member

ksdme commented Jul 22, 2018

It looks like 120 CPL isn't enough for old files, you should use the max length that passes all the current files and then ignore them in a new section with 80 CPL limit.

@Man-Jain
Copy link
Member

Man-Jain commented Jul 22, 2018

@ksdme Yes 120 was not enough so I fixed all those lines that were over 120 characters by splitting them up. And I have created the new section which ignores the old files and sets 80 cpl limit to new files.

@ksdme
Copy link
Member

ksdme commented Jul 22, 2018

120 CPL limit for old files is flexible, changing so many files isn't. Change the limit to whatever is required to pass all the files without splitting all of them.

@Man-Jain
Copy link
Member

I don't think there is a cpl limit that can be applied for all the files as many files were in the range of 200 and 300 and even more than that in some.

@ksdme
Copy link
Member

ksdme commented Jul 22, 2018

Maybe have 300+ CPL, I think it is better than changing all files.

WDYT @jayvdb

@Man-Jain
Copy link
Member

Okay If the two of you agree. I will do it. 😄

@ishanSrt
Copy link
Contributor

Maybe have 300+ CPL, I think it is better than changing all files.

@ksdme that surely isn't better that changing files

Man-Jain added a commit to Man-Jain/projects that referenced this issue Jul 23, 2018
This commit ensures that the files in the _projects folder are having
a maximum line length of 120 characters for existing files whereas 80
characters for new files. This feature helps to improve the ability to
read the code and review it.

Fixes coala#583
Man-Jain added a commit to Man-Jain/projects that referenced this issue Jul 23, 2018
Some files have links that are being reported as broken by the coala.
These changes fixes these links so that the build passes.

Fixes coala#583
Man-Jain added a commit to Man-Jain/projects that referenced this issue Jul 23, 2018
This commit ensures that the files in the _projects folder are having
a maximum line length of 120 characters for existing files whereas 80
characters for new files. This feature helps to improve the ability to
read the code and review it.

Fixes coala#583
Man-Jain added a commit to Man-Jain/projects that referenced this issue Jul 24, 2018
This commit ensures that the files in the _projects folder are having
a maximum line length of 120 characters for existing files whereas 80
characters for new files. This feature helps to improve the ability to
read the code and review it.

Closes coala#583
Man-Jain added a commit to Man-Jain/projects that referenced this issue Jul 25, 2018
This commit ensures that the files in the _projects folder are having
a maximum line length of 120 characters for existing files whereas 80
characters for new files. This feature helps to improve the ability to
read the code and review it.

Closes coala#583
Man-Jain added a commit to Man-Jain/projects that referenced this issue Jul 29, 2018
This commit ensures that the files in the _projects folder are having
a maximum line length of 120 characters for existing files whereas 80
characters for new files. This feature helps to improve the ability to
read the code and review it.

Closes coala#583
Man-Jain added a commit to Man-Jain/projects that referenced this issue Jul 29, 2018
This commit ensures that the files in the _projects folder are having
a maximum line length of 120 characters for existing files whereas 80
characters for new files. This feature helps to improve the ability to
read the code and review it.

Closes coala#583
Man-Jain added a commit to Man-Jain/projects that referenced this issue Jul 30, 2018
This commit ensures that the files in the _projects folder are having
a maximum line length of 120 characters for existing files whereas 80
characters for new files. This feature helps to improve the ability to
read the code and review it.

Closes coala#583
Man-Jain added a commit to Man-Jain/projects that referenced this issue Jul 31, 2018
This commit ensures that the files in the _projects folder are having
a maximum line length of 120 characters for existing files whereas 80
characters for new files. This feature helps to improve the ability to
read the code and review it.

Closes coala#583
Man-Jain added a commit to Man-Jain/projects that referenced this issue Aug 5, 2018
This commit ensures that the files in the _projects folder are having
a maximum line length of 120 characters for existing files whereas 80
characters for new files. This feature helps to improve the ability to
read the code and review it.

Closes coala#583
Man-Jain added a commit to Man-Jain/projects that referenced this issue Aug 5, 2018
This commit ensures that the files in the _projects folder are having
a maximum line length of 120 characters for existing files whereas 80
characters for new files. This feature helps to improve the ability to
read the code and review it.

Closes coala#583
Man-Jain added a commit to Man-Jain/projects that referenced this issue Aug 6, 2018
This commit ensures that the files in the _projects folder are having
a maximum line length of 120 characters for existing files whereas 80
characters for new files. This feature helps to improve the ability to
read the code and review it.

Closes coala#583
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

5 participants