I'm booking this in the next day or so -- does anyone have any
recommendations for good nearby hotels?
(The best recommendation is if you're coming, too and staying there
yourself!)
Ian
On Tue, Aug 12, 2014 at 6:43 PM, Gorkem Ercan
wrote:
>
> Just returning from PTO, great timing :)
>
:)
>
> On Tue, Aug 12, 2014 at 04:06:14PM -0400, Andrew Grieve wrote:
> > Played around with it and it's pretty clear to me that the ability to
> > record your plugins & platforms in config.xml is
On Tue, Aug 12, 2014 at 6:58 PM, Gorkem Ercan
wrote:
> On Tue, Aug 12, 2014 at 08:40:25PM +, Chuck Lantz wrote:
> > +1
> >
> > That same pattern could be applied to platforms actually with an
> additional version attribute:
> >
> >
> > ... things like icons, splaschreens, and maybe eve
Yes, good point - Took a look at the PR with platforms - seems like a similar
concept but using the engine element which as I think about it would probably
be better anyway.
More consistent with the existing plugin.xml
Would we need / want to support restoring from git repos or oth
Github user asfgit closed the pull request at:
https://github.com/apache/cordova-plugin-contacts/pull/38
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the
I have made changes to cordova-lib/src/plugman/prepare-browserify.js to
support this work for cordova.js.
It is now:
1) Computing commit Ids. I had to update my cordova-js branch to set cwd to
be cordova-js repo root. That way `git rev-list HEAD --max-count=1` is
always run from the right locatio
Github user asfgit closed the pull request at:
https://github.com/apache/cordova-plugin-media/pull/21
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the fe
Given that we are looking at decoupling engine and platform releases, there
should be ways to specify them separately, right ? In this case, I am assuming
it is basically version of cordova-cli/cordova-lib and the platform, assuming
that cordova-cli can work with older platform versions.
-
On Tue, Aug 12, 2014 at 08:40:25PM +, Chuck Lantz wrote:
> +1
>
> That same pattern could be applied to platforms actually with an additional
> version attribute:
>
>
> ... things like icons, splaschreens, and maybe even packaging details
> go here ...
>
>
> We could also follow a
Just returning from PTO, great timing :)
On Tue, Aug 12, 2014 at 04:06:14PM -0400, Andrew Grieve wrote:
> Played around with it and it's pretty clear to me that the ability to
> record your plugins & platforms in config.xml is a big step up.
>
> I do have some specific comments about the curren
Hi, Martin
Another problem i have is using createmobilespec module. I am testing
mobilespec on amazon-fireos platform and script doesn't work for me. So i
create mobilespec manually. Hence, its quite broken and most tests are not
working. Are you familiar with createmobilespec.js?
Archana
On Tu
Cool. Thank you Martin. I will clone plugin-test-framework and try it out.
One question: I see mobilespec's main page still has other buttons for some
the tests that were removed. Dont we want to remove those from index.html?
Archana
On Tue, Aug 12, 2014 at 2:43 PM, Martin Gonzalez <
martin.c.g
Hi Archana,
The recent changes in mobile spec are aimed to use cordova
plugin-test-framework.
plugin-test-framework brings a new style, usage of jasmine 2.0, and several
other features (https://github.com/apache/cordova-plugin-test-framework).
All tests were moved to its respective plugin, and por
Hi, Devs
Recently, I noticed mobilespec has gone through changes and want to know
whats the new workflow to test all the plugins? Can someone please
summarize?
Thanks
Archana
Believe me, I want to go all node -- but all in for all scripts --
which we don't have time to do yet (maybe 4.0?).
But seeing that it's just replacing the contents of the current bash
script with python code, it's the path of least resistance, and path
of least potential conflict imo. No one will
+1
That same pattern could be applied to platforms actually with an additional
version attribute:
... things like icons, splaschreens, and maybe even packaging details
go here ...
We could also follow a similar model if we wanted to say what top level cordova
version was used to cre
Github user asfgit closed the pull request at:
https://github.com/apache/cordova-js/pull/72
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is e
Shaz, that's technically true, but how many users actually use that path
these days?
I thought the last stats overwhelmingly suggest our users are drinking the
kool-aid and using cli, node, etc.
On Tue, Aug 12, 2014 at 4:19 PM, Shazron wrote:
> Not if they are installed manually. It's not wort
On Tue, Aug 12, 2014 at 4:18 PM, Lorin Beer wrote:
> the version scripts are just about the only thing about the platform
> scripts that are simple enough to rewrite trivially.
Which is why the Android version scripts are literally
console.log("whatever_the_version_was_when_we_updated_this_fil
yeah, do what is easiest, and adds the least dependencies.
My point was just that I wonder if non-CLI paths still exist. #offtopic ...
@purplecabbage
risingj.com
On Tue, Aug 12, 2014 at 1:19 PM, Shazron wrote:
> Not if they are installed manually. It's not worth having some
> dependency just t
is nice, but why not just as plugin.xml already uses?
config.xml and plugin.xml share lots of tags already, why fork here?
-Michal
On Tue, Aug 12, 2014 at 4:06 PM, Andrew Grieve wrote:
> Played around with it and it's pretty clear to me that the ability to
> record your plugins & platforms
Github user asfgit closed the pull request at:
https://github.com/apache/cordova-mobile-spec/pull/80
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the fea
Not if they are installed manually. It's not worth having some
dependency just to read a version, that's nuts.
On Tue, Aug 12, 2014 at 1:15 PM, Jesse wrote:
> the non-cordova cli path depends on node to install/uninstall plugins
>
> @purplecabbage
> risingj.com
>
>
> On Tue, Aug 12, 2014 at 1:08
the version scripts are just about the only thing about the platform
scripts that are simple enough to rewrite trivially. Leveraging a
rewrite/refactor of the version script into a total rewrite of platform
level scripts == 'lulz'
On Tue, Aug 12, 2014 at 1:15 PM, Jesse wrote:
> the non-cordova
the non-cordova cli path depends on node to install/uninstall plugins
@purplecabbage
risingj.com
On Tue, Aug 12, 2014 at 1:08 PM, Shazron wrote:
> Of course I considered nodejs, but no, this is for the non-cordova CLI
> path, which does not need another dependency.
>
> On Tue, Aug 12, 2014 at
Of course I considered nodejs, but no, this is for the non-cordova CLI
path, which does not need another dependency.
On Tue, Aug 12, 2014 at 11:52 AM, Jesse wrote:
> Yeah, if you are going to replace bash, replace it with nodejs!
>
>
> @purplecabbage
> risingj.com
>
>
> On Tue, Aug 12, 2014 at 11
Played around with it and it's pretty clear to me that the ability to
record your plugins & platforms in config.xml is a big step up.
I do have some specific comments about the current design though:
- Right now the plugin save saves all plugins to config.xml rather than
just explicitly-installed
sync xhr ftw!
On Aug 12, 2014 12:40 PM, "Brian LeRoux" wrote:
> we talked about this years ago…pretty cool stuff. basically the only use
> for a sync xhr call ever.
>
>
> On Tue, Aug 12, 2014 at 12:32 PM, Andrew Grieve
> wrote:
>
> > Very cool! Thanks for sharing.
> >
> > The technique used is t
I think that A) is likely the feasible solution for 3.6.0.
B) seems like a 4.0-scope project (but an interesting one; I hadn't
considered it working in that direction)
We'd probably spend a while debating the particulars -- I don't know if
changing the CSP syntax is a good idea, or if we'd want an
we talked about this years ago…pretty cool stuff. basically the only use
for a sync xhr call ever.
On Tue, Aug 12, 2014 at 12:32 PM, Andrew Grieve
wrote:
> Very cool! Thanks for sharing.
>
> The technique used is the same that JsHybugger uses. Differences though:
> - This is x-platform
> - This
But what should we store in the shed?
On Aug 12, 2014 12:33 PM, "Brian LeRoux" wrote:
> In another life I'd love to paint that shed.
>
>
>
> On Tue, Aug 12, 2014 at 12:25 PM, Jesse wrote:
>
> > Michal, exactly!
> > Brian, hilarious, I was implying rewrite ALL the platform scripts in
> nodejs
> >
In another life I'd love to paint that shed.
On Tue, Aug 12, 2014 at 12:25 PM, Jesse wrote:
> Michal, exactly!
> Brian, hilarious, I was implying rewrite ALL the platform scripts in nodejs
> ... as was recently done for cordova-windows. May be more work than it is
> worth though.
>
> @purpleca
Very cool! Thanks for sharing.
The technique used is the same that JsHybugger uses. Differences though:
- This is x-platform
- This requires a server
- This doesn't integrate with devtools
- This is free to use.
On Tue, Aug 12, 2014 at 1:56 PM, Jesse wrote:
> Sergey, this is brilliant!
>
> In
=)
I'm a bit of a bash nerd but in seriousness we have many Windows users so
keeping it pure Node is probably the best idea.
On Tue, Aug 12, 2014 at 12:17 PM, Myles Borins wrote:
> A, I forgot to shim my path with a sense of humor... Oops!
> On Aug 12, 2014 12:16 PM, "Brian LeRoux" wrote
Michal, exactly!
Brian, hilarious, I was implying rewrite ALL the platform scripts in nodejs
... as was recently done for cordova-windows. May be more work than it is
worth though.
@purplecabbage
risingj.com
On Tue, Aug 12, 2014 at 12:17 PM, Myles Borins wrote:
> A, I forgot to shim my pat
A, I forgot to shim my path with a sense of humor... Oops!
On Aug 12, 2014 12:16 PM, "Brian LeRoux" wrote:
> main benefit was humor though an argument could be made for other ones ;)
>
>
>
> On Tue, Aug 12, 2014 at 12:06 PM, Myles Borins wrote:
>
> > Brian, would there be any benefit to usin
A malicious XSS that navigates the page would probably opt to not include
cordova.js :P.
So - I see there being three kinds of things to have whitelists for:
1) Top-level navigations,
2) intents (on supported platforms)
3) resources / CSP-related requests.
According to CSP docs here:
https://deve
main benefit was humor though an argument could be made for other ones ;)
On Tue, Aug 12, 2014 at 12:06 PM, Myles Borins wrote:
> Brian, would there be any benefit to using bash to call node rather than a
> script that started with #! /bin/env node?
> On Aug 12, 2014 12:01 PM, "Brian LeRoux"
Brian, would there be any benefit to using bash to call node rather than a
script that started with #! /bin/env node?
On Aug 12, 2014 12:01 PM, "Brian LeRoux" wrote:
> OR BOTH
>
> #!/bin/sh
> node -e 'console.log(require("./package.json").version)'
>
>
> On Tue, Aug 12, 2014 at 11:52 AM, Jesse w
..re reading your email, I think thats exactly what you mean and also
suggested the same solution :P alrighty!
On Tue, Aug 12, 2014 at 3:00 PM, Michal Mocny wrote:
> Jesse what did you mean about output projects and package.json?
>
> Do you mean the result of running cordova create / ./create
OR BOTH
#!/bin/sh
node -e 'console.log(require("./package.json").version)'
On Tue, Aug 12, 2014 at 11:52 AM, Jesse wrote:
> Yeah, if you are going to replace bash, replace it with nodejs!
>
>
> @purplecabbage
> risingj.com
>
>
> On Tue, Aug 12, 2014 at 11:48 AM, Myles Borins wrote:
>
> > Have
Jesse what did you mean about output projects and package.json?
Do you mean the result of running cordova create / ./create script? If the
intention is to know what version of the platform was used to create the
project, then we can either copy the source package.json to the root of the
platform
+1
- Installed from archives
https://dist.apache.org/repos/dist/dev/cordova/CB-7267/
- Built a test app on OSX and ran it on Android (Nexus 7 with KitKat).
On Tue, Aug 12, 2014 at 12:31 PM, Sergey Grebnov (Akvelon) <
v-seg...@microsoft.com> wrote:
> +1
>
> * Verified archives
> * Performed fun
Yeah, if you are going to replace bash, replace it with nodejs!
@purplecabbage
risingj.com
On Tue, Aug 12, 2014 at 11:48 AM, Myles Borins wrote:
> Have you considered writing a small node script to pass the json? This
> would make it as simple as requiring in the package json an piping the
>
Have you considered writing a small node script to pass the json? This
would make it as simple as requiring in the package json an piping the
relevant info to stdout
On Aug 12, 2014 11:47 AM, "Shazron" wrote:
> Yeah I value life and my sanity - I'll probably replace the bash
> script with python
GitHub user emilioeduardob opened a pull request:
https://github.com/apache/cordova-plugin-contacts/pull/39
[CB-7131] Check for profile photo existance
This tries to solve the bug checking for the existance of the picture
before adding to the photo array
You can merge this pull req
A related topic as it pertains to CSP: If we are going down the CSP path, what
are people's thoughts on the security risks associated with inline script
("unsafe-inline") use as a part of the default policy - particularly from
content originating from outside of the Cordova app package? It's ob
Yeah I value life and my sanity - I'll probably replace the bash
script with python
On Tue, Aug 12, 2014 at 11:40 AM, Lorin Beer wrote:
> one source for version information is better
>
> although parsing json with bash scripts seems janky
>
>
> On Tue, Aug 12, 2014 at 11:31 AM, Jesse wrote:
>
>>
one source for version information is better
although parsing json with bash scripts seems janky
On Tue, Aug 12, 2014 at 11:31 AM, Jesse wrote:
> I think it still needs to exist in an output project ... which is not
> (yet?) an npm project, and so does not have a package.json.
>
> The individu
Heh now I have to read up on how to parse json using bash O_o
On Tue, Aug 12, 2014 at 11:30 AM, Ian Clelland wrote:
> I presume that the alternative would be for the version scripts to parse
> package.json instead. I think that Android used to work the same way, but
> now its version script just
For iOS, the only file I can see that depends on this is:
1.
https://github.com/apache/cordova-ios/blob/master/bin/templates/scripts/cordova/version
Not sure of the alternative.
This references it but can be removed:
https://github.com/apache/cordova-ios/blob/master/bin/create
On Tue, Aug 12,
I think it still needs to exist in an output project ... which is not
(yet?) an npm project, and so does not have a package.json.
The individual platform repos can get rid of it, they will just need to
modify the way they `create` new projects to read the value from
package.json and output it to N
I presume that the alternative would be for the version scripts to parse
package.json instead. I think that Android used to work the same way, but
now its version script just reports a hard-coded number (which coho
updates).
As long as the package.json files are included in the release artifacts,
Thanks for the feedback! I have made the changes on my branch. Here is the
commit with the suggested changes.
https://github.com/stevengill/cordova-js/commit/71113eb61cfba1a354b8bdc61a72b9d73a43c287
On Tue, Aug 12, 2014 at 10:42 AM, Jesse wrote:
> so did I, good otherwise.
>
> @purplecabbage
>
Most of our repos have a package.json. It keeps track of versions. I think
we should work towards removing the VERSION files from the repos we can.
Thoughts?
This would require some changes to coho so it doesn't try to update the
version file when doing releases.
Sergey, this is brilliant!
In case anyone missed it, Sergey just posted this in another thread,
related to hooks and cordova-lib, but I think this deserves some attention
of it's own so I am reposting here.
A JS debugger as a cordova plugin !!
===
... sharing some my experiments with plugin hook
so did I, good otherwise.
@purplecabbage
risingj.com
On Tue, Aug 12, 2014 at 7:58 AM, Andrew Grieve wrote:
> add some nitpick-type comments on your commit, but looks good!
>
>
> On Mon, Aug 11, 2014 at 8:45 PM, Steven Gill
> wrote:
>
> > So I have removed cordova.version and added cordova.pla
Thanks for pointing this out Paul,
looks to involve minor fixes, most of the generated errors are date/time
formatting related
On Tue, Aug 12, 2014 at 8:30 AM, Andrew Grieve wrote:
> Thanks for pointing this out Paul. I've created an issue for it:
> https://issues.apache.org/jira/browse/CB-729
OP Delivers!
On Tue, Aug 12, 2014 at 8:58 AM, Andrew Grieve wrote:
> But since I've piqued your interests now, that's an internal codename for
> Chrome Dev Editor: https://github.com/dart-lang/chromedeveditor
>
>
>
>
> On Tue, Aug 12, 2014 at 11:54 AM, Ray Camden wrote:
>
> > Do you have an ex
Github user rodms10 commented on the pull request:
https://github.com/apache/cordova-plugin-inappbrowser/pull/58#issuecomment-51946902
Thanks for the fix! @zalun should take a look. (may take some time, he's on
vacation).
---
If your project is set up for it, you can reply to this e
Github user rodms10 commented on the pull request:
https://github.com/apache/cordova-firefoxos/pull/5#issuecomment-51946435
This has been fixed in
https://github.com/apache/cordova-firefoxos/commit/20def3b060620e71113d8fa8c6cd671fb6ca2504.
---
If your project is set up for it, you ca
GitHub user stacic opened a pull request:
https://github.com/apache/cordova-plugin-file-transfer/pull/38
CB-6466 created mobile-spec test
Added a test to check that filetransfer.download() automatically creates
any nonexistent directories.
Also removed some unused code
Github user stacic commented on the pull request:
https://github.com/apache/cordova-mobile-spec/pull/58#issuecomment-51940760
Yep, this needs to get added to plugin tests instead. I'll close this one
and add a new PR to file-transfer.
---
If your project is set up for it, you can rep
Github user stacic closed the pull request at:
https://github.com/apache/cordova-mobile-spec/pull/58
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the fea
..and it supports "deploy to mobile" thanks to Chrome Apps flavour of App
Harness. There was an I/O talk here:
https://www.youtube.com/watch?v=NNLnTz6yIc4 complete with demo fail for
mobile :(
On Tue, Aug 12, 2014 at 11:58 AM, Andrew Grieve
wrote:
> But since I've piqued your interests now, t
+1
* Verified archives
* Performed functional testing on Windows
• the app builds successfully
• the app starts successfully (Android 4.4.2 (API19) emulator)
Thx!
Sergey
-Original Message-
From: iclell...@google.com [mailto:iclell...@google.com] On Behalf Of Ian
Clelland
Github user csantanapr commented on the pull request:
https://github.com/apache/cordova-lib/pull/55#issuecomment-51937234
@sgrebnov I agree now that plugin hooks are located in a unique place like
plugins/mypluginid/src I can have my node module dependencies isolated without
impacting
Any page navigations have to include cordova.js, right? What about
injecting the meta tag before deviceready?
On Tue, Aug 12, 2014 at 11:54 AM, Ian Clelland
wrote:
> I love the idea of using CSP for this, especially because it handles things
> that our whitelist intercept code can't, like tag
But since I've piqued your interests now, that's an internal codename for
Chrome Dev Editor: https://github.com/dart-lang/chromedeveditor
On Tue, Aug 12, 2014 at 11:54 AM, Ray Camden wrote:
> Do you have an external URL? :)
>
> From: Andrew Grieve
> S
Sorry - that wasn't meant for the Cordova list :(. It's going one of those
days it seems...
On Tue, Aug 12, 2014 at 11:53 AM, Parashuram Narasimhan (MS OPEN TECH) <
panar...@microsoft.com> wrote:
> http://goto ? The URL does not seem to resolve. Is there an external URL ?
>
> -Original Messa
I love the idea of using CSP for this, especially because it handles things
that our whitelist intercept code can't, like tags and WebSockets.
The problem that I've found with it is that we can only enforce it for
single-page apps. We can inject our own CSP headers into the application's
start pa
Do you have an external URL? :)
From: Andrew Grieve
Sent: Tuesday, August 12, 2014 10:49 AM
To: dev
Subject: Chrome-axiom
New mailinglist that it may be worth joining.
"Chrome Axiom is the Chrome Developer Tooling platform. We're building the
well lit pat
http://goto ? The URL does not seem to resolve. Is there an external URL ?
-Original Message-
From: Andrew Grieve [mailto:agri...@google.com]
Sent: Tuesday, August 12, 2014 8:49 AM
To: dev
Subject: Chrome-axiom
New mailinglist that it may be worth joining.
"Chrome Axiom is the Chrome D
New mailinglist that it may be worth joining.
"Chrome Axiom is the Chrome Developer Tooling platform. We're building the
well lit path for web developers to succeed."
http://goto/chrome-axiom
Had a quick question on the whitelists. I remember that there was talk of using
CSP to fix this issue. A CSP file may not be backward compatible, but could
potentially just give us one list instead of 2 whitelists. The CSP file may be
like the following
Content-Security-Policy:
script-
Thanks for pointing this out Paul. I've created an issue for it:
https://issues.apache.org/jira/browse/CB-7292
On Sat, Aug 9, 2014 at 12:47 PM, Paul Topping wrote:
> Just FYI. Here's output from
> http://feedvalidator.org/check.cgi?url=http%3A%2F%2Fcordova.apache.org%2Frss.xml
>
>
>
> This feed
I've created CB-7291 for the whitelist issue, and I've ported the code from
June to the new-style configuration architecture and committed it to a
named CB-7291 branch on cordova-android.
If anyone has any thoughts/opinions on the syntax or the proposal itself,
or on what the defaults should be fo
add some nitpick-type comments on your commit, but looks good!
On Mon, Aug 11, 2014 at 8:45 PM, Steven Gill wrote:
> So I have removed cordova.version and added cordova.platformVersion on the
> branch cb-7219 on my repo. Check it out at
> https://github.com/stevengill/cordova-js/tree/cb-7219. I
+dev -- replied too quickly
-- Forwarded message --
From: Ian Clelland
Date: Tue, Aug 12, 2014 at 8:32 AM
Subject: Re: Cordova 3.5.1 NOT published ?
To: Axel Nennker
See the "[Vote] Tools Release for Android 3.5.1" thread -- the "cordova"
npm package is part of the tools, and w
GitHub user vladimir-kotikov opened a pull request:
https://github.com/apache/cordova-plugin-contacts/pull/38
CB-7290 Adds support for universal Windows platform.
Fix for [CB-7290](https://issues.apache.org/jira/browse/CB-7290)
You can merge this pull request into a Git repository b
Github user sgrebnov commented on the pull request:
https://github.com/apache/cordova-lib/pull/55#issuecomment-51905210
+ sharing some my experiments with plugin hooks: WP8 JavaScript debugger
implemented as cordova plugin
https://github.com/sgrebnov/cordova-debug-wp8
https://v
Github user sgrebnov commented on the pull request:
https://github.com/apache/cordova-lib/pull/55#issuecomment-51904349
Answering Carlos questions
1. I believe recommended solution here is placing required dependencies
along with plugin (node_modules folder inside your scripts fol
Github user sgrebnov commented on the pull request:
https://github.com/apache/cordova-lib/pull/55#issuecomment-51903709
Thx for review and notes. +1000 to wrap parameters to special class
instance, but I would prefer to push this version since it takes time to
re-base it all over the
> Looks like messages about missing dependencies not very friendly.
cordova 3.5 contains duplicated dependency check for ubuntu platform.
(one of them with unclear message.).
issue already fixed in trunk
https://github.com/apache/cordova-lib/commit/153092f2c235185c7f141979a550779800954a92
---Maxi
There some random guy have a problems adding Ubuntu as a platform on SO:
http://stackoverflow.com/questions/25259980/adding-platform-in-apache-cordova-does-not-work
Looks like messages about missing dependencies not very friendly.
- Andrey
Hi,
when I do a "sudo npm install -g cordova" I still get 3.5.0-0.2.6
cheers
Axel
ignisvulpis@ubuntu:~$ npm view cordova
{ name: 'cordova',
description: 'Cordova command line interface tool',
'dist-tags': { latest: '3.5.0-0.2.6', rc: '3.5.0-0.2.4' },
versions:
[ '0.0.0-fake',
'0.
Github user sgrebnov commented on the pull request:
https://github.com/apache/cordova-plugin-file/pull/54#issuecomment-51885991
@Bosman, thx for sharing this trick!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your
Github user purplecabbage commented on the pull request:
https://github.com/apache/cordova-plugin-file/pull/54#issuecomment-51885637
Thanks for the github lesson @BBosman, extremely useful! I will use this
lots.
---
If your project is set up for it, you can reply to this email and ha
Github user BBosman commented on the pull request:
https://github.com/apache/cordova-plugin-file/pull/54#issuecomment-51883097
You can see a commit diff ignoring whitespace changes by appending `?w=1`
to the querystring.
So for this pull request's commit that would be:
https:
89 matches
Mail list logo
