Not pipes, but stuff like `&&` or `;` in bash/zsh looks like this in fish: `command; and echo "Success"; or echo "Fail!"`. Not bad, but incompatible with other shells. Still, scripting small bits is way more sane. Thats what I like about Fish. :)

Command interpolation:
```
$ echo This is (uname).
This is Linux.
```
or loops:
```
$ for file in *.yml
                              wc -l $file
                          end|sort -r
348 the-twelve-factor-app-best-practices-for-jruby-deployment.yml
344 how-you-can-fix-your-community-in-one-day.yml
342 jruby-file-io-using-mmap.yml
323 scaling-ruby-applications-with-mesos-and-docker.yml
285 quiet-and-versatile-rubyist-using-the-tools-you-ve-got.yml
279 get-your-shoes-back-on.yml
228 best-of-both-worlds-building-a-mirror-trading-platform-on-top-of-the-java-ruby-ecosystem.yml
224 mirah-where-java-meets-ruby.yml
217 a-case-against-showmanship.yml
140 jruby-with-java-code-in-data-processing-world.yml
0 will-the-real-technical-people-please-stand-up.yml
0 the-struggle-to-stay-technical.yml
0 the-power-of-small-abstractions.yml
0 the-junior-jump.yml
0 programming-parents.yml
0 programming-as-a-performance.yml
0 passing-down-the-pain-difficulties-in-teaching-software-development.yml
0 orders-of-magnitude.yml
0 opal-the-journey-from-javascript-to-ruby.yml
0 mruby-a-packaging-story-filled-with-freedom.yml
0 moving-the-web-into-the-physical-world-with-beacons.yml
0 keynote.yml
0 jruby-gradle-bringing-java-power-tools-to-ruby.yml
0 jruby-9000-is-here.yml
0 humanising-math-and-physics-on-computer-science.yml
0 how-teaching-kids-to-code-made-me-a-better-developer.yml
0 game-programming-in-haskell.yml
0 deep-diving-how-to-explore-a-new-code-base.yml
0 cultivating-empathy.yml
0 crystal-the-programming-language.yml
0 calling-native-code-from-ruby.yml
0 ask-vs-guess-culture-communication.yml
0 amelia-bedelia-learns-to-code.yml

```


You can remove Jacob McSwain his comments from here, its totally bullshit. he didnt have to change anything to make it work. He only stole it from our private repo's and made the bliss copy/paste kids happy with it. I already made it work for cm based roms lol. The second commit for the airplanemode, data and wifi button is not needed. just include hyde his original buttons in this commit and it will work.

Cheers
BlackDragon


Yes, `.ToString()` was better than `.Message`, but I suspect sending the entire exception will give the same behavior.
So I think we should try to go with the official solution.


fu~


Yes, I agree, this looks like a typo made by Vinai. Why haven't you created a pull request?


Upper-cased country code `zh-TW` will not take effect my computer.


Oh, and this one we want to keep.


Один коммит охуенней другого просто))


It also looks like you are trying to imply which map to look apply the operation on by its existence in either map. This is not clear from the code and only those with intimate domain knowledge understand why this happens to work.

It would be better to store whether it is a buy or a sell order on the Order and check that.


It works for this demo, but it's not the optimal solution. I had something in mind more similar to how CML multicast streams work in the Reppy book:

![dsc_0389](https://cloud.githubusercontent.com/assets/193115/9653008/c3497eca-5220-11e5-9c78-2287038cfe57.JPG)

Where read values from the stream disappear by GC. Is there a similar facility in Hopac?


fixed in b20853d


This error should be captured, but it should be within a closer scope


Actually, remove this function entirely. You don't need it.


Leave a comment on commit_id6f75b00


Looks like support for environment variables was added: https://github.com/docker/compose/pull/845


Does this condition need a correction?
Which aspect do you try to express with a double negation here?


Composer command form 0.3 is better.


hum, what happen if the command return `latin1` encoded output? I guess it won't work.
What we could do instead is to add an `encoding` configuration option to every hooked config sections so that the user can configure that the command output must be decoded as `utf8`, `latin1`, or other encoding.
The default config value for `encoding` could be `utf8`.


AddStuffToUI (kbenne) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg)


On line 43:
	auto it = m_buyOrdersById.find(order->m_orderId);
	assert (it != m_buyOrdersById.end())


Apply same logic to the Sell stuff


I think you should return a NotFoundException.


93651760-RowPerObjectReport (JasonGlazer) - Win64-Windows-7-VisualStudio-12: OK (1815 of 1815 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


Remove


Cool, I think understand now. Thanks for explaining :-)


Yes, I use it for `url_for` calls


much cleaner +1


That's a nice feature!


Yep :yum:
`root@d802:/ # sysctl net.ipv4.tcp_available_congestion_control
`net.ipv4.tcp_available_congestion_control = veno cubic westwood reno bic`


I think you should not return Optional in these methods :
- If the file does not exists -> NotFoundException
- If the line does not exists -> Empty Iterable


Fixed.  I did test it, but the code changed after I enabled the plugin for testing, and the bug only manifested itself during enabling.


This method checks, over all the answers, if the final answer is Y-Y. If this is the case true is returned, otherwise false.
To analyze the answers I need first to summarize/aggregate them and then check if the number of yes yes answers are bigger than the yes-no or no answers.


Btw you can read my comment for this here: https://github.com/TeamBliss-LP/android_frameworks_base/commit/dc889169c85e1e7556faa6d05f775b3b9ed39355


that's a typo


I don't know how exacly the called functions work.... so just a stupid question: If any of the `files`, `first` should become *nullptr* will the application survive?


Sorry I forgot the date, I 'm going to fix this now.


No need for ```return``` here


You declared auto yet set the value to 0. This will cause totalPrice to be cast to an int. Pretty sure that's not what you want.


you used auto in all places except here. Why? IMHO, you should just name the type unless it is complex like a std::reverse_iterator<std::string::iterator> that you're getting from some algorithm and you'd have to spend 10 minutes looking up how to syntaxually set that up, AND the type doesn't matter. Here, it is useful.


Fixed.


i can see the item group, however i'm not able to search by it


Thanks for fix :thumbsup:


Just wanted to point out that in my experience, strict mode can't ever introduce any tricky issues. It errors out early when the code is screwed up, but never changes semantics in a silent way. It also potentially may help performance, at least in future (V8 will feel safer to do more optimizations at runtime).

All libraries I have written for the past few years have `use strict` and I never had any problem because of it, including browserified bundles.


1. You should not assert here.
2. You should be explicit when testing if a value is greater than 0.


Checkmarx scan results 03 Sep 2015 11:02 UTC +03:00

High
Medium
Low
Info

New
0
0
0
0

Recurrent
3
3
2
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

066f603e097afc8f97b178984ff8475794e758a7	@cxehud


Was this linked to the old days when food and minerals were stored somewhere for distribution?


I suppose this could work. Thanks for the tip :)


Should be good to rewrite these tests one day...


Test


This has nothing to do with modularization, it's part of an upcoming iTunes-like image blurring thinga.


@NL_BlackDragon, you know comments like this is what pissed us off last time, coming on our repo and saying things that are just not true and trying to belittle us for the sake of making your self sound better. "WE" did copy and paste anything, for shit sake you have a private repo that our core team has no access to. It was pushed to our gerrit. Now did Jacob copy and paste, I don't know. There is nothing wrong with manual editing of files as long as authorship is maintained. Correct? And really why did you have to go private in the first place????

Yes I have made mistakes our our repos, but I corrected them. I don't claim to be a "REAL" dev as you so bluntly put it, but I feel as though me and Team Bliss can put together a pretty decent rom.

So instead of coming hear and trying to blast on our repo, take the issue up at the source.


I have no idea about that, maybe yes. This seems correct though :)


Will this work on cabled Ethernet devices? Frame larger then 1500 should be supported by gigabit interfaces (jumboframes) but I don't know if it is really well supported by hardware/driver we use, some real tests would be helpful.


meh.. sure. Generally this is frowned upon as it lacks any sort of semantic context. What you're actually doing is iterating until

1. You reach 0 in the targetSize
2. Run out of orders (should never happen though) However, this while true will continue to run forever until you break, so if anything does go wrong you have the potential to crash (would a bug somewhere, but bounding this loop properly would avoid the possibility of a bug)


(gentle ping) @addyosmani


see feedback for priceToSellShares


I think the description should more explicitly distinguish this setting from excludeServers.


You have typedefs for your shared_ptrs but not for much more complex types such as these. Why?


name and tags attributes?


@Soullivaneuh Done.


Fix Add more toast animations
@@ -405,50 +405,58 @@ public void handleShow() {
                 // the layout direction
                 final Configuration config = mView.getContext().getResources().getConfiguration();
                 final int gravity = Gravity.getAbsoluteGravity(mGravity, config.getLayoutDirection());
-            switch(Settings.System.getInt(context.getContentResolver(), Settings.System.TOAST_ANIMATION, 1)) {
+
+                switch(Settings.System.getInt(context.getContentResolver(), Settings.System.TOAST_ANIMATION, 1)) {
                 case 0:
-                mParams.windowAnimations = -1;
-                break;
+                        mParams.windowAnimations = -1;
+                        break;
                 case 1:
-                mParams.windowAnimations = com.android.internal.R.style.Animation_Toast;
-                break;
+                        mParams.windowAnimations = com.android.internal.R.style.Animation_Toast;
+                        break;
                 case 2:
-                mParams.windowAnimations = com.android.internal.R.style.Animation_Toast_Fade;
-                break;
+                        mParams.windowAnimations = com.android.internal.R.style.Animation_Toast_Fade;
+                        break;
                 case 3:
-                mParams.windowAnimations = com.android.internal.R.style.Animation_Toast_SlideRight;
-                break;
+                        mParams.windowAnimations = com.android.internal.R.style.Animation_Toast_SlideRight;
+                        break;
                 case 4:
-                mParams.windowAnimations = com.android.internal.R.style.Animation_Toast_SlideLeft;
-                break;
+                        mParams.windowAnimations = com.android.internal.R.style.Animation_Toast_SlideLeft;
+                        break;
                 case 5:
-                mParams.windowAnimations = com.android.internal.R.style.Animation_Toast_Xylon;
-                break;
+                        mParams.windowAnimations = com.android.internal.R.style.Animation_Toast_Xylon;
+                        break;
                 case 6:
-                mParams.windowAnimations = com.android.internal.R.style.Animation_Toast_Toko;
-                break;
+                        mParams.windowAnimations = com.android.internal.R.style.Animation_Toast_Toko;
+                        break;
                 case 7:
-                mParams.windowAnimations = com.android.internal.R.style.Animation_Toast_Tn;
-                break;
+                        mParams.windowAnimations = com.android.internal.R.style.Animation_Toast_Tn;
+                        break;
                 case 8:
-                mParams.windowAnimations = com.android.internal.R.style.Animation_Toast_Honami;
-                break;
+                        mParams.windowAnimations = com.android.internal.R.style.Animation_Toast_Honami;
+                        break;
                 case 9:
-                mParams.windowAnimations = com.android.internal.R.style.Animation_Toast_FastFade;
-                break;
+                        mParams.windowAnimations = com.android.internal.R.style.Animation_Toast_FastFade;
+                        break;
                 case 10:
-                mParams.windowAnimations = com.android.internal.R.style.Animation_Toast_GrowFade;
-                break;
+                        mParams.windowAnimations = com.android.internal.R.style.Animation_Toast_GrowFade;
+                        break;
                 case 11:
-                mParams.windowAnimations = com.android.internal.R.style.Animation_Toast_GrowFadeCenter;
-                break;
+                        mParams.windowAnimations = com.android.internal.R.style.Animation_Toast_GrowFadeCenter;
+                        break;
                 case 12:
-                mParams.windowAnimations = com.android.internal.R.style.Animation_Toast_GrowFadeBottom;
-                break;
+                        mParams.windowAnimations = com.android.internal.R.style.Animation_Toast_GrowFadeBottom;
+                        break;
                 case 13:
-                mParams.windowAnimations = com.android.internal.R.style.Animation_Toast_Translucent;
-                break;
+                        mParams.windowAnimations = com.android.internal.R.style.Animation_Toast_Translucent;
+                        break;
+                case 14:
+                        mParams.windowAnimations = com.android.internal.R.style.Animation_Toast_SlideLeftRight;
+                        break;
+                case 15:
+                        mParams.windowAnimations = com.android.internal.R.style.Animation_Toast_SlideRightLeft;
+                        break;
                 }
+
                 mParams.gravity = gravity;
                 if ((gravity & Gravity.HORIZONTAL_GRAVITY_MASK) == Gravity.FILL_HORIZONTAL) {
                     mParams.horizontalWeight = 1.0f;


We found an issue with this specific commit. The problem is that when running snpEff for example, the absolute_filepath is as follows: /mnt/galaxy/tmp/tmp-toolshed-mtdN93bg4/snpEff/snpEff/' but should be: /mnt/galaxy/tmp/tmp-toolshed-mtdN93bg4/snpEff. The chmod that's done later errors out as follows:

```
File "/mnt/galaxy/galaxy-app/lib/tool_shed/galaxy_install/tool_dependencies/recipe/step_handler.py", line 89, in extract
    os.chmod( absolute_filepath, unix_permissions )

