-
Notifications
You must be signed in to change notification settings - Fork 4
Boilerplate review #4
Comments
As for the last point - the README.md could consist of two parts - description of the boilerplate and how to use it and a part to merge into inheriting project's README. |
Doesn't sound like a mandatory feature and should not be a blocker. A new issue should be opened for it for future enhancement.
I really want to avoid having the root bloated. It already has lots of files and my idea was leaving there just the files that will be touched by every project. Unfortunately just JSCS and JSHint config files shouldn't be touched, ideally, as all projects will be following the same standards. So it looks like ok to have them in the "protected" place they are right now. As for IDEs, it may be a pity to not have this automated, but at least with mine I'm able to configure the project to read it from a different location. Hopefully the benefit of having a less bloated root compensates it.
I don't understand this. Right now we're using .gitignore only.
Right now they already do this. It's the .gitignore file.
If you mean anything more than this, please feel free to push fixes directly.
I think the way it is described right now is more than sufficient. It's concise. I understand your point though, but we're really taking about a small part of the README file, while all the rest would be dropped. I think that most of the people would not even consider using it because of this. One idea would be having a template for README in the wiki, having the main README point to it. |
Fair enough :).
Here you have them twice: https://github.com/ckeditor/ckeditor-boilerplate/blob/master/gruntfile.js. And project like CKEditor has some assets which should not be checked, so physically I need to add them to both these arrays. I can reuse one array and that's exactly what I meant - there should be only one array defined in that file. It's a detail of course.
It's good if you already know how it works, but I don't think it's enough for other developers who may not be familiar with Grunt and git hooks. So it's ok for boilerplate's README, but projects' READMEs should be written from other perspective. I will need to write something for ckeditor-dev, so I'll put it in the wiki. |
I'll address the remaining points (mainly API docs and the wiki article) and close this ticket. |
While reviewing http://dev.ckeditor.com/ticket/12721 I made a short list of things that we could improve:
I miss an ability to check a dir or path using the(Extracted to Grunt jscs and jshint tasks checking only specified directory/file #5)jscs
andjshint
tasks. In other words I want to be able to execute:grunt jscs core/
.I think that we could keep JSCS and JSHint configs in the root dir (the same way as they are now in https://github.com/ckeditor/ckeditor-dev). They will be easier to find there and this is the standard location and file names so perhaps some IDEs or other tools can benefit from that automatically. The thing that I believe we can't do is to keep ignored files lists as they are now in https://github.com/ckeditor/ckeditor-dev, because: a) we want to share the list, b) we want to extend them with(Dropped).gitignore
.jscs
andjshint
can share the list of ignored files - it's hard to think of a realistic case in which these lists should differ.The text was updated successfully, but these errors were encountered: