Mainly for compatibility to pure Vim.
I disabled many keymaps by default for the feature which was not supported in pure Vim.
Another reason is to avoid conflict with Atom's default keymap.

[Commands-which-have-no-default-keymap](https://github.com/t9md/atom-vim-mode-plus/wiki/Commands-which-have-no-default-keymap)

Althoug it is disabled, I will strongly recommend to try by enabling each feature, it might improve your effectiveness on editing codes.


Hi @wdblair, I just finished it following your initial example. Your code is very helpful, thank you! Please double check that I'm doing it correctly.


assert may better?


No reason to have ```as e``` here.


test comment


OH HI JACK! :D


Instead of using old and uncool arm_boot_info, I recommend new and QOM'ed (i.e. cool) MachineState which has all these kernel_filename, initrd_filename, etc. You can use sPAPRMachineState as an example.


FYI the new localisation load order is still causing vanilla to override localisations from `am_titles.csv`, `am_loadtips.csv`, `am_flavor.csv`, and `am_events2.csv`


Also use `.safe_find` to prevent defaults from failing


primer msg


Just a question, is using blank return a good practice in a method that is decorated with ```@inlineCallbacks```? Should it be replaced with ```returnValue(None)``` instead?


Looks good conceptually. Thanks!


No. In other smtp servers example Amazon it is not an email

Enviado desde Yahoo Mail para Android

  El mié., feb. 24, PM a 9:13 PM, R.S<notifications@github.com> escribió:
@degaray Shouldn't the username always be the from address?

—
Reply to this email directly or view it on GitHub.


As a note, this also trimmed trailing spaces


Not sure I'm following--the two tests are actually flaky in the same way (they can pass when they shouldn't), but this one should have a false negative around 0.1% of the time, while the existing one has false negative rate of 0.625


AspeedState should QOM-inherit from MachineState, not be allocated on a stack which is thrown away when ast2400_init is done. And I believe it does not need @ram and @address_space members, cannot see these in (for example) sPAPRMachineState


