Re: Please write good commit messages before asking for code review

2017-03-09 Thread Mark Côté
On 2017-03-09 4:48 PM, Boris Zbarsky wrote: On 3/9/17 4:35 PM, Eric Rescorla wrote: I'm in favor of good commit messages, but I would note that current m-c convention really pushes against this, because people seem to feel that commit messages should be one line. They feel wrong, and we should

Re: Please write good commit messages before asking for code review

2017-03-09 Thread Mark Côté
Oops just saw this after I posted separately about this feature. Yeah, I agree it's a bit confusing. We have a few ideas for making this better differentiated; will open a bug. Mark On 2017-03-09 3:29 PM, Kyle Machulis wrote: This has actually been confusing me in reviews, since the commit

Phabricator Update, July 2017

2017-07-11 Thread Mark Côté
Hi all, here's a brief update on the project to deploy and integrate Phabricator at Mozilla: * Development Phabricator instance is up at https://mozphab.dev.mozaws.net/, authenticated via bugzilla-dev.allizom.org. * Development, read-only UI for Lando (the new automatic-landing service) has bee

Re: Phabricator Update, July 2017

2017-07-11 Thread Mark Côté
push-to-review support? Chris. On Wednesday, July 12, 2017 at 8:42:06 AM UTC+12, Mark Côté wrote: Hi all, here's a brief update on the project to deploy and integrate Phabricator at Mozilla: * Development Phabricator instance is up at https://mozphab.dev.mozaws.net/, authen

Re: Phabricator Update, July 2017

2017-07-11 Thread Mark Côté
On 2017-07-11 9:51 PM, Martin Thomson wrote: On Wed, Jul 12, 2017 at 6:41 AM, Mark Côté wrote: * MozReview and Splinter turned off in early December. Is this bugzilla-wide? I know that other project use splinter still. Will those projects be able to use phabricator for their projects

Re: Phabricator Update, July 2017

2017-07-13 Thread Mark Côté
On 2017-07-13 3:54 PM, Randell Jesup wrote: On Wed, Jul 12, 2017 at 11:27 AM, Byron Jones wrote: But indeed having also the patches in bugzilla would be good. no, it would be bad for patches to be duplicated into bugzilla. we're moving from bugzilla/mozreview to phabricator for code review

Re: Phabricator Update, July 2017

2017-07-14 Thread Mark Côté
On 2017-07-14 1:31 AM, Jim Blandy wrote: Many people seem to be asking, essentially: What will happen to old bugs? I'm trying to follow the discussion, and I'm not clear on this myself. For example, "Splinter will be turned off." For commenting and reviewing, okay, understood. What about viewing

Re: Phabricator Update, July 2017

2017-07-17 Thread Mark Côté
I filed a central tracker bug for production Phabricator deployment: https://bugzilla.mozilla.org/show_bug.cgi?id=1381498. I have filed blockers and dependencies for a variety of related tasks as discussed in these threads. Mark On 2017-07-14 11:33 AM, Milan Sreckovic wrote: Replying in ge

Re: Phabricator Update, July 2017

2017-07-17 Thread Mark Côté
On 2017-07-17 8:46 PM, Edmund Wong wrote: Mike Hoye wrote: Given that we've been talking about this stuff for years now, I think it's very clear that we haven't come to this point by "somebody at the top issuing an edict that they want something modern"; the decision to commit to Phabricator w

Re: Phabricator Update, July 2017

2017-07-24 Thread Mark Côté
On Wednesday, 19 July 2017 16:19:02 UTC-4, Randell Jesup wrote: > >On 2017-07-14 1:31 AM, Jim Blandy wrote: > >> Many people seem to be asking, essentially: What will happen to old bugs? > >> I'm trying to follow the discussion, and I'm not clear on this myself. > >> > >> For example, "Splinter wi

Phabricator and confidential reviews

2017-08-08 Thread Mark Côté
(Cross-posted to mozilla.tools) Hi, I have an update and a request for comments regarding Phabricator and confidential reviews. We've completed the functionality around limiting access to Differential revisions (i.e. code reviews) that are tied to confidential bugs. To recap the original plan

