Beefy Boxes and Bandwidth Generously Provided by pair Networks
Your skill will accomplish
what the force of many cannot
 
PerlMonks  

Re^2: On Coding Standards and Code Reviews

by eyepopslikeamosquito (Chancellor)
on Feb 19, 2009 at 11:19 UTC ( #745008=note: print w/replies, xml ) Need Help??


in reply to Re: On Coding Standards and Code Reviews
in thread On Coding Standards and Code Reviews

Thanks for the feedback. This is the sort of feedback I was hoping for, so I can be prepared before I meet with our QA guy. :-) To answer your question, we have one QA guy and I'll need to refine these notes before I meet with him. As you might expect, he's more interested in process than I am, his primary concerns being things like "how should we enforce the coding standard?". In the past, we've sorta had coding standards (actually, too many of them) and informal code reviews, but they've never been enforced consistently across the organisation, especially when deadlines are tight.

  • Comment on Re^2: On Coding Standards and Code Reviews

Replies are listed 'Best First'.
Re^3: On Coding Standards and Code Reviews
by puudeli (Pilgrim) on Feb 19, 2009 at 11:38 UTC

    One way to enforce code quality and integrity in the process is to choose a process model that can facilitate it. Agile methods are good at it. Especially I would recommend looking at Scrum and pay attention to the Definition of Done.

    A self guiding process is the best tool to get teams happy, efficient and committed. This includes code quality and standards. I would not dictate them except on a very high level (setting some thresholds). IMO the tools do not matter that much, the result is what I would be interested in. And a good process that includes tools for correcting bad ways of working is priceless.

    --
    seek $her, $from, $everywhere if exists $true{love};

Log In?
Username:
Password:

What's my password?
Create A New User
Node Status?
node history
Node Type: note [id://745008]
help
Chatterbox?
[stevieb]: did my pull request challenge today, Locale::gettext. Weird layout for a dist, and I have no idea how to use it, but it's always nice when you can force someone else's work be use strict; and use warnings; compliant :)
[stevieb]: ahem... s/no idea how to use it/didn't read enough as to what it's used for/
[stevieb]: interestingly enough, someone else got my Devel::Examine:: Subs distribution for their PRC, and I applaud the change. This dist is extremely complicated and mostly obfu, but the person doing it understood PPI enough to change...
[stevieb]: ...something I had overlooked in the extreme depths of the core functionality. After merging, then a couple of extra tweaks, I still have 100% test coverage. Yay for people who write tests!

How do I use this? | Other CB clients
Other Users?
Others pondering the Monastery: (7)
As of 2017-01-24 01:16 GMT
Sections?
Information?
Find Nodes?
Leftovers?
    Voting Booth?
    Do you watch meteor showers?




    Results (199 votes). Check out past polls.