Page MenuHomePhabricator

jparise (Jon Parise)
User

Projects

Today

  • Clear sailing ahead.

Tomorrow

  • Clear sailing ahead.

Friday

  • Clear sailing ahead.

User Details

User Since
Aug 8 2011, 7:59 PM (383 w, 1 d)
Availability
Available

Recent Activity

Oct 10 2018

jparise added a comment to T8971: Auto-fix patch is wrong when multiple issue affect the same line.

One additional perspective on this:

Oct 10 2018, 7:03 PM · Restricted Project, Arcanist

Mar 23 2018

jparise edited the content of Community Resources.
Mar 23 2018, 2:56 PM

Nov 28 2017

jparise edited the content of Community Resources.
Nov 28 2017, 3:46 PM

Nov 6 2017

petr.biza awarded T9720: Provide a way to link to Diffusion files a Like token.
Nov 6 2017, 10:46 AM · Restricted Project, Remarkup

Mar 24 2017

jparise committed rARCd1db9a72b552: Improve the flake8 line-matching regex (authored by jparise).
Improve the flake8 line-matching regex
Mar 24 2017, 4:19 PM
jparise closed D17552: Improve the flake8 line-matching regex by committing rARCd1db9a72b552: Improve the flake8 line-matching regex.
Mar 24 2017, 4:19 PM
epriestley awarded D17552: Improve the flake8 line-matching regex a Cup of Joe token.
Mar 24 2017, 4:17 PM
jparise updated the diff for D17552: Improve the flake8 line-matching regex.

Restore the flake8 error code suffx

Mar 24 2017, 4:16 PM
jparise added a comment to D17552: Improve the flake8 line-matching regex.

Ideally, the "name" would be a short human-readable title for the error that's as specific as possible ("Undefined Variable") while the "description" provides more details ("The variable $x is not defined in this context. Define variables prior to use.") but they're both human-readable text so any reasonable behavior should be fine.

Mar 24 2017, 4:14 PM
jparise created D17552: Improve the flake8 line-matching regex.
Mar 24 2017, 3:36 PM

Jul 28 2016

20after4 awarded T9720: Provide a way to link to Diffusion files a Like token.
Jul 28 2016, 1:14 AM · Restricted Project, Remarkup

Jun 15 2016

jparise edited the content of Community Resources.
Jun 15 2016, 3:19 PM

May 11 2016

jparise added a comment to T9030: Need specify the minimal version of each linter.

This is also a duplicate of T4954, which was fixed by the addition of per-linter version support in D14298.

May 11 2016, 4:12 PM · Lint

May 7 2016

jparise edited the content of Community Resources.
May 7 2016, 7:43 PM
jparise edited the content of Community Resources.
May 7 2016, 7:38 PM

Apr 29 2016

jparise added a comment to T10622: Auto add reviewers based on custom logic.

For reference, the Chromium project has a formal definition of their OWNERS files, including parsing code (in Python).

Apr 29 2016, 3:10 PM · Restricted Project, Arcanist, Feature Request

Apr 27 2016

jparise committed rARC9b07d1c480f4: Recognize error codes from Flake8 extensions (authored by jparise).
Recognize error codes from Flake8 extensions
Apr 27 2016, 10:11 PM
jparise closed D15810: Recognize error codes from Flake8 extensions by committing rARC9b07d1c480f4: Recognize error codes from Flake8 extensions.
Apr 27 2016, 10:11 PM
jparise updated the diff for D15810: Recognize error codes from Flake8 extensions.

Simplify regular expression. No need to capture the code character.

Apr 27 2016, 9:06 PM
jparise updated D15810: Recognize error codes from Flake8 extensions.
Apr 27 2016, 9:05 PM
jparise retitled D15810: Recognize error codes from Flake8 extensions from to Recognize error codes from Flake8 extensions.
Apr 27 2016, 9:00 PM

Apr 26 2016

jparise awarded T5055: Distribution mechanism for arc extensions a Mountain of Wealth token.
Apr 26 2016, 9:50 PM · Arcanist
Herald updated subscribers of T6060: Arcanist should skip linters that have missing dependencies.
Apr 26 2016, 9:20 PM · Restricted Project, Arcanist

Apr 20 2016

jparise added a comment to D15472: Make callsigns optional in arcanist.

Despite the status, owners.search is probably more stable than owners.query at this point. Is there any technical reason you can't switch to owners.search?

Apr 20 2016, 3:51 PM
jparise added inline comments to D15472: Make callsigns optional in arcanist.
Apr 20 2016, 3:34 PM

Mar 7 2016