[Errno 2] No such file or directory: '/mnt/galaxy/tmp/tmp-toolshed-mtdN93bg4/snpEff/snpEff/'
```
Reverting this commit solves the problem.


This takes care that MeV units will be used for the bg rates per default. There is though a flag to avoid this if desired (turned off per default).


The context menu is shown only when the user clicks on files/folders; otherwise another menu is shown, which is dealt with in another file. So, there's no possibility for a null pointer. I'm using this commit for a week now and have encountered no problem so far.

BTW, my next commit fixes a rare, random crash in addition to its main purpose; I forgot to mention that in its message.


This one breaks rst, since the length of the underline does not match the headline.


Previously the behaviour was to show players which have not yet been connected to any server.


Please use an other identifier for such a jump label so that the potential for confusion with [a specific function](http://pubs.opengroup.org/onlinepubs/9699919799/functions/_exit.html "Terminate a process.") will be avoided.


A nice way to make easy with callbacks in repl. (see below)


Forget it, I didn't see that it's when there's no source code that you're returning an Optional.absent(), which make sense


is this really needed?
i would say that there comes ```0``` as default


Dunno. I thought it might have been intended for calculating supply / resource sharing group stuff.


91958608DOASEffectOnZoneSizing (wfbuhl) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (763 of 763 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-10.33%25-green.svg)


yeah, if you could put this into `preStart` at least. If it throws here we're in deep do do.


1. Why is this here and not in the "default" scope.
2. The semantic meaning of this is invalid. True means we reached it, false means we did not AND an error.

You should change this to return an int.
- return 1 // success
- return 0 // failure
- return -1 // error


@mdo is there an explanation on this delete ? Thanks


i can see the item group, however i'm not able to search by it


I haven't read your logic yet but I think there is a world that exists where this function does not. But maybe it will make sense when I read the code


ok, point taken, it's fundamentally broken and needs to move over to use the `SourceResolver`


It's needed for edge cases such as exceptions:

![exception_priority](https://cloud.githubusercontent.com/assets/73419/9653582/a4c3cb82-5224-11e5-9098-eb8aeef63a89.png)


That actually looks pretty nice :+1:


Ist das so sinnvoll?!


Yeah, that one of the bugs i was wondering nobody discovered before :wink:


@ph0b strange. The compiler shouldn't reorder these.


I pushed fixes for this. It seems to be working properly still.


Ок, буду знать. Да, див самое то.


Thank you , be safe, hope a better future


Неа, не угадал. Давай, перечитай мой прошлый коммент, пойми как оно работает и как сделать лучше.


この辺インデントがズレてるね


looks good to me.


Так. Почему в коммите про добавление liquibase/h2 внезапно обновился Spring?!


Agree: fixed in next push


I see you have dropped the last digit of the version, was this intended or just a typo?


Why do you first hide the element if at the end you remove the entire parent?


I don't think I follow. I'm interested in pictures and pictures are png. Why would I want to remove the tools which make the pngs?


Probably yes: I'll check what `@@` is at this point :-)


哈哈不要大白了 :smile:


I think I don't understand. Such a method at this point doesn't make sense to me. We have all the answers of a snippet and we want to know if we have a majority of Y-Y over all other cases.
For this reason the method "answerIsYes" checks over all the collected answers, after aggregating them, if the number of yes-yes is higher than the other cases. If this is the case then true is returned otherwise false.
Please correct me if I'm wrong :)


const if you're going to do this. tempOrderAction will not be modified (and shouldn't be)


Бесполезный комментарий.


csvとか読み込んで処理するやつ
⇒消えたｗ


Ты их так слепил, что я даже не сразу понял где ключ, а где значение.

Лучше добавь пробелов, не скупись:
```
driver: org.h2.Driver
```


hello world


By now you should feel how silly it is to duplicate the same amount of code for a type that differs by a single character or so.

There is a way to avoid this code duplication, but you can worry about that when you finish.


У тебя в properties-файле указана БД `db1`, желательно ее здесь явно указать, т.к. помнится спринг по умолчанию генерит это название сам.


It is the environment.
Empty when used with spring.
I named it to make it more clear.


This is a good idea :) I will try to implement this today


Lol you know why we (Cyanide and Fusion) went private. dont act like you dont know. A while back when we did not have private repo you also stole stuff from us. remember....
Besides that, i have nothing more to say to you about this. other devs also know about you kids copy/paste work. And yes the source issue Jacob McSwain has been removed from our organisation so that is taken care off. And to your response on "Team Bliss can put together a pretty decent rom" Other devs did the work you mean, you guys just copy/pasted everything from them. it is a miracle everything still works decent. but that is thanks to Tobias (Thats a real dev) he fix all of your bullshit codes that you pulled and fucked up. Learn from him if he continues to help you guys.

Enough said, happy copy/pasting dude


pyROOT, I suppose.
A occhio manca un `import ROOT` all'inizio.
Bel lavoro, comunque.


Не советую использовать импорт со звездочкой.


It's "canola" and not "canols"


Note: scrapped is needed? maybe removed after


in libUDB/libUDB.c
-6ty// TODO: do something about these declarations being here - RobD
should be
// TODO: do something about these declarations being here - RobD

have not yet found when this happened..


Ты забыл про ключевое слово `private`. И вынеси аннотацию на отдельную строчку выше.


How to enter Swype Settings now?
Only through the App drawer then?
Maybe the best way, to use the Swype Key's long-press, if it is not possible to use the Dragon Key's long-press, which would be much better I think, only if possible.

When will this release? Its not in the Xposed Download updates yet :(


Known issues:
-> flash message is shown 'x' times in form, where 'x' equals number of refreshed available bikes
-> public function updateAvailableBikesAction() code is duplicated (see execute function in UpdateAvailableBikesFromProviderCommand class)


Checkmarx scan results 03 Sep 2015 11:28 UTC +03:00

High
Medium
Low
Info

New
0
0
1
0

Recurrent
1
1
1
0

Fixed
1
1
0
0

 Open Viewer
Included commits

Commit ID	Owner

066f603e097afc8f97b178984ff8475794e758a7	@cxehud


Старая какая-то версия. Обнови с 2.0 до 3.0 хотя бы.


When ScaleIO is configured with SingleMDM, the API Methos from ViPR is not connecting / Discovering the system !!!!


I agree, though I'm not sure `stderr` is best as in a daemon situation it's likely `/dev/null`. Fortunately, the old log file is still open (and we're going to continue to use it for logging) so I can send the error there. Fixed in next push.


modified


There is to test that covers this case, but the second method does not cover the case where otherNumber is lower than the number, which is a requirement from the header file.

> Either @c number or @c otherNumber may be the lower number, but the string always includes numbers from lowest to highest.

This is essentially the same task as AwesomeCounter in Loops chapter and can be done like this:

```objective-c
- (NSArray *) arrayOfNumbersBetweenNumber:(NSInteger)number andOtherNumber: (NSInteger)otherNumber {
	/* WORK HERE */
	NSMutableArray *arr = [NSMutableArray array];
	NSInteger low = (number < otherNumber) ? number : otherNumber;
	NSInteger high = (number > otherNumber) ? number : otherNumber;

	for (NSInteger i=low; i<=high; i++) {
		[arr addObject:@(i)];
	}

	return arr;
}
```


Can you please get rid of this commit?


Verry prety STUPIDs little boy & little girl  :p


@DiscanX Just a small remark: s + pawn_push(Us) returns a square, not a Bitboard.


As this 'When I select to add a property to the case' step is used on both scenarios then it can be added to the background section just to clean up the individual scenarios a bit.


It works


Is it supposed to be mutated later ? if not, use ImmutableMap.of()


Измени `id` на что-то более конкретное. id должен быть уникальным, а этот id настолько общий, что когда ты создашь еще одну таблицу, то как ты будешь называть следующий id?

Зачем тут `runOnChange`? Я не знаю зачем, никогда не использовал и не понимаю зачем он тебе здесь понадобился. Убеди меня, что он нужен или выкинь :)


I noticed that for many themes the cache is missing (Oxygen, Tango, Breeze*).

Am I correct to assume that other themes than `hicolor` and `HighContrast` will only come from a single package so we should best just distribute them with the caches?

Another option might be to regenerate them after buildenv like we do for `hicolor` (in system env at least). BTW there isn't a reason why we don't regenerate `HighContrast` and it should be added, right?


I'm troubled by changelog so i didn't bother reading the patch.

> himem   : 0xfc000000 - 0x100000000   (  64 MB)

How is this ever going to work. 0xFC00_0000 is strictly peripheral address space, so memory of any sorts can NOT be hooked up here


Are you sure of this ? Because if my understanding is correct a project view can never be a root.


so this is a patch off the series, and I can apply it to for-next rightaway !


Reopened https://bugzilla.redhat.com/show_bug.cgi?id=767998


No, I think that is unnecessary. `logrotate` rotates before compression and normally keeps one compressed; even if you put geodns aside, it *must* do that (think about it), not least as it has no idea *when* a `SIGHUP` would have been responded to - how long would it have to wait before compressing the file if it did it the other way? Given `SIGHUP` itself has no 'acknowledgement' that the process is now logging to a new file, I don't see any harm in the code. I could add `SIGHUP` handling too but it seems like unnecessary complexity (oh, and some poor person might be running this on Windows).


Wena


Would this make sense?
`assert_equal 'mah=body', resp.env.request_body`


Зачем тут стиль скриптом устанавливается?


Не уверен, что здесь нужно явно указывать `nullable="false"`, т.к. первичный ключ не может быть nullable.


I think I understand now. I am trying to do something similar. However, I need a transition every 9+(4123/6000)  cycles. I was under the impression I would then need fraction<=(fraction&5999)+4123. The 5999 comes from 6000-1, as I thought you were doing 1024-1. Now my guess is I actually need to do this:  fraction<=(fraction-5999)+4123. Correct? I guess you were lucky that you got 1024 in the fraction.


if a projectView is a root, the code really should fail, loud and strong


You'll need to add the wtforms module to requirements.txt and run ./update_vendor.sh in the project root.


Third solution is fine example where KVC operators shine:

```objective-c
NSNumber *minNum = [arrayOfNumbers valueForKeyPath:@"@min.integerValue"];
return [minNum integerValue];
```

This `valueForKeyPath` will enumerate the array and find the minimum integerValue.


I'm pretty sure we want to be able to edit the rating on the review itself. The bug is about being able to rate when doing an edit of a reply to an existing review.


Needs one more space for indentation (sorry for being so picky :-) )


Почему `name` и `age` не объявления как nullable?


Unfortunately closing reporterHelperClassLoader leads to build failure when generating schema. I'll need to find a way to work around the issue (e.g. make ReporterHelper expose releaseResources() method or sth similar).

[ERROR] /.../radargun/target/antrun/build-main.xml:20: java.lang.NoClassDefFoundError: freemarker/template/TemplateModelException
[ERROR] around Ant part ...<ac:for xmlns:ac="antlib:net.sf.antcontrib" param="reporter" list="reporter-default,reporter-perfrepo">... @ 14:106 in /.../radargun/target/antrun/build-main.xml: freemarker.template.TemplateModelException


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/20040/testReport/) for this commit**


This breaks my code and I'm not sure how to fix it:
```Julia
abstract FixedArray{T, NDim, SIZE}
typealias FixedMatrix{Row, Column, T} FixedArray{T, 2, Tuple{Row, Column}}
immutable Mat{Row, Column, T} <: FixedMatrix{Row, Column, T}
    _::NTuple{Column, NTuple{Row, T}}