Re: Phabricator and confidential reviews

2017-08-09 Thread Mark Côté
For brevity and clarity I'm just replying to Dan here, but I am attempting to address other points raised so far in this thread. On Wednesday, 9 August 2017 13:07:08 UTC-4, Daniel Veditz wrote: > On Tue, Aug 8, 2017 at 5:30 PM, Mark Côté wrote: > > > I am not sure how oft

Re: Phabricator and confidential reviews

2017-08-28 Thread Mark Côté
On Saturday, 26 August 2017 00:40:08 UTC-4, Randell Jesup wrote: > >And don't forget reporter and assignees. Occasionally a reporter not in the > >security group will notice that a patch is insufficient which is nicer to > >find before the patch is committed than after the commit link is added to

Phabricator/Lando update, November 2017

2017-11-29 Thread Mark Côté
I posted an update on Phabricator and Lando to my blog a couple weeks ago, but I figured I should share it here too: https://mrcote.info/blog/2017/11/17/phabricator-and-lando-november-update/ There are two important points: 1. Our Phabricator instance has been up and running for a few months no

Re: Phabricator/Lando update, November 2017

2017-11-29 Thread Mark Côté
Right, I should have mentioned that. We are working right now on enforcing MFA for Phabricator via BMO. See https://bugzilla.mozilla.org/show_bug.cgi?id=1393950. Should go out next week. Mark On Nov 29, 2017 12:41 PM, "Andreas Tolfsen" wrote: > Also sprach Mark Côté: > > &g

Re: Phabricator/Lando update, November 2017

2017-11-29 Thread Mark Côté
On Wednesday, 29 November 2017 12:43:58 UTC-5, Steve Fink wrote: > On 11/29/2017 08:35 AM, Mark Côté wrote: > > I posted an update on Phabricator and Lando to my blog a couple weeks ago, > > but I figured I should share it here too: > > https://mrcote.info/blog/2017/11/17/

Re: Phabricator/Lando update, November 2017

2017-12-07 Thread Mark Côté
cation > >> option was TOTP that requires a smartphone. I do not have a smartphone > >> like Mark. > >> > >> How can I continue to contribute after we are forced to use Phabricator? > >> Mozilla no longer wants volunteer contributors? > >> > >

Re: Commit messages in Phabricator

2018-02-13 Thread Mark Côté
On Tuesday, 13 February 2018 14:05:55 UTC-5, Haik Aftandilian wrote: > On Mon, Feb 12, 2018 at 6:30 AM, Eric Rescorla wrote: > > > On Mon, Feb 12, 2018 at 6:09 AM, Boris Zbarsky wrote: > > Instead, maybe we can arrange for Phab/Lando to put the bug #in the short > > message, potentially also wi

Re: BMO service degradation

2018-03-26 Thread Mark Côté
The issue has been resolved and all queued bugmail has been sent out. Thank you for your patience. Mark On Monday, 26 March 2018 11:48:04 UTC-4, Mark Côté wrote: > As a result of the migration of bugzilla.mozilla.org (BMO) from SCL3 to AWS, > mail from BMO is currently being sen

BMO service degradation

2018-03-26 Thread Mark Côté
As a result of the migration of bugzilla.mozilla.org (BMO) from SCL3 to AWS, mail from BMO is currently being sent very slowly. The mail servers are processing requests much more slowly than expected. There is no data loss; the mail will be sent out eventually. The BMO and Cloud Operations teams

Re: Phabricator and Bugzilla

2018-03-31 Thread Mark Côté
Regarding comment and flag mirroring, we've discussed this before: https://groups.google.com/d/msg/mozilla.dev.platform/Y8kInYxo8UU/e3Pi-_FpBgAJ https://groups.google.com/d/msg/mozilla.dev.platform/Y8kInYxo8UU/tsF7UfxvBgAJ Given that Phabricator is still new, I don't see any reason to reopen that

