These are chat archives for bluefin-octopusdeploy/chrome-extension

16th
May 2016
Matt Richardson
@matt-richardson
May 16 2016 05:46
@davidroberts63 - just saw you merged.
I've just realised there's a bug in it
David Roberts
@davidroberts63
May 16 2016 05:47
Okay. Good timing. Was about to publish it in about 15 minutes.
Matt Richardson
@matt-richardson
May 16 2016 05:47
Where it doesn't handle the async responses properly, so the scoping doesn't always show up
Phew 😀
David Roberts
@davidroberts63
May 16 2016 05:48
Okay, I'll hold off on it for now. Async is hard, especially in JavaScript. At least for me anyway.
Matt Richardson
@matt-richardson
May 16 2016 05:49
Aye.
I'll probably have a fix available in about 15, but I am aware it's the middle of the night for you, so don't wait up on my account.
David Roberts
@davidroberts63
May 16 2016 05:50
No problem. Laundry in the dryer. I'll be up for at least an hour.
Once this is done a published I'm going to git push the code to the bluefin-octopusdeploy/chrome-extension repository. I'll then point the build and the website link to the new repository. So we should both start using the new one for pull requests and issues. I'll copy over the old issues too.
Matt Richardson
@matt-richardson
May 16 2016 05:53
You should just be able to move the repo
(Unless you want to keep the old one)
David Roberts
@davidroberts63
May 16 2016 05:54
Really?.....O.O look there is a 'Transfer' button.
Oh this will make it so much easier and keep all the history plus PR and issues.
Glad I mentioned it then. :-)
Matt Richardson
@matt-richardson
May 16 2016 06:01
:)
github adds redirects as well
David Roberts
@davidroberts63
May 16 2016 06:03
I noticed that. Reading up on it to see what all it does. It does a lot.
Yeah, will definitely just transfer the repository. But only after this publish. I know it redirects but I can wait a bit.
Matt Richardson
@matt-richardson
May 16 2016 06:14
okay, looks harder to fix than I thought
Matt Richardson
@matt-richardson
May 16 2016 06:24
Will take a look while on the train
In half an hour or so
David Roberts
@davidroberts63
May 16 2016 06:26
'While on the train'. I envy people living in places with real usable public transportation systems.
Matt Richardson
@matt-richardson
May 16 2016 07:11
Heh.
Bet you don't envy the average house price here though.
Nor the
244 USD I spent on my weekly train ticket.
But I must admit the trains in and around London are pretty good
David Roberts
@davidroberts63
May 16 2016 07:35
$244 USD a week? Yeah, okay I'm good with my 20 minute commute in my Toyota.
Yeah, average house prices in my area (Oklahoma) are very inexpensive compared to a lot of the hot spots.
Matt Richardson
@matt-richardson
May 16 2016 07:36
okay, it is a first class ticket, but still.
and its cheaper if you get monthly etc
okay, pull request raised
Matt Richardson
@matt-richardson
May 16 2016 07:52
And another one, cos I could :)
Matt Richardson
@matt-richardson
May 16 2016 11:11
Whats your thoughts about JsHint?
Worth adding to the build?
Matt Richardson
@matt-richardson
May 16 2016 11:21
potentially even adding to coveralls.io - looks like there is support for jasmine tests (via a somwhat convulted route - https://github.com/nickmerwin/node-coveralls#istanbul)