TeamCity LabKey :: LabkeyRelease15.3 :: CDS postgres 9.4 [Build 159](https://teamcity.labkey.org/viewLog.html?buildId=305483&buildTypeId=LabkeyRelease153_CdsPostgres94) outcome was **FAILURE**
Summary: Tests failed: 11 (4 new), passed: 40 Build time: 01:30:31
### Failed tests
```
org.labkey.test.tests.CDSTest.verifyFilterDisplays: <no details avaliable>

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

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

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

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

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

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

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

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

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

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

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


ARM926EJ-S does not have documentation either? What is its exact relationship to AST2400?


I'm not sure keep a reference here is safe


There is actually an caching policy called [TinyLFU](http://arxiv.org/pdf/1512.00727.pdf). You might enjoy implementing that. :)


If caller keep, it should be ok


I was thinking about creating a new extension since this module name is "gmail...". Did you test this on Amazon ?

Sent from my iPhone

> On Feb 24, 2016, at 11:28 PM, degaray <notifications@github.com> wrote:
>
> No. In other smtp servers example Amazon it is not an email
>
> Enviado desde Yahoo Mail para Android
>
> El mié., feb. 24, PM a 9:13 PM, R.S<notifications@github.com> escribió:
> @degaray Shouldn't the username always be the from address?
>
> —
> Reply to this email directly or view it on GitHub.
>
> —
> Reply to this email directly or view it on GitHub.
>


You need a better commit log. Template:

AST2400 is lalala used in IBM lalala. Unfortunately the documentation is not available. Basically it is a SOC with XXX architecture CPU on board and lala1, lala2, lala3 peripheral devices. This implements only lala1, lala2.


Cool, and the new description makes sense to me.


"The implementation is a little awkward as the hardware uses 32bit" - either
move it to the code as comment or
make the implementation not awkward or
add so much information in the commit log so this comment would make sense then


On fait le test sur la permission. Au sens django-rules, une permission est une rule spéciale, dans le sens ou elle est couplée au backend d'auth. Si tu fais référence par contre à une rule du modèle qu'on a fait pour déclarer qui peut faire quoi sur qu'elle objet (HasPermission), ce prédicat est bel et bien dans la permission (rule).

d'après ce qu'on s'est dit, le contenu de cette permission va changer, car ce sont des personnels de érudit qui feront ce travail de review pour toutes les revues.


@athanatos  Shouldn't it be based on aio_write_queue_bytes ?


17, 18, 35, ... - what are these numbers? :) Some pinout explanation would not hurt.


should be in the Railtie IMHO


Kconfig is missing the flag u made will not appear or will not work later on


Thanks for the explanation.  It is horrifying.

On Wed, Feb 24, 2016 at 7:45 PM, Marcos Caceres <notifications@github.com>
wrote:

> I don't know. I don't know what fixup does.
> It starts aggressively looking for DOM elements that it assumes should be
> there - and when they are not, it just crashes. There are lots of cases
> where those elements might not be present.
>
> In other cases, it starts creating new elements when certain elements are
> missing.
>
> It might be better if the script is callable, so when the spec is ready,
> it can invoke W3C.styles.fixup(); of something. I might suggest that if
> it gets too messy.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/w3c/respec/commit/cceb8ada6e96c9f64dcad7f40d16e1802beb8ffa#commitcomment-16316671>
> .
>

--
Shane McCarron
halindrome@gmail.com


Is this still a thing in Ruby land? Wasn't it an unsupported runtime a couple of years ago?


I think return is fine, but I don't know for sure.


alternatively, in the Link include `ActiveModelSerializers::Support::UrlHelpers` and in the railtie `ActiveModelSerializers::Support::UrlHelpers.include ActiveModelSerializers::Support::UrlHelpers::Rails` or something like that


Getting a crash with this feature if the timing grid is inactive and you hit an arrow key.


Ah, I see. I was comparing it to `new_identifier` above, which is somewhat parallel but not as close as that passage in `parsestr`.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 17163](https://teamcity.mesosphere.io/viewLog.html?buildId=145320&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 2 (new) Build time: 00:00:50


(I don't actually see this change in your latest commit.)


Yes and it works

Enviado desde Yahoo Mail para Android

  El mié., feb. 24, PM a 10:32 PM, R.S<notifications@github.com> escribió:   I was thinking about creating a new extension since this module name is "gmail...". Did you test this on Amazon ?

Sent from my iPhone

> On Feb 24, 2016, at 11:28 PM, degaray <notifications@github.com> wrote:
>
> No. In other smtp servers example Amazon it is not an email
>
> Enviado desde Yahoo Mail para Android
>
> El mié., feb. 24, PM a 9:13 PM, R.S<notifications@github.com> escribió:
> @degaray Shouldn't the username always be the from address?
>
> —
> Reply to this email directly or view it on GitHub.
>
> —
> Reply to this email directly or view it on GitHub.
>

—
Reply to this email directly or view it on GitHub.


then we could have

```ruby
module UrlHelpers
  def url_for(options)
    fail 'template method'
  end
  def default_url_options
    fail 'template method'
  end

  module Rails
     include Rails.application.routes.url_helpers

     def default_url_options
       ActionMailer::Base.default_url_options
     end
  end
end
```


Ah, fair enough.


Hey a new extension with queued jobs would be awesome.

Enviado desde Yahoo Mail para Android

  El mié., feb. 24, PM a 10:32 PM, R.S<notifications@github.com> escribió:   I was thinking about creating a new extension since this module name is "gmail...". Did you test this on Amazon ?

Sent from my iPhone

> On Feb 24, 2016, at 11:28 PM, degaray <notifications@github.com> wrote:
>
> No. In other smtp servers example Amazon it is not an email
>
> Enviado desde Yahoo Mail para Android
>
> El mié., feb. 24, PM a 9:13 PM, R.S<notifications@github.com> escribió:
> @degaray Shouldn't the username always be the from address?
>
> —
> Reply to this email directly or view it on GitHub.
>
> —
> Reply to this email directly or view it on GitHub.
>

—
Reply to this email directly or view it on GitHub.


@luiscalcano La parte de arriba de las vistas (en la seccion de filtrar) podria estar mejor organizado, te ense;are como hacerlo.


Looks good. thoughts on interface? tests?


Yeah, makes sense.


Reversed the default side. Right is right, left is wrong.


Prefix "Edu" = ("Re" or "")


are these still at /api/users/... or are they just at /api/cart and using the auth header to provide the user id as opposed to the user surrogate key being in the route?


Aqui en vez de usar un input seria mejor usar un textarea ya que es una descripcion. Ademas que el tama;o que tiene le quita la alineacion que tiene el formulario.


Not a tracepoint?


magic constant 0x30 - usually folks ask to define these to give them self-explanatory names. But sometime they are ok.


@ravnon I could not open up the `candidate_1.jpg` on my computer. Perhaps you have a corrupted file?

Eventually we would want to have candidate pictures for every single candidate in the system. And perhaps a helper method as well, e.g. `getCandidateAvatar(candidateId)`, that returns the appropriate avatar for that candidate, or the default avatar if none is present.


Good


Keep it up! You checked in your key o.0


Wrong indent here and below - align to the opening bracket.


Thanks for this. Def. felt the animations were a tad too slow before. 😂


OK!


Looks good, can anybody else with iterm2 confirm that this actually works?


what do we need this for? this is the class i'm using to convert to an `ExportInstance` (i think, gotta double check)


checking coment


made a mistake here


ERROR: 'cast' is not followed by whitespace.


ERROR: 'sessions' hides a field.


ERROR: 'tabPos' hides a field.


ERROR: Missing a Javadoc comment.


ERROR: Name 'SEARCH' must match pattern '^[a-z]([_a-zA-Z0-9]+)*$'.


ERROR: [PMD-CPD] 65 lines duplicated.
app/src/main/java/org/fossasia/openevent/activities/SessionDetailActivity.java


ERROR: [PMD-CPD] 15 lines duplicated.
app/src/main/java/org/fossasia/openevent/activities/SessionDetailActivity.java


WARNING: [SimpleDateFormat] To get local formatting use `getDateInstance()`, `getDateTimeInstance()`, or `getTimeInstance()`, or use `new SimpleDateFormat(String template, Locale locale)` with for example `Locale.US` for ASCII dates.
 Almost all callers should use `getDateInstance()`, `getDateTimeInstance()`, or `getTimeInstance()` to get a ready-made instance of SimpleDateFormat suitable for the user's locale. The main reason you'd create an instance this class directly is because you need to format/parse a specific machine-readable format, in which case you almost certainly want to explicitly ask for US to ensure that you get ASCII digits (rather than, say, Arabic digits).

Therefore, you should either use the form of the SimpleDateFormat constructor where you pass in an explicit locale, such as Locale.US, or use one of the get instance methods, or suppress this error if really know what you are doing.


WARNING: [UnusedAttribute] Attribute `fontFamily` is only used in API level 16 and higher (current min is 15)
 This check finds attributes set in XML files that were introduced in a version newer than the oldest version targeted by your application (with the `minSdkVersion` attribute).

This is not an error; the application will simply ignore the attribute. However, if the attribute is important to the appearance of functionality of your application, you should consider finding an alternative way to achieve the same result with only available attributes, and then you can optionally create a copy of the layout in a layout-vNN folder which will be used on API NN or higher where you can take advantage of the newer attribute.

Note: This check does not only apply to attributes. For example, some tags can be unused too, such as the new `<tag>` element in layouts introduced in API 21.


FATAL: [MissingTranslation] "`menu_schedule`" is not translated in "hi" (Hindi), "ro" (Romanian), "zh" (Chinese), "zh-CN" (Chinese: China)
 If an application has more than one locale, then all the strings declared in one language should also be translated in all other languages.

If the string should *not* be translated, you can add the attribute `translatable="false"` on the `<string>` element, or you can define all your non-translatable strings in a resource file called `donottranslate.xml`. Or, you can ignore the issue with a `tools:ignore="MissingTranslation"` attribute.

By default this detector allows regions of a language to just provide a subset of the strings and fall back to the standard language strings. You can require all regions to provide a full translation by setting the environment variable `ANDROID_LINT_COMPLETE_REGIONS`.

You can tell lint (and other tools) which language is the default language in your `res/values/` folder by specifying `tools:locale="languageCode"` for the root `<resources>` element in your resource file. (The `tools` prefix refers to the namespace declaration `http://schemas.android.com/tools`.)


FATAL: [MissingTranslation] "`filter_sessions`" is not translated in "hi" (Hindi), "ro" (Romanian), "zh" (Chinese), "zh-CN" (Chinese: China)
 If an application has more than one locale, then all the strings declared in one language should also be translated in all other languages.

If the string should *not* be translated, you can add the attribute `translatable="false"` on the `<string>` element, or you can define all your non-translatable strings in a resource file called `donottranslate.xml`. Or, you can ignore the issue with a `tools:ignore="MissingTranslation"` attribute.

By default this detector allows regions of a language to just provide a subset of the strings and fall back to the standard language strings. You can require all regions to provide a full translation by setting the environment variable `ANDROID_LINT_COMPLETE_REGIONS`.

You can tell lint (and other tools) which language is the default language in your `res/values/` folder by specifying `tools:locale="languageCode"` for the root `<resources>` element in your resource file. (The `tools` prefix refers to the namespace declaration `http://schemas.android.com/tools`.)


FATAL: [MissingTranslation] "`select_all`" is not translated in "hi" (Hindi), "ro" (Romanian), "zh" (Chinese), "zh-CN" (Chinese: China)
 If an application has more than one locale, then all the strings declared in one language should also be translated in all other languages.

If the string should *not* be translated, you can add the attribute `translatable="false"` on the `<string>` element, or you can define all your non-translatable strings in a resource file called `donottranslate.xml`. Or, you can ignore the issue with a `tools:ignore="MissingTranslation"` attribute.

By default this detector allows regions of a language to just provide a subset of the strings and fall back to the standard language strings. You can require all regions to provide a full translation by setting the environment variable `ANDROID_LINT_COMPLETE_REGIONS`.

You can tell lint (and other tools) which language is the default language in your `res/values/` folder by specifying `tools:locale="languageCode"` for the root `<resources>` element in your resource file. (The `tools` prefix refers to the namespace declaration `http://schemas.android.com/tools`.)


FATAL: [MissingTranslation] "`select_none`" is not translated in "hi" (Hindi), "ro" (Romanian), "zh" (Chinese), "zh-CN" (Chinese: China)
 If an application has more than one locale, then all the strings declared in one language should also be translated in all other languages.

If the string should *not* be translated, you can add the attribute `translatable="false"` on the `<string>` element, or you can define all your non-translatable strings in a resource file called `donottranslate.xml`. Or, you can ignore the issue with a `tools:ignore="MissingTranslation"` attribute.

By default this detector allows regions of a language to just provide a subset of the strings and fall back to the standard language strings. You can require all regions to provide a full translation by setting the environment variable `ANDROID_LINT_COMPLETE_REGIONS`.

You can tell lint (and other tools) which language is the default language in your `res/values/` folder by specifying `tools:locale="languageCode"` for the root `<resources>` element in your resource file. (The `tools` prefix refers to the namespace declaration `http://schemas.android.com/tools`.)


FATAL: [MissingTranslation] "`add_to_calendar`" is not translated in "hi" (Hindi), "ro" (Romanian), "zh" (Chinese), "zh-CN" (Chinese: China)
 If an application has more than one locale, then all the strings declared in one language should also be translated in all other languages.

If the string should *not* be translated, you can add the attribute `translatable="false"` on the `<string>` element, or you can define all your non-translatable strings in a resource file called `donottranslate.xml`. Or, you can ignore the issue with a `tools:ignore="MissingTranslation"` attribute.

By default this detector allows regions of a language to just provide a subset of the strings and fall back to the standard language strings. You can require all regions to provide a full translation by setting the environment variable `ANDROID_LINT_COMPLETE_REGIONS`.

You can tell lint (and other tools) which language is the default language in your `res/values/` folder by specifying `tools:locale="languageCode"` for the root `<resources>` element in your resource file. (The `tools` prefix refers to the namespace declaration `http://schemas.android.com/tools`.)


WARNING: [LogNotTimber] Using 'Log' instead of 'Timber'
 Since Timber is included in the project, it is likely that calls to Log should instead be going to Timber.


WARNING: [Overdraw] Possible overdraw: Root element paints background `@color/more_background` with a theme that also paints a background (inferred theme is `@style/AppTheme`)
 If you set a background drawable on a root view, then you should use a custom theme where the theme background is null. Otherwise, the theme background will be painted first, only to have your custom background completely cover it; this is called "overdraw".

NOTE: This detector relies on figuring out which layouts are associated with which activities based on scanning the Java code, and it's currently doing that using an inexact pattern matching algorithm. Therefore, it can incorrectly conclude which activity the layout is associated with and then wrongly complain that a background-theme is hidden.

If you want your custom background on multiple pages, then you should consider making a custom theme with your custom background and just using that theme instead of a root element background.

Of course it's possible that your custom drawable is translucent and you want it to be mixed with the background. However, you will get better performance if you pre-mix the background with your drawable and use that resulting image or color as a custom theme background instead.


WARNING: [UnusedResources] The resource `R.color.item_session_divider` appears to be unused
 Unused resources make applications larger and slow down builds.


WARNING: [AlwaysShowAction] Prefer "`ifRoom`" instead of "`always`"
 Using `showAsAction="always"` in menu XML, or `MenuItem.SHOW_AS_ACTION_ALWAYS` in Java code is usually a deviation from the user interface style guide.Use `ifRoom` or the corresponding `MenuItem.SHOW_AS_ACTION_IF_ROOM` instead.

If `always` is used sparingly there are usually no problems and behavior is roughly equivalent to `ifRoom` but with preference over other `ifRoom` items. Using it more than twice in the same menu is a bad idea.

This check looks for menu XML files that contain more than two `always` actions, or some `always` actions and no `ifRoom` actions. In Java code, it looks for projects that contain references to `MenuItem.SHOW_AS_ACTION_ALWAYS` and no references to `MenuItem.SHOW_AS_ACTION_IF_ROOM`.


WARNING: [ContentDescription] [Accessibility] Missing `contentDescription` attribute on image
 Non-textual widgets like ImageViews and ImageButtons should use the `contentDescription` attribute to specify a textual description of the widget such that screen readers and other accessibility tools can adequately describe the user interface.

Note that elements in application screens that are purely decorative and do not provide any content or enable a user action should not have accessibility content descriptions. In this case, just suppress the lint warning with a tools:ignore="ContentDescription" attribute.

Note that for text fields, you should not set both the `hint` and the `contentDescription` attributes since the hint will never be shown. Just set the `hint`. See http://developer.android.com/guide/topics/ui/accessibility/checklist.html#special-cases.


WARNING: [HardcodedText] [I18N] Hardcoded string "Speakers", should use `@string` resource
 Hardcoding text attributes directly in layout files is bad for several reasons:

* When creating configuration variations (for example for landscape or portrait)you have to repeat the actual text (and keep it up to date when making changes)

* The application cannot be translated to other languages by just adding new translations for existing string resources.

In Android Studio and Eclipse there are quickfixes to automatically extract this hardcoded string into a resource lookup.


WARNING: [HardcodedText] [I18N] Hardcoded string "2016.fossasia.com", should use `@string` resource
 Hardcoding text attributes directly in layout files is bad for several reasons:

* When creating configuration variations (for example for landscape or portrait)you have to repeat the actual text (and keep it up to date when making changes)

* The application cannot be translated to other languages by just adding new translations for existing string resources.

In Android Studio and Eclipse there are quickfixes to automatically extract this hardcoded string into a resource lookup.


WARNING: [HardcodedText] [I18N] Hardcoded string "\@fossasia", should use `@string` resource
 Hardcoding text attributes directly in layout files is bad for several reasons:

* When creating configuration variations (for example for landscape or portrait)you have to repeat the actual text (and keep it up to date when making changes)

* The application cannot be translated to other languages by just adding new translations for existing string resources.

In Android Studio and Eclipse there are quickfixes to automatically extract this hardcoded string into a resource lookup.


WARNING: [HardcodedText] [I18N] Hardcoded string "Rate Fossasia", should use `@string` resource
 Hardcoding text attributes directly in layout files is bad for several reasons:

* When creating configuration variations (for example for landscape or portrait)you have to repeat the actual text (and keep it up to date when making changes)

* The application cannot be translated to other languages by just adding new translations for existing string resources.

In Android Studio and Eclipse there are quickfixes to automatically extract this hardcoded string into a resource lookup.


WARNING: [HardcodedText] [I18N] Hardcoded string "Subscribe to mailing list", should use `@string` resource
 Hardcoding text attributes directly in layout files is bad for several reasons:

* When creating configuration variations (for example for landscape or portrait)you have to repeat the actual text (and keep it up to date when making changes)

* The application cannot be translated to other languages by just adding new translations for existing string resources.

In Android Studio and Eclipse there are quickfixes to automatically extract this hardcoded string into a resource lookup.


But there are! Its just the dev task.


`ExportInstance` has a `last_built` property. i guess i what are the advantages to having a separate model that stores the last_updated/last_accessed fields for an export instance?


I know right, super weird =/


It's entirely possible that that is a separate bug in that specific version of ImageMagick - that maybe only shows up for you on Ubuntu?

I added 6.8.9-0 to the versions tested against - https://travis-ci.org/Danack/imagick/builds/111665283 and the tests are passing there.

Also, 6.8.9 is in a version earlier than where I noticed the behaviour changed.....honestly I think the code in ImageMagick was just a bit flaky about getting the value exactly right. I'd suggest just ignoring that test, unless you need values to be that precisely accurate.


Thanks for digging and your PR, glad it worked out :)