Re: Phabricator and Bugzilla

2018-04-05 Thread Mark Côté
As I indicated, those posts go into detail on why we are avoiding both comment and more complicated flag mirroring. Mark On Sat, Mar 31, 2018 at 10:14 AM, Ben Kelly wrote: > On Sat, Mar 31, 2018, 10:09 AM Mark Côté wrote: > >> Regarding comment and flag mirroring, we'

Re: Phabricator and Bugzilla

2018-04-12 Thread Mark Côté
I linked in one of my replies above. Mark > > >On Sat, Mar 31, 2018 at 10:14 AM, Ben Kelly wrote: > > > >> On Sat, Mar 31, 2018, 10:09 AM Mark Côté wrote: > >> > >>> Regarding comment and flag mirroring, we've discussed this before:

Re: Is super-review still a thing?

2018-04-25 Thread Mark Côté
A few comments on Phabricator and Lando: Phabricator has two types of review requests: "reviewer" and "blocking reviewer". These are only really differentiated if there is more than one reviewer on a revision. In that case, if there is a blocking reviewer, the revision is only marked "accepte

Re: [Update] Revert RESOLVED:INACTIVE State

2018-05-30 Thread Mark Côté
The process finished this morning. The vast majority of bugs were successfully reverted. As intended, some were skipped due to having been updated since the first script ran. There were also a handful of unexpected errors that we will be cleaning up manually. See https://bug1465190.bmoattac

Launch of Phabricator and Lando for mozilla-central

2018-06-06 Thread Mark Côté
The Engineering Workflow team is happy to announce the release of Phabricator and Lando for general use. Going forward, Phabricator will be the primary code-review tool for modifications to the mozilla-central repository, replacing both MozReview and Splinter. Lando is an all-new automatic-landi

Re: Launch of Phabricator and Lando for mozilla-central

2018-06-06 Thread Mark Côté
On Wednesday, 6 June 2018 11:18:43 UTC-4, Boris Zbarsky wrote: > On 6/6/18 10:57 AM, Mark Côté wrote: > > An upcoming post will outline the plans for the deprecation, archival, and > > decommission of MozReview, with Splinter to follow. > > Just a quick question:

Re: Launch of Phabricator and Lando for mozilla-central

2018-06-06 Thread Mark Côté
On Wednesday, 6 June 2018 12:08:49 UTC-4, Boris Zbarsky wrote: > On 6/6/18 11:48 AM, Mark Côté wrote: > > Good question. Probably, as it has different uses, but it shouldn't be > > used to work around Phabricator. :) > > A related question: How is Phabricator'

Re: Launch of Phabricator and Lando for mozilla-central

2018-06-06 Thread Mark Côté
On Wednesday, 6 June 2018 15:18:43 UTC-4, Jan-Ivar Bruaroey wrote: > On 6/6/18 3:03 PM, Boris Zbarsky wrote: > > On 6/6/18 2:52 PM, Jan-Ivar Bruaroey wrote: > > Mozreview will show me the equivalent of "diff -r B -r P1", "diff -r P1 > > -r P2" and "diff -r P2 -r P3".  If there are then edits to P

Re: Launch of Phabricator and Lando for mozilla-central

2018-06-07 Thread Mark Côté
On Wednesday, 6 June 2018 19:58:43 UTC-4, Xidorn Quan wrote: > On Thu, Jun 7, 2018, at 12:57 AM, Mark Côté wrote: > > Phabricator is a suite of applications, but we are primarily using the > > code-review tool, called Differential, which will be taking the place of > > M

Re: Update: Phabricator and commit series

2018-07-05 Thread Mark Côté
Yup, patches will be archived. We have a plan for the migration that I will announce soonish (was intending on doing so earlier, but the commit-series work required some adjustments to the plan). Mark On Wed, Jul 4, 2018 at 6:11 AM, Dão Gottwald wrote: > 2018-07-03 21:34 GMT+02:00 Mark C

Update: Phabricator and commit series

