search

grisu / gricli

Grisu commandline client
7 stars 2 forks source link

rename hostCount to hostcount #210

Closed sina-masoud-ansari closed 12 years ago

sina-masoud-ansari commented 12 years ago

Just to bring it in line with the other globals eg jobtype, walltime

ndjones commented 12 years ago

As a key interaction element, variable syntac would be important to review with users - I'd assume that readability is important, so a common delimiter would then be the proposal. Agree completely that it should be standard across variables.

Can you test some options with a variety of users, e.g:

jobtype jobType JobType job-type job_type

I find job-type the best, as it works when used in documentation too - if it's job_type, then the underscore becomes obscured in documentation, whereas job-type provides readability and clarity independent of treatment.

On 5/01/12 12:28 PM, "Sina Masoud-Ansari" <reply+i-2729813-8df9ef5c1ccccba56c45a399db91f48c2c8714fd-583502@reply.gith ub.com> wrote:

Just to bring it in line with the other globals eg jobtype, walltime

Reply to this email directly or view it on GitHub: https://github.com/grisu/gricli/issues/210

sina-masoud-ansari commented 12 years ago

Yeah that's a good point, I'll round up these issues and meet with users over the next couple of weeks.

-- Same scheme should apply to print job x results

makkus commented 12 years ago

hostcount renamed

makkus commented 12 years ago

Done.