reddit-india / indianelectionmeter.github.io

Website dedicated to track promises made in Indian Elections, as suggested on reddit.com/r/india
http://indianelectionmeter.in
MIT License
40 stars 19 forks source link

Implementing CI #61

Open okpatil4u opened 8 years ago

okpatil4u commented 8 years ago

Using travis-ci or circle-ci for bug free builds.

jimaldon commented 8 years ago

I have experience with jenkins, but not travis and circle.

okpatil4u commented 8 years ago

Anything could work. How do you set it up ? Can you write few basic tests ? I

ndarville commented 8 years ago

I’ve got my own repo with custom rules and linters: https://github.com/ndarville/style. You can see an example of it in action at another repo.

You’ll have to decide on the linter rules, if you don’t want to stick with the defaults.

thel3l commented 8 years ago

This is easy enough. First go to travis-ci.org and link your github account ( the one which owns this repository). Then enable the integration - screenshot attached. After this, anyone with push access can push the .Travis.yml file required for Travis to run - I'm willing to write a .Travis.yml file, but I'll need you to get set up the repo first. screenshot_2015-11-22-07-11-24

ndarville commented 8 years ago

Also check out LintHub. Really easy to enable, and it lints your pull requests.

okpatil4u commented 8 years ago

@ndarville @thel3l I am not looking for information. I am looking someone to take over this issue. Can any of you take it on ?

thel3l commented 8 years ago

Only the owner of this repo can set it up. None of us can.

jimaldon commented 8 years ago

I can take it up after you set it up.

okpatil4u commented 8 years ago

Thanks @jimdsouza. Should it be travis-ci or circle-ci ?

jimaldon commented 8 years ago

Whichever looks easily integrable. Since I'll have to do a lot of digging either way, I suggest you pick one with good documentation and a big community.

On 24 Nov 2015 10:47, at 10:47, Omkar Patil notifications@github.com wrote:

Thanks @jimdsouza. Should it be travis-ci or circle-ci ?


Reply to this email directly or view it on GitHub: https://github.com/reddit-india/indianelectionmeter.github.io/issues/61#issuecomment-159135892

thel3l commented 8 years ago

I'd choose Travis.

jimaldon commented 8 years ago

Okay. Since you started this repo, @okpatil4u, could you go here (https://docs.travis-ci.com/user/getting-started/)  and complete steps 1 and 2? And then add me as a contributor to this repo?

On 25 Nov 2015 08:48, at 08:48, the lel notifications@github.com wrote:

I'd choose Travis.


Reply to this email directly or view it on GitHub: https://github.com/reddit-india/indianelectionmeter.github.io/issues/61#issuecomment-159450138

okpatil4u commented 8 years ago

Done

ndarville commented 8 years ago

What exactly is it you want to be checked?

I tried my hand at it, and it looks like Meteor.js can’t be checked directly with Travis; you’ll have to write a test suite for it instead, from what I gather. You can just run a meteor instance, because it’ll time out and end the test suite.

Some other things to test and lint would be:

The last one is the most straightforward, but again, it depends on what you want in a linter. I see that people are using very, very different coding styles in JS, and we need to agree on a standard, unless you just want me to enforce one (four-space indentation and so on).


The best linter to go by is jscs, because it supports automatic code fixes, even in editors like Sublime Text. This makes it a little easier to rewrite the code base as it currently exists.

I don’t have any JSX-specific linter rules, though, but there are still the basics.

thel3l commented 8 years ago

@ndarville I think it's possible. If @okpatil4u is done with the first two steps, then I'll try my hand at this.

okpatil4u commented 8 years ago

@thel3l, yeah, I have completed first two steps, but @jimdsouza is taking over it. It won't make sense all of us working on this one. I will try to resolve most of content generator issues by this monday. Why don't you take care of web scraper ?

thel3l commented 8 years ago

Ook. That's fine with me.

okpatil4u commented 8 years ago

@jimdsouza any progress ?

jimaldon commented 8 years ago

@okpatil4u In the thick of my exams this week. Plan to dive from Wednesday next week.

okpatil4u commented 8 years ago

Sounds good. I was also busy last week, finishing most of the tasks today.