jparise edited the content of Community Resources.
Mar 7 2016, 8:29 PM

Nov 5 2015

jparise created T9720: Provide a way to link to Diffusion files.
Nov 5 2015, 5:03 PM · Restricted Project, Remarkup

Oct 22 2015

joshuaspence awarded D14298: External linters can now specify a version requirement. a Like token.
Oct 22 2015, 11:20 AM · Lint

Oct 21 2015

jparise committed rARCb3ea439f4d86: External linters can now specify a version requirement. (authored by jparise).
External linters can now specify a version requirement.
Oct 21 2015, 4:46 PM
jparise closed T4954: Allow version dependencies to be specified for external linters as Resolved by committing rARCb3ea439f4d86: External linters can now specify a version requirement..
Oct 21 2015, 4:46 PM · Lint, Arcanist
jparise closed D14298: External linters can now specify a version requirement. by committing rARCb3ea439f4d86: External linters can now specify a version requirement..
Oct 21 2015, 4:46 PM · Lint
jparise updated the diff for D14298: External linters can now specify a version requirement..

Fix silly errors.

Oct 21 2015, 4:29 PM · Lint
jparise added a comment to D14298: External linters can now specify a version requirement..

@epriestley, I've acted on those two suggested action items.

Oct 21 2015, 4:21 PM · Lint
jparise updated D14298: External linters can now specify a version requirement..
Oct 21 2015, 4:20 PM · Lint
jparise updated the diff for D14298: External linters can now specify a version requirement..

Prefer '=' to '==' for user-facing equality.

Oct 21 2015, 4:19 PM · Lint

Oct 19 2015

jparise updated the diff for D14298: External linters can now specify a version requirement..

Introduce getUpgradeInstructions() for upgrade-specific instruction text.

Oct 19 2015, 1:48 AM · Lint

Oct 18 2015

jparise added a comment to D14298: External linters can now specify a version requirement..

I had considered a getUpgradeInstructions() method and got stalled on the idea that some cases might call for a "downgrade". I suppose that's really not a good reason to forego the actual functionality, though. :)

Oct 18 2015, 5:48 PM · Lint
jparise updated the diff for D14298: External linters can now specify a version requirement..

Fix the version requirement regular expression.

Oct 18 2015, 4:35 AM · Lint
jparise added a project to D14298: External linters can now specify a version requirement.: Lint.
Oct 18 2015, 4:31 AM · Lint
jparise added a comment to D14298: External linters can now specify a version requirement..

I just signed the CLA. Sorry, I thought I had already done that a while back.

Oct 18 2015, 4:30 AM · Lint
jparise retitled D14298: External linters can now specify a version requirement. from to External linters can now specify a version requirement..
Oct 18 2015, 4:27 AM · Lint
jparise added a revision to T4954: Allow version dependencies to be specified for external linters: D14298: External linters can now specify a version requirement..
Oct 18 2015, 4:27 AM · Lint, Arcanist

Aug 13 2015

jparise created T9160: Hiding inline comments confuses keyboard navigation shortcuts.
Aug 13 2015, 3:17 AM · Differential

Jun 23 2015

jparise added a comment to T5791: Write Herald rules for outbound mail.

A partial solution is to (ab)use the List-ID header, which Gmail does understand. GitHub took this approach.

Jun 23 2015, 8:21 PM · Restricted Project, Restricted Project, FreeBSD, Mail

Jun 8 2015

jparise added a comment to T8069: Exception: Argument 1 passed to DifferentialChangesetHTMLRenderer::renderImageStage().

I see this error about once a week. Repeatedly clicking the "Close" button in the error pop up will eventually dismiss it. I don't know what the number of required clicks correlates to, but it's consistent per problematic diff.

Jun 8 2015, 9:46 PM · Differential

May 7 2015

jparise added a comment to T7447: Bring inline comments forward across revision updates.

I've been enjoying the combination of the "Done" flag and inherited comments, but one thing that I've found bothersome is that I need to click "Done" for each comment in a "thread" to "close out" that line of conversation (once I've actually done the requested thing).

May 7 2015, 6:33 PM · Prioritized, Differential

May 5 2015

jparise added a comment to T7674: Remove support for arc commit hooks.

This appears to break cassowary which uses getCommitHookMode().

May 5 2015, 9:11 PM · Lint, Arcanist

Apr 30 2015

jparise added a comment to T7447: Bring inline comments forward across revision updates.

Perhaps a reviewer could flag a comment as "Changes Requested" or similar to indicate whether the comment should be inherited by future revisions (until it is marked "Done" by the owner).

Apr 30 2015, 6:21 PM · Prioritized, Differential