I don't know why it's there, but @aldeed is one of the original creators of cfs, so he might have had a reason? Providing fixes upstream is always good i guess :)

To make this publish ready though, you would also need to increase the version number of the upload-http package (top of package.js) and the pinned versions in any other cfs package that pulls it in.


in general i think this is what we do. sometimes the base class will have a `wrap_correctly` method that does it


I fixed the accessibility issues https://github.com/mpc-hc/mpc-hc.org/commit/1f1cba3f607f9beb4ac568d4c18f681faa3ce964.

But since those are all upstream, I'll wait until I add grunt-accessibility in Bootstrap.


cn still has muramana you incredibly wise black friend


it's not very wise to reference to source code line of moving target. use some tag to reference by line, not `master` branch!


112796613_DesicDehum (Nigusse) - Win64-Windows-7-VisualStudio-12: OK (2120 of 2123 tests passed, 1 test warnings)

 * 1 test had: AUD diffs.
 * 1 test had: Table small diffs.

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


Excellent.


You're a bit late to the party. This was already committed, and later reverted because of that..

On 24 February 2016 19:38:15 GMT-08:00, xiong-gang <notifications@github.com> wrote:
>"SET_VAR $I" at line 1377 didn't set GP_HOSTNAME
>after removing HOST_LOOKUP, all segments will have the same
>GP_HOSTNAME, that is wrong
>
>---
>Reply to this email directly or view it on GitHub:
>https://github.com/hlinnaka/gpdb/commit/aada3a17f975e6f87be1ef6662512435e482754b#commitcomment-16316593


This should be a joke at the buttom of the README, for those who read it
all :)
On 25 Feb 2016 00:40, "Dylan Araps" <notifications@github.com> wrote:

> I've changed it but that point of it was that it's an inside joke. In the
> scene
> Regina says "Stop trying to make fetch happen" so I though it'd be funny
> to have as the screenshot in the readme. :)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/dylanaraps/fetch/commit/d4a518526135ef8a1493b7a07ecacff9a2abb2e3#commitcomment-16311859>
> .
>


i think the `ExportInstance` can return some of these things: https://github.com/dimagi/commcare-hq/blob/app-mgr-parent-select-filters/corehq/apps/export/models/new.py#L306


fff


aha i see


hi there not sure what I am doing and trying to learn


请问salt是怎么获取的？


Lesson 1！


Undefined is a valid left or right value add a test for
```
var s = stream(Either.right(undefined));
assert.equals(s.right(), undefined);
```
same for left


#471 #340


Looking at `git log -Gname_i -p Python/ast.c`, it's a sad, sad tale.

When the AST code first landed, it meant something pretty reasonable -- the index `i` corresponding to the function name. In that version, the relevant code was also responsible for handling decorators, so sometimes it had to skip over the decorators to find the name. https://github.com/python/cpython/blob/0d45400bf/Python/ast.c#L808

Then someone refactored to support class decorators, and the relevant code now always found the function name as child 1. Fine so far, but rather than make it overtly a constant, they kept this zombie name `name_i` around which was always just 1. Bad move, but at least the code is still truthful. https://github.com/python/cpython/commit/5d6153956#diff-4d35cf8992b795c5e97e9c8b6167cb34R838
(That's the Python 2 backport; for some reason the version in master now is in a squash of a bunch of different changes.)

Then someone else goes and adds PEP 3107 support in a big patch that probably nobody actually really read. They are confused (?) by this zombie name, and appropriate it for their own purposes to now track where the *colon* goes, only with a funky offset of 2 that comes from the variable's origin (and still name!) of tracking where the function name goes. Boom, now the code lies.
https://github.com/python/cpython/commit/a801935eb#diff-4d35cf8992b795c5e97e9c8b6167cb34R1011


23302ec73a10e2d8d433687295a560e23a12bee1 and done.


No, this looks good - although stylistically I prefer using unless instead of an if-not statement.  But either way is fine, so I'll just merge this.


Applied this one


Thanks! :1:


Sure. Huzzah for good static analysis.


Wrong logic we dont want to expose EIther to the user of the api except during `.mapAll`. If s.val is a right either return s.val.right not the either make the same change for the `.left()`.


hallo hallo


Oh, and you don't have to do the version thing, i was still in wiki-mode ^^P
Might better leave that for the publishers ;)


I know you are silly.


The partest model is saved in its own file (model-partest.RData) so no need to save .RData when it is run, but it won't hurt anything if you do!


<!-- PMABOT:SPACE -->
This commit contains trailing whitespace, what is prohibited in phpMyAdmin. Please check our [Developer guidelines](http://wiki.phpmyadmin.net/pma/Developer_guidelines#Indentation) for more information.

Offending files: libraries/Util.php


OK, so be it.


Yep, that looks good to me! Happy to help


Nice.


Write `.isLeft` and `.isRight` more like `.map` and `.mapAll` as static methods (use the static methods where possible) then create a bound version to put on the stream during creation.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 17164](https://teamcity.mesosphere.io/viewLog.html?buildId=145331&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 236 Build time: 00:09:40


Cool. With your changes in 072d6da I think it all looks good, except the one small comment that I didn't see make it into that, the TODO you already have for yourself, and Guido's suggestion to encapsulate your vector data structure into its own small helpers.


yes you are, you little monkey you


Aha. I knew you had it under control. Maybe I'll take it out then, since saving .RData does take time. I made other changes to all.r, so will fix it back manually. Thanks.


This seems fine. I don't know if we even capture semantic exception codes right now.


Is anything left from that i.MX31? :)


Should we also sleep for implicit even though we won't loop?


no need to add a color - the element is never shown while it has this class.


Nice change.


You don't need this


For those reviewing this one... just a tip that's way easier to review with:
```
git show --color-words --word-diff-regex=[^[:space:]\(\)/\*,]+ <commit>
```


Why is this not an array anymore? Are we assuming we always are connecting to the seed?


Checks like this are dangerous if `this.val.left = 0` it will be coerced into false. You need to actually check for `this.val.left !== undefined`. Also the !!(...) is the same as new Boolean(...) because you are and-ing something together you are guaranteed a boolean value to start with so the extra cast should be unnecessary.


Unless someone has stated otherwise, I think it's a good idea to leave methods like that to be explicit that they exist. This comes from the Zen of Python, but I wholeheartedly agree with the phrase, "explicit is better than implicit".


:) no worries.


I think it would be cleaner to use `.show()` and `.hide()` (or `.toggle()`) - this way you only need one css class.


A few comments. Question, can I start A, connect B to A, and then connect C to B and it will still work?


