-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
Consider Adding Continuous Integration #45
Comments
Hi @ashfurrow, yeah, sure. I'm all for that. I've done some investigation for CircleCI, but it asked me for admin permission to all of my repositories, which unfortunately I cannot do. That would mean that I would need to create a dummy github account just for that? I'm sure you have more experience then I have with that, so I would appreciate your help on this. |
Cool, I can take a look at that. I understand it's not practical to give access to app repos – a dummy account could work. Alternatively, if you're willing, we can use my account (I would need write access to the repo, which I totally understand if you're not OK with). I have already given CircleCI access to my account, so it's fine. |
I'm all for simple solutions :) I've added you as a collaborator, this should give you write access. |
Perfect – I'll try and get that setup this weekend. |
The write access is not enough for CircleCI, as discussed and tested few months ago, it requires the repo ownership and Kruno can't give access to all repos. The good solution would be to Kruno to create an association being the administrator, but the RxSwift name has been taken and not used since last year. According to Github rules is a violation, but I don't know how much they would care about it. :( |
Huh, you're right. Looks like I can only have CI on my fork. Hmm. If it's OK with everyone, I can contact GitHub to see about getting the organization name. |
(That is, of course, if @kzaher wants to move it to an org.) |
(I am on a phone, sorry for typos and missing words) I think Kruno is ok with that, we discussed this weeks ago, on that time the necessity wasn't big, now that the exposure of the project is getting bigger, it would probably make sense. I think the name is important, but on the other hand the Reactive scene has his own association, so I don't know if a dedicated one would be better than join the original one. The current RxSwift is an extreme accurate port of Rx.Net and has a lot to share with RxJava (some names and conventions are different, but mostly because of the language, not because of the implementation, as discussed in the Slack channel). I will let Kruno answer, also I would also love to push the website with valid tutorials and content, I will manage to draft a roadmap and publish it on Slack tomorrow. |
Whats' wrong with Travis-CI? Worked great for me in the past. |
I've already contacted GitHub :) The account doesn't have any publicly visible activity, but it isn't dormant. The person responded, but I didn't get the impression that he was willing to let the name go :( In theory, I don't have an issue moving this project to RxSwift organization or to a part of some other organization, but I don't want to do it over night because I wanted to include CI for my repository. It is also kind of backwards that continuous integration asks more then write access. What is more weird is that I can't give it admin access to only one of repositories I have access to. What's even more weird is that it can't integrate with write only access to my repository, but it can integrate with organizations what I am only member of and not admin. This is insane. https://help.github.com/articles/about-third-party-application-restrictions/
Have I misunderstood something? |
Wow, that is insane. Travis doesn't let me set up this repo, either. @kzaher: here's the list of permissions that Travis requested. It applies to all my orgs, too, which is silly. Why is it up to the org to opt out? Why can't I, as the user granting access, determine which orgs an app has access to? Dunno if that situation is acceptable, either. Quite the pickle. Totally understand not wanting to transfer this to an org just for CI – we can keep this issue tucked away and reevaluate when it makes sense 😄 |
@kzaher Is it the same between the Private Travis and the Open Source (public only) Travis? |
It's pretty bananas, but I'm not on that many organizations with close source repos. Just @artsy, really, but it kind of freaks me out that any Artsy org member could approve a malicious app by accident, then anyone could have access to all our private repos – all of Artsy's code. Some org members aren't developers and might not understand the significance of the approval. The more I think about this, the more I agree with you @kzaher. |
Well, eventually I can write a script pulling from repo on github on a personal virtual machine and run the script using the ownership of a fork. This would workaround the ownership and Kruno can avoid to transfer the repo to an association just for CI. He's still the only one really writing and investing time on coding the core of the project, so CI wouldn't give any real advantage right now. |
On core I mean operators, schedulers, subjects, etc... Most of the external commits are about RxCocoa or documentation/useful resources, so things that are not directly related to the necessity of having a CI on place for RxSwift :) |
I've thought about this, and I think I know what to do. I need to create another github account for private client repositories and leave their organizations with this public account. In that way CI tools will only have access to my public work. It's a stupid solution, but the only one that is "low risk" and secure. It will take some time to do this since I'm in the middle of some projects :(, but I'll start the process probably next week. |
I'm trying to add CircleCI Warning: Sorry! Your build didn't run because we are still adding capacity to the iOS beta and haven't enabled your organization yet. We know you're waiting and we'll get you building as soon as we can. If you have any questions, please free to contact us in support. :( Does anybody know somebody there to help us? :) |
Hm, looks like Travis has a chance of working, I'll see if I can maybe set up Travis. |
I can finally say, considered, done :) 🎉 |
👏 |
We should consider adding continuous integration from Travis or CircleCI. This would run tests on pull requests to make sure everything is 🍏 before we merge 🎉
Happy to help out with this.
The text was updated successfully, but these errors were encountered: