text:
Test the derive crate on beta.

@bors-servo r+

:pushpin: Commit be216d7 has been approved by nox

:hourglass: Testing commit be216d7 with merge 12f1ef4...

:sunny: Test successful - status-travis

authors: Ms2ger bors-servo nox

repo: servo/heapsize


text:
fixed the sidebar styling

No issue # in PR body.

source: gharchive/pull-request

authors: CongoCash esthersweon

license: MIT

license_type: permissive

license_source: github-api


text:
сделать выступления "оценка сроков и того хуже, тайм-трекинг"

готовлюсь тут: https://github.com/sharovatov/teamlead/blob/master/talks/estimation_and_tracking.md

Огромное спасибо, задонатил немного 😉

ничего себе!!! увидел! спасибо огромное, это прямо мега-приятно! :)

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

спасибо, тоже добавлю и рассмотрю

https://www.youtube.com/watch?v=tqoJOEjeAEw&t=2s

Спасибо было интересно посмотреть, у меня есть на тему прогнозов заметка

source: gharchive/issue

authors: Luchanso sharovatov

repo: sharovatov/teamlead


text:
[ja] Machine learning tips and tricks

Just realized that your work was already ready to be reviewed @umu1729! Please feel free to invite friends to take a look at it.

@shervinea
This cheat sheet has been translated and reviewed in the following PR. Even though it's a different translation from umu1729's, I think we could call this cheat sheet 'done'.
https://github.com/shervinea/cheatsheet-translation/pull/164#issuecomment-510325217

Hi @ytknzw, thanks for your message. It is my understanding that PR #164 is not the same as the current one, right? (deep learning tips and tricks as opposed to machine learning tips and tricks)

Hi @shervinea, yes, sorry! My misunderstanding!

No worries @ytknzw, thanks for helping out in seeing what can be already done! By the way, would you be interested in reviewing the translation?

By the way, would you be interested in reviewing the translation?
Yes!

@ytknzw, thanks for proposing your help!
Hi @umu1729, please feel free to invite anyone else who you think could be interested in this process. Looking forward to the final version of this translation!

Hello @umu1729, a team from Machine Learning Tokyo completed reviewing your translation and added some suggestions. Could you check and incorporate our suggestions?
Here is how to incorporate suggestions:
https://help.github.com/ja/github/collaborating-with-issues-and-pull-requests/incorporating-feedback-in-your-pull-request

Hi @umu1729, would there be anything I could do to help out in finishing the reviewing process?

Hi @shervinea, is it possible for you to incorporate suggestions and merge? I think this translation is as good as other completed translations.

Thanks @yoshiyukinakai for pinging me on this PR. I went ahead and manually merged in all relevant reviewers suggestions. Hopefully it should be good now. Thanks again to everyone who helped on this translation, really appreciate your time and work!

authors: shervinea umu1729 yoshiyukinakai ytknzw

repo: shervinea/cheatsheet-translation


text:
opendata facitily csv

Changes Unknown when pulling 01316017ef6218c65b3fbd089abe4ac280e56e91 on sunny4381:features/opendata-facitily-csv into ** on shirasagi:master**.

Changes Unknown when pulling 5569df0e3fb5f1f44a347c3a47e8c08ef2d7aee3 on sunny4381:features/opendata-facitily-csv into ** on shirasagi:master**.

repo: shirasagi/shirasagi

license: mit

license_source: bigquery


text:
[fix] banner random sort element

Coverage remained the same at 83.55% when pulling e3782bc3e1eaf62dfbe97dd110494a91cfcb37ad on itowtips:fix-ads into 46c58fccacb35ad7014b42f1917c560775b2aaaf on shirasagi:master.

authors: coveralls itowtips


text:
[fix] weather xml issues

Coverage decreased (-7.0%) to 56.23% when pulling 833a269042eacf423a6e8c28fbf459013b018d92 on itowtips:fix-weather_xml into 17686ad0c612aa9a480709d3f7b7fb40a97b09b2 on shirasagi:master.

@itowtips シラサギっぽい修正方法ではありますが、普通のオブジェクト指向っぽくサブクラスで private_show_path をオーバーライドする方法ではダメだったのでしょうか？
また新しく導入した @_show_path は広く有用なものでしょうか？

@sunny4381
サイト内検索などでサブクラスに変換せずに private_show_path を使っている箇所があります。
サブクラスのインスタンスを経由せずに文字列を参照する方法として、今回の修正のようにしてみました。
今回の気象庁XMLページのように、DBに保存している route と管理画面のパスは必ずしも一致しないパターンがあるので、その場合は @@_show_path を設定すればと思います。
実装的には private_show_path をオーバーライドしても同様の動作にできるのでダメではないです。この場合、各クラスに @@_show_path を設定するのと、各クラスに private_show_path をオーバーライドするのは同じ意味です。

@itowtips route と管理画面のパスは必ずしも一致しないパターンがあるというのは同意です。
Rss::WeatherXmlPage クラスとして利用する場合にのみ @@_show_path が正しくセットされ、private_show_path が正しいパスを応答します。
サブクラスに変換せずに処理すれば（つまり Cms::Page のまま処理すれば） @@_show_path はセットされていないので、private_show_path は正しいパスを応答できないのではないかと思うのですが、どうでしょうか？
サブクラスに変換せずに処理しているのが問題だと思いますが、どうでしょうか？

authors: coveralls itowtips sunny4381


text:
Feature/plugin config by file

Opened against hyperledger :slightly_smiling_face:

authors: Luis-GA dbluhm

repo: sicpa-dlab/aries-cloudagent-python


