Page MenuHomePhabricator
Feed Advanced Search

Jan 12 2016

dserafin added a comment to T4171: Building reporting and data systems.

As cspeckmim, I'm working in a company where all code is safety related and must be certified. We'd like to have a report on the code review activity. At present we have to submit a lot of screenshots to demonstrate we performed code review using Phabricator...

Jan 12 2016, 5:02 PM · Restricted Project, FreeBSD, Facts, Wikimedia

Oct 16 2014

dserafin created T6325: Report generation. Any?.
Oct 16 2014, 6:49 AM · Facts

Oct 8 2014

dserafin added a comment to T2999: When we detect a file as binary, explain why.

The first time you commit a good (UTF-8) file it will still list as binary because the old side of the diff is binary. The second good commit will result in a good new and old version that can be text diff'ed. Also, pure ASCII is not needed, UTF-8 is fine so you should be able to use whatever characters you need for your native language.

Oct 8 2014, 1:16 PM · Diffusion, Arcanist

Oct 6 2014

dserafin added a comment to T2999: When we detect a file as binary, explain why.

Same problem with our assembly files (working directory on Windows7 disk folder): is not possible to view them in Audit because they're seen as "binary" (while they are visible as diffs in Differential probably because unix diff we use fixes encoding). Verifying their nature with cygwin 'file' tool they are seen as:

Oct 6 2014, 2:48 PM · Diffusion, Arcanist

Sep 30 2014

dserafin added a comment to T6210: Detect if pygmentize exists but fails to execute.

That would be very useful. You can also add a warning on the Pygments installation instructions to carefully check that temporary directory and, in any case, double check it's functional using the debug function page you showed me: that was really clear and shot the bug.

Sep 30 2014, 3:47 PM · Setup, Phabricator
dserafin added a comment to T6210: Detect if pygmentize exists but fails to execute.

Great!
With the debug code I (in fact indirectly YOU) solved the issue. This was my output of the ../debug page:

Sep 30 2014, 3:10 PM · Setup, Phabricator
dserafin added a comment to T6210: Detect if pygmentize exists but fails to execute.

Thank you,
I must confess that I'm using Linux since Linus' first version, but I have little sysadmin experience.

Sep 30 2014, 2:48 PM · Setup, Phabricator
dserafin created T6210: Detect if pygmentize exists but fails to execute.
Sep 30 2014, 1:08 PM · Setup, Phabricator