-
Notifications
You must be signed in to change notification settings - Fork 115
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
implement StyleCI #92
Comments
It allows all repos you're "admin" on, rather than ones you physically own. |
@GrahamCampbell Ah, alright. Can't have multiple admins on personal repos, I believe. |
Nope, you deffo can. I know repos that have like 12 admins on them. |
GitHub let's admins choose "read", "write", "admin" when you're added as a collaborator. |
@GrahamCampbell I can't find that option in any of my repos. Would you post a screenshot / tutorial to demonstrate? +github docs
-- from Adding collaborators to a personal repository
|
anyways, putting us back on topic. @GrahamCampbell feel free to email me if you find out, email is on github profile
|
To be honest, if boris has multiple maintainers, it would make sense to move it to an organisation. We did that a few weeks ago with class preloader. https://github.com/ClassPreloader/ClassPreloader |
@GrahamCampbell - Makes sense. Chaining @d11wtq in on this 👍 |
I've converted this repository to an organization and you have admin access now. No objections to StyleCI, nope. I'm easy on coding style, just as long as it's consistent, although I do have a dislike for tabs mostly because they render terribly/unpredictably in things like cat/less/the browser. |
StyleCI only supports fixing to spaces atm anyway. :) |
Why would you want tabs? lol |
@GrahamCampbell I mean tabs do have their advantages, like being able to set the appearance custom in your environment, but @d11wtq brings up a good point so we will continue to use spaces. I'll implement this before EOD tmrw. |
https://styleci.io/
@d11wtq, what do you think about implementing StyleCI? I would myself, but it only allows you to do this on repos you own, not collaborate to.
The text was updated successfully, but these errors were encountered: