Copyright © UnaMesa Association 2007-2016

Typo? 2017?


ah don't bother that comment, i saw the other comment below


Doesn't `React` still needs to be imported when JSX is used?

```
ReferenceError: React is not defined
```


(L) (L) (L)


Possibly useful to you: if you install [shellcheck](https://github.com/koalaman/shellcheck) it will save you the googling by flagging the incorrect one as an error in your editor :slightly_smiling_face:.


NewErrorFixAgainWhenHoursShownInFirstColumnOfTable (JasonGlazer) - Win64-Windows-7-VisualStudio-14: OK (2496 of 2496 tests passed, 0 test warnings)

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


you build all 3 classes here, but I only will have one. this means the work is superfluous at best


YOU TOLD ME TO


well done!


forcing choices. incorrect


not handled properly as per instruction in class.


not sure about that one, it doesnt look like av_free does any null check, and primedata could be null if the earlier checks fail it seems.


forcing choice. incorrect


Figura Nº1


not handled properly


Probably shouldn't hardcode values (`-1e100`) even if they are just meant to be large numbers.  Define a constant at the top of the module and use that instead.


AFN2017 (macumber) - i386-Windows-7-VisualStudio-12: Build Failed

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


not handled at all


You must add revison also, default is "xosp-n"


handled incorrectly


Changed to `FilteringSourceRootsContainer(filter = { ... })`.


AFN2017 (macumber) - Win64-Windows-7-VisualStudio-12: Build Failed

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


what


Good question.

@dhruska, any insight into why you removed that?


@AndropaX isn't that cm-14.1?
https://github.com/XOSP-Project/platform_manifest/blob/xosp-n-rebase/snippets/xosp_devices_deps.xml#L6


I've got some more fixes for this, ready to make a pull request for it ;)
Expect it in 3 minutes


This commit fixed #39.


Not sure that I quite follow because I haven't spent much time setting up Fog. Is the idea that `fog_public` exposes a public URL by changing file ACL's but `fog_serve_public` somehow exposes a (different? same?) public URL without changing file ACL's? And if so, is there a corresponding bit of code in Fog that reads this CarrierWave setting and acts on it accordingly?


He's right  @AndropaX if none revision is specified the default one is used


It explicitly tolerates NULL.


@aviraxp well... fuck! Thanks.


This element can be already existing in a header morkup - you just need to insert username with jquery text() method by certain selector. So the header building will be in one place.


Этот сценарий не нужен, повторяюсь. У тебя сценарий выше, и так уже запускает сервер с чефом, оставляй только второй сценарий (тест план в тикете поправим)


With no revision set roomservice will add xosp-n to manifest
https://github.com/XOSP-Project/android_build/blob/xosp-n-rebase/tools/roomservice.py#L41


**Whoops!**

There were *49 warnings* that will prevent this PR from being merged. Please take a look, and either fix, or provide a justification for why they can't be fixed.

**WARNINGS**
[`src/content/en/fundamentals/engage-and-retain/push-notifications/common-notification-patterns.md#L4`](https://github.com/Google/WebFundamentals/blame/79342a44d9bccdca5a8c51c4e782087b8d02526a/src/content/en/fundamentals/engage-and-retain/push-notifications/common-notification-patterns.md#L4) - WF Tag `wf_updated_on` must be within the last 7 days.
[`src/content/en/fundamentals/engage-and-retain/push-notifications/common-notification-patterns.md#L260`](https://github.com/Google/WebFundamentals/blame/79342a44d9bccdca5a8c51c4e782087b8d02526a/src/content/en/fundamentals/engage-and-retain/push-notifications/common-notification-patterns.md#L260) - Line must be at most 100 characters
[`src/content/en/fundamentals/engage-and-retain/push-notifications/common-notification-patterns.md#L264`](https://github.com/Google/WebFundamentals/blame/79342a44d9bccdca5a8c51c4e782087b8d02526a/src/content/en/fundamentals/engage-and-retain/push-notifications/common-notification-patterns.md#L264) - Line must be at most 100 characters
[`src/content/en/fundamentals/engage-and-retain/push-notifications/common-notification-patterns.md#L330`](https://github.com/Google/WebFundamentals/blame/79342a44d9bccdca5a8c51c4e782087b8d02526a/src/content/en/fundamentals/engage-and-retain/push-notifications/common-notification-patterns.md#L330) - Line must be at most 100 characters
[`src/content/en/fundamentals/engage-and-retain/push-notifications/common-notification-patterns.md#L357`](https://github.com/Google/WebFundamentals/blame/79342a44d9bccdca5a8c51c4e782087b8d02526a/src/content/en/fundamentals/engage-and-retain/push-notifications/common-notification-patterns.md#L357) - Line must be at most 100 characters
[`src/content/en/fundamentals/engage-and-retain/push-notifications/display-a-notification.md#L4`](https://github.com/Google/WebFundamentals/blame/79342a44d9bccdca5a8c51c4e782087b8d02526a/src/content/en/fundamentals/engage-and-retain/push-notifications/display-a-notification.md#L4) - WF Tag `wf_updated_on` must be within the last 7 days.
[`src/content/en/fundamentals/engage-and-retain/push-notifications/display-a-notification.md#L78`](https://github.com/Google/WebFundamentals/blame/79342a44d9bccdca5a8c51c4e782087b8d02526a/src/content/en/fundamentals/engage-and-retain/push-notifications/display-a-notification.md#L78) - Line must be at most 100 characters
[`src/content/en/fundamentals/engage-and-retain/push-notifications/display-a-notification.md#L82`](https://github.com/Google/WebFundamentals/blame/79342a44d9bccdca5a8c51c4e782087b8d02526a/src/content/en/fundamentals/engage-and-retain/push-notifications/display-a-notification.md#L82) - Line must be at most 100 characters
[`src/content/en/fundamentals/engage-and-retain/push-notifications/display-a-notification.md#L86`](https://github.com/Google/WebFundamentals/blame/79342a44d9bccdca5a8c51c4e782087b8d02526a/src/content/en/fundamentals/engage-and-retain/push-notifications/display-a-notification.md#L86) - Line must be at most 100 characters
[`src/content/en/fundamentals/engage-and-retain/push-notifications/display-a-notification.md#L91`](https://github.com/Google/WebFundamentals/blame/79342a44d9bccdca5a8c51c4e782087b8d02526a/src/content/en/fundamentals/engage-and-retain/push-notifications/display-a-notification.md#L91) - Line must be at most 100 characters
[`src/content/en/fundamentals/engage-and-retain/push-notifications/display-a-notification.md#L93`](https://github.com/Google/WebFundamentals/blame/79342a44d9bccdca5a8c51c4e782087b8d02526a/src/content/en/fundamentals/engage-and-retain/push-notifications/display-a-notification.md#L93) - Line must be at most 100 characters
[`src/content/en/fundamentals/engage-and-retain/push-notifications/display-a-notification.md#L101`](https://github.com/Google/WebFundamentals/blame/79342a44d9bccdca5a8c51c4e782087b8d02526a/src/content/en/fundamentals/engage-and-retain/push-notifications/display-a-notification.md#L101) - Line must be at most 100 characters
[`src/content/en/fundamentals/engage-and-retain/push-notifications/display-a-notification.md#L103`](https://github.com/Google/WebFundamentals/blame/79342a44d9bccdca5a8c51c4e782087b8d02526a/src/content/en/fundamentals/engage-and-retain/push-notifications/display-a-notification.md#L103) - Line must be at most 100 characters
[`src/content/en/fundamentals/engage-and-retain/push-notifications/display-a-notification.md#L119`](https://github.com/Google/WebFundamentals/blame/79342a44d9bccdca5a8c51c4e782087b8d02526a/src/content/en/fundamentals/engage-and-retain/push-notifications/display-a-notification.md#L119) - Line must be at most 100 characters
[`src/content/en/fundamentals/engage-and-retain/push-notifications/display-a-notification.md#L123`](https://github.com/Google/WebFundamentals/blame/79342a44d9bccdca5a8c51c4e782087b8d02526a/src/content/en/fundamentals/engage-and-retain/push-notifications/display-a-notification.md#L123) - Line must be at most 100 characters
[`src/content/en/fundamentals/engage-and-retain/push-notifications/display-a-notification.md#L150`](https://github.com/Google/WebFundamentals/blame/79342a44d9bccdca5a8c51c4e782087b8d02526a/src/content/en/fundamentals/engage-and-retain/push-notifications/display-a-notification.md#L150) - Line must be at most 100 characters
[`src/content/en/fundamentals/engage-and-retain/push-notifications/display-a-notification.md#L154`](https://github.com/Google/WebFundamentals/blame/79342a44d9bccdca5a8c51c4e782087b8d02526a/src/content/en/fundamentals/engage-and-retain/push-notifications/display-a-notification.md#L154) - Line must be at most 100 characters
[`src/content/en/fundamentals/engage-and-retain/push-notifications/display-a-notification.md#L178`](https://github.com/Google/WebFundamentals/blame/79342a44d9bccdca5a8c51c4e782087b8d02526a/src/content/en/fundamentals/engage-and-retain/push-notifications/display-a-notification.md#L178) - Line must be at most 100 characters
[`src/content/en/fundamentals/engage-and-retain/push-notifications/display-a-notification.md#L182`](https://github.com/Google/WebFundamentals/blame/79342a44d9bccdca5a8c51c4e782087b8d02526a/src/content/en/fundamentals/engage-and-retain/push-notifications/display-a-notification.md#L182) - Line must be at most 100 characters
[`src/content/en/fundamentals/engage-and-retain/push-notifications/display-a-notification.md#L241`](https://github.com/Google/WebFundamentals/blame/79342a44d9bccdca5a8c51c4e782087b8d02526a/src/content/en/fundamentals/engage-and-retain/push-notifications/display-a-notification.md#L241) - Line must be at most 100 characters
[`src/content/en/fundamentals/engage-and-retain/push-notifications/display-a-notification.md#L255`](https://github.com/Google/WebFundamentals/blame/79342a44d9bccdca5a8c51c4e782087b8d02526a/src/content/en/fundamentals/engage-and-retain/push-notifications/display-a-notification.md#L255) - Line must be at most 100 characters
[`src/content/en/fundamentals/engage-and-retain/push-notifications/display-a-notification.md#L296`](https://github.com/Google/WebFundamentals/blame/79342a44d9bccdca5a8c51c4e782087b8d02526a/src/content/en/fundamentals/engage-and-retain/push-notifications/display-a-notification.md#L296) - Line must be at most 100 characters
[`src/content/en/fundamentals/engage-and-retain/push-notifications/display-a-notification.md#L300`](https://github.com/Google/WebFundamentals/blame/79342a44d9bccdca5a8c51c4e782087b8d02526a/src/content/en/fundamentals/engage-and-retain/push-notifications/display-a-notification.md#L300) - Line must be at most 100 characters
[`src/content/en/fundamentals/engage-and-retain/push-notifications/faq.md#L4`](https://github.com/Google/WebFundamentals/blame/79342a44d9bccdca5a8c51c4e782087b8d02526a/src/content/en/fundamentals/engage-and-retain/push-notifications/faq.md#L4) - WF Tag `wf_updated_on` must be within the last 7 days.
[`src/content/en/fundamentals/engage-and-retain/push-notifications/faq.md#L53`](https://github.com/Google/WebFundamentals/blame/79342a44d9bccdca5a8c51c4e782087b8d02526a/src/content/en/fundamentals/engage-and-retain/push-notifications/faq.md#L53) - Line must be at most 100 characters
[`src/content/en/fundamentals/engage-and-retain/push-notifications/faq.md#L74`](https://github.com/Google/WebFundamentals/blame/79342a44d9bccdca5a8c51c4e782087b8d02526a/src/content/en/fundamentals/engage-and-retain/push-notifications/faq.md#L74) - Line must be at most 100 characters
[`src/content/en/fundamentals/engage-and-retain/push-notifications/good-notification.md`](https://github.com/Google/WebFundamentals/blame/79342a44d9bccdca5a8c51c4e782087b8d02526a/src/content/en/fundamentals/engage-and-retain/push-notifications/good-notification.md) - Unable to read file, was it deleted?
[`src/content/en/fundamentals/engage-and-retain/push-notifications/handling-messages.md#L4`](https://github.com/Google/WebFundamentals/blame/79342a44d9bccdca5a8c51c4e782087b8d02526a/src/content/en/fundamentals/engage-and-retain/push-notifications/handling-messages.md#L4) - WF Tag `wf_updated_on` must be within the last 7 days.
[`src/content/en/fundamentals/engage-and-retain/push-notifications/how-push-works.md#L4`](https://github.com/Google/WebFundamentals/blame/79342a44d9bccdca5a8c51c4e782087b8d02526a/src/content/en/fundamentals/engage-and-retain/push-notifications/how-push-works.md#L4) - WF Tag `wf_updated_on` must be within the last 7 days.
[`src/content/en/fundamentals/engage-and-retain/push-notifications/how-push-works.md#L69`](https://github.com/Google/WebFundamentals/blame/79342a44d9bccdca5a8c51c4e782087b8d02526a/src/content/en/fundamentals/engage-and-retain/push-notifications/how-push-works.md#L69) - Line must be at most 100 characters
[`src/content/en/fundamentals/engage-and-retain/push-notifications/index.md#L5`](https://github.com/Google/WebFundamentals/blame/79342a44d9bccdca5a8c51c4e782087b8d02526a/src/content/en/fundamentals/engage-and-retain/push-notifications/index.md#L5) - WF Tag `wf_updated_on` must be within the last 7 days.
[`src/content/en/fundamentals/engage-and-retain/push-notifications/notification-behaviour.md#L4`](https://github.com/Google/WebFundamentals/blame/79342a44d9bccdca5a8c51c4e782087b8d02526a/src/content/en/fundamentals/engage-and-retain/push-notifications/notification-behaviour.md#L4) - WF Tag `wf_updated_on` must be within the last 7 days.
[`src/content/en/fundamentals/engage-and-retain/push-notifications/notification-behaviour.md#L138`](https://github.com/Google/WebFundamentals/blame/79342a44d9bccdca5a8c51c4e782087b8d02526a/src/content/en/fundamentals/engage-and-retain/push-notifications/notification-behaviour.md#L138) - Line must be at most 100 characters
[`src/content/en/fundamentals/engage-and-retain/push-notifications/notification-behaviour.md#L158`](https://github.com/Google/WebFundamentals/blame/79342a44d9bccdca5a8c51c4e782087b8d02526a/src/content/en/fundamentals/engage-and-retain/push-notifications/notification-behaviour.md#L158) - Line must be at most 100 characters
[`src/content/en/fundamentals/engage-and-retain/push-notifications/notification-behaviour.md#L171`](https://github.com/Google/WebFundamentals/blame/79342a44d9bccdca5a8c51c4e782087b8d02526a/src/content/en/fundamentals/engage-and-retain/push-notifications/notification-behaviour.md#L171) - Line must be at most 100 characters
[`src/content/en/fundamentals/engage-and-retain/push-notifications/notification-behaviour.md#L185`](https://github.com/Google/WebFundamentals/blame/79342a44d9bccdca5a8c51c4e782087b8d02526a/src/content/en/fundamentals/engage-and-retain/push-notifications/notification-behaviour.md#L185) - Line must be at most 100 characters
[`src/content/en/fundamentals/engage-and-retain/push-notifications/permission-ux.md#L4`](https://github.com/Google/WebFundamentals/blame/79342a44d9bccdca5a8c51c4e782087b8d02526a/src/content/en/fundamentals/engage-and-retain/push-notifications/permission-ux.md#L4) - WF Tag `wf_updated_on` must be within the last 7 days.
[`src/content/en/fundamentals/engage-and-retain/push-notifications/permission-ux.md#L48`](https://github.com/Google/WebFundamentals/blame/79342a44d9bccdca5a8c51c4e782087b8d02526a/src/content/en/fundamentals/engage-and-retain/push-notifications/permission-ux.md#L48) - Line must be at most 100 characters
[`src/content/en/fundamentals/engage-and-retain/push-notifications/permission-ux.md#L57`](https://github.com/Google/WebFundamentals/blame/79342a44d9bccdca5a8c51c4e782087b8d02526a/src/content/en/fundamentals/engage-and-retain/push-notifications/permission-ux.md#L57) - Line must be at most 100 characters
[`src/content/en/fundamentals/engage-and-retain/push-notifications/permission-ux.md#L62`](https://github.com/Google/WebFundamentals/blame/79342a44d9bccdca5a8c51c4e782087b8d02526a/src/content/en/fundamentals/engage-and-retain/push-notifications/permission-ux.md#L62) - Line must be at most 100 characters
[`src/content/en/fundamentals/engage-and-retain/push-notifications/permission-ux.md#L106`](https://github.com/Google/WebFundamentals/blame/79342a44d9bccdca5a8c51c4e782087b8d02526a/src/content/en/fundamentals/engage-and-retain/push-notifications/permission-ux.md#L106) - Line must be at most 100 characters
[`src/content/en/fundamentals/engage-and-retain/push-notifications/permission-ux.md#L111`](https://github.com/Google/WebFundamentals/blame/79342a44d9bccdca5a8c51c4e782087b8d02526a/src/content/en/fundamentals/engage-and-retain/push-notifications/permission-ux.md#L111) - Line must be at most 100 characters
[`src/content/en/fundamentals/engage-and-retain/push-notifications/permission-ux.md#L115`](https://github.com/Google/WebFundamentals/blame/79342a44d9bccdca5a8c51c4e782087b8d02526a/src/content/en/fundamentals/engage-and-retain/push-notifications/permission-ux.md#L115) - Line must be at most 100 characters
[`src/content/en/fundamentals/engage-and-retain/push-notifications/permissions-subscriptions.md`](https://github.com/Google/WebFundamentals/blame/79342a44d9bccdca5a8c51c4e782087b8d02526a/src/content/en/fundamentals/engage-and-retain/push-notifications/permissions-subscriptions.md) - Unable to read file, was it deleted?
[`src/content/en/fundamentals/engage-and-retain/push-notifications/sending-messages-with-web-push-libraries.md#L4`](https://github.com/Google/WebFundamentals/blame/79342a44d9bccdca5a8c51c4e782087b8d02526a/src/content/en/fundamentals/engage-and-retain/push-notifications/sending-messages-with-web-push-libraries.md#L4) - WF Tag `wf_updated_on` must be within the last 7 days.
[`src/content/en/fundamentals/engage-and-retain/push-notifications/sending-messages.md`](https://github.com/Google/WebFundamentals/blame/79342a44d9bccdca5a8c51c4e782087b8d02526a/src/content/en/fundamentals/engage-and-retain/push-notifications/sending-messages.md) - Unable to read file, was it deleted?
[`src/content/en/fundamentals/engage-and-retain/push-notifications/subscribing-a-user.md#L4`](https://github.com/Google/WebFundamentals/blame/79342a44d9bccdca5a8c51c4e782087b8d02526a/src/content/en/fundamentals/engage-and-retain/push-notifications/subscribing-a-user.md#L4) - WF Tag `wf_updated_on` must be within the last 7 days.
[`src/content/en/fundamentals/engage-and-retain/push-notifications/web-push-protocol.md#L4`](https://github.com/Google/WebFundamentals/blame/79342a44d9bccdca5a8c51c4e782087b8d02526a/src/content/en/fundamentals/engage-and-retain/push-notifications/web-push-protocol.md#L4) - WF Tag `wf_updated_on` must be within the last 7 days.
[`src/content/en/fundamentals/engage-and-retain/push-notifications/web-push-protocol.md#L456`](https://github.com/Google/WebFundamentals/blame/79342a44d9bccdca5a8c51c4e782087b8d02526a/src/content/en/fundamentals/engage-and-retain/push-notifications/web-push-protocol.md#L456) - Line must be at most 100 characters


To me it looks like this line is missing, at least, a closing curly bracket -> }


is this stab for testing purposes?


forcing choices. incorrect


handled incorrectly


Did I? I thought I didn't and anyway I wanted to fill up some of the missing docs. I don't care either way.


not handled at all


Change jQuery position


handled incorreclty


Didn't commit originally


@Montellese includes wss:


Put here one more <li> element - for username inserting


Ugh.... Anyway I'll add revision prop to every repo to avoid any problem, gimme a sec


@Calvarez14 - your copy pasting cost you a lot of points. you propagated the same errors over and over and over. remember when I said "dont copy paste"? Here is a perfect example why


Documenting the diagnosis and the fix on GitHub is one thing we could do to communicate understanding within the team. Better of course would be to have something in the code.

I'm personally interested in understanding the assumptions / guarantees of each phase of the compiler and it would take me more than a few minutes to understand this issue. I just looked and couldn't find any discussion of invariants in LocalCopyProp{h.cpp}. The .h file has no comments at all and the .cpp file doesn't discuss any properties, unless you meant this (https://github.com/p4lang/p4c/blob/master/midend/local_copyprop.cpp#L215)?

I see that moveConstructors.cpp has a nice description of what it does (https://github.com/p4lang/p4c/blob/master/midend/moveConstructors.cpp#L32) but I don't see how this is related to the `pred` variable introduced by predication...


`else` after return!


Looks like we implemented the same _polishing_. Now I'll have to rebase and refactor. 😉


// https://openclipart.org/search/json/?query=elephants - query changed from unicorn

// CLASS 3 CODE

var elephants; // originally was unicorns; changed all unicorns in code to elephants

function loadJSON(file, callback) {
  var xobj = new XMLHttpRequest();
    xobj.overrideMimeType("application/json");
  	xobj.open('GET', file, true);
  	xobj.onreadystatechange = function () {
    	if (xobj.readyState == 4 && xobj.status == "200") {
     // Required use of an anonymous callback as .open will NOT return a value but simply returns undefined in asynchronous mode
     callback(xobj.responseText);
    }
  };
  xobj.send(null);
}

function getRandomThing(array) {
  var randomNumberBetween0And1 = Math.random();
  var highestNumber = array.length;
  var randomInteger = Math.floor(
    randomNumberBetween0And1 * highestNumber
  );
  return array[randomInteger];
}

function getImage() {
  var img = document.createElement('img');
  var elephant = getRandomThing(elephants);
  img.src = elephant.svg.url;
  return img;
}

function handleClick(event) {
  if (event.target.childElementCount > 0) {
    return;
  }
  var today = new Date().getDate();
  if (event.target.innerHTML > today) {
    return;
  }
  var img = getImage();
  event.target.appendChild(img);
}

function getDay(num) {
  var day = document.createElement('div');
  day.classList.add('day');
  day.innerHTML = num;
  day.addEventListener('click', handleClick);
  return day;
}

function drawCalendar() {
  var container = document.getElementsByClassName('container')[0];
  var year = new Date().getFullYear();
  var month = new Date().getMonth() + 1;
  var daysInMonth = new Date(year, month, 0).getDate();

  for (var i = 1; i <= daysInMonth; i ++) {
    var day = getDay(i);
    container.appendChild(day);
  }
}

function getElephantsThenDrawCalendar() { // originally getUnicornsThenDrawCalendar
  loadJSON('elephantsData.json', function(data) { // originally 'unicornData.json'
    elephants = JSON.parse(data).payload;
    drawCalendar();
  });
}

getElephantsThenDrawCalendar();


Let's move stuff regarding to the clearing of local storage and config object to the main controller.


Asked HanziQ in a twitch chat for info and an example a day or two before the update went live so I could update this library!


@Montellese , comments ? This change keeps the correct selected item rather than punting back to top item.


not handled correctly


(au cas où tout ce code est encore dans general.html)


@Montellese , thoughts ? I just could not figure out why the item text would change, so kicked back to a hammer.


`truckDriveChoice` might now have been initialized if exception was thrown


youre forcing a choice. incorrect


not handled properly


This is a quote from local_copyprop.h:

```
/* Local copy propagation and dead code elimination within a single action function.
   This pass is designed to be run after all declarations have received unique
   internal names.  This is important because the locals map uses only the
   declaration name, and not the full path.

   Requires expression types be stored inline in the expression
   (obtained by running Typechecking(updateProgram = true)).
   Requires that all declaration names be globally unique
   (obtained by running UniqueNames)
 */
```

The pass we are talking about is MoveDeclarations, not MoveConstructors.

The LocalCopyProp also requires declarations at the top-level control scope - that is not documented, which is what MoveDeclarations does.


Постараться уйти от этого приведения типов. См. Nash, Accelerated C#, page 329 (Ограничения)


Oh, right, because you had the one bug. We good here. 👍


youre forcing a spurious choice


@xboxfanj Would we just merge our closed vendor/oneplus into this? Or how would we handle it?


could be `numOfOptions` from previous choice since you didn't handle the exception properly


do not leave in youre unwanted messes. why do i want them in my repo?


`0 - 1` is `-1`, so if prompter threw an exception, thats what youre trying to get. crashes application


spurious used previous choice if this prompter throws an exception


forcing a spurious choice


TeamCity libpalaso :: palaso-win32-master Continuous [Build 4.0.16.ebedcbd3ad27e8953016314ab66c2859b60b0959](http://build.palaso.org/viewLog.html?buildId=88823&buildTypeId=Libpalaso_PalasoWin32masterContinuous) outcome was **FAILURE**
Summary: Compilation error: SIL.Windows.Forms\SIL.Windows.Forms.csproj; compilation error: SIL.DblBundle\SIL.DblBundle.csproj Build time: 00:00:38


again, you could be accessing position -1


again, you could be accessing a previous choice if this prompter failed.


got it fixed. Yes thank you for pointing it out.


since you copied and pasted classes, you propagated **all** the same errors. please reference your long horn builder for the same exact issues.


暂时注释，不用经过服务器可自动登录


@SilentImp, ап


Sigh. I was looking in the midend directory and interspersed the names.

Thanks. That helps.


:tada: :smile:


Are these resolved now? Last I checked, it looks like this, still:

<img width="888" alt="screen shot 2017-04-26 at 12 03 28 pm" src="https://cloud.githubusercontent.com/assets/1310293/25446856/82a70e32-2a78-11e7-9752-78ed5b89eec4.png">


TeamCity libpalaso :: palaso-win32-libpalaso-3.1 Continuous [Build 3.1.293.ebedcbd3ad27e8953016314ab66c2859b60b0959](http://build.palaso.org/viewLog.html?buildId=88824&buildTypeId=bt223) outcome was **SUCCESS**
Summary: Tests passed: 3587, ignored: 128 Build time: 00:03:45


You propagated all the same errors with your lonestar builder. this is because you copy and pasted. so please reference that class for issues in this one.


@kopacki I've found that the preset works with this code but the package didn't seem to update with the changes when I tried to do `yarn upgrade neutrino-preset-sass`. Wasn't sure if I did it wrong or not. Has the package been updated yet? Thanks in advance.


was missing


TeamCity libpalaso :: palaso-precise64-libpalaso-2.6 Continuous [Build 2.6.1287.ebedcbd3ad27e8953016314ab66c2859b60b0959](http://build.palaso.org/viewLog.html?buildId=88820&buildTypeId=PalasoPrecise64v26Cont) outcome was **SUCCESS**
Summary: Tests passed: 3565, ignored: 111 Build time: 00:06:02


arses! I fixed that up in the commit I rolled back then forgot about it :(.


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 4.0.15.ebedcbd3ad27e8953016314ab66c2859b60b0959](http://build.palaso.org/viewLog.html?buildId=88826&buildTypeId=Libpalaso_PalasoPrecise64masterContinuous) outcome was **FAILURE**
Summary: Failed to resolve artifact dependency Build time: 00:00:05


you propagated all your errors from copy and pasting. because this is your extra credit, you make up some of your other classes issues, but still suffer the same catastrophic point losses. *DO NOT COPY/PASTE... EVER*


@lbjay I verified in my dev env that changing the config name makes Karaf 4.0.8 use that param. Putting the pid file in the karaf data directory is consistent with what it does with the  karaf port file.
https://bitbucket.org/opencast-community/matterhorn/pull-requests/1488/mh-12188-changed-karaf-pid-config-param/diff


I found it more practical. For example, if you'd want to do it from a velocity script, you can easily access the `XWikiContext` and set the batchId before calling `deleteDocument` for a number of references.

How easy is it for a script (velocity, maybe others?) to access and write to the ExecutionContext with the current API?

On the other hand, it could just be a bad habit on my side, but at least it's easy to change inside the `DefaultModelBridge` implementation.

Could you please list some quick pros and cons on the 2 approaches? It might help me better understand when to use one and when to use the other.


spurious forced selection


not handled properly


forced selection


not handled AT ALL


ᕕ( ᐛ )ᕗ


@nourelhoda89 -  obviously you created this one and then let your team copy/paste off of you. all the same errors are propagated throughout the whole source code and in the same exact places.


fixes #34


these variance numbers aren't accurate anymore with the unified model (predicting whole trajectory rather than AoA)


TeamCity libpalaso :: palaso-precise64-libpalaso-3.1 Continuous [Build 3.1.245.ebedcbd3ad27e8953016314ab66c2859b60b0959](http://build.palaso.org/viewLog.html?buildId=88829&buildTypeId=bt322) outcome was **SUCCESS**
Summary: Tests passed: 3565, ignored: 111 Build time: 00:05:51


TeamCity libpalaso :: palaso-trusty64-lfmerge Continuous [Build 2.6.278.ebedcbd3ad27e8953016314ab66c2859b60b0959](http://build.palaso.org/viewLog.html?buildId=88825&buildTypeId=PalasoTrusty64lfmergeContinuous) outcome was **SUCCESS**
Summary: Tests passed: 3565, ignored: 111 Build time: 00:05:48


Did you mean to comment out the `ranlib` call here?


not sure whether/how to make a similar point


again, same errors. same places. **DO NOT COPY/PASTE ... EVER*


nope, this is on `untilOther`, that we decided to keep for now (even though it is probably a bit too convoluted and the new `delayUntilOther` should be used in most cases)


Thanks. Didn't know about it


Great!  Sorry about that.  We run multiple instances of edal-java as well,
but they are each in separate Tomcat instances.

On Wed, Apr 26, 2017 at 8:29 AM, Guy Griffiths <notifications@github.com>
wrote:

> Works like a charm, thanks. I've cherry-picked it into our develop branch.
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/axiom-data-science/edal-java/commit/c6a13333947017c794ad2af4815c9d8723949055#commitcomment-21916940>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AAJ6NyBt6MJathuDdjN90fBJ9Gl03g-kks5rz2LRgaJpZM4NJCRH>
> .
>


Scripts criado com linguagem Python


The following doc(s) have been deployed to the [2310backup](https://2310backup-static-docs.pantheonsite.io/docs) Multidev environment:
- [docs/backups](https://2310backup-static-docs.pantheonsite.io/docs/backups)
- [source/docs/assets/images/backup-progress.png](https://github.com/pantheon-systems/documentation/commit/ee21a106e4090cde1d73b51349010ce56946a7be/source/docs/assets/images/backup-progress.png)


> I found it more practical. For example, if you'd want to do it from a velocity script, you can easily access the XWikiContext and set the batchId before calling deleteDocument for a number of references.

That's only if you have programming right so I don't really see this as a feature.

> How easy is it for a script (velocity, maybe others?) to access and write to the ExecutionContext with the current API?

I don't think there is any generic API for this but you could add a method dedicated to setting the batch id in the refactoring script service if you really need that.

> Could you please list some quick pros and cons on the 2 approaches? It might help me better understand when to use one and when to use the other.

PROS ExecutionRequest:
* this is the standard way to setting contextual stuff, XWikiContext is just legacy
* no need to go tough any "special bridge because I don't have access to oldcore here", you just use Execution component


"PORT" is probably a fairly common env var, to avoid potential conflicts, so I think it makes sense to make this a flag instead, and start the server with `./commento --port=8080` or `./commento -p 8080` instead.


not handled


youre forcing a choice by you not handling the exception. there is "no default" if an exception is thrown


again, forcing a choice


@roblourens yes, that makes sense!


@gregw can you review this? is this sane from a Filter point of view?


not handled properly


Sounds fine.


This looks good, I might use it :)


ok. not going to mark every bad exception handling and forced choices. you basically said that if you, then developer, used the prompter wrong, you will ignore the exception. problem is, if the exception is thrown, the user chioce is lost, except, you choose already, hence they get your choice. not good


[Edit] Soucre Code Version 0.5


**Whoops!**

There were *2 warnings* that will prevent this PR from being merged. Please take a look, and either fix, or provide a justification for why they can't be fixed.

**WARNINGS**
[`src/content/en/updates/2017/03/chrome-58-deprecations.md#L5`](https://github.com/Google/WebFundamentals/blame/5cba6ebbdb10e85d7899ad6afcf003fdc61f1b8f/src/content/en/updates/2017/03/chrome-58-deprecations.md#L5) - WF Tag `wf_updated_on` must be within the last 7 days.
[`src/content/en/updates/2017/03/chrome-58-deprecations.md#L148`](https://github.com/Google/WebFundamentals/blame/5cba6ebbdb10e85d7899ad6afcf003fdc61f1b8f/src/content/en/updates/2017/03/chrome-58-deprecations.md#L148) - Updates post is missing comment widget: `{% include "comment-widget.html" %}`


确认


forced choice


just want to write and see what happens next


nice catch


**good**


And basic python (CSE, the "computer science" course)


Ich würde sogar das margin-left wegnehmen. Das Leerzeichen dazwischen reicht.


please read your commit names and tell me where I am to grade? All semester I have asked for commit names that have meaning. "Add files via upload" has no meaning. "Adds SXTBuilder" has meaning. It is nearly impossible for me make grading comments as is. I will do my best. So if you have questions, you probably will have to see me.


This opaque wrapper can't actually be totally opaque, because we need to allow access to cross-origin accessible properties. We need to duplicate https://dxr.mozilla.org/mozilla-central/source/js/xpconnect/wrappers/FilteringWrapper.h#22 and https://dxr.mozilla.org/mozilla-central/source/js/xpconnect/wrappers/FilteringWrapper.h#62. Together those inherit from https://dxr.mozilla.org/mozilla-central/source/js/xpconnect/wrappers/XrayWrapper.h#530 which is at least a type that is defined by SpiderMonkey for us: https://dxr.mozilla.org/mozilla-central/source/js/xpconnect/wrappers/XrayWrapper.h#446


FixZoneVolumeCalcs (JasonGlazer) - x86_64-MacOS-10.9-clang: OK (2463 of 2496 tests passed, 195 test warnings)

Messages:

 * 151 tests had: EIO diffs.
 * 21 tests had: ESO big diffs.
 * 30 tests had: Table big diffs.
 * 198 tests had: ESO small diffs.
 * 160 tests had: MTR small diffs.
 * 104 tests had: Table small diffs.
 * 3 tests had: SSZ big diffs.
 * 3 tests had: ZSZ big diffs.
 * 8 tests had: ERR diffs.
 * 10 tests had: MTR big diffs.
 * 8 tests had: ZSZ small diffs.
 * 3 tests had: SSZ small diffs.

Failures:

integration Test Summary
 * Passed: 596
 * Failed: 2

regression Test Summary
 * Passed: 565
 * Failed: 31

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


@SeanNaren In Python 3 (which I'm using -- have a couple other misc issues in the ted data loader related to that), filter returns an iterator instead of a list (as in Python 2), which causes this to blow up later on when `len()` is called on `transcript`.  This is pretty easily worked around by adding an additional `list()` call, i.e.
```python
transcript = list(filter(None, [self.labels_map.get(x) for x in list(transcript)]))
```
which is what I am testing with now.


@LegitSoulja what is pc?
i would like to learn to help out sometime =/


@joakime LGTM.


Since we don't technically own that file - yes please leave it in tact. I can't hardly see what was changed otherwise.


![](https://github.trello.services/images/mini-trello-icon.png) [As a WM, I want to be able to search prospect records so that I can access them quickly [1 day]](https://trello.com/c/Ny9PTwXi/12-as-a-wm-i-want-to-be-able-to-search-prospect-records-so-that-i-can-access-them-quickly-1-day)


@miguelcleon This looks good


Program to add SystemInfo


Finde ich nicht, ich wollte das margin eher erhöhen. Die Glocke hebt sich zu wenig vom Rest ab, das Auge sucht zuviel

Via Mobile, deshalb kurz. Nicht unhöflich gemeint!

> Am 26.04.2017 um 19:24 schrieb MatthiasApsel <notifications@github.com>:
>
> Ich würde sogar das margin-left wegnehmen. Das Leerzeichen dazwischen reicht.
>
> —
> You are receiving this because you are subscribed to this thread.
> Reply to this email directly, view it on GitHub, or mute the thread.
>


Program to display information about current python version


Please don't consider to make it well-formed. It would be too easy to parse.


Можно было просто написать содержит рекламу https://forum.ru-board.com/topic.cgi?forum=5&topic=48272&start=2700#16


It's a purple Sphere, of course! Literally, no other reason than that... and just because I like ANSI.


This commit introduces the new pcl::DrizzleData class, which implements serialization/deserialization of XML drizzle data files (XDRZ format, .xdrz file suffix). XDRZ files will be used for all image registration and integration tasks since the next version 1.8.5 of PixInsight, and the old plain text format will only be supported for compatibility with existing .drz files (read-only support).

The new XDRZ format is faster, does not have file buffering problems on Windows, and supports Bayer drizzle directly from monochrome CFA frames. During the next days I'll be upgrading all tools involved in drizzle tasks: StarAlignment (still not open-source), CometAlignment, ImageIntegration and DrizzleIntegration.


okay 4 additional comment


The following section deals with the OBIS formatting in case F value is included


AFN2017 (jasondegraw) - x86_64-MacOS-10.10-clang: Build Failed

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


Bluemix [toolchain](https://console.ng.bluemix.net/devops/toolchains/64ac4856-ca8a-4cca-8fab-dfaef1cd503e "simple-toolchain-20170426172315300"): [Delivery Pipeline](https://console.ng.bluemix.net/devops/pipelines/c4dfb98e-bde8-43c5-a393-5d951e513ba4 "simple-toolchain-20170426172315300") deployed [simple-toolchain-20170426172315300](https://github.com/BluemixLabs4/simple-toolchain-20170426172315300/releases/tag/deploy-ibm-yp-us-south-bluemixlabs4-gmail.com-dev-20170426-173657 "deploy-ibm-yp-us-south-bluemixlabs4-gmail.com-dev-20170426-173657") to [dev](https://console.ng.bluemix.net/apps/d5678c38-82c0-4fe0-8488-135dfd7dee09 "ibm:yp:us-south:bluemixlabs4@gmail.com:dev"), including this commit


These are new OBIS codes used by Iskra. Currently only active power is available but more should become available once meter has been re-parameterised.


It'd be great if you had configuration of your WS


Il s'agit du nombre de noms pas de taxons ?


Custom nightly build resources


E.g. using CXF + Spring


TeamCity Code :: 1. Development :: DDK :: Feature [Build 261](http://mp-ci.centralus.cloudapp.azure.com/viewLog.html?buildId=21446&buildTypeId=Code_Development_Ddk_Feature) is now running


Thanks! BitHub has sent payment of  $63.08USD for this commit.


This annotations should be present on particular methods that need them


Да я его и не ставил. Так, мимо проходил.


undefined attempts and max_start_attempts


Thanks @OctoNezd but I mean, what is this commit in general because it reads as "merge branch" and I haven't done it xD
Anyway, it's fine :P


Oui on avait dit que la modification des listes et leur contenu est seulement pour les droits 4 et plus


Is this really necessary?  "my shifts" will no, no? xD


Where is @Path annotation?
Input argument should be Document class, not String


You can configure your service to marshall/unmarshall entities automatically


Hi ciarams87,
i used multiple_arrays_example it work great.
our environment have 160 VMAX  and 20 Unispehre this script is great.
need to learn python and adapt some of your scripts
once again thanks


WS should be stateless. You have state here - doc


TeamCity Code :: 1. Development :: DDK :: Feature [Build 261](http://mp-ci.centralus.cloudapp.azure.com/viewLog.html?buildId=21446&buildTypeId=Code_Development_Ddk_Feature) outcome was **SUCCESS**
Summary: Tests passed: 59 Build time: 00:02:25


Something changed in Clang 4 but I was able to use `-Wno-invalid-partial-specialization` (https://clang.llvm.org/docs/DiagnosticsReference.html#winvalid-partial-specialization) to workaround the error:
- https://github.com/bitshares/bitshares-fc/compare/6d386e442b57f4da6099ebd934e071a73d46f398...453b3de77efeed3d159c67b9eaa9e0595e0f9091
- https://github.com/bitshares/bitshares-core/compare/365987e77fede5c78bd50dbe4d603577f0a02f06...7d77cbf15be8fbd89ed1d04f0ae8106984e19772


neeej hvor fint


The following doc(s) have been deployed to the [2310backup](https://2310backup-static-docs.pantheonsite.io/docs) Multidev environment:
- [docs/backups](https://2310backup-static-docs.pantheonsite.io/docs/backups)
- [source/docs/assets/images/backup-progress.png](https://github.com/pantheon-systems/documentation/commit/ee21a106e4090cde1d73b51349010ce56946a7be/source/docs/assets/images/backup-progress.png)


How should this work?


C++ still seems like the common language between us.
Oh, just to warn you. My coding style might be, foreign.
The Dyalog Student Competition just launched. APL's array oriented, which can lead to really weird programming when translated to other languages.


should this be [0,4] instead of [0,5] ?


Why do you have method for deleting the 1st chapter of document?
You must have method for deleting all Document’s Chapters


Did you mean to leave this sentence in?  It looks like it's made redundant by the next one.


Looks good.


Done by Pablo Cabezali in Maria´s laptop


Your client project can't be run because of wrong project structure (no pom.xml)


@evanrivera  - you have the same issues in rt as you do sxt and sxt plus. So please refer to those issues created for whats wrong with rt. Plus, you have no uml at the moment, so things like engine are not the same for rt as they are for sxt and sxt plus. This class is 100% incorrect because you copy/pasted the sxt and changed string values.


You have some public methods for get/post/delete etc. operations
But you should have WS's methods encapsulated in client


Approved


Bluemix [toolchain](https://console.ng.bluemix.net/devops/toolchains/64ac4856-ca8a-4cca-8fab-dfaef1cd503e "simple-toolchain-20170426172315300"): [Delivery Pipeline](https://console.ng.bluemix.net/devops/pipelines/c4dfb98e-bde8-43c5-a393-5d951e513ba4 "simple-toolchain-20170426172315300") deployed [simple-toolchain-20170426172315300](https://github.com/BluemixLabs4/simple-toolchain-20170426172315300/releases/tag/deploy-ibm-yp-us-south-bluemixlabs4-gmail.com-dev-20170426-174450 "deploy-ibm-yp-us-south-bluemixlabs4-gmail.com-dev-20170426-174450") to [dev](https://console.ng.bluemix.net/apps/e7e40802-2a7e-4cba-8f62-eed48dd9da90 "ibm:yp:us-south:bluemixlabs4@gmail.com:dev"), including this commit


Some of your client's methods don't work - like this one


Bluemix [toolchain](https://console.ng.bluemix.net/devops/toolchains/64ac4856-ca8a-4cca-8fab-dfaef1cd503e "simple-toolchain-20170426172315300"): [Delivery Pipeline](https://console.ng.bluemix.net/devops/pipelines/c4dfb98e-bde8-43c5-a393-5d951e513ba4 "simple-toolchain-20170426172315300") deployed [simple-toolchain-20170426172315300](https://github.com/BluemixLabs4/simple-toolchain-20170426172315300/releases/tag/deploy-ibm-yp-us-south-bluemixlabs4-gmail.com-dev-20170426-174617 "deploy-ibm-yp-us-south-bluemixlabs4-gmail.com-dev-20170426-174617") to [dev](https://console.ng.bluemix.net/apps/3e42f053-1686-4e6b-9937-23b9bd2431df "ibm:yp:us-south:bluemixlabs4@gmail.com:dev"), including this commit


for your rt plus build, it is again the sxt with string values changed. because of so, it is not the rt plus and 100% incorrect.


updated.


Новый модуль


Towards #10


These 2 methods shouldnt be public


Live Project


Теперь в словаре вопросы это Тюплы так удобнее в разы


这么屌的 commit，什么时候发版本啊


тут обрати внимание, теперь с ним можно немного поговорить, а не просто шаблонные вопросы=)


Ready bug = no changes in order combos.
Fixed by adding trigger("change") in ready
fill_order is no needed anymore


Thanks da3dsoul :)

I need to remove the pri.longpath nuget, and add jmm/pri.longpath as a module.


Thanks for commenting.  This file is currently deprecated but we can remove those two lines over here:

https://github.com/dvcs/gitignore/blob/53e4705511dd7fc391deffd452d9095df3496cde/templates/Code.gitignore


PlayCanvas


No, it's not necessary, but it looks better when personalised and is added with a simple

> filename="shifts-with-" + session::get('userName') + ".ics"

So why not?


@sertal70 sorry this fix was supposed to be reverted, please use an older version for now until I publish the next release (soon).


```Lua
testVar = 0x100500
```

![obraz](https://cloud.githubusercontent.com/assets/9157726/25448683/8eb6980a-2ab9-11e7-9da4-09ec66ba980f.png)

Now, we have a problem. We do not know when offset will be treated as hexadecimal and when not.

```Asm
[ENABLE]
$testVar+50:
db 'a test'

[DISABLE]
```
It will write at 00100532


"easily" not "more easily"


@giuliavezzani ❓
the code style is not uniform regarding the use of brackets!
What about tabs vs. spaces or even different number of spaces?

I suggest you use a beautifier.


в общем мы стремимся к идее «один тест = 1 expectation». так как сейчас — просто не читается («it is expected. is expected», почему вы повторяете, вы повторяете, я уже понял, уже понял). то есть можно попробовать (я навскидку не помню, что нормально будет работать, а что нет, но это норм):
```ruby
# вариант 1
it do
  is_expected.to all (
    be_a(Cinema::NewMovie)
    .and have_attributes(period: :new, genre: array_including('Sci-Fi'))
  )
end
# вариант 2
it do
  is_expected
    .to all(be_a(Cinema::NewMovie))
    .and all(have_attributes(period: :new, genre: array_including('Sci-Fi')))
end
# вариант 3, если ничего не помогает!
it  { is_expected.to all be_a(Cinema::NewMovie) }
it  { is_expected.to all have_attributes(period: :new, genre: array_including('Sci-Fi')) }
```

Хороший способ выбрать самый лучший вариант — использовать `rspec --format doc` и выбрать то, что выводит наиболее осмысленное описание наших ожиданий.


As Dan pointed, we remove ov even if the refcount is not zero. Eventually, if the code is right, the refcount should go to zero and it should be freed there. If there is a bug where there is a ov->refcnt leak, then neither the old code nor the new code is going to free it. So there is nothing wrong removing ov whose refcount is not zero here from drc.


ahh, ok. so my fix wasn't even really fixing it. good catch.


looks good! `name.split.map { |i| i.capitalize }.join(' ')` could work if you want to do a one-liner :) But yours works, too! And is probably more readable?


I'm not sure about how these examples relate to the text.


This will never be shown to the user as the @error will always be overwritten by the line below, as the check_column will fail


Any change we get a new release  with that stuff?


@fichtner nice!


"Now you can push changes and the get built tested and deployed." > "Now you may push changes, which will trigger the build and deployment of the codebase." @edewit


I guess this is supposed to read "not available [everywhere]"?


@giuliavezzani 😲
the local `t0` hides `this->t0`? Is this wanted?


Э, ну я ж по памяти пишу. Предполагается, что ты всё время смотришь в rspec matchers: https://www.relishapp.com/rspec/rspec-expectations/docs/built-in-matchers и ищешь подходящее по смыслу. То что тут на самом деле было надо — `satisfy` https://www.relishapp.com/rspec/rspec-expectations/v/3-5/docs/built-in-matchers/satisfy-matcher


FixZoneVolumeCalcs (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2502 of 2536 tests passed, 197 test warnings)

Messages:

 * 139 tests had: EIO diffs.
 * 22 tests had: ESO big diffs.
 * 31 tests had: Table big diffs.
 * 200 tests had: ESO small diffs.
 * 160 tests had: MTR small diffs.
 * 90 tests had: Table small diffs.
 * 3 tests had: SSZ big diffs.
 * 3 tests had: ZSZ big diffs.
 * 7 tests had: ERR diffs.
 * 10 tests had: MTR big diffs.
 * 7 tests had: ZSZ small diffs.
 * 4 tests had: SSZ small diffs.

Failures:

integration Test Summary
 * Passed: 599
 * Failed: 2

regression Test Summary
 * Passed: 584
 * Failed: 32

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


![](https://github.trello.services/images/mini-trello-icon.png) [Repositório](https://trello.com/c/Hu9t8DqS/17-repositorio)


@pdiwadkar Can you please update the README with instructions to build, deploy and run this code.  Thank you.

CC/ @sreekantho, @inertialbit


Erf, http://cyos.babylonjs.com show "Not Found" content here :/


@giuliavezzani
either you use always `true` or always `on` to avoid confusion.


1. location - I would have it match a path like bigquery's - https://github.com/apache/beam/blob/master/sdks/java/io/google-cloud-platform/src/main/java/org/apache/beam/sdk/io/gcp/bigquery/BigQueryIO.java

3. cleaning up files - yeah, that can be tricky. That's a good question to ask on the mailing list.


This looks good!


There are a bunch of open questions on the design doc - I'd like to make sure we answer those before I spend much more time on the code


Hi, think parse_url($requestUri, PHP_URL_PATH);  would be the better variant because of NPath Complexity


`filtered_or_not` is not a proper name 😉
The standard is to term a variable with a tag that is meaningful when it's true.


Yeap! It uses the SysEx Midi range to control whatever you want inside the keyboard - it's still a work in progress right now, but eventually it'll be called the API, and probably won't use Midi :)


I'll fix it by having the merge script use a different branch for the manifests folder. Thanks.


oui !


FixZoneVolumeCalcs (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


This should fix #110


@giuliavezzani
that doesn't make sense: if `radiusValue` is out of bounds, then threshold it at the limits.


This breaks module https://github.com/Icinga/puppet-icinga2-legacy

Error: Facter: error while resolving custom fact "java_version": undefined method `with_env' for Facter::Util::Resolution:Class
Error: Could not retrieve catalog from remote server: Error 500 on SERVER: Server Error: Evaluation Error: Error while evaluating a Resource Statement, Evaluation Error: Error while evaluating a Function Call, Duplicate declaration: Icinga2::Config::Objectdir[apiusers] is already declared in file /etc/puppetlabs/code/environments/production/modules/icinga2/manifests/object/apiuser.pp:24; cannot redeclare at /etc/puppetlabs/code/environments/production/modules/icinga2/manifests/object/apiuser.pp:24 at /etc/puppetlabs/code/environments/production/modules/icinga2/manifests/object/apiuser.pp:24:3 at /etc/puppetlabs/code/environments/production/modules/icinga2/manifests/objects.pp:15 on node XXX

I'm currently running Puppet 4.8.2, stdlib from GIT as well as puppet-icinga2-legacy module from Git.

in line 24 of the apiuser.pp I have just:
ensure_resource('icinga2::config::objectdir', 'apiusers')

reverting stdlib to just before the merge, let it work again for me.


No


dont'use `cout`, stick to `yInfo` and other similar routines.


Ugh -- I inserted that because without it, I got duplicate symbols for log2 with the MSVC+Bazel build. I'll make it an explicit namespace with explicit qualification.


@tonybruess For some reason I couldn't fork [Your repository](https://github.com/tonybruess/tonybruess.com) and pull request this commit...

```
The following changes since commit 9d92602c6d26bcdafac04785ced7c178dacea9e9:

  Update resume (2017-04-02 16:40:43 -0700)

are available in the git repository at:

  https://github.com/TheMolkaPL/tonybruess.com.git rip-project-ares

for you to fetch changes up to dba844687a9e351e19c321a4e5108f47f81956f6:

  oc.tc forums doesn't exist anymore (2017-04-26 19:43:48 +0200)

----------------------------------------------------------------
TheMolkaPL (1):
      oc.tc forums doesn't exist anymore

 index.html | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

```


To my knowledge, translators agreed to have no copyright on their translations for the installation folder. They can add own copyrights to their official language packs.
Normally, those files are posted on the translation forum and I pick them from there and merge them manually. You just don't see me fixing those headers then :)


Don't upset developers! :) i think it should be upsert aka shorthand for 'update-or-insert'


Hi @nidup , there is any reason to have all these locales hardcoded?


user-friendly


closes #3


indeed, fixed


/dev/sdb1 on Windows? Is this ok?


Can we ifdef this with an environmental var?


up-to-date


(to opt out)


Why is that?


This paragraph is missing a thesis.


Oops. It should be G:

On 26-Apr-2017 11:38 PM, "Alin Trăistaru" <notifications@github.com> wrote:

> /dev/sdb1 on Windows? Is this ok?
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/mbusb/multibootusb/commit/160e1a75d221d565b307a8264120b8652765be6c#commitcomment-21919953>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AEC34IFdW450KAMXIfILWJTBd7CfL2O1ks5rz4gvgaJpZM4NJOmV>
> .
>


test


home pictures


Yes, I agree. I will work on answering the open questions and arrive at the same page with all the folks before doing any code.


Just getting to looking at this. I would very strongly prefer that we do not duplicate configurations (such as these), and I think we should be cautious about making deeply-nested structures such as this as well. By and large, I've hoped to make a structure where there's top-level groupings (e.g. `irods`, `elasticsearch`, `kifshare`, etc.) but once you're into those folders in the consul UI, it's easy to look at everything relating to that at once, which having subfolders undermines. Outright duplicates are even more directly counterproductive: if one needed to update one of these properties ad-hoc, it would be necessary for them to know both locations (which, due to the nested structure, don't appear on the same page) and change them both in order to change a value in all applicable configurations.

So, I'd really prefer if the two `elasticsearch/indices/data/*` keys went away, and the preexisting `elasticsearch/data-index` and `elasticsearch/data-alias` keys used instead.


Vinod:


is this where rulers can go and like buy weapons and stuff? will the AI do it?


Curious why you approached it this way? This is not a restful route. Were you trying to combine all of the actions into one? I would stick with the Restful routes, but curious if you were getting an error and that is why you switched  it up?


Oh merci, j'ai pas fait gaffe !

On Apr 26, 2017 10:03 PM, "Kuntz Thomas" <notifications@github.com> wrote:

> nice .swp file son.
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/pallamidessi/annoying.js/commit/4fd8cc25311b66282677346427d6a5d0d8b60596#commitcomment-21915284>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/ABeOUn8T0OGzbOzf2ppmz3xrf-REgzsiks5rz06ygaJpZM4NI6Lv>
> .
>


Hi, i dont see function that fetches authors/commiters of a version as it's described in #37. Did you decide to skip that?


Actually the framework can do that for us. I will open a PR tomorrow.


The way we use the logger is very specific, which makes it a lot easier to identify the data associated with the logs. Debug only takes two arguments, a string and an object. Put the user and org ids in an object please.


Insta-shame of having someone looking at all the garbage code in my repo
(>~>)

On Apr 27, 2017 2:11 AM, "Joseph Pallamidessi" <
joseph.pallamidessi@gmail.com> wrote:

Oh merci, j'ai pas fait gaffe !

On Apr 26, 2017 10:03 PM, "Kuntz Thomas" <notifications@github.com> wrote:

> nice .swp file son.
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/pallamidessi/annoying.js/commit/4fd8cc25311b66282677346427d6a5d0d8b60596#commitcomment-21915284>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/ABeOUn8T0OGzbOzf2ppmz3xrf-REgzsiks5rz06ygaJpZM4NI6Lv>
> .
>


Never mind, get well soon!


it's much better to use C++ here:
```c++
std::string line;
std::getline(pointsFile,line);
```
or similar... double check


Yep, that was wrong.  We can collapse None == 0 I guess.


The following doc(s) have been deployed to the [2310backup](https://2310backup-static-docs.pantheonsite.io/docs) Multidev environment:
- [docs/backups](https://2310backup-static-docs.pantheonsite.io/docs/backups)
- [source/docs/assets/images/backup-progress.png](https://github.com/pantheon-systems/documentation/commit/fffdbb9c7714cd09a42a8c733dc703372930dccb/source/docs/assets/images/backup-progress.png)


old version was correct


Well done


А может нам просто перенести creatKeyboard а методы createFiguresButtonsPane и createOperationsButtonsPanel?


Is this right? type may equal `USB_TRANSACTION_IN`


DNS update is on its way


Modern `nixpkgs` have `haskellPackages.callCabal2nix` which does this.


hai git hub


What gradle task do I have to perform to make it download the dependency?


Thanks for fixing the actors Virg


Thanks man


I would expect that cocoapods gets the podspec from master branch.

Have you used `pod update`?


Change 1


You can use macros with yield. For example: https://github.com/galaxyproject/tools-iuc/blob/350ab347625ed5941873ba0deb3e1cf219d60052/tools/gff3_rebase/macros.xml


It's not right, that's why those are removed lines. :)


I believe the following should work:

```
if ( class_exists( 'WooCommerce' ) && is_product() ) {
    wc_get_template ( 'single-product-reviews.php' );
    return;
}
```

Any chance that this fix will be published to the WordPress repo anytime soon?


I have been wondering why do you still keep QMPlay2 as Qt4 compatible? when Qt4 has been EOL for a long time now


fixed in 087d54e6ab7216f94e0a8102946215e62a649f4b


I prefer "constants" not "constant variables" which is an oxymoron.


试试修改


Ok, I just didn't see that IntelliJ was asking me to approve auto update... Problem solved for me.


![](https://github.trello.services/images/mini-trello-icon.png) [Search Button - Add a button to the search bar](https://trello.com/c/6A3grK57/23-search-button-add-a-button-to-the-search-bar)


"build" should be sufficient if you are using IntelliJ. For Eclipse, you need to run "cleaneclipse eclipse" afterwards to update the classpath.

> On 26. Apr 2017, at 20:19, x-moe-x <notifications@github.com> wrote:
>
> What gradle task do I have to perform to make it download the dependency?
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub, or mute the thread.
>


@Andrews54757 I found a weird issue with the plugins.

For some reason, it executes the plugin 3 times.

How I found out? Accidentally using console.log. using main log, output once as normal. I've did some testing with my API plugin, and 1 request sent via minirequest is sent 3 times, which makes me think the plugin is being loaded 3 times? Also tried adding an object to data, to check for a type to ignore loading another instance of itself, however that doesn't work, so I don't know why it's executing more than once.


Thanks @3noch btw this is me and @andrewthad's successor to servant


you'll notice how close each of these diagonal checks are, and how close the code looks to each other.

are there any easy tricks you can think of that would reduce the duplication here?


I am going to disable Travis caching for now, since that didn't work correctly without shrinkwrap.


You are crazy :) But hey, this is not a big deal actually. It's not that we want the bar to be drawn directly on top of the actor, it's okay as it is now. The only thing I would like to have (as it shows the player differences between "strong enemies" (+BOSS) and "real important bosses" (+SUPERBOSS e.g.) by a different background graphic for the HUD enemy health display.


<img src="https://cdn.frankerfacez.com/emoticon/128585/4" alt="tooDank">


@mtnbarreto
Yeah 'pod update' would've probably worked. I just set it up to pull directly from master. Thanks


ага, проверила, вариант 1 и вариант 2 оба вполне читаемые, но мне первый больше нравится


if you want to disable interpretation of meta-characters use: "@^\Q{$match}\E$@"

It's safer to use this anyway with untrusted output.


All the methods below should be public and client should be a separate app that you will give to user of your service


Associating with #3  with this commit.


Don't know how so many changes have occured.....


My future CTO, you're so cute 💟 💟 💟


You should override chapters variable to be called "chapter" for Jax-B.
Now you have
<Documents>
    <Document>
        <chapters>
            <id>1</id>
            <number>2</number>
            <pagesCount>3</pagesCount>
        </chapters>
        <chapters>
            <id>2</id>
            <number>4</number>
            <pagesCount>4</pagesCount>
        </chapters>
        <chapters>
            <id>3</id>
            <number>5</number>
            <pagesCount>7</pagesCount>
        </chapters>
        <id>0</id>
        <name>One</name>
    </Document>
</Documents>


FixZoneVolumeCalcs (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1302 of 1302 tests passed, 0 test warnings)

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


After updating Joomla 3.7 in module mod_news_pro_gk5_J! 3. Settings have stopped working - Data source and Basic setting ..
Is it possible to make corrections to the general file of the module. I'm not a webmaster - I do not understand how to edit the code. Thank you in advance


The loop could be replaced with `lines.contains(line)`


Response class could be used as return type


- Ubuntu 14.04 and similar (e.g. Mint) - I must recompile recent Qt5 and append it into DEB. Also style problem - Qt 5.6 supports GTK2 style, but Qt4 does it better. Qt 5.7 and higher doesn't support GTK2 style, so I must use additional library with GTK2 style which IIRC crashes together with new GTK3 integration - additional work for me for native Ubuntu style...

- Windows - [this commit](https://github.com/zaps166/qt/commit/0111edb296c481b537ac86a229351ecfb15399fe) allows me to use properly any scroll area on touch screen. Qt5 has `QScroller` which is horrible, so I must have more time to adjust and set it properly to get it work as good as Qt4 Windows implementation...

- On Windows 7 and old Intel GPU I've got crashes with OpenGL (I've got this hardware, too) - this is a driver BUG, but Qt4 doesn't trigger this bug and it works fine - at least I think I can do a workaround in Qt for this.

---

In the future I'll drop any Qt older than 5.6 (this year I hope 😄).


This code should be thread-safe


@kelscahill I don't think this syntax is supported in `twig`?


Do you need this?


Simpler to just do this:

```js
const globPath = path.posix.join(relativePath, '**', '*');
```


And here.


Do you need this page?


The following doc(s) have been deployed to the [2310backup](https://2310backup-static-docs.pantheonsite.io/docs) Multidev environment:
- [docs/backups](https://2310backup-static-docs.pantheonsite.io/docs/backups)
- [source/docs/assets/images/backup-progress.png](https://github.com/pantheon-systems/documentation/commit/1f77f34de03626f93caf36516037300a49f51d1a/source/docs/assets/images/backup-progress.png)


@fichtner it's not really untrusted input, it has to be in a menu.xml. if you don't mind, I would like to keep this change small as it is. maybe we can replace/test another approach  later.


I just remove the line manually (which is why this commit was needed).  Not sure where it got lost but didn't feel like looking into it.  I wouldn't think it'd be much easier to define the env var than to just delete the line.


yeah, nothing being touched in either is related to the other, so a git merge would work fine.


This might not be safe - use `gs.board` instead.


@SharkFinPro Pointless to learn, when there's no way for you to help. Client is off limits, as we wouldn't want people with copies of it. But I think I got everything on my own, and andrew will take over afterwards.


`private display_context_proxy`

(Because the functions were private before, so let's keep it that way.)


Deleting CNAME


Indeed, now it's work perfectly


i think you may need a space to end this line


space after `!`


Well, its gonna fail on Windows 11. As you can see in the build script, the (pre)build is based on a Windows 10 `.winmd` file (bunch of Windows 10 specific meta data). You can also easily create builds for Windows 8 and 8.1 with the correct `.winmd` files. But I just haven't had the time to do that yet.


yes thanks, as discussed on IRC I'll do it.


hello,mamoo i signed up but i dont understand a word!!haha


spaces around concat operator (`.`)


I tried on my F10.  I used Zeitkriterium=0315 and VO code HKFM and CAS. I have i-level F010-15-11-501. Is not working.


untrusted output from the caller perspective. It looks like this is just a normal string compare? the next meta-character can break this (e.g "+")


closes #195


Woops


If you leave this out, some (read: one) tests are not longer executed.


Relevant issue: #2009


These ideas should be rolled into a design document somewhere.


nvm, ln 331 is also a part of this statement.


unfortunately the gtk integration will always be a nightmare...most because gnome/gtk devs don't care about breaking their API's and its fine for them because they only care about gnome

what makes everything even more annoying  is that Qt5.8 was really broken release ..even no 5.8.1 fix release will be done... Qt team said that they want to focus on 5.9 instead, so it should be more solid release with some [nice features](https://wiki.qt.io/New_Features_in_Qt_5.9)...and Qt5.10 seems to get some nice feature also.

btw. what do you think about making the UI (at least some parts) with QML instead of C++?


Приветствую! Скачал твою CMS, запускаю пишет
![123](https://cloud.githubusercontent.com/assets/16428889/25451152/e9cc0e18-2ac9-11e7-8ddc-6385c08f099d.png)


Delete this Hello.


Нужно указать логотипу, относительно чего позиционироваться (по умолчанию от верхнего левого угла окна). Для этого можно в .container дописать правило position: relative;
И ещё один момент. Я сразу не обратила внимания, потому что весь фон белый, но шапка и футер не должны тянуться на всю ширину окна браузера, У них фиксированная ширина 1200рх и центровка, а у body - серый фон. По сути, можно убрать div class="container" из футера и хедера, а класс container добавить им самим, потому что границы шапки и подвала в Седоне совпадают с шириной контейнера.


Platformus.WebApplication выделен жирным


Write here Hello world.


А по какому полю рельсы должны искать связанные билеты, если в билете 2 ассоциации? То, как сделано в билете - верно, а вот тут нужно поправить


@j4yk FYI: had to force push to master


I must be missing something obvious. The ref count of dupreq entry is 2 to start with. The only place where refcnt is decremented is as part of traversing the dupreq_q list. As part of the traversal, the entry is removed from the dupreq_q and refcnt is decremented to 1 but not freed. How will this entry be eventually freed.


btw. about windows 7 support...isnt M$ trying to kill it really fast? I mean with offical update/support wise ...I think there was only year or two left for win7 (don't know about enterprise version)


i prefer the tests to be sandboxed by project,
i.e. to run the test on its project, a user has to add the tester as parameter of Pro.run()


Happy to see this. When can one expect to see this on crates.io?


Please do not commit IDE files


afwe


I don't understand the logic.
If `config_ok==false` you should better `return false` straightway.

Then, when you `return false`, make sure you dispose all the resource you opened.


Well, to be honest I thought you mentioned putting it together for dry ha ha.


Even if `Bind` needs to support the two-argument form (which I question), I don't see the purpose of the `GLOBALS` table and its associated machinery in `ModuleLoad`. Why not just assign a `proc` to `Bind` and `export` it? @JacquesCarette


Your client should be a separate app that you'll provide to users of your service
It should have methods for working with WS's methods: not just facade for using http methods. It should contains methods of your WS like getTeachers(...), addTeacher(...) etc.


This class is not required


I don't see the purpose of the `GLOBALS` table and its associated machinery in `ModuleLoad`. Why not just assign `proc`s to `DOMAIN`, etc. and `export` them? @JacquesCarette


For now just players, we can worry about the AI later. It allows players to buy/commission weapons to be made. The better it is the more expensive it is. Such as a Valyrian weapon costing 5k gold.


It's not clear such separation of interfaces. This method should be a part of TeachersService


if it starts with a $ the whole calculation is lua


Because of such try/catches you exception mapper will never be used


Do you think a more descriptive variable name helps readability here?  Although `arg_value` isn't used elsewhere, I think it's good to get into the habit of using descriptive variable names.

Example:
```
if (cache_number < 1 || cache_number > 2)
  ...
```


ok


The way this should be done is with validation. See http://flowframework.readthedocs.io/en/stable/TheDefinitiveGuide/PartII/Validation.html


Добрый вечер! Платформус это просто набор расширений. Его нельзя запустить напрямую. Чтобы его запустить необходимо приложение-хост. Посмотрите на https://github.com/Platformus/Platformus-Sample/. Вот это приложение подключает Платформус в виде нюгет-зависимостей, и запускает его. Так он будет работать.


I believe this was intellij's / android studio's doing.
When I replaced the .jar this was changed.
I left it like that because I suppose this might be the "lint-friendly" way, as it hadn't done any of this in the past.


Cool beans!


Banter


Refcounts are put in nfs_dupreq_rele(), nfs_dupreq_delete(), and nfs_dupreq_finish().  All of these are called from nfs_rpc_execute() as part of the NFS execution code.  Am I missing something?


@dmoisset This commit did something wrong with typeshed. Now it points to the three weeks old version of typeshed.


**Whoops!**

There were *2 warnings* that will prevent this PR from being merged. Please take a look, and either fix, or provide a justification for why they can't be fixed.

**WARNINGS**
[`src/content/en/tools/chrome-devtools/evaluate-performance/index.md#L5`](https://github.com/Google/WebFundamentals/blame/dc5f5d325df3c1ed85a6b524c4e4391ebe0d5bc2/src/content/en/tools/chrome-devtools/evaluate-performance/index.md#L5) - WF Tag `wf_updated_on` must be within the last 7 days.
[`src/content/en/tools/chrome-devtools/evaluate-performance/index.md#L395`](https://github.com/Google/WebFundamentals/blame/dc5f5d325df3c1ed85a6b524c4e4391ebe0d5bc2/src/content/en/tools/chrome-devtools/evaluate-performance/index.md#L395) - Line must be at most 100 characters


TODO, just do it ;)


Spelling


Ok, I still don't make my mind about what's the "right/best/pretty" way in gradle :)


Maybe the quickest fix is for me to sync typeshed again.


gigas - freekick.cpp has a regression (emulation hangs quickly after booting)
![gigas-diff](https://cloud.githubusercontent.com/assets/9075896/25451706/10a0c7ba-2a91-11e7-9141-f59bb2c6738c.png)


Making the pop up close makes very good sense. I always wished it did that :)


Usually, the order goes as follows:
- List
- Show
- Create
- Delete
- Edit
- Used

If you don't have show, you might wonder why and see, if you want to have it or not. Maybe it's not necessary with the editor system, but maybe it is.


we need this for media queries to work


FixZoneVolumeCalcs (JasonGlazer) - i386-Windows-7-VisualStudio-14: OK (2467 of 2502 tests passed, 198 test warnings)

Messages:

 * 150 tests had: EIO diffs.
 * 23 tests had: ESO big diffs.
 * 30 tests had: Table big diffs.
 * 200 tests had: ESO small diffs.
 * 160 tests had: MTR small diffs.
 * 105 tests had: Table small diffs.
 * 3 tests had: SSZ big diffs.
 * 3 tests had: ZSZ big diffs.
 * 7 tests had: ERR diffs.
 * 12 tests had: MTR big diffs.
 * 7 tests had: ZSZ small diffs.
 * 3 tests had: SSZ small diffs.

Failures:

integration Test Summary
 * Passed: 599
 * Failed: 2

regression Test Summary
 * Passed: 566
 * Failed: 33

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


Я не уверен, что это общепринятая практика, но пометил.


den pop


![](https://github.trello.services/images/mini-trello-icon.png) [Homepage Tests](https://trello.com/c/EpkEXUR0/9-homepage-tests)


After this changes `@log_tags` does not work. Can you update the documentation how to use log_tags in version 0.3.0 ? https://github.com/actindi/act-fluent-logger-rails#usage


Better than having uncommited changes all the time


AFN2017 (macumber) - x86_64-MacOS-10.10-clang: Build Failed

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


Там ещё одна непереведённая строка осталась, Generic profile, по-русски я это обозвал "Универсальный профиль" - профиль CSV, который пытается вытащить из контакта всё по-максимуму.


Testing a commit on "set-page theme"


I would combine these three lines into a method in PreferenceProfile: addAuthor(MPlayer mplayer). This is to me clearly a task that the preference object should handle itself.

Maybe consider using senderObject instead, but you'll figure something out I am sure.


I will amend to better type.

Sorry, what do you mean "picked elsewhere"?  I was seeing name as the friendly identifier to be used in the UI.


?  Prefix with Log?  So becomes "LogMerchantAdded"?


Yes.  I will see if I can implement.


add a test code to study


Go for it if you'd like to


This should be done with a TypeName. You can look up TypeNameAbstract and how it is used:

https://github.com/MassiveCraft/MassiveCore/commit/c7e3d3dffabfd029d02e3ba69c1628eb63dac743
I am afraid, there are no open source usages yet, but you can ask me in skype for a screenshot.


hey


@arter97 please port this to n-mr2 if it's still necessary.  Lot of commits on r15 are dependent on this.


should probably move this into galaxy_ie_helpers eventually.


So I don't have to sync


I personally would prefer if all setters are grouped up with the append function. This way, all change to the profile is done in one block. See how it feels and decide accordingly.


Have not worked the "buy" process yet.  Was thinking of having and orders collection and passing on the value once the order has been fulfilled.  I'd stop any suicide calls if outstanding orders.  Disputes will need thinking about too.  Maybe an agreed arbitrator.


Hhm. Do we all have windows home computers here?


May need a steer here.  Been playing with refactoring the Merchant or Product responsibilities but as their tied to structs, mappings and arrays it's a little tricky.  Any pointers or links?


This is really something that @yuriy0 should answer.  This particular use seems like overkill.  Other uses are 'clever' because it means that the update script does not need to be changed anytime new globals get added to Hakaru.  I think the goal was to make reloading the library simpler, including making it actually work properly on Windows (which it never did unless you remembered to not have any active Maples when you did it).


Neat :)


This message will be white. This is not the standard color to send information. Use <i> as the base color and <h> for the name. <h> stands for "highlight" and that is surely what we want to do with the name. Also, since you are setting the name to "name" a few lines earlier, why not re-use that variable? :)


Colors again, see above comment.


I think this is intended.  Wasn't there something with compression or similar which sometimes meant that the exact size of a target was not known?  (Feel free to correct.)  So, I think that 'up to' is correct.


So delete is pointless?


Again, something @yuriy0 should answer.  I also don't remember why 2-argument Bind exists.  Rest of this comment answered in comment on other commit.


Ok great :)


Another great tip :)


IF we don't use the MConf to fetch them, you can alternatively set them in the constructor.


@tvolkert, I think this line is missing an `await`


Are you calling out the use of the private "updateProduct" function here?


"AngularJS is a JavaScript structural framework." not "AngularJS is a type of JavaScript structural framework."


I'm not reusing the parameter variable for setting the name so that I can safely add any cleanup logic to the setter and have those manipulations to the name appear in the message


TypeEntity.get(PreferenceProfileColl.get() has now appeared a second time. It should now definitely get it's own singleton object. A rule of thumb is, that besides typeList and TypeNullable, we very rarely construct the Types in this line. So those types deserve their own object.


@emilioicai @secondsun should this be in the public repo? If not, we may want to remove this and generate a new secret...


Oh yeah, you're right! I forgot about that. Yes, one of our independent implementors wanted to support a use case where the exact target file is unknown, but they know its upper bound, and need a way to prevent endless data attacks. So, yes, absolutely, you're right.

As for compression, I'm not sure what the consensus here was. Do we want to list metadata about the _compressed_ files themselves? IIRC, I think @endophage was arguing against this. So he might like the idea of listing an upper bound here, too.


Yep, makes sense.


No problem !
It is my fault for the merge conflict, I often forget to do a "git pull" before the push.
Still, github shows strangely the polluted history in such case.


Bluemix [toolchain](https://console.ng.bluemix.net/devops/toolchains/a1c1f084-c18b-4ecb-99f8-9a9b22da21b5 "FB Face Recognition"): [Delivery Pipeline](https://console.ng.bluemix.net/devops/pipelines/63e7d3d4-ea51-4079-91c8-7e11b1d3f83b "FB Face Recognition") deployed [FB-Face-Recognition](https://github.com/iris-bot/FB-Face-Recognition/releases/tag/deploy-ibm-yp-us-south-iris-bot-dev-20170426-191351 "deploy-ibm-yp-us-south-iris-bot-dev-20170426-191351") to [dev](https://console.ng.bluemix.net/apps/47bcbc3c-0115-4065-99e5-49b693b7caac "ibm:yp:us-south:iris-bot:dev"), including this commit


Please group up id and tooltip, there is no need for the empty line.


Желательно из проекта удалить всё, что к нему не относится
Примеры и прочее лучше хранить отдельно (в другом репозитории, например)


Hey , the      .Remove/Add/Savechanges are in red and the VS show an error telling that cannot  resolve symbole .. can someone explain to me why ? the DbSet is working but those default methodes are not working


Please use messageAsync() here, we don't know how performance heavy this could get.


Please elaborate on the high rope comment?   updateProduct does a couple of things like maintain the stock value and event trigger.  I thought I'd go DRY there but sounds like a bad idea.


Ok ta


<i> is sufficient.


Можно использовать метод isEmpty()


See request for help above ;)


Copy-paste


i compiled a new version some minutes ago and i'm testing right now. gimme 20 minutes


You should have endpoint interface


Why ResponseEntity parametrized here?


I don't see anything "fake" about this ticket. Besides it's use of course. So why not name it "ticket" instead?


You don't have method for getting one teacher


Всё, что к проекту не относится - вынести отсюда (например, в другой репозиторий)


![](https://github.trello.services/images/mini-trello-icon.png) [Több nyelven is megjelenik a naptár, ki lehet választani a nyelvet](https://trello.com/c/hdF0pE3M/11-tobb-nyelven-is-megjelenik-a-naptar-ki-lehet-valasztani-a-nyelvet)


For update URI should be /teachers/{id}


@emilioicai do you know why this was upgraded? I'd like to stay in sync with what React Native 0.43 recommends, and upgrade gradle when they do.


Same here, no need to type <info> out.


Why do you have produces here? You dont return anything


![](https://github.trello.services/images/mini-trello-icon.png) [As a customer when I search for a Pension Wise appointment in a particular location I am able to see the location page in the search results](https://trello.com/c/Q18mwqlt/141-as-a-customer-when-i-search-for-a-pension-wise-appointment-in-a-particular-location-i-am-able-to-see-the-location-page-in-the-se)


@jorcalle11 I think documenting each route would be nice. What about a brief description that includes what one expects to find when accessing a route and what params are needed? What do you think?


One message that the command was run should be sufficient imo.


@david-winder-kaltura I think there's a bit of an issue with the new code, the new variable is defined inside a private function: https://github.com/kaltura/server/blob/Lynx-12.15.0/plugins/drop_folder/batch/DropFolderWatcher/KAsyncDropFolderWatcher.class.php#L94

while you attempt to use it in a public function outside of the main class.

In addition you're not checking whether the $dropfolders is empty, causing a huge spam if there's no dropfolders to update.

Possible solutions:

1. pass the dropfolder ID to preKill()
2. check whether  $dropFolders is an object before trying to use its properties (

Let me know which one you prefer and I can make a pull request for it


w testach leci:
wdsr.exercise3.client.ProductServiceTest > deleteProduct_shouldDeleteProduct FAILED
    java.lang.RuntimeException at ProductServiceTest.java:30
        Caused by: java.net.BindException at ProductServiceTest.java:30
    java.lang.NullPointerException


I'm going to be learning Java this summer, so maybe we could all learn it or some other new language to use with a project.


"It can convey the application’s components efficiently." - I don't know what this means.


Your mapper isn't used anywhere


It  occured to me that you already have TypePreferenceProfile, please make use of it if applicable.


Сервлеты должны с сервисами работать, а не с дао


> btw. what do you think about making the UI (at least some parts) with QML instead of C++?

Btw. Do you know how to disable ugly font rendering (RGB hinting) in QtQuick? I have colored fonts, but fontconfig uses grayscale hinting (every other app uses grayscale)...

I don't know this technology. QtQuick Controls 2 is only for mobiles (inconvenient and ugly for desktops). QMPlay2 is not designed for mobiles (Android, iOS). Other UI controls are similar to QtWidgets so it makes no sense to use QtQuick instead of QWidgets. C++ is fast and easy to use language, but JS/QML does not (at least for me 😄). But QML has only one advantage - doesn't need to be compiled, so I can update a module without recompilation (bit I can use Lua or any other script for this...). Unfortunately, QtWidgets doesn't get modern widgets just like QML/QtQuick :/

It'll be good to see full OpenGL rendering in all QtWidgets and also new widgtes and layouts especially for mobiles... Also touchscreen support should be improved in QtWidgets (again, `QScroller` is horrible).

> btw. about windows 7 support...isnt M$ trying to kill it really fast? I mean with offical update/support wise ...I think there was only year or two left for win7 (don't know about enterprise version)

IMHO yes, M$ tries to kill Win 7 and 8 just like it killed their best product - Windows XP. M$ thinks that Windows 10 is the only OS which should exists on out planet - spyware, adware, etc 😃

---

I hope that all games and professional software will arrive to Linux natively with full performance...


Dajesz dajesz!


Ah.  Good point.  Need the ability to create multiple arrangements and remover value after a said time.  Will re-design.


++ thanks


Yay! A thesis! Something missing from too many papers.


You can always do a `git pull --rebase` if you made a local commit on top of the old state and you don't want to create unnecessary merge commits


По структуре, помимо работы с дао из сервлетов - всё верно.
Исправила правильно


_Режим ждуна_ включён.


So, just send detail regardless is sender had set one up?  Is my use of a "isActive" flag not good practice?


Такие callback'и обычно пишут как:
```js
var searchInput = document.getElementById("search_id_1");
searchInput.onkeyup = function() {
  var userList = document.getElementById("user_list_id_1");
  // . . .
}
```


Just added a table of tagId to the Segment model so that tags can be added to the where filter in _galleryFilter['gallery.view.classifiers] to simplify integration with the existing gallery infinite scroll methods.


As the idiom in my native language says: "Pucha perrin". Why did you added the example for ```.env``` file? @tractorcow shame on you. When installing it using composer it wasn't there and I got confused with the docs as it said clearly

> An example ```.env``` file is included in the default installer named ```.env.example```.

[https://docs.silverstripe.org/en/4/getting_started/environment_management/#managing-environment-variables-with-env-files]


Little section header missing:
// FIELD: MODERATOR PREFERENCES

Also, the order should be as usually:
- get
- set
- has (or other related)


Got a typo in the Changelog... `auto-loabable`


Later if `x` amount of questions & answers is supplied we could set up `x` amount of fragments and launch upon quiz completion of the previous fragment until `x` amount of fragments have been exhausted from the collection. Thereafter the PlayActivity will resume with the next checkpoint.


Красиво написал `||` мне нравится, дух Js)


Empty line missing between methods.


I did at first, but I was running into issues with it not accepting the name of the profile and only the id. I'll take a look at how TypeEntity does it so that I can get similar functionality in TypePreferenceProfile.


![](https://github.trello.services/images/mini-trello-icon.png) [[1] iCal: name the genereated iCal files accordingly](https://trello.com/c/7W9R4YJo/214-1-ical-name-the-genereated-ical-files-accordingly)


researchgate icon added to icon.html file.


Also, how many times do we get this? Would it be worth while storing it into a transient field perhaps?


should be `const [, thisScheme, thisDomain, thisPath] = this.urlParts`


hareket is the same it means move


скачал, еще есть ошибки An unhandled exception occurred while processing the request.

InvalidOperationException: The view 'Index' was not found. The following locations were searched:
/Views/Installation/Index.en.cshtml
/Views/Installation/Index.cshtml
/Views/Shared/Index.en.cshtml
/Views/Shared/Index.cshtml
Microsoft.AspNetCore.Mvc.ViewEngines.ViewEngineResult.EnsureSuccessful(IEnumerable<string> originalLocations)

Stack Query Cookies Headers
InvalidOperationException: The view 'Index' was not found. The following locations were searched:
 /Views/Installation/Index.en.cshtml
 /Views/Installation/Index.cshtml
 /Views/Shared/Index.en.cshtml
 /Views/Shared/Index.cshtml


TeamCity libpalaso :: palaso-win32-libpalaso-3.1 Continuous [Build 3.1.294.e8f20a3844da914db781d3007cc68e3fce3f1846](http://build.palaso.org/viewLog.html?buildId=88832&buildTypeId=bt223) outcome was **SUCCESS**
Summary: Tests passed: 3585, ignored: 128 Build time: 00:03:43


I think that further tuning is not needed. I saw everything I wanted to see. A number of tests will follow soon.


TeamCity libpalaso :: palaso-win32-master Continuous [Build 4.0.17.e8f20a3844da914db781d3007cc68e3fce3f1846](http://build.palaso.org/viewLog.html?buildId=88839&buildTypeId=Libpalaso_PalasoWin32masterContinuous) outcome was **FAILURE**
Summary: Compilation error: SIL.Windows.Forms\SIL.Windows.Forms.csproj; compilation error: SIL.DblBundle\SIL.DblBundle.csproj Build time: 00:00:29


Huh. Done with implementation of this class. Testing is ahead.


Ok.  I take it that is best practice that I should adopt?


TeamCity libpalaso :: palaso-trusty64-lfmerge Continuous [Build 2.6.279.e8f20a3844da914db781d3007cc68e3fce3f1846](http://build.palaso.org/viewLog.html?buildId=88831&buildTypeId=PalasoTrusty64lfmergeContinuous) outcome was **SUCCESS**
Summary: Tests passed: 3563, ignored: 111 Build time: 00:05:28


I wasn't able to retrieve PDD puzzles from the code base (if you think that it's a bug on your our side, please submit it to [yegor256/0pdd](https://github.com/yegor256/0pdd/issues)):

```
set -x;set -e;set -o pipefail;cd /tmp/0pdd20170426-4-7vgssj/lautarobock/newBoM && pdd -q -f /tmp/pdd.xml20170426-3655-1g6ro6y: pid 3744 exit 1 (not zero):
+ set -e
+ set -o pipefail
+ cd /tmp/0pdd20170426-4-7vgssj/lautarobock/newBoM
+ pdd -q -f /tmp/pdd.xml20170426-3655-1g6ro6y
/app/vendor/bundle/ruby/2.2.0/gems/pdd-0.17.4/lib/pdd/source.rb:144:in `rescue in puzzles': ["in /tmp/0pdd20170426-4-7vgssj/lautarobock/newBoM/src/app/domain/recipe.ts", #<PDD::Error: @todo found, but puzzle can't be parsed in line #90>] (PDD::Error)
	from /app/vendor/bundle/ruby/2.2.0/gems/pdd-0.17.4/lib/pdd/source.rb:142:in `puzzles'
	from /app/vendor/bundle/ruby/2.2.0/gems/pdd-0.17.4/lib/pdd.rb:97:in `block (3 levels) in xml'
	from /app/vendor/bundle/ruby/2.2.0/gems/pdd-0.17.4/lib/pdd.rb:96:in `each'
	from /app/vendor/bundle/ruby/2.2.0/gems/pdd-0.17.4/lib/pdd.rb:96:in `block (2 levels) in xml'
	from /app/vendor/bundle/ruby/2.2.0/gems/nokogiri-1.6.8.1/lib/nokogiri/xml/builder.rb:391:in `call'
	from /app/vendor/bundle/ruby/2.2.0/gems/nokogiri-1.6.8.1/lib/nokogiri/xml/builder.rb:391:in `insert'
	from /app/vendor/bundle/ruby/2.2.0/gems/nokogiri-1.6.8.1/lib/nokogiri/xml/builder.rb:375:in `method_missing'
	from /app/vendor/bundle/ruby/2.2.0/gems/pdd-0.17.4/lib/pdd.rb:95:in `block in xml'
	from /app/vendor/bundle/ruby/2.2.0/gems/nokogiri-1.6.8.1/lib/nokogiri/xml/builder.rb:293:in `initialize'
	from /app/vendor/bundle/ruby/2.2.0/gems/pdd-0.17.4/lib/pdd.rb:93:in `new'
	from /app/vendor/bundle/ruby/2.2.0/gems/pdd-0.17.4/lib/pdd.rb:93:in `xml'
	from /app/vendor/bundle/ruby/2.2.0/gems/pdd-0.17.4/bin/pdd:63:in `<top (required)>'
	from /app/vendor/bundle/ruby/2.2.0/bin/pdd:23:in `load'
	from /app/vendor/bundle/ruby/2.2.0/bin/pdd:23:in `<main>'

/app/objects/exec.rb:43:in `block in run'
/app/vendor/ruby-2.2.2/lib/ruby/2.2.0/open3.rb:199:in `popen_run'
/app/vendor/ruby-2.2.2/lib/ruby/2.2.0/open3.rb:93:in `popen3'
/app/objects/exec.rb:41:in `run'
/app/objects/git_repo.rb:61:in `xml'
/app/objects/puzzles.rb:46:in `deploy'
/app/objects/job.rb:42:in `proceed'
/app/objects/job_starred.rb:35:in `proceed'
/app/objects/job_recorded.rb:33:in `proceed'
/app/objects/job_emailed.rb:36:in `proceed'
/app/objects/job_commiterrors.rb:36:in `proceed'
/app/objects/job_detached.rb:51:in `block in exclusive'
/app/vendor/ruby-2.2.2/lib/ruby/2.2.0/timeout.rb:89:in `block in timeout'
/app/vendor/ruby-2.2.2/lib/ruby/2.2.0/timeout.rb:34:in `block in catch'
/app/vendor/ruby-2.2.2/lib/ruby/2.2.0/timeout.rb:34:in `catch'
/app/vendor/ruby-2.2.2/lib/ruby/2.2.0/timeout.rb:34:in `catch'
/app/vendor/ruby-2.2.2/lib/ruby/2.2.0/timeout.rb:104:in `timeout'
/app/objects/job_detached.rb:49:in `exclusive'
/app/objects/job_detached.rb:39:in `block in proceed'
/app/objects/job_detached.rb:39:in `fork'
/app/objects/job_detached.rb:39:in `proceed'
/app/0pdd.rb:243:in `block in <top (required)>'
/app/vendor/bundle/ruby/2.2.0/gems/sinatra-1.4.7/lib/sinatra/base.rb:1611:in `call'
/app/vendor/bundle/ruby/2.2.0/gems/sinatra-1.4.7/lib/sinatra/base.rb:1611:in `block in compile!'
/app/vendor/bundle/ruby/2.2.0/gems/sinatra-1.4.7/lib/sinatra/base.rb:975:in `[]'
/app/vendor/bundle/ruby/2.2.0/gems/sinatra-1.4.7/lib/sinatra/base.rb:975:in `block (3 levels) in route!'
/app/vendor/bundle/ruby/2.2.0/gems/sinatra-1.4.7/lib/sinatra/base.rb:994:in `route_eval'
/app/vendor/bundle/ruby/2.2.0/gems/sinatra-1.4.7/lib/sinatra/base.rb:975:in `block (2 levels) in route!'
/app/vendor/bundle/ruby/2.2.0/gems/sinatra-1.4.7/lib/sinatra/base.rb:1015:in `block in process_route'
/app/vendor/bundle/ruby/2.2.0/gems/sinatra-1.4.7/lib/sinatra/base.rb:1013:in `catch'
/app/vendor/bundle/ruby/2.2.0/gems/sinatra-1.4.7/lib/sinatra/base.rb:1013:in `process_route'
/app/vendor/bundle/ruby/2.2.0/gems/sinatra-1.4.7/lib/sinatra/base.rb:973:in `block in route!'
/app/vendor/bundle/ruby/2.2.0/gems/sinatra-1.4.7/lib/sinatra/base.rb:972:in `each'
/app/vendor/bundle/ruby/2.2.0/gems/sinatra-1.4.7/lib/sinatra/base.rb:972:in `route!'
/app/vendor/bundle/ruby/2.2.0/gems/sinatra-1.4.7/lib/sinatra/base.rb:1085:in `block in dispatch!'
/app/vendor/bundle/ruby/2.2.0/gems/sinatra-1.4.7/lib/sinatra/base.rb:1067:in `block in invoke'
/app/vendor/bundle/ruby/2.2.0/gems/sinatra-1.4.7/lib/sinatra/base.rb:1067:in `catch'
/app/vendor/bundle/ruby/2.2.0/gems/sinatra-1.4.7/lib/sinatra/base.rb:1067:in `invoke'
/app/vendor/bundle/ruby/2.2.0/gems/sinatra-1.4.7/lib/sinatra/base.rb:1082:in `dispatch!'
/app/vendor/bundle/ruby/2.2.0/gems/sinatra-1.4.7/lib/sinatra/base.rb:907:in `block in call!'
/app/vendor/bundle/ruby/2.2.0/gems/sinatra-1.4.7/lib/sinatra/base.rb:1067:in `block in invoke'
/app/vendor/bundle/ruby/2.2.0/gems/sinatra-1.4.7/lib/sinatra/base.rb:1067:in `catch'
/app/vendor/bundle/ruby/2.2.0/gems/sinatra-1.4.7/lib/sinatra/base.rb:1067:in `invoke'
/app/vendor/bundle/ruby/2.2.0/gems/sinatra-1.4.7/lib/sinatra/base.rb:907:in `call!'
/app/vendor/bundle/ruby/2.2.0/gems/sinatra-1.4.7/lib/sinatra/base.rb:895:in `call'
/app/vendor/bundle/ruby/2.2.0/gems/rack-protection-1.5.3/lib/rack/protection/xss_header.rb:18:in `call'
/app/vendor/bundle/ruby/2.2.0/gems/rack-protection-1.5.3/lib/rack/protection/path_traversal.rb:16:in `call'
/app/vendor/bundle/ruby/2.2.0/gems/rack-protection-1.5.3/lib/rack/protection/json_csrf.rb:18:in `call'
/app/vendor/bundle/ruby/2.2.0/gems/rack-protection-1.5.3/lib/rack/protection/base.rb:49:in `call'
/app/vendor/bundle/ruby/2.2.0/gems/rack-protection-1.5.3/lib/rack/protection/base.rb:49:in `call'
/app/vendor/bundle/ruby/2.2.0/gems/rack-protection-1.5.3/lib/rack/protection/frame_options.rb:31:in `call'
/app/vendor/bundle/ruby/2.2.0/gems/rack-1.6.5/lib/rack/logger.rb:15:in `call'
/app/vendor/bundle/ruby/2.2.0/gems/rack-1.6.5/lib/rack/commonlogger.rb:33:in `call'
/app/vendor/bundle/ruby/2.2.0/gems/sinatra-1.4.7/lib/sinatra/base.rb:219:in `call'
/app/vendor/bundle/ruby/2.2.0/gems/sinatra-1.4.7/lib/sinatra/base.rb:212:in `call'
/app/vendor/bundle/ruby/2.2.0/gems/rack-1.6.5/lib/rack/head.rb:13:in `call'
/app/vendor/bundle/ruby/2.2.0/gems/rack-1.6.5/lib/rack/methodoverride.rb:22:in `call'
/app/vendor/bundle/ruby/2.2.0/gems/sinatra-1.4.7/lib/sinatra/base.rb:182:in `call'
/app/vendor/bundle/ruby/2.2.0/gems/sinatra-1.4.7/lib/sinatra/base.rb:2013:in `call'
/app/vendor/bundle/ruby/2.2.0/gems/sinatra-1.4.7/lib/sinatra/base.rb:1487:in `block in call'
/app/vendor/bundle/ruby/2.2.0/gems/sinatra-1.4.7/lib/sinatra/base.rb:1787:in `synchronize'
/app/vendor/bundle/ruby/2.2.0/gems/sinatra-1.4.7/lib/sinatra/base.rb:1487:in `call'
/app/vendor/bundle/ruby/2.2.0/gems/rack-1.6.5/lib/rack/handler/webrick.rb:88:in `service'
/app/vendor/ruby-2.2.2/lib/ruby/2.2.0/webrick/httpserver.rb:138:in `service'
/app/vendor/ruby-2.2.2/lib/ruby/2.2.0/webrick/httpserver.rb:94:in `run'
/app/vendor/ruby-2.2.2/lib/ruby/2.2.0/webrick/server.rb:294:in `block in start_thread'
```


this needs to be an addition, otherwise the balls will be placed above the 0 y point of the view and clip.


TeamCity libpalaso :: palaso-precise64-libpalaso-2.6 Continuous [Build 2.6.1288.e8f20a3844da914db781d3007cc68e3fce3f1846](http://build.palaso.org/viewLog.html?buildId=88830&buildTypeId=PalasoPrecise64v26Cont) outcome was **SUCCESS**
Summary: Tests passed: 3563, ignored: 111 Build time: 00:05:57


Id nie przychodzi tutaj w obiekcie produkut. Warto zobaczyć jak to wygląda od strony testów.


So restricting access to constant data pulls is pointless full stop?


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 4.0.16.e8f20a3844da914db781d3007cc68e3fce3f1846](http://build.palaso.org/viewLog.html?buildId=88833&buildTypeId=Libpalaso_PalasoPrecise64masterContinuous) outcome was **FAILURE**
Summary: Failed to resolve artifact dependency Build time: 00:00:06


don't need this much padding...


```
quiz: [{
  answers: [{
    answer: string,
    correct: bool
  }]
  ...
}]
```


I was wondering!!


quiz: [{
  ...
  question: string
}]


Ok, so no loops and always hardwire to 2?  Is there a better approach when dealing with dynamic scenarios such as this?


JSDoc comment, looks like it's not necessary in this case, because I just saw that there is no JSDoc in this project. I will remove it in next commit.


this can be simplified to `for (ShapeDrawable shape : myShapes) {