end
Mat(((1,2,3),)) # ERROR: TypeError: Tuple: in parameter, expected Type{T}, got Int64
```
```Julia
abstract FixedArray{T, NDim, SIZE}
typealias FixedMatrix{Row, Column, T} FixedArray{T, 2, (Row, Column)}
ERROR: TypeError: FixedArray: in parameter, expected Type{T}, got Tuple{TypeVar,TypeVar}
```


I've added some comments in the next push. Not sure it really deserves a test of its own as `chan chan error` is the standard technique described in the go synchronisation tutorial!


Wouldn't it be better to define the federate value like this?
```
#define XYZ_RATE 50*60
#define MANUAL_FEEDRATE {XYZ_RATE, XYZ_RATE, XYZ_RATE}  // For Delta Configurations, all axes use the same feedrate
```
where you can substitute some name of your choice for the XYZ_RATE above


Эм. Что кто-то ищет бин `springLiquibase`?


I thought so thus didn't proceed further and didn't commit to my main branch.  Thanks for the heads up.


IMO ```path.parent()```should always be preceded by the condition ``ìsRoot()```. Now you're talking about the error handling. Maybe you're right, it should fail in this case.


This code would be simpler if the `xkeyboard_country_code` table used the countries as key instead of values.

Honest question (I'm not sure!): Does it make sense to change `local xkeyboard_country_code = { "de", "fr", ... }` to `local xkeyboard_country_code = { de = true, fr = true, ... }` so that this code here just becomes `if not xkeyboard_country_code[file] then break end`?

And while I am thinking out loud:
Should the local table `xkeyboard_country_code` be moved to a "global" table as `keyboardlayout.xkeyboard_country_code` so that people can add/remove entries if they really want to?


I don’t think Linking the whole `plug` spec to `String` Is a good choice, as it’s is inconsistent with keyword highlighting in Vim script files. `plug` should be treated as a command keyword, the rest of the spec as command arguments.


`RegformsBase`? What about `RegFormBase`?


This seems like a random location to force units to a default.
Shouldn't this happen at the time of object construction or read / write?


Ben, if you remove control statement, its former body should not be indented any more and shifted back to the main script level (where closing php tag is).


Перед словом pressed поставьте пробел.


Wonderful! Once you're done with this, we'll start the **level 2** :dancer:


Yes.
Taking into account that we have a configuration file available for this command, I assume it has the definitions we use in our project?
Bu I do not always run it to avoid having my name all over the "git blame" output.


Seem to have a duplicate assertion here for '42A Broad Street' can probably remove one of these.


I recommend to [complete the error detection and exception handling](https://github.com/Shardcached/shardcached/issues/5 "Completion of error handling") here.


This plural -> singular inheritance thing is very confusing!


93651760-RowPerObjectReport (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1815 of 1815 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


it should be plural for sure since it's the base for anything in the registration module related. `RHManageRegFormBase` would be the base class that handles a specific registration form object


mm/memory.c still exports this and there is atleast one user in kernel: mm/cma so i think we need to retain this !

So NACK


Could probably stay as it is for now, but I think we have to make curly braces compulsory in the future, especially for cases when if/else is used, otherwise it is difficult to read.


I'd add a linebreak before `.filter_by` so the first line points to the query object with each following line containing one element in the call chain


I just tried `Src.tap src` in `subscribe` but that doesn't throw away the data in Src even when there are no `Stream<_>` references.


`'success'`


how this could be possible to not have this value initialized in constructor?


s/the Content Security Policy specification/`<cite>Content Security Policy</cite>`/


There is one problem, if it fails, you can't reproduce it (ie, `x` is not related to the tested input).

Perhaps let it generate a random date time, and extract the year,month, and day from that?


Ok


Should we rethrow the actual remote error or the generic `RemoteException` in this case? If the intention is to behave similar to `map` it should be the underlying remote exception.


Usually unserialization. Metadata is serialized and cached.


потому что мне нужно только в этом случае выровнять по центру весь <main>, который описан в родительском темплейте.  любые предложения по улучшению кода приветствуются


agreed with @gab1one. we should take out too generic formats.


We try to use `test` instead of `[ ... ]`. I know that there are some instances of `[ ... ]` elsewhere but let's try not to add more, especially here as this is between two instances of `test`.


The code related to 'Action' is in [here](https://github.com/codecombat/codecombat/blob/master/app/templates/play/level/tome/spell_palette_entry_popover.jade#L18).


what fix are you talking about ????

https://github.com/crdroidandroid/android_frameworks_base/commit/3bb4dfae1659048a1f98669d803cc1bbfafb43a5


101884090--NewVenting-Availability-Schedule (lgu1234) - i386-Windows-7-VisualStudio-12: OK (1814 of 1814 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


Вы используете get-методы для доступа к данным полям, следовательно модификаторы доступа логичнее было бы сделать private.


И тут


Done.


93651760-RowPerObjectReport (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg)


Exactly @ThiefMaster. That is the way we do it in the `Surveys` module.


1. Привести поле sum к int и отобразить в итоге как текст в списке (TIP: Integer.toString(i), i - int).


The code is replaced by later patch:
-        self.client.get_connection_by_uuid(dev_cfg.con_uuid).delete()
+        dev_cfg.con.delete()
but there we assume dev_cfg.con existing as well.
The code (callback on [-] button) is called only on items of the (device/connection) list that always have con set (virtual devices - vlan, bond, bridge, team) - we are actually removing the connection from the list in these cases. But adding the check on the attribute wouldn't hurt here, so I'll do it.


I agree, we can just return after a single call to cmp_handler. Fixed.


np mate, btw nice work on those clock size and font stuff


Another thing to consider for the future (leave it for now), commit messages policy. They look nicer if the length of the title is not more than 72 characters. If more details needed, they may be added to the commit body. This one I would rewrite as: "Bugfix: Fix customised text load issue in email message.", then in the body would state something liken "Customised text in experiment enrolment confirmation email was not loaded correctly, this patch resolves the issue.".


Add missing


I rather move `kubernetes_version` to group_vars or common/defaults to avoid mixing up different releases by accident.


then please do a patch and send the PR ..... is your fix man .... thanks


Is closing </center> tag missing? Or it is appended in some different place not covered by this diff?


What was the reason for the revert? It's been working for us but really sorry if it's broken something else.


Ah, one thing I have to ask quickly.

In the original `keyboardlayout.lua`, while there's vim mode line which sets `shiftwidth=4 softtabstop=4`, the code itself has been formatted as if `shiftwidth=3 softtabstop=3`.   Following the instruction of `docs/02-contributing.md`,  I've kept 3 space indentation.   How should I do with it?   Since I'm using Vim, I'd be happy if you permit me to reformat the whole code with 4 space indentation.


Если возникает такая ситуация, то нужно подумать, а правильно ли ты выделил рабочую зону из основного шаблона?
Такой подход сильно усложняет дальнейшую поддержку.


I gave you the fixes


AFAIK the cache is optional. On NixOS we regenerate it but it's hard to handle this kind of stateful cache on nix you know. Oxygen, tango ecc. should work fine without the cache.


XYZ


94006714-MultipleTowers (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1818 of 1818 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


develop (macumber) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2296 of 2332 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-98.46%25-yellow.svg)


I believe the old rule (x\\/-y)/\\y  =  x/\\y was correct, and am quite certain that the new rule (x\\/-y)/\\y  =  y-x is not.
Maybe (x\\/-y)/\\x  =  x-y holds, is this the rule you intended?


ABC


Should all of the locals be called source instead of notifier?


AVC


Me acabas de descubrir el de la [RAI](http://diccionario.raing.es/). Aunque no me trae `interpolar`, por ejemplo. Creo que `particionar` es lo más usado, pero así nos ahorramos la posibilidad de equivocarnos.


Prefer `/`


if I add them the FIX COMMIT will not be displayed with your name


Fixed this in https://github.com/tambetm/DeepMind-Atari-Deep-Q-Learner/commit/ca6b4025de57ec3c9d08ff1164e66be60ac4c74b. Creating pull request for single commit is too much hassle, so you can merge this manually. Or include it in that fancy Makefile, which really is better solution.


As stated earlier, curly braces in control statements would make this easier to follow, we need them in coding style rules. No need to change anything for now, unless you really want.


Yes, I've already pylinted and fixed it, it seems that the PR shows some slightly older version (only missing minor fixups like this) of the branch - hence the strange order of commits I was talking about before.


Would `flat_map` be a good fit here?


Will do


</center> is already there in the file, the opening tag was missing.


take a look at this code:  https://github.com/cwi-swat/rascal/blob/master/src/org/rascalmpl/library/cobra/RandomValueTypeVisitor.java#L214

That is how the current random date-times are generated.


91958608DOASEffectOnZoneSizing (wfbuhl) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1292 of 1292 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-70.05%25-green.svg)


No way to add test file ?


Does it make sense to move the changes for ctrucal-winioctl.c to commit c573a7874acc7873e092d51ea3c8c23b17cf4192?


it doesn't test different time-zones though, that could be improved.


It breaks the deploy task with a specific sha, see this [comment](https://github.com/getsentry/freight/pull/45#discussion_r38570277) for more information


The most important thing to fix


This is exactly what I meant :-) Thanks for taking care if it.


Nice project website! :+1:


Muchas gracias por la aclaración lorenzo :smile:. Me estaba volviendo loco porque no encontraba esto referenciado en ningún lado y yo recordaba haber empezado utilizándolo para CakePHP3. Entiendo que cuando hicisteis el cambio de los plugins (que se instalaran en Vendor en lugar de en Plugin) dejó de hacer falta este argumento.

Un saludo :wink:


Good catch! Will fix.


was on purpose, I'm making many commits and also unclean commit in this branch, and when everything is stable I was supposed to do clean commit, in a clean and separated way, everything.


*crypt


Zkus napsat test který vrátí tuto podmínku jako true. $this->balances <= 0 řekl bych že ta podmínka je zbytečná ale jistej si nejsem.


Checkmarx scan results 03 Sep 2015 11:58 UTC +03:00

High
Medium
Low
Info

New
1
1
2
0

Recurrent
0
0
0
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

066f603e097afc8f97b178984ff8475794e758a7	@cxehud


pIoctlFileChunk_p NULL check


Tady chybí ověření isAmountPositive.


I suspect it reduces disk IO, which might be nontrivial for rotating drives (memory etc. probably aren't that important nowadays).


101884090--NewVenting-Availability-Schedule (lgu1234) - Win64-Windows-7-VisualStudio-12: OK (1814 of 1814 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


You could make vm:set_lighting and vm:calc_lighting set a boolean in emerge.cpp where the on_generateds are executed to true and after they all were executed, if it's true, the light becomes calculated.
https://forum.minetest.net/viewtopic.php?f=5&t=13058


Do you mean commit 93def6da4b9fc78196fd57dbee39eef32367afc4 ?


locked for reference on fix patch


Thx!


please review /cc @gouzouni625 @makistsantekidis


你们要做成网页游戏？


Ah, ok, ignore this comment then.


TEST COMMENT ON COMMIT FROM LAPTOP


Is there a plan to merge this into stable/kilo?

Does this also kill the bug of puppet adding multiple copies of an image? (Because sometimes the glance image-list failed, so it thought the image does not exist and tried to added and then the glance image-create went through successfull and voila, a duplicate image is born)


develop (macumber) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg)


Nice work :+1:

I have a small suggestion: I would like it if the `Your latest check...` and the `Their latest update...` would be less prominent (smaller/lighter) than the normal text. Also I would combine it into one line (somehow), to make ik take up less space at the top. This information is interesting, but less interesting than the actual updates.


I suggest to [complete error detection and exception handling](https://github.com/Shardcached/libshardcache/issues/7 "Completion of error handling") for the used functions here.


Fixes #33


buildnes bez tohto el6 balik?


yes. sorry copied the wrong sha :(


Code does not work contains errors


"Por eso, aprender a buscar bibliografía y otras herramientas también es importante."

Esa es muy buena. A esto le falta un párrafo sobre buscar bibliografía.


102497980-Improve-waste-heat-function-handling-for-CoilCoolingDXMultiSpeed-and-CoilHeatingDXMultiSpeed-#5162 (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1289 of 1289 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


я подумал - и выделил так, если есть предложения лучше, буду только благодарен


I don't think it's necessary ... pthread_cond_wait() can fail only because of memory corruption.


Fixed: https://github.com/abligh/geodns/commit/76fd4483818a6f8acc18a28dc368745ab2513dd3


Good try, but the timeout was in mocha - I've updated here: c75a3bf5413615385ad6742d356752e422d066a3


I would add the acceptance criteria in the feature file, either in it's own section at the top or put each criteria under the scenario that covers it off, just makes it clear what the tests are covering off.


改行マン


What's the point of dropping top 20% if the floor is chosen?


this should be `deviceId`


Ага, молодец, сам дошел до JSTL-я.

Могу ссылкой поделиться еще http://ndpsoftware.com/JSPXMLCheatSheet.html иногда пригождается.


Should the program execution stop if important error situations will be detected?


To be even better this could be <button type="submit">Go</button> :-)


holder.js関連はモック用なのでもう消しましょう

holder.js はダミー画像を簡単に導入できるjsプラグインです


要啥自行车，哈哈。


Would be nice if Sphinx could add some special styling on this. It's quite easy to overlook like this:

https://docs.weblate.org/en/latest/admin/install.html#running-weblate-under-path


二箇所プチ修正したらLGTMで！
あと6日間！！


Urgh. I didn't know/notice, sorry.

Yeah, that should be reformatted. I guess when merging your PR, things should be squashed into two or three commits: Reformatting, "changing stuff" and the needed rename from `keyboardlayout` to `self`.


can this be consistent? camel vs. snake case.


@sabrecat Should this `false` be `true`?

At the moment there's no code to prevent the modal being shown every time the player's health decreases after 30 HP has been reached.


Можно было использовать `myIndex.count`


Why not convert to `orddict` (or `dict`) and use their merge functions?


Perhaps would make sense to use stream().filter().foreach() and thus filter all entries instead of the internal "if".


Structure and member description is missing.


TEST COMMENT


`results` there will always be one single `device`. You might want to skip the "Array" check and `forEach` then.


True, the parameter that is now abused should be replaced once we can [https://github.com/cwi-swat/rascal/issues/861](use keyword parameters as custom generators).

It is also true that it does not yet take different time zone in account, please remember ... it just a very small first setup which needs to be extended :)


Ok, nie załapałem z tym globalnym .gitignore - nigdy z tego nie korzystałem.
Co, jeszcze z obecnego .gitignore powinno trafić do globalnego?
Tyko *,iml? znalazłem przykład, z którego mógłbym skorzystać: https://github.com/github/gitignore/blob/master/Global/JetBrains.gitignore
Chyba, że możesz odesłać mnie do lepszego rozwiązania.


Why is `outputs` needed? Seems ugly to mess with that in `all-packages.nix`.


`instanceID` will not always be the BT address but will be guaranteed to be unique from what I understood.


あああ


You must've meant at least `sftp(smth).open` here. Do use get_astute_yaml.


TEST


Możesz skorzystać. Jak chcesz to mogę podesłać Ci mój global.


You might want to use `instanceID` instead of `device.uuids[0]` to uniquely identify a peripheral.


Использовать ссылки вот таким образом не совсем правильно. В этом случае нужно использовать `c:url` либо даже `spring:url`. Почитай про них и отпишись здесь об отличиях твоего текущего подхода и предлагаемого моной. В каких случаях твои ссылки не будут работать? Это тебе такой quiz :)


Ok, tak zrobię


ASCII significantly reduces size.
base64 plain - `4428 bytes (100%)`, ASCII85 plain - `1656 bytes (37.4%`)
base64 gzipped - `1573 bytes (100%)`, ASCII85 gzipped - `632 bytes (40.2%)`

![1441265193093](https://cloud.githubusercontent.com/assets/6908417/9654617/aafc7eb2-5234-11e5-8265-2018c2295ac4.png)

base64 (4428 bytes)
```scss
base64,PD94bWwgdmVyc2lvbj0iMS4wIiBlbmNvZGluZz0idXRmLTgiPz48c3ZnIHZlcnNpb249IjEuMSIgaWQ9IkNoZWNrIiB4bWxucz0iaHR0cDovL3d3dy53My5vcmcvMjAwMC9zdmciIHhtbG5zOnhsaW5rPSJodHRwOi8vd3d3LnczLm9yZy8xOTk5L3hsaW5rIiB4PSIwcHgiIHk9IjBweCIgdmlld0JveD0iMCAwIDYxMiA3OTIiIGVuYWJsZS1iYWNrZ3JvdW5kPSJuZXcgMCAwIDYxMiA3OTIiIHhtbDpzcGFjZT0icHJlc2VydmUiPjxwYXRoIGZpbGw9IiM1Q0I4NUMiIGQ9Ik0yMzMuOCw2MTAuMWMtMTMuMywwLTI1LjktNi4yLTM0LTE2LjlMOTAuNSw0NDguOEM3Ni4zLDQzMCw4MCw0MDMuMyw5OC44LDM4OS4xYzE4LjgtMTQuMyw0NS41LTEwLjUsNTkuOCw4LjNsNzEuOSw5NWwyMjAuOS0yNTAuNWMxMi41LTIwLDM4LjgtMjYuMSw1OC44LTEzLjZjMjAsMTIuNCwyNi4xLDM4LjcsMTMuNiw1OC44TDI3MCw1OTBjLTcuNCwxMi0yMC4yLDE5LjQtMzQuMywyMC4xQzIzNS4xLDYxMC4xLDIzNC41LDYxMC4xLDIzMy44LDYxMC4xeiIvPjwvc3ZnPg==
base64,PD94bWwgdmVyc2lvbj0iMS4wIiBlbmNvZGluZz0idXRmLTgiPz48c3ZnIHZlcnNpb249IjEuMSIgaWQ9Ildhcm5pbmciIHhtbG5zPSJodHRwOi8vd3d3LnczLm9yZy8yMDAwL3N2ZyIgeG1sbnM6eGxpbms9Imh0dHA6Ly93d3cudzMub3JnLzE5OTkveGxpbmsiIHg9IjBweCIgeT0iMHB4IiB2aWV3Qm94PSIwIDAgNjEyIDc5MiIgZW5hYmxlLWJhY2tncm91bmQ9Im5ldyAwIDAgNjEyIDc5MiIgeG1sOnNwYWNlPSJwcmVzZXJ2ZSI+PHBhdGggZmlsbD0iI0YwQUQ0RSIgZD0iTTYwMyw2NDAuMmwtMjc4LjUtNTA5Yy0zLjgtNi42LTEwLjgtMTAuNi0xOC41LTEwLjZzLTE0LjcsNC4xLTE4LjUsMTAuNkw5LDY0MC4yYy0zLjcsNi41LTMuNiwxNC40LDAuMiwyMC44YzMuOCw2LjUsMTAuOCwxMC40LDE4LjMsMTAuNGg1NTcuMWM3LjUsMCwxNC41LTMuOSwxOC4zLTEwLjRDNjA2LjYsNjU0LjYsNjA2LjcsNjQ2LjYsNjAzLDY0MC4yeiBNMzM2LjYsNjEwLjJoLTYxLjJWNTQ5aDYxLjJWNjEwLjJ6IE0zMzYuNiw1MDMuMWgtNjEuMlYzMDQuMmg2MS4yVjUwMy4xeiIvPjwvc3ZnPg==
base64,PD94bWwgdmVyc2lvbj0iMS4wIiBlbmNvZGluZz0idXRmLTgiPz48c3ZnIHZlcnNpb249IjEuMSIgaWQ9IkNyb3NzIiB4bWxucz0iaHR0cDovL3d3dy53My5vcmcvMjAwMC9zdmciIHhtbG5zOnhsaW5rPSJodHRwOi8vd3d3LnczLm9yZy8xOTk5L3hsaW5rIiB4PSIwcHgiIHk9IjBweCIgdmlld0JveD0iMCAwIDYxMiA3OTIiIGVuYWJsZS1iYWNrZ3JvdW5kPSJuZXcgMCAwIDYxMiA3OTIiIHhtbDpzcGFjZT0icHJlc2VydmUiPjxwYXRoIGZpbGw9IiNEOTUzNEYiIGQ9Ik00NDcsNTQ0LjRjLTE0LjQsMTQuNC0zNy42LDE0LjQtNTEuOSwwTDMwNiw0NTEuN2wtODkuMSw5Mi43Yy0xNC40LDE0LjQtMzcuNiwxNC40LTUxLjksMGMtMTQuNC0xNC40LTE0LjQtMzcuNiwwLTUxLjlsOTIuNC05Ni40TDE2NSwyOTkuNmMtMTQuNC0xNC40LTE0LjQtMzcuNiwwLTUxLjlzMzcuNi0xNC40LDUxLjksMGw4OS4yLDkyLjdsODkuMS05Mi43YzE0LjQtMTQuNCwzNy42LTE0LjQsNTEuOSwwYzE0LjQsMTQuNCwxNC40LDM3LjYsMCw1MS45TDM1NC43LDM5Nmw5Mi40LDk2LjRDNDYxLjQsNTA2LjgsNDYxLjQsNTMwLDQ0Nyw1NDQuNHoiLz48L3N2Zz4=
base64,PD94bWwgdmVyc2lvbj0iMS4wIiBlbmNvZGluZz0idXRmLTgiPz4NCjwhLS0gR2VuZXJhdG9yOiBBZG9iZSBJbGx1c3RyYXRvciAxNy4xLjAsIFNWRyBFeHBvcnQgUGx1Zy1JbiAuIFNWRyBWZXJzaW9uOiA2LjAwIEJ1aWxkIDApICAtLT4NCjwhRE9DVFlQRSBzdmcgUFVCTElDICItLy9XM0MvL0RURCBTVkcgMS4xLy9FTiIgImh0dHA6Ly93d3cudzMub3JnL0dyYXBoaWNzL1NWRy8xLjEvRFREL3N2ZzExLmR0ZCI+DQo8c3ZnIHZlcnNpb249IjEuMSIgaWQ9IkxheWVyXzEiIHhtbG5zPSJodHRwOi8vd3d3LnczLm9yZy8yMDAwL3N2ZyIgeG1sbnM6eGxpbms9Imh0dHA6Ly93d3cudzMub3JnLzE5OTkveGxpbmsiIHg9IjBweCIgeT0iMHB4Ig0KCSB2aWV3Qm94PSIwIDAgOCA4IiBlbmFibGUtYmFja2dyb3VuZD0ibmV3IDAgMCA4IDgiIHhtbDpzcGFjZT0icHJlc2VydmUiPg0KPHBhdGggZmlsbD0iI0ZGRkZGRiIgZD0iTTYuNCwxTDUuNywxLjdMMi45LDQuNUwyLjEsMy43TDEuNCwzTDAsNC40bDAuNywwLjdsMS41LDEuNWwwLjcsMC43bDAuNy0wLjdsMy41LTMuNWwwLjctMC43TDYuNCwxTDYuNCwxeiINCgkvPg0KPC9zdmc+DQo=
base64,PD94bWwgdmVyc2lvbj0iMS4wIiBlbmNvZGluZz0idXRmLTgiPz4NCjwhLS0gR2VuZXJhdG9yOiBBZG9iZSBJbGx1c3RyYXRvciAxNy4xLjAsIFNWRyBFeHBvcnQgUGx1Zy1JbiAuIFNWRyBWZXJzaW9uOiA2LjAwIEJ1aWxkIDApICAtLT4NCjwhRE9DVFlQRSBzdmcgUFVCTElDICItLy9XM0MvL0RURCBTVkcgMS4xLy9FTiIgImh0dHA6Ly93d3cudzMub3JnL0dyYXBoaWNzL1NWRy8xLjEvRFREL3N2ZzExLmR0ZCI+DQo8c3ZnIHZlcnNpb249IjEuMSIgaWQ9IkxheWVyXzEiIHhtbG5zPSJodHRwOi8vd3d3LnczLm9yZy8yMDAwL3N2ZyIgeG1sbnM6eGxpbms9Imh0dHA6Ly93d3cudzMub3JnLzE5OTkveGxpbmsiIHg9IjBweCIgeT0iMHB4Ig0KCSB3aWR0aD0iOHB4IiBoZWlnaHQ9IjhweCIgdmlld0JveD0iMCAwIDggOCIgZW5hYmxlLWJhY2tncm91bmQ9Im5ldyAwIDAgOCA4IiB4bWw6c3BhY2U9InByZXNlcnZlIj4NCjxwYXRoIGZpbGw9IiNGRkZGRkYiIGQ9Ik0wLDN2Mmg4VjNIMHoiLz4NCjwvc3ZnPg0K
base64,PD94bWwgdmVyc2lvbj0iMS4wIiBlbmNvZGluZz0idXRmLTgiPz4NCjwhLS0gR2VuZXJhdG9yOiBBZG9iZSBJbGx1c3RyYXRvciAxNy4xLjAsIFNWRyBFeHBvcnQgUGx1Zy1JbiAuIFNWRyBWZXJzaW9uOiA2LjAwIEJ1aWxkIDApICAtLT4NCjwhRE9DVFlQRSBzdmcgUFVCTElDICItLy9XM0MvL0RURCBTVkcgMS4xLy9FTiIgImh0dHA6Ly93d3cudzMub3JnL0dyYXBoaWNzL1NWRy8xLjEvRFREL3N2ZzExLmR0ZCI+DQo8c3ZnIHZlcnNpb249IjEuMSIgaWQ9IkxheWVyXzEiIHhtbG5zPSJodHRwOi8vd3d3LnczLm9yZy8yMDAwL3N2ZyIgeG1sbnM6eGxpbms9Imh0dHA6Ly93d3cudzMub3JnLzE5OTkveGxpbmsiIHg9IjBweCIgeT0iMHB4Ig0KCSB2aWV3Qm94PSIwIDAgOCA4IiBlbmFibGUtYmFja2dyb3VuZD0ibmV3IDAgMCA4IDgiIHhtbDpzcGFjZT0icHJlc2VydmUiPg0KPHBhdGggZmlsbD0iI0ZGRkZGRiIgZD0iTTQsMUMyLjMsMSwxLDIuMywxLDRzMS4zLDMsMywzczMtMS4zLDMtM1M1LjcsMSw0LDF6Ii8+DQo8L3N2Zz4NCg==
```
ASCII85 (1656 bytes)
```scss
charset=utf8,%3Csvg xmlns='http://www.w3.org/2000/svg' viewBox='0 0 612 792'%3E%3Cpath fill='%235CB85C' d='M233.8 610.1c-13.3 0-25.9-6.2-34-16.9L90.5 448.8C76.3 430 80 403.3 98.8 389.1c18.8-14.3 45.5-10.5 59.8 8.3l71.9 95 220.9-250.5c12.5-20 38.8-26.1 58.8-13.6 20 12.4 26.1 38.7 13.6 58.8L270 590c-7.4 12-20.2 19.4-34.3 20.1h-1.9z'/%3E%3C/svg%3E
charset=utf8,%3Csvg xmlns='http://www.w3.org/2000/svg' viewBox='0 0 612 792'%3E%3Cpath fill='%23D9534F' d='M447 544.4c-14.4 14.4-37.6 14.4-51.9 0L306 451.7l-89.1 92.7c-14.4 14.4-37.6 14.4-51.9 0-14.4-14.4-14.4-37.6 0-51.9l92.4-96.4-92.4-96.5c-14.4-14.4-14.4-37.6 0-51.9s37.6-14.4 51.9 0l89.2 92.7 89.1-92.7c14.4-14.4 37.6-14.4 51.9 0 14.4 14.4 14.4 37.6 0 51.9L354.7 396l92.4 96.4c14.3 14.4 14.3 37.6-.1 52z'/%3E%3C/svg%3E
charset=utf8,%3Csvg xmlns='http://www.w3.org/2000/svg' viewBox='0 0 612 792'%3E%3Cpath fill='%23F0AD4E' d='M603 640.2l-278.5-509c-3.8-6.6-10.8-10.6-18.5-10.6s-14.7 4.1-18.5 10.6L9 640.2c-3.7 6.5-3.6 14.4.2 20.8 3.8 6.5 10.8 10.4 18.3 10.4h557.1c7.5 0 14.5-3.9 18.3-10.4 3.7-6.4 3.8-14.4.1-20.8zm-266.4-30h-61.2V549h61.2v61.2zm0-107.1h-61.2V304.2h61.2v198.9z'/%3E%3C/svg%3E
charset=utf8,%3Csvg xmlns='http://www.w3.org/2000/svg' viewBox='0 0 8 8'%3E%3Cpath fill='%23FFF' d='M6.4 1l-.7.7-2.8 2.8-.8-.8-.7-.7L0 4.4l.7.7 1.5 1.5.7.7.7-.7 3.5-3.5.7-.7L6.4 1z'/%3E%3C/svg%3E
charset=utf8,%3Csvg xmlns='http://www.w3.org/2000/svg' width='8' height='8' viewBox='0 0 8 8'%3E%3Cpath fill='%23FFF' d='M0 3v2h8V3H0z'/%3E%3C/svg%3E
charset=utf8,%3Csvg xmlns='http://www.w3.org/2000/svg' viewBox='0 0 8 8'%3E%3Cpath fill='%23FFF' d='M4 1C2.3 1 1 2.3 1 4s1.3 3 3 3 3-1.3 3-3-1.3-3-3-3z'/%3E%3C/svg%3E
```


Done.


If I move this var from here and put it into group_vars, this role won't be self-contained anymore and anybody wants to use it in her playbook should be aware to declare this var herself. Putting this into common/defaults makes more sense to me because this role already has a dependency to common.
Anyway I didn't get what you mean by mixing up different releases?


I use a rotating drive, it takes time at startup without cache. Don't know about HighContrast, it has a separate cache?


I was actually thinking of **following** that link, there is already steps available for doing that. Something like this (pseudo code):

    When I follow "Manage Pending Minions (1)"
      Then I should see a "All Minions" text
      And I should see a "<minion id>" text below "Pending"

I think it would be a bit better since we would actually verify if the link works? Further it would not break the test in case the actual link URI changes!

Apart from that this step definition sounds like it is very generic ("the link XXX should point to YYY URL"), but in fact it is not: inside you are verifying the "Pending Minions" text in a hard-coded way. How should someone reuse this step definition for "checking" other links?


OK


@tgalal, I think we should still send  the delivery receipt here, otherwise we get an ack error


Now fixed, I think.


call me paranoïd but you don't protect your code from a null in the Autocloseable array


What's wrong with this:

````
test bool canConstructDate(datetime input)
    = date.year == input.year && date.month == input.month && date.day == input.day
    when datetime date := createDate(input.year, input.month, input.day);
```


I don't think such test beds belong in Nixpkgs. You can easily keep development versions in your own tree.


there should be a space after the * otherwise it's a syntax error


Yep, sorry, my mistake - suspiciously all the tests did run without this


TeamCity mobile-music-automatization :: PR_BUILD_CONFIGURATION [Build 60](http://teamcity.mobile.dev.yandex.net/viewLog.html?buildId=1191721&buildTypeId=MobileMusicAutomatization_TestBuildConfiguration) is now running


93651760-RowPerObjectReport (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (761 of 761 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-10.22%25-green.svg)


do those checks in `wrap` directly, you get a cleaner stacktrace in which does not appear `CloseablesIteratorWrapper`


#9612 is likely to be a better solution. And there's a more intrusive one: https://github.com/NixOS/nixpkgs/issues/287#issuecomment-136310741 (testing is welcome).


instead, you could make the blackboard a `vector<char>`.


Agreed


Зачем? Используя чистый SQL мы теряем один из главных плюсов liquibase -- не зависимость от БД. Верни пока на XML.


Do we need this patch - last time when I reworked highmem code, didn't we decide to ditch special tables for kmap and reuse swapper pg dir (used by vmalloc) and simplify all of the low level code. The vmalloc range is start of all the virtual mapping and kmap and friends will follow it so we don't need to check ?

Am i missing something here !


Why did you start with this? Couldn't you start with the basic HTML structure instead?


Okay, I'll move the change..


TeamCity mobile-music-automatization :: PR_BUILD_CONFIGURATION [Build 60](http://teamcity.mobile.dev.yandex.net/viewLog.html?buildId=1191721&buildTypeId=MobileMusicAutomatization_TestBuildConfiguration) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:01:18


Ok, nice !


ok


(the timezone comment was about the RandomValueTypeVisitor code)


How ofRandomGaussian() it now?


rax retry tests


I reset the branch and did another commit.


trying to see how it works


This is certainly NOT needed !


разве точка по размеру как буква в заданном шрифте?


This I need to check. So we use swapper pgdir, but dedicated pgtables !


Wzorowałem się na metodzie public List<Map> getProductsList() {...}. W jaki sposób należy zrobić to na obiektach - możesz odesłać mnie do jakiegoś źródła? Nie za bardzo wiem jak to ugryźć.


To rozwiązanie będzie najlepsze :)


I'll add log message for future investigating.


AAAAA


Два причастных оборота подряд нехорошо, лучше так: для сборки файлов мультиязыковой поддержки в проектах, ...


So `fCtrlInitialized_l` provides capability to invoke service module and general stack configuration from a single application right?


C'est une normalisation utilisée pour ne pas avoir de cube d'étoiles dès que le seuil est franchis, quand on passe le seuil on a quelques étoiles qui apparaissent et non une flopée. Ça vient de Rasera et Teyssier 2006.


this is different from other models... how are we supposed to override this?


AAA


Caused any jQuery event handler doesn't run in second page of dataTables plugin.
http://stackoverflow.com/questions/25414778/jquery-onclick-not-working-in-datatables-2nd-page-or-rows-past-11


Очень много


TTT


непонятно, о модуле ЧЕГО для интернационализации идет речь? общего модуля в собранном проекте? нужно тут немного развернуть, чтобы это стало ясно.


Здесь просто рассадник! :)

1) А где `finaly` для закрытия ресурсов? Ну или вместо них сразу используй try-with-resources из Java7 (http://habrahabr.ru/post/178405/) для того, чтобы у тебя правильно закрывались ресурсы.

2) **ВАЖНО!** Когда ты создаешь SQL-запрос на основе пользовательских данных, то ты сразу же получешь дыру в приложении. Прочитай про SQL injection, например, здесь: https://ru.wikipedia.org/wiki/%D0%92%D0%BD%D0%B5%D0%B4%D1%80%D0%B5%D0%BD%D0%B8%D0%B5_SQL-%D0%BA%D0%BE%D0%B4%D0%B0 Подсказка, чтобы защититься от этого в Java нужно использовать prepared statements, вместо обычных statements.

3) Если ты не можешь обработать исключение, то не надо его ловить и особенно вот так вот "глушить". Если приложение на работает, то пользователь должен увидеть ошибку, а не нормальную страницу с некорректными данными. Поэтому просто возвращай исключение из контроллера (добавь `throws`). Для начала это подойдет. В идеале, нужно отображать специальную страницу с ошибкой.


I don't understand the use of `fIgnoreMismatch_p` here?


имеются в виду .po-файлы? если да, стоит это указать


TeamCity Xbim :: Xbim Web UI :: Xbim Web UI [Build 8](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=141&buildTypeId=Xbim_XbimWebUi_XbimWebUi) is now running


AAAA


The flag makes sure that ctrlu_init is not called twice, e.g. oplk_init is called and the also oplk_serviceInit.
Couldn't find a better way to workaround calling oplk_init with api initialization parameters in the firmware update tool.

Do you have a better idea?


TeamCity Xbim :: Xbim Web UI :: Xbim Web UI [Build 8](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=141&buildTypeId=Xbim_XbimWebUi_XbimWebUi) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:00:19


The commit message has a typo:
... Windows __NDIS__ PCIe driver.


когда в дальнейшем, кем и для чего использоваться? нужно или выкинуть все, начиная с которые, или поподробнее прокомментировать юзкейс


Stwórz obiekt `DTO`, przepisz do niego i go zwróć.


I'm pretty sure you need to close this unless you use `File.open` with a block. You probably just want to use `log = File.read(fixture_path)`


Good suggestion.
I thought it was a good idea to be independent of the current datetime (test generator) implementation since its the subject under test ... might be overkill though


два «для», трудно разобраться, кто кого использует, нужно перестроить предложение.


Instead of this, you should benefit from associations and use `category.videos.each ...` - as each video is associated to a category.


I believe Alys is right.


Usually the flag is false (when calling oplk_init()), but in case of using oplk_initService() the firmware tool can ignore version and/or feature missmatch. E.g. if the kernel is version 2.3.0 but the update tool bases on V2.3.1 or so!


Not needed in assembly as far i can see !
Better to prefix the C symbols with @ as that makes is clearer


Done


Thanks! I'll change that


Its ok to have only `@categories` as you can access videos from each category.


js файлы называем маленькими буквами, названия из нескольких слов - через черточку (например, long-file-name.js)


Также как и в сосдением ревью:
- используй try-with-resources
- не лови исключение, если не знаешь, что с ним делать

И, конкретно для этого случая: старайся не использовать `SELECT *`, а явно указывай названия нужных тебе колонок. Это позволит тебе не выбирать лишние записи в будущем, когда изменится таблица `people`.


Great work @Kacper3331 :+1:


почему обязательно --save-dev? это требует комментария, либо, если это не обязательно, то нужно удалить эту опцию


I'd break this out in to a separate spec:

```ruby
it "parses each line in the log file" do
  # …

it 'sanitizes bad utf-8 lines' do
  # …
```


You can rather use `= image_tag video.small_cover_url` and change the url to cover the `/tmp` path. `image_tag` is Rails helper that makes it more readable.


Same here as mentioned above about `image_tag`


I added Rally to AllyEntersBattlefieldTriggeredAbility. Seems logical.

Set the rule text of Lantern Scout with setText.


требование — зависимость несколько двусмысленно. либо «enb-dem-i18n» зависит от, либо **Зависимости**: пакет enb версии...


Each theme seems to have a separate cache. IIRC the file is mmapped by apps so it's actually shared in RAM.


I would be tempted to wrap all of these checks (for title no, address and tenure) in one step that says 'the correct address details are displayed' or something along those lines. Within this step you could then check the individual pieces of data. What this means is it can then be used later when checking for the address in the case details page so reusing steps. But in a way that the scenario doesn't end being overly wordy and hard to  follow the flow.


I couldn't find any code that generates slugs - you have to manually enter them in Panopticon when creating a new Publisher artefact.


You can break the string over multiple lines:

```ruby
expected_log_line = "2015-07-09 15:41:40 [29933] foi+request-1234" \
                    "@example.com SMTP protocol synchronization " \
                    "error (next input" # etc
```


опечатка: интернаЦИОНАлизации


This is a wrong construction, you can access videos from the category by simply using `@category.videos`. I recommend deleting this line.


**ВАЖНО** нельзя отображать данные, пришедшие от пользователя как есть. Там могут быть плохие данные от злых пользователей. Почитай про XSS: https://ru.wikipedia.org/wiki/%D0%9C%D0%B5%D0%B6%D1%81%D0%B0%D0%B9%D1%82%D0%BE%D0%B2%D1%8B%D0%B9_%D1%81%D0%BA%D1%80%D0%B8%D0%BF%D1%82%D0%B8%D0%BD%D0%B3 Подсказка: используй `c:out`.


So you are trying to get to a different PGD - but we dont have special PGD - we have special page table - that's 2nd level - im totally confused what we are trying to do here.


Better way would be `@category.videos` here. Try to use as least ivars as possible.


@raunakkathuria this is done.


you are testing the library DateTime, so that is different than our `datetime` value in a way..

and otherwise:

```
test bool canConstructDate(int y, int m, int d)
    = date.year ==year && date.month == month && date.day == day
    when year := abs(y) % 5000
    && month := (abs(m) % 12) + 1
    && day := (abs(d) % daysIn(year, month)) + 1
    ;

int daysIn(int year, int month) = .. ;
```


```diff
- done = MailServerLogDone.new(:filename => "foo", :last_stat => DateTime.new(2012, 10, 10))
+ done = MailServerLogDone.new(:filename => "foo",
+                              :last_stat => DateTime.new(2012, 10, 10))


nitpick: replace `size_t` with `Blackboard::difference_type` ?


Same as I mentioned about `image_tag` in previous commit.


While the discussions are taking place, evidence is being gathered, appropriate tangible alternatives have yet to appear, `warning` or `note` would be an appropriate class than `critical`.


I would use short-hand routes now in Course 3. `video_path(video)` should be just `video`.


License header


Very true! Thanks. I reverted a fix of Stef, because yesterday we introduced another absorbion rule, but that gave some Sentinel errors. In combination of some messy merge conflicts, I reversed a coupel of commits, including this one. I thought I checked it all, but happily I have a keen audience! :-))


У вас тут стоит 15 анимаций ...А в core/java/android/widget/Toast.java у вас 13 ...Ошибка ,я дал вам там исправления


container


That commit link is incorrect. Currently links to https://github.com/korobi/Web/commit/5a4249767432873e16cb3238123cb207a0af6fef which doesn't exist.


Вы меня заблокирывали


по примеру непонятно, где имя блока, а где переводы. предлагаю сделать чуть более развернутый и чуть более реалистичный пример, чтобы легче было сориентироваться


непонятно: а предыдущий keyset-файл был для чего тогда?


Well spotted - it wasn't intentional. Rebased and fixed.


Can we make suggestions for environment variable names namespaced like ``SPENDB_AWS_KEY_ID`` or ``SPENDB_MAIL_USERNAME``?

So e.g.
```
AWS_KEY_ID = env('SPENDB_AWS_KEY_ID')
```
instead of
```
AWS_KEY_ID = env('AWS_KEY_ID')
```
?


Perhaps the handler method name should be "save". Would be more readable.


Thanks for starting this project! Is there anyway I can convince you to add EAN-13 support? I'm finding it really difficult to convert the C# code to Delphi.


Building in protect in obtaining to introduce a communication on a retraining to tradition in a global language to finally notice the educational and trades to improve the mentality of committed to make great improvements to families and there professional was developed and achieved I'm units of leading to prove there lives and have the skills to cultivate In education was well implemented as processors did there job to enhance the proper way that was needed to confirm with our work well done.


This is a little guide (in Spanish) to learn how to use the basic Unix commands.


falsify if also __-r__ is given
Do you mean __-f__?


непонятна модальность: могут или должны содержать? и на что это влияет?


Just reading all the changes from the decruft branch makes me jump in excitement :wink:  :+1:


Until last wednesday, the software supported the rule (x\/-y)/\y = y-x.  This is incorrect, as @sjcjoosten signals.
We're still trying to fix this, but the quick fix brought us from the rain into the drip. To be continued....


Ooh cool


We follow the CANlib API which states to return the id_.


Sounds good. Make it so.
On 3 Sep 2015 11:39, "jrudinskij" <notifications@github.com> wrote:

> Perhaps the handler method name should be "save". Would be more readable.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/OpenESDH/OpenESDH-UI/commit/81871657c4bef6c1bcb58e59a63947154832125a#commitcomment-13042991>
> .
>


"to hack the games" -> "to hack games"


слишком длинный комментарий, можно отрезать «для разных языков» (и так понятно)


No.  We're wanting to move everything towards getting all the config from environment variables.  Leaving this in place will allow us to migrate to passing this in the environment instead of overwriting files at deploy time.


А этот комментарий можно сделать покороче «Файл с общими переводами (для русского и английского)»


Ah sorry, I'll do that. I can't read it though because the code I'm testing wants a file handle, not a string.


непонятно: так в keyset-файлах или в одной из библиотек?


Put attributes as arguments of the openTag(String, Attribute...) and automatically calculate indentation level.


Oh yeah, sorry!


@bendem Any idea why? Is the hash incorrect - does staging generate its own commits/hashes? The URL itself looks okay?


ZoneHVACBaseboardRadiantConvectiveElectric-UI (kbenne) - i386-Windows-7-VisualStudio-12: OK (2307 of 2332 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-98.93%25-yellow.svg)


I have the latest Android SDK installed but I am still getting an error that my zipalign is outdated.

djalik@djalik-VirtualBox:~/Android/Sdk/build-tools/23.0.0$ ls -lart zipalign
-rwxrwxr-x 1 djalik djalik 92759 Aug 27 01:08 zipalign


:) I made that in 1 min and 3 secs.
But I searched (googled) for this one: rd /S /Q plugin


Why did you make password 20? Bcrypt hashed password returns at least 60 characters


Doh, doesn't look like the actual code that uses this closes it either https://github.com/mysociety/alaveteli/blob/d904d4e7f22eb7aa4e3903b19353b3b45062dbda/app/models/mail_server_log.rb#L53-L62


что значит «инициализировано данными»? как это сделать? нужна ссылка или пояснение


I'm not really using this as a spec, more as a way of mocking out the calls to the model so that I can return a known InfoRequest, is there a way of making that less of an expectation and more of an explicit mock?


Cool, Idk how to make .bat files.


непонятен технический процесс, скрытый за выражением «для получения». нужно где-то его выше отдельно описать: откуда вообще берется ядро, что должен сделать вручную разработчик, а что будет выполнено автоматически.


Container


нелишним будет уточнить, куда (в каком файле) писать эту зависимость


Firmware update module will not call oplk_init and oplk_serviceInit in the same execution right?


missing docu


@uhafner AFAICT the bug on trend graphs is only present on Workflow jobs. It is working for any other job type extending `AbstractProject`, right?


\since 1.5


addressings -> addressing


P.S. Есть еще один вопрос: а кому вообще нужен этот файл? о_О Может его можно удалить? :)


"Returns true if"
or
"Returns whether"

(whether = 'true or false depending on...')


I think you want a [method stub](http://www.rubydoc.info/gems/rspec-mocks/frames#Method_Stubs)

```ruby
# Add .with and .exactly if required
allow(InfoRequest).to receive(:find_by_incoming_email).and_return(ir)
```


94006714-MultipleTowers (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1818 of 1818 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


can be removed, let the compiler generate it


@pitr-ch ping


непонятно, при чем здесь ядро, если в разделе речь идет о технологии keysets? нужно где-то прописать, в каком отношении состоят ядро и технологии


#4854-ZoneHVACOutdoorAirUnit-does-not-autosize-child-objects-correctly (rraustad) - Win64-Windows-7-VisualStudio-12: Build Failed

![Build Badge](http://img.shields.io/badge/build%20status-failing-red.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-89.39%25-red.svg)


How about ```if @project.repositories.none? ``` ? Or if blank?


I try to make a Drag and Drop work with Protractor but i'm facing a problem I cannot figure out.
The drag seems to be too fast and ui-tree doesn't trigger any change. This is like if the move part of the drag and drop is not done.
Example of code is:

browser.actions().dragAndDrop(browser.findElement(by.css(sourceNodeCss)), {x : -50, y: 0 }).perform();

But if you make some extra step in the move it's works. Example:

browser.actions().mouseMove(browser.findElement(by.css(sourceNodeCss))).mouseDown().mouseMove({x:-10, y:0}).mouseMove({x:-10, y:0}).mouseMove({x:-10, y:0}).mouseMove({x:-10, y:0}).mouseMove({x:-10, y:0}).perform();

I hope this helps.


this break is never reached, due to the return above -> remove it
(clang even warns about that)


shorter: return QString()


неинформативное название разделов — непонятно, кто работает с технологиями — вроде бы это для пользователя, но в разделе описан внутренний процесс сборки. предлагаю название переформулировать


Nah, you only skip writing the properties. The message will still be sent ;)
Better remove the confusing comment.


8: M102 unrecognized bullet/signature
9: M102 unrecognized bullet/signature
19: M102 unrecognized bullet/signature
20: M102 unrecognized bullet/signature
11: M130 no bullets are allowed after signatures
15: M130 no bullets are allowed after signatures
17: M130 no bullets are allowed after signatures
22: M130 no bullets are allowed after signatures


the metadata and referenceParameters are still not written out, are they?


Нужно вынести в начало раздела «Быстрый старт», что в результате мы получаем функцию i18n которая работает так-то. Иначе непонятно, как вообще это все использовать.


Hi, I hope you dont mind me contacting you.  I am a windows user trying to connect via Putty to cakebox.  I have followed this http://cakebox.readthedocs.org/en/latest/tutorials/hardening-box-authentication/ but when I try vagrant rebuild --provision it always fails with authentication failed - retrying

Any ideas?


These 2, 4, 6 seem to be magic numbers, can we have named constants please?


At least 60? That means 60 is not right to put there. Should I use 80?

On Thu, Sep 3, 2015 at 10:46 AM, Otemuyiwa Prosper <notifications@github.com
> wrote:

> Why did you make password 20? Bcrypt hashed password returns at least 60
> characters
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/andela/pibbble/commit/cd2671dbca1966383e4c813bba579b1ddd74b22a#commitcomment-13043128>
> .
>


непонятно, при чем тут опять ядро, вроде бы речь шла о функции i18n — все-таки нужно как-то в самом начале объяснять архитектуру всего этого.


Результатом ЧЕГО?


eh, I don't particularly care for these


remove one of the two spaces after the comma


This kinda follows on a bit from a comment I made in the feature file about merging the address checks into one step. On this step though try to limit a step to one logical function such as checking that some values exist, entering a value, or clicking a button. Here you've got a step checking a piece of data exists and then clicking on a button. The issue with this is that the step description doesnt exactly match what the step actually does, it also limits its reuse if someone should want to do either of these functions but not both.


наличия модульной системы ГДЕ?


Rename the class name


should be a const ref, no?


94006714-MultipleTowers (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg)


Yeah, 80 would be good.


Moreover, fm_file_info_list_peek_head is derived from g_queue_peek_head (libfm -> fm-list.h), which handles empty queues. Also fm_file_info_list_get_length is derived from g_queue_get_length, which is zero for empty queues. So, everything seems safe logically.

BTW, do I need separate pull requests for my next commits?


Both lines mean the same thing, plus the earlier line is better since it shows that the secret key would be none if not found. Why not drop this commit? It doesn't introduce any change.


Файлы i18n могут подключаться? (и лучше без возвратных форм на -ся, потому что непонятно, сами они подключаются автоматически или разработчик должен как-то подключить) файлы можно подключить в node.js или...


I guess this shouldn't be part of the public API, but rather be private, with the calling class as friend.

App developers can't and shouldn't call this directly, it's part of the overall "interface->call(KDSoapMessage)" implementation.


Ок, в ближайшее время поправлю.


Hm, that could be very large and we have the information on other place too.


s/KDSoapMessage/KDSoapMessageAddressingProperties/ (already reported)


сам подключается или разработчик должен подключить?


The test is not okay. From spec:

- data contains "primary data"

"Primary data: MUST be:
- a single resource object, a single resource identifier object, or null, for requests that target single resources => not applied here.
- an array of resource objects, an array of resource identifier objects, or an empty array ([]), for requests that target resource collections

And null is not a valid resource object or resource object identifier because both MUST have "id" and "type".


Code Samsung TV device un32f5500


Oh, also -- we should probably deprecate the no-params case in api as well (if all you want to do is advance the order then you should be calling "next" or "advance").  So this is where we're trying to get with this method in api:
```ruby
def update_params
  params.require(:order).permit(permitted_checkout_attributes)
end
```
While frontend will need at least the massaged params stuff for the foreseeable future.


Can you squash all the empty lines within the methods? The code style in the repo doesn't include them...


It's inside the check if instigator=self, so that won't happen

On Thursday, September 3, 2015, Chris Kitching <notifications@github.com>
wrote:

> What does IsAlly(yourself) return? I suspect that's truthy.
>
> Does this code get triggered if you kill yourself?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/quark036/fa/commit/fd88618e0dab75b1eed9183f3f8450be9d259662#commitcomment-13060020>
> .
>


Nevermind, I can't read apparently :)


me is also an option


ruh-roh. I'll fix that up now (not sure what change I was trying to make here :\ )


trying it


@mrmarktyy the old method didnt actually do anything: check.x and check.y are not variables, the real variables that point to coordinates are check.position.x and check.position.y


あ、ちょっとまった！
restaurant tableのprimary_keyってprimary keyじゃない？
あとただのkeyって何やってるんだっけ？
key parentって親カテゴリーのこと？


Photovoltaics (evanweaver) - x86_64-Linux-Ubuntu-14.04-clang-3.5: Build Failed

![Build Badge](http://img.shields.io/badge/build%20status-failing-red.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-20.42%25-red.svg)


Can't link to callbacks in docs so am unsure how to handle. I would prefer to remove the duplication where possible.


@bleege Yes so glad I wrote that function. Silly boilerplate.


This change seems off.  *argv is currently NULL (0).  **argv is an undefined value at address 0x0.  The remainder seems perfect.


I can't figure out how to revert this!!


81211464-LinkInfiltrationAndReturnFlows (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1287 of 1287 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-70.0%25-green.svg)


This commit partially addresses #17


Oh yeah oops! Sorry didn't see those, I'll remove them later. Thanks!


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.570](http://teamcity.monogame.net/viewLog.html?buildId=10027&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


en realidad ninguno, seria necesario solo quisiéramos incluir soporte para acceso restringido a datasets; quizá en todo caso lo podríamos agregar mas adelante; lo dejamos fuera por ahora?


Yes it fixed everything!

And yes I know about the tree issue. They (the programmers) actually got confused about that themselves and used the wrong ones in places as well, mostly near the pyramids. So I will need to hack the rom to fix the levels where that occurs.

And I did notice the flipped sprite problem when I added in the goombas.

Very nice work. :D


Photovoltaics (evanweaver) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg)


Web sites are using this. But, like the other WebKit APIs we support, this will be considered deprecated.  We see the Directory Upload proposal that's in this commit as the right way forward here. But there's still standards work ongoing for that.


@binarylogic postgres doesn't support count with order


Ok I close the case then. Thanks for your assistance.


si :+1: me encanta la idea, incluso me parece que podríamos agregar soporte para distintos mecanismos de distribución a través de plugins/controladores independientes por ejemplo, si lo hiciéramos así el soporte para el API de consulta seria solo una de la lista y podríamos hacer mas en el futuro; se me ocurre organizarlos por protocolo, por ejemplo:

- HTTPS, el controlador API que ya tenemos
- WSS, para secure websockets
- Magnet, para torrents

Cada zona ( que de acuerdo a DCAT es un Dataset ) podría activar/incluir los mecanismos que quisiera habilitar


I need to double check the impact of changing the DataBoundConstructor to add an additional argument. There may be an impact for the upgrade scenario moving from an earlier version of the plugin to this version of the plugin.

Have you checked an upgrade scenario moving jobs from the previous version of the plugin to a plugin version allowing a submodule reference repository?


NP, any time man, glad to be of assistance (well I didn't do anything but you know what I mean lol)


Maybe coerce localeTerritory to lowercase before testing, for robustness?  Or are we positive that this jibber_jabber library will always provide this as lowercase?  One of our tests uses uppercase "IR".  It will correctly pass, but it's slightly confusing because testing "CN" will give possibly-surprising results.


Cierto, era un ejemplo viejo, lo corrijo en el siguiente commit


97215920---Fill-Gaps-in-HVAC-Components---Initial-Code (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (762 of 762 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-10.81%25-green.svg)


Wow! That's gorgeous! :+1:


can I help pls pls pls


this should be set as `SOCIAL_AUTH_USER_FIELDS` instead, so it is less likely to conflict with other settings/apps. (when `python-social-auth` code reads this value, it checks `strategy.setting('USER_FIELDS')` and the `setting` method tries various lookups including `SOCIAL_AUTH_USER_FIELDS`)


I seem to have it fixed if you look at the latest commit.  Works for me.  I should add a test.  Let me add a test


Yeah, mostly I don't see why we need to extract the module. That pattern works good for helpers or controllers, but I think with actors we can use a dependency injection model instead.


Please add a comment here explaining what this setting is.


gpio.c ?


Please keep the English comments.


Can you override interpret_visibility (or whatever), do custom stuff and then call super?  If we have to override too much then I think we need to refactor the actor methods into smaller chunks


Maybe we should open a separate issue to fix those problems with the syntax highlighting, otherwise we might leave this PR on a perpetual 'waiting' status ;) But a moderately good default would be good for the first iteration of this.


'RT_USING_TIMER_DRIVER' maybe better.


Please use the new COPYRIGHT file header.


Probably should log a warning about deprecation, at the very least to remind maintainers what can be removed in the next major version bump :)


'RT_Device_Class_Timer' is enough.


develop (kbenne) - Win64-Windows-7-VisualStudio-12: OK (2315 of 2336 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.1%25-yellow.svg)


練習の練習


Теперь можно добавить в CSS что-то вроде `.de-panel-buttons { width: 162px; }` и значки будут переходить на следующую строку:
![screenshot_20150904032820](https://cloud.githubusercontent.com/assets/1163958/9674104/1151e564-52b5-11e5-82a7-1e0cf8eb4f5f.png)

Только надо пофиксить окна настроек, чтобы они динамически высчитывали высоту панели, а то сейчас там захардкожено 25px.


Please provide a COPYRIGHT file header and provide a markdown format document.


Why don't raise an exception instead of returning nil when the value is invalid for the enum? Just like `parse` does.


javabeans


http://stackoverflow.com/questions/31452202/ios-swift-merge-and-convert-wav-files-to-mp3

https://github.com/wuqiong/mp3lame-for-iOS

http://stackoverflow.com/questions/24002369/how-to-call-objective-c-code-from-swift

http://stackoverflow.com/questions/16324095/custom-completion-block-for-my-own-method

https://thatthinginswift.com/completion-handlers/

https://www.codefellows.org/blog/writing-completion-blocks-with-closures-in-swift


Good find, but the "$notify =" part isn't needed, just the 'both' string.


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.570](http://teamcity.monogame.net/viewLog.html?buildId=10027&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 538, ignored: 6 Build time: 00:23:35


This loop reads data and prints it out. We will have to modify this to also write data that is typed into the command line.


1. 我建议你多考虑几个平台，例如换到LPC的平台上会如何实现，感觉驱动侧的接口有些多。
2. 请提供相关的文档和例程，从代码上看，确实看不出来如何使用。例如应用可以通过rt_device*的接口来访问timer设备，那么然后呢？能够在timer device超时时，调用用户的函数？貌似不行。。。


falta añadir que el juego vuelva a empezar cuando se presiona una tecla en el game over y mejorar la estetica del mismo.


What in God's name are you doing you idiot!!!  You broke the whole build with this crap.  You are fired!!

Naw just kidding :-)


Pano i-revert sir? Ayaw po ma-revert e


AHHHHH! what?

you have already boo'ed, bar'ed and foo'ed, why are you AHHHHHH! ing now?


I actually tried it with just the string first and got an error after adding a new user, so I went with this option. Must have been something else going on at that point as it's working now with just the string.


I think this test might overflow actually. The range 769126 -> 773157 includes 769641 which was listed in Taylor's list of overflow numbers [here](https://piazza.com/class/idgl07a8yrx5m6?cid=123) as an overflow number.


@rbgarga Some time ago you commented to me that we should put 4 spaces for 2nd and later lines of multi-line "if" statements. I did that (or preserved that) during the code style review.
Has the policy changed?
It seems that many of these are now being changed to tabs.


TeamCity Architecture Team :: Java Platform :: otj-components :: otj-data-structures :: build [Build 51](http://teamcity.otenv.com/viewLog.html?buildId=1494177&buildTypeId=ArchitectureTeam_JavaPlatform_OtjComponents_OtjDataStructures_Build) is now running


The Russian translation is read in a way of "me no spek Englush."

@EmmittJ Do you use Crowdin or just commits to the branch?


TeamCity Architecture Team :: Java Platform :: otj-components :: otj-data-structures :: build [Build 51](http://teamcity.otenv.com/viewLog.html?buildId=1494177&buildTypeId=ArchitectureTeam_JavaPlatform_OtjComponents_OtjDataStructures_Build) outcome was **FAILURE**
Summary: Tests passed: 30; exit code 1 Build time: 0:0:0


Sadly,
s/SAdly/Sadly/


nice catch.  wonder how i never noticed this


typo: "length"


typo "length"


:star:


I hope this will be done


bây giờ tất cả chỉ dùng một repo này thôi nha


101884090-Venting-Availability-Schedule-does-not-work-properly-in-the-AFN-model (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1803 of 1804 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.94%25-yellow.svg)


I'll take a look at Problem #4 for profiles.


Added a test.


101884090-Venting-Availability-Schedule-does-not-work-properly-in-the-AFN-model (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg)


Can we also create an English version for this docs?


Wow


Should this really be wep_enable or ?


@waj Hmmm... yes, we could do that. I'm using it [here](https://github.com/manastech/crystal/blob/master/src/http/web_socket.cr#L97-L100). I wanted to customize the error message so checking with nil I can do it. But probably raising "Invalid {{@type}}: #{value}" is the same.

On the other hand, we could make `parse` return nil. I wanted to avoid exceptions for performance reasons, but I don't know.


Change it and create a pull request, please.


Ừ.để test thêm xem nó có ổn không đã


Merge conflict stuff...


Sorry, I wasn't the one who translated it. Just commits
On Sep 3, 2015 5:45 PM, "cuss" <notifications@github.com> wrote:

> The Russian translation is read in a way of "me no spek Englush."
>
> @EmmittJ <https://github.com/EmmittJ> Do you use Crowdin or just commits
> to the branch?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/EmmittJ/PoESkillTree/commit/67aa3918c17fd5b08767a2b3b87e117f872a4523#commitcomment-13060472>
> .
>


Is this accidentally incorporated?


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 505](https://teamcity.labkey.org/viewLog.html?buildId=270052&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 5, passed: 0 Build time: 00:10:20
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSVisualizationTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSMeasureStoreTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSLoginTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSSecurityTest: <no details avaliable>

```


In this case, the scheduleJob() must have failed due to some problem other than that the schedule already exists. How should that be handled? Two possibilities:
1. we say that if one schedule fails, all schedules should fail. In that case, you have to delete the schedules that were already created. Otherwise, your call will fail, but it has already created the schedules. And if the caller retries later, it will fail again because those schedules already exist.
2. we say that we ignore this failure with a warning (not my favorite). In that case you would have to continue here, and create the remaining schedules.


This is still not working as expected. I need to add media queries to scale the navbar contents.


Jose submission


Delete Branches


a


Yeah, but with a group clause aggregates are valid. For example, counting the number of rows within each group. I'll submit a PR with tests shortly.


According to the tooltip, and the full red blocks, I think that means full deletion.


Hey Eric. Thanks for this PR. I think this makes total sense - the current implementation is an oversight really.

I do think it should be the default behaviort - no "deactivateBindings" parameter even needed (I can't think of a situation where it should ever be false).

What do you think? If you want to commit a version where the parameter is not needed (the PR will automatically be updated) I'll merge it.


This has been in the same package for a long time (https://github.com/SpongePowered/SpongeAPI/commit/3b68e24765fd8ddbfb866f5655655e83ab158a47#diff-2ab421167ec5d1c019761332abe9dec3), I don't really see why it needed to be changed now.


Delete this shit


Go rails go!


ok you were right this file is stupid


Biến ngay Đạo chếch ai cho m phát biểu ở đây


This is in the wrong spot, it should be above soft volume. When you look at the original revert comment, you can see it was above soft volume. https://github.com/CyanogenMod/android_kernel_htc_msm8974/commit/bd8f19398258ee417d471f866f13b4bbac116cde


I tried to commit this to my personal fork (even using smartgit to help me not mess it up) but it still wound up in the imatix fork. My success rate using git remains zero.


đm thằng đạo cút ngay


Setup bootstrap which makes it _sexy_
* Gemfile and Gemfile.lock are related to bundler and the gems we needed to add to include bootstrap, annotate and friends.

* We also added a link on the static_pages home page


We ran `rails generate controller books new index show edit`


Needs the `Makefile` changes so I can start using this before it's publicly documented :p


Change this repo's name to jon5477.github.io instead of mygithubpage


Originally our `rails generate` command setup `get` routes, but we'll switch to using `resources` which covers the usual RESTful routes cases.

You can read a little more here: http://rubylearning.com/blog/2014/01/29/routing-basics-ruby-on-rails-for-front-end-developers/


We should add our other routes to this controller as well.  Note that they're not complete yet.


Merge problems?


Interesting. I wonder how you discovered it :stuck_out_tongue:


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 504](https://teamcity.labkey.org/viewLog.html?buildId=270049&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 33 (5 new), passed: 4 Build time: 00:32:21
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSTest: <no details avaliable>

org.labkey.test.tests.CDSTest.testLearnAboutStudiesSearch: <no details avaliable>

org.labkey.test.tests.CDSTest.testSummaryPageSingleAxisLinks: <no details avaliable>

org.labkey.test.tests.CDSTest.testLearnAboutStudyProductsSearch: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSLoginTest: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyAntigenScatterPlot: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyBoxPlots: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifySubjectCountsWithFilters: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyTimeAxisWithMultipleSchedules: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifySubjectCountsWithFiltersAdvancedOptions: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyScatterPlotColorAxis: <no details avaliable>

##### there are 23 more failed tests, see build details
```


You don't see that with merge tools too often, I love it. Too bad we don't get that with Clearcase.


Changed the link on home to point to books_path
* if you run `rake routes` you should see this route
* link_to is a rails helper method that sets up an html link

html links look like
`<a href="somewhere">text</a>`

http://api.rubyonrails.org/classes/ActionView/Helpers/UrlHelper.html#method-i-link_to


I think i may have messed up some of the offset stuff, so you should check that, but I think the rest is ok


canaloop isn't correct. I think you mean "canola".


we ran: `    rails g model book title:string year_published:integer author:string available:boolean genre:string image_url:string`


Ok I got it it was because of #191


`rake db:migrate`

In the previous step we created a new migration file.
When we migrate it updates the schema.rb file which is generated ruby/rails code to create the database structure.

If you later did `rake db:schema:load` this file would be used to build new tables in your database


this debug line is the case of https://github.com/wwivbbs/wwiv/issues/169


annotate was kind enough to generate some info in our models about the db structure.


101884090-Venting-Availability-Schedule-does-not-work-properly-in-the-AFN-model (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (754 of 754 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-8.88%25-green.svg)


I thought about this but the issue is that there is not much we can do here.
The SchedulerException is thrown if the scheduler has shutdown for some reason. Now since the scheduler has shutdown we cannot delete the schedules to which we deployed so far neither carry on with deploying schedules.
One option will be to try starting the schedule again and keep retrying for it to start and then only we can do something.
I am not sure as what is the behavior of quartz scheduler in case of shutdown due to failure as whether it needs to be started manually or it tries to restart.


We create the @books instance variable on the index action of the controller.  That makes @books available to the index view (found in app/views/books/index.html.erb).  Any instance variable set in the controller method is shared with its view.

`Book.all` is rails for the SQL: `SELECT * FROM "books".  Basically, get all books from the database.


Keep it up!


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 506](https://teamcity.labkey.org/viewLog.html?buildId=270059&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 28, passed: 9 Build time: 00:22:08
### Failed tests
```
org.labkey.test.tests.CDSMeasureStoreTest.testDemoMeasureStore: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyAntigenScatterPlot: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyBoxPlots: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifySubjectCountsWithFilters: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyTimeAxisWithMultipleSchedules: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifySubjectCountsWithFiltersAdvancedOptions: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyScatterPlotColorAxis: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyAntigenBoxPlot: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyBinnedPlot: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyScatterPlot: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyGutterPlotBasic: <no details avaliable>

##### there are 18 more failed tests, see build details
```


向hosts添加新规则不必重启即可生效，另外
让所有规则生效最简单的方法是断一下网，重新连接网络后，即生效


This was more for debugging purposes.  We'd never really do @books.inspect cause it's ugly.

Output:
`"#<ActiveRecord::Relation [#<Book id: 1, title: \"The Hobbit\", year_published: nil, author: \"J.R.R. Tolkien\", available: nil, genre: nil, image_url: nil, created_at: \"2015-09-04 01:30:22\", updated_at: \"2015-09-04 01:30:22\">]>"`


This is the line for the checkpoint submission


Now we write a loop in ~ruby~ scratch that, embedded ruby (erb) that goes over each book in the @books instance variable and outputs the title.

Actually there's a small bug in this code, can you spot it? (it's related to the html used)