I renamed the properties to both start with lower case and match the name of the attributes. I think it looks more consistent like this (matching the exact parameter name), but if you want the capitalization, I can change it.

From: Brendan Forster [mailto:notifications@github.com]
Sent: 8 de dezembro de 2013 16:37
To: octokit/octokit.net
Cc: André Rodrigues
Subject: Re: [octokit.net] Implement labels CRUD (#256)


In Octokit/Models/Request/LabelUpdate.cs:

> @@ -0,0 +1,41 @@

> +﻿using System;

> +using System.Text.RegularExpressions;

> +

> +namespace Octokit

> +{

> +    public class LabelUpdate

> +    {

> +        private string _color;

> +

> +        public LabelUpdate(string name, string color)

> +        {

> +            Ensure.ArgumentNotNullOrEmptyString(name, "Label name can't be null");

> +            Ensure.ArgumentNotNullOrEmptyString(color, "color can't be null");

Ok, let's capitalize Color here (and in the other case) and I think I'm done with "things to find wrong" here [:grinning:]

—
Reply to this email directly or view it on GitHub<https://github.com/octokit/octokit.net/pull/256/files#r8185076>.