Page MenuHomePhabricator
Feed Advanced Search

Feb 3 2015

joshuaspence placed T6154: XHPAST fails to recognise some syntax errors up for grabs.
Feb 3 2015, 7:37 PM · XHPAST
joshuaspence lowered the priority of T6061: `arc linters` should do basic environment checking from High to Low.
Feb 3 2015, 7:35 PM · Lint, Arcanist
joshuaspence triaged T6857: Moved file is showing incorrectly in Differential as Normal priority.
Feb 3 2015, 7:33 PM · Differential
joshuaspence added a project to T7071: PHPCS is broken: Lint.
Feb 3 2015, 7:32 PM · Lint, Arcanist
joshuaspence added a comment to T7071: PHPCS is broken.

Ubuntu 14.10 with PHP 5.5

Feb 3 2015, 7:31 PM · Lint, Arcanist
joshuaspence triaged T7097: Following documentation on Aphlict+nginx results in empty client-port for Aphlict as Normal priority.
Feb 3 2015, 7:30 PM · FreeBSD, Restricted Project, Documentation, Aphlict

Feb 2 2015

joshuaspence edited P1713 D11528.
Feb 2 2015, 8:34 PM
joshuaspence created T7101: Allow creating links to books.
Feb 2 2015, 2:29 AM · Diviner

Feb 1 2015

joshuaspence claimed T7097: Following documentation on Aphlict+nginx results in empty client-port for Aphlict.

I have a few ideas on how to improve this.

Feb 1 2015, 8:06 AM · FreeBSD, Restricted Project, Documentation, Aphlict
joshuaspence added a comment to T7097: Following documentation on Aphlict+nginx results in empty client-port for Aphlict.

I think I fixed this in D11423, particularly:

Feb 1 2015, 1:05 AM · FreeBSD, Restricted Project, Documentation, Aphlict

Jan 31 2015

joshuaspence added a comment to T7091: Grep Linter.

I think that this is already achievable with the ScriptAndRegexLinter. In addition, ArcanistXHPASTLinter also has this functionality (for PHP only). I feel that language specific linters would be better than just simply grep.

Jan 31 2015, 12:53 AM · Lint, Arcanist

Jan 30 2015

joshuaspence added a project to T7091: Grep Linter: Lint.
Jan 30 2015, 9:04 PM · Lint, Arcanist

Jan 29 2015

joshuaspence edited P1709 (An Untitled Masterwork).
Jan 29 2015, 9:47 PM
joshuaspence closed T7078: Add a `./bin/storage interactive` command as Resolved by committing Restricted Diffusion Commit.
Jan 29 2015, 8:15 PM · Database
joshuaspence placed T5097: `arc lint --everything` uses a large amount of memory in when executed in large repositories up for grabs.
Jan 29 2015, 9:15 AM · Arcanist
joshuaspence added a comment to T7071: PHPCS is broken.

I can't reproduce that unfortunately:

Jan 29 2015, 9:07 AM · Lint, Arcanist
joshuaspence added a comment to T7071: PHPCS is broken.

I am not able to replicate this. What version of phpcs are you using?

Jan 29 2015, 7:24 AM · Lint, Arcanist
joshuaspence added a revision to T7078: Add a `./bin/storage interactive` command: Unknown Object (Differential Revision).
Jan 29 2015, 7:21 AM · Database
joshuaspence awarded T7074: `arc patch Dx` shouldn't ask for authentication if Dx is public a Doubloon token.
Jan 29 2015, 7:02 AM · Arcanist
joshuaspence awarded T7076: Show status of linked revisions for tasks a Evil Spooky Haunted Tree token.
Jan 29 2015, 7:01 AM · Restricted Project, Differential, Maniphest

Jan 28 2015

joshuaspence created T7078: Add a `./bin/storage interactive` command.
Jan 28 2015, 11:36 PM · Database
joshuaspence added a comment to T7076: Show status of linked revisions for tasks.

I approve.

Jan 28 2015, 9:28 PM · Restricted Project, Differential, Maniphest
joshuaspence added a comment to T7077: Add a linter rule for arrays not being allowed as class constants.

Actually, this is perfectly fine in PHP 5.6 I believe.

Jan 28 2015, 8:49 PM · Lint
joshuaspence created T7077: Add a linter rule for arrays not being allowed as class constants.
Jan 28 2015, 8:48 PM · Lint
joshuaspence added a comment to T7067: Fatal error running XHPast (arc lint).

Update libphutil, it seems that you are not using the latest version.