Fixed at [https://github.com/CapnBry/HeaterMeter/pull/31](url)


Can we use [assert_raises] (http://docs.seattlerb.org/minitest/Minitest/Assertions.html#method-i-assert_raises)?


deleting to update


As a tip use 0 and 1 as values to lefts and rights they catch bad boolean checks for example using `if(val.left)` instead of `if(val.left===undefined)`


Here we have a problem. AP_GPS is using double on purpose. Even if slower on STM32, it's really needed. With your re-declaration in AP_Math.h it would actually become a float here.

Some possible solutions:
  - Make AP_GPS to declare its own RAD_TO_DEG_DOUBLE and use it (then you can remove from AP_Math.h)
  - Make AP_Math.h to provide both RAD_TO_DEG and RAD_TO_DEG_DOUBLE regardless if `DBL_MATH` is defined.

I don't really have a preference in this case, but I'm leaning towards the second. Thoughts?


//***comment***//
Comment


Just saw this issue now.

I did
https://github.com/despairblue/jumpy/commit/d7d95d91112d35355450f9345c565db5074b287c?w=1
for clarity of fix...

Great fix Danny!  Thanks as always!

How do you like the new version.  Mostly pretty good right?  It's fast again.  Faster than ever?

Once in a while with some kind of resizing or something the labels jump to the edge.. or jump around.. the thing is so do other packages.  I've seen the linter do it.  It's an issue with the decorations, but overall was worth the switch over to markers/decorations I think.


Can you please make it so that if the file is not there, things still work? This is pretty useful for development. Currently the package fails loading if the version file is not present. So you cannot use it directly from the checkout.


deleting


since this define is more a generic one to tweak the behavior of AP_Math could it remain in AP_Math.h?


真的是这样？？测试


@morozkina Счетчик реализован не корректно, студент с оценками 2 5 5 так же пройдет, комментарии про stud остался не замеченным


インデント


Revert this, please


> Is -latomic required on x86 and x64?

No. It is not required on x86 and x64.
You mean that ac_check_lib should not be executed on x86 and x64, don't you?
If so, I'll fix it later.


Hmm, I see. How about just reordering so that "add all files we might eventually want" and "remove the ones we don't actually want right now" are right next to each other? Then it's possible to use "remove the ones we don't want" as the base for diffs to see the overall changes, just as if it had been a single commit that effectively squashed those two, but there's also the more comprehensive commit for that other use case. Though I'm not sure that other use case will quite be served in the end anyway -- the base commit for that purpose will inevitably be different for, e.g., the Python 3.6 version.

Yeah, if you keep around a branch containing these commits that should ensure GitHub won't GC them and the comments on them, and it provides a way to navigate to them. Even if you didn't, any link to the commits and their comments would continue to work, though it's possible they'd get GC'd and the links stop working.

That API for handling different Python versions sounds reasonable to me.


So here you are relying on AP_Math.h to be included after some other defines. Instead you should include here: AP_HAL/AP_HAL.h to get the CONFIG_HAL_BOARD definition and math.h to potentially get the definition of M_PI.


このインデントも


remove the extra negation and make it easier to read at the same time `!s.left()` -> `s.isRight()`


Compile this version of WebView with pawitp's patch: https://chromium.googlesource.com/chromium/src/+/40.0.2214.114


I like these changes. Should they also be available for HString so that it, too, can be extended?


インデントおかしいっす。


> You mean that ac_check_lib should not be executed on x86 and x64, don't you?

Yes. We don't want to link needless libraries.


math.h defines it explicitly but I think there's no advantage in doing so... ?


ここらへんもインデントっすね。


these look good


This should not be changed, not related to your PR or changes - it might be a fluke change due to timing issues


Ditto


Same here


OK, I'll fix it later.


Good catch lol!


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 17165](https://teamcity.mesosphere.io/viewLog.html?buildId=145342&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 236 Build time: 00:07:05


test


link_toのところでインデントがずれているところが多いっす。hamlはインデントがずれるとエラーがでるので一回動かしてみるといいかも。あと、link_to image_tagのところとそうでないところがあるので全てlink_to image_tagで統一しましょう。
後半の方はこまかく見ていないですが、まずは上記の点を、前半後半含め修正しましょう！

ロケットハッシュが全部なくなってるのは素晴らしい！！！あとlink_to image_tagを初めから使ってきたとこも良い！


This feature was comprised from an idea on this [issue](https://github.com/InternalError503/forget-it/issues/1)  and a few other similar suggestions from other users.


+razetime you made quite a mistake adding the uncompressed WAV for Silet Echoes :D


When can `is_ocf_ip(ipv6) != is_ocf_ip(ipv4)`?


I got the certificate error from `curl` when building the image, but not when I downloaded it on the host. So apparently having `ca-certificates` installed would solve the error. I was trying to install the minimum required packages (hence the explicit list of dependencies instead of something like `libqt5gui5`), but of course the better way is not to use `--insecure`. So I'll change as you suggested. Thanks @cdown ;)


Awesome job on the html2pdf stuff!

I just committed your code to the codebase:
https://github.com/openemr/openemr/commit/09b37d7334a13d6724b02e302ec7af3885dc314d

I also followed it with a commit with some very minor changes:
https://github.com/openemr/openemr/commit/5ef2c14518e75b4a849a8a9eceda8ea93bb3b3c5

I also updated the developer wiki page with your method:
http://www.open-emr.org/wiki/index.php/Html2pdf

-brady


FYI - received this message multiple times:

````
PHP Notice:  Undefined variable: a in /var/www/hubzilla/include/session.php on line 127
````


@duke12468 It's not technically a carat/exp hack, and these method(s) have been public for **years**. The two/three round boss mode can be accomplished several ways. The most notable being, to simply change the room's type state. As far as the carat/exp gain, they're using the renown slot/equipment swap method.

As I've repeated multiple times, the carat and exp hacks featured in PyE's source **still** work with little to no effort.


This is probably the smallest nitpick ever, but I think the convention for keybindings is to have them not seperated by spaces.


training ... machine code  by hand and brain


Regarding the html2pdf stuff, do you think it would make sense to remove the examples directory in the TCPDF package:
https://github.com/openemr/openemr/tree/master/library/html2pdf/vendor/tecnickcom/tcpdf/examples
(if this isn't needed, then it's always good to remove for security purposes)


增加一个文件。


@mmmries what is the reason for not having a default_exchange any more?


rest of the patch LGTM


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 17166](https://teamcity.mesosphere.io/viewLog.html?buildId=145372&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 236 Build time: 00:07:05


af3.0上传进度展示


Regarding the Finis project, it's still in the works. I'm working on a more complex and secure front end, that will map with the level 3 REST api.

http://prntscr.com/a7msg4

My work schedule has interfered with my development process slightly, but the web application will still be live before the end of the month as originally planned. I'll also be making a lot of the web application's front end open source via my personal Github.


增加


I've merged this as this isn't that important and I don't wanna pester you more, but for your benefit and learning more about Rust: This collect here is also not required, because `extend` takes an `I: IntoIterator<Item=T>`. So, since `new_headers` is already an iterator, it trivially satisfies this trait. What you're doing here, on the other hand, is first creating a temporary vector and then having it converted into an iterator, by using its implementation of the `IntoIterator` trait.

Basically, just giving `new_headers` to extend would have sufficed -- the code would have been more explicit, clean, and potentially more efficient (not sure on the last one, since in optimized builds, llvm might be able to see that there's no point in the temporary vector, but the other case, we'd know that for sure).


spasinya coy


KURANG AJAR


You don't really need the if block for skill_list here, it's redundant.

The removing and re-adding of skill list are done automatically if you pass `skill_list: "skill1, skill2"` to the update method.


If understand this correctly, if you're passing this as keyword args you should use `task_name` but if you're passing it inside `q_options` you should name it as`name`. Is that correct ?


I agree that relying on old libs for such case is risky, we need to check the impact of known security issues (cc @daniel-beck)

:sadpanda: regarding the API deprecation policy.
It does not allow to bump the Guava version after 2.0 :(


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 17167](https://teamcity.mesosphere.io/viewLog.html?buildId=145381&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 234 Build time: 00:03:51


`self.ipv4.packed[3]` should work under python3


Hi @arkenthera,
You can also use the format like
```c
// TODO(arkenthera) check if the values are valid
```
This will create a `TODO` - `check if the values are valid` & auto assign it to `arkenthera`


Can you check for the existence of spinnaker service in a more portable way with:

status spinnaker
SPINNAKER_EXISTS_CHECK=$(echo $?)
if [ $SPINNAKER_EXISTS_CHECK != 0 ]; then
  initctl emit rush_triggered
  exit 0
fi


ok. sure


|Assembly|Errors/Failures/Totals|
|---------------|---------------|
|DomainsCICD.sln|0/4/302|

|Assembly|Coverage (%)|Cyclomatic Complexity|
|---------------|---------------|---------------|
|Summary|14.11|63.0|
|domainsCICDApi|38.41|7.0|
|cicdInfrastructure|68.63|5.0|
|godaddyCICDTests|73.27|7.0|
|cicdTestingInfrastructure|60.47|5.0|
|Octokit|0.36|63.0|
|cicd|69.17|11.0|
|IntegratedTests|0.0|3.0|
|VersionControl|47.08|8.0|
|SlackNotification|67.16|10.0|
|Notifications.Infrastructure|0.0|0.0|
|VersionControlTest|86.78|4.0|
|GitHubNotifier|40.91|6.0|
|GitHubBotTests|100.0|1.0|
|cicdInfrastructureTests|80.0|1.0|


коммит должен содержать сообщение, no message нельзя


and:

PHP Notice:  Trying to get property of non-object in /var/www/hubzilla/include/session.php on line 127


all the get_distance_* variant functions are still in the other header. Should probably be removed in favor of these ones?


totally NOT ready to merge, but here so you can check it out. PS, SO SORRY if I caused this, but I noticed in the current master the refactoring you did to get rid of all those "directory"'s is missing.


@tsearle @nbansalvox In order to be able to update the configuration sent to jssip when reusing a voxbone object, I added the following line.


I am OK with this commit. But Could you please help to provide more background about the use case of this API?
For Android WebView,   this API is used at:
WebViewContentsClientAdapter.useHttpAuthUsernamePassword. But I found useHttpAuthUsernamePassword was not used.


Sounds good :)

Regarding example directory in TCPDF package, we can remove that but I did not want to make changes inside vendor directory because if we want to update TCPDF or FPDI in future we just need to run composer update and then example directory will back again with updated package but if you want we can remove it


Hi,

If we are stuck needing to upgrade Smarty, would it be possible to only use one instance of Smarty for all 3 of them in openemr/calendar/phpgacl?

If this is possible, then would recommend installing the new smarty version into the following directory:
library/smarty/

For example, I think all the smarty code in the openemr smarty can be redirected to new package here:
https://github.com/openemr/openemr/blob/master/library/classes/Controller.class.php#L3

Then we can direct all 3 of them in openemr/calendar/phpgacl to use the new smarty library. Recommend keeping the cache and compiler in the same directories(interface/main/calendar/modules/PostCalendar/pntemplates/compiled and interface/main/calendar/modules/PostCalendar/pntemplates/cache). Then we can ensure we have all the custom smarty plugins as we converge them all through the same smarty class.

Then when done, we can remove the 3 outdated smarty packages.

If we did this, would be a super awesome code clean up also.

-brady


Yes, was thinking of doing that. This import only needed to maintain compatibility with existing applications.


You should :)


@chintanb Puchii dekho bhala :* :*


`[]byte` internally gets encodes as base64 JSON automatically


Stanford Univercity, developing Apps for iOS8, lesson 7 : Mutiple MVCs, demo


This commit is already merged. Switch Yii2 requirement to dev stability and run `composer update` in order to get this commit.


Hi Faidy , I have installed the latest Appium 1.4.16.1 . The runcuke file I converted it into bat file since I use Windows system and tried running it , but Its opening appium application but not starting it with the port number u mentioned.

These are the error logs shown on command prompt.

`D:\AppiumparallelFaidy\appium-android-parallel-exec-master\cucumber>#!/usr/bin/e
nv bash
'#!' is not recognized as an internal or external command,
operable program or batch file.

D:\AppiumparallelFaidy\appium-android-parallel-exec-master\cucumber>appium -p 44
91 -bp 2252

`

Can u please help me to run appium scripts in parallel I am a fresher in mobile automation.


instead of adding the ifdef, could you add the proper definition into `libraries/AP_Common/missing/cmath`. Then you will ifdef inside the header for HAVE_STD_ISNAN. The idea of this is to later allow the build system to check what the toolchain has.

I think just a "namespace std { using ::isnan; }` would suffice there, right?


Just wondering, does this throw a traceable event to the reason that the world was unloaded? If not should it? if it does throw an event, is it cancellable? what happens if it's cancelled?


Looks good but I think you forgot to add requests.R..


Hello,

May be provide:

- manpage
- dynamic library

What do you think about this ?


I actually finished this within an hour after making this repository, but I didn't commit it until now.


Not necessarily.  BlackBoxes in Chisel2 behave as normal Modules for the C backend, so that you can write a behavioral description of the module inside the BlackBox and simulate it.  They emit no Verilog, so you can substitute in the real implementation of the BlackBox for phyiscal implementation.


テスト初期状態へ変更した


Hello,

May be provide:

    manpage
    dynamic library

What do you think about this ?


以前にも言ったかもしれないですが, 明らかな conflictがない限り, masterをマージする必要はありません. PRを行うブランチはコンフリクトなくマージできれば何の問題もなく, 最新版する必要は全くありません. conflictの解消が面倒くさかったらブランチを破棄して, 最新版から新しくブランチを作った方がよいです(`git cherry-pick`で簡単に別ブランチのコミットも取り込める). 特に今回のように conflictした状態のファイルを誤って突っ込んでしまうなら.(近藤さんにより修正されています. https://github.com/CPFL/Autoware/commit/8945cd69d33ae4f9b05e0220e1d6ca8181bd9ff8)

conflictは場合により避けられませんが, なるべく発生しないように **変更量は小さく, 早めに PRする**というようにした方がいいと思います.


We've had a lot of maintenance issues with configurable values and defaults. In ActionSubscriber there is pretty good reason to deal with this maintenance, but for publishing it is so easy to create your own method like:

```
def publish_event(message, routing_key, options={})
  ::ActivePublisher.publish("events", message, routing_key, options)
end
```

That gives you a default exchange at your application layer (or library layer if you have a private gem). In this case it is no more difficult for the client to provide this than it is for them to use a configuration option so I would prefer to let them handle defaults.


45f2f19f9998324c5098a5b0b2b36d491e459c04 按此做了处理


@ap is there a specific reason for leaving out `</li>` in these? They are the only tests which break on my patch for #119


line 47-49
```
@pioneertype_images = @pioneertype.pioneertype_images.sub
```
で取れると思います。


oh you fixed the typo :) .


I can give it a try but latest smarty package have many method name changes like they have changed method names with underscore to lower camel case

I think if I will be able to upgrade smarty in calendar module then I can update on the other part as well

I am going to try it and will keep you updated


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 17168](https://teamcity.mesosphere.io/viewLog.html?buildId=145407&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 236 Build time: 00:04:26


Looks good, but gives me conflict to apply without the others


I like coffeescript as well


また、2回と決めてしまうと、画像が1枚だったとき、3枚だったときに不都合がありそうです。


Hopefully fixed in 21719bdb8ae0c41b28144488d361a8f9fe340842


|Assembly|Errors/Failures/Totals|
|---------------|---------------|
|DomainsCICD.sln|0/4/302|

|Assembly|Coverage (%)|Cyclomatic Complexity|
|---------------|---------------|---------------|
|Summary|14.13|63.0|
|domainsCICDApi|38.41|7.0|
|cicdInfrastructure|68.63|5.0|
|godaddyCICDTests|73.51|7.0|
|cicdTestingInfrastructure|60.47|5.0|
|Octokit|0.36|63.0|
|cicd|69.17|11.0|
|IntegratedTests|0.0|3.0|
|VersionControl|47.08|8.0|
|SlackNotification|67.16|10.0|
|Notifications.Infrastructure|0.0|0.0|
|VersionControlTest|86.78|4.0|
|GitHubNotifier|40.91|6.0|
|GitHubBotTests|100.0|1.0|
|cicdInfrastructureTests|80.0|1.0|


Come on. You know what i am doing? No! You don't. Let me do,what i am doing. K?


For the method name changes, we can likely create wrappers in the smarty plugins directory to deal with that.


This is a dependency substitution test, the permutations related to the PR are in DependencyResolveRulesIntegrationTest


This has broken the code input for Verisure locks. No input field for code displayed anymore.


NEW OK


It's a thing in enterprise ruby-land. I've seen it still being alive.


They have their own SmartyBC (backward compatibility) I trying to integrate it


Probably just `LMMS_BUILD_APPLE` would be sufficient.


@shama: How about publishing a new release too?


Oh I see it now, it should be `code_format`, instead of `codeFormat`.


@plushyskirt
1) ConsoleApplication9 - не самое читабельное название
2) Переменные имеет смысл определять и инициализировать непосредственно перед их использованием.
3) workers a - читабельно, переименуйте в более ясное имя (b туда же)
4) Программа заканчиваться должна return 0 в случае успеха если функция int main ()


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 17169](https://teamcity.mesosphere.io/viewLog.html?buildId=145423&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 236 Build time: 00:03:49


@rafaelfranca Would you mind publishing a new version that contains this change? jQuery v2.2.1 includes a significant bug fix. https://github.com/jquery/jquery/commit/412c5dfb0d6301dc0b3d7c98325cc9908dd97511


Is that something that still makes sense for Chisel3, where there's really no distinction between C and Verilog (everything is just FIRRTL)?

The ability to do that appears useful, but overloading BlackBoxes could potentially be confusing to people who don't know about this feature. Perhaps this could (should?) be accomplished through some other mechanism?


Fixed 5c2b2c12ec850089d540a94a9d2e5012ecfdedd0


Hi i copied the your code & ran the function in R, and i am getting following error:
"Error in num == "best" :
  comparison (1) is possible only for atomic and list types"
I am getting the same error with my code and with another guys code. Could you help me out here?

Thanks
Bharat


Looks like this was added by @piranna : https://github.com/gulpjs/vinyl-fs/commit/35e9b3a68907167b51fa31edd0701ae24915fd84#diff-22158b56af64af8f8b937f00240fbba7R22

I think it was needed at the time, but may have become unnecessary (and wrong, apparently) after rebasing. In particular @Klowner added `end:false` to the pipe call in writeStream : https://github.com/gulpjs/vinyl-fs/commit/24119a862ad062f39ed15db767feff459843c2d8#diff-22158b56af64af8f8b937f00240fbba7R29

Does that make sense?


So this means we'll have .odex instead of .odex.gz?


|Assembly|Errors/Failures/Totals|
|---------------|---------------|
|DomainsCICD.sln|0/4/302|

|Assembly|Coverage (%)|Cyclomatic Complexity|
|---------------|---------------|---------------|
|Summary|14.11|63.0|
|domainsCICDApi|38.41|7.0|
|cicdInfrastructure|68.63|5.0|
|godaddyCICDTests|73.27|7.0|
|cicdTestingInfrastructure|60.47|5.0|
|Octokit|0.36|63.0|
|cicd|69.17|11.0|
|IntegratedTests|0.0|3.0|
|VersionControl|47.08|8.0|
|SlackNotification|67.16|10.0|
|Notifications.Infrastructure|0.0|0.0|
|VersionControlTest|86.78|4.0|
|GitHubNotifier|40.91|6.0|
|GitHubBotTests|100.0|1.0|
|cicdInfrastructureTests|80.0|1.0|


Actual changed code, actual fix.


Yep, will need to use SmartyBC since there is prevalent use of {php} tags which aren't supported in smarty 3.


Should I change this manually ? As I cannot update partial mismatches through God mode.


Then we need to skip the lazy loading I guess.


And BTW,https://chromium.googlesource.com/chromium/src/+/40.0.2214.114
This branch is not supported with M, M uses new api,i have already encountered the issue,and working on it. Maybe soon fix will be released. And about pawitp patch,pawitp patch only ignore egl_sync_failures which prevents browser from crash but our issue is different. eglCreateImageKHR ,it failed because of unknown textures from our lib,earlier it was ignored,and still create Image. That is the main issue.I already have found fix. So just wait/sleep. Let me and sandpox do the work.


I don't think manual changes are allowed anymore, instead - do this, revert this file to master state and re-run God mode on this file and ensure that we do not have unintended changes by looking through the changes that God mode made

For instance, ```highlight``` related changes should not result in ```disabled``` changes


**are**


Sorry, Booth. I used [Lombok](https://projectlombok.org/index.html). [IntelliJ](https://github.com/mplushnikov/lombok-intellij-plugin) supports it just fine. Boilerplate sucks, especially `equals` and `hashCode`. Also builders. Also getters. It all sucks. Lombok is nice.


reviewed


This is legendary.


well, that didn't merge right...


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 17170](https://teamcity.mesosphere.io/viewLog.html?buildId=145434&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 236 Build time: 00:03:36


Принято


The question mark is not a valid character in a file name.


Awesome, thanks for looking into it!.

I don't think you want to assume that. Ideally the packager should be able to support HMR even if you're developing different *apps* (even on different platforms if you want). I think the code should actually be easier if you remove that assumption :).

Each client, identified by the bundleURL (entryFile + platform) would cache all it's dependencies information (`dependenciesCache`, `dependenciesModulesCache`, `shallowDependencies`) and when a file is changed you'd simply have to iterate over all the clients and do what we're currently doing. Does this makes sense?


Also, maybe we should move the discussion to a separate issue to avoid spamming people here :D.


nrrd reader and writer were fixed.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 17171](https://teamcity.mesosphere.io/viewLog.html?buildId=145445&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 235 Build time: 00:04:11


I don't have any issues with Wi-Fi.


Wat is a cn


I only care about EUW


For more examples of DRF stuff ... https://github.com/erasche/shed/tree/master/api_drf


I am doing the same. But how can I ensure that the mismatches , that were not due to highlight part, dont change. As God mode changes all the mismatches. Is there some way that while matching expected file to the actual, God mode confirms every mismatch to change with me ? If not how can I do that ?


113837367-Fanger-PMV-Equation-Bug (RKStrand) - Win64-Windows-7-VisualStudio-12: OK (2122 of 2122 tests passed, 0 test warnings)

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


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 17172](https://teamcity.mesosphere.io/viewLog.html?buildId=145446&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 236 Build time: 00:07:54


啥？


Awesome!


@pat, you missed a comma

our CI is fails now =)


