things to automate in the code review

We encourage users to post events happening in the community to the community events group on https://www.drupal.org.
You are viewing a wiki page. You are welcome to join the group and then edit it. Be bold!

Please update this wiki for things that are being worked on and any associated issues.

  • Coder review of sandbox projects
  • A tool to do status changes on issues. We need it to do "in needs work status for 4 weeks and no response, move to something else like won't fix and post a really nice message" but this seems like a nice tool to have for project_issue.module in general
  • General coder cleanup/enhancements

Comments

dreditor VBO

klausi's picture

Regarding status changes: dreditor has a semi-secret vbo branch that allows you to execute mass operations on drupal.org issues: http://drupal.org/project/dreditor So it should not be a problem to close old issues manually every few months.

Sure, but even better would

greggles's picture

Sure, but even better would be if it happened automatically by a robot instead of a human.

It saves time, makes sure it happens consistently, and I think it's easier to hear the message if it's coming from a robot.

robot prototype

attiks's picture

I was getting bored doing this by hand, so I created http://projectapp.h001.attiks.com/ to automate it a bit.

Do you have the source online

klausi's picture

Do you have the source online somewhere? I'm using a simple bash script to automate the same things, currently not in a mature state ... which I will publish soon.

This functionality is being

ELC's picture

This functionality is being built into drupal.org .. perhaps you should join in the coding effort there?

On a side note, these FSCKING captchas are CRAP! I've entered it correct 5 times already.

@ELC I just found out this

attiks's picture

@ELC I just found out this afternoon, missed that one, do you have a link to the issue?

http://drupal.org/node/1299710

There isn't really a single

ELC's picture

There isn't really a single issue there are quite a few things that need to change. The issues all seem to be listed in that post. The rest of the related material is just on the code-review group page - mostly the titles with "Project Application" in them.

The modules being modified are project and coder. Apparently there's talk of creating a new module specifically to deal with project reviews, but I haven't been able to find reference to it yet.

And here is my Bash script:

klausi's picture

And here is my Bash script: http://drupal.org/sandbox/klausi/1320008

Have a look at

attiks's picture

Have a look at http://drupal.org/sandbox/attiks/1319432, be advised code can be messy ;p

Code assumes presence of drush make and content type isn't added yet, I'll add it later.

did you forget to push?

klausi's picture

did you forget to push?

almost, i forgot to commit

attiks's picture

almost, i forgot to commit first ;p

new version based on features added

Script is executed using

attiks's picture

Script is executed using drush patest-all

Code review for security advisory coverage applications

Group organizers

Group notifications

This group offers an RSS feed. Or subscribe to these personalized, sitewide feeds:

Hot content this week