Jan 28 2015, 7:45 PM · XHPAST
joshuaspence claimed T7071: PHPCS is broken.
Jan 28 2015, 7:41 PM · Lint, Arcanist
joshuaspence added a revision to T784: Allow Differential changesets to be marked with various attributes: Unknown Object (Differential Revision).
Jan 28 2015, 9:05 AM · Restricted Project, Restricted Project, Restricted Project, Differential, Arcanist
joshuaspence added a revision to T784: Allow Differential changesets to be marked with various attributes: Unknown Object (Differential Revision).
Jan 28 2015, 9:04 AM · Restricted Project, Restricted Project, Restricted Project, Differential, Arcanist
joshuaspence added a revision to T784: Allow Differential changesets to be marked with various attributes: Restricted Differential Revision.
Jan 28 2015, 8:53 AM · Restricted Project, Restricted Project, Restricted Project, Differential, Arcanist
joshuaspence added a revision to T784: Allow Differential changesets to be marked with various attributes: Unknown Object (Differential Revision).
Jan 28 2015, 8:42 AM · Restricted Project, Restricted Project, Restricted Project, Differential, Arcanist
joshuaspence added a comment to T7053: Restart daemons automagically.

I'm sure this question has been asked previously, but why can't the daemons read the updated configuration themselves? I imagine that the configuration would be cached, but surely it could be purged periodically?

Jan 28 2015, 7:48 AM · Daemons
joshuaspence claimed T784: Allow Differential changesets to be marked with various attributes.

I have half a diff for this.

Jan 28 2015, 7:42 AM · Restricted Project, Restricted Project, Restricted Project, Differential, Arcanist

Jan 27 2015

joshuaspence created T7053: Restart daemons automagically.
Jan 27 2015, 11:46 AM · Daemons
joshuaspence created T7052: Unable to scroll after clicking in left sidebar.
Jan 27 2015, 10:30 AM · PHUI
joshuaspence updated subscribers of T7029: Tokenizers lose state changes after navigation actions.
Jan 27 2015, 7:04 AM · Mobile, Maniphest
joshuaspence merged task T7051: Moving back to a task does not re-create correct Status/CC/Project field despite remembered 'Action' value into T7029: Tokenizers lose state changes after navigation actions.
Jan 27 2015, 7:04 AM · Maniphest

Jan 26 2015

epriestley awarded T7046: Add a linter rule for detecting missing variables for `pht` a Mountain of Wealth token.
Jan 26 2015, 8:23 PM · Lint
joshuaspence placed T5616: Add tuple support to `PhutilTypeSpec` up for grabs.

I haven't been able to make any progress on this.

Jan 26 2015, 8:16 PM · libphutil
joshuaspence triaged T7046: Add a linter rule for detecting missing variables for `pht` as Low priority.
Jan 26 2015, 7:57 PM · Lint
joshuaspence claimed T7046: Add a linter rule for detecting missing variables for `pht`.
Jan 26 2015, 7:57 PM · Lint
joshuaspence created T7046: Add a linter rule for detecting missing variables for `pht`.
Jan 26 2015, 7:56 PM · Lint
joshuaspence created T7045: Allow linters to be skipped.
Jan 26 2015, 7:50 PM · Differential, Lint
joshuaspence updated the task description for T7037: Allow `.arclint` files to be templated.
Jan 26 2015, 5:49 AM · Lint
joshuaspence updated the task description for T7037: Allow `.arclint` files to be templated.
Jan 26 2015, 5:44 AM · Lint
joshuaspence updated the task description for T7037: Allow `.arclint` files to be templated.
Jan 26 2015, 5:41 AM · Lint
joshuaspence updated the task description for T7037: Allow `.arclint` files to be templated.
Jan 26 2015, 5:39 AM · Lint
joshuaspence created T7037: Allow `.arclint` files to be templated.
Jan 26 2015, 5:35 AM · Lint

Jan 25 2015

joshuaspence updated the task description for T7032: Scrolling of revisions is broken on Chrome 41 all.
Jan 25 2015, 7:53 AM · Google Chrome, Differential, Mobile
joshuaspence created T7032: Scrolling of revisions is broken on Chrome 41 all.
Jan 25 2015, 5:55 AM · Google Chrome, Differential, Mobile
joshuaspence added a comment to T7029: Tokenizers lose state changes after navigation actions.

Serializing into a hidden field sounds like the correct approach to me.

Jan 25 2015, 12:11 AM · Mobile, Maniphest
joshuaspence added a comment to T7029: Tokenizers lose state changes after navigation actions.

You can reproduce this in a normal web browser (at least, in Safari):

  • Click "Edit Task" on any task.
  • Add a project to the "Projects" field.
  • Type some text in the "Description" field.
  • Click a link on the page, like the Phabricator logo.
  • Press the browser back button.

The description change persists, but the field change does not.

This is probably fixable, but also probably involves some stupid garbage nonsense where we serialize tokens into a hidden field value.