2018-07-05 Thread Mark Côté
Hi all, After listening to feedback and discussing requirements with some of engineering leadership, the Engineering Workflow team has decided to write a new command-line application to better support submitting and updating commit series with Phabricator. We’ve been in touch with Phacility, and i

Re: PSA: pay attention when setting multiple reviewers in Phabricator

2018-07-10 Thread Mark Côté
te interested in and has been a long time coming. Mark On Thu, Jul 5, 2018 at 3:27 PM, James Graham wrote: > On 05/07/2018 18:19, Mark Côté wrote: > >> I sympathize with the concerns here; however, changing the default would >> be a very invasive change to Phabricator, which wou

Re: PSA: pay attention when setting multiple reviewers in Phabricator

2018-07-10 Thread Mark Côté
I sympathize with the concerns here; however, changing the default would be a very invasive change to Phabricator, which would not only be complex to implement but troublesome to maintain, as we upgrade Phabricator every week or two. This is, however, something we can address with our new custom c

Re: Update: Phabricator and commit series

2018-07-10 Thread Mark Côté
The diffs for all review requests will be archived. More details soon. :) Mark On Thu, Jul 5, 2018 at 4:23 PM, Henrik Skupin wrote: > Mark Côté wrote on 03.07.18 21:34: > > > We’re aiming for late July for these implementations, which will allow us > > to close down MozRev

Plan for Sunsetting MozReview

2018-07-26 Thread Mark Côté
To follow up on some previous threads, here is the plan for deprecating, archiving, and decommissioning MozReview. The MozReview shutdown deadline is approaching. Although enhanced commit-series support is still in progress (see update below), MozReview users should start familiarizing themselves

Re: Plan for Sunsetting MozReview

2018-07-26 Thread Mark Côté
ll be putting a bundle of the review repo up on S3 as well for anyone who wants to dig deeper. Mark On Thu, Jul 26, 2018 at 3:02 PM, Botond Ballo wrote: > On Thu, Jul 26, 2018 at 2:37 PM, Mark Côté wrote: > > Every landed, in-progress, and abandoned patch will be downloaded > >

Re: Plan for Sunsetting MozReview

2018-07-26 Thread Mark Côté
18 at 5:55 PM, Karl Tomlinson wrote: > Mark Côté writes: > > > On August 20, we will remove public access to MozReview and archive > > patches. Every landed, in-progress, and abandoned patch will be > downloaded > > from MozReview and stored in an S3 bucket. The “stub a

Re: Plan for Sunsetting MozReview

2018-07-27 Thread Mark Côté
I talked to gps, and yes, we can totally do that. Mark On Thu, Jul 26, 2018 at 6:39 PM, Mike Hommey wrote: > On Thu, Jul 26, 2018 at 06:31:34PM -0400, Mark Côté wrote: > > The problem there is that the review repo will be bundled and stored. We > > don't want to run anot

Re: Plan for Sunsetting MozReview

2018-07-27 Thread Mark Côté
h bootstrap, it is no problem. > > > -- Makoto Kato > > *1 https://developer.mozilla.org/en-US/docs/Mozilla/Developer_ > guide/Introduction#Step_4_Get_your_code_reviewed > *2 http://moz-conduit.readthedocs.io/en/latest/phabricator-user.html > > On Fri, Jul 27, 2018 at 3:37 AM,

Re: Plan for Sunsetting MozReview

2018-07-27 Thread Mark Côté
ished commits that were part of a diff's history. There is no change to the update-only period from August 6 to August 20. Mark On Thu, Jul 26, 2018 at 2:37 PM, Mark Côté wrote: > To follow up on some previous threads, here is the plan for deprecating, > archiving, and decommiss

Initial release of Arcanist wrapper

2018-08-08 Thread Mark Côté
The first iteration of our commit-series-friendly Arcanist wrapper is ready for use. At this time, it only supports Mercurial (without mq) and doesn't yet support Windows, but we wanted to get what we have out in front of users. We are continuing work on Windows support ( https://bugzilla.mozilla.o

