Page MenuHomePhabricator
Feed Advanced Search

Jun 14 2016

malandrew added a comment to D16085: ArcanistTextLinter: Option to support UTF8.

@avivey Maniphest ticket filed: T11150

Jun 14 2016, 11:56 PM
malandrew created T11150: ArcanistTextLinter: Support UTF8.
Jun 14 2016, 11:53 PM · Restricted Project, Lint, Feature Request
malandrew updated D16085: ArcanistTextLinter: Option to support UTF8.
Jun 14 2016, 11:42 PM
malandrew added a comment to D16085: ArcanistTextLinter: Option to support UTF8.

Thanks for that second document. I didn't find it earlier.

Jun 14 2016, 11:34 PM
malandrew added a comment to D16085: ArcanistTextLinter: Option to support UTF8.

Sorry I read that differently. I though that was only addressing when you might build a feature, not when you might review a feature. In my mind (having maintained projects myself), asking a maintainer to build something new is very different from asking for review of something someone has already implemented. The former is the request from someone who takes and doesn't give back and the latter is someone who is (usually) working towards the same goal of making something better and can be a valuable productive member of the community if their interest in the project is nurtured.

Jun 14 2016, 11:33 PM
malandrew added a comment to D16085: ArcanistTextLinter: Option to support UTF8.

@chad Forgive me if I misunderstood, but is that document suggesting that I need to pay to get a diff reviewed and accepted?

Jun 14 2016, 11:25 PM
malandrew added a comment to D16085: ArcanistTextLinter: Option to support UTF8.

*bump*

Jun 14 2016, 5:47 PM

Jun 9 2016

malandrew added a comment to D16085: ArcanistTextLinter: Option to support UTF8.

@chad resolved lint warnings.

Jun 9 2016, 12:05 AM
malandrew updated the diff for D16085: ArcanistTextLinter: Option to support UTF8.
  • Fix lint warnings
Jun 9 2016, 12:04 AM

Jun 8 2016

malandrew added a comment to D16085: ArcanistTextLinter: Option to support UTF8.

@chad So I should ignore the convention currently in the file for all new additions?

Jun 8 2016, 11:45 PM
malandrew added a comment to D16085: ArcanistTextLinter: Option to support UTF8.

Can I add ¯\_(ツ)_/¯ to the project README and update .arclint to support UTF8?

Jun 8 2016, 11:43 PM
malandrew added reviewers for D16085: ArcanistTextLinter: Option to support UTF8: epriestley, avivey.
Jun 8 2016, 11:30 PM