Jan 25 2015, 12:01 AM · Mobile, Maniphest

Jan 24 2015

joshuaspence added a comment to T7029: Tokenizers lose state changes after navigation actions.

Actually, it seems that the subscribers field doesn't persist either (I originally thought that it did, but it seems that only the default subscriber of myself persists). So yeah, this may not be fixable.

Jan 24 2015, 11:59 PM · Mobile, Maniphest
joshuaspence added a comment to T7029: Tokenizers lose state changes after navigation actions.

Seeing as the other fields persist, I suspect that it should be fixable.

Jan 24 2015, 11:55 PM · Mobile, Maniphest
joshuaspence created T7029: Tokenizers lose state changes after navigation actions.
Jan 24 2015, 11:52 PM · Mobile, Maniphest
joshuaspence created T7028: Consider making HTML email the default.
Jan 24 2015, 11:45 PM · Mail
joshuaspence created T7027: "Leaving high security" page looks a bit off on mobile.
Jan 24 2015, 11:39 PM · PHUI

Jan 23 2015

joshuaspence created T7024: Document cluster configuration.
Jan 23 2015, 9:59 PM · Restricted Project, Infrastructure, Documentation
joshuaspence claimed T5163: User hitting undefined index in changeset rendering.
Jan 23 2015, 10:00 AM · Differential
joshuaspence updated the task description for T6378: differential.generated-paths doesn't collapse files.
Jan 23 2015, 8:50 AM · Differential

Jan 22 2015

joshuaspence added a comment to T6915: Allow the Aphlict server to scale across multiple hosts.

As an aside, I still think it would be nice to send the notification contents rather than just the key (to avoid the extra request to /notification/individual/). Possibly this would be easier with WebSockets than it would have been with Flash.

Jan 22 2015, 10:22 PM · Restricted Project, Infrastructure, Aphlict
joshuaspence added a comment to T6915: Allow the Aphlict server to scale across multiple hosts.

Does that solve the scaling issue though? Essentially, all Aphlict servers need to be aware of (I.e. store in memory) all clients and all subscriptions. Realistically, I wouldn't expect this to cause any issues for any installs today, but possibly as Aphlict receives more usage it might?

Jan 22 2015, 10:20 PM · Restricted Project, Infrastructure, Aphlict
joshuaspence closed T6890: Diviner stops reading return type after <whitespace> as Resolved by committing Restricted Diffusion Commit.
Jan 22 2015, 8:18 PM · Diviner
joshuaspence closed T6950: conduit.query should provide return values as Resolved by committing Restricted Diffusion Commit.
Jan 22 2015, 8:17 PM · Conduit
joshuaspence closed T6858: "Depends on Dx" generates a superfluous edge as Resolved by committing Restricted Diffusion Commit.
Jan 22 2015, 8:12 PM · Differential, Transactions
joshuaspence added a revision to T5655: Standardize naming conventions: Unknown Object (Differential Revision).
Jan 22 2015, 8:09 PM · libphutil, Arcanist
joshuaspence added a revision to T6890: Diviner stops reading return type after <whitespace>: Unknown Object (Differential Revision).
Jan 22 2015, 8:04 PM · Diviner
joshuaspence added a revision to T5163: User hitting undefined index in changeset rendering: Unknown Object (Differential Revision).
Jan 22 2015, 7:55 PM · Differential
joshuaspence added a project to T4209: Multiserver / High-Availability Configuration: Phacility.
Jan 22 2015, 7:40 PM · Restricted Project, Phacility
joshuaspence added a comment to T5163: User hitting undefined index in changeset rendering.

I was able to reproduce this behavior with the following diff:

Jan 22 2015, 12:21 PM · Differential
joshuaspence added a comment to T6378: differential.generated-paths doesn't collapse files.

I was able to get this to work (i.e. the files were collapsed), but I had to purge the cache first.

Jan 22 2015, 11:57 AM · Differential
joshuaspence added a comment to T6482: arc patch does not pull in dependencies.

What exactly does arc patch --trace D489 output?

Jan 22 2015, 11:32 AM · Restricted Project, Arcanist
joshuaspence added a revision to T6950: conduit.query should provide return values: Unknown Object (Differential Revision).
Jan 22 2015, 11:20 AM · Conduit
joshuaspence claimed T6950: conduit.query should provide return values.
Jan 22 2015, 11:19 AM · Conduit
joshuaspence placed T6884: Logging when a task has been marked as a blocker up for grabs.
Jan 22 2015, 11:12 AM · Transactions, Maniphest
joshuaspence added a revision to T6857: Moved file is showing incorrectly in Differential: Unknown Object (Differential Revision).
Jan 22 2015, 9:40 AM · Differential
joshuaspence added a comment to T6857: Moved file is showing incorrectly in Differential.

