text:
e2e: Add kube-hunter configs and script

The original commit failed with following error:
+ pushd ../..
/tmp/build/8c0986c9/lokomotive-kubernetes /tmp/build/8c0986c9/lokomotive-kubernetes/ci/aws
+ RUN_FROM_CI='"true"'
+ make run-e2e-tests
KUBECONFIG="/root/assets/auth/kubeconfig" ./scripts/kube-hunter.sh
W1107 14:47:25.279377     467 loader.go:223] Config not found: /root/assets/auth/kubeconfig
unable to recognize "/tmp/build/8c0986c9/lokomotive-kubernetes/scripts/kube-hunter-manifests/config.yaml": Get http://localhost:8080/api?timeout=32s: dial tcp 127.0.0.1:8080: connect: connection refused
unable to recognize "/tmp/build/8c0986c9/lokomotive-kubernetes/scripts/kube-hunter-manifests/config.yaml": Get http://localhost:8080/api?timeout=32s: dial tcp 127.0.0.1:8080: connect: connection refused
unable to recognize "/tmp/build/8c0986c9/lokomotive-kubernetes/scripts/kube-hunter-manifests/config.yaml": Get http://localhost:8080/api?timeout=32s: dial tcp 127.0.0.1:8080: connect: connection refused
make: *** [Makefile:11: kube-hunter] Error 1
+ RET=2
+ popd
/tmp/build/8c0986c9/lokomotive-kubernetes/ci/aws

Got a weird error this time:
--------------------------------------------------
unable to recognize "/tmp/build/8c0986c9/lokomotive-kubernetes/scripts/kube-hunter-manifests/config.yaml": Get https://loko-ci-1573139279.test.lokomotive-k8s.net:6443/api?timeout=32s: dial tcp 3.124.161.255:6443: connect: connection refused
unable to recognize "/tmp/build/8c0986c9/lokomotive-kubernetes/scripts/kube-hunter-manifests/config.yaml": Get https://loko-ci-1573139279.test.lokomotive-k8s.net:6443/api?timeout=32s: dial tcp 3.124.87.148:6443: connect: connection refused
unable to recognize "/tmp/build/8c0986c9/lokomotive-kubernetes/scripts/kube-hunter-manifests/config.yaml": Get https://loko-ci-1573139279.test.lokomotive-k8s.net:6443/api?timeout=32s: dial tcp 3.124.87.148:6443: connect: connection refused
make: *** [Makefile:11: kube-hunter] Error 1
+ RET=2
+ popd

Packet tests are failing because of unavailability of the machines.

The output of the CI run on AWS:
Apply complete! Resources: 105 added, 0 changed, 0 destroyed.
+ '[' 0 = 0 ']'
+ pushd ../..
/tmp/build/8c0986c9/lokomotive-kubernetes /tmp/build/8c0986c9/lokomotive-kubernetes/ci/aws
+ RUN_FROM_CI='"true"'
+ make run-e2e-tests
KUBECONFIG="/root/assets/auth/kubeconfig" ./scripts/kube-hunter.sh
The connection to the server loko-ci-1573198900.test.lokomotive-k8s.net:6443 was refused - did you specify the right host or port?
Waiting for the cluster to be responsive...
The connection to the server loko-ci-1573198900.test.lokomotive-k8s.net:6443 was refused - did you specify the right host or port?
Waiting for the cluster to be responsive...
The connection to the server loko-ci-1573198900.test.lokomotive-k8s.net:6443 was refused - did you specify the right host or port?
Waiting for the cluster to be responsive...
The connection to the server loko-ci-1573198900.test.lokomotive-k8s.net:6443 was refused - did you specify the right host or port?
Waiting for the cluster to be responsive...
The connection to the server loko-ci-1573198900.test.lokomotive-k8s.net:6443 was refused - did you specify the right host or port?
Waiting for the cluster to be responsive...
The connection to the server loko-ci-1573198900.test.lokomotive-k8s.net:6443 was refused - did you specify the right host or port?
Waiting for the cluster to be responsive...
NAME             STATUS   ROLES    AGE   VERSION
ip-10-0-0-153    Ready    <none>   61s   v1.16.2
ip-10-0-23-197   Ready    <none>   60s   v1.16.2
ip-10-0-37-216   Ready    <none>   59s   v1.16.2
serviceaccount/kube-hunter created
clusterrolebinding.rbac.authorization.k8s.io/privileged-psp-kube-hunter created
job.batch/kube-hunter created
The pod has not started yet...
The pod has not started yet...
The pod has not started yet...
The pod has not started yet...
The pod has not started yet...
The pod has not started yet...
The pod has not started yet...
The pod has not started yet...
The pod has not started yet...
The pod has not started yet...
The pod has not started yet...
The pod has not started yet...
The pod has not started yet...
The pod has not started yet...
The pod has not started yet...
Waiting until the 'kube-hunter' job is complete...
Waiting until the 'kube-hunter' job is complete...
Waiting until the 'kube-hunter' job is complete...
Waiting until the 'kube-hunter' job is complete...
Waiting until the 'kube-hunter' job is complete...
Waiting until the 'kube-hunter' job is complete...
Waiting until the 'kube-hunter' job is complete...
Waiting until the 'kube-hunter' job is complete...
Waiting until the 'kube-hunter' job is complete...
Waiting until the 'kube-hunter' job is complete...
Waiting until the 'kube-hunter' job is complete...
Waiting until the 'kube-hunter' job is complete...
Waiting until the 'kube-hunter' job is complete...
Waiting until the 'kube-hunter' job is complete...
Waiting until the 'kube-hunter' job is complete...
Waiting until the 'kube-hunter' job is complete...
Waiting until the 'kube-hunter' job is complete...
Waiting until the 'kube-hunter' job is complete...
Waiting until the 'kube-hunter' job is complete...
--------------------------------------------------
kube-hunter output:
--------------------------------------------------
~ Started
~ Discovering Open Kubernetes Services...

repo: kinvolk/lokomotive-kubernetes


text:
Upload code used to parse wayback machine

Closed by 1d12710

source: gharchive/issue

repo: kitchenknif/SlowChinese

license: MIT

license_type: permissive

license_source: github-api


text:
MMB: Share e-ticket-pdf

@mrtnzlml please check again
I added default props to pdfview component, so you can still override cache and expiration if you want to. default is true and Infinity

source: gharchive/pull-request

repo: kiwicom/mobile


text:
[test] get meta-evidence

Pull Request Test Coverage Report for Build 844

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 49.475%

Totals

Change from base Build 842:
0.0%

Covered Lines:
592

Relevant Lines:
1087

💛  - Coveralls

repo: kleros/kleros-api


text:
local: capture Docker logs in 'local' mode

