Page MenuHomePhabricator

sectioneight (Aiden Scandella)
User

Projects

User does not belong to any projects.

Today

  • Clear sailing ahead.

Tomorrow

  • Clear sailing ahead.

Thursday

  • Clear sailing ahead.

User Details

User Since
Dec 17 2014, 12:49 AM (522 w, 6 d)
Availability
Available

Recent Activity

Jan 8 2016

sectioneight added a comment to T9456: Evaluate upstream support for third-party build systems.

Not super-familiar with PowerShell. If the issue linked above would be helpful to you, could you please add your requirements there and we can coordinate getting it in the Uber plugin? (I'm the maintainer)

Jan 8 2016, 3:09 AM · Restricted Project, Harbormaster
sectioneight added a comment to T9456: Evaluate upstream support for third-party build systems.

I have a branch up that allows for non-differential builds to submit
harbormaster from Jenkins. I dropped it before the holidays since there
didn't seem to be much interest but can resurrect it if people are
interested:

Jan 8 2016, 1:14 AM · Restricted Project, Harbormaster

Sep 8 2015

sectioneight created T9365: Harbormaster results are hard to aggregate .
Sep 8 2015, 11:02 PM · Harbormaster, Restricted Project, Restricted Project

Aug 31 2015

sectioneight added a comment to T6518: Add "Notify Jenkins on Commit" step for Harbormaster.

Hey there, I am the maintainer of the Phabricator Jenkins plugin. I am working on adding coverage (and unit) reporting from Jenkins -> Harbormaster now that the API exists. You can track our progress here: https://github.com/uber/phabricator-jenkins-plugin/issues/34 (I wish I had a public Phabricator to develop this on :))

Aug 31 2015, 5:39 PM · Harbormaster

Aug 11 2015

sectioneight added a comment to D13873: Restore default "yes" behavior for lint patch.

Sounds good.

Aug 11 2015, 10:06 PM
sectioneight added a comment to D13873: Restore default "yes" behavior for lint patch.

It looks like this is the only one with $console->confirm() and $default_no.

Aug 11 2015, 9:54 PM
sectioneight added a comment to D13873: Restore default "yes" behavior for lint patch.
NOTE: If this is the correct strategy and I'm not confused, there are other $default_no = false instances that should be fixed.
Aug 11 2015, 9:45 PM
sectioneight added a comment to D13873: Restore default "yes" behavior for lint patch.

Before:

pasted_file (192×674 px, 28 KB)

Aug 11 2015, 9:44 PM
sectioneight added a task to D13873: Restore default "yes" behavior for lint patch: T9029: $default_no is misleading / potentially wrong for PhutilConsole::confirm in various workflows.
Aug 11 2015, 9:40 PM
sectioneight added a revision to T9029: $default_no is misleading / potentially wrong for PhutilConsole::confirm in various workflows: D13873: Restore default "yes" behavior for lint patch.
Aug 11 2015, 9:40 PM · Arcanist
sectioneight added a comment to T9029: $default_no is misleading / potentially wrong for PhutilConsole::confirm in various workflows.

@joshuaspence should I just submit a diff for this? Or am I misunderstanding the code?

Aug 11 2015, 9:28 PM · Arcanist

Aug 6 2015

sectioneight awarded D13811: Unprototype Harbormaster a Pterodactyl token.
Aug 6 2015, 3:34 PM

Jul 31 2015

sectioneight added a comment to T9029: $default_no is misleading / potentially wrong for PhutilConsole::confirm in various workflows.

And if this is indeed a bug, and not just me reading things terribly wrong, I'm happy to submit a patch to flip the binary values and rename, but wanted to make sure I wasn't wasting yall's time if this is just a misunderstanding on my part.

Jul 31 2015, 2:09 AM · Arcanist
sectioneight added a comment to T9029: $default_no is misleading / potentially wrong for PhutilConsole::confirm in various workflows.

In other words, I was surprised to discover that the "Apply this patch" prompt default was "N", and it looks like before it was switched from phutil_console_confirm the default used to be "Y".

Jul 31 2015, 2:08 AM · Arcanist
sectioneight added a comment to T9029: $default_no is misleading / potentially wrong for PhutilConsole::confirm in various workflows.

For an example of a conversion, see {D3151}

Jul 31 2015, 1:59 AM · Arcanist
sectioneight created T9029: $default_no is misleading / potentially wrong for PhutilConsole::confirm in various workflows.
Jul 31 2015, 1:58 AM · Arcanist

Jun 17 2015

sectioneight awarded T8549: Feedback on 2015 Redesign a Pterodactyl token.
Jun 17 2015, 8:42 PM · Design

Jun 11 2015

sectioneight added a comment to T8513: `differential.querydiffs` returns array instead of object when lacking permissions.

thanks for the quick replies! internally to uber, arc patch has served us well. we git reset --hard to the base revision, then patch --nocommit --nobranch. it wasn't until we open sourced our jenkins plugin and got this bug report that we even noticed this behavior. i'll follow the task you mentioned and update the jenkins plugin when a more robust solution is available.

Jun 11 2015, 4:49 PM · Conduit
sectioneight updated the task description for T8513: `differential.querydiffs` returns array instead of object when lacking permissions.
Jun 11 2015, 3:37 PM · Conduit
sectioneight created T8513: `differential.querydiffs` returns array instead of object when lacking permissions.
Jun 11 2015, 3:34 PM · Conduit

Apr 30 2015

sectioneight added a comment to D12621: Add a Go unit test engine with support for Race and Godep..

nice!

Apr 30 2015, 4:35 PM
sectioneight awarded T6526: Implement "Badges" or "Medals", for annotating special users a Like token.
Apr 30 2015, 6:28 AM · Restricted Project, Badges (v0), People
sectioneight added a comment to D12537: Add a Go unit test engine (through go and godep)..

I have a version of this with coverage support if you're interested:

Apr 30 2015, 1:42 AM

Apr 29 2015

sectioneight added a comment to D12120: Add GoVet Linter.

@joshuaspence is this ready for review?

Apr 29 2015, 3:41 AM

Apr 24 2015

sectioneight awarded D12484: Port comments through time and space in the common/best case a Like token.
Apr 24 2015, 8:56 PM
sectioneight added a comment to D12120: Add GoVet Linter.

This is great, thanks for getting the tests passing!

Apr 24 2015, 4:08 PM

Mar 28 2015

sectioneight added a comment to D12120: Add GoVet Linter.

I don't know if this is useful, or how to get the tests pass, but I like the functionality of go vet and I like phabricator, so maybe the two can be friends?

Mar 28 2015, 12:04 AM

Mar 20 2015

sectioneight added a comment to D12120: Add GoVet Linter.

I also need to detect when the tool is not installed and output the proper warning. Unfortunately it's not as simple as a path check, since it's executed as go tool vet <file> and having go in the path is not enough.

Mar 20 2015, 10:33 PM
sectioneight added a task to D12120: Add GoVet Linter: T6867: Add linter and unit test engine for Go(lang).
Mar 20 2015, 10:28 PM
sectioneight added a revision to T6867: Add linter and unit test engine for Go(lang): D12120: Add GoVet Linter.
Mar 20 2015, 10:28 PM · Lint

Dec 17 2014

sectioneight added a comment to T6769: Confusing UI when harbormaster fails but differential is accepted.

eek, sorry about that!

Dec 17 2014, 1:53 AM · Harbormaster
sectioneight created T6769: Confusing UI when harbormaster fails but differential is accepted.
Dec 17 2014, 12:52 AM · Harbormaster