As discussed in Haacked/Scientist.net#12 this needs to be protected by a lock because it's not safe to call `Next` simultaneously from multiple threads.


I prefer it in metric units ;-)


`input-group-btn` would provides better rendering in this case


I don't get why do you put search field in a list


also you should use standard html structure from bootstrap, this will prevent unexpected behaviors and easiest maintenance.


Pretty sure it's singular when **no** is involved :wink:


Shit wala na update ang example nyahaha @wadiwasi/wadiwasi


It's plural so it doesn't make sense to me. But well, not a native English speaker.


Is this a brute-force attack?


Is the newest version of bundler not 1.11.2?


cc @MichaelHatherly I think this can be removed given the function redesign?


 ryu/lib/ofctl_v1_3.py：90行目の部分ではデータタイプが'base64'を確認した上でbase64.b64decode()を動作させています。
189行目の部分でbase64.b64encode()を動作後にデータタイプがact.dataが'base64'かを確認していませんが、確認処理は必要ではないのでしょうか？


|Assembly|Errors/Failures/Totals|
|---------------|---------------|
|DomainsCICD.sln|0/4/302|

|Assembly|Coverage (%)|Cyclomatic Complexity|
|---------------|---------------|---------------|
|Summary|14.12|63.0|
|domainsCICDApi|38.41|7.0|
|cicdInfrastructure|69.54|5.0|
|godaddyCICDTests|73.45|7.0|
|cicdTestingInfrastructure|60.47|5.0|
|Octokit|0.36|63.0|
|cicd|69.17|11.0|
|IntegratedTests|0.0|3.0|
|VersionControl|47.08|8.0|
|SlackNotification|67.16|10.0|
|Notifications.Infrastructure|0.0|0.0|
|VersionControlTest|86.78|4.0|
|GitHubNotifier|40.91|6.0|
|GitHubBotTests|100.0|1.0|
|cicdInfrastructureTests|80.0|1.0|


Thanks! Just pushed a fix :)


Good catch.


ryu/lib/ofctl_v1_3.py:87行目、88行目でact.dataかbase64かを確認し、どちらでもなければエラーログを出力するようになっていますが、ログ出力のみでそのまま処理を継続してもよいのでしょうか？
OFPActionExperimenterUnknown()の動作が保証できないのではないかと思います。


Disagree. Looks ugly and hard to work with.


@willdean I see that you split it here as you wanted from #478 but now if someone passes a Guid.Empty it will insert that into the DB and all assumptions wolud not hold anymore. I think a Create function is a very good point to check assumptions that are made to hold true, here for instance that you should not create an user with a Empty Guid.

I thnk this could then become one function with the default being Guid.Empty which means the Guid for this new user will be provided.


I think we're coming at this from two different angles.