Before

after.png (744×1 px, 101 KB)

Jan 22 2015, 7:56 AM · Differential
joshuaspence added a comment to T6857: Moved file is showing incorrectly in Differential.

It seems like this was broken in D10865.

Jan 22 2015, 7:50 AM · Differential
joshuaspence claimed T6857: Moved file is showing incorrectly in Differential.
Jan 22 2015, 6:59 AM · Differential
joshuaspence claimed T6884: Logging when a task has been marked as a blocker.
Jan 22 2015, 6:58 AM · Transactions, Maniphest

Jan 21 2015

joshuaspence created T7004: Don't send email whilst online.
Jan 21 2015, 8:36 PM · Notifications, Mail
joshuaspence closed T6998: After removing `process.exit(1)`, Node exits with error code 0 after throwing as Resolved by committing Restricted Diffusion Commit.
Jan 21 2015, 7:47 PM · Aphlict
joshuaspence added a revision to T6858: "Depends on Dx" generates a superfluous edge: Unknown Object (Differential Revision).
Jan 21 2015, 12:14 PM · Differential, Transactions
joshuaspence claimed T6858: "Depends on Dx" generates a superfluous edge.
Jan 21 2015, 12:11 PM · Differential, Transactions
joshuaspence claimed T6888: Extending revert edges.
Jan 21 2015, 11:53 AM · Maniphest, Differential
joshuaspence closed T6822: Fix methods which override other methods with the wrong visibility as Resolved.
Jan 21 2015, 11:52 AM · Infrastructure
joshuaspence added a revision to T5105: Ensure that `ArcanistTextLinter` respects `.editorconfig` settings: Unknown Object (Differential Revision).
Jan 21 2015, 7:56 AM · Arcanist
joshuaspence added a revision to T6998: After removing `process.exit(1)`, Node exits with error code 0 after throwing: Unknown Object (Differential Revision).
Jan 21 2015, 7:34 AM · Aphlict
joshuaspence claimed T6998: After removing `process.exit(1)`, Node exits with error code 0 after throwing.
Jan 21 2015, 7:00 AM · Aphlict
joshuaspence added a comment to T6998: After removing `process.exit(1)`, Node exits with error code 0 after throwing.

Interesting. So ./bin/aphlict debug exits with 1 but running node './support/aphlict/server/aphlict_server.js' '--client-port=22280' '--admin-port=22281' '--admin-host=localhost' '--log=/var/log/aphlict.log' exits with 0.

Jan 21 2015, 1:27 AM · Aphlict

Jan 20 2015

joshuaspence added a revision to T5105: Ensure that `ArcanistTextLinter` respects `.editorconfig` settings: Unknown Object (Differential Revision).
Jan 20 2015, 11:15 AM · Arcanist
joshuaspence added a revision to T5105: Ensure that `ArcanistTextLinter` respects `.editorconfig` settings: Unknown Object (Differential Revision).
Jan 20 2015, 10:45 AM · Arcanist

Jan 19 2015

joshuaspence reassigned T6987: Aphlict server does not respond to http requests on configured ports. from joshuaspence to epriestley.

Didn't realize that @epriestley stole this one.

Jan 19 2015, 7:29 PM · Aphlict
joshuaspence claimed T6987: Aphlict server does not respond to http requests on configured ports..
Jan 19 2015, 7:27 PM · Aphlict

Jan 18 2015

joshuaspence added a project to T6983: Best practices for repository creation: Documentation.
Jan 18 2015, 7:36 PM · Documentation, Diffusion

Jan 15 2015

joshuaspence added a revision to T6822: Fix methods which override other methods with the wrong visibility: Unknown Object (Differential Revision).
Jan 15 2015, 8:40 PM · Infrastructure
joshuaspence added a revision to T6822: Fix methods which override other methods with the wrong visibility: Unknown Object (Differential Revision).
Jan 15 2015, 8:33 PM · Infrastructure
joshuaspence added a revision to T6822: Fix methods which override other methods with the wrong visibility: Unknown Object (Differential Revision).
Jan 15 2015, 8:14 PM · Infrastructure
joshuaspence closed T5344: Multiplex AJAX calls from Aphlict clients through the JX.Leader as Resolved by committing Restricted Diffusion Commit.
Jan 15 2015, 8:12 PM · Aphlict
joshuaspence closed T5344: Multiplex AJAX calls from Aphlict clients through the JX.Leader, a subtask of T5364: Conpherence v2 (Original Discussion), as Resolved.
Jan 15 2015, 8:12 PM · Conpherence
joshuaspence claimed T6822: Fix methods which override other methods with the wrong visibility.

I think this is done now (with the exception of the super-secret Phacility classes)

Jan 15 2015, 12:43 PM · Infrastructure