MozReview now uses Bugzilla's auth delegation

2015-08-26 Thread Mark Côté
tl;dr We improved the authentication system in MozReview. Please log out (if necessary) and back into the MozReview UI (Review Board) before pushing any commits up for review. Today we deployed a change to MozReview's authentication system. Rather than logging into Review Board with your Bugzill

New BMO SSL certificate coming 2015/10/29

2015-10-27 Thread Mark Côté
BMO's SSL certificate expires on 2015/10/30, so please be advised we will be installing a new one around 13:30 UTC on 2015/10/29 (9:30 am EDT/6:30 am PDT). The new fingerprint will be 7c:7a:c4:6c:91:3b:6b:89:cf:f2:8c:13:b8:02:c4:25:bd:1e:25:17 `mach hgsetup` will be patched as soon as the new ce

Re: Just Autoland It

2016-01-29 Thread Mark Côté
On 2016-01-29 10:27 AM, Eric Rescorla wrote: > On Fri, Jan 29, 2016 at 6:22 AM, Andrew Halberstadt < > ahalberst...@mozilla.com> wrote: > >> On 28/01/16 06:31 PM, Eric Rescorla wrote: >> >>> On Thu, Jan 28, 2016 at 10:58 AM, Gregory Szorc >>> wrote: >>> >>> I'd like to thank everyone for the feed

Re: MozReview/Autoland in degraded state

2016-02-05 Thread Mark Côté
We will be deploying a fix for the ssh-level restrictions to MozReview shortly, around 2:30 pm EST/11:30 am PST. MozReview will be down for about 10 minutes if all goes smoothly. We'll be able to rollback not long after that if there are unresolvable issues. You can follow along in #mozreview.

Re: MozReview/Autoland in degraded state

2016-02-05 Thread Mark Côté
;ll report back here when they're back, hopefully Monday. Mark On 2016-02-05 1:42 PM, Mark Côté wrote: > We will be deploying a fix for the ssh-level restrictions to MozReview > shortly, around 2:30 pm EST/11:30 am PST. MozReview will be down for > about 10 minutes if all goes smooth

Re: Why is Mozreview hassling me about squashed commits?

2016-04-04 Thread Mark Côté
On 2016-04-04 10:07 AM, Eric Rescorla wrote: > On Sun, Apr 3, 2016 at 10:09 PM, L. David Baron wrote: > >> On Saturday 2016-04-02 18:51 -0300, Eric Rescorla wrote: >>> 1. I write a bunch of code, committing along the way, so I have a lot of >>> commits named "Checkpoint" and "Fix bug" and the lik

Re: Why is Mozreview hassling me about squashed commits?

2016-04-04 Thread Mark Côté
On 2016-04-04 8:41 PM, Eric Rescorla wrote: > On Mon, Apr 4, 2016 at 9:23 PM, Mark Côté wrote: >> To answer the original question, though, at this time we have no plans >> to completely do away with the squashed-commit view. However, in the >> interests of ensuring that

MozReview's interdiffs

2016-04-07 Thread Mark Côté
MozReview, specifically, Review Board, has a number of bugs related to interdiffs[1]. Many of these bugs are related to Review Board's algorithms to filter out changes caused by rebases (as opposed to intentional updates to commits). Both the MozReview and the Review Board teams have done some inve

Re: [Bug 1224726] High memory consumption when opening and searching a large Javascript file in debugger.

2016-04-11 Thread Mark Côté
Tagging the comments as spam will autoban the account after a certain number. It will also autocollapse the comments. Mark On 2016-04-11 6:35 PM, Lawrence Mandel wrote: > Good intentions or not, we need to stop this activity. > > Mark - What's our usual approach to address bug spam? > > Lawre

Re: Triage Plan for Firefox Components