From my perspective, this change is a fix for a security vulnerability in two different codebases that I maintain. The vulnerability is listed by both the [node security project](https://nodesecurity.io/advisories/61) and [snyk.io](https://snyk.io/test/github/barc/express-hbs). Currently every one of [handlebars dependencies](https://www.npmjs.com/browse/depended/handlebars) which use the 3.x branch, and every sub dependency of those projects, is being flagged as insecure when checked against the available tools.

As the fix isn't available on the 3.x branch, the entire ecosystem has to, currently, do a breaking update and handle several other changes in order to do a security release.

E.g. Ghost is dependent on handlebars through express-hbs. The only way that I can update express-hbs is to move to the 4.x branch, meaning I have to also figure out and publish a full set of compatibility notes, and all of the dependents of express-hbs have to do the same work.

It makes sense to do that when moving forward to get new features from the 4.x line. It does not make sense to do that when you want to update your project to get rid of a [security vulnerability](https://snyk.io/test/github/TryGhost/Ghost).

I'm not sure what the use case is in which adding escaping for `=` might cause a breaking change, but if this is a serious concern, then `3.1.0` makes more sense than `3.0.3`. However, I do think that this vulnerability fix needs to be available to the 3.x branch because handlebars is so heavily depended upon.


thanks. I noticed already 😃


Okay thank you for the update Veritas. When you do release your new project Finis I will support it 100%. Also thank you for clearing things up, I now know they basically just took the basics of wnd made a "c.@neetVeritasyou I


There are more changes then just an ok key fix... Please describe them better or all...


Would it be better to modify the extension from `.txt` to `.cer` and `.pem`, even though the string would be the same ?


This string makes Java 7 required. It is imposible to use Java 6 any more.
Class java.nio.file.Path added in Java 7 [(javadoc for Path)](https://docs.oracle.com/javase/7/docs/api/java/nio/file/Path.html)


添加或改变的绿色显示


Sync fork


Argh. No, just an editing mistake.


1. Difficulty: 7
2. Struggles: I found the obtuse triangle tricky – got side tracked into cosine / arcosine.... ugly.
I also found it difficult accessing the appropriate element in the bank account.
3. More info: Would it make more sense to have each account as an object within the larger accounts object?
4. Time: ~3 hrs


> Where did I 'break' code style?

Omitted spaces around `<` and C++ style comments.

> posix_memalign fails if size is < sizeof(void*)

OK, yes this is mentioned in my man page. Not sure how we missed this before. But then we should only do it for posix_memalign() and not for other implementations (similar to the weird size fix for `aligned_alloc()`). Same goes for the Sparc fix. How about this?

```diff
diff --git a/src/memory.c b/src/memory.c
index 0034a62..e57288d 100644
--- a/src/memory.c
+++ b/src/memory.c
@@ -251,13 +251,13 @@ void *mem_alloc_align_func(size_t size, size_t align
        )
 {
        void *ptr = NULL;
-       if (align<sizeof(void*))
-               align = sizeof(void*);
-       if (size < 1)
-               size = 1; // memalign on sparc fails
+
 #if defined (MEMDBG_ON)
        ptr = (char*) MEMDBG_alloc_align(size, align, file, line);
 #elif HAVE_POSIX_MEMALIGN
+       /* posix_memalign requires a minimum alignment of sizeof(void*) */
+       if (align < sizeof(void*))
+               align = sizeof(void*);
        if (posix_memalign(&ptr, align, size))
                pexit("posix_memalign ("Zu" bytes)", size);
 #elif HAVE_ALIGNED_ALLOC
@@ -270,6 +270,9 @@ void *mem_alloc_align_func(size_t size, size_t align
 #elif HAVE_MEMALIGN
        /* Let's just pray this implementation can actually free it */
 #if defined(__sparc__) || defined(__sparc) || defined(sparc) || defined(__sparcv9)
+       /* Needed for Sparc's memalign() */
+       if (size < 1)
+               size = 1;
        if (!(ptr = memalign(align, size)))
 #else
        if (!(ptr = memalign(&ptr, align, size)))
```


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 17174](https://teamcity.mesosphere.io/viewLog.html?buildId=145454&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 236 Build time: 00:04:24


Hi, thanks guys for the great job, this library is really useful. Any chance we can include this commit in a new tag, ie v2.9.1 so I can fix the dependency in my projects. I had to point to dev-master#7555ea6 because of hundreds of deprecation errors in my functional tests. Cheers


No problem..
That's software development...test test and then...test again ;)

On Wed, Feb 24, 2016 at 10:47 PM, Lukáš Hozda alias magnusi <
notifications@github.com> wrote:

> Thank you for fixing it. I am still in process of learning how to use
> markdown, so I unintetionally do some stupid stuff like that from time to
> time.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/c2lang/c2compiler/commit/538a30d04d15683dd60dea76f041d46d80025cb1#commitcomment-16310512>
> .
>


####  bug Screenshot :
![bug](https://cloud.githubusercontent.com/assets/15491308/13312047/619b3a1e-dbce-11e5-969f-8b51365a803d.gif)
#### ----------------------------------------------------------
####  fixed Screenshot :
![fix bug](https://cloud.githubusercontent.com/assets/15491308/13312062/8d98195c-dbce-11e5-8158-7a7ed6c1837e.gif)


To early this morning :) where is it?


@davidhadrianus89 mohon perhatikan coding standard, untuk indenting adalah 4 spasi. Di geolevels.py masih menggunakan tab.

Di Notepad++ ada fitur Edit > Blank Operations > TAB to Space, untuk Sublime Text coba refer https://css-tricks.com/changing-spaces-tabs-sublime-text/


Dangling `` ` `` before `` `ngClass` ``.


Thanks for your reply ! I am very excited about using vim-mode in Atom and get more involved with building the tools one needs to be efficient when editing code.
Shameless plug: I am currently developing a [plugin on my own](https://github.com/Byron/atom-byrons-edittools) which will me with that, and I am sure that VMP is going to affect the way I think about things !


Is `img` really needed?


_"mpaa"_  -->  _"title"_ (?)


what compiler did you need this on? I thought --std=gnu99 implied -D_GNU_SOURCE


@yargalot: which node.js version are you using? I get a lot more changes with `npm shrinkwrap --dev`.


good spot!


@davidhadrianus89 tolong sesuaikan geolevels/models.py supaya mengacu ke ERD terbaru di https://github.com/geoenvo/opendims/issues/11#issuecomment-188042323.

Yang perlu dilakukan adalah poin2 berikut:
* Penambahan tabel RT
* Perubahan nama field poly menjadi polygon


Oh, don't forget to push the git tags too.


Хорошо бы тесты написать на алгоритм разбора.


_"good way"_  -->  _"a good way"_ (?)


Hi Ben, looks interesting. Though I'm bummed I've chosen a dumb/confusing name for this implementation now. Naming things, huh? :/


Yeh I wasn't sure about this one. The existing debug_marker header reused that enum since it's used for precisely the same purpose as debug_report. We could create our own but it feels better to have a dependency on debug_report rather than duplicating that enum. Also if any future extensions come along and add new object types, they are very likely to update debug_report's enum (since it will be a very common/well used extension), and if there was a unique one here in debug_marker it might get forgotten.


We've never followed branching in the line sense, more of semver as a method to advertise that something may be breaking as the code line progresses forward. There are a few other things since 4.0.0 proper which are flagged in security warnings due to upstream projects ([4.0.5](https://github.com/wycats/handlebars.js/blob/master/release-notes.md#v405---november-19th-2015)).

At this point we don't really have the resources to maintain two code lines, i.e. myself, but if you're willing to maintain a security codeline for 3.x then we should talk to @wycats to make that happen.


Changed 'enviroment' to 'environment' in the hyperlink text.


Good catch but I'm comfortable making Java 7 the requirement.


function gì đây. nhớ comment khi viết code nhé


yes. fixing it


Never used?


đặt id cho tử tế vào nhé


Caching, naming things, and off-by-one are the bane of our profession. Looks like you nailed it. =)


This was testing a exercise to for better understanding of the topic.


empty line, please remove :)


Hi Sean. I thought I'd comment on this line since I wrote it originally. The thought behind using bool explicitly rather than var is that with these Mediator calls it's not really clear in the code what type you get back. I went the route of explicitly defining the type to make the code a bit more readable in these cases. Happy to go with whatever the team agree is the best standard though as it doesn't make too much difference.


poor @Zaffy  :( ^^


@shahmishal Oops, sorry, I think this was my mistake. Thanks for fixing this, I hope you weren't inconvenienced too much by it!


The `MAYBE_INLINE` macro is actually trying to `always inline` (the MAYBE is somewhat misleading) so that macro should be used with some care. But in this case, that is exactly what we want.


The problem is indeed that the bvals have non-integer values with zero
variance, resulting in the b0 shell being selected instead.

Fixed it in the script to make sure it would not fail early users. In the
long run, it might indeed be better to support it in the backend. Will have
a look.
On 25 Feb 2016 12:51 a.m., "Robert Smith" <notifications@github.com> wrote:

> @bjeurissen <https://github.com/bjeurissen> What precisely was going
> wrong here? Making them floats was primarily because these values get used
> to create file names (might change it to round explicitly just for that
> purpose); but even if it's ~ 0.5 off the true mean, I would have thought
> the shell selection should still work? Or do you have a non-integer b-value
> shell with no variance? Either way it might be worth having a peek at the
> Shells class operation here.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/MRtrix3/mrtrix3/commit/d1db05417eb17d659cbe7bd64e48e45b9cd01fa2#commitcomment-16313233>
> .
>


_"href"_  -->  _"ng-href"_ (?)


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 17179](https://teamcity.mesosphere.io/viewLog.html?buildId=145512&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:00:19


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


At the moment I said that these commands are only valid for primary command buffers. We could also support secondary command buffers but then I think we would want to make the restriction that marker regions don't cross between primary and secondary - ie. don't push in a primary and pop in a secondary or vice-versa.

The MarkerEnd could be a in a different QueueSubmit, the only restriction currently is that they have to be submitted in the right order. I'll try and clarify that.


Yep, was using github desktop, did't realise it wasnt pushing tags


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 17177](https://teamcity.mesosphere.io/viewLog.html?buildId=145496&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 2 Build time: 00:02:02


using 4.2.4 atm


_"has a length"_  -->  _"'s length is > 0"_ (or something)
(All arrays have a length; but it can be 0.)


This is not a forward-compatable solution either if sa-mp increases this limit. IsValidVehicle should be used, it's a native not exposed via any default include but works: native IsValidVehicle(vehicleid);


电子排班系统源码


OK, then it must be because of that. 5.x is using npm 3.x.


Dangling `*`.


Perfect! We are already done with 27% of the functions :)


Adding athenahealth.

Contributor: Mimi Liu, Architect, mliu@athenahealth.com
Source: internal headcount


got it


Is there a reason to include NSDate tests if including NSDate in the library is still being debated?


Most projects use phantomjs and not the prebuilt one, so this results in downloading two phantomjs.  What's the gain from using the prebuilt package?


it needs a `return` after the notification


Well it is the relevant bit of change log from distlib 0.2.2 (cf previous changelog with 0.2.1).
Maybe it should be removed.


Alternatively, you could do: `return "\n".join(map(str, game))`


It doesn't feel like dbschema needs a model.  Just pull it from the db with a query.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 17176](https://teamcity.mesosphere.io/viewLog.html?buildId=145495&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 236 Build time: 00:08:05


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 17178](https://teamcity.mesosphere.io/viewLog.html?buildId=145497&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 235 Build time: 00:09:03


This should be in the API controller, not the web controller.


thanks a lot.


It tests the extensibility of the library. It is also an example for people
to drop directly into their projects or use as a starting point to get date
support.

On Thursday, February 25, 2016, Brian Mullen <notifications@github.com>
wrote:

> Is there a reason to include NSDate tests if including NSDate in the
> library is still being debated?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/utahiosmac/Marshal/commit/4f6bfedc24668703a2b8043d7f57d96b69d333b7#commitcomment-16318606>
> .
>


I noticed you are using the same pattern for all directives, so this is fine (I guess) :smiley:


Cool. Just confirming for another patch. :+1:


In case you lost the combined commit. here it is (after some fiddling)


fix key OK` +			if not self.in_menu:
+				self.toggleShow()`


zdfzzgf


fgdfdg


`res[i]` would be even faster, because it does no bounds checks.


コメント１つ目に対して：
　口頭でも確認させていただきましたが、base64.b64encode()したあとは必ず文字列に変換できている
　ことを前提としているため、data_strの状態の確認は不要かなと思います。

コメント２つ目に対して：
　指摘のとおりですので、修正致します。

以上、宜しくお願い致します。


siiiiiiick!


Replace strncpy with memcpy, maybe?


Yeah, I see that. I don't know what caused it. Maybe Git decided that it was a mistake when it was calculating it's merge. We'll fix it first thing tomorrow.


sessionStorage.setItem(currentDir, this.$container.scrollTop().toString());


Double clear :)


Right. We're saying there is nothing for that purpose. If we wrote `ng-href` someone might think we meant that it DID exist.  It's a guess on our part.


It would be nice to mention the difference or provide some link for reference :smiley:


@modocache No worries.

Its causing this failure:
```
$ swift/utils/build-script --preset=buildbot_incremental,tools=RA,stdlib=RA
swift/utils/build-script: using preset 'buildbot_incremental,tools=RA,stdlib=RA', which expands to swift/utils/build-script --test --validation-test --ios --tvos --build-subdir=buildbot_incremental --release --assertions --llbuild -- --reconfigure --verbose-build=1 --build-ninja --build-swift-static-stdlib=0 --build-swift-stdlib-unittest-extra=1 --compiler-vendor=apple --skip-test-ios --skip-test-tvos --skip-test-watchos
Traceback (most recent call last):
  File "swift/utils/build-script", line 995, in <module>
    sys.exit(main())
  File "swift/utils/build-script", line 991, in main
    return main_normal()
  File "swift/utils/build-script", line 893, in main_normal
    "--install-symroot", os.path.abspath(args.install_symroot),
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/posixpath.py", line 360, in abspath
    if not isabs(path):
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/posixpath.py", line 54, in isabs
    return s.startswith('/')
AttributeError: 'NoneType' object has no attribute 'startswith'
swift/utils/build-script: command terminated with a non-zero exit status 1, aborting
```


I don't think this change is necessary. You can just use `make watch` to recompile constantly riot on any change. `make riot` needs to clean the `dist` folder because this folder is temporary and it could contain stuff that we don't want to publish ( the `make release` picks all the files from the dist folder and brings them to the root of the project) .


Feel free to come up with something and submit a PR. We're moving on.


controllerはページをまたぐと管理が面倒になる。
ページごとに作る、共通にする。
mvcはもともとservletにはない考え方なので、今回は独自にMVCパターンを使った作りにする。


You set its default value few [lines above](https://github.com/smarterclayton/origin/commit/be766800915f66ea78633c0a5987965fc6778dcf?diff=split#diff-df3b6baf7e3b18ed4b1ff84012467504R121). This way its length can be zero only when invoked with `--service-account=''` which I doubt will ever be the case.


That's not the right place for that error message. The 'environment variable' message belongs in the bot itself. (The kit doesn't care where the URL comes from in the config, it could well come from some far better source).


Here will be good to have comment with explanation of regular expression


这两个变量没用？


`limitTo` also supports a starting offset as a 2nd param.
It might be useful to use it and point out the difference in param orders.


Commit reverted to unblock master branch.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 17181](https://teamcity.mesosphere.io/viewLog.html?buildId=145532&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 236 Build time: 00:03:43


Phantomjs was deprecated in favor of prebuilt


As far as I know there is no way to do that, we will need to eyeball the changes and be mindful that the change is correct - and that's what reviews are for too, to make sure that the changes are correct if the person who is developing it missed it :)


LGTM


In `filter` above, there is an extra `:marked` under `none` (not sure which one is the "correct").
Just saying...


Could you tell to how to get DMS or Lattitude and longitude using x,y coordination?


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 17182](https://teamcity.mesosphere.io/viewLog.html?buildId=145533&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 236 Build time: 00:07:05


@zalox tagging you in case you missed the two messages above


Oh yea, forgot to remove it.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 17180](https://teamcity.mesosphere.io/viewLog.html?buildId=145513&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 236 Build time: 00:08:19


Missing `:marked` ?


测试能写不


I agree.

I wish Github would allow us to make an API key for an entire Organization, but [they don't support that feature right now](http://stackoverflow.com/questions/31159275/how-do-i-generate-the-github-oauth-token-for-organization-accounts).

However, I will make a test user named `dvcoders-dummy` and issue an API key we can use for testing purposes.

Kyle also created a test user account which we can send invitations to called [delete-guy](https://github.com/delete-guy).

I will let you know once I have our testing API key setup!


Thanks for the tip! Yeah, I modified the MCStats implementation slightly just to get it to work at all, but I've not tested all the features. I'll be sure check out your client and integrate it if I like what I see!


got it


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 17183](https://teamcity.mesosphere.io/viewLog.html?buildId=145553&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 236 Build time: 00:07:13


BTW, Python supports chained comparisons, e.g.

`if not (0 < x < 2 and 0 < y < 2):`


@erikkemperman thanks for digging into this.  I think we are good with it being handled by autoClose then and it is fine I removed it.  I hope to put some finishing touches on this when I wake up.


[]を増やしてemailのバリデーションを管理


I don't see it. The display is fine


新しいファイル・既にあるファイルかどうかはfileidではなく別に与えられたキー("s")で判別されていました。
そのため、fileidを新しくセットする方針とは別のやり方を検討したいと思います。


Perhaps, it's better to pass `int`s  to `make_move`, i.e.: `make_move(play, '0', int(input("insert x -> ")), int(input("insert y -> ")))`


@hustKiwi Should be ok.. thanks


The pattern is to throw from ed.cpp and try-catch-rb_raise in rubymain.cpp.


I've got a changelog generator...


Fixes #36


a


Do we ever need the `encrypted` option now? I thought we were leaning towards omitting it, if we are doing that, why have the option at all given it only works if `cipher` is provided?


修正しました。
https://github.com/takahashiminoru/ryu/commit/17ebbd89a0b6b4da4cd21773b23a8e27d2483a98


I think this should be:

```py
if args.install_symroot:
    build_script_impl_args += [
        "--install-symroot", args.install_symroot,
    ]
```

Earlier in the `build-script` a check is made that `install_symroot` is an absolute path, and the build script raises an error if it is. So we don't need to convert it here.


_"even after minification"_ - LoL, cheateeeer :stuck_out_tongue:
Yeah, you just need a whole new language (TypeScript) and a compile step.
All ng1 needs is a tiny little extra processing step (`ngAnnotate`).


test


Not following? (what method are U referring to?)


The APIs document is at https://docs.google.com/document/d/13GjIizusc7iUZq7TuVrsp8JZ2eEx52xUtoX9uRNzFgc/edit?usp=sharing


> (provided if provided else default)

Not keen on this wording given other developers who may not speak English as a first language have to use this as the spec.  Also typo in minimum (mininum).

Can it not simply be:

Checks that the provided options are valid and self-consistent as best it can, raises an exception if not. At a minimum, this should include checking the calculated keyLength is a valid key length for the encryption algorithm (for example, 128 or 256 for @AES@)


:+1: yes, should be fine to remove now.


As above, I propose we remove this attribute completely


aaa


A comment


没用


corbel.validate.value('id', id)

el que tiene que cambiar es al construir dentro la url, en el método anterior (rescue) lo has hecho bien


## Task 1

Previously, you wrote a program that used a recursive function to compute the sum of the integers from 0 to 1000 (inclusive).  This time, I want you to accomplish the same thing, but using a `while` loop. Compute the sum of 0 to 1000 (inclusive) in a function using `while`. Return the sum of these numbers to the main loop and print the result. Write the program to include a `main()` function and the `ifmain` statement.

- [ ] Program is correctly formatted with shebang, description, code (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program computes sum using a `while` loop in it's own function (1 pt)
- [ ] Program correctly computes and displays sum of 0 to 1000, inclusive (1 pt)

## Task 2

Above, you wrote a program that used a `while` loop to compute the sum of the integers from 0 to 1000 (inclusive).  This time, I want you to accomplish the same thing, but using a `for` loop. Compute the sum of 0 to 1000 (inclusive) in a function using `for`. Return the sum of these numbers to the main loop and print the result. Write the program to include a `main()` function and the `ifmain` statement.

- [ ] Program is correctly formatted with shebang, description, code (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program computes sum using a `for` loop in it's own function (1 pt)
- [ ] Program correctly computes and displays sum of 0 to 1000, inclusive (1 pt)

## Task 3

In Python, we tend to use `for` loops over different types of iterators much more than `while` loops - they're more flexible and iterators are **everywhere** in Python.  Write a program that includes 5 different functions, each of which should demonstrate the use of `for` over **different types** of iterators. Your program should include these five functions, a `main()` function that calls all the other functions, and the `ifmain` statement.

- [ ] Program is correctly formatted with shebang, description, code (1 pt)
- [ ] Program includes `main()` function and "ifmain" statement (1 pt)
- [ ] Program contains 5, correct subfunctions that use `for` over **different types** of iterators (5 pt)
- [ ] Program's `main()` function calls all subfunctions (1 pt)
- [ ] Program runs and correctly demonstrates use of `for` (1 pt)


There seems to be a single space indentation for this file. Let's do two spaces.


updated to work on the photon


Or just `git add std-lib`. I'm using SourceTree so I'm unsure of the exact command. There are some notes in [HACKING](https://github.com/agda/agda/blob/master/HACKING#L172).


@173210  tried it in my fork 105d5a8c5b6182bf9d11c04db36de8590768e0dd, looks wokring fine. Still an intermediate commit along with other changes, so just FYI.


@mizapf, was sagst du dazu?


I like `ngAnnotate` too. Can't assume it. Someone who know about it (like you do) will make the adjustment.

You're not wrong about the baggage of TS (or Babel). But it does give back in so many ways ... and this happens to be one that someone coming from A1 might wonder about.


Copy and assignment do different things for this class?


Если функция не принимает параметров, то `() ->` писать не обязательно. Можно просто `->`


This should add support for OSX after a `brew install coreutils findutils gnu-tar` #10


Just noticed 'X-Permitted-Cross-Domain-Policies' is listed twice.


emails work as normal in safari (iPad), chrome and firefox. IE and Edge now display a 'leaving Edge' or 'do you want to open this links?' message.


Wie sollen wir hier jetzt verbleiben? Ich bin prinzipiell offen für alle 3 Vorschläge, wobei mir "einhängen" und "laden" doch besser als "einlegen" gefällt.


I think that's `INIT_DATA` that's being put in? It does take a while, yes - but it's nice and easy if someone is just getting started with it... At least on 512k where I have been able to test it's just a simple `esptool.py write_flash 0 file.bin`


All of our controllers are capitalized.

s/aboutCtrl/AboutCtrl/

This well need to be changed in the .spec and about.js files as well.


It was originally supposed to pull from a running TWRP, hence the /sbin


add picture


Ok, so this can be a whole lot simpler - so the only reason the 4M file would have to be bigger is for the INIT_DATA?


Why were these consts removed?


pushed a Readme.md file by gitbash


I have no idea what it's purpose is (OK, I have an idea, but I don't know for sure :wink:), but all other sections seem to have a `:marked` right under the heading, e.g.:

```
:marked
  ### heading
:marked
  ...content...
```

This one is missing the 2nd `:marked` (but I don't know what difference it makes in the actual output - if any).


Don't miss this "aboutCtrl" in the refactoring. It doesn't break anything if it isn't changed so it is an easy one to miss.


Hi @cedricmessiant @sgeulette

If I understand well, maybe I am wrong but this is recomputing the number of items of every displayed collections, no matter it needs to display the counter or not...  So at every change, every searches are executed?  I should take care of only executing necessary searches, aka where collection.showNumberOfItems is True.

Could you please correct, including tests to show that it only recompute necessary collections?  And do a PR?

Thank you,

Gauthier


qqqqqqqqqq


xxception -> ception? Spíš exception ne? :D


The idea is good, but the implementation needs work. It ignores a failure in xrdp-keygen, which wasn't just a theoretical possibility until very recently.


This LGTM. Just change the capitalization of aboutCtrl to match the other controllers and ship it.


Let's change this to `Remove Fields`


@SpongePowered/developers

Let me know your thoughts, if this commit goes into master it WILL wipe API set gamemodes the moment the server saves.


Why `.vm` if it's js code?


Синтаксическая ошибка "гпуппа". Переименуй, пожалуйста, на "активная/занятия завершены"


I totally agree :wink:
<sub>(I wad just saying... :grin:)</sub>


This is for Terry,
Won't the above line throw a php parsing error since missing the right parenthesis?


I see, thanks. I'll need to dig into my deps and see which one is using the old one then.


It's just odd, because I haven't seen this stuff added for lots of other new fields that have been added to the patient_data any dem layout.


Looks fine, Davey son


@phated Yup, that sounds good.


Line 113


Doesn't the interface/billing/sl_eob_search.php file throw a parsing php error since a line is missing the right parenthesis?
(at line 1026)


I suppose you wanted to delete this.


And this.


But, the one you put up is a lot nicer than the generated one


Will we see this fix soon in a new release? I noticed it on my personal Jenkins setup and also on ci.jenkins-ci.org -- https://ci.jenkins-ci.org/job/jenkins_main_trunk/configure


@modocache Thanks, I committed a fix.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 17184](https://teamcity.mesosphere.io/viewLog.html?buildId=145600&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 236 Build time: 00:04:00


Doesn't happen in PHP7 tho...I'm using it on my public server and there have been no complaints. But the php5 version does...and it's really annoying :/


``!selected_db`` will return ``false`` for db ``0`` but ``true`` for other DB, so if user selected a DB, the condition will not trigger

I think you can skip the condition and use just ``client.selected_db !== self._redis_default_db`` (Redis client connexion does a selected if a DB is provided, else it will select only on the first release


I mean, I tested glibc-2.23 which is what the branch contains at merge-time. The naming just confused me.

BTW, I'm sorry this introduced some new master changes separately. I didn't realize before pushing.


https://github.com/ClearSkyTeam/ClearSky/commit/3214872510c0316bace705001001eca4382de4dd

Line 217


picture


In the meantime you can install the latest version from github directly if you absolutely need the latest commits.


@RenanGamerz  line 168.


It seems that sending a body with a DELETE request isn't against the HTTP RFC and lots of APIs require it, so I'm happy to accept a PR for this.  However, please can you change the specs, so they pass too.


This should actually be "true" as it's the legacy behavior


For the first 3 seconds of composing that commit message I was like »Shall I mention @moonglum?« and then it got me that you'll be monitoring the lot of my commits anyway. <3


This is a test comment on a commit


One more test comment on a commit


ありがとうございます！！！


This is a comment on a changed file in a commit


The dynamic library is there since 1.1, thanks to @joncfoo

I never wrote a manpage and it would take me quite a bit to learn the format methinks.
Perhaps we could *lend* a manpage from elsewhere, if the license is comparable.


integrando


TYPO : Update**d**


wd


Steps to compile in Mac

Method 1
1. compile it with the command -> cc hello.c
2. Above step will make an executable file called a. out
3. Run a.out by typing the command-> ./a.out

Method 2
1. run the command -> make hello
2. above command will create the executable file hello
3. run hello -> ./hello


github push  commit  by upload file.好有意思


Actually @tve it might make sense to just have a 512k combined image? presumably that should work on 4M parts too and still flashes relatively quickly. I'll be packaging the tgz file as well so each block can be written separately if needed.


Still not getting javaDoc comments through getJavaDoc() method for classorinterface declaration and method declaration


I think just checking ``client.selected_db !== db`` would be simpler for the code readability at the price of 1 extra select over the lifecycle of the app for each client, your call


LGTM
Acked-by: scui@redhat.com


:) I appreciate you taking the time to fix this!


FYI

You've changed  this from downloading a .tar file which you subsequently extract to downloading a .deb package that you're still trying to extract :)


@waqas19921 I think this is unrelated to this commit, please comment in the appropriate issue or open a new one


fc ads,d cs


This is code duplication.

What I suggest instead is
in widget.jade
var show_voxbone_branding = using_widget ? request.query.voxboneBranding : true

So the footer is always present in the generator view..
Then in the above method invoke hide() and show() as needed


I agree with this -- a forced gamemode should always be forced.
If an admin didn't want all dimensions to be a certain gamemode, then they should use a plugin to interact with the GameMode API, imo.


No problem. Thanks


Added.


Better now ;)


@kragniz Oh I forgot it did that, I had it in my head that it was the same as renaming your actual github username as I know it didn't redirect. Cheers!


Releasing 1.11.


...maybe someone should build that plugin before this is merged


Only change in Comment section


Written entirely while holding down Shift instead of enabling CapsLock.


ok thanks


As using `data.table`, I'd suggest the `flightsDTFactor[, month := ...]` syntax over the `<-` operator. Same below.


There seems to be a lot of copy/paste here -- that you could eliminate by creating the base `ggplot2` object (eg map data) once and tweaking that further for the specific plots.


@mikelambert: it seems tests fail on Windows after this. I also have https://github.com/giakki/uncss/tree/deps; feel free to push there and then merge if you get it working.


There at least  2 ways to go from here.

1. A pool of `mod_http_notification_requestor` hidden behind esl/cuespor` pool manager as it is done for `fusco`. Actually in this approach we can remove pool of `fusco` workers and start one `fusco` worker for each `mod_http_notification_requestor`. Additionally a monitor process for all the workers is needed here to kill workers with too big message queue.
2. Probably better approach would be to have `fusco` behind other pool manager than `cuesport`, f.e inaka/worker_pool, which can quickly return an error if there is no spare `fusco` workers to make the request. In such a case a metric counting request erros could be increased and a message to log on loglevel info or debug.

Other than that it would be good to have also a metric counting successful requests.


first commit


Do we really need this ?


@addyosmani: **ping**. We need a new release.


Er, this was with the notifier unplugged. Ah well.


I'm not sure here, since I'm not the best ruby programmer (or even a programmer at all...). I hope I've changed the correct parts.


You havent turned off allow-flight, i bet cha.

> Am 25.02.2016 um 08:59 schrieb Muqsit Rayyan <notifications@github.com>:
>
> Doesn't happen in PHP7 tho...I'm using it on my public server and there have been no complaints. But the php5 version does...and it's really annoying :/
>
> —
> Reply to this email directly or view it on GitHub.
>


It seems `as.Date(input$weatherDate)` is used pretty often -- so you could compute that once as a `reactive` and reuse that.


what is artifact


일단은 Alt+F 로 다시 하는게 좋을 것 같음


hhh


Making the old username clickable doesn't make much sense since it doesn't exist anymore, no? I tried clicking on it, it does nothing but my other instance receives a bogus /whois reply (although that one is likely ZNC's fault).


You could either define your own custom scale for this or to keep simple at least to save this named character vector (define once and reuse later).


yo


## Task 1

Previously, you wrote a program that used a recursive function to compute the sum of the integers from 0 to 1000 (inclusive).  This time, I want you to accomplish the same thing, but using a `while` loop. Compute the sum of 0 to 1000 (inclusive) in a function using `while`. Return the sum of these numbers to the main loop and print the result. Write the program to include a `main()` function and the `ifmain` statement.

- [ ] Program is correctly formatted with shebang, description, code (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program computes sum using a `while` loop in it's own function (1 pt)
- [ ] Program correctly computes and displays sum of 0 to 1000, inclusive (1 pt)

## Task 2

Above, you wrote a program that used a `while` loop to compute the sum of the integers from 0 to 1000 (inclusive).  This time, I want you to accomplish the same thing, but using a `for` loop. Compute the sum of 0 to 1000 (inclusive) in a function using `for`. Return the sum of these numbers to the main loop and print the result. Write the program to include a `main()` function and the `ifmain` statement.

- [ ] Program is correctly formatted with shebang, description, code (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program computes sum using a `for` loop in it's own function (1 pt)
- [ ] Program correctly computes and displays sum of 0 to 1000, inclusive (1 pt)

## Task 3

In Python, we tend to use `for` loops over different types of iterators much more than `while` loops - they're more flexible and iterators are **everywhere** in Python.  Write a program that includes 5 different functions, each of which should demonstrate the use of `for` over **different types** of iterators. Your program should include these five functions, a `main()` function that calls all the other functions, and the `ifmain` statement.

- [ ] Program is correctly formatted with shebang, description, code (1 pt)
- [ ] Program includes `main()` function and "ifmain" statement (1 pt)
- [ ] Program contains 5, correct subfunctions that use `for` over **different types** of iterators (5 pt)
- [ ] Program's `main()` function calls all subfunctions (1 pt)
- [ ] Program runs and correctly demonstrates use of `for` (1 pt)


update tg


@thebigsmileXD try not to turn the commit comments page into a chatroom.