text:
feat: Add support for promises in handlers (#30)

Coverage increased (+0.03%) to 99.176% when pulling eb1c65fdbd1544633022ade4dc4e07a8a6592d8d on promise_support into c7396116de1686f60ac8e7970729ba2ed6e2f610 on master.

Coverage increased (+0.03%) to 99.176% when pulling eb1c65fdbd1544633022ade4dc4e07a8a6592d8d on promise_support into c7396116de1686f60ac8e7970729ba2ed6e2f610 on master.

Since GitHub still doesn't like force pushes...
diff --git a/src/interfaces.ts b/src/interfaces.ts
index f12cb80..3a2581d 100644
--- a/src/interfaces.ts
+++ b/src/interfaces.ts
@@ -40,6 +40,12 @@ export interface RequestProcessor {
     * processing. Failure to either send a response or call `next` will result in a hung
     * process.
     *
+    * If this function returns a Promise object, a `catch` method will be automatically
+    * attached to the promise. If the promise is rejected, that `catch` method will call
+    * `next` passing along the rejected value or an error if the value is empty. If the
+    * returned promise is resolved, `next` will *not* be called automatically. It is up to
+    * the handler to call `next` when appropriate.
+    *
     * @param req The request to be handled
     * @param resp The response that will be sent when the request-handling process is
     *             complete
diff --git a/tests/test-utils/makeRequestProcessor.ts b/tests/test-utils/makeRequestProcessor.ts
index aa9c3d4..406824e 100644
--- a/tests/test-utils/makeRequestProcessor.ts
+++ b/tests/test-utils/makeRequestProcessor.ts
@@ -49,9 +49,9 @@ export default function makeRequestProcessor(name: string, userOpts?: MakeFuncti
          } else if (opts.passesErrorToNext) {
             return next(err);
          } else if (opts.returnsRejectedPromise) {
-            return new Promise((_resolve, reject) => setTimeout(() => reject(`Rejection from "${name}"`), 4));
+            return new Promise((_resolve, reject) => setTimeout(() => { reject(`Rejection from "${name}"`); }, 4));
          } else if (opts.returnsEmptyRejectedPromise) {
-            return new Promise((_resolve, reject) => setTimeout(() => reject(), 4));
+            return new Promise((_resolve, reject) => setTimeout(() => { reject(); }, 4));
          } else if (opts.returnsResolvedPromise && opts.callsNext) {
             return new Promise((resolve) => setTimeout(() => {
                next(); // eslint-disable-line callback-return
@@ -75,9 +75,9 @@ export default function makeRequestProcessor(name: string, userOpts?: MakeFuncti
          } else if (opts.callsNextRoute) {
             return next('route');
          } else if (opts.returnsRejectedPromise) {
-            return new Promise((_resolve, reject) => setTimeout(() => reject(`Rejection from "${name}"`), 4));
+            return new Promise((_resolve, reject) => setTimeout(() => { reject(`Rejection from "${name}"`); }, 4));
          } else if (opts.returnsEmptyRejectedPromise) {
-            return new Promise((_resolve, reject) => setTimeout(() => reject(), 4));
+            return new Promise((_resolve, reject) => setTimeout(() => { reject(); }, 4));
          } else if (opts.returnsResolvedPromise && opts.callsNext) {
             return new Promise((resolve) => setTimeout(() => {
                next(); // eslint-disable-line callback-return

diff --git a/tests/test-utils/makeRequestProcessor.ts b/tests/test-utils/makeRequestProcessor.ts
index 406824e..647e130 100644
--- a/tests/test-utils/makeRequestProcessor.ts
+++ b/tests/test-utils/makeRequestProcessor.ts
@@ -27,6 +27,14 @@ const DEFAULT_OPTS: MakeFunctionOpts = {
    returnsEmptyRejectedPromise: false,
 };

+function delay<T>(value: T, ms: number): Promise<T> {
+   return new Promise((resolve) => {
+      setTimeout(() => {
+         resolve(value);
+      }, ms);
+   });
+}
+
 /**
  * When using sinon assertions (especially for asserting that many functions were called,
  * and that call order was correct), it's best to have named functions. So, this function
@@ -49,16 +57,15 @@ export default function makeRequestProcessor(name: string, userOpts?: MakeFuncti
          } else if (opts.passesErrorToNext) {
             return next(err);
          } else if (opts.returnsRejectedPromise) {
-            return new Promise((_resolve, reject) => setTimeout(() => { reject(`Rejection from "${name}"`); }, 4));
+            return delay(Promise.reject(`Rejection from "${name}"`), 4);
          } else if (opts.returnsEmptyRejectedPromise) {
-            return new Promise((_resolve, reject) => setTimeout(() => { reject(); }, 4));
+            return delay(Promise.reject(), 4);
          } else if (opts.returnsResolvedPromise && opts.callsNext) {
-            return new Promise((resolve) => setTimeout(() => {
+            return delay(undefined, 4).then(() => {
                next(); // eslint-disable-line callback-return
-               resolve();
-            }, 4));
+            });
          } else if (opts.returnsResolvedPromise) {
-            return new Promise((resolve) => setTimeout(resolve, 4));
+            return delay(Promise.resolve(), 4);
          } else if (opts.callsNext) {
             return next();
          }
@@ -75,16 +82,15 @@ export default function makeRequestProcessor(name: string, userOpts?: MakeFuncti
          } else if (opts.callsNextRoute) {
             return next('route');
          } else if (opts.returnsRejectedPromise) {
-            return new Promise((_resolve, reject) => setTimeout(() => { reject(`Rejection from "${name}"`); }, 4));
+            return delay(Promise.reject(`Rejection from "${name}"`), 4);
          } else if (opts.returnsEmptyRejectedPromise) {
-            return new Promise((_resolve, reject) => setTimeout(() => { reject(); }, 4));
+            return delay(Promise.reject(), 4);
          } else if (opts.returnsResolvedPromise && opts.callsNext) {
-            return new Promise((resolve) => setTimeout(() => {
+            return delay(undefined, 4).then(() => {
                next(); // eslint-disable-line callback-return
-               resolve();
-            }, 4));
+            });
          } else if (opts.returnsResolvedPromise) {
-            return new Promise((resolve) => setTimeout(resolve, 4));
+            return delay(Promise.resolve(), 4);
          } else if (opts.callsNext) {
             return next();
          }

diff --git a/tests/test-utils/makeRequestProcessor.ts b/tests/test-utils/makeRequestProcessor.ts
index 647e130..fb3cbd0 100644
--- a/tests/test-utils/makeRequestProcessor.ts
+++ b/tests/test-utils/makeRequestProcessor.ts
@@ -27,7 +27,7 @@ const DEFAULT_OPTS: MakeFunctionOpts = {
    returnsEmptyRejectedPromise: false,
 };

-function delay<T>(value: T, ms: number): Promise<T> {
+function delay(value: unknown, ms: number = 4): Promise<unknown> {
    return new Promise((resolve) => {
       setTimeout(() => {
          resolve(value);
@@ -57,15 +57,15 @@ export default function makeRequestProcessor(name: string, userOpts?: MakeFuncti
          } else if (opts.passesErrorToNext) {
             return next(err);
          } else if (opts.returnsRejectedPromise) {
-            return delay(Promise.reject(`Rejection from "${name}"`), 4);
+            return delay(Promise.reject(`Rejection from "${name}"`));
          } else if (opts.returnsEmptyRejectedPromise) {
-            return delay(Promise.reject(), 4);
+            return delay(Promise.reject());
          } else if (opts.returnsResolvedPromise && opts.callsNext) {
-            return delay(undefined, 4).then(() => {
+            return delay(undefined).then(() => {
                next(); // eslint-disable-line callback-return
             });
          } else if (opts.returnsResolvedPromise) {
-            return delay(Promise.resolve(), 4);
+            return delay(Promise.resolve());
          } else if (opts.callsNext) {
             return next();
          }
@@ -82,15 +82,15 @@ export default function makeRequestProcessor(name: string, userOpts?: MakeFuncti
          } else if (opts.callsNextRoute) {
             return next('route');
          } else if (opts.returnsRejectedPromise) {
-            return delay(Promise.reject(`Rejection from "${name}"`), 4);
+            return delay(Promise.reject(`Rejection from "${name}"`));
          } else if (opts.returnsEmptyRejectedPromise) {
-            return delay(Promise.reject(), 4);
+            return delay(Promise.reject());
          } else if (opts.returnsResolvedPromise && opts.callsNext) {
-            return delay(undefined, 4).then(() => {
+            return delay(undefined).then(() => {
                next(); // eslint-disable-line callback-return
             });
          } else if (opts.returnsResolvedPromise) {
-            return delay(Promise.resolve(), 4);
+            return delay(Promise.resolve());
          } else if (opts.callsNext) {
             return next();
          }

I pushed a couple small typographical changes. I need to review the tests and then I can merge this.
diff --git a/src/interfaces.ts b/src/interfaces.ts
index 3a2581d..b95b75a 100644
--- a/src/interfaces.ts
+++ b/src/interfaces.ts
@@ -40,9 +40,9 @@ export interface RequestProcessor {
     * processing. Failure to either send a response or call `next` will result in a hung
     * process.
     *
-    * If this function returns a Promise object, a `catch` method will be automatically
+    * If this function returns a Promise object, a `catch` method will automatically be
     * attached to the promise. If the promise is rejected, that `catch` method will call
-    * `next` passing along the rejected value or an error if the value is empty. If the
+    * `next`, passing along the rejected value or an error if the value is empty. If the
     * returned promise is resolved, `next` will *not* be called automatically. It is up to
     * the handler to call `next` when appropriate.
     *

authors: coveralls jthomerson onebytegone

repo: silvermine/lambda-express


text:
Cannot see documentation on window 10.

This was a tough problem to track down.  It was a build issue that did not show up on my development box.  The issue was that when Qt was upgraded from 5.13.1 to 5.13.2, some pieces of the help file viewer were left over from 5.13.1 and the mix was being installed.  This is now all resolved.  The program works on a Windows 10 machine with a fresh install of Windows 10 with no other software installed, so all DLLs needed are correct and installed with the application.

repo: sjcbulldog/xeropathgen


text:
host detail migration

Rebase and push again

authors: kmkanagaraj yazhiniloganathan

repo: skyrings/kitoon


text:
make shortcuts unified format

thanks


text:
Added platform.js as a dependency

Could you specify this package through npm and require it in purescript code?
It would be much more convenient I think, because we can specify lib version in package.json and then it works. In case of bower.json we need to add additional step for concatenation of platform.js with our script.

Ah, I see it was already mentioned in package.json so we need not to mention it in bower.json

still need to mention it in bower.json for when installed through bower

I close this because we really need not this in bower.json

authors: beckyconning cryogenian

repo: slamdata/purescript-platform


text:
Removing unused imports

Coverage remained the same at 96.866% when pulling 053fd2bd55570933c9589f24cb5049efe3d8a88a on Sam-Burns:unused-imports into a51018c1044496c6d08a817539402ea08938a3a4 on slimphp:4.x.

Coverage remained the same at 96.866% when pulling 4d7d517af8fcc7e58a20315fad0d174709e8911f on Sam-Burns:unused-imports into a51018c1044496c6d08a817539402ea08938a3a4 on slimphp:4.x.

authors: Sam-Burns coveralls

repo: slimphp/Slim


text:
Created task to delete redis keys with specified prefix [refs #119]

@snc I don't know what to do. I tried git rebase one more time but it looks not good :(

@tomcyr @snc I don't think this should be merged as is. It is not recommended that the keys command be run in production (see this warning).

I'm closing this PR as it has become unmergeable by now, and should be rewritten to use SCAN instead of KEYS for the reason mentioned above.
I'm also not sure I see the use case of the command as a whole as it implies highly application-specific applicability, which should usually be implemented in safer ways, which it has been for the original use case mentioned in #119.

authors: curry684 peterjmit tomcyr

repo: snc/SncRedisBundle


text:
Schema DDL: add Scala 2.12 support

@chuwy, schema-ddl uses scalaz-specs2 whose only version supporting scala 2.12 requires scalaz-core_2.12;7.2.7 which isn't compatible for us. Should we postpone this issue until snowplow/snowplow supports scalaz 7.2.x?

repo: snowplow/iglu


text:
Bump semver to 4.3.2

Wondering if there's anything potentially wrong with npm-shrinkwrap.json changing significantly. I ran npm install, npm install semver@4.3.2 and npm shrinkwrap.

I'm afraid I don't know enough about this to have a view - perhaps @BenFradet, @chuwy or @lhache?

@mhadam seems related to your issue: https://github.com/angular/angular/issues/13935

Ok. Seems like the npm-shrinkwrap.json file was outdated/referencing a package that isn't used in the project. I've updated the semver package to 4.3.2, and now npm-shrinkwrap.json defines the production packages(which is the default behaviour for npm). npm prune --production did the trick.

authors: alexanderdean lhache mhadam

repo: snowplow/snowplow-javascript-tracker


text:
Move mashlib-dev scripts to solidos repo

Will solidos replace mashlib-dev then, or what is the plan?

Yes

See https://gitter.im/solid/solidos?at=5f9689206c8d484be2ab6674

authors: angelo-v michielbdejong

repo: solid/solidos


text:
Dev edits

Accidentally sent here rather than my fork! SORRY!

repo: spacetelescope/notebooks


text:
make sure we've got analytics set up

set up & tested on all main pages

assuming that it'll be copied across now :) cc @lorrainechu3n

repo: sparcopen/doathon-dev


text:
Fix invalid html so flash message appears within div.

Good catch. Thanks.

authors: jdutil sporkd


text:
Remove http-source test that uses default port

LGTM; merged

authors: ilayaperumalg markfisher


text:
integrated pipeline

@GD-Dheer Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

Accident, pull request on the wrong repo

authors: GD-Dheer pivotal-issuemaster

repo: spring-projects/spring-petclinic


text:
chore: upgrade individual packages to latest

:tada: This PR is included in version 4.0.2 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

authors: ryanrd sprucelabs-ci

repo: sprucelabsai/spruce-test


text:
Link to Travis CI Build Status

lgtm

authors: drcapulet syamp

repo: square/metrics


text:
Wip/translation

Honestly, I think you should make a new script for translation. It's not the same as paraphrasing. You can pull some of the code into a shared file that gets imported from both -job.sh scripts.

Ok, reverted changes in paraphrase and put new stuff in translation scripts. Is there anything else?


text:
Can't remove rating tag from my blog

Can you post this to the mailing list?

repo: stephenmcd/mezzanine


text:
ADD grommit to docs

You forgot to rename the image.

I didn't really forget, rather was hoping no-one would notice...
Fine I'll change it...
I can't get away with any yolo-ing anymore these days.

Merge when ready

authors: ndelangen pksunkara

repo: storybooks/storybook


text:
Handle students who graduate from Elementary School to High School

One way to do this -- override the enrollment_status coming in from X2 and set it to "High School"? Would prefer that to actually importing all of the high school students in order to avoid importing unused / unnecessary data into the database.

. . . on the other hand, we would have to include the High School in the school filter in order to even have these graduated student records processed by our importer code.

Did some local manual testing against #618, 👍

Merged, pushed to production, ran a fresh import task. Looking good! The example student that Uri sent me now has an enrollment_status of High School, a school of nil, and doesn't show up in the ESCS overview page anymore. Once I have confirmation from Uri's side I'll close this issue.

repo: studentinsights/studentinsights


text:
ES6 syntax

@michael u can merge now

authors: Integral michael

repo: substance/texture


text:
add jsx support to readthedocs build env

Looks like readthedocs has support for conda envs.
http://docs.readthedocs.io/en/latest/yaml-config.html

https://pypi.python.org/pypi/pygments-lexer-babylon/0.10.0

Getting a weird error both in rtd build env and locally:
ubuntu@docker-machine:~$ conda env create -n rtd10 -f environment.yml
Fetching package metadata .................
Solving package specifications: .
An unexpected error has occurred.
Please consider posting the following information to the
conda GitHub issue tracker at:

    https://github.com/conda/conda/issues

Current conda install:

               platform : linux-64
          conda version : 4.3.22
       conda is private : False
      conda-env version : 4.3.22
    conda-build version : not installed
         python version : 2.7.13.final.0
       requests version : 2.12.4
       root environment : /home/ubuntu/miniconda2  (writable)
    default environment : /home/ubuntu/miniconda2
       envs directories : /home/ubuntu/miniconda2/envs
                          /home/ubuntu/.conda/envs
          package cache : /home/ubuntu/miniconda2/pkgs
                          /home/ubuntu/.conda/pkgs
           channel URLs : https://repo.continuum.io/pkgs/free/linux-64
                          https://repo.continuum.io/pkgs/free/noarch
                          https://repo.continuum.io/pkgs/r/linux-64
                          https://repo.continuum.io/pkgs/r/noarch
                          https://repo.continuum.io/pkgs/pro/linux-64
                          https://repo.continuum.io/pkgs/pro/noarch
            config file : None
             netrc file : None
           offline mode : False
             user-agent : conda/4.3.22 requests/2.12.4 CPython/2.7.13 Linux/4.4.0-78-generic debian/stretch/sid glibc/2.23
                UID:GID : 1000:999

`$ /home/ubuntu/miniconda2/bin/conda-env create -n rtd10 -f environment.yml`

    Traceback (most recent call last):
      File "/home/ubuntu/miniconda2/lib/python2.7/site-packages/conda/exceptions.py", line 632, in conda_exception_handler
        return_value = func(*args, **kwargs)
      File "/home/ubuntu/miniconda2/lib/python2.7/site-packages/conda_env/cli/main_create.py", line 108, in execute
        installer.install(prefix, pkg_specs, args, env)
      File "/home/ubuntu/miniconda2/lib/python2.7/site-packages/conda_env/installers/pip.py", line 8, in install
        pip_cmd = pip_args(prefix) + ['install', ] + specs
    TypeError: unsupported operand type(s) for +: 'NoneType' and 'list'

Supposably fixed:
https://github.com/conda/conda/issues/4618
https://github.com/conda/conda/issues/2254
It's weird as our travis.ci builds for superphy/backend run fine and so do superphy/docker-flask-conda

Should be fixed in https://github.com/superphy/backend/pull/108/commits/bff7ecc977a3c89e0d8f83af7d9470482b9485d3
Needed to add pip as a dep to the conda env

Fixed in https://github.com/superphy/backend/commit/278e7762a9de59a2e92d0fc3186e71b2d6a6b4ff

repo: superphy/backend


text:
env check

not present as sub-command.

contained by another sub-commands.

repo: suzuki-hoge/has-hub


text:
App 1203 - TokenUtils added and AppToken with equals() and hashCode()

Codecov Report

:exclamation: No coverage uploaded for pull request base (dev@c25306a). Click here to learn what that means.
The diff coverage is 38.88%.

@@          Coverage Diff           @@
##             dev     #103   +/-   ##
======================================
  Coverage       ?   89.39%
======================================
  Files          ?      117
  Lines          ?     2131
  Branches       ?      235
======================================
  Hits           ?     1905
  Misses         ?      147
  Partials       ?       79

Impacted Files
Coverage Δ

.../org/symphonyoss/integration/utils/TokenUtils.java
0% <0%> (ø)

...integration/authentication/api/model/AppToken.java
72.72% <53.84%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c25306a...ff5012c. Read the comment docs.

authors: campidelli codecov-io

repo: symphonyoss/App-Integrations-Commons


text:
Refresh specs for API Spec 1.50.0 Final

commit f72db137dfbd1a1207b647791f8f830657be1ea3
1.50.1 released to maven central.

authors: dovie ftbb

repo: symphonyoss/symphony-java-api


text:
Using stable discovery and make sure we have message factories.

Coverage remained the same at 0.0% when pulling 98a47a6a669a0b99999babe77bb5ea52821c67a5 on Nyholm:patch-1 into bddbf1c077cd5f6fbf45e9064b4a8a3cb2d8bd85 on syntaxera:develop.

Can you help me understand why message factories are desired here?

One need message factories to actually create HTTP request. I've added a block in the readme about it.
I also updated some other code. A user does not need to use the CURL client. He/She may install Guzzle or Buzz if they want to.

Coverage remained the same at 0.0% when pulling 552347e333065b6b06010017412c91f5df06d0ef on Nyholm:patch-1 into bddbf1c077cd5f6fbf45e9064b4a8a3cb2d8bd85 on syntaxera:develop.

Lovely. Thanks very much for the info. The update to the README was a nice touch.

You are welcome

authors: Nyholm coveralls robojamison

repo: syntaxera/cakephp-sparkpost-plugin


text:
transport correlation in all logs

Coverage increased (+0.2%) to 92.632% when pulling 4357d1a18a770ad5a8431add1a23991df9fe950a on logging-enhancements into cd3ac44a3e6ecec530d356df3dfe9449bd68e76b on master.

+1

authors: coveralls domano smancke

repo: tarent/lib-compose


text:
switch run view to using new view mixing in encapsia-api

Superceded by https://github.com/tcorbettclark/encapsia-cli/pull/11

authors: petrem tcorbettclark

repo: tcorbettclark/encapsia-cli


text:
Debug visibility

Is this what #8 is for?

yes

authors: dskora tebeka

repo: tebeka/selenium


text:
Upgrade to Node 13

Duplicate to #53

repo: technology-ebay-de/zapperment


text:
Improve the readme

Added lots of new info in #11

authors: dvidsilva fvcproductions

repo: techqueria/website


text:
Decoder module

Ray and I reviewed. Could use some more testing but it is a very simple module and has been sufficiently tested.

repo: terminalstderr/Mips-FPGA


text:
Simplifications around command log

Do you want to merge the PR?

Hi Anton,
I have just pushed another bunch of changes.
I believe most of them make sense. Please have a look.
Cheers,
Marco

On 28 Nov 2016, at 10:23 PM, Anton Anufriev notifications@github.com wrote:
Do you want to merge the PR?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/terzerm/hover-raft/pull/5#issuecomment-263247329, or mute the thread https://github.com/notifications/unsubscribe-auth/AJqHGxXRLnJkZgd-gnlZWVYEicV8rScxks5rCrnOgaJpZM4K9HFS.

Will add copyFrom back

On 28 Nov 2016, at 11:42 PM, Anton Anufriev notifications@github.com wrote:
@anton-anufriev commented on this pull request.
In src/main/java/org/tools4j/hoverraft/command/InMemoryCommandLog.java https://github.com/terzerm/hover-raft/pull/5#pullrequestreview-10307729:

 }

 @Override

public void readKeyTo(final long position, final LogKey logKey) {

   final LogKey readKey = read(position).logKey();

   logKey.term(readKey.term()).index(readKey.index());

public synchronized void readTo(final long index, final LogEntry target) {

   final LogEntry source = read(index);

   target.writeBufferOrNull()

           .putBytes(0, source.readBufferOrNull(), source.offset(), source.byteLength());

target.copyFrom(source)?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/terzerm/hover-raft/pull/5#pullrequestreview-10307729, or mute the thread https://github.com/notifications/unsubscribe-auth/AJqHGysLjFuECbO499tC0xpUeMqnR-4Iks5rCswngaJpZM4K9HFS.

good point — bug — fixed.

On 28 Nov 2016, at 11:52 PM, Anton Anufriev notifications@github.com wrote:
@anton-anufriev commented on this pull request.
In src/main/java/org/tools4j/hoverraft/command/InMemoryCommandLog.java https://github.com/terzerm/hover-raft/pull/5#pullrequestreview-10309102:

 }

 @Override

public synchronized void append(final LogEntry logEntry) {

   entries.add(clone(logEntry));

public void append(final int term, final Command command) {

   final LogEntry logEntry = directFactory.logEntry();

   logEntry.logKey().term(term);

   logEntry.command()

           .writeBufferOrNull()

           .putBytes(0, command.readBufferOrNull(), command.offset(), command.byteLength());

does command.byteLength() include the byteLength size (4 bytes)?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/terzerm/hover-raft/pull/5#pullrequestreview-10309102, or mute the thread https://github.com/notifications/unsubscribe-auth/AJqHG7L9sswmAzSHtYIEn6PFQakWwP6Lks5rCs6TgaJpZM4K9HFS.

donnow. don’t like this redundancy stuff. it confused me completely.

On 28 Nov 2016, at 11:57 PM, Anton Anufriev notifications@github.com wrote:
@anton-anufriev commented on this pull request.
In src/main/java/org/tools4j/hoverraft/command/InMemoryCommandLog.java https://github.com/terzerm/hover-raft/pull/5#pullrequestreview-10309860:

 }

 @Override

public synchronized void append(final LogEntry logEntry) {

   entries.add(clone(logEntry));

public void append(final int term, final Command command) {

   final LogEntry logEntry = directFactory.logEntry();

   logEntry.logKey().term(term);

   logEntry.command()

           .writeBufferOrNull()

           .putBytes(0, command.readBufferOrNull(), command.offset(), command.byteLength());

   synchronized (this) {

       final long index = entries.size();

       logEntry.logKey().index(index);//TODO this is redundant, do we need this?

I would leave it for now. I thought it was nice to pass prevLogKey to appendRequest and lastLogKey in the voteRequest and apply compare and containment operation on them. But may be it is redundant.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/terzerm/hover-raft/pull/5#pullrequestreview-10309860, or mute the thread https://github.com/notifications/unsubscribe-auth/AJqHGyK2t31J0F1XpBD6f_fR3hyqnfWxks5rCs_JgaJpZM4K9HFS.

yea rebuilding index later

On 29 Nov 2016, at 12:02 AM, Anton Anufriev notifications@github.com wrote:
@anton-anufriev commented on this pull request.
In src/main/java/org/tools4j/hoverraft/command/InMemoryCommandLog.java https://github.com/terzerm/hover-raft/pull/5#pullrequestreview-10310413:

 }

 @Override
 public synchronized void truncateIncluding(final long index) {

   if (index >= entries.size()) {

       throw new IllegalArgumentException("Truncate index " + index + " must be less than the size " + entries.size());

   }

   for (long idx = lastIndex(); idx >= index; idx--) {

       entries.remove((int)idx);

   for (int idx = entries.size() - 1; idx >= index; idx--) {

       final LogEntry removed = entries.remove(idx);

       maxCommandIndexBySourceId.remove(removed.command().commandKey().sourceId());

it does not look right to me. If you truncate the log there should be other commands from the SourceId left in the log, however the SourceId keyed entry would be removed.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/terzerm/hover-raft/pull/5#pullrequestreview-10310413, or mute the thread https://github.com/notifications/unsubscribe-auth/AJqHG0Lp3LPcCZkLhhK2c382volOqTQZks5rCtDMgaJpZM4K9HFS.

you are right. would be nicer to rebuild on truncate.
will change.

On 29 Nov 2016, at 12:10 AM, Anton Anufriev notifications@github.com wrote:
@anton-anufriev commented on this pull request.
In src/main/java/org/tools4j/hoverraft/command/InMemoryCommandLog.java https://github.com/terzerm/hover-raft/pull/5#pullrequestreview-10311656:

private LogEntry clone(final LogEntry e) {

   final LogEntry clone = directFactory.logEntry();

   clone.writeBufferOrNull().putBytes(0, e.readBufferOrNull(), e.offset(), e.byteLength());

   return clone;

public synchronized boolean contains(final CommandKey commandKey) {

   final int sourceId = commandKey.sourceId();

   long maxCommandIndex = maxCommandIndexBySourceId.get(sourceId);

   if (maxCommandIndex < 0) {

       for (int idx = entries.size() - 1; idx >= 0; idx--) {

           final CommandKey ckey = entries.get(idx).command().commandKey();

           if (ckey.sourceId() == sourceId) {

               maxCommandIndex = ckey.commandIndex();

               maxCommandIndexBySourceId.put(sourceId, maxCommandIndex);

               break;

           }

       }

I see that you are rebuilding the command index.
Is it a right place in contains() method? Shouldn't it be part of truncate method?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/terzerm/hover-raft/pull/5#pullrequestreview-10311656, or mute the thread https://github.com/notifications/unsubscribe-auth/AJqHG5Bs_FApP10kXno6f4KRnNZBA0_iks5rCtKugaJpZM4K9HFS.

ok will remove

On 29 Nov 2016, at 12:17 AM, Anton Anufriev notifications@github.com wrote:
@anton-anufriev commented on this pull request.
In src/main/java/org/tools4j/hoverraft/command/machine/StateMachine.java https://github.com/terzerm/hover-raft/pull/5#pullrequestreview-10312659:

public interface StateMachine {

//Should it be bare Command or CommandMessage
void onMessage(CommandMessage message);

//Should it be bare CommandPayload or Command
remove comment as we know that it is the command that is applied the the state machine

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/terzerm/hover-raft/pull/5#pullrequestreview-10312659, or mute the thread https://github.com/notifications/unsubscribe-auth/AJqHGzvEPpj8TvC9TY-u3iE9fFuZMc80ks5rCtRggaJpZM4K9HFS.

yep

On 29 Nov 2016, at 12:18 AM, Anton Anufriev notifications@github.com wrote:
@anton-anufriev commented on this pull request.
In src/main/java/org/tools4j/hoverraft/direct/AllocatingDirectFactory.java https://github.com/terzerm/hover-raft/pull/5#pullrequestreview-10312784:

@@ -72,9 +73,9 @@ public TimeoutNow timeoutNow() {
}

 @Override

public CommandMessage commandMessage() {

   final DirectCommandMessage directCommandMessage = new DirectCommandMessage();

   directCommandMessage.wrap(newBuffer(DirectCommandMessage.EMPTY_COMMAND_BYTE_LENGTH), 0);

public Command commandMessage() {
command() ?

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/terzerm/hover-raft/pull/5#pullrequestreview-10312784, or mute the thread https://github.com/notifications/unsubscribe-auth/AJqHG4YJrLa12gty6ZMh-l9sySpgzjTDks5rCtSWgaJpZM4K9HFS.

renamed

On 29 Nov 2016, at 12:18 AM, Anton Anufriev notifications@github.com wrote:
@anton-anufriev commented on this pull request.
In src/main/java/org/tools4j/hoverraft/direct/AllocatingDirectFactory.java https://github.com/terzerm/hover-raft/pull/5#pullrequestreview-10312836:

@@ -72,9 +73,9 @@ public TimeoutNow timeoutNow() {
}

 @Override

public CommandMessage commandMessage() {

   final DirectCommandMessage directCommandMessage = new DirectCommandMessage();

   directCommandMessage.wrap(newBuffer(DirectCommandMessage.EMPTY_COMMAND_BYTE_LENGTH), 0);

public Command commandMessage() {

   final DirectCommand directCommandMessage = new DirectCommand();

directCommand
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/terzerm/hover-raft/pull/5#pullrequestreview-10312836, or mute the thread https://github.com/notifications/unsubscribe-auth/AJqHG3JeFIQ_WHoWODRZHcISX1_Z_GGMks5rCtS0gaJpZM4K9HFS.

actually i did it this way cause then this also works if we open a (possibly huge) command log at the start.

On 29 Nov 2016, at 12:10 AM, Anton Anufriev notifications@github.com wrote:
@anton-anufriev commented on this pull request.
In src/main/java/org/tools4j/hoverraft/command/InMemoryCommandLog.java https://github.com/terzerm/hover-raft/pull/5#pullrequestreview-10311656:

private LogEntry clone(final LogEntry e) {

   final LogEntry clone = directFactory.logEntry();

   clone.writeBufferOrNull().putBytes(0, e.readBufferOrNull(), e.offset(), e.byteLength());

   return clone;

public synchronized boolean contains(final CommandKey commandKey) {

   final int sourceId = commandKey.sourceId();

   long maxCommandIndex = maxCommandIndexBySourceId.get(sourceId);

   if (maxCommandIndex < 0) {

       for (int idx = entries.size() - 1; idx >= 0; idx--) {

           final CommandKey ckey = entries.get(idx).command().commandKey();

           if (ckey.sourceId() == sourceId) {

               maxCommandIndex = ckey.commandIndex();

               maxCommandIndexBySourceId.put(sourceId, maxCommandIndex);

               break;

           }

       }

I see that you are rebuilding the command index.
Is it a right place in contains() method? Shouldn't it be part of truncate method?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/terzerm/hover-raft/pull/5#pullrequestreview-10311656, or mute the thread https://github.com/notifications/unsubscribe-auth/AJqHG5Bs_FApP10kXno6f4KRnNZBA0_iks5rCtKugaJpZM4K9HFS.

already fixed by now

On 29 Nov 2016, at 12:51 AM, Anton Anufriev notifications@github.com wrote:
@anton-anufriev commented on this pull request.
In src/main/java/org/tools4j/hoverraft/message/direct/DirectCommandPayload.java https://github.com/terzerm/hover-raft/pull/5#pullrequestreview-10317737:

+import org.agrona.DirectBuffer;
+import org.agrona.MutableDirectBuffer;
+import org.tools4j.hoverraft.command.CommandPayload;
+import org.tools4j.hoverraft.direct.AbstractDirectPayload;

+import java.nio.ByteBuffer;
+
+public class DirectCommandPayload extends AbstractDirectPayload implements CommandPayload {
+

private static final int BYTE_LENGTH_OFF  = 0;
private static final int BYTE_LENGTH_LEN  = 4;
private static final int BYTES_OFF = BYTE_LENGTH_OFF + BYTE_LENGTH_LEN;

@Override
public int byteLength() {

   return readBuffer.getInt(offset + BYTE_LENGTH_OFF);

I think here is the catch. DirectCommandPayload.byteLength() does not include the bytelength field size.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/terzerm/hover-raft/pull/5#pullrequestreview-10317737, or mute the thread https://github.com/notifications/unsubscribe-auth/AJqHG2bSgJkFSNcd6aXKe1jZEjW0rNvbks5rCtx-gaJpZM4K9HFS.

authors: anton-anufriev terzerm

repo: terzerm/hover-raft


text:
fix for windows environment

I have checked this on my machine that uses Windows 10 and this failed. This because __dirname is refer to
'jsdoc-webpack-plugin' folder which is not what you need. See what I mean bellow. Thanks for the fix.
        if(/^win/.test(process.platform)){             jsdoc = spawn(__dirname + '/../.bin/jsdoc.cmd', ['-c', jsDocConfTmp]);         }

@ViktorSoroka it is because in your environment jsdoc module is installed in parent module and so npm does not install it again in jsdoc-webpack-plugin. It is complex problem because jsdoc can be anywhere in project node+modules folder. I dont have idea how to solve it right now. Weather to search for it in project structure, or force to install jsdoc-webpack-plugin local instance of jsdoc dependency.. or whatever else. Or create setting parameter with path to jsdoc in plugin class? Hmm..

Got it. Thanks for the detailed comment. As I understand this is because I use npm 3 which makes flat tree dependencies.

authors: ViktorSoroka luckylooke

repo: tfiwm/jsdoc-webpack-plugin


text:
Onboarding @aleksey-zhidkov

Hello @aleksey-zhidkov. Thanks for your interest in becoming a member of Nuvola Apps project. There are a few actions to take. You can tick them here for your convenience.

[ ] I've invited you to Nuvola Maintainers team. After you accept the invitation, you will have write access to the Yandex Music repository.
[ ] Please create a Tiliado account and tell me your username. Your account will be then upgraded to developer account and you will gain free access to flatpak builds of Nuvola 4.x including all premium features.
[ ] Subscribe to the nuvola-player-devel mailing list as important announcements are posted there. This mailing list has low traffic so don't be afraid that your mailbox will be full of spam.
[ ] Take a look at Service Integration Guidelines: Rules and guidelines you should follow. Please make sure you are familiar with the process how releases are made. If you have any questions about the guidelines or suggestions to improve them, don't hesitate to ask here.
[ ] Make yourself familiar with Nuvola ADK and try to build & run your script. If there are any issues, don't hesitate to ask here.

Ping me once you are ready for next steps.

@fenryxo I already have Tiliado account - aleksey.zhidkov

I already have Tiliado account - aleksey.zhidkov

:+1: Upgraded to Tiliado developers

@fenryxo, I'm try to build script and getting this error:
atomic-armchair:Alive/nuvola-apps/nuvola-app-yandex-music> ./configure --with-dbus-launcher                       (master|✔)
Makefile written. Run `make all` and then `make install`.
atomic-armchair:Alive/nuvola-apps/nuvola-app-yandex-music> make all                                               (master|✔)
valac  --vapidir="/home/azhidkov/.local/lib/python3.5/site-packages/nuvolasdk/data/vapi" --pkg gio-2.0 --pkg gtk+-3.0 --pkg gio-unix-2.0 --pkg nuvolaruntime-runner -X "-DNUVOLASDK_APP_ID=\"yandex_music\"" -X "-DNUVOLASDK_UNIQUE_ID=\"eu.tiliado.NuvolaAppYandexMusic\"" -X "-DNUVOLASDK_FLATPAK_BUILD=0" -X "-DNUVOLASDK_APP_DATA_DIR=\"/usr/local/share/nuvolaruntime/web_apps/yandex_music\"" -o nuvola-app-yandex-music /home/azhidkov/.local/lib/python3.5/site-packages/nuvolasdk/data/launch_app.vala
error: Package `nuvolaruntime-runner' not found in specified Vala API directories or GObject-Introspection GIR directories
Compilation failed: 1 error(s), 0 warning(s)
Makefile:25: recipe for target 'nuvola-app-yandex-music' failed
make: *** [nuvola-app-yandex-music] Error 1

And cannot understand how to fix it

ADK seems to be trimmed not to include Nuvola VAPI files. You can build the script without -with-dbus-launcher as it is unnecessary for development. It is used in regular flatpak builds.

tiliado/nuvolasdk#4

I recommend going through Service Integration Tutorial, which provides information on how to use Nuvola SDK and to create web app scripts.

I've runned Yandex Music script, so I believe, that I'm ready for next steps

Great :+1:

[ ] Change maintainer_* in https://github.com/tiliado/nuvola-app-yandex-music/blob/master/metadata.in.json
[ ] Make sure you understand all code in integrate.js.
[ ] Increase api_major & api_minor to 4.8 in metadata.json.
[ ] Use custom actions API to integrate the like button - ticket #2.
[ ] Look at Media Player Integration and consider adding relevant missing functionality, e.g. track rating, progress bar and volume management integration.

Look at Media Player Integration and consider adding relevant missing functionality, e.g. track rating, progress bar and volume management integration.

Reticketed as #12.

authors: aleksey-zhidkov fenryxo

repo: tiliado/nuvola-app-yandex-music


text:
Level

xcv

repo: tolearngitforu/NextgenF


text:
updating PR

update PR


text:
選択中のチャンネルをホバーすると通知ONマークが大きくなる

#828で解決したはず

authors: Eleven-Lines unipota

repo: traPtitech/traQ-UI


text:
Endpoint for project config update

@anero saw your PR ... can you explain a bit about what you are trying to achieve?  I'm open to it, I just want to better understand your intent.

@anero Thanks for the info, making the config more configurable has come up before, I'll take a look at your commit.  Thanks!

authors: anero matthewjackowski

repo: transifex/txgh


text:
Update cats-effect and Cats

Codecov Report

Merging #243 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master    #243   +/-   ##
======================================
  Coverage    97.2%   97.2%
======================================
  Files          23      23
  Lines         786     786
  Branches       58      58
======================================
  Hits          764     764
  Misses         22      22

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6bdee37...1d815b1. Read the comment docs.

authors: codecov-io travisbrown

repo: travisbrown/iteratee


text:
Update README.md

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Thank you @harshp2124 but we don't have an issue related to this documentation change.

authors: CLAassistant harshp2124 nopcoder

repo: treeverse/lakeFS


text:
How can i contribute in order to make this widget supports 5 items ?

Was there a problem for limiting the tabs to 4 ?
Looks fine to me, reduce the circle size and added support for 5 items

Any news regarding that improvements?

This library implement the same widget, without the limitation.
https://pub.dev/packages/circular_bottom_navigation

@HilalHakla Would you mind to submit your work as PR to this library, It really looks great with 5 tabs by reducing the size.

authors: HilalHakla abdulwahid24 samy-baili


text:
Update steampipe-plugin-sdk to v1.4.1

Closing this PR since we are updating to the latest SDK in https://github.com/turbot/steampipe-plugin-jira/pull/40

authors: bigdatasourav misraved

repo: turbot/steampipe-plugin-jira


text:
Updated workflow to checkout chaos plugin with last released tag

Closing it as the updates are made as part of https://github.com/turbot/steampipe-plugin-sdk/pull/88

repo: turbot/steampipe-plugin-sdk


text:
cumulative lost when normed for histograms #3407

:+1:

authors: mnavasiolava scottdraves

repo: twosigma/beaker-notebook


text:
Instructions for installing libuv on Debian|Ubuntu

Why is there no zlib or ssl packages? And why libuv1 when libuv1-dev has this as dependency. So many questions

@alexhultman indeed. Please excuse me. I just want to inform you that libuv does not have candidate to install. So on Debian/Ubuntu you have to install libuv1

Shouldn't it be -dev packages?

To be sure I did make on clean fresh installed Ubuntu. So I found that zlib (zlib1g-dev) library is already installed. So I removed it. Also I found that on fresh Ubuntu libssl-dev is absent . So I've added it. Thanks for the tip.

authors: alexhultman alexslipknot

repo: uNetworking/uWebSockets


text:
Py3 compatibility changes for tests

Thanks for this, I will see if I can get configure Travis to run the tests with Py3 as well.

authors: eklitzke tijko

repo: uber/pyflame


text:
Added functionality to query previous version models

Can one of the admins verify this patch?

jenkins add to whitelist

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Clipper-PRB/1200/
Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Clipper-PRB/1209/
Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Clipper-PRB/1211/
Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Clipper-PRB/1214/
Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Clipper-PRB/1215/
Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Clipper-PRB/1216/
Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Clipper-PRB/1217/
Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Clipper-PRB/1279/
Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Clipper-PRB/1280/
Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Clipper-PRB/1281/
Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Clipper-PRB/1291/
Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Clipper-PRB/1330/
Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Clipper-PRB/1335/
Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Clipper-PRB/1340/
Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Clipper-PRB/1390/
Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Clipper-PRB/1406/
Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Clipper-PRB/1408/
Test FAILed.

jenkins test this please

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Clipper-PRB/1409/
Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Clipper-PRB/1414/
Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Clipper-PRB/1415/
Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Clipper-PRB/1422/
Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Clipper-PRB/1429/
Test FAILed.

jenkins test this please

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Clipper-PRB/1431/
Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Clipper-PRB/1435/
Test FAILed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Clipper-PRB/1444/
Test PASSed.

authors: AmplabJenkins daminig dcrankshaw

repo: ucbrise/clipper


text:
Create new website

good

repo: umpox/TinyEditor


text:
Petition creator notified of sponsors support spec migration

LGTM

authors: alanth jnormington

repo: unboxed/e-petitions


text:
added me as author, added code-review post

@gaffa - please run this on your machine locally. Instructions [here] (https://github.com/upday/upday.github.io) if needed.
Most of the paragraphs look like blocks of text. You need two return characters to put the text on a new line.
Also, it would be nice if you can bold some key words here and there in the text, to make it easier to read.
For headlines, in all the other blog posts we are using " ## " not just " # " so the text size is a bit smaller. It would be great if we try to have some consistency in our posts.

There are several "dont"s and "thats" and other words that require apostrophes. Use an editor like Atom, Google Docs or Word, make sure that your language is set to English and correct all of them. I pointed some of the mistakes but I might have missed some. Similar for typos.

After you make some of these changes suggested, could you notify Peter T. also, so he can check the English if he has time, please?

@florina-muntenescu : i incorporated most of your suggestions. are you happy with the current state?
@peter-tackage : florina asked me to ask you to check the english :)
Thanks for all the feedback @florina-muntenescu . I can see you invest a lot in this and am thankful for your input.

Thank you, @peter-tackage , for your detailled review. I tried to solve all mentioned issues. Would you mind to review the commit I just pushed and check if you are happy with the changes?
Thank you!

@peter-tackage : i just pushed another commit. Thanks again for investing the time!

@peter-tackage I changed most of it. Can you live with the rest?
@pkrauss-asideas / @florina-muntenescu : how does this proceed now /what are the final steps to get it live?

@gaffa - as soon as a PR is marked as merge, the creator of the PR is responsible of pressing the merge button. Everything that is on master is live on the blog so, as soon as you merge, it will be live.

@florina-muntenescu : who is responsible to mark as "merge"? 💃

@gaffa - the reviewer. In this case me or @peter-tackage.

I dont think @peter-tackage  feels like doing so. You should step in. Have you still oversight? Whats missing?

finally... thank you!

authors: florina-muntenescu gaffa


text:
tr: add truncate flag

👍

authors: Arcterus shutefan

repo: uutils/coreutils


text:
CI for Node

test/game.test.js

repo: vailtra/github-actions-for-ci


text:
Mark offset only after sending to channel

No longer applicable/necessary

authors: dduyVeritone tnguyen-veri

repo: veritone/go-messaging-lib


text:
Log when certificate has been refreshed

Nice

repo: vespa-engine/vespa


text:
build: add pre commit hooks for black and flake8

Should be good to merge if ok

repo: vinteo/py-opensprinkler


text:
Weaver passes cmd/rpc/targetrpc/event names instead of hashcodes now. HLAPI worries about sending hashcodes instead.

Codecov Report

Merging #192 into master will decrease coverage by 0.01%.
The diff coverage is 0%.

@@            Coverage Diff            @@
##           master    #192      +/-   ##
=========================================
- Coverage    6.75%   6.74%   -0.02%
=========================================
  Files          28      29       +1
  Lines        6099    6109      +10
  Branches      856     856
=========================================
  Hits          412     412
- Misses       5674    5684      +10
  Partials       13      13

Impacted Files
Coverage Δ

Mirror/Runtime/Extensions.cs
0% <0%> (ø)

Mirror/Runtime/NetworkBehaviour.cs
0% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 86c482c...283727b. Read the comment docs.

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


text:
20161031

不好意思，merge方向反了

repo: vnpy/vnpy


text:
Update Node in travis

Coverage remained the same at 73.955% when pulling 14bc7fa0613479090deccb7dd29c91aa79a1a4cd on master into 7f5d20d5a106ecd03fd6ede91b55bee6e8fe07ff on update-travis.

authors: coveralls lukasoppermann


text:
Add missing Gateway Close Event Codes, fixes #18

https://github.com/vsTerminus/Mojo-Discord/blob/cd25b3edcaed0df872d8c8bcd5ecee17b8495a6b/lib/Mojo/Discord/Gateway.pm#L148-L151
Double Invalid shard?

https://github.com/vsTerminus/Mojo-Discord/blob/cd25b3edcaed0df872d8c8bcd5ecee17b8495a6b/lib/Mojo/Discord/Gateway.pm#L148-L151
Double Invalid shard?

Fixed.

authors: Disinterpreter incognico

repo: vsTerminus/Mojo-Discord


text:
add CountdownLabel.

Thank you (:

authors: kc-sanrenp victorgama

repo: vsouza/awesome-ios


text:
added vue-spinner-component

ping @da-sloysha

close due to in-activity.
if you hope push this PR again, you can push it as new PR. :)

authors: DannyFeliz da-sloysha kazupon

repo: vuejs/awesome-vue


text:
fix proton false positive

Fixed in #59

I'd rather merge a one-line bug fix (if approved) immediately than waiting for a long PR that also includes it but will need more review.
Reviewing is easier when done in several small independent PR than fixing all at once.

repo: vuillaut/ctaplot


text:
Admin Panel multilingual support

Pulled out from 1.0 milestone and will implement in future


text:
Fixes #74: clicking on previous month's day 31 from month with only 30 days

Thanks!

authors: nemoDreamer wangzuo

repo: wangzuo/input-moment

license: isc


text:
Added iojs-v3

#boomshake

nooooo: https://travis-ci.org/waterlock/waterlock-local-auth/jobs/75946218

fix https://travis-ci.org/waterlock/waterlock-local-auth/builds/75947351

Good work :)

authors: listepo wayne-o

repo: waterlock/waterlock-local-auth


text:
Travis indentation fixes

Thanks

authors: amilajack watson

repo: watson/airplay-protocol


text:
Restructure docs

Https://1

0

authors: kitikhunp spawnia

repo: webonyx/graphql-php


text:
Also take subjects from MARC 600

Silver has tweaked some of the transform rules for subjects, so go back to the sierratransformable and check we're still doing the correct thing.

^ this includes the following:
"Can we add to the 600 work stripping out any subjects other than those from lc and mesh?
Not to be applied to genre"

authors: alexwlchan jtweed

repo: wellcometrust/platform