2016-04-12 Thread Mark Côté
On 2016-04-07 2:50 AM, L. David Baron wrote: > (I'd much rather a bug report be editable text, with history > available, for answers to these or similar questions -- rather than > a stream of permanent comments. But we seem stuck with the horrid > stream-of-comments Bugzilla format, which means I

Re: Triage Plan for Firefox Components

2016-04-13 Thread Mark Côté
On 2016-04-13 9:34 AM, Gervase Markham wrote: > On 12/04/16 21:01, Mark Côté wrote: >> Meant to reply to this earlier... BMO has a User Story field that sounds >> like it does exactly what you want. It's an editable field that keeps >> history (admittedly not in an

Re: Readable Bug Statuses in Bugzilla

2016-05-24 Thread Mark Côté
Indeed, see tracking bug: https://bugzilla.mozilla.org/show_bug.cgi?id=1150541 Mark On 2016-05-24 1:03 PM, Emma Humphries wrote: > Yes, the plan is that the 'Modal' view will become the default, and the bmo > team is working on that. > > Meanwhile, you can beat the rush and switch over to the v

Re: review stop-energy (was 24hour review)

2013-07-10 Thread Mark Côté
On 2013-07-09 4:48 PM, Boris Zbarsky wrote:> On 7/9/13 4:29 PM, Chris Peterson wrote: >>> Bugzilla's interdiff is totally unsuitable for this >>> purpose, unfortunately, because it fails so often. >> >> Can we fix Bugzilla's interdiff? > > Not easily, because it does not have access to the original

Re: review stop-energy (was 24hour review)

2013-07-10 Thread Mark Côté
On 2013-07-10 2:18 PM, Boris Zbarsky wrote: > On 7/10/13 1:58 PM, Mark Côté wrote: >> The BMO team is again considering switching to ReviewBoard, which should >> fix this problem > > How does ReviewBoard address this? > > Again, what we have in the bug is diff 1 aga

Re: review stop-energy (was 24hour review)

2013-07-11 Thread Mark Côté
On 2013-07-11 7:59 AM, Gervase Markham wrote: > On 09/07/13 21:29, Chris Peterson wrote: >> I've seen people change their Bugzilla name to include a comment about >> being on PTO. We should promote this practice. We could also add a >> Bugzilla feature (just a simple check box or a PTO date range)

Re: bugzilla can now show bugs that have been updated since you last visited them

2014-06-05 Thread Mark Côté
On 2014-06-04, 3:01 PM, Neil wrote: > Byron Jones wrote: > >> thanks to dylan's work on bug 489028, bugzilla now tracks when you >> view a bug, allowing you to search for bugs which have been updated >> since you last visited them. > > I shared a basic search which I call "Unseen Changes". > > I

Re: BzAPI Compatibility API has been rolled out to production BMO

2014-06-25 Thread Mark Côté
Those are just the API root paths, for reference. For example, to view a bug, they would be https://bugzilla.mozilla.org/bzapi/bug/35 https://bugzilla.mozilla.org/rest/bug/35 Mark On 2014-06-22, 4:42 AM, Josh Matthews wrote: > [5] https://bugzilla.mozilla.org/bzapi > [6] https://bugzilla.mozil

Re: BzAPI Compatibility API has been rolled out to production BMO

2014-06-26 Thread Mark Côté
On 2014-06-25, 3:21 PM, jmor...@mozilla.com wrote: > This is terrific! > > The docs make mention of POST under bz_rest_options. Do you now (or will you > at some point) support bug creation via API? Would you do full CRUD at some > point? Yes, the native REST API already supports bug creation

Review Board Preview

2014-09-04 Thread Mark Côté
I know lots of people are very interested in the on-going project to replace Splinter with a modern code-review tool. After a colourful variety of setbacks, this project[1], based on Review Board[2], is very nearly ready for initial deployment. I put up a preview screencast on my blog[3] to give

Re: Review Board Preview

2014-09-15 Thread Mark Côté
uld nicely solve this if it's possible > to work with projects other than gecko, right? > > On Fri, Sep 5, 2014 at 2:57 AM, Mark Côté wrote: > >> I know lots of people are very interested in the on-going project to >> replace Splinter with a modern code-review tool. After

MozReview ready for general use

2014-11-05 Thread Mark Côté
A couple months ago I gave a sneak peak into our new repository-based code-review tool based on Review Board. I'm excited to announce that this tool, now named (descriptively but unimaginatively) MozReview, is ready for general use. In the interests of beta-testing our documentation at the same t

Re: MozReview ready for general use

2014-11-10 Thread Mark Côté
On 2014-11-10 5:51 AM, David Rajchenbach-Teller wrote: > Is there any chance we could log in with Persona? > > Cheers, > David > > On 06/11/14 05:50, Mark Côté wrote: >> A couple months ago I gave a sneak peak into our new repository-based >> code-review tool ba

Re: MozReview ready for general use

2014-11-10 Thread Mark Côté
, Gijs Kruitbosch wrote: > It looks like all reviews (and patches) are currently public. Is there > some way to have them not be so, for security/confidential bugs/reviews? > > ~ Gijs > > On 06/11/2014 04:50, Mark Côté wrote: >> A couple months ago I gave a sneak peak into

You can now log into BMO with your GitHub account

2015-04-27 Thread Mark Côté
This morning we enabled a feature on bugzilla.mozilla.org that allows users to log in with their GitHub credentials, similar to our existing Persona support. If you have several email addresses associated with your GitHub account, you will be prompted to choose one. In either case, if your chosen

Re: You can now log into BMO with your GitHub account

2015-04-27 Thread Mark Côté
On 2015-04-27 10:41 AM, Mike Hoye wrote: > On 2015-04-27 10:15 AM, Mark Côté wrote: >> This morning we enabled a feature on bugzilla.mozilla.org that allows >> users to log in with their GitHub credentials, similar to our existing >> Persona support. If you have several email

Voting in BMO

2015-06-09 Thread Mark Côté
In a quest to simplify both the interface and the maintenance of bugzilla.mozilla.org, we're looking for features that are of questionable value to see if we can get rid of them. As I'm sure everyone knows, Bugzilla grew organically, without much of a road map, over a long time, and it experienced

Re: Voting in BMO

2015-06-09 Thread Mark Côté
On 2015-06-09 5:39 PM, Sören Hentzschel wrote: > On 09.06.15 23:24, Chris Peterson wrote: >> I vote for bugs as a polite (sneaky?) way to watch a bug's bugmail >> without spamming all the other CCs by adding myself to the bug's "real" >> CC list. > > Same here. Removing the voting feature means th

Re: Voting in BMO

2015-06-09 Thread Mark Côté
On 2015-06-09 5:24 PM, Xidorn Quan wrote: > On Wed, Jun 10, 2015 at 9:09 AM, Mark Côté wrote: > >> In a quest to simplify both the interface and the maintenance of >> bugzilla.mozilla.org, we're looking for features that are of >> questionable value to see if we can

Re: Voting in BMO

2015-06-09 Thread Mark Côté
On 2015-06-09 6:00 PM, Justin Dolske wrote: > That said, there are much bigger issues with Bugzilla's UI, and removing > voting is probably the smallest possible improvement. But it's probably > easy to just disable it for a while, and see what happens? Indeed, it's a minor thing. Consider it a t

Re: Voting in BMO

2015-06-10 Thread Mark Côté
On 2015-06-09 11:58 PM, Wayne wrote: >> That said, there are much bigger issues with Bugzilla's UI, and removing >> voting is probably the smallest possible improvement. But it's probably >> easy to just disable it for a while, and see what happens? > > I never have seen the voting UI as being the

Re: Voting in BMO

2015-06-10 Thread Mark Côté
On 2015-06-10 7:06 AM, Philipp Kewisch wrote: > I could live without this feature if the number of people on CC gives > some indication of how wanted a feature may be. Can you check > correlation between the number of votes and the number of people on CC? A quick scan of the Core & Firefox product

Re: Voting in BMO

2015-06-10 Thread Mark Côté
Thanks for all the input on this feature. This was a good discussion. Here's what I've learned: * Almost no one makes decisions based on the number of votes (Thunderbird and related may be an exception). ** Ergo, most users voting on bugs are probably being misled into thinking their vote ac

Re: Voting in BMO

2015-06-11 Thread Mark Côté
On 2015-06-11 4:46 PM, Chris Hofmann wrote: > On Thu, Jun 11, 2015 at 1:18 PM, Mike Hoye wrote: > >> On 2015-06-11 3:48 PM, R Kent James wrote: >> >>> Maybe the correct fix is to start paying attention to votes. >>> >> If you choose your project priorities based on internet voting, you're >> gonn

Re: Voting in BMO

2015-06-11 Thread Mark Côté
On 2015-06-11 6:43 PM, Chris Hofmann wrote: >> Furthermore, since bugs with lots of votes also have lots of CCs (see an >> earlier post of mine), if we want to just acknowledge that a bug is >> popular, we can just use CC counts above a certain threshold. >> Admittedly there's no way to search for

Windows support for moz-phab and Arcanist

2018-08-21 Thread Mark Côté
Quick update: - Landed Windows 10 support for moz-phab (our commit-series-friendly Arcanist wrapper). - Improved installation and usage instructions for moz-phab. - Referenced moz-phab from our Phabricator user guide (mainly just a link to moz-phab's README). - Published our own Arcanist installat

Re: Plan for Sunsetting MozReview

2018-08-28 Thread Mark Côté
See https://bugzilla.mozilla.org/show_bug.cgi?id=1480887 for the redirect service, which is in progress. Mark On Mon, Aug 27, 2018 at 6:57 PM, Botond Ballo wrote: > > Until this gets fixed, a workaround for closed bugs is to go to the > bottom of the bug, and look for https://hg.mozilla.org/ >

Changes to BMO-Phabricator integration

2018-09-12 Thread Mark Côté
To reduce confusion and a growing maintenance burden, the Engineering Workflow team plans to remove two pieces of Phabricator-Bugzilla integration: 1. The setting of r+ flags on the stub attachments in that link to Phabricator revisions ( https://bugzilla.mozilla.org/show_bug.cgi?id=

Changes to BMO-Phabricator integration

2018-09-13 Thread Mark Côté
(apologies: this was sent out to firefox-dev and dev.platform yesterday but for some reason appears not to have made it to the latter) To reduce confusion and a growing maintenance burden, the Engineering Workflow team plans to remove two pieces of Phabricator-Bugzilla integration: 1. Th

Re: PSA: Phabricator silently drops the content of binary files on Windows

2018-10-01 Thread Mark Côté
On Saturday, 29 September 2018 21:59:48 UTC-4, Boris Zbarsky wrote: > On 9/29/18 11:31 AM, tom...@gmail.com wrote: > > Our modern roadmap is almost entirely driven by paying customers, and no > > customers have expressed interest in this. > > Are we a paying customer? Yes we are. I'll file an

Phabricator and updates to commit messages

2018-10-01 Thread Mark Côté
There’s been a bit of confusion around Phabricator and updates to commit messages lately (https://bugzilla.mozilla.org/show_bug.cgi?id=1476425). Arcanist's default workflow is to not update a revision’s Summary field even if the associated commit message is updated (e.g. by `hg commit --amend`) and

Re: PSA: Phabricator silently drops the content of binary files on Windows

2018-11-13 Thread Mark Côté
On Monday, 1 October 2018 11:07:18 UTC-4, Mark Côté wrote: > On Saturday, 29 September 2018 21:59:48 UTC-4, Boris Zbarsky wrote: > > On 9/29/18 11:31 AM, tom...@gmail.com wrote: > > > Our modern roadmap is almost entirely driven by paying customers, and no > > &g

Re: Workflow Apropos!

2018-11-28 Thread Mark Côté
On Wed, Nov 28, 2018 at 1:34 PM James Graham wrote: > Can you share with us the long term vision for what the workflow is > going to look like here? I've recently seen a few cases where > experienced develoeprs who have either never contributed to gecko before > or contribute infrequently tried t