At least to start with, should be enough to pipe the stdout/err of individual containers into console log of node-red container (leveraging streams in https://www.npmjs.com/package/dockerode).

repo: kpavel/node-red-node-openwhisk


text:
Updated with the new items.

This also fixes #37

license: mit

license_source: bigquery


text:
Update config.toml

/lgtm
/approve

authors: jimangel zacharysarah

repo: kubernetes/website


text:
vue devtoolsのダウンロードを勧めるメッセージを出力しないように設定する

PR #36で対応したのでクローズする。

repo: kubotama/glipah


text:
Updated Skraps contract ABI and Address

do  you have link to a public post explaining the change to contract address?

Yes https://medium.com/@SKRAPS_IO/skraps-agrees-to-extend-token-sale-to-accommodate-market-dip-52c592aab87d

authors: kvhnuke nodermann

repo: kvhnuke/etherwallet


text:
Alert Manager integration

Prometheus integrate with Consul for Service Discovery

repo: laoshanxi/app-mesh


text:
[5.2] password validation override in ResetsPasswords

Thanks, but the tests are failing.

From the build log:
$ phpenv global 7.0
rbenv: version `7.0' not installed
The command "phpenv global 7.0" failed and exited with 1 during .

Seems as though it's not the tests but something in the build process that failed.

What are you wanting to customize about the validation?

The validation sets the minimum password length to 6 characters. I would like to be able to override this to 8 chars. The alternative solution is:
class PasswordController extends Controller
{
    use ResetsPasswords {
        reset as parentReset;
    }

    public function reset(Request $request)
    {
        $this->validate($request, [
            'token' => 'required',
            'email' => 'required|email',
            'password' => 'required|confirmed|min:8',
        ]);
        return $this->parentReset($request);
    }

}

With this change I can do:
class PasswordController extends Controller
{
    use ResetsPasswords;

    $this->passwordValidation = [
        'password' => 'required|confirmed|min:8',
    ];
}

repo: laravel/framework


text:
开发 yeoman生成器

暂时不用 yeoman 了

repo: larkjs/lark


text:
HOMS-231 Speed up docker build

Resolved conflict with CHANGELOG.md

repo: latera/homs


text:
remove pullapprove

LGTM

repo: leancloud/ChatKit-OC


text:
Mobile UX Test Data Serialization Structure

Issue moved to leapmotion/UnityModulesInternal #3 via ZenHub

repo: leapmotion/UnityModules


text:
Edit text debugging.md

Can one of the admins verify this patch?

authors: centos-ci jhornice

repo: leapp-to/leapp


text:
It's 2016 :tada:

Thanks, Happy New Year! 🎇


text:
Fix a typo in application conf to allow run java

Coverage remained the same at 27.444% when pulling 04664a8fecea76995ce9768ac4b2d9f2b254e5cf on camelliazhang:master into ae801c8d24a25758742c724467c04371931165f1 on linkedin:master.

authors: camelliazhang coveralls

repo: linkedin/WhereHows


text:
Install mysql 8.0 on travis server (WIP)

Codecov Report

Merging #1644 into master will increase coverage by 0.07%.
The diff coverage is 80.00%.

@@             Coverage Diff              @@
##             master    #1644      +/-   ##
============================================
+ Coverage     73.58%   73.66%   +0.07%
- Complexity     8683     8697      +14
============================================
  Files           615      615
  Lines         47656    47702      +46
  Branches       6026     6028       +2
============================================
+ Hits          35067    35139      +72
+ Misses        10764    10728      -36
- Partials       1825     1835      +10

Impacted Files
Coverage Δ
Complexity Δ

...ls/src/main/java/com/github/ambry/utils/Utils.java
78.16% <66.66%> (-0.27%)
128.00 <1.00> (+1.00)
:arrow_down:

...va/com/github/ambry/account/MySqlAccountStore.java
96.00% <100.00%> (+96.00%)
12.00 <1.00> (+12.00)

...in/java/com/github/ambry/store/StorageManager.java
89.06% <100.00%> (+0.03%)
64.00 <1.00> (+1.00)

...n/java/com/github/ambry/clustermap/MockDiskId.java
77.77% <0.00%> (-7.94%)
12.00% <0.00%> (-1.00%)

...hub/ambry/protocol/CatchupStatusAdminResponse.java
75.00% <0.00%> (-6.25%)
6.00% <0.00%> (-1.00%)

...a/com/github/ambry/network/SSLBlockingChannel.java
75.67% <0.00%> (-5.41%)
4.00% <0.00%> (ø%)

...java/com/github/ambry/store/CompactionManager.java
89.94% <0.00%> (-2.96%)
27.00% <0.00%> (ø%)

.../main/java/com/github/ambry/router/PutManager.java
87.09% <0.00%> (-1.30%)
30.00% <0.00%> (ø%)

...java/com/github/ambry/network/SSLTransmission.java
70.46% <0.00%> (-0.93%)
72.00% <0.00%> (-2.00%)

...va/com/github/ambry/replication/ReplicaThread.java
83.61% <0.00%> (-0.60%)
133.00% <0.00%> (+1.00%)
:arrow_down:

... and 72 more

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 7ac904b...84ec9cd. Read the comment docs.

authors: codecov-io lightningrob

repo: linkedin/ambry


text:
Fix for failing createProxiedFileSystemUsingToken

Trivial bug fix. Merging.

repo: linkedin/gobblin


text:
Rename packages.txt to requirements.txt

@cybernobie Hey, thank you so much for improve the project, your nick was included in contributors on main README <3

authors: cybernobie loseys

repo: loseys/BlackMamba


text:
Revert LazyList back to Stream for Shrink

Thanks @ashawley, i pushed this to https://github.com/rickynils/scalacheck/pull/411/commits

authors: ashawley lrytz

repo: lrytz/scalacheck


text:
Prettier

Added the following packages and configuration:
npm i -D prettier eslint-plugin-prettier eslint-config-prettier

eslintrc.json:
{
  "extends": [
    "prettier/@typescript-eslint",
    "plugin:prettier/recommended"
  ],
  "plugins": [
    "prettier"
  ]
}

Results
Prettier worked well, it indicates errors and has features like auto-formatting long lines that ESLint doesn´t have
These are negative points noted:
Errors just indicate how to solve, but don't show the rule of why. ESLint shows a better tooltip and it's possible to search for rule for more information.
(Prettier + ESLint)

(just ESLint)

The so opinionated format isn't of my preference. ESLint Airbnb format looks more close to a pattern that I see around many repos
(Prettier with error)

(Prettier fixed)

Conclusion
Auto-formatting of long lines is not worth with these trade-offs
Prettier (once again) doesn't suit me.

repo: luanorlandi/gaming-bot-sdk


text:
rename retransmittable to ack-eliciting

Codecov Report

Merging #1866 into master will not change coverage.
The diff coverage is 89.8%.

@@           Coverage Diff           @@
##           master    #1866   +/-   ##
=======================================
  Coverage   86.27%   86.27%
=======================================
  Files          98       98
  Lines        6666     6666
=======================================
  Hits         5751     5751
  Misses        655      655
  Partials      260      260

Impacted Files
Coverage Δ

internal/ackhandler/received_packet_handler.go
65.63% <ø> (ø)
:arrow_up:

internal/ackhandler/ack_eliciting.go
100% <100%> (ø)

packet_packer.go
89.02% <100%> (ø)
:arrow_up:

internal/ackhandler/sent_packet_handler.go
76.9% <100%> (ø)
:arrow_up:

session.go
75.96% <66.67%> (ø)
:arrow_up:

internal/ackhandler/received_packet_tracker.go
80% <66.67%> (ø)
: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 9ad6a7b...b5336be. Read the comment docs.

authors: codecov-io marten-seemann

repo: lucas-clemente/quic-go


text:
Batch update dependencies from other PRs

Coverage remained the same at 100.0% when pulling 76a4de906b0c33c4e24a196c3c13258c2ea543c1 on batch-update-deps into ddc04cc5b3c7e03afe2019d431e1708923759b76 on master.

authors: coveralls lukegb

repo: lukegb/SpongeAuth


text:
Update aria2 to 1.31.0

I removed extract_dir accidentally.

repo: lukesampson/scoop

license: unlicense


text:
Merge feature/vnext to master

@kroniak You are the best!

authors: kroniak lvermeulen


text:
[bugfix] fix timeout interval when calling df

:+1:

:bow:

authors: Songmu kazeburo

repo: mackerelio/mackerel-agent


text:
feat(api): fully migrate to async/await exposing an async API

Closing as this is superseded by PR #128


text:
improve page seo

Using this lib: https://github.com/garmeeh/next-seo

Using this lib: https://github.com/garmeeh/next-seo


text:
Get to 100% code coverage for Jest tests

Done in the HTMLRewriter branch. Waiting for a code contributor to agree to license MIT code to merge that.

repo: markjaquith/cf-worker-utils


text:
Little Edizz

Throwing away all usings is usually very annoying when you want to add another code and default usings are missing.


text:
What stream should I use for rate limiting (i.e. for object mode)?

I think just setting the high water mark will pause stream consumption in object mode. See https://nodejs.org/api/stream.html#stream_buffering
One type of throttling you can add in though, is https://github.com/maxogden/byte-stream which lets you batch up streams by count and time in object mode, or byte count + time not in object mode.

Didn't mean to close, but please close if that answers your questions.

Closing.  Speak up if there are unaddressed issues.

authors: bcomnes sprijk

repo: maxogden/mississippi


text:
Added known issue when committing / Updating

@arjan-h is there a ticket number for this?

authors: MarkvanMents arjan-h

repo: mendix/docs


text:
Fix unstable test mesosphere.marathon.storage.repository.RepositoryTest

Duplicate of #4716
Please search first.

authors: jasongilanfarr marcomonaco

repo: mesosphere/marathon


text:
Removed aio/mcp naming from docs and templates

:+1:

repo: metacloud/molecule


text:
Fix security issue in dependencies

Landed in https://github.com/metarhia/config/commit/15fd8e223704c9c6a8ccdf7f5569783155c098a0

repo: metarhia/config


text:
Update phantom peerDependency to more lax.

Phantomjs minor versions have been pretty major... Not to mention 1.10 will likely never come as phantomjs 2 is the focus.
What's the motination for this?

Well this version was quite old and it was preprint npm installs (due to peerdep hell) from working ... So eh I kinda wanted to loosen the restriction and allow users to be explicit and choose and earlier version if they would like.

Earlier than 1.9.1?

Phantomjs 2 just came out - I doubt 1.10 will ever come out.

dope

authors: metaskills nathanboktae samccone

repo: metaskills/mocha-phantomjs


text:
Add proxies support to Api.py

Thanks for this!

authors: jeflopo michaelhelmick

repo: michaelhelmick/python-tumblpy


text:
correção de erro gramatical

"orinalmente" corrigido para originalmente!

Thanks for taking the time to contribute this localization pull request. However, we are not able to accept contributions to this historical reference repository at this time.

authors: hugovms microsoftopensource

repo: microsoft/MS-DOS


text:
codeql analysis

Codecov Report

Merging #14 (c49409d) into main (be8bb6d) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #14   +/-   ##
=======================================
  Coverage   81.81%   81.81%
=======================================
  Files          12       12
  Lines         176      176
=======================================
  Hits          144      144
  Misses         32       32

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 be8bb6d...c49409d. Read the comment docs.

authors: codecov-commenter mike-north


text:
Add a dropdown menu to landing page

No dropdown menu, only search bar

repo: mikemiksch/Gather-The-Magic


text:
Create LICENSE

Nada

repo: mindpad/Higher-Interaction-Models


text:
Fix post merge issues

merged.

repo: minerva-ml/steppy


text:
Copy changed of jurisdiction names in seed data

Looks good to me :+1:
You are good to merge

authors: colinbruce kelliematheson

repo: ministryofjustice/fr-staffapp


text:
Fix/minor release fixes

Coverage remained the same at 61.683% when pulling 11bbae4f32af45c32a3c61595042c897cce1cf1b on jennapederson:fix/minor-release-fixes into 4322a68dcd342a55155f850d8eac98cf6a616b9b on minnestar:master.

authors: coveralls jennapederson

repo: minnestar/sessionizer


text:
STITCH-2742: Update release scripts and README to reflect new Github requirements

I used a tool called hub to open the pull request because git request-pull doesn't have the desired behaviour: https://stackoverflow.com/questions/34945947/git-request-pull-how-to-create-a-github-pull-request-on-the-command-line

repo: mongodb/stitch-android-sdk


text:
response: include matcher for IndividualName

Codecov Report

Merging #3 (246582e) into master (f3f0451) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master       #3   +/-   ##
=======================================
  Coverage   22.47%   22.47%
=======================================
  Files           5        5
  Lines          89       89
=======================================
  Hits           20       20
  Misses         63       63
  Partials        6        6

Impacted Files
Coverage Δ

pkg/service/model_config.go
0.00% <ø> (ø)

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 f3f0451...246582e. Read the comment docs.

authors: adamdecaf codecov-io

repo: moov-io/ach-test-harness


text:
build: enable windows in TravisCI

Codecov Report

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

@@           Coverage Diff           @@
##           master      #49   +/-   ##
=======================================
  Coverage   93.65%   93.65%
=======================================
  Files          12       12
  Lines         473      473
=======================================
  Hits          443      443
  Misses         22       22
  Partials        8        8

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 2812ceb...1fbece7. Read the comment docs.

repo: moov-io/base


text:
build: update Copyright headers for 2020

Codecov Report

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

@@           Coverage Diff           @@
##           master      #58   +/-   ##
=======================================
  Coverage   94.36%   94.36%
=======================================
  Files          12       12
  Lines         479      479
=======================================
  Hits          452      452
  Misses         20       20
  Partials        7        7

Impacted Files
Coverage Δ

docker/docker.go
100% <ø> (ø)
:arrow_up:

idempotent/idempotent.go
77.77% <ø> (ø)
:arrow_up:

k8s/kubernetes.go
100% <ø> (ø)
:arrow_up:

idempotent/lru/lru.go
100% <ø> (ø)
:arrow_up:

error.go
100% <ø> (ø)
:arrow_up:

id.go
62.5% <ø> (ø)
:arrow_up:

time.go
95.23% <ø> (ø)
:arrow_up:

http/bind/bind.go
84.21% <ø> (ø)
:arrow_up:

http/server.go
100% <ø> (ø)
:arrow_up:

http/response.go
93.61% <ø> (ø)
:arrow_up:

... and 2 more

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 1c705e9...4699c68. Read the comment docs.


text:
Push notification with subscribe and unsubscribe

There is still a need to react to unsubscribe from outside of the page

I filed https://bugzilla.mozilla.org/show_bug.cgi?id=1221965, which should make it easier to test push subsrciption changes.

Recipe should react to cancelled (via notification UI) and expired push subscriptions

server should remove subscription if endpoint doesn't exist (server responds with 4xx error)
front-end should change from subscribed to unsubscribed (if cancelled) or reissue subscription (if expired)

Because of a weak storage this recipe fails when server is restarted and the subscription remains in push service. It shows that user is subscribed to notifications, but notifications do not show. Consider if handling this use case should be a part of the recipe or simply add this info to readme.

pushsubscriptionchanged is an event fired only if subscription is expired. This recipe should act if this happens

authors: marco-c zalun

repo: mozilla/serviceworker-cookbook


text:
Fixed messed up table

Looking more closely, I think the actual content of this table needs review. The "index of" example seems to have gotten mixed up with the ".." notation for selecting descendants. Maybe this pull request should be rejected in favor of a more complete correction?

Closing as whole doc already updated https://docs.mulesoft.com/mule-runtime/4.1/dataweave-cookbook-extract-data

authors: davidvancina-mulesoft kevintroller

repo: mulesoft/docs-mule-runtime


text:
Refactor embedded expectations

Will need to manually reimport all guides in production, since it changes DB model in a non-backward compatible way

repo: mumuki/mumuki-platform


text:
Add readme

Merged.

repo: mxaly/linkguru-api


text:
Contact 32 accept drag on dock

relasted https://github.com/namhokim/cocoa_app/issues/32


text:
Specific acronyms spell fix

Acronyms automatically sync to GitHub frequently from an internal list. Added spell checking to the syncing script's list of tasks.

authors: jhonatantft joelmalissa

repo: nasa/NASA-Acronyms


text:
Fix Issue: Subscriber usage text is wrong #239

Thanks for the fix!

authors: kiates watfordgnf

repo: nats-io/csharp-nats


text:
react.forwardRef support (requires react >=16.8.3)

Coverage remained the same at 100.0% when pulling dc1205b75599810263c290bb29fedcda6e33b437 on feature/react-forward-ref into cc097d734d14e4e80604c96fa91c903730d1571a on master.

Coverage remained the same at 100.0% when pulling dc1205b75599810263c290bb29fedcda6e33b437 on feature/react-forward-ref into cc097d734d14e4e80604c96fa91c903730d1571a on master.

authors: coveralls natterstefan

repo: natterstefan/react-component-catalog


text:
Add php7.4/php8 to travis config

Looks like travis isn't ready for php8.. We should anyway migrate to GH actions, if you'd like to help with that.

It looks like Travis does support php8 by now: https://travis-ci.community/t/php-8-0-missing/10132
Also, I could spend some time to work on GH actions in a separate PR if you want @Seldaek

authors: KartaviK Seldaek smatyas

repo: nelmio/NelmioCorsBundle


text:
Do I need to compile it myself if I want +clientserver?

No, all features are included. FAQ mentions this.

repo: neovim/homebrew-neovim


text:
Add ReScript language server configuration

Can anyone share vim configuration for working with Rescript on a pure LSP? (At the moment, it turned out to get COC support, but LSP is needed)

repo: neovim/nvim-lspconfig


text:
GetTodosByProject call to simplify some operations

Thanks for your PR @andrewevansmith
It's hard to review the diff because you changed the indentation of all the arrays in the README.md files. Could you revert those changes?
Could you also add a test for the getTodosByProject call?

authors: andrewevansmith ricbra

repo: netvlies/basecamp-php


text:
Visualizer enhancement

kann einfach germergt werden, oder? @pdeubel

Ich gucks mir nochmal an und merge dann :+1:

Ich merge sieht soweit OK aus. Kann man bestimmt noch mehr machen (Layout verbessern etc.) aber ist gerade nicht im Scope.

authors: bjuergens pdeubel


text:
Add essential metrics under the same name as in TiKV, make them visible in grafana.

Fixed by https://github.com/ngaut/unistore/pull/624

authors: coocood hslam

repo: ngaut/unistore


text:
Add visible handle to move the ball position without hiding it with a finger

Fixed by #61ec83c4438dc5d2d384f2d56bd6b39df264129b

repo: nhoizey/esviji


text:
Use MemoryWarehouse instead of JSONWarehouse in MemoryWarehouse get

Excellent, thank you!

repo: nickoneill/Pantry


text:
Add ssl configurations for https server mock

Hey @gr2m , here's the configuration I'll be requiring to mock localhost https servers.
I've tested them out on one of the test cases on my machine.
Can you please check on whether the configs are at the right place in the project?

Haw man, totally dropped the ball here, I’m very sorry :( I’ll try to look into it this week

Hey @gr2m . How are you?
We've got to wrap this up! It has been quite a while now ...

@mbad0la He'll probably get to this when he can - 20 days isn't that long in open source! I'm pretty sure that @gr2m does this in his spare time, which he may or may not have a lot of these days. :)

Thanks @RichardLitt :)
@mbad0la I’ve pushed an update with an example that uses a local https server. Can you work with that?

Hey @gr2m ,
Sorry for the delay here. I was intending to fix up tests separately (had them figured out already!) and that is why was waiting on a merge on this ...
Can we merge this without having to add tests here itself?

sure thing! Feel free to send pull requests with more than one tests if they go together well. What ever you think is easiest to review :)

repo: node-nock/nock


text:
Fix typos in config descriptions

Thanks !

authors: cassiersg jonas-schievink

repo: nrc/rustfmt


text:
Fixes

Codecov Report

Merging #1 into master will increase coverage by -3.62%.

@@            Coverage Diff             @@
##           master       #1      +/-   ##
==========================================
- Coverage   90.43%   86.82%   -3.62%
==========================================
  Files           7        7
  Lines         617      645      +28
==========================================
+ Hits          558      560       +2
- Misses         59       85      +26

Impacted Files
Coverage Δ

voropy/mesh_tetra.py
99.41% <ø> (ø)
:white_check_mark:

voropy/about.py
100% <100%> (ø)
:white_check_mark:

voropy/base.py
61.9% <7.14%> (-27.39%)
:x:

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 d043d3c...e70b205. Read the comment docs.

authors: codecov-io nschloe

repo: nschloe/voropy


text:
[T5A3][W15-B2]Marcus Ng Wen Jian

Accepting this PR without a review because another of your PRs for this tutorial was reviewed in detail.

authors: AlvianPrasetya marcusngwj


text:
nuxt js

WHAT?? 🙄😁

Codecov Report

Merging #869 into master will decrease coverage by 2.21%.
The diff coverage is 57.14%.

@@            Coverage Diff             @@
##           master     #869      +/-   ##
==========================================
- Coverage    99.3%   97.08%   -2.22%
==========================================
  Files          13       13
  Lines         578      584       +6
==========================================
- Hits          574      567       -7
- Misses          4       17      +13

Impacted Files
Coverage Δ

lib/nuxt.js
100% <100%> (ø)
:arrow_up:

lib/render.js
86.76% <50%> (-11.75%)
:arrow_down:

lib/module.js
87.03% <58.33%> (-8.89%)
:arrow_down:

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 208f6b9...9295185. Read the comment docs.


text:
ovirt-vm-infra: allow set state and sd for vm

The commiter is not a member of organization nor whitelist. Please contact maintainer for patch verification.

The commiter is not a member of organization nor whitelist. Please contact maintainer for patch verification.

The commiter is not a member of organization nor whitelist. Please contact maintainer for patch verification.

The commiter is not a member of organization nor whitelist. Please contact maintainer for patch verification.

The commiter is not a member of organization nor whitelist. Please contact maintainer for patch verification.

The commiter is not a member of organization nor whitelist. Please contact maintainer for patch verification.

The commiter is not a member of organization nor whitelist. Please contact maintainer for patch verification.

The commiter is not a member of organization nor whitelist. Please contact maintainer for patch verification.

The commiter is not a member of organization nor whitelist. Please contact maintainer for patch verification.

The commiter is not a member of organization nor whitelist. Please contact maintainer for patch verification.

The commiter is not a member of organization nor whitelist. Please contact maintainer for patch verification.

The commiter is not a member of organization nor whitelist. Please contact maintainer for patch verification.

The commiter is not a member of organization nor whitelist. Please contact maintainer for patch verification.

The commiter is not a member of organization nor whitelist. Please contact maintainer for patch verification.

The commiter is not a member of organization nor whitelist. Please contact maintainer for patch verification.

The commiter is not a member of organization nor whitelist. Please contact maintainer for patch verification.

Looks OK, did you verify the ovirt_vms file is compatible with Ansible 2.3? If yes, I will merge.

@mwperina

I ran this role with ansible --version
ansible 2.3.1.0
config file = /home/talayan/git/ansible-playbooks/ansible.cfg
configured module search path = Default w/o overrides
python version = 2.7.13 (default, May 10 2017, 20:04:28) [GCC 6.3.1 20161221 (Red Hat 6.3.1-1)]

The commiter is not a member of organization nor whitelist. Please contact maintainer for patch verification.

The commiter is not a member of organization nor whitelist. Please contact maintainer for patch verification.

The commiter is not a member of organization nor whitelist. Please contact maintainer for patch verification.

The commiter is not a member of organization nor whitelist. Please contact maintainer for patch verification.

The commiter is not a member of organization nor whitelist. Please contact maintainer for patch verification.

The commiter is not a member of organization nor whitelist. Please contact maintainer for patch verification.

The commiter is not a member of organization nor whitelist. Please contact maintainer for patch verification.

The commiter is not a member of organization nor whitelist. Please contact maintainer for patch verification.

The commiter is not a member of organization nor whitelist. Please contact maintainer for patch verification.

The commiter is not a member of organization nor whitelist. Please contact maintainer for patch verification.

The commiter is not a member of organization nor whitelist. Please contact maintainer for patch verification.

The commiter is not a member of organization nor whitelist. Please contact maintainer for patch verification.

The commiter is not a member of organization nor whitelist. Please contact maintainer for patch verification.

The commiter is not a member of organization nor whitelist. Please contact maintainer for patch verification.

The commiter is not a member of organization nor whitelist. Please contact maintainer for patch verification.

The commiter is not a member of organization nor whitelist. Please contact maintainer for patch verification.

ansible --version
ansible 2.3.1.0
config file = /home/talayan/git/ansible-playbooks/ansible.cfg
configured module search path = Default w/o overrides
python version = 2.7.13 (default, May 10 2017, 20:04:28) [GCC 6.3.1 20161221 (Red Hat 6.3.1-1)]

@tareqalayan have we covered the option of multiple storage domains per VM here?
say a VM with one disk on SSD SD and additional disk on HDD SD?

@mgoldboi  No if the vm is created from a template and storage_domain parameter is defined we simply copy all templates disks to the specified storage domain.

authors: machacekondra mgoldboi ovirt-infra tareqalayan

repo: oVirt/ovirt-ansible


text:
relations with datatype properties

Use library RDF4J or some other present to parse the datatypes value parts.

@josef-janousek Can you pls just summarize here in one sentence what was the issue?

authors: brodecva josef-janousek tomas-knap

repo: odalic/sti


text:
Incorp train

Bringing incorp_train branch and main in line....


text:
Activate manual test

Closing. This may be something we want to add to some repos, but no pressing need.
cc: @sbesson @jburel

repo: ome/ansible-role-ice


text:
refactor(lint): @typescript/prefer-optional-chain

Codecov Report

Merging #65 (39fb826) into master (26a52c6) will increase coverage by 0.15%.
The diff coverage is 66.19%.

@@            Coverage Diff             @@
##           master      #65      +/-   ##
==========================================
+ Coverage   32.83%   32.99%   +0.15%
==========================================
  Files          59       59
  Lines        3170     3167       -3
  Branches      494      491       -3
==========================================
+ Hits         1041     1045       +4
+ Misses       1965     1958       -7
  Partials      164      164

Impacted Files
Coverage Δ

src/WSManProcessor.ts
4.84% <0.00%> (ø)

src/actions/ACMActivator.ts
43.96% <0.00%> (ø)

src/actions/CCMActivator.ts
45.09% <0.00%> (ø)

src/actions/CIRAConfigurator.ts
6.92% <0.00%> (+0.10%)
:arrow_up:

src/repositories/AMTDeviceVaultRepository.ts
10.25% <0.00%> (ø)

src/Validator.ts
56.08% <33.33%> (ø)

src/ProfileManager.ts
63.20% <66.66%> (ø)

src/repositories/CiraConfigDbFactory.ts
46.66% <66.66%> (ø)

src/repositories/NetConfigDbFactory.ts
46.66% <66.66%> (ø)

src/DomainCredentialManager.ts
79.16% <75.00%> (ø)

... and 10 more

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 26a52c6...39fb826. Read the comment docs.

Codecov Report

Merging #65 (39fb826) into master (26a52c6) will increase coverage by 0.15%.
The diff coverage is 66.19%.

@@            Coverage Diff             @@
##           master      #65      +/-   ##
==========================================
+ Coverage   32.83%   32.99%   +0.15%
==========================================
  Files          59       59
  Lines        3170     3167       -3
  Branches      494      491       -3
==========================================
+ Hits         1041     1045       +4
+ Misses       1965     1958       -7
  Partials      164      164

Impacted Files
Coverage Δ

src/WSManProcessor.ts
4.84% <0.00%> (ø)

src/actions/ACMActivator.ts
43.96% <0.00%> (ø)

src/actions/CCMActivator.ts
45.09% <0.00%> (ø)

src/actions/CIRAConfigurator.ts
6.92% <0.00%> (+0.10%)
:arrow_up:

src/repositories/AMTDeviceVaultRepository.ts
10.25% <0.00%> (ø)

src/Validator.ts
56.08% <33.33%> (ø)

src/ProfileManager.ts
63.20% <66.66%> (ø)

src/repositories/CiraConfigDbFactory.ts
46.66% <66.66%> (ø)

src/repositories/NetConfigDbFactory.ts
46.66% <66.66%> (ø)

src/DomainCredentialManager.ts
79.16% <75.00%> (ø)

... and 10 more

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 26a52c6...39fb826. Read the comment docs.

authors: codecov-io rsdmike

repo: open-amt-cloud-toolkit/rps


text:
Fix Markdown warnings

Core extension. Heading style changes only. No impact on translations as Transifex ignores heading styles.


text:
fix(testing-karma-bs): use edge 17 instead of latest because of flaky bs

Do we still want this? CI is stilly flaky, but not sure if this fixes that?

I've been testing BS a lot recently. And Edge 18 was the least flaky. Not sure if this can really help.

authors: LarsDenBakker bashmish daKmoR

repo: open-wc/open-wc


text:
the side counts should be thousands (in some cases) not tens

It was agreed upon that we would leave the counting system the way it is to avoid false advertising.

authors: reesjones timm


text:
More fixes for hive RBAC.

This is included in #818
/close

authors: dgoodwin staebler

repo: openshift/hive


text:
Fix fabric8-analytics-build-master template

@riuvshin PTAL Roman, thanks :wink:

LGTM

authors: msrb skryzhny

repo: openshiftio/openshiftio-cico-jobs


text:
Windows: Testing Fixes and Improvements

Codecov Report

Merging #1282 into master will not impact coverage by -1.5%.

@@            Coverage Diff            @@
##           master    #1282     +/-   ##
=========================================
- Coverage    70.6%   69.11%   -1.5%
=========================================
  Files         165      170      +5
  Lines       24854    26700   +1846
  Branches     3120     3319    +199
=========================================
+ Hits        17549    18453    +904
- Misses       5975     6837    +862
- Partials     1330     1410     +80

Impacted Files
Coverage Δ

third_party/mbedtls/repo/library/ecp_curves.c
90.16% <ø> (-3.06%)
:x:

...d_party/mbedtls/repo/library/memory_buffer_alloc.c
60.25% <ø> (-2.33%)
:x:

third_party/mbedtls/repo/library/bignum.c
50.68% <ø> (-0.42%)
:x:

third_party/mbedtls/repo/library/ssl_tls.c
54.34% <ø> (-0.04%)
:x:

examples/apps/ncp/main.c
100% <ø> (ø)

src/core/api/link_raw_api.cpp
0% <ø> (ø)

src/ncp/hdlc.cpp
76.34% <ø> (ø)

src/ncp/ncp_uart.cpp
85.13% <ø> (ø)

src/ncp/ncp_base.cpp
34.56% <ø> (ø)

src/core/thread/mesh_forwarder.cpp
75.89% <ø> (+0.11%)
:white_check_mark:

... and 20 more

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 186d300...b1c5d1a. Read the comment docs.

authors: codecov-io nibanks

repo: openthread/openthread


text:
[Review] deploy only a consul server on main

IPG 158 was green

repo: openwhisk/openwhisk


text:
写的文章字数较多的情况下无法继续编辑，看看能不能使编辑的字数提升多点

好吧。超过5000啦😅

好吧，显然是超过啦！😋

😂 厉害了我的哥

不对~~~~(>_<)~~~~。你看一下：链接《字数1276》这么多内容不能在博客中却可以在 简书，掘金上面发表。😋

5w包含样式这些，不是纯文字

@AnneBlair 已经将字数调整至 20w ，你的那篇文章我测试一下，直接复制过来是 10w+

authors: AnneBlair Sunzhuokai biezhi

repo: otale/tale


text:
Custom fields api link broken - http://camaleon.tuzitio.com/custom-fields-api

@jebingeosil
Thank you for the report, I solved the problem!


text:
Exit confirmation dialogs

End turn confirmation is moved to another issue: #566 ("End turn confirmation")

repo: ozkriff/zemeroth


text:
Update zh-CN.json

Thanks!

repo: oznu/homebridge-config-ui-x


text:
record metrics per host

todos are blocking on metric tagging (possibly: https://github.com/palantir/tritium/pull/39)

repo: palantir/http-remoting


text:
Connector -> Container -> Servlet

对于一个应用来说，Connector 负责和外界沟通，解析HTTP，然后传递给 Container，根据相关 request 和 response 负责动态加载 Servlet 实例，通过 Servlet 实例的 service 方法处理请求。

https://github.com/hellmonky/note/blob/master/tomcat原理/tomcat启动时序图.jpg
tomcat 时序图

repo: path-to-concurrency/how-tomcat-works


text:
why cannot add the same label name?

looks like a duplicate of #81

authors: pahnpiseth paragvaidya

repo: pchmn/MaterialChipsInput


text:
配置有点略繁琐，不知道能不能更简洁一点。

谢谢，我再精简下吧

将Readme中的使用步骤写得更傻瓜一点，会吸引更多的人使用。比如哪句话初始化，哪些语句打开相册，打开相机，又怎么获取到选择结果之类这样。配置的化只用配置一个ImageLoader。那些个性化的配置再放到后面介绍。
可能因为我是新手，所以会这样想。请多指教

非常感谢你的建议，我会尽可能的去简化.

authors: pengjianbo whyalwaysmea

repo: pengjianbo/GalleryFinal


text:
8.0.0.1

Hello, This is an automated comment from our build system. Since this pull request originated from a user  not in the Pentaho organization, we will not proceed  with automatic validation of the pull request.  However, rest assured that it will be reviewed manually as soon  as we get the chance. Thanks for your contribution!

Hello, This is an automated comment from our build system. Since this pull request originated from a user  not in the Pentaho organization, we will not proceed  with automatic validation of the pull request.  However, rest assured that it will be reviewed manually as soon  as we get the chance. Thanks for your contribution!

authors: guduxinmo wingman-pentaho

repo: pentaho/pentaho-kettle


text:
things

Build Completed
:x: This pull request has errors. They will need to be addressed before it can be accepted. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
mvn -B -f 'pom.xml' -pl 'core' -P '!assemblies' -Dsurefire.runOrder=alphabetical -Daudit -amd clean install

Cleanup Commands
mvn -B -f 'pom.xml' -pl 'core' -P '!assemblies' -amd build-helper:remove-project-artifact

Changed files
  core/src/main/java/org/pentaho/di/core/encryption/CertificateGenEncryptUtil.java

Unit Test Coverage
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.

:warning: Coverage Changes (click to expand)
org.pentaho.di.job.entries.waitforsql.JobEntryWaitForSQL

Branch Change: -0.8% :small_red_triangle_down:
Complexity Change: -1% :small_red_triangle_down:
Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.steps.constant.ConstantMeta

Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.steps.dbproc.DBProcMeta

Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.steps.dimensionlookup.DimensionLookupMeta

Branch Change: -1.8% :small_red_triangle_down:
Complexity Change: -1.1% :small_red_triangle_down:
Instruction Change: -0.4% :small_red_triangle_down:
Line Change: -0.3% :small_red_triangle_down:

org.pentaho.di.trans.steps.ldapoutput.LDAPOutputMeta

Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.steps.replacestring.ReplaceStringMeta

Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.steps.gpload.GPLoadMeta

Branch Change: -0.9% :small_red_triangle_down:
Complexity Change: -0.8% :small_red_triangle_down:
Instruction Change: -0.2% :small_red_triangle_down:

org.pentaho.di.trans.steps.salesforceinsert.SalesforceInsertMeta

Instruction Change: -0.1% :small_red_triangle_down:

License header violations

:heavy_exclamation_mark: Copyright year is not the current year on core/src/main/java/org/pentaho/di/core/encryption/CertificateGenEncryptUtil.java. Found 2017. Was expecting 2019

authors: hbfernandes wingman-pentaho


text:
K8SPXC-172 fix tests

Test name
Status

recreate
passed

big-data
passed

scaling
passed

storage
passed

init-deploy
passed

self-healing
passed

scaling-proxysql
passed

limits
passed

demand-backup
passed

upgrade
passed

self-healing-advanced
passed

monitoring
passed

upgrade-consistency
passed

security-context
passed

operator-self-healing
passed

monitoring-2-0
passed

affinity
passed

scheduled-backup
failed

one-pod
failed

auto-tuning
passed

proxysql-sidecar-res-limits
passed

authors: JNKPercona delgod

repo: percona/percona-xtradb-cluster-operator


text:
Activate code coverage with scrutinizer-ci

Major :+1:

@andresgutierrez @steffengy : Any chance to merge this PR ? :beers:

@fezfez
Thanks for your effort, but I think that currently does not much, since scrutinizer does not seem enabled for the development branch, or am I misreading that?

@steffengy : First you have to activate the coverage in you scrutinizer config (https://scrutinizer-ci.com/docs/tools/external-code-coverage/#changes-in-scrutinizer-configuration) but i think if the total configuration (scrutinizer config + send coverage) is not in master scrutinizer will not show coverage.

authors: Green-Cat fezfez steffengy

repo: phalcon/zephir


text:
add exfat utils for media players

Move these to filesystem_packages in the config.

authors: brett pigmonkey

repo: pigmonkey/spark

license: Unlicense


text:
=15.11 support

Merging this PR will publish the following release:
7.0.0 (2020-02-04)
Bug Fixes

update plugin wrappers for 15.11+ compat (0433891)

BREAKING CHANGES

This plugin now only supports semantic-release 15.11 and above.

This is better than flipping back ad forth. I'm a plus one here too.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

authors: hweeks pmowrer updaterci

repo: pmowrer/semantic-release-monorepo


text:
v0.50.0

done

repo: pocke/mry


text:
new features

1

repo: poeticloud/eva


text:
Ci

This is everything to get CI passing.
It changes some layout to match pony-stable. We should look at which tests we can steal from there to bring over as best we can.

repo: ponylang/corral


text:
awesome-cv.cls

Unless this addresses a particular issue, I don't think we should change this for everyone. If you'd like a different font size in your own CV, you can override the command to set it to your preferred size.

authors: MiaAverianova OJFord


text:
Dev plans - install postgres from source

Approved

repo: postgresqlaas/docker-postgresql


text:
Integrate unicore-hub SSO functionality

@miltontony Hey, I have this here. What am I doing right, wrong, and where can I go next? :+1:

@codiebeulaine reviewed the password reset bits of the PR, looks good, left some comments.

:+1:  :cake: all around!!

authors: codiebeulaine justinvdm miltontony nathanbegbie


text:
GEMMOLO 534: Tag cloud update

@projectmushroom please review
Add span between anchor :before natural space

repo: praekelt/molo-gem


text:
ci(github): add lint pr action

 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.

authors: CLAassistant steebchen

repo: prisma/prisma-client-go


text:
Asset enrichment

@adityagoenka94 - code coverage should be 95%+.

repo: project-sunbird/knowledge-platform-jobs


text:
Containerize static checks

LGTM

authors: robbrockbank tomdee

repo: projectcalico/libnetwork-plugin


text:
#286: Convert tests to junit5-jupiter

@rultor merge

@rultor merge

@proshin-roman OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@proshin-roman Done! FYI, the full log is here (took me 2min)

authors: ccostin93 proshin-roman rultor

repo: proshin-roman/finapi-java-client


text:
Where is export form chooser going to be placed?

Group fields in 'edit forms', so there is an Exporter tab. That's where data pickers & export button can be added

repo: punkave/apostrophe-forms


text:
Fixed sandboxing documentation link

@googlebot I signed it

repo: puppeteer/puppeteer


text:
manual doesn't explain how to install BioNetGen

Now that Anaconda is the default install method, BNG is installed via a package, so we don't need to provide separate instructions.

authors: alubbock jmuhlich

repo: pysb/pysb


text:
Added Pipfile and gitignore for venvs

Oops wrong repo sorry.


text:
NOMRG Estimate time gain of not running CPU tests on GPU machines

"Run test" part of the CI jobs:
For linux, 16min -> 8min = 8min gain
For Windows, 21min  -> 10min = 11min gain
In both cases this amounts for ~50% of the "Run test" parts, and about ~30% of the whole CI job (including spin up, installations, etc)
The unittest_windwos_gpu used to be the slowest test job taking about 10min longer than all the other ones:

Now, we've got those 10 minutes back. So potentially this saves 10 mins of developer time on every single commit.

repo: pytorch/vision


text:
chore: 修改ss链接生成

Coverage decreased (-0.1%) to 93.907% when pulling 4922668e5fb3a2af18e7e8e14ccf85c9e374bfae on cloverzrg:master into 78e3597350f8d1d11d918a338fcbc10c757a5b40 on qious:master.

https://github.com/shadowsocks/shadowsocks-org/issues/121

authors: cloverzrg coveralls

repo: qious/ss-panel


text:
Adds specs2 showtimes to show test times for each test.

@mossprescott assigned this to you since you were doing other test suite and build work :)
example:
[info]   + convert a simple read with path projects (998 ms)
[info]   + convert a basic invoke (1 second, 149 ms)
[info]   + convert project object and make object (1 second, 119 ms)
[info]   + convert a basic reduction (322 ms)

It's already enabled on travis for everything which extends QuasarSpecification (which at the time was all of them) and personally I'm not such a fan of them on a continuous basis, especially because they are unparseable. If it would simply emit a single number, that'd be a lot better. Anyway, any who want them can enable them on the command line with
./sbt -Dshowtimes test

So, I vote against this.

I have no problem enabling them locally and closing this PR. I pretty much always want them, which is why I suggested the PR. Any other opinions?

I do like to see them locally for regression tests, and only decided not to re-enable that in my current PR at the last minute.
But I guess I agree with @paulp that it's more noisy than useful as a general thing. Good to know about that syntax for when you do want them, though.

@mossprescott @psp thanks for the discussion. Closing PR.

authors: alissapajer mossprescott paulp

repo: quasar-analytics/quasar


text:
Replace 'while' and 'for' loop to 'foreach'

Thanks @yorlov

authors: johnktims yorlov

repo: querydsl/querydsl


text:
Release binaries

Thanks to https://github.com/goreleaser/releaser

repo: qvl/sleepto


text:
Add Support for Reddit Ads Pixel

Hi @artwerks! Do you like to contribute that?

I’m a pretty basic rails developer that just works on my own side projects, I would love to contribute but I don’t know if I would actually be able to help?!
Mike
Sent from my iPhone

On Sep 29, 2020, at 16:50, Marco notifications@github.com wrote:

Hi @artwerks! Do you like to contribute that?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

No worries, I'm sure you will figure this out. Just look how the other integrations were added. It's a good opportunity to learn and we're always happy to help if you get stuck. :) Also it would be great that you can test it directly in your side project if it tracks what you expect. :)

authors: DonSchado artwerks

repo: railslove/rack-tracker


text:
WIP: Multi-arch hosts support

@imikushin There are two modes we run rancher/server in CATTLE_USE_LOCAL_ARTIFACTS=true or CATTLE_USE_LOCAL_ARTIFACTS=false.  true is the default.  Can you test your code change against CATTLE_USE_LOCAL_ARTIFACTS=false to?  Meaning run docker run -e CATTLE_USE_LOCAL_ARTIFACTS=false rancher/server.
What false does is that the agents will download the assets from the CDN URL and not from rancher server directly.  I'm concerned this mode will not work with your changes.

@ibuildthecloud from rancher-agent logs, it looks cattle-global.properties URLs are indeed used to download the binaries:
INFO: Updating configscripts
INFO: Downloading http://147.75.201.33:8080/v1//configcontent//configscripts current=&arch=arm
INFO: Running /var/lib/cattle/download/configscripts/configscripts-1-779a2b9e18687e6355ae911fdf248f6266c988eea14f2574ef1ce40627459625/apply.sh
INFO: Sending configscripts applied 1-779a2b9e18687e6355ae911fdf248f6266c988eea14f2574ef1ce40627459625
INFO: Updating pyagent
INFO: Downloading http://147.75.201.33:8080/v1//configcontent//pyagent current=&arch=arm
INFO: Running /var/lib/cattle/download/pyagent/pyagent-1-9e3dcab12585149a65d818082279708965cd826df1937a55b24f48428f8f0c065013e2bd046ea82e358d87d3d345bbab/apply.sh
INFO: Updating host-api-arm
INFO: Downloading https://github.com/imikushin/host-api/releases/download/v0.32.0_arm/host-api_arm.tar.gz current=&arch=arm
INFO: Running /var/lib/cattle/download/host-api-arm/8af523b779bb0942e1ac0d8e4272e3e4/apply.sh
INFO: Updating python-agent
INFO: Downloading https://github.com/rancher/python-agent/releases/download/v0.81.0/python-agent.tar.gz current=&arch=arm
INFO: Running /var/lib/cattle/download/python-agent/74079d5d5ffe805bd17f8e4f581ea710/apply.sh
INFO: Updating cadvisor-arm
INFO: Downloading https://github.com/imikushin/cadvisor-package/releases/download/v0.22.0-1_arm/cadvisor_arm.tar.gz current=&arch=arm
INFO: Running /var/lib/cattle/download/cadvisor-arm/37c45d0b6f05e14eaefa388e4d983f5f/apply.sh

@imikushin Perfect, thanks for checking that out.

I'm updating the binaries' build scripts and then this PR soon.

authors: ibuildthecloud imikushin

repo: rancher/cattle


text:
remove boolean check for rfc params

Closing this outdated PR. Please submit this to the latest sprint branch if needed.

authors: madanagopalt vijs

repo: rdkcentral/rdkservices


text:
port_compiler: consistently format default_env/0

Would be great if we can merge this trivial fix before the attempt to sync port_compiler with the rebar3 plugin.

repo: rebar/rebar


text:
Adding hubzilla.nl to the fallback servers

merged by hand

authors: jeroenpraat redmatrix

repo: redmatrix/hubzilla


text:
Do not allow size 0 files to be uploaded

Closed in https://github.com/refgenomics/onecodex/commit/5192c28560cd75cfb3ebd2f4ed884bfa01cdfdf6

repo: refgenomics/onecodex


text:
feat: 支持百度小程序

很棒啊，兄弟

文档也顺便更新下

👍  💯

请问这个特性不再支持了吗？

authors: Damaged-Code Darmody noyobo watsonhaw5566 yesmeck

repo: remaxjs/remax


text:
Fix AngularJS viewFinder

Won't merge after update to v2.
I'll put it on the list for the angular directive for version 2

authors: gregoiremotot rendro

repo: rendro/vintageJS


text:
feat(ci): build docker image

@ViceIce will this run on all pull requests that target master?

Build time ~30s

It should also run on renovate/* branches

authors: ViceIce rarkins

repo: renovatebot/docker-base


text:
port 53 in in use, but is not?

ter

repo: reswitched/pegaswitch

license: ISC


text:
transactions: small doc fix to 'signature_hash_internal_input'

Now part of #80


text:
add some basic documentation

@rexdivakar could you add the hacktobrefest accepted label plz and thanks

repo: rexdivakar/Telegram-Notifly


text:
Add files via upload

Thanks

authors: MahmoudFayed faisal-alsulaiman

repo: ring-lang/ring


text:
Fix RISC-V website URL

@mgielda Could you please take a look?

repo: riscv/risc-v-getting-started-guide


text:
timelocked multisig outputs not visible in rivinec wallet output

A bug that comes up as a consequence of commit 34ab7ab.

repo: rivine/rivine


text:
Dev

Einheitliche Suchstrings in Webinterface & Hintergrund.


text:
Added xsense-mvn dependency in Appveyor

Waiting the fix of robotology/yarp#1132


text:
remove NonASCII function and upgrade jhillyerd/enmime

Coverage decreased (-0.2%) to 56.581% when pulling bcc8e8d2d4808c367d812bedd245848ce3d05579 on RFC2047 into 9e61bef898594ed3814a7f0c2bbe3a2b9ee82f37 on master.

authors: coveralls rodcorsi

repo: rodcorsi/mattermail


text:
Centralized require spec_helper in .rspec

Awesome, thanks!

Uh nice I had no idea that this is even possible

authors: benlovell jmgcrespo pxlpnk

repo: roidrage/lograge


text:
reCAPTCHA says “ERROR for site owner: Invalid domain for site key”

Done ✅
https://rtcharity.slack.com/archives/GBUD95PT8/p1549339881492100

Done ✅
Turns out the new domain hadn't been added to the recaptcha config

authors: Manoj-nathwani michaltrzesimiech

repo: rtcharity/hubreboot


text:
Exclude openssl related tests from opal suite.

@eregon Is opal the only platform supported by mspec that can't work with openssl? If yes - I'm ok with it. I thought that openssl support detection can be organized as a more abstract filter, not just for opal.

Most likely not, so these examples should either be changed to not use OpenSSL stuff or create their data locally so they can easily be excluded.

Is a literal constant like OpenSSL::X509::Name in non-executed code a problem for Opal?

@eregon Well, now I understand that it's not related to openssl directly, it's here because openssl has some T_DATA type inside (that is used in this test). Using some different class in this test will not give any benefits: all platforms except of opal support it.

There's no high-level constant OpenSSL in Opal

Right, I was asking if Opal can parse (and compile to JS) a file where it meets constants it does not know about.
Seems it can:
$ bundle exec bin/opal -e 'if false; p OpenSSL; end; p :ok'
"ok"

@eregon Yes, everything is fine when the code is not invoked. But I invoke it in my patch to Opal and it raises a load error on require (and then a name error on resolving a constant name). Simulating a class hierarchy in Opal might fix an error, but that's not a solution.

OK, I'll work on moving it or using another class which is less problematic.
Thanks for reporting the issue!

@eregon Thank you. However I'm not sure whether ruby stdlib has such class. It must be available without explicit require-ing and it must have a C pointer inside :smile:

Dir has it (a Dir* in C). So I changed the spec to use that in 73c4c70b2ddff7db73429819b1d36ad2de51146c.
That should fix the issue :smile:

@eregon Thanks!

:+1:

authors: elia eregon iliabylich

repo: ruby/rubyspec


text:
add website 2 and 3 to profiles [#238]

Thanks for the feedback @tyranja :) I've done this.

Ha, after adding another test similiar to the one in profile editing I even found out that we missed a field in the admin show view :roll_eyes: Good to have good tests :)

👍

I have changed some of the view so it looks like that now:
In profile show with different amount of websites added:

In profile edit:

Some thoughts:

how do we handle fallbacks? Currently when I enter 2 websites in my german profile, these 2 websites are shown in the :de and :en locale. When I enter 1 website in my english profile, this website is shown in the :en locale but the second website from the german profile still shows up.
--> Suggestion if possible: use fallback only when no website is given

@sabrina-ulbrich I can't reproduce your bug you mention above. Is it still a problem?

Do you mean use the fall-back only when no website in that language is given at all?
I have changes the code now that way:
When only the first website in one language is provided:

use normal fall-back

When only the first website and one of the additional websites in one language is provided:

the first website uses normal fall-back
the other is only shown in their language ( no fall-back )

When the first website is in both language there and one of the additional websites in one language:

the first website in both languages
the other are only shown in their language ( no fall-back )

That means the 2 additional languages have no fall-back option.
How does that sound?

@zaziemo could yo have a look at it today?

New behaviour now:
no website in current language scope:

use fall-back

at least one website in current language scope:

don't use fall-back for all the other websites

Here is the test:
  describe 'globalize fall-back options for websites' do
    before do
      sign_in ada
      click_on 'EN', match: :first
    end

    it 'when there is no website given for the current language scope you are in use the fall-back' do
      expect(page).to have_content('www.ada.de')
      expect(page).to have_content('www.ada2.de')
      expect(page).to have_content('www.ada3.de')
      click_on 'DE', match: :first
      expect(page).to have_content('www.ada.de')
      expect(page).to have_content('www.ada2.de')
      expect(page).to have_content('www.ada3.de')
    end

    it 'when there is at least one website given for the current language dont use fall-back' do
      ada.website_en = "www.ada.en"
      ada.website_2_en = "www.ada2.en"
      ada.save!
      visit current_path
      expect(page).to have_content('www.ada.en')
      expect(page).to have_content('www.ada2.en')
      expect(page).not_to have_content('www.ada3.de')
      click_on 'DE', match: :first
      expect(page).to have_content('www.ada.de')
      expect(page).to have_content('www.ada2.de')
      expect(page).to have_content('www.ada3.de')
    end
  end

Yay, great work @sabrina-ulbrich @tyranja ❤🌈🙌🏼

authors: sabrina-ulbrich tyranja zaziemo


text:
Use 'dyn' since trait objects without an explicit 'dyn' are deprecated

Thanks for the PR @Atul9! Unfortunately this would require bumping our minimum supported Rust version, so I think we can solve this by allowing the bare trait object instead:
#[allow(unknown_lints, bare_trait_objects)]
fn cause(&self) -> Option<&Error> {
  ..
}

@KodrAus done. I have updated the PR.

repo: rust-lang-nursery/glob


text:
Add issue templates

r? @QuietMisdreavus


text:
Update highfive's configs

r? @pietroalbini

repo: rust-lang/highfive


text:
link to our shiny new Zulip stream

@bors r+

:pushpin: Commit e204c0fbaba21ba3771cdbfbbbd2aab9abfc4864 has been approved by RalfJung

:hourglass: Testing commit e204c0fbaba21ba3771cdbfbbbd2aab9abfc4864 with merge d895f6e22991fdcb61c41e57a80337cd83c2c0e5...

:sunny: Test successful - checks-actions
Approved by: RalfJung
Pushing d895f6e22991fdcb61c41e57a80337cd83c2c0e5 to master...

authors: RalfJung bors

repo: rust-lang/miri


text:
专题复制功能疑似未实现

Fix: https://github.com/ryanlee2014/CUP-Online-Judge-Frontend/commit/c0cccc6f63486720739a6ce5f156d16161d4a3ba


text:
fixed PointPointCollisionTest

Changes Unknown when pulling 67e22b1fdf493d993d905282562db7b90f637b8c on vladfedorenko:lecture_02_task into ** on rybalkinsd:master**.

authors: coveralls vladfedorenko

repo: rybalkinsd/atom


text:
Compile file and lines

I think something got foo#'d in my coverage thingy which I have not touched in years :) Good idea combining the two, I can think of some neat use cases for it 👍 Let me fix the CI stuff and ill merge it in.
Thank you for your contribution!

This feature is already used in ipfs add command in this pull request. I will change my fork address to original address, thank you.

authors: crackcomm sabhiram

repo: sabhiram/go-gitignore


text:
Check if "jokers" could be extended to other search modes

exact mode: not possible

at least these: has no meaning, since we want to include all unknown ingredients in the result

any of these: also has no meaning as we want all recipes with all unknown ingredients

group mode: could have meaning.

It would rather be more meaningful to limit the number of unknown ingredeints in the search result.


text:
Changing Act On Redirect URI to https://oauth.us/auth

dotmailer works but doesn't access their api, only for single sign on right now
changed redirect uri for act-on to use oauth uri

repo: saltt/oauthd


text:
Less verbose errors

TODO: add to history once #89 is merged.

repo: samuelcolvin/pydantic


text:
consistent api across all platforms

@jacobrosenthal thanks!
Manually merged, with some minor changes: https://github.com/sandeepmistry/bleno/commit/0cff272e2c93a13ff7c0b71612075b1211d10b3f

authors: jacobrosenthal sandeepmistry

repo: sandeepmistry/bleno


text:
Create 0000-01-02-sbrock314.md

hey


text:
Fix a terrible bug in core.

Current coverage is 15.80% (diff: 78.68%)

Merging #44 into master will increase coverage by 5.43%

@@             master        #44   diff @@
==========================================
  Files            30         31     +1
  Lines           839        886    +47
  Methods         800        849    +49
  Messages          0          0
  Branches         39         37     -2
==========================================
+ Hits             87        140    +53
+ Misses          752        746     -6
  Partials          0          0

Powered by Codecov. Last update f921306...fb88538

repo: scalax/fsn


text:
Update delete api.

Current coverage is 15.06% (diff: 0.00%)

Merging #52 into master will increase coverage by 0.30%

@@             master        #52   diff @@
==========================================
  Files            44         44
  Lines          1084       1062    -22
  Methods        1036       1014    -22
  Messages          0          0
  Branches         48         48
==========================================
  Hits            160        160
+ Misses          924        902    -22
  Partials          0          0

Powered by Codecov. Last update beda5ea...3cdb41f


text:
Add commands to a seperate shell script.

Travis tests have failed
Hey @kmehant,
Please read the following log in order to understand the failure reason.
It'll be awesome if you fix what's wrong and commit the changes.
TravisBuddy Request Identifier: aeeff770-afa8-11ea-ae15-c70b0cf57a1c

authors: TravisBuddy kmehant

repo: scorelab/BassaClient


text:
Issues setting default filters fixed, further testing and refactoring

Coverage decreased (-13.5%) to 70.604% when pulling 71e724a6ad26b1a5b9ff1950e10f2013f97b272f on develop into f46dc73c206d2eb983486804077202b6f58eb580 on master.

authors: coveralls sean-ww

repo: sean-ww/react-redux-datatable


text:
Link to a windows installation guide

Thanks!

authors: sebcrozet ubershmekel

repo: sebcrozet/nphysics


text:
Process Section

Process icons look great on standard desktop, mobile and tablet. When resizing the window on desktop, they get a little bit misaligned


text:
Mission Numpy type

My pythran installation wouldn't even run without this type added :)

Well, now it works... I don't really know why. I think I can close this one..

repo: serge-sans-paille/pythran


text:
feat: support custom entry file and all scf config

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

authors: slsplus yugasun

repo: serverless-components/tencent-express


text:
#251

closes #251

repo: setlife-network/trinary


text:
235 enforce staysignedin

Coverage decreased (-93.2%) to 5.297% when pulling 10bb7513064e77a17c9e422f73c37b1beb65d9f0 on 235-enforce-staysignedin into 6cc54c28725f22a50415d978f719a02cd1baa187 on production.

Coverage remained the same at 98.491% when pulling 6c34cb7ec470805b1692f421fca0e2cf386c14b5 on 235-enforce-staysignedin into 6cc54c28725f22a50415d978f719a02cd1baa187 on production.

@jszwedko fyi, I wasnt able to use the devise route mappings, as in the controller test files (:create, :destroy, etc) . I did a quick Google search and apparently the devise library isn't compatible with Integration Tests for some reason... so hardcoding the route was the quickest solution. Thanks!

Looks good, thanks for the explanation @jafowler49 !

authors: coveralls jafowler49 jszwedko

repo: sfbrigade/adopt-a-drain


text:
removing console.log from ngu-carousel.component.ts

+1

authors: em-zet ohotniksergey

repo: sheikalthaf/ngu-carousel


text:
Did fluffypony change his mind after telling he wants to pump and dump XMR?

“I thought, ‘I’m going to pump it and dump it,’ because I was interested and taking the ideas
and implementing them in bitcoin. The bitcoin code base was far more interesting to me than monero,
and I thought, ‘I’m not going to work on this codebase, it’s terrible” he recalls.

fluffypony on Monero

https://www.coindesk.com/drugs-code-icos-moneros-long-road-blockchain-mainstream

Spam or truth? There is only one way to find out. Look into the sources yourself.

tari-project/tari#2904

tari-project/tari#2893

tari-project/tari#2892

tari-project/tari#2859

tari-project/tari#2979

tari-project/tari#2977

tari-project/tari#2976

tari-project/tari#2975

repo: sicTrust/tari


text:
fix build

@palashmon Can I merfe this ?

Yup, please go ahead :+1:

authors: SaraVieira palashmon

repo: siddharthkp/bundlesize


text:
added plugin tag

merged

repo: sivabalansethu/devops-techtalk


text:
Fixed if inline issues #252 #253 #309

Closing pull request, another one with fixed branch was submitted.

repo: sjbarag/brs


text:
Format slack messages and shit

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.0 out of 2 committers have signed the CLA.:x: chrislu1207:x: tfshocknesschrislu1207 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

oops

repo: slackapi/easy-peasy-bot


text:
Add remote continuous integration.

I've found Travis-CI pretty easy to set up. I've only used it for Python and C++, but the way it works (using a new VM instance for each build) should make it straightforward to set up for any language, and it integrates well with github.

I turned on Travis CI for the repo. What happens next (for me) is usually copying some other PureScript .travis.yml file. :smile:

authors: beckyconning daira jdegoes

repo: slamdata/purescript-aff-reattempt


text:
Recruit 를 생성한다. : 도메인 어그리게이트 생성 저장

Recruit이 Managed URL에 의해 관리된다면, Recruit 엔티티는 값이 아닐까?

RecruitId는 존재하지않고, RDB를 위한 id만 존재될 수 있다.

값으로, 정의하는 순간 RDB에서 대리 식별자가 정의되나, 값 변경시 필요하니까 엔티티가 되는게 맞다고 판단됨

Managed URL: Owner가 Recruit를 관리하기 위한 URL으로 Recruit 생성시 만들어진다.
ManagedUrl 은 추후 인증과 권한과 관련이 있을것 같다.

authors: LenKIM yangwansu

repo: slippStudy/moim


text:
Add two more progress bars: global and task

See https://github.com/smith-chem-wisc/MetaMorpheus/issues/233

repo: smith-chem-wisc/MetaMorpheus


text:
rewrite toolbar to use react-toolkit

closed due to lack activity

repo: solsort/bogapp


text:
Add job manager

This is ready to review!

@molant ready to review again

It would also be kinda nice if you used Docker and docker-compose to containerize all of your service dependencies like MongoDB and for local development.

I'll take a look to this and also the other packages you mention. :)

@sarvaje You don't have to use any of the modules, but I thought I would just throw them out there as they may make things easier.

@austinkelleher I've been taking a look and definitely I'm going to use tri hehehe. For urgly it looks really nice, but in this case optionator have some features that I'm using and urgly doesn't have (like enumerators for the options).

@austinkelleher changes done!

After my comment, your changes look good to me! Glad tri worked nicely for you. 👍

It would also be kinda nice if you used Docker and docker-compose to containerize all of your service dependencies like MongoDB and for local development.

@austinkelleher I don't know if I get it or not (also because docker is new for me hehehe). Do you mean a Docker with all the code and a composer to add MongoDB and develop with that. Or add some script to auto install a docker with MongoDB when people clone the code and run npm install (for example)

@sarvaje You don't necessarily have to do this in the same PR. It would probably be better as a separate issue. Just wanted to suggest it! 😄
With that being said, you assume that the user has Docker installed and you use docker-compose for defining the different services and repositories where those containers are stored. Feel free to take a look at one of my repos as an example (Dynograte). In that project, I run a Docker container with a local version of DynamoDB to run my tests against. I'm not sure if there is already a docker repo for running local Azure Service Bus though. There are definitely ones to run Mongo.

@alrra done!

@molant done!

@sarvaje looks like you need more code coverage to pass the tests :P

ouch!
Ok!

done!

authors: austinkelleher molant sarvaje

repo: sonarwhal/online-service


text:
Fixed DMS orientation of trace

Pull Request Test Coverage Report for Build 84

2 of 2 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 69.699%

Totals

Change from base Build 83:
0.0%

Covered Lines:
835

Relevant Lines:
1198

💛  - Coveralls

repo: spacetelescope/awesimsoss


text:
tpl: Add basic tests for all namespace init funcs

This is the kind of tests that I only like when I can get them for free. 700 LOC in 18 files isn't free -- and making sure every package has at least 1 example test will accomplish the same test coverage.

authors: bep moorereason


text:
Fixed typo

Good catch, thanks @pborreli.

authors: fabpot pborreli

repo: splitsh/lite


text:
support loading stack file addresses from query string

Hi @craynic, thanks for this!
Could you please add:

An example-dump.txt file with an example thread dump

A clickable URL to both index.html and README.md showing how to use the by-query-string feature to analyze the example dump.
Regards /Johan

http://craynic.github.io/tda/index.html?url=stacks/jstack.sample.txt
It could be a demo for this feature.

I don't known how to write a readme. Is there any sample or template?

There's a README.md file at the top of this repo, just add it there!
And to the index.html, since that's what people will see when they just come from Google or wherever.

I just realized I don't think this works across domains; if I merged this it would only work for URLs on the spotify.github.io domain, which would be pretty useless.
For a demo, try this link and watch your Javascript console, note the CORS warning:
http://craynic.github.io/tda/index.html?url=https://www.kth.se
Google "CORS" to learn more, but I think it's not possible to get this working cross domains :(.

Sorry I'm a little busy these days. In our usage we do serve the script and stack files on a same domain.
Very sorry for that. I will close this PR.

authors: craynic walles

repo: spotify/threaddump-analyzer


text:
spring.cloud.discovery.enabled=false should behave the same as spring.cloud.consul.discovery.enabled=false

Closed via ed2ae39

authors: OlgaMaciaszek ryanjbaxter

repo: spring-cloud/spring-cloud-consul


text:
Time source enhancements

squashed and merged as https://github.com/spring-cloud/spring-cloud-stream-modules/commit/b83f18e399bf692241d1b161e55fbe6e99b6f951

authors: dturanski markfisher


text:
Upgrade Vault version to 1.3.4

Thanks, that's merged now.

repo: spring-projects/spring-vault


text:
LIBS-456 - support for remote-hosts fixed

Jenkins build was successful.


text:
Test + fix for out ints larger than a byte

Thanks. Pointer code is so goofy.

authors: kg mispy

repo: sq/JSIL


text:
IOException instead of Throwable in EventListener API

@adriancole I took call.enqueue and Callback as my guide after @swankjesse suggested it
public interface Callback {
  /**
   * Called when the request could not be executed due to cancellation, a connectivity problem or
   * timeout. Because networks can fail during an exchange, it is possible that the remote server
   * accepted the request before the failure.
   */
  void onFailure(Call call, IOException e);

I tried testing it by making my EventListener throw a NPE and it actually just broke and I never received onFailure :( . I was hoping it was going to return a IOException that wrapped the NPE.

Consider this for discussion, a bullet in the chamber if you will.

If there's only one bullet in the chamber, well done I'd say! ;)
On 27 Aug 2017 13:45, "Yuri Schimke" notifications@github.com wrote:

Consider this for discussion, a bullet in the chamber if you will.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/square/okhttp/pull/3543#issuecomment-325178531, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAD615BUUo5EIuBZwyZusNytvJpqFS-Xks5scQJqgaJpZM4PDowW
.

authors: adriancole yschimke

repo: square/okhttp


text:
Implement is_tag(ref_name)bool UDF

Depends on https://github.com/src-d/go-mysql-server/issues/1

repo: src-d/gitquery


text:
drone: parallelize docker builds

This looks great! Let's wait for it to pass CI now that I've reviewed https://github.com/src-d/landing/pull/215. Sorry for the delay

http://docs.drone.io/pipelines/#parallel-execution
TL;DR: drone will execute all the tasks in a group at the same time. In our case, it will follow these steps:

build
docker, docker-api, docker-slackin in parallel
deploy

authors: rporres smeruelo

repo: src-d/landing


text:
Add kit-filesystem package

You need to add the new package to planet-express.py

pacman doesn't allow for packages to overlay on top of each other.

authors: raccube trickeydan

repo: srobo/kit-packages


text:
update travis

#536

repo: srz-zumix/iutest


text:
Fixing up some tests, changing code coverage strategy

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@8ebf649). Click here to learn what that means.
The diff coverage is 0%.

@@            Coverage Diff            @@
##             master     #236   +/-   ##
=========================================
  Coverage          ?   11.07%
=========================================
  Files             ?       70
  Lines             ?     4117
  Branches          ?        0
=========================================
  Hits              ?      456
  Misses            ?     3661
  Partials          ?        0

Impacted Files
Coverage Δ

lib/src/application/isolate_supervisor.dart
0% <0%> (ø)

lib/src/application/application.dart
26.92% <0%> (ø)

...ib/src/application/isolate_application_server.dart
0% <0%> (ø)

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 8ebf649...b2905c7. Read the comment docs.

authors: codecov-io joeconwaystk

repo: stablekernel/aqueduct


text: Feature/10 prettify dmg

repo: stefanjudis/forrest


text:
Separate Client and Server Actions?

Actually, not needed ;-) https://github.com/facebook/flux/pull/172

No I mean. The way it is now we can't have "media client" actions and "media server" actions in separate files which is a requirement for avoiding circular imports.
i.e. a common scenario is:
media client action -> webapi -> media server action

PR please.

I don't have any good suggestion for solution. Just pointing out a problem.

I am not aware of this issue yet, maybe it's not problem with ES6 modules. Let's wait.

authors: ronag steida

repo: steida/este


text:
Properly determine if an asset is regulated

closed for the same reasons as #75

closed for the same reasons as #75

repo: stellar/stellar-demo-wallet


text:
Fixes #54 - respect JWT policy setting.

I'll make a new release that includes your changes.

No problem at all, really appreciate this tool.  Thanks very much for the swift reply @stephaneey

authors: simonness stephaneey

repo: stephaneey/azure-apim-extension


text:
fixing random typos in readme.md

My fat fingers thank you :)

authors: stevenosloan waynn

repo: stevenosloan/slack-notifier


text:
Json rx rpc

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

repo: streamich/json-joy


text:
Do not crash in production on rules the browser doesn't recognize

In the latest version I added safe addon, which basically does exactly what is proposed in this PR.

authors: steadicat streamich

repo: streamich/nano-css


text:
1.2.0

Coverage remained the same at 99.663% when pulling c604272c27f6a76aae1c7ddbd46ca886ee6f39d9 on 1.2.0 into 59ba60788b5a683571d280e01e1e9e6d2b7416e7 on master.

authors: coveralls emilgoldsmith


text:
User has to know either the eye icon is active or not

Fixed on Master

Fixed in Prod.

repo: subutai-io/control-center


text:
add new separate group/class fields

Coverage remained the same at 94.074% when pulling 243549bd2bb428cf483da909279afda2f5dd6441 on add-new-fields into 88328268aeef8144fa6ecf4a7a095e274578fdf3 on master.

authors: coveralls peetucket

repo: sul-dlss/revs-indexer-service


text:
Update pairing.md

merge update of pairing matrix

authors: gillsuGeorge tom-reith


text:
[WIP] Add dependency analyzer

@kdy1 PTAL

Thanks a lot!

repo: swc-project/swc


text:
Test commit 2

Thank you for your pull request and welcome to our community! We require contributors to sign a Contributor License Agreement and we don't seem to have CLAs on file for these contributors to the Pull Request: (@lmacedodaitan). In order for your PR to be reviewed and merged, please follow the directions at the link above.
Project team: please do not merge this Pull Request until Foundation staff have confirmed that a CLA is in place for the new contributor(s) listed above.
If there are any questions, please don't hesitate to get in touch with our Infrastructure Support team. /CC @finos-admin

Codecov Report

Merging #151 into dev will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev     #151   +/-   ##
=======================================
  Coverage   88.06%   88.06%
=======================================
  Files         135      135
  Lines        2405     2405
  Branches      280      280
=======================================
  Hits         2118     2118
  Misses        176      176
  Partials      111      111

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 87ab81b...186e3bd. Read the comment docs.

authors: codecov-io finos-admin lmacedodaitan

repo: symphonyoss/App-Integrations-Commons


text:
[CS] Fixes #2365 Add ForbidNewOutsideFactoryServiceRule

All checks have passed 🎉 @TomasVotruba it is ready for review.

All checks have passed 🎉 @TomasVotruba I think it is ready.

rebased.

Thank you :+1:

rebased.

Oh, this should've been merged ages ago. I forgot to apply :heavy_check_mark:
Merged, thank you

authors: TomasVotruba samsonasik

repo: symplify/symplify


text:
Post coverage results to coveralls.io

The reason that Travis CI failed is because this GitHub repo is not registered with coveralls.io. See https://docs.coveralls.io/ for details.

If you're curious what the output looks like:

repo: t-yuki/gocover-cobertura


text:
ADD: Most Recent Url Dropdowns

Coverage increased (+0.04%) to 98.0% when pulling 3b5ca754b6d480e1086c5782a197a0c3a9ad6c0f on agreene/mru into f120aa62d12ccf329945f3e444fb8a6f56a07dcc on dev.

Looks awesome! Love the placement of the recent list and that it's pre populated with the samples.
Two comments:

Let's axe the reset button.  It's useless now that we have the url persisted.  It's functionally equivalent to refreshing the browser. (yeah we don't have to reload any js when just doing reset, but not worth having it in the UI just for that, I think it can be confusing to have that button there).
The added margins on the header look strange to me because of the offset the header now has with the URL input.

This is a super useful feature.  Thanks!

Coverage increased (+0.04%) to 98.0% when pulling e5cb14153c91cff0b3d4f9bf992493f47eb5c1f6 on agreene/mru into f120aa62d12ccf329945f3e444fb8a6f56a07dcc on dev.

Coverage increased (+0.04%) to 98.0% when pulling 387a59b52617c11ae646613ca6013216c1677411 on agreene/mru into f120aa62d12ccf329945f3e444fb8a6f56a07dcc on dev.

I pinged @jagreene to raise a small npm dependency issue and it's resolved in e8b3dd0. I kind of like the experience of the Reset button but I'll leave that to @lbrendanl. Looks good by me!

authors: brandoconnor coveralls jagreene lbrendanl

repo: tableau/webdataconnector


text:
Release (2021-05-30)

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

repo: team-aie/app


text:
Api tests register route

Codecov Report

Merging #87 into master will increase coverage by 1.53%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##           master     #87      +/-   ##
=========================================
+ Coverage   55.26%   56.8%   +1.53%
=========================================
  Files          80      80
  Lines         845     845
  Branches       70      70
=========================================
+ Hits          467     480      +13
+ Misses        338     326      -12
+ Partials       40      39       -1

Impacted Files
Coverage Δ

server/database/db_connection.js
88.88% <100%> (ø)
:arrow_up:

server/validation/register-trainer-val.js
100% <0%> (+20%)
:arrow_up:

server/controllers/register-trainer.js
93.33% <0%> (+46.66%)
: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 2447881...f26888d. Read the comment docs.

authors: codecov-io dupreesi


text:
fix version emission

done

repo: terminal-labs/inflation


text:
CI for Node

Initializes with two players
Starts the game with a random player
Sets the current player to be whoever it is not

repo: thiagodff/github-actions-for-ci


text:
Support scopes

Oops! Excuse me, that PR was supposed to be on my own fork and was NOT supposed to be a proper implementation of scopes but rather a hack to get some scoping up and running for my fork.
But since I'm here already; we want to see this merged thomseddon/node-oauth2-server#203! :+1:


text:
THORN-2089 upgrade wildfly-nosql to 1.0.0.Beta1 and nosql-config-api to 1.3.1

Hey @scottmarlow, what's the difference between 1.3.0 and 1.3.1? I cannot see anything.

@mkouba the only change is that I updated wildfly-nosql from alpha to beta, getting ready for first 1.0 final.  From a project coordination correctness point of view, I thought we should regenerate the config, but yes, no change needed.  Should we stich with 1.3.0 of nosql config?

I'm not sure. @kenfinnigan what's your opinion?

I don't have a preference

I'll sync with master, to address being out of date.
With regard to the other underlying question, of whether we should bump a minor release of nosql-config-api, when releasing a wildfly-nosql update.  I don't plan any actual api changes, in the wildfly-nosql subsystems until the next wildfly-nosql version, of which I would like to start on.
So, I think the take away for me, is that I won't release another new nosql-config-api until wildfly-nosql 1.1/2.0 release.

@mkouba I reverted back to use 1.3.0 for nosql config.

The windows build failed with usual MP FT TCK failure (BulkheadAsynchRetryTest).

@mkouba do people merge their own PRs for simple changes like this or is it best to always have someone else merge?
Second question, do you all want me to approve/merge changes like this pr if they come from someone else?  I seem to have commit rights now, thanks, but wanted to make sure that was intentional before using them.

@mkouba do people merge their own PRs for simple changes like this or is it best to always have someone else merge?

Sometimes yes, esp. when someone else approved the PR. But I think it's usually better when someone else merge... I can merge this one.

Second question, do you all want me to approve/merge changes like this pr if they come from someone else? I seem to have commit rights now, thanks, but wanted to make sure that was intentional before using them

I'd say we would be happy if you review/approve PRs where someone requested you to do so (in a comment or github reviewers).

authors: kenfinnigan mkouba scottmarlow

repo: thorntail/thorntail


text:
Add unstyled search bar

If this is an “unstyled” component, do we need to have an image within the button? Personally, I would immediately strip it out, because:

Usually a button with the word “Search” in it will get me pretty far to start with; i.e. no icon

If I did want an icon in the button, I would use SVG
Right now, there’s no fallback text for the button image

authors: Magnus-G tysongach

repo: thoughtbot/refills


text:
master直接上2.0了？稳定吗 看2.0目录变动好大 sql备份文件也没有找到呢

看到更新了 感谢

repo: thousmile/spring-admin-vue


text:
refactor: improve label help messages

/merge

repo: ti-community-infra/tichi


text:
[core] Remove embedded server abstraction

JavalinJettyServer javalinJettyServer = new JavalinJettyServer(jettyServer, new JavalinServlet(
This part should probably just be a function which attaches handlers to the server, a JavalinJettyServer class is not needed.

Also did https://github.com/tipsy/javalin/issues/238 in this.

Thanks @majorpasza. The threadpool has always been configurable:
QueuedThreadPool tp = new QueuedThreadPool(250, 8, 60_000);

app.server(() -> new Server(tp)) // after this change

app.embeddedServer(new EmbeddedJettyFactory(() -> new Server(tp))) // before this change

Ouch, yea, got triggered by those defaults and forgot about that

authors: majorpasza tipsy

repo: tipsy/javalin


text:
Move to Travis-CI

Moved to GitHub Actions

repo: tkrs/poston


text:
sliding on 45 angles is super weird

#good_enough


text:
Add Notification Module

Added in PR https://github.com/tomochain/tomodex/pull/53

repo: tomochain/tomodex


text:
CI for Node

ok

ok

OK


text:
Add codecs for JsonObject and JsonNumber

Thanks! Looks good to me—I'll merge when green and publish a new snapshot shortly.
One thing: do you mind adding an issue noting that we should have an Arbitrary instance for JsonNumber so that we can test its codecs? I don't think that needs to happen in this PR, but I'd like to remember that it should happen eventually.

Also, for the record, these additions are discussed briefly here.

Current coverage is 68.73%

Merging #74 into master will increase coverage by +0.12% as of e9a897d

Powered by Codecov. Updated on successful CI builds.

authors: codecov-io ochrons travisbrown

repo: travisbrown/circe


text:
feat: Add installing node dependencies with Gemfury

Awesome, thanks! Give it a try in prox-trace-ui and let me know how it goes

Awesome, thanks! Give it a try in prox-trace-ui and let me know how it goes

authors: dbanty z-triax

repo: triaxtec/github-actions


text: Images


text:
WebWallet to support DID Exchange

already implemented
https://github.com/trustbloc/edge-agent/blob/master/cmd/user-agent/src/pages/chapi/wallet/get/didConn.js#L22
https://github.com/trustbloc/edge-agent/blob/master/cmd/user-agent/src/pages/chapi/wallet/common/didExchange.js#L18

repo: trustbloc/edge-agent


text:
Adding support for enums

Current coverage is 86.53%

Merging #38 into master will increase coverage by +0.40% as of c31a592

@@            master     #38   diff @@
======================================
  Files            2       2
  Stmts          101     104     +3
  Branches         0       0
  Methods
======================================
+ Hit             87      90     +3
  Partial          0       0
  Missed          14      14

Review entire Coverage Diff as of c31a592

Powered by Codecov. Updated on successful CI builds.

authors: codecov-io sjturley

repo: turowicz/swift-serializer


text:
Tab Cycling

Duplicate of #20

repo: turtlebrowser/turtlebrowser


text:
Bumping to bijection 0.8.0

merge when green

authors: ianoc jnievelt

repo: twitter/algebird


text:
[BUG] Unnecessary attribute target = "_blank" of link in "Table of contents" list on "Hooks" page

Duplicate of #115

repo: uStudioCompany/ustudio-ui


text:
Otherwise it is crashing on torchvision 0.10.

Thanks!

repo: ubc-vision/COTR


text:
Upgrade to OpenTracing API 1.0rc4

@abhinav

LGTM

Ok, this looks good to me.
-P

authors: abhinav blampe thepaulm yurishkuro


text:
fix #4: correct some inconsistencies with closing tags

Thank you! 😁

authors: SergeyKhval robdodson


text:
Use the function type as the base for spy.

I found this problem as I was working on a new test suite using spys:
> require('unexpected')(require('sinon').spy(), 'to be a function');
Promise (fulfilled)
> require('unexpected').use(require('unexpected-sinon'))(require('sinon').spy(), 'to be a function');
UnexpectedError:
expected spy1 to be a function

Coverage remained the same at 92.573% when pulling beb3ece28f8d29cfeefa721b20ea1e9b9a86c375 on feature/useFunctionAsBaseTypeForSpy into 3cfd6683c76d1cbef068446ea4849c2606a07695 on master.

This exposed a bug in unexpected-documentation-site-generator, which I've now fixed. Trying again...

Coverage remained the same at 92.573% when pulling 8e79f4618b76189404662f214e71bd0018ccc616 on feature/useFunctionAsBaseTypeForSpy into 3cfd6683c76d1cbef068446ea4849c2606a07695 on master.

Coverage remained the same at 92.573% when pulling 8e79f4618b76189404662f214e71bd0018ccc616 on feature/useFunctionAsBaseTypeForSpy into 3cfd6683c76d1cbef068446ea4849c2606a07695 on master.

Released in 10.5.0.

authors: coveralls gustavnikolaj papandreou

repo: unexpectedjs/unexpected-sinon


text:
Added USA Freedom Act

Thanks, @nickom! Also, it looks like you made the edit to an outdated version of the CSV, so I had to resolve the merge conflict myself.
I'm happy to give you write permissions to this repo, given all your work so far. You can make future PRs in branches off of master, rather than working in a fork that may get stale.

My bad! Always glad to have someone as diligent as you checking over my shoulder! Thanks @konklone

authors: konklone nickom

repo: unitedstates/bill-nicknames


text:
amCalendar. Advanced arguments referenceTime, formats added

Coverage remained the same at 92.76% when pulling 708dc9bbd687d46a6271bfd41acabfd8acc5a6bb on Nitro-N:am_calendar_advanced_arguments into 7b1f68d45058ec43a537fa59c3a1a8426404173e on urish:master.

Thanks!

authors: Nitro-N coveralls urish

repo: urish/angular-moment


text:
.img and .bin support

Please explain?

That img files can be used like iso files on UTM

Does it not work?

No it doesn’t try yourself

You can add them to the vm. But depending on the file format qemu may or may not support it. There’s no standard for .img or .bin so you need to know the actual file format.

repo: utmapp/UTM


text:
Develop

Coverage remained the same at 97.632% when pulling 8d0a038bf37d0c01633f90e41a4c141a8fc824b4 on develop into a3f12b452a79a90a41e6ac59c81f46138f2df4f3 on master.

authors: coveralls fanglinfang

repo: uw-it-aca/uw-restclients-trumba


text:
hack #480 - show 'mean' when func is average

I added a todo to remove this hack.

authors: domoritz kanitw

repo: uwdata/vega-lite-ui


text:
Add Docker image link in README

Do you know pip install --user manuale?

@szepeviktor Yes, what is your point?

I think --user provides separation.
But I am not against a container, I do things in Docker also.

Thanks!

Thanks!

authors: jgiannuzzi szepeviktor veeti

repo: veeti/manuale


text:
Release 1.5 Canelo

@lpozo @codeshard Great news! So, is this actually ready for release? I see one test still failing... what's the next step in this case?

@lpozo @codeshard Great news! So, is this actually ready for release? I see one test still failing... what's the next step in this case?

@leonel-lordford Next step is to make the test pass.

@leonel-lordford Next step is to make the test pass.

@codeshard What do you think this is: ERROR: Failure: ValueError (Command ffmpeg not found). Is TravisCI failing to install ffmpeg?

@codeshard What do you think this is: ERROR: Failure: ValueError (Command ffmpeg not found). Is TravisCI failing to install ffmpeg?

@leonel-lordford Next step is to make the test pass.

Ok! I think the process is failing because ffmpeg is missing in the env... I think :-D So... are we sure about what's failing?

@leonel-lordford Next step is to make the test pass.

Ok! I think the process is failing because ffmpeg is missing in the env... I think :-D So... are we sure about what's failing?

@codeshard What do you think this is: ERROR: Failure: ValueError (Command ffmpeg not found). Is TravisCI failing to install ffmpeg?

Exactly! I think that is the problem...

@codeshard What do you think this is: ERROR: Failure: ValueError (Command ffmpeg not found). Is TravisCI failing to install ffmpeg?

Exactly! I think that is the problem...

authors: leonel-lordford lpozo

repo: videomorph-dev/videomorph


text:
Merge vit9696's repo

Update

Update


text:
Adjustment in ZoneDateTime pattern for 1-6 nanoseconds digits

Thanks for your Pull Request. Whenever I will have some time, I will take a look at it. For the moment, I'm very busy working on my Hypersistence Optimizer project so I don't have time to allocate to these open-source projects.

Applied upstream, thanks

authors: calvequi vladmihalcea

repo: vladmihalcea/hibernate-types


text:
Integrate new repo/tag view with Admiral

Done with PR #4401

authors: ninjadq steven-zou

repo: vmware/harbor


text:
账号与密码怎么修改呀

后台得密码应该是这里

authors: myxss renblog


text:
AMD requires are not properly linked

This was improperly closed. The commit should have referenced #19.

Released in 2.0.0

repo: walmartlabs/circus


text:
Fix gs-bgcolor and display-frame + decompiler types

Pull Request Test Coverage Report for Build 784663085

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 69.321%

Totals

Change from base Build 783324199:
0.0%

Covered Lines:
34047

Relevant Lines:
49115

💛  - Coveralls

authors: ManDude coveralls


text:
Do not merge!

 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.

authors: CLAassistant anweshan

repo: watson-developer-cloud/openwhisk-sdk