Gijs gijsk

gijsk commented on issue mozilla/readability#237
@gijsk

Happy to do that, though I'm not sure how many consumers we have beyond iOS...

gijsk commented on issue mozilla/readability#237
@gijsk

Is this reproducible on android or desktop in some way? I don't have iOS available so it's hard to know what's wrong right now...

@gijsk
Fix some of the remaining errors that show up in stderr during session restore and/or other circumstances unclear to me...
gijsk commented on issue gijsk/triage-helper#3
@gijsk

Fixed in 0.1.5 by commit c133a75 - was due to restored tabs

gijsk commented on issue gijsk/triage-helper#4
@gijsk

Fixed in 0.1.5 by commit 13b3124

@gijsk
Try using the BMO builtin api-key before prompting users to make using the add-on easier
@gijsk
Add checkbox for "add in-triage keyword"
@gijsk
@gijsk
@gijsk
@gijsk
Reader View is available for various websites although it shouldn't
gijsk commented on issue mozilla/readability#227
gijsk commented on issue mozilla/readability#227
@gijsk

So the page does some unholy replaceState magic and we seem to get 0 pagehide/pageshow events - just an onLocationChange in the browser that asks u…

gijsk commented on issue mozilla/readability#227
@gijsk

This should have been filed on bugzilla. Anyway, let's investigate here because this looks very broken. @leibovic , ideas on what's going on here? :-(

gijsk commented on pull request armenzg/mozilla_ci_tools#198
@gijsk

It is not clear to me how this is meant to work because I don't know enough about these scripts. I was originally warned that for the talos script,…

@gijsk
  • @gijsk c1ae713
    Add a way to filter out android/desktop and/or specific talos jobs fr…
gijsk opened pull request armenzg/mozilla_ci_tools#198
@gijsk
Add a way to filter out android/desktop and/or specific talos jobs
1 commit with 39 additions and 0 deletions
gijsk commented on issue armenzg/mozilla_ci_tools#197
@gijsk

Trying again produces: (virtualenv_) E:\dev\mozilla_ci_tools\mozci\scripts>python alltalos.py --times 14 --rev 188165e33ad0 --repo-name try 05/14/2…

gijsk commented on issue armenzg/mozilla_ci_tools#196
@gijsk

Maybe my earlier comment wasn't clear enough - I want the *sh versions of the virtualenv activate/deactivate scripts to be present in the repo on c…

gijsk commented on issue armenzg/mozilla_ci_tools#196
@gijsk

I'm confused - don't I want to clone because I need to change the script in question? It seemed evil to get the package with pip/easy_install and t…

@gijsk
Script fails without indicating if anything was triggered or not
gijsk commented on issue armenzg/mozilla_ci_tools#196
@gijsk

Ah! This helps, it seems. I will note, which I forgot to mention in the previous comment, that on a Windows machine virtualenv will only generate b…

gijsk commented on issue armenzg/mozilla_ci_tools#196
@gijsk

So I've easy_install virtualenv on my Windows box, and created a virtualenv, and activated it, and then running: (virtualenv_) E:\dev\mozilla_ci_to…

@gijsk
Readme and instructions are not usable for non-python-wizards
gijsk commented on issue gijsk/triage-helper#8
@gijsk

Yes. I'm a little confused what you mean, though - you want a checkbox on every page that does this when ticked? Or a pref in the add-on? Right now…

gijsk commented on issue mozilla/readability#225
@gijsk

Because it ends up in a and the text is long enough, that is why.

gijsk opened pull request mozilla/readability#224
@gijsk
Fixes #220 - ignore on-air-board and slider things
1 commit with 1,826 additions and 7 deletions