text:
package_name_to_exported_wrapper

This is untested on macos/linux.
It works on Windows.
I hope either a that I can pull in some CI testing, or something tests this with rextendr on those platforms.

This is untested on macos/linux.
It works on Windows.
I hope either a that I can pull in some CI testing, or something tests this with rextendr on those platforms.

This won't work. See here: https://github.com/extendr/extendr/issues/123#issuecomment-760383469

This won't work. See here: https://github.com/extendr/extendr/issues/123#issuecomment-760383469

I've thought about it, and now see what the problem is: The wrapper in R should be agnostic to whether the platform used was
windows, linux or mac.
I would like to still suggest that that code is still part of the exported wrapper code, as the crate-name is the same, and
one only needs to detect what platform the current R instance is on. That code can be amended to the wrapper exporter code in Rust.
I'll do that here as to experiment with it.

I've thought about it, and now see what the problem is: The wrapper in R should be agnostic to whether the platform used was
windows, linux or mac.
I would like to still suggest that that code is still part of the exported wrapper code, as the crate-name is the same, and
one only needs to detect what platform the current R instance is on. That code can be amended to the wrapper exporter code in Rust.
I'll do that here as to experiment with it.

Honestly, this is not that useful. If you want to work on this part of the code, it would be more helpful to add a code path that outputs the info required to reconstruct wrappers into a csv file. (Function names, argument names, return/no return, docstrings if available.)

Honestly, this is not that useful. If you want to work on this part of the code, it would be more helpful to add a code path that outputs the info required to reconstruct wrappers into a csv file. (Function names, argument names, return/no return, docstrings if available.)

I would welcome you to take a look at the metadata branch. This may make R code generation a lot easier.

I would welcome you to take a look at the metadata branch. This may make R code generation a lot easier.

Yes, but see: https://github.com/extendr/extendr/pull/128#issuecomment-761631432
I think we'll still have to generate output during the macro expansion phase. Otherwise we're back to regular expressions.

Yes, but see: https://github.com/extendr/extendr/pull/128#issuecomment-761631432
I think we'll still have to generate output during the macro expansion phase. Otherwise we're back to regular expressions.

I've taken this on board for the metadata wrapper builder.
I may remove the old wrapper generator as it has been subsumed into the metadata variety.
Would you mind if I closed this PR?

I've taken this on board for the metadata wrapper builder.
I may remove the old wrapper generator as it has been subsumed into the metadata variety.
Would you mind if I closed this PR?

The old wrapper generator is still being used by rextendr, though. Please don't remove it until we have ported rextendr. Relevant issue: https://github.com/extendr/rextendr/issues/28

The old wrapper generator is still being used by rextendr, though. Please don't remove it until we have ported rextendr. Relevant issue: https://github.com/extendr/rextendr/issues/28

But yes, this PR can be closed I think.

But yes, this PR can be closed I think.

Closing as this been incorporated in the metadata.

Closing as this been incorporated in the metadata.

authors: CGMossa andy-thomason clauswilke

repo: extendr/extendr


text:
Finalize addons infra

@rultor please merge.

@rultor please merge.

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

@rultor please merge.

@extsoft Done! FYI, the full log is here (took me 1min)

source: gharchive/pull-request

authors: extsoft rultor

repo: extsoft/elegant-git

license_type: permissive

license_source: bigquery


text:
Add support for overriding entry file in react-native-xcode.sh

Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. In order for us to review and merge your code, please sign up at https://code.facebook.com/cla - and if you have received this in error or have any questions, please drop us a line at cla@fb.com. Thanks!

By analyzing the blame information on this pull request, we identified @frantic and @clickthisnick to be potential reviewers.

Thank you for signing our Contributor License Agreement. We can now accept your code for this (and any) Facebook open source project. Thanks!

Hey @kennethlynne, thank you for your pull request. The react-native-xcode.sh script was designed to make the initial experience with React Native seamless. Indeed, the script is very simple and makes a lot of assumptions about the structure of your project.
However, if it doesn't fit your needs anymore, I recommend copying it into your source tree and changing Xcode to use your forked script. You will have more freedom over what happens and when. There is a trade-off in accepting this PR – it adds another configuration change that serves only small percent of the users. It will require maintenance and also encourage other people to tune it to fit their needs.

repo: facebook/react-native

license: MIT

license_source: github-api


text:
Adds emojies to ui messages

Here's what the first commit looks like:

Here's what the latest commit 28133b6 looks like

license: mit


text:
Test

Coverage decreased (-0.01%) to 74.969% when pulling 4a3d558472264c8c4ba951335aa5e3e9a7016514 on camilamacedo86:TEST into cbd257acb4809f6f9073e579afc6d8c2085c4a1c on feedhenry:master.

Coverage decreased (-0.01%) to 74.969% when pulling 4a3d558472264c8c4ba951335aa5e3e9a7016514 on camilamacedo86:TEST into cbd257acb4809f6f9073e579afc6d8c2085c4a1c on feedhenry:master.

IGONERE IT

repo: feedhenry/fh-forms


text: Unity: Add an avatar and control its movement using arrow keys

source: gharchive/issue

repo: felixerdy/origami


text:
Bug-PrintPrescriptionPatient

O pull request será aceito, entretanto verifica-se página em branco no pdf do paciente que será consertado em uma história posterior.


text:
Update installation.md

The new, updated documentation is at https://github.com/flarum/docs, you may want to check there.

@Arkinn What do you say? Do we want this here or in the flarum/docs repo?

@franzliedke I'd prefer it to remain in the flarum/docs repo, keep everything together then.

Okay, in that case... @punit2502 Could you re-submit this pull request to https://github.com/flarum/docs, please? :smiley:

Stale and obsolete, the text has already been removed.
@Arkinn As has been practice for the last few weeks (months?), we will accept contributions into this repo, so that they go live quickly. We will have to merge the contents of the old and new documentation once the new site (integrating the docs repo) goes live.

authors: Arkinn datitisev franzliedke punit2502


text:
Update composer so the plugin works with Craft 3.1

Much appreciated.

authors: jcherniak nateiler


text:
added fips-libnoise to registry

Thanks!

authors: floooh mgerhardy

repo: floooh/fips


text:
gRPC takes forever to build after adding cloud firestore dependency

Adding gRCP as a dependency reduced build time for 200s to 57s.
Has anyone else had this issue?

Please add the output of flutter doctor -v.

Which versions of grpc and cloud_firestore were added to pubspec.lock?

I actually leave them blank so the most up to date version. No issues with flutter doctor at all.

the most up to date version

That doesn't guarantee that you get the latest version.
If for example grpc and cloud_firestore depend on the same transitive dependency B and latest grpc doesn't yet support the latest version of B, flutter packages get might end up fetching an older version of cloud_firestore that is compatible with the version of B grpc is compatible with.
So it's usually a good idea to occassionally check if pubspec.lock actually contains the version you expect. Especially if something is behaving weirdly.

Is there a way to find the recommended versions to use with Flutter so I can add them to my pubspec?

That depends on the Flutter version you are using and how up-to-date your other dependencies are that you are using. Usually the latest available version is a good bet unless it introduced a bug and you want revert to a previous version.
I'm just saying that you should check if you actually got what you expected.
When #12627 is fixed this will make this information available immediately when flutter packages get is run.

No issues with flutter doctor at all.

that's the least relevant output of flutter doctor -v ;-) Please post the rest.

[✓] Flutter (Channel beta, v1.0.0, on Mac OS X 10.14.3 18D21c, locale en-US)
    • Flutter version 1.0.0 at /Users/josephdunivan/library/development/flutter
    • Framework revision 5391447fae (3 weeks ago), 2018-11-29 19:41:26 -0800
    • Engine revision 7375a0f414
    • Dart version 2.1.0 (build 2.1.0-dev.9.4 f9ebf21297)

[✓] Android toolchain - develop for Android devices (Android SDK 28.0.3)
    • Android SDK at /Users/josephdunivan/Library/Android/sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-28, build-tools 28.0.3
    • ANDROID_HOME = /Users/josephdunivan/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1136-b06)
    • All Android licenses accepted.

[✓] iOS toolchain - develop for iOS devices (Xcode 10.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 10.1, Build version 10B61
    • ios-deploy 1.9.4
    • CocoaPods version 1.5.3

[✓] Android Studio (version 3.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 31.1.1
    • Dart plugin version 181.5656
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1136-b06)

[✓] VS Code (version 1.25.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 2.21.1

To clarify, before adding any Firestore dependency I experienced normal build times under a min. After adding any Firestore dependecy I experienced build times that could take up to 200 seconds or longer, almost 4 min long(that's ridiculous). When I went into xcode during the build process I noticed that gRCP now wanted to download over 300 files, which was not the case before adding Firestore.

Do all versions of cloud_firestore cause this issue? On Bitrise CI after adding cloud_firestore dependency the build time increased from few minutes to about 20 minutes. More about similar issue here #27771

Adding gRCP as a dependency reduced build time for 200s to 57s.
Has anyone else had this issue?

Can you link to said gRCP dependency? I can't find it

This is being tracked in FirebaseExtended.  See https://github.com/FirebaseExtended/flutterfire/issues/349.

a working solution with more detail at https://github.com/flutter/flutter/issues/37582#issuecomment-667997839

I have this issue. The build time is over 20 minutes after adding Firebase packages to pubspec.yaml. This is with adding a couple lines of code to the default app:
import 'package:flutter/material.dart';
import 'package:firebase_core/firebase_core.dart';
import 'package:firebase_analytics/firebase_analytics.dart';

Future main() async {
  WidgetsFlutterBinding.ensureInitialized();

  await Firebase.initializeApp();

  runApp(MyApp());
}

These are the only changed lines of code. Somehow the build time takes 15-20 minutes without exception. What's going on?

authors: JosephDunivan fresswolf jimmont jmagman orestesgaolin peternielsen112 zoechi

repo: flutter/flutter


text:
Make "pub get" retry when it fails

Thanks for explaining 69.
LGTM!

authors: HansMuller Hixie


text:
Clean up test infrastructure

Most of the failures seem to be because I messed up the RAM requirements. I'm going to minimize the RAM requirements on all the shards and then increase them until each shard works.

Aah! I was wondering if that would break things. It seemed like it should work, based on the YAML spec and the lack of error message about parse errors. :-)
I'll fix it.

cc @jonahwilliams @dnfield @godofredoc @tvolkert @fkorotkov @yjbanov
Please review this carefully (probably by looking at the actual files in the case of .cirrus.yml and test.dart, since I basically entirely replaced them). I tried to self-review many times, and tried to leave comments where some of my decisions may be surprising, but it's likely I missed things, since each day that I worked on this for the past 2+ months I had to do really sketchy merges.
@fkorotkov you may have interesting input on the .cirrus.yml file in particular
@yjbanov you may be interested in my rewrite of your web test sharding logic
@dnfield @godofredoc you may be interested in my resharding in the context of the LUCI migration


text:
foam.flow should be upgraded to foam.u2

Or at the very least FOAMByExample/FOAMBook need to be upgraded

I agree.  I would like to write some U2 docs in U2 FLOW.

authors: adamvy kgrgreer

repo: foam-framework/foam


text:
Make pogs part of namespace and then change name.

Done.

repo: foges/pogs


text:
Add fn to get ciphersuite of a session

bors r+

authors: jack-fortanix jethrogb

repo: fortanix/rust-mbedtls


text:
Add support for skipped Test Results

ci-retest

ci-retest

repo: foundriesio/jobserv


text:
Add jose-jwt library to build-constraints.yml

Thanks!

I just ran the test suite. The doctests are a bit fragile due to their reliance on the package database having unique module names, e.g.:
[1 of 1] Compiling Main             ( doctests.hs, dist/dist-sandbox-ff51c802/build/doctests/doctests-tmp/Main.o )
Linking dist/dist-sandbox-ff51c802/build/doctests/doctests ...
Running 2 test suites...
Test suite tests: RUNNING...
Test suite tests: PASS
Test suite logged to: dist/dist-sandbox-ff51c802/test/jose-jwt-0.4.1-tests.log
Test suite doctests: RUNNING...

Jose/Jwt.hs:30:8:
    Could not find module ‘Control.Error’
    Perhaps you meant Control.Arrow (from base)
    Use -v to see a list of the files searched for.

Jose/Jwt.hs:34:8:
    Ambiguous module name ‘Crypto.Random’:
      it was found in multiple packages:
      crypto-random-0.0.8 crypto-api-0.13.2
Test suite doctests: FAIL
Test suite logged to:
dist/dist-sandbox-ff51c802/test/jose-jwt-0.4.1-doctests.log
1 of 2 test suites (1 of 2 test cases) passed.
cabal: Error: some packages failed to install:
jose-jwt-0.4.1 failed during the tests phase. The exception was:

I usually deal with this in Stackage by either disabling the tests for the library entirely, or by passing in a flag to disable doctests. I'll start off with the former, if you have a flag for the latter, please let me know.

Thanks, I don't at the moment, but I'll add one now.

Ok, I created a new release with a doctest flag:
http://hackage.haskell.org/package/jose-jwt-0.4.1.1
Thanks for the suggestion!

Thanks for implementing it so quickly! I've updated the build-constraints file to match.

authors: snoyberg tekul

repo: fpco/stackage


text:
vite.config properties root and build.outDir is ignored

@juvirez Thanks!
I've moved some of the config logic to Vitedge here so we can do some more cleanup.

authors: frandiox juvirez

repo: frandiox/vite-ssr


text:
Updated example to use es6 imports

Hi, we're currently fixing up the namespace.

wow, that is pretty nice for me

Hi @amilajack! Thanks for contributing!
We've updated the docs post including both cjs and es build in /dist (Took us a while, unfortunately). So this PR was actually ahead of its time :)
Even so, do keep following for when we update and contribution guidelines for stuff to come. Cheers!

authors: Leakermeli achillesrasquinha amilajack pratu16x7

repo: frappe/charts


text:
fix soap encoding error

This has worked before. Has the standard always been soap-env etc? Is this a change that is compatible with most legacy devices? Has it been tested against easycwmp just for a start?

The build error is Just a docker secret issue.

This has worked before. Has the standard always been soap-env etc? Is this a change that is compatible with most legacy devices? Has it been tested against easycwmp just for a start?

I remember the initial code does not work with my Huawei 820 cpe device which does not accept the soap envelop so I did some fix.  for which xmlns attribute I think it is not important but we should keep it consistent everyone in the code. And the effective fix I think is this one:

` tr069/src/main/java/com/github/freeacs/tr069/xml/Response.java

sb.append("\n");
`

@yrong sorry about not answering for ages. About the encoding part, does it work for more than one type of device? I am not interested in merging this if I am not confident that most devices will handle this change. yes its a simple namespace change, but yey there are lot of badly implemented tr069 clients out there ...

I don't have to time to evaluate this for now. I can't just merge it without knowing why I should

authors: jarlah yrong

repo: freeacs/freeacs


text:
Fixes merge of plugin options

@cascornelissen I didn’t even notice “extend” was a dependency, haha

Thanks! I'll test, merge and release it tomorrow!

authors: carlcs cascornelissen

repo: freshheads/svg-spritemap-webpack-plugin


text:
Added Tower of Hanoi implementation in Python (issue #304)

please read the rules

authors: iam-abbas rdestrada


text:
Added finalizers check in query mode.

looks good, thanks

authors: davidor ftomassetti

repo: ftomassetti/effectivejava


text:
Implemented and tested the AsyncMethodsCheck

see some comments above

@JimmyRojas Fixed the observations, ready to merge

Looks good to me, thank you

@ziur if you don't have some observation, we can merge the changes

repo: fundacionjala/enforce-sonarqube-plugin


text:
libs.encoding_utils: improve clean_base64

Codecov Report

Merging #45 into develop will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff            @@
##           develop      #45   +/-   ##
========================================
  Coverage    71.41%   71.41%
========================================
  Files           16       16
  Lines          780      780
  Branches       106      106
========================================
  Hits           557      557
  Misses         177      177
  Partials        46       46

Impacted Files
Coverage Δ

cl_sii/libs/encoding_utils.py
78.26% <100%> (ø)
: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 1bf6fa0...468f7a9. Read the comment docs.

authors: codecov-io glarrain

repo: fyndata/lib-cl-sii-python


text:
Add spark operation examples to the Wiki

Merged into develop.

repo: gchq/Gaffer


text:
404Pageを追加

👍

close #9

close #6

#6

repo: gedorinku/koneko-client


text:
Issue 15249

Here is the problem:
https://github.com/geneontology/go-ontology/pull/15370/files#diff-3cf7b2b4f9db4df4cae7c72388fdba78R112728

Unfortunately you have gone beyond the event horizon here, you've saved a file that is impossible to read again, the only fix is to hand edit file (or redo the work).
This is currently an undesirable feature of the owlapi obo parser/writer (which I wrote). What is written should always be readable, but sadly there are edge cases like this. It's hard to get around this as it arises from a property of .obo.
As a workaround, we'd like better lint checks within Protege, as it's easy to accidentally slip and use the wrong AP for something. I think it should be straightforward to write a robot plugin for Protege: https://github.com/ontodev/robot/issues/265

How dd you find that ? I think I made that mistake in the past, I don't remember it being so lethal.

When I ran locally I scanned the ginormous OWLAPI error log. the trick is to look for the OBOFormat section (I have done this many times for many different projects... paying the sins for .obo format). It gives the line number. I think it should have been there in the travis log too

authors: cmungall pgaudet

repo: geneontology/go-ontology


text:
Default demo image

The program will now display one of five default images when opened:

https://imgur.com/gallery/svViHqm
https://imgur.com/gallery/BfktK
https://imgur.com/gallery/u5OUfBF
https://imgur.com/gallery/PT3Nh7B
https://imgur.com/gallery/EtXIdFP


text:
bug fixed in leaky_relu

adding negative tests for all tests

@geohot added neg tests

more clean now
@geohot @marcelbischoff what you say.

@geohot @marcelbischoff
tests failing for negative values
conv
test_div (because of np.log(x))

conv and tensordot needs lot of tolerance  around 1-e3

@geohot PR completed
Added leaky_relu fix
Added Negative Testing
Adde neg support.

closing this will add 'neg' support in future pr.

repo: geohot/tinygrad


text:
Assets docs are missing

there is an incomplete version

done

repo: geomorillo/prestomvc


text:
Check that Connect has been Initialized when calling Connect methods

Addresses #6

:+1:

authors: chadedrupt mattfurness

repo: getconnect/connect-net-sdk


text:
Remove shouldproxy param

This may cause the same issue as https://github.com/getlantern/lantern-internal/issues/761 @atavism @myleshorton

authors: atavism joesis

repo: getlantern/autoupdate


text:
gfx_window_sdl is not available on crates.io

It's published now! https://crates.io/crates/gfx_window_sdl

thanks @csherratt !

authors: csherratt kvark murarth

repo: gfx-rs/gfx


text:
Make integration tests work on Windows

Epic :clap:

This is really close. The env tests fail because something is converting the \t in the paths (such as my home dir: c:\Users\techn...) to tabs.

Sorry if you prefer the mystery approach to debugging, but here are some spoilers:
test: env with bare repo ...                                       FAILED
-- stdout --
    Initialized empty Git repository in C:/Users/techn/git-lfs-tests/test-env.sh-5768/env-with-bare-repo/
    ASSERT
    --- a.txt   2015-10-22 18:36:51.247455400 -0700
    +++ b.txt   2015-10-22 18:36:51.346460800 -0700
    @@ -3,7 +3,7 @@
     git config filter.lfs.clean = "git-lfs clean %f"
     git config filter.lfs.smudge = "git-lfs smudge %f"
     git version 2.5.1.windows.1
    -GIT_LFS_TEST_DIR=/c/Users/techn/git-lfs-tests
    +GIT_LFS_TEST_DIR=C:/Users/techn/git-lfs-tests
     git-lfs/1.0.0 (GitHub; windows amd64; go 1.5.1; git 6fad0c7)
     LocalGitDir=C:\Users\techn\git-lfs-tests\test-env.sh-5768\env-with-bare-repo
     LocalGitStorageDir=C:\Users\techn\git-lfs-tests\test-env.sh-5768\env-with-bare-repo

I changed contains_same_elements to use diff, making the results readable in stdout. Turns out the problem comes from my own GIT_LFS_TEST_DIR setting of /c/Users/techn/git-lfs-tests. However, it doesn't seem to work in Git Bash if I use c:\Users\techn\git-lfs-tests instead.
test/testlib.sh: line 78: c:Userstechngit-lfs-tests/test-env.sh-3432/out: No such file or directory

BTW I made my commits in a dummy branch. I like the idea of using diff for these long string comparisons. It also fixes the \t issue by using a '%s' arg for printf.
https://github.com/github/git-lfs/commits/sinbad-windows-tests

One thought I had is that Git LFS could detect whether it's in a git bash env. If $SHELL is set (especially to something like /Git/usr/bin/*sh\z), then I think we can assume it's Git Bash. That env is empty in Git cmd. Then, the paths could all be be output as the unixified paths, like /c/Users/....
That sounds like kind of a headache. The test suite would have to know about all this too.
Since a lot of this seems to be coming from git lfs env, we could even add some flags to disable ENV output or something.

This is really close. The env tests fail because something is converting the \t in the paths

Aaargh, I fixed loads of those cases but obviously missed a couple because they didn't blow up for me, nice catch with the '%s'.

I like the idea of using diff for these long string comparisons.

Why didn't I think of that instead of copy/pasting the terminal output and staring at it? Nice.
I'll have a poke around with GIT_LFS_TEST_DIR today.

I've ported the diff approach from your branch, I just redirected the output to stderr so it was easier to find in the output (next to the test failure).
The workaround for GIT_LFS_TEST_DIR  is to use the form:
 export GIT_LFS_TEST_DIR=C:\\Users\\steve\\temp\\git-lfs-test-dir

That works fine in Git Bash for me and the tests pass now with GIT_LFS_TEST_DIR  (with one other small fix which is on this branch)
As for detecting bash & using different paths, I think it's simplest to just make the rule that git-lfs always deals in native paths? All we then need to do is be aware that some data from either env vars or git may come in with bash-style paths and we may need to clean it up before using it internally, or passing it back to git (we had this in SourceTree a lot). Luckily filepath does a lot of the heavy lifting internally and any external scripts can use cygpath if they know that output from git-lfs will be native and they want Bash (although my experience is that most of the time Windows tools would rather have native paths).
I don't think it's too onerous to require that GIT_LFS_TEST_DIR is set to a native path (with escaping) just for the tests. I could write something which runs cygpath on every environment variable in the tests (it can be made safe for non-paths) but I think that's overkill.

some data from either env vars or git may come in with bash-style paths and we may need to clean it up before using it internally, or passing it back to git

So Git uses native (c:\ ?) paths? I'm fine with using native paths on windows.

I don't think it's too onerous to require that GIT_LFS_TEST_DIR is set to a native path

Agreed.

So Git uses native (c:\ ?) paths? I'm fine with using native paths on windows.

No, you get paths in bash-style but Go's filepath can cope with those as well as native paths. Also when you pass paths to git it can handle native paths if they're relative (but not absolute paths because it thinks they're outside the repo - must do a string test). So the main thing is to be careful about using string tests on paths unless you clean them first - this is why lfs.FilenamePassesIncludeExcludeFilter uses filepath.Clean before doing any comparisons.
It's just easier to use native paths inside Go generally since they'll all behave the same way, you just have to be aware of the potential for non-native paths coming in from Git, which will still work anyway inside Go but will throw off string comparisons.

Huzzah, works on my vm too :+1:

authors: sinbad technoweenie

repo: github/git-lfs


text:
Ignore PyCharm IDE project file

Thanks but see Global/JetBrains.gitignore for this

authors: arcresu tanmaythakur

repo: github/gitignore


text:
change doc

Pull Request Test Coverage Report for Build 1811

0 of 0   changed or added relevant lines in 0 files are covered.
14 unchanged lines in 6 files lost coverage.
Overall coverage decreased (-0.02%) to 61.53%

Files with Coverage Reduction
New Missed Lines
%

core/registry/healthz.go
1
43.84%

session/session_manager.go
2
64.93%

core/config/config.go
2
64.81%

core/loadbalancer/struct.go
3
0.0%

core/handler/transport_handler.go
3
19.28%

core/lager/logrotate.go
3
36.24%

Totals

Change from base Build 1810:
-0.02%

Covered Lines:
5326

Relevant Lines:
8656

💛  - Coveralls

authors: coveralls tianxiaoliang

repo: go-chassis/go-chassis


text:
add more UT and support request body limit

Can one of the admins verify this patch?

Coverage increased (+0.8%) to 67.834% when pulling 1d054eac617000879ac3be8bd5b405e25790496d on lizihuai:master into 273aa991d1ca3e471e8874040dfaa3fc958c84c3 on go-chassis:master.

authors: coveralls lizihuai tianxiaoliang


text:
Add cloudron as a deployment method

Thanks your PR!
But README already mentions the CloudRon and I think that's pretty much enough information for users to know, since install is as far as click a button.
Add additional page to documentation adds more cost to maintain and organize.

@Unknwon fair enough, it's your call :-) I thought it would be nice to have it listed in https://gogs.io/docs since that's the main gogs docs page. I see your concern though, so maybe a line in  https://gogs.io/docs/installation/install_from_packages ? I will close this PR and if you have an alternate suggestion I can provide another PR.

authors: Unknwon gramakri

repo: gogits/docs


text:
BERT论文里面有#task1和#task2，在训练模型时这两个task是先训练task1再训练task2，还是同时训练同时训练的话，怎么做的？训练语料是同一份吗？

用pytorch的transform吧，那个封装程度更好一些

用pytorch的transform吧，那个封装程度更好一些

不是使用，是论文里的内容理解，

一起训练 loss是两个taskloss的叠加 代码里model_fn_builder写的比较清楚

一起训练，是同一份语料，在构建训练数据时 create_pretraining_data.py，同时为两个任务构建训练数据。
instance的结构为:
[CLS] sentence1 [SEP] sentence2 [SEP]
sentence1 和sentence2 构成NSP 任务，sentence1，sentence2 句子内部词语MASK构成任务一Masked LM
instance 实例（实际构建instance时会句子有截断）：
[CLS] 魏 孝 静 帝 初 年 ， 李 神 俊 代 理 并 州 刺 史 ， 不 久 [MASK] [MASK] 骠 骑 大 将 军 、 肆 州 刺 史 ， 回 朝 担 任 [MASK] [MASK] 。 [SEP] 去 世 ， 虚 岁 六 十 四 ， 朝 廷 赠 予 都 督 雍 秦 泾 三 州 诸 军 事 、 [MASK] [MASK] 大 将 军 、 尚 书 左 仆 射 、 司 徒 公 、 雍 州 刺 史 ， 侍 中 、 [MASK] [MASK] 公 [MASK] 故 ， [MASK] 号 [MASK] [MASK] ， 兴 [MASK] 三 年 岁 次 辛 酉 十 二 月 廿 三 日 （ [MASK] [MASK] 年 1 月 24 日 ） [MASK] [MASK] 邺 城 [SEP]

authors: Brentbin mianzhiwj real-brilliant yuanjun1991

repo: google-research/bert


text:
Fix the bug of digital label error

@googlebot I signed it!

@googlebot I signed it!

repo: google-research/google-research


text:
Chrome 46 is going live

LGTM

authors: addyosmani jakearchibald

repo: google/WebFundamentals


text:
regen

Abandoned in favour of b69ba1387ce2108ac9bc8e8e5e5a46e7d5c72313

repo: google/go-genproto


text:
Turn vectorization on by default for DispatchWorkgroupsOp path

Shouldn't the commit message be "Turn vectorization on by default for Linalg on Tensor path"?

Either way works. I sometimes feel linalg on tensors path a bit not really capturing it: we still use bufferization even in the new path and previously we also have some ops going through linalg on tensors. OTOH, DIspatchWorkgroupsOp is entirely new. Naming is hard. :D

repo: google/iree


text:
Fix bad link and update graph text in hello_world_android.md.

Thanks for reporting this. Merged into our internal code base, and will be pushed to GitHub in the next release.

repo: google/mediapipe


text:
Misc alive patches

LGTM thanks

repo: google/souper


text:
Splitting ServiceApiSettings back out of ApiCallSettings

One small nit. Otherwise, LGTM.

@anthmgoogle , take a look.

LGTM.

authors: anthmgoogle garrettjonesgoogle pongad

repo: googleapis/gax-java


text:
Java: Updating import paths

LGTM

Codecov Report

Merging #1209 into master will not change coverage.
The diff coverage is 100%.

@@            Coverage Diff            @@
##             master    #1209   +/-   ##
=========================================
  Coverage     86.18%   86.18%
  Complexity     3765     3765
=========================================
  Files           380      380
  Lines         14929    14929
  Branches       1663     1663
=========================================
  Hits          12866    12866
  Misses         1615     1615
  Partials        448      448

Impacted Files
Coverage Δ
Complexity Δ

.../transformer/java/JavaGapicSurfaceTransformer.java
98.89% <100%> (ø)
43 <0> (ø)
:arrow_down:

...nsformer/java/JavaGapicSurfaceTestTransformer.java
100% <100%> (ø)
27 <0> (ø)
: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 d2c7619...ff3f615. Read the comment docs.

I missed some package moves so I have updated the PR.

LGTM

repo: googleapis/toolkit


text:
Python: Change samples to use dictionaries rather than instantiated protos.

Codecov Report

Merging #1445 into master will decrease coverage by 0.02%.
The diff coverage is 87.5%.

@@             Coverage Diff              @@
##             master    #1445      +/-   ##
============================================
- Coverage     87.16%   87.13%   -0.03%
- Complexity     4031     4037       +6
============================================
  Files           386      386
  Lines         16190    16204      +14
  Branches       1768     1773       +5
============================================
+ Hits          14112    14120       +8
- Misses         1619     1622       +3
- Partials        459      462       +3

Impacted Files
Coverage Δ
Complexity Δ

...n/transformer/py/PythonModelTypeNameConverter.java
89% <100%> (ø)
29 <0> (ø)
:arrow_down:

...transformer/py/PythonImportSectionTransformer.java
97.79% <100%> (-0.07%)
35 <0> (-2)

...nsformer/py/PythonGapicSurfaceTestTransformer.java
100% <100%> (ø)
19 <0> (+1)
:arrow_up:

...api/codegen/transformer/py/PythonSurfaceNamer.java
92.72% <100%> (+0.34%)
52 <3> (+3)
:arrow_up:

...ransformer/py/PythonApiMethodParamTransformer.java
91.76% <45.45%> (-6.89%)
28 <0> (+4)

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 6952453...5559c9b. Read the comment docs.

authors: codecov-io landrito


text:
Formatted string

I signed it!

repo: googlesamples/android-vision


text:
fix a missing verbose condition for logging

LGTM

Can this be merged?

authors: erlichmen gcochard

repo: gotwarlost/mock-spawn


text:
Debug clock skew on macos workers

[trickle] No significant performance differences

[microbenchmarks] No significant performance differences

Needed to debug kokoro macos flakes, merging.

authors: grpc-testing jtattermusch

repo: grpc/grpc


text:
PHP: attempt to add v1.21.3 to interop matrix

https://github.com/grpc/grpc/pull/19230 would need to be backported to make this PR work.

@jtattermusch This is ready for another review. The Adhoc interop matrix job was succesful - I manually triggered another build, PTAL. Thanks.

authors: jtattermusch stanley-cheung


text:
Win32 python

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Okay; if we're really going to be sticklers about the binaries for the sake of size, I'll set up a pull from GCS.

@soltanmm: no, that's missing the point. We're trying to build artifacts. That work you've done isn't going to be used by the end users (hopefully). We'll build the dll artifacts on Jenkins, move them over to the proper location, then build the python eggs.
Always having up-to-date synchronised code across everything is the essential point of having everything within a single repository. We know the DLL will be of the proper revision because we'll build it on the same revision as the rest of the python code.

@nicolasnoble Ah, so, I was under the impression that we'd have things set up such that any user could build a package for Windows if he/she wanted to. That thus seems to either be something we'll sort out later or are scrapping?

authors: grpc-jenkins nicolasnoble soltanmm


text:
Reimplemented ZUC in ANSI C99.

我在周日或者周一会有一次更新，@edsgerlin 你能否在下周二重新提交一次？便于我把你的代码并入主分支。多谢！

OK，那我先close。

authors: edsgerlin guanzhi

repo: guanzhi/GmSSL


text:
Cluster Mines and Ion Bombs missing "device" object

Closed as resolved

authors: XPav danrs


text:
Corrected top nav link to Svelte site

Excellent, thanks 👍

authors: bohtho halfnelson

repo: halfnelson/svelte-native


text:
have blocks for faye callback?

going to close this since we already have a callback based api.

authors: hamin shams-ahmed

repo: hamin/FayeSwift


text:
v0.1.0 Release

In order to approve this PR sign the head of this branch with:
git signatures add

My yubikey broke off in the USB port of my laptop this morning. Ordered a new one today. (what i'm saying is that i won't be able to sign anything for a bit)

@RyanSquared @benharri final approvals and I merge this and start using it on real projects?
@KellerFuchs looking at it would be ideal but she has not had many cycles recently, but she can always make a PR if we really screwed anything up here.

authors: benharri lrvick

repo: hashbang/git-signatures


text:
Fix tiny typo in docs

Thank you @vrom911!

repo: haskell/haddock


text:
Add gem.js to the repository

I'd like to also upload it to a javascript package manager or two.

repo: helephant/Gem


text:
HARP-16283 build-* now builds with the production flag set

No longer needed. The production builds, at least in GitHub are handled within the ci.yaml file.


text:
Bump up gRPC version

@fedefernandez Anyway we are on 1.16.x, which will cause also an eviction with the scalapb version. I've asked for a new release in gitter though: https://gitter.im/trueaccord/ScalaPB?at=5c137b41ca317a0e25c59b4e

repo: higherkindness/mu


text:
Use pipenv in docker container

I merged this PR.
However, I often encounter the problem of pipenv's slowness.
I'll rethink whether it should use pipenv or not.

repo: himkt/pyner


text:
tmux: string-split -> split-string

Good catch. Thanks!

authors: MaskRay hlissner

repo: hlissner/doom-emacs


text:
substitution bugfix and test

@whoburg ready for review

repo: hoburg/gpkit


text:
YARN-1947

Can one of the admins verify this patch?

authors: HopsJenkins giannokostas

repo: hopshadoop/hops


text:
Implement Trust Authorization and Update PermissionVerifier to check "network"

Hi @agunde406 Could you give a high level overview of what is happening in each of the handlers and why?

Descriptions added to Authorization handlers

repo: hyperledger/sawtooth-core


text:
Port this to Node and publish on npm

https://devcenter.heroku.com/articles/developing-toolbelt-plug-ins

authors: sindrenm timkurvers

repo: hyperoslo/heroku-pages


text:
Updated the base-microservice-example to version 2.3.0-alpha

@daniel-warren - I have committed your changes


text:
split build-lib rule to statuc and dynamic lib

done

repo: igagis/prorab


text:
ISPN-5736 Hot Rod protocol 1.4 does not exist

Integrated, thanks Galder!

authors: danberindei galderz

repo: infinispan/infinispan


text:
Pattern synonyms for degenerate HFC instances

Replaced by https://github.com/input-output-hk/ouroboros-network/pull/2412 due to GitHub problems.

repo: input-output-hk/ouroboros-network


text:
user endpoints added, teamMember table name updated and seeded users …

 BE - Setup route / endpoint to pull user data

 BE - Setup initial Route directory

 BE - Setup initial knex db, migrations, seeds

authors: ChelseaTolnai adrianadames erinc35


text:
New req changes

@ptrthomas for now you can close the pull request for now, may be i can resubmit later after incorporating all the comments.

repo: intuit/karate


text:
Stop using git-flow

I deleted develop branch from this repo and of course from my local. Don't forget to do the same @rsjavier14
Good bye git-flow.

repo: iporaitech/react-to-mdl


text:
use one portManager for all agents

@irisdingbj: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-local-tests.sh
eb8e691d5e2ce50d1b3c86aaa793e6a52a0f8e6e
link
/test istio-integ-local-tests

prow/istio-unit-tests.sh
eb8e691d5e2ce50d1b3c86aaa793e6a52a0f8e6e
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@irisdingbj: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
eb8e691d5e2ce50d1b3c86aaa793e6a52a0f8e6e
link
/test istio-unit-tests

prow/istio-integ-local-tests.sh
f675b3abb46072513784d2fce7cf912aae77f761
link
/test istio-integ-local-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@irisdingbj: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-local-tests.sh
f675b3abb46072513784d2fce7cf912aae77f761
link
/test istio-integ-local-tests

prow/istio-unit-tests.sh
f675b3abb46072513784d2fce7cf912aae77f761
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

https://github.com/istio/istio/issues/10023

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: irisdingbj, nmittler
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pkg/OWNERS [nmittler]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

authors: irisdingbj istio-testing

repo: istio/istio


text:
Fix a DestinationRule for the bookinfo egress test

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: douglas-reid
Assign the PR to them by writing /assign @douglas-reid in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@vadimeisenbergibm: PR needs rebase.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@vadimeisenbergibm: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
c265555d0892743122027d5226f55525220b05c4
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@vadimeisenbergibm: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
c265555d0892743122027d5226f55525220b05c4
link
/test istio-unit-tests

prow/istio-presubmit.sh
c265555d0892743122027d5226f55525220b05c4
link
/test istio-presubmit

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test istio-presubmit

/test istio-unit-tests

@vadimeisenbergibm: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
c265555d0892743122027d5226f55525220b05c4
link
/test istio-pilot-e2e-envoyv2-v1alpha3

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@vadimeisenbergibm: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
c265555d0892743122027d5226f55525220b05c4
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/e2e-bookInfoTests-v1alpha3.sh
c265555d0892743122027d5226f55525220b05c4
link
/test e2e-bookInfo-envoyv2-v1alpha3

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@vadimeisenbergibm: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
c265555d0892743122027d5226f55525220b05c4
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/e2e-bookInfoTests-v1alpha3.sh
c265555d0892743122027d5226f55525220b05c4
link
/test e2e-bookInfo-envoyv2-v1alpha3

prow/istio-pilot-e2e.sh
c265555d0892743122027d5226f55525220b05c4
link
/test istio-pilot-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

@vadimeisenbergibm: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-bookInfoTests-v1alpha3.sh
c265555d0892743122027d5226f55525220b05c4
link
/test e2e-bookInfo-envoyv2-v1alpha3

prow/istio-pilot-e2e.sh
c265555d0892743122027d5226f55525220b05c4
link
/test istio-pilot-e2e

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
c265555d0892743122027d5226f55525220b05c4
link
/test istio-pilot-e2e-envoyv2-v1alpha3

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@vadimeisenbergibm: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e.sh
c265555d0892743122027d5226f55525220b05c4
link
/test istio-pilot-e2e

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
c265555d0892743122027d5226f55525220b05c4
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/e2e-bookInfoTests-v1alpha3.sh
c265555d0892743122027d5226f55525220b05c4
link
/test e2e-bookInfo-envoyv2-v1alpha3

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@vadimeisenbergibm: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
c265555d0892743122027d5226f55525220b05c4
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/e2e-bookInfoTests-v1alpha3.sh
c265555d0892743122027d5226f55525220b05c4
link
/test e2e-bookInfo-envoyv2-v1alpha3

prow/istio-pilot-e2e.sh
c265555d0892743122027d5226f55525220b05c4
link
/test istio-pilot-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/lgtm

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: rshriram
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

samples/OWNERS [rshriram]
tests/OWNERS [rshriram]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/test istio-pilot-e2e-envoyv2-v1alpha3

/test istio-pilot-e2e-envoyv2-v1alpha3

Removing the no-cla label, both of the commits are mine, including the one that is cherry-picked.

@vadimeisenbergibm: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-bookInfoTests-v1alpha3.sh
c265555d0892743122027d5226f55525220b05c4
link
/test e2e-bookInfo-envoyv2-v1alpha3

prow/istio-pilot-e2e.sh
c265555d0892743122027d5226f55525220b05c4
link
/test istio-pilot-e2e

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
c265555d0892743122027d5226f55525220b05c4
link
/test istio-pilot-e2e-envoyv2-v1alpha3

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


text:
Sonar view does not work when activating a fresh sonar

see #44

repo: iti-luebeck/MARS


text:
Add a function somewhere to force-focus an element

  local finder = GUI.findElementByName("finder")
  window.state.focusedElm = finder

  -- Start the main loop
  GUI.Main()

this snippet worked for me if anyone stumbles on this. it autofocuses an element on window start.

Update in case I ever get back to this:
Right now there's no way for an event handler to set the focus, only to lose it. This might just be a matter of:
  -- window.lua @ window:updateInputEvents
  if state.shouldLoseFocus then elementToLoseFocus = state.shouldLoseFocus end
+ if state.elementToFocus then elementToFocus = state.elementToFocus end

authors: gwatcha jalovatt

repo: jalovatt/scythe


text:
Update tslint-config-airbnb to the latest version 🚀

Coverage remained the same at 70.0% when pulling 5d0e726944230590a81eb48e6b6763d5342aaf03 on greenkeeper/tslint-config-airbnb-5.4.2 into 004a8cfbf996b47f6f792241ab8c305ad50c86a8 on master.

authors: coveralls jameswlane

repo: jameswlane/status-board-cli


text:
Fix translating a translated polyline

Thanks!

authors: bugadani jamwaffles

repo: jamwaffles/embedded-graphics


text:
appveyor failure

Testing. Homu shouldn't merge this
@homunkulus r+

:pushpin: Commit 1a6eab8 has been approved by japaric

:hourglass: Testing commit 1a6eab8 with merge 1a6eab8...

:broken_heart: Test failed - status-appveyor

@homunkulus good bot!

authors: homunkulus japaric

repo: japaric/rustc-builtins


text:
MatchFunction tests

Current coverage is 96.30%

Merging #1063 into master will increase coverage by +0.26% as of a291659

@@            master   #1063   diff @@
======================================
  Files           75      75
  Stmts         9402    9402
  Branches      1733    1733
  Methods          0       0
======================================
+ Hit           9030    9055    +25
  Partial        101     101
+ Missed         271     246    -25

Review entire Coverage Diff as of a291659

Powered by Codecov. Updated on successful CI builds.

Thank you!

authors: codecov-io danieldietrich ruslansennov

repo: javaslang/javaslang


text:
Packaging of the code for easy distribution.

@nirav1997 I assigned you this issue since I think you have already started working on this

Almost there


text:
Added Icelandic localisation for user-facing text

@barnabywalters thanks!

authors: barnabywalters manelclos

repo: jbittel/django-mama-cas


text:
Implement antd

tarea recibida empezare con la implementacion


text:
fix: update broken links

Hi @hervelemeur. Thanks for your PR.
I'm waiting for a jenkins-x or todo member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign jstrachan
You can assign the PR to them by writing /assign @jstrachan in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/ok-to-test

/approve

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: jstrachan
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [jstrachan]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

repo: jenkins-x/octant-jx


text:
Type checking for all methods -> migrate to TypeScript

Keep going with JS for now

repo: jesusprubio/shodan-client


text:
Make it possible to not include ssl_ciphers in the resulting vhost.

Closing in favor of #794

repo: jfryman/puppet-nginx


text:
Remove SSLv3 as it is insecure

Found some references to SSLv3 that were statically defined and not removed with the previous efforts to disable SSLv3

:heart:s and :lock:!

authors: ghoneycutt jfryman


text:
Fix compiler warning

Thanks!

authors: jlawton wlisac

repo: jlawton/UIColor-HTMLColors


text:
Update CHANGELOG.md

bot, retest this please

repo: jlebon/docker


text:
Fix https download: add 'use_ssl' for actual download

@jlindsey Hey, can you please merge this?

@sh0oki Sorry about the long wait, this project was made for a company I'm no longer with and I haven't looked at it in a while. Your PR email must have got lost in the shuffle!

repo: jlindsey/ApkDownloader


text:
Small typo in README Nginx section

Thank you (sorry didn't say so sooner)

No problem. Thanks for making django-sendfile ;)

authors: davegaeddert johnsensible

repo: johnsensible/django-sendfile


text:
Add integrations for Hound CI and Travis CI

@jonas-p I'd like to enable Travis and Hound but according to their websites I am still lacking the admin permissions for this repo.

repo: jonas-p/go-shp


text:
Restructure

Do we want to restructure the pipeline folder in this PR as well?

Suggestions?

Pull Request Test Coverage Report for Build 466

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.2%) to 97.645%

Totals

Change from base Build 436:
0.2%

Covered Lines:
622

Relevant Lines:
637

💛  - Coveralls

authors: coveralls joskuijpers wzorgdrager


text:
Make modal with pdf text for each requirement

ya esta con dummy links, toca decidir a donde queremos que lleven, se pueden ver en https://github.com/jp-hidalgo/vehicle-profile-editor/blob/master/src/App.js desde la linea 137 (los route que tienen url)

Ready the chicken pero creo que faltan algunos más? Puedo meter URL para cada "here" que ponga, solo es que me diga.

ya tienen los HERE que estan en el translation.json, no se donde mas faltan

Solo faltaba en "riskReq":  y lo puse (creo jajaj sigo aprendiendo).

authors: carlosfpardo jp-hidalgo

repo: jp-hidalgo/vehicle-profile-editor


text:
There are .source.js files left in the fixture and some of them don't pass

Ah, I was wrong...

repo: jquery/esprima


text:
Pass flags to daemon

How did I not even notice that? Shh... that never happened. :D

LGTM :+1:

repo: jwrb/paycoin-electron


text:
Implement Collapse and Accordion

done

repo: kattunga/IWBootstrapFramework


text:
when adding a new item image it doesnt fill the whole circle!...how can i fix that?

What do you mean?
Do you have any screenshot?

I have a similar issue. See this:

Context: I used the same icons with the same method and it worked perfectly fine. However, I needed to make the buttons smaller since it wouldn't fit some device screens. So I made the following changes:
    if UIScreen.mainScreen().bounds.height < 600 {
        fab.itemSpace = 8
        fab.itemSize = 36
        fab.size = 40
    }

However, changing itemSize parameter break the image centering. How do I make sure images stay centered when I change the itemSize?

Hello guys
This issue may be same as Issue 44.
Check it out please. 😄

Tried the solution there but didn't worked for me. When I change the itemSize, it does not calculate the center correctly.

It really my mistake. I fixed it.

Thanks, altough I can't seem to get the latest version from cocoapods. When I do pod search KCFloatingActionButton it shows the latest version is 1.3.4.

@yigitserin
$ pod search KCFloatingActionButton

-> KCFloatingActionButton (1.3.6)
   Floating Action Button for iOS
   pod 'KCFloatingActionButton', '~> 1.3.6'
   - Homepage: https://github.com/kciter/KCFloatingActionButton
   - Source:   https://github.com/kciter/KCFloatingActionButton.git
   - Versions: 1.3.6, 1.3.5, 1.3.4, 1.3.3, 1.3.2, 1.3.1, 1.3.0, 1.2.1, 1.2.0, 1.1.9, 1.1.8, 1.1.7, 1.1.6, 1.1.5, 1.1.4, 1.1.3, 1.1.2, 1.1.1, 1.1.0, 1.0.1, 1.0.0 [master repo]

https://github.com/CocoaPods/Specs/blob/master/Specs/KCFloatingActionButton/1.3.6/KCFloatingActionButton.podspec.json
🤔

This broke fab.buttonImage, now my icon appears out of the fab

Hello, @allaire
I've update to version 1.3.7.
Check it out please. 😄

It worked for me, thanks! Also had to update my pod repository for new versions to appear.

Good :)
This issue is closed.

Worked, thanks!

authors: AmrSayedAhmed allaire kciter yigitserin

repo: kciter/KCFloatingActionButton


text:
moves styles from kbc pt2

konfliktyu resolved, mergujem

authors: janmichek ujovlado

repo: keboola/indigo-ui


text:
[wip] move TablesListingTest to Common

one more thing coming here.

added a test that will fail on master but pass on https://github.com/keboola/connection/pull/695
apropos https://github.com/keboola/connection/issues/692

@pivnicek please check always that builds are passing. I am going to fix this code style issue now.

authors: Halama pivnicek

repo: keboola/storage-api-php-client


text:
独秀你坐下，让后边的大钊发言

方圆几公里的橘子树都给你, 成了吧

起来，我来扫扫地上的橘子皮

repo: kelseyhightower/nocode


text:
Export

Not sure why tabs and indents are not preserved in cellml export text??


text:
Fix typo in README

Thanks!

authors: iaguis krnowak

repo: kinvolk/kube-spawn


text:
Fix Buffer error "If encoding is specified then the first argument must be a string"

Fixed in https://github.com/kkaefer/utf7/pull/4

authors: Bolisov kkaefer


text:
Document gotcha for running mssql tests locally

@elhigu Have you ever encountered this issue yourself? Running the code on Windows is the only way how I can explain why it works in CI but fails locally, although I have no idea why it would make a difference for Docker environment.

Nope I have never seen that 🤔

@elhigu Does it work locally without a hitch for you? Which OS are you running?

I've been using mostly MacOS for knex work, but also I haven't ran mssql (or any other) knex tests for a long time.

authors: elhigu kibertoad

repo: knex/knex


text:
add getting long value

PR-#15

repo: koichi-murakami/kut


text:
preview page

とりあえずの機能を
commit: 1de9721
で追加しました．
ただもう少しUIなどを考慮する必要がありそうです．

repo: komagata/lokka


text:
adding file upload data to values in multipart body

@sisoje Shouldn't we create a PR to the official project so the community can benefit from these changes?

@sisoje Shouldn't we create a PR to the official project so the community can benefit from these changes?

i don't think this particular change is beneficial to the community. it looks more like a special case just for komed, and we might need to even extend it further, right @Gorshtak ?

@sisoje as I understood the PR it's to enhance the file upload method so not only a file can be uploaded but additionally other data can be sent with the request. Is that correct?
If that is so then I think the community can benefit greatly from the feature 🙂

@BorntraegerMarc i managed to add two more pr-s

authors: BorntraegerMarc sisoje

repo: komed-health/http


text:
播放局域网内rtsp视频流时，延时很大能否控制到1s内？

我在播放局域网内的rtsp视频流时，发现首屏时间达到了6s左右，而且视频延时也有4s左右，能否修改设置延时控制到1s内呢，如果可以请告诉我下怎么设置？


text:
Use helm 2.4.1 and set install timeout to 10m

LGTM

repo: kubernetes/charts


text:
wip - test for Update in kube client

hey @tojosephyang, is this PR intentional? Seems like this is @michelleN's work. Can you provide more context?

closing

authors: bacongobbler tojosephyang

repo: kubernetes/helm


text:
add China Unicom case study and update Case Study homepage with new f…

/approve

@zacharysarah @kbarnard10  this looks good to me, please review and merge when you have a chance, thank you!

/lgtm
/approve

repo: kubernetes/website


text:
Fix a typo

I signed the CLA

/retest

/check-cla

@mingrammer CLA 상태 확인 부탁드립니다!

@ClaudiaJKang 단계 하나를 빼먹었네요. 방금 완료 했습니다 :)

@mingrammer Thank you for the contribution !!
/lgtm
/approve

authors: ClaudiaJKang mingrammer seokho-son


text:
Agent implemantation

All components are implented and incorporated


text:
[ci skip] Fix indentention in README

thx!

repo: kyuden/banken


text:
Support Swagger YAML format.

No work required. Underlying library supports it. See scripted tests

repo: lagom/sbt-lagom-descriptor-generator


text:
Feature/public user profile

@stephanerangaya I did some basic styling (see attached). Let me know if you were thinking something different or want me to make some adjustments

LGTM! Thanks @kristysnet!

authors: kristysnet phil-monroe

repo: lale-help/lale-help


text:
how much speech data for training ?

If I recall, the VCC2016 dataset was not big, each person only has less than 100 sentences. So I expect for your own dataset, you would not require too much for training.

Thank you for your reply!


text:
是否可以支持修改表字段的类型，比如原来是interger 现在改为text

sqlite 本身不支持，但是 sqlite 本身的 column 是无类型的， 所以 你直接改下 model属性  就会自动转换的

sqlite 本身不支持，但是 sqlite 本身的 column 是无类型的， 所以 你直接改下 model属性  就会自动转换的

model 改了类型，但是sqllite并不会变的

model 改了类型，但是sqllite并不会变的


text:
Engine uses provided world; fixes issue #230

Cheers!

authors: iamjoshua liabru

repo: liabru/matter-js


text:
Support REST API with DocService

Duplicate with #1072.

repo: line/armeria


text:
Adds a better concat and better test cases

@sashajustice Let me know what you think of this replacement test

repo: lizzkats/npmModules


text:
Implement payment item view in payments.php

This view won't be implemented for now. Focusing on the multi-event aspects of the system.

repo: luizricardodelima/eve


text:
Can I use Graphql in polka?

With usage of external module like (graphql-js)[https://github.com/graphql/graphql-js] yes

Thank you sir

Hey, I don't actually use GraphQL but I added a quick little demo to show how you might approach.
There's likely better ways & better tools out there -- but it all comes down to how much abstraction you want / are comfortable with.
Hope it helps!

Wow Thank you sir

authors: ThiccCat lagmanzaza lukeed

repo: lukeed/polka


text:
Could you update GracePlot.jl to julia 1.x ?

It was working in September for Juila 1.0.  I will take a look at it - maybe the problem is with the newer versions of Julia.
Could you tell me which version you are using/which platform?
Thanks.

Update:
I just ran GracePlot with Julia 1.1 on Linux 18.04.1 LTS.
Everything looks fine with this configuration.
...I would still need feedback on the configuration that gives you trouble.

Just in case:
GracePlot.jl does not automatically install the Grace application itself (it assumes it is already accessible in the system paths simply from the command xmgrace).
For more information on how to install, see: https://github.com/ma-laforge/HowTo/blob/master/grace/grace_install.md#Installation
Also:
If you install xmgrace to a different path, you can set an environment variable in your ~/.julia/config/startup.jl file:
ENV["GRACEPLOT_COMMAND"] = "/home/laforge/bin/xmgrace2"

It works fine.
Thank you!

authors: brioglade ma-laforge


text:
changesetがない場合にdeployがエラーになる問題に対応

Codecov Report

Merging #20 (73281c6) into master (15964b5) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #20   +/-   ##
=======================================
  Coverage   98.28%   98.28%
=======================================
  Files           7        7
  Lines         175      175
  Branches       23       23
=======================================
  Hits          172      172
  Partials        3        3

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 15964b5...73281c6. Read the comment docs.

authors: codecov-commenter masaminh


text:
add fill method

Looks great! 👍
Thank you so much for taking the time to contribute.

authors: Xuefeng-Zhu mateogianolio

repo: mateogianolio/vectorious


text:
Fix type of variadic

Coverage remained the same at 58.318% when pulling 81b9db81264a832370667d9d1eff384530b2dfe6 on fix-variadic into 5994cc52dfa89a4ee21ac891b06fbc1ea02c52d3 on master.

authors: coveralls mattn


text:
add url pathmap functionality and path based routing rule

I suggest we create another function for PathBasedRule
Add-ArmApplicationGatewayPathBasedRequestRoutingRule

the split is done and tested on my local sub

repo: mbernard/PoshArm-Deployment


text:
please delete generator with key java_models_via_mobile_lambda

have released a new version of apidoc with the ability to delete generator services.
Goto:
http://www.apidoc.me/generators/servies/d1395bb5-ee13-42e5-8eaa-7afae9d264ef/show
As the author, you should see a link to delete
Once deleted, re-add the generator and it will resync immediately the generators provided by that service

Confirmed that UI change allowed successful delete

authors: mbryzek mchimirev

repo: mbryzek/apidoc


text:
Dev

LGTM!


text:
1.4.x

@estenberg is this branch open for check-in now?
@oleorhagen note that I had to cherry-pick the reboot hardening commit here to get the others to apply cleanly.  Is that documentation relevant for the 1.4.x branch?

No, it did not make the release

OK.  Thanks for confirming.  I've rebased this and dropped the hardening docs.

I think you can commit to this branch, yes.

authors: drewmoseley estenberg oleorhagen

repo: mendersoftware/mender-docs


text:
1.3.3 (#1)

ping

repo: mesonbuild/zstd


text:
backoff: avoid Timer.Reset() problems; honor minWait promise

Coverage increased (+0.3%) to 57.883% when pulling 119079d7b9868f08dd96a863718ce8ce8b85d0fa on jdef:notifier_fixes into 1558d48b04cb19deb7065191bc7966705ba918a2 on mesos:master.

Coverage increased (+0.3%) to 57.883% when pulling 119079d7b9868f08dd96a863718ce8ce8b85d0fa on jdef:notifier_fixes into 1558d48b04cb19deb7065191bc7966705ba918a2 on mesos:master.

authors: coveralls jdef

repo: mesos/mesos-go


text:
create git aliases

g-ct "message" (git commit -m)
g-add (git add --all)
g-cta "message" (git add --all && git commit -m)

repo: mhernandes/dotfiles


text:
Code/initial

Changes Unknown when pulling 68864d29211e8c99e9fe7b28263c99dde6e0297b on code/initial into ** on master**.

authors: coveralls philwinder

repo: microservices-demo/orders


text:
How to categorise  the workbooks programmatic

@praba984 need a lot more information about this issue.  fill in a description and more information and you can re-open the issue.


text:
fix Chinese typo

@laosb @stkevintan PTAL

repo: microsoft/TypeScript-Website


text:
Create apt sofacy.txt

Ok, I'll figure that out :-). I made some tests:
Folderpath search for --> C:\Windows\System32\conhost.exe
where FolderPath matches regex @'conhos.' -->match
where FolderPath matches regex @'conhos.' -->match
where FolderPath matches regex @'.*Windows.*conhost.exe' -->match
where FolderPath matches regex @'Windows.*conhost.exe' -->match
where FolderPath matches regex @'Wind.System32.' -->match
where FolderPath matches regex @'Wind.*System32' -->match

So as you suggested: I removed the .* at the end
I replaced . with . in rundll32.exe
I decided to just remove the \ from %APPDATA%


text:
sql assessment version 0.6.1

Gallery has been updated

authors: Charles-Gagnon vchvlad

repo: microsoft/azuredatastudio


text:
Port message reaction scenario

PR:
https://github.com/microsoft/botbuilder-python/pull/442
This isn't fully done since there's an issue with swapping reactions

https://stackoverflow.com/questions/47841985/python3-asyncio-call-from-flask-route

Look into Quartz

repo: microsoft/botbuilder-python


text:
Authndev

Coverage increased (+0.01%) to 51.235% when pulling ab82962bf2c00fc868c5d75ba82a6e79a2a9598b on authndev into cfc67e8409c782558dae07b6c94e036f7a693712 on pai-0.14.y.

Coverage increased (+0.01%) to 51.235% when pulling ab82962bf2c00fc868c5d75ba82a6e79a2a9598b on authndev into cfc67e8409c782558dae07b6c94e036f7a693712 on pai-0.14.y.

authors: coveralls ydye

repo: microsoft/pai


text:
New retrofit api services

Local Rebase conflict. Please do not merge

repo: microsoft/rural-crowdsourcing-toolkit


text:
Preparing for release

Codecov Report

Merging #7828 into release will decrease coverage by 58.78%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           release   #7828       +/-   ##
===========================================
- Coverage    58.78%      0%   -58.79%
===========================================
  Files          496      57      -439
  Lines        22126     509    -21617
  Branches      3560      60     -3500
===========================================
- Hits         13006       0    -13006
+ Misses        8308     509     -7799
+ Partials       812       0      -812

Impacted Files
Coverage Δ

src/client/common/installer/serviceRegistry.ts
0% <0%> (-100%)
:arrow_down:

...lient/datascience/jupyter/jupyterSelfCertsError.ts
0% <0%> (-100%)
:arrow_down:

src/client/debugger/extension/serviceRegistry.ts
0% <0%> (-100%)
:arrow_down:

src/client/common/serviceRegistry.ts
0% <0%> (-100%)
:arrow_down:

src/client/interpreter/locators/services/conda.ts
0% <0%> (-100%)
:arrow_down:

src/client/common/utils/regexp.ts
0% <0%> (-100%)
:arrow_down:

src/client/common/utils/icons.ts
0% <0%> (-100%)
:arrow_down:

src/client/debugger/constants.ts
0% <0%> (-100%)
:arrow_down:

src/client/datascience/jupyter/jupyterUtils.ts
0% <0%> (-100%)
:arrow_down:

src/client/interpreter/serviceRegistry.ts
0% <0%> (-100%)
:arrow_down:

... and 467 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 9af1d4f...24ba659. Read the comment docs.

authors: codecov-io karthiknadig

repo: microsoft/vscode-python


text:
Update the link to the Open Source forum

@AppInventorWorkerBee retest this please

@AppInventorWorkerBee retest this please

repo: mit-cml/appinventor-sources


text:
Do not attempt to send requests to mixpanel if adblocker is enabled

Have you tried it with AdBlocker on?

Yup

authors: patedit yinfeiru

repo: mixpanel/mixpanel-android


text:
Update README.md

I prefer to keep the basic example as simple as possible, see https://github.com/mll-lab/php-cs-fixer-config/commit/e83cdf0bceedd7838daa0c379ac5bb390d8b58e1

authors: SimBig spawnia

repo: mll-lab/php-cs-fixer-config


text:
changelog for 2.3.0

Manually rebased and merged in https://github.com/mllrsohn/gulp-protractor/commit/fc79938c1c5631eddcdcb65ca4975f528b45b0a8 . Thanks!

repo: mllrsohn/gulp-protractor


text:
Restructuring

I just made this to look at the diff

Outdated.


text:
Fix #119. update systems diagram

already got r+ at https://github.com/mozilla/watchdog-proxy/issues/119#issuecomment-411174631

repo: mozilla/watchdog-proxy


text:
Added Xanitizer to list of Java and Scala tools

Thank you @normanwenzel. 🎃

authors: mre normanwenzel

repo: mre/awesome-static-analysis


text:
Remove discontinued tool

👍 Great @viniciusbds.

authors: mre viniciusbds


text:
broken on rotation

Thanks, Please check the last commit


text:
Add support to arabic

What is the need of this change??
Adding support for Arabic is not really need for this.

Lots of Arabic apps need to display country name with Arabic.I will use my fork then.thanks for the library anyway.

@anasanasanas Could you please spend some time and provide Arabic localization in the following fork https://github.com/clockbyte/country-picker-android ? Please put it to the values-eg\strings.xml (refer values-ru\string.xml for instance). Thx in advance! Let's help this great lib to be translated in a correct way!! :traffic_light:

authors: anasanasanas kot331107 mukeshsolanki

repo: mukeshsolanki/country-picker-android


text:
3.8

3.8 branch is not supposed to be merged to 3.7, both must remain separate versions.

authors: ciscoiseguest pmantellini

repo: mulesoft/anypoint-examples


text:
[EP-2124] Fixed results rendering on homepage

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

authors: ancashoria dan-qc

repo: natgeosociety/marapp-frontend


text:
transactional update for arkivering

Tror ikke vi trenger dette allikevel nå som vi sletter i jobben om noe som er arkivert om den fortsatt skulle være arkivert.
Endret på main branchen at vi kun logger en warning om det skulle skje noe feil når vi sletter fra inFlight;
https://github.com/navikt/k9-vaktmester/blob/82b6ade2a35965a33092cd2d935eb971e757cfc3/src/main/kotlin/no/nav/k9/vaktmester/river/ArkivRiver.kt#L47

authors: andersnav fraadsbrandth


text:
Update README.md

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

authors: CLAassistant hmert


text:
support https

Importing Neo4j certificat to your JVM

Open your browser on this : https://localhost:7473
Export the certificat as a binary DER and save it on your filesystem (for example: /tmp/neo4j-cert.der)
Execute this command  (replace @JAVA_HOME@ with your JVM location) :

$> cd @JAVA_HOME@/jre/
$>  bin/keytool -import -alias neo4j -keystore @JAVA_HOME@/jre/lib/security/cacerts -file /tmp/neo4j-cert.der


text:
added luac maker for lua

I don't think both should be enabled by default, since luacheck seems to report the same fatal parse errors, doesn't it?
So I suggest to use only luacheck by default if it exists.
Do you agree?

@blueyed sounds like a plan

Thanks!

authors: blueyed sbdchd

repo: neomake/neomake


text:
fix mix archive.install link

Thanks for catching that! I had missed updating that link. There's a new and improved way of installing the archive now: mix archive.install hex nerves_bootstrap. If you'd like to have the honors, feel free to update this PR. If not, I'll update it later today. Thanks again for catching it.

Thank you for all your work on nerves. It is a really cool project.

Merged! Thanks for the update.

authors: Raphexion fhunleth


text:
Add specs for Surrealist::Utils

Coverage increased (+4.5%) to 99.756% when pulling 52beb9fe1b84424e7305e58421afdf0ca670ed7d on specs into ea91b1980158a3f157da908ff5ab4ca2b5dc7ee3 on master.

Coverage increased (+4.5%) to 99.756% when pulling 52beb9fe1b84424e7305e58421afdf0ca670ed7d on specs into ea91b1980158a3f157da908ff5ab4ca2b5dc7ee3 on master.

Coverage increased (+4.5%) to 99.756% when pulling 52beb9fe1b84424e7305e58421afdf0ca670ed7d on specs into ea91b1980158a3f157da908ff5ab4ca2b5dc7ee3 on master.

Coverage increased (+4.5%) to 99.756% when pulling 52beb9fe1b84424e7305e58421afdf0ca670ed7d on specs into ea91b1980158a3f157da908ff5ab4ca2b5dc7ee3 on master.

Coverage increased (+4.5%) to 99.756% when pulling 52beb9fe1b84424e7305e58421afdf0ca670ed7d on specs into ea91b1980158a3f157da908ff5ab4ca2b5dc7ee3 on master.

authors: coveralls nesaulov

repo: nesaulov/surrealist


text:
fix formatting of commit message

The build fails cause of wrong JAVA_HOME :(
Maybe its time to upgrade from Java 1.6?

When applying the PR locally (on OSX), I get extra quotation marks in my git log:

This is not desirable. What is the behavior on your Windows machine?

git log
commit 54a11fcee5ee866c7ee3ee2c0ead57f2356515c2
Author: Stefan Lehmann stefan.lehmann@unic.com
Date:   Wed Sep 20 15:49:29 2017 +0200
Release v0.0.3

commit c2bb4107923a572dcf34373544d0a1bce3cff923
Author: Stefan Lehmann stefan.lehmann@unic.com
Date:   Wed Sep 20 15:46:59 2017 +0200
Prepare next release v0.0.3-SNAPSHOT

commit 87d0338b9b3a646dc6a07ad24da883cc68cd703e
Author: Stefan Lehmann stefan.lehmann@unic.com
Date:   Wed Sep 20 15:46:59 2017 +0200
Release v0.0.2

commit 89ef44c0d373941d8bb446071efd2c815f9f8d49
Author: Stefan Lehmann stefan.lehmann@unic.com
Date:   Wed Sep 20 15:46:49 2017 +0200
test

commit 49304fd82afaf3dd7dc57dfa036e6061ed20eb92
Author: Stefan Lehmann stefan.lehmann@unic.com
Date:   Wed Sep 20 15:41:48 2017 +0200
Prepare next release v0.0.2-SNAPSHOT

Ok, i removed the quotation marks and it does not complain about it (but still the exit status is bad). Sorry about the trouble. I will close this PR and make a new one only containing the fix for the output logging.

authors: netzwerg stefanleh

repo: netzwerg/gradle-release-plugin


text:
Add modify channel information feature

Pull Request Test Coverage Report for Build 759499672

14 of 18 (77.78%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.2%) to 87.033%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

channels.go
7
9
77.78%

helix.go
7
9
77.78%

Totals

Change from base Build 754838219:
-0.2%

Covered Lines:
745

Relevant Lines:
856

💛  - Coveralls

authors: Scorfly coveralls


text:
Add more http status code

Awesome, thanks!

authors: impegoraro nathan-osman

repo: nitroshare/qhttpengine


text:
added tags

added tags

repo: njvack/piggybank


text:
Which version is this? I'm using c++14 and it's giving me error when I run engine.cpp

@xFighter1209  Hmm, I can't reproduce this. Interestingly, anything before C++14 throws an error for me, but C++14 and later always compiles. Could you summarise/show me the error message you're getting?

repo: nkarve/surge


text:
Update dependencies

Webpack build requires NPM package of ccv-js: https://github.com/forresto/ccv-js/issues/1

@automata @forresto ping on this one

@automata should be good to go now

repo: noflo/noflo-ccv


text:
Resizing logo to fit better and adding SVG for download use

Current coverage is 100% (diff: 100%)

Merging #94 into master will not change coverage

@@           master    #94   diff @@
====================================
  Files          63     63
  Lines        1844   1844
  Methods         0      0
  Messages        0      0
  Branches        0      0
====================================
  Hits         1844   1844
  Misses          0      0
  Partials        0      0

Powered by Codecov. Last update 28fe182...363e3b5

authors: AbsentSemicolon codecov-io

repo: not-nexus/shelf


text:
[T2A3][T09-A3]Chew Wan Theng Ruth

ready for review

that is one way to reducing SLAP on the code! i am sure there's more so do look around the other pull requests!

authors: chewwt mauris


text:
Encapsulated feedbackToUser in CommandResult class

Ready for review

Ready for review


text:
[T6A3][W14-C3]Abel Lim

Well done! :)

authors: abellimz michelletan


text:
[T2A3][F11-A3]Hsieh Hsin Han

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

authors: Tony-Hsieh xpdavid


text:
[W6.4h][W11-B1]WANG ZI

@alexawangzi
Some comments added. Please ack & close the PR after reading comments.

authors: alexawangzi okkhoy


text:
Add MonadFail constraints in obelisk-cliapp for GHC 8.6

Definitely seems like progress; can revist whether we want MonadFail or something else later.

repo: obsidiansystems/obelisk


text:
Add spider for Rr Boa Vista

Thanks @rennerocha I just updated the code

repo: okfn-brasil/diario-oficial


text:
pod上面的版本还是1.9.3，是还没有更新么？

更新了，请检查你是不是用了自己的源

authors: onevcat tripleCC

repo: onevcat/Kingfisher


text:
fix thread safety issues of AnimateImageView frames

@luochaojing LGTM. Thanks!

authors: luochaojing onevcat


text:
Generate coverage report for Scrutinizer.

moved to #42

authors: MindaugasR gyKa

repo: ongr-io/ongr-sandbox


text:
Add Ghostscript to dependent libraries

👍

This has been resolved and is in docker-build-base: https://github.com/onmodulus/docker-build-base/blob/master/bootstrap.sh#L16 Closing

authors: bitomule franvarney tzmanics

repo: onmodulus/docker-build-node


text:
Fix some tf2.x conversion bugs.

Did we verify this PR on efficientNet model conversion?

Did we verify this PR on efficientNet model conversion?
Yes, it can convert.
BTW, the EfficientNet doesn't run though the code path which I fixed.

authors: jiafatom wenbingl

repo: onnx/keras-onnx


text:
Bug fix, ensure krnl.iterate can lower in the degenerate case.

Could we have a test for this?

@doru1004 yes, sorry this is a WIP.

repo: onnx/onnx-mlir


text:
check-in witherspoon xml with full boot in hb standalone simics

@austinwrx - Please take a look

@op-jenkins ok to test

i looked at the recent changes i made. they are correct. looks good.

authors: austinwrx dcrowell77 pragupta

repo: open-power/witherspoon-xml


text:
V 9.0 Release

Fixing save issue.

repo: open-source-labs/reactime


text:
Code | Numeric Pattern 177

@aryashah2k kindly assign it to me.

repo: openAOD/Python-PatternHouse


text:
LDAP: Additional test case automations

[ ] - Verify Valid Group_Name update and privilege works - Good path

[ ] -  Verify InValid Group_Name update and privilege not works - Bad path

[ ] TBD

[ ] TBD

[ ] Update valid LDAP Group Name And Verify Operations with LDAP user with Administrator privilege
[ ] Update  valid LDAP Group Name And Verify Operations with LDAP user with Operator privilege
[ ] Update  valid LDAP Group Name And Verify Operations with LDAP user with User privilege
[ ] Update  valid LDAP Group Name And Verify Operations with LDAP user with Callback privilege
[ ] Update Invalid LDAP Group Name And Verify Operations with LDAP user with Administrator privilege
[ ] Update  Invalid LDAP Group Name And Verify Operations with LDAP user with Operator privilege
[ ] Update  Invalid LDAP Group Name And Verify Operations with LDAP user with User privilege
[ ] Update  Invalid LDAP Group Name And Verify Operations with LDAP user with Callback privilege

All above test cases are automated using https://gerrit.openbmc-project.xyz/c/openbmc/openbmc-test-automation/+/24159

repo: openbmc/openbmc-test-automation


text:
Added function to retrieve player friends, fixed some typos

Please ignore the hash files inside the NDLL folders and check return types on getPlayerFriends and gamecenter_playerfriends on different platforms.

authors: fbricker joackobengochea

repo: openfl/extension-gamecenter


text:
Fix outdated example documentation

Supersedes #73 which I will close.

repo: openforcefield/openforcefield


text:
Change heading for school authority eng column.

Thanks for the fix.

authors: johnollier robmckinnon

repo: openregister/school-data


text:
Fix sketch canvas not resizing properly. Fix #973

@avandecreme Thank you!
@leesei I've gone ahead and merged it, but can you verify that it's working to fix your issue?

authors: avandecreme iangilman

repo: openseadragon/openseadragon


text:
blog: getting started contributing

@flynnduism I was wondering if you could help me fix two things:

I've added the octocat emoji to the last paragraph using a png but it's creating a new line after itself: https://deploy-preview-32--openservicemesh.netlify.app/blog/getting-started-contributing/
Is it possible to reduce the spacing between each title and paragraph?
. Not super necessary if it's not possible but a nice to have!

@SanyaKochhar I've tweaked how images can be aligned in blog posts to get the result you want (using the figure method which allows easier alignment / sizing in markdown).

Reduced the spacing between h3 and p also. Hope that all looks ok now.

@flynnduism this looks perfect, thanks!

authors: SanyaKochhar flynnduism

repo: openservicemesh/osm-www


text:
enable draining taint within kubevirt

@rmohr could you please review?

Will require kubevirt 1.16.

0.16 is suppose @rmohr

Ah yes. I guess we don't want to wait for 1.16.

Agreed.

yeah, we already deployed 0.16

@hardys ready for merging :)

authors: fabiand karmab rmohr

repo: openshift-metalkube/dev-scripts


text:
Add option to set a PR number in fork_ami

I don't think we are moving forward with enhancing AMIs anymore.

repo: openshift/aos-cd-jobs


text:
Bug 1802161: regenerate-certificates command blocked by error illegal base64 data at input byte 3

/assing @sttts

/hold
for a manual smoke test

Clean pick?

Clean pick?

almost, had to regenerate bindata. Anyway, I have just successfully performed a manual smoke test - all good.  Ready to be merged.

/retest

/hold cancel

/lgtm
/approve

/bugzilla refresh

/bigzilla refresh

authors: p0lyn0mial sttts tnozicka


text:
Dockerfile.ocp: upgrade Go to 1.13

Remember to propagate this change to https://github.com/openshift/release/blob/master/ci-operator/config/openshift/prometheus-alertmanager/openshift-prometheus-alertmanager-master.yaml and similar files responsible for 4.4 and 4.5 releases

authors: paulfantom simonpasquier

repo: openshift/prometheus-alertmanager


text:
Release 1.1.1c

Rebased on top of #9034

repo: openssl/openssl


text:
Access: implement account endpoints

So how do we want to access this data from the indexed payloads? I couldn't find examples of a similar process happening in Flow so I'm not sure how it's supposed to be done. I assume there's a specific key type to look for with account in it, and then we should unmarshal the value into a flow.Account?
In flow the only cases of getting accounts I could find were done with scripts, so maybe that's also the way to go for us? I have literally no idea haha.

Fixed in #296

repo: optakt/flow-dps


text:
Some trivial fixes

Pulled, thanks!

authors: AkihiroSuda mitake

repo: osrg/earthquake


text:
Embed Client ID, Secret

https://github.com/otofune/seaside/commit/7da60cb66ea5539c56c395c716e0266125b03c1d

repo: otofune/seaside


text:
Makefiles updates

Needed to add a flag to have gcs access within the cluster.
GCS is mostly working here, still a few integration tests aren't passing but it's clearly writing to buckets. So I think it must be something other than the storage that's going wrong.

repo: pachyderm/pachyderm


text:
Fix wrong PR number in CHANGELOG.md

Thanks for your interest in palantir/tslint, @rail44! Before we can accept your pull request, you need to sign our contributor license agreement - just visit https://cla.palantir.com/ and follow the instructions. Once you sign, I'll automatically update this pull request.

Thanks @rail44!

authors: JKillian palantirtech rail44

repo: palantir/tslint


text:
Fix Simple Typo: serverly -> severely

Thank you for the contribution!

repo: pallets/click


text:
helper is not working in master branch for common messages

Fixed in 4682c39830db5e503d243f4a105bbad79e3c1a22


text:
feat: update text

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: patrickleet
If they are not already assigned, you can assign the PR to them by writing /assign @patrickleet in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/test this

/test this

/test this

/test this

/retest

:star: PR built and available in a preview environment patrickleet-node-http-pr-3 here

repo: patrickleet/node-http


text:
Build with Kobalt

Code is on kobalt branch. I'm closing this PR.

authors: cbeust pawegio

repo: pawegio/KAndroid


text:
[BACKLOG-26731][TEST] Fixed failing test

Build Completed
:fire: This pull request has some issues. It would be preferable to fix them in order for it to be just perfect. 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 'plugins/json/core' -P '!assemblies' -Dsurefire.runOrder=alphabetical -Daudit -amd clean install

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

Changed files
  plugins/json/core/src/test/java/org/pentaho/di/trans/steps/jsoninput/analyzer/JsonInputAnalyzerTest.java

Still Broken Tests:
org.pentaho.di.trans.steps.jsoninput.analyzer.JsonInputAnalyzerTest.testJsonInputExternalResourceConsumer:
:red_circle: java.lang.AssertionError
java.lang.AssertionError
	at org.pentaho.di.trans.steps.jsoninput.analyzer.JsonInputAnalyzerTest.testJsonInputExternalResourceConsumer(JsonInputAnalyzerTest.java:190)

Build Completed
:fire: This pull request has some issues. It would be preferable to fix them in order for it to be just perfect. 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 'plugins/json/core' -P '!assemblies' -Dsurefire.runOrder=alphabetical -Daudit -amd clean install

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

Changed files
  plugins/json/core/src/test/java/org/pentaho/di/trans/steps/jsoninput/analyzer/JsonInputAnalyzerTest.java

Still Broken Tests:
org.pentaho.di.trans.steps.jsoninput.analyzer.JsonInputAnalyzerTest.testJsonInputExternalResourceConsumer:
:red_circle: java.lang.AssertionError
java.lang.AssertionError
	at org.pentaho.di.trans.steps.jsoninput.analyzer.JsonInputAnalyzerTest.testJsonInputExternalResourceConsumer(JsonInputAnalyzerTest.java:190)

Build Completed
:white_check_mark: This pull request has passed all validations.
Build Commands
mvn -B -f 'pom.xml' -pl 'plugins/json/core' -P '!assemblies' -Dsurefire.runOrder=alphabetical -Daudit -amd clean install

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

Changed files
  plugins/json/core/pom.xml
  plugins/json/core/src/test/java/org/pentaho/di/trans/steps/jsoninput/analyzer/JsonInputAnalyzerTest.java
  plugins/json/core/src/test/java/org/pentaho/di/trans/steps/jsonoutput/analyzer/JsonOutputAnalyzerTest.java

Newly Fixed Tests:
org.pentaho.di.trans.steps.jsoninput.analyzer.JsonInputAnalyzerTest.testJsonInputExternalResourceConsumer:
:large_blue_circle: java.lang.AssertionError
java.lang.AssertionError
	at org.pentaho.di.trans.steps.jsoninput.analyzer.JsonInputAnalyzerTest.testJsonInputExternalResourceConsumer(JsonInputAnalyzerTest.java:195)

org.pentaho.di.trans.steps.jsonoutput.analyzer.JsonOutputAnalyzerTest.testJsonOutputExternalResourceConsumer:
:large_blue_circle: java.lang.AssertionError
java.lang.AssertionError: expected:<2> but was:<3>
	at org.pentaho.di.trans.steps.jsonoutput.analyzer.JsonOutputAnalyzerTest.testJsonOutputExternalResourceConsumer(JsonOutputAnalyzerTest.java:148)

@pentaho/r2d2

Build Completed
:white_check_mark: This pull request has passed all validations.
Build Commands
mvn -B -f 'pom.xml' -pl 'plugins/json/core' -P '!assemblies' -Dsurefire.runOrder=alphabetical -Daudit -amd clean install

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

Changed files
  plugins/json/core/pom.xml
  plugins/json/core/src/test/java/org/pentaho/di/trans/steps/jsoninput/analyzer/JsonInputAnalyzerTest.java
  plugins/json/core/src/test/java/org/pentaho/di/trans/steps/jsonoutput/analyzer/JsonOutputAnalyzerTest.java

Newly Fixed Tests:
org.pentaho.di.trans.steps.jsoninput.analyzer.JsonInputAnalyzerTest.testJsonInputExternalResourceConsumer:
:large_blue_circle: java.lang.AssertionError
java.lang.AssertionError
	at org.pentaho.di.trans.steps.jsoninput.analyzer.JsonInputAnalyzerTest.testJsonInputExternalResourceConsumer(JsonInputAnalyzerTest.java:195)

org.pentaho.di.trans.steps.jsonoutput.analyzer.JsonOutputAnalyzerTest.testJsonOutputExternalResourceConsumer:
:large_blue_circle: java.lang.AssertionError
java.lang.AssertionError: expected:<2> but was:<3>
	at org.pentaho.di.trans.steps.jsonoutput.analyzer.JsonOutputAnalyzerTest.testJsonOutputExternalResourceConsumer(JsonOutputAnalyzerTest.java:148)

authors: pentaho-amessier wingman-pentaho

repo: pentaho/pentaho-kettle


text:
configure: error: Package requirements (protobuf >= 2.6.0) were not met:

export PKG_CONFIG_PATH=$PKG_CONFIG_PATH:/path/to/protobuf/lib/pkgconfig


text:
add typescript typings

@peterhaldbaek

repo: peterhaldbaek/mt-osgridref


text:
Move assert_allclose from chainer.gradient_check to chainer.testing

I feel this PR causes a lot of conflicts with existing PRs. I want to merge this PR after almost all PRs are merged.

I split PR #1302.
Could you review this?

LGTM except comments.

Can you rebase or merge the mater branch?

LGTM

authors: okuta unnonouno

repo: pfnet/chainer


text:
Creature anchor point absolute fix

I have also updated the kestrel moon creature runtimes for the setAnchorPoint() method at:
https://github.com/kestrelm/Creature_WebGL/pull/6

repo: photonstorm/phaser-ce


text:
Add buildpack buildpackage info to the store

Codecov Report

Merging #372 into master will decrease coverage by 0.08%.
The diff coverage is 64.28%.

@@            Coverage Diff             @@
##           master     #372      +/-   ##
==========================================
- Coverage   67.60%   67.52%   -0.09%
==========================================
  Files          80       81       +1
  Lines        3294     3307      +13
==========================================
+ Hits         2227     2233       +6
- Misses        796      800       +4
- Partials      271      274       +3

Impacted Files
Coverage Δ

pkg/apis/experimental/v1alpha1/store_types.go
0.00% <ø> (ø)

pkg/cnb/remote_store_reader.go
73.43% <64.28%> (-3.93%)
:arrow_down:

pkg/apis/experimental/v1alpha1/stack_types.go
0.00% <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 9d6e17b...0107ada. Read the comment docs.

authors: codecov-io djoyahoy

repo: pivotal/kpack


text:
deleting a trace and adding one populates editor with deleted trace

I wonder if somehow the useAsDefaultTrace default value isn't getting mutated somewhere along the line by plotly.js... And the reason I don't see it in Amex is that I'm actually overwriting it?

I wonder if the default value for  useAsDefaultTrace isn't getting mutated somewhere along the line and the reason I'm not seeing it in jupyterlab is that I'm overriding it?

authors: VeraZab nicolaskruchten

repo: plotly/react-chart-editor


text:
Latest master

@BurntSushi Thanks!

repo: pombredanne/fst

license: Unlicense


text:
Last Week in Pony - February 18, 2018

A language server is being developed by irc user patroclos. Preview: https://gfycat.com/accuratehonesteeve


text:
Use specific AWS driver for Fog

Note: at the moment fog is used only to upload compiled assets to S3, but there's an open issue to remove async_assets as suggested by Heroku docs, see https://github.com/potomak/tomatoes/issues/161.

Ok, I didn't notice the #161 issue.

authors: dalpo potomak

repo: potomak/tomatoes


text:
Deployment readiness

@gsvr review please

:+1:

authors: gsvr imsickofmaps

repo: praekelt/seed-identity-store


text:
Add product test for nested Hive views

Updated, thanks

repo: prestosql/presto


text:
Release 1.4.0

Coverage remained the same at 97.101% when pulling 68c4e642894eaeb032430ac96f5ff35dcce225c1 on release-one-four-oh into fc5ecf0be2c04ae59bb033b5db8f5bde976753ce on master.

authors: coveralls trek

repo: pretenderjs/pretender


text:
DFU Runtime support

Oh, this is cool! Does the DFU support just extend to rebooting into DFU mode, and does it work with dfu-util etc? I should definitely backport the DFU part to FFP.

I guess you don't if it's working but I'd usually expect to write VTOR before jumping to the built-in bootloader - on the FFP code it's a CM0 without VTOR and instead does the ST-specific memory remap so the system memory starts at 0 and the vector table offset is done that way, but on the CM7 in the hs-probe we can write VTOR instead.

@adamgreig this code only supports rebooting into "full" DFU with dfu-util --detach. It works on F723 without VTOR change or remap, but on other chips you may need a different jump-to-bootloader code.

Thanks, I read through the DFU spec and it all makes sense. I implemented it for FFP here, much nicer than having to use my custom software to request bootloading.
When I was doing this before, using a vendor-specific code but otherwise the same thing, I found if I didn't ACK the request before resetting the stm32, the host got upset at the USB state, so I reworked my USB stack a bit to be able to finish and acknowledge the request first, and then bootload afterwards. Does the usb stack here do the same thing or is it just not a problem with dfu?

My implementation doesn't respond to the DFU_DETACH request, but it's ok as far as I understand the spec.

authors: Disasm adamgreig

repo: probe-rs/hs-probe-firmware


text:
Add 'podman restart' asciinema

LGTM

LGTM, but it woulda been nice to attach to the restarted container and cat test.txt one more time.
Did you sign this commit?  I'm not seeing it in the description.

Commit should be signed, yeah. Good call about the attach/cat, but it took the better part of an hour to do it without major typos, so I'll save that bit for later

@rh-atomic-bot r+

authors: TomSweeneyRedHat mheon rhatdan

repo: projectatomic/libpod


text:
能否实现拼音转汉字？求指教

这个项目虽然支持多音字，但最终处理结果（根据上下文和自定义字典）还是一对一。
拼音转汉字是一对多，即使有上下文也几乎无法实现一对一，与这个项目的需求工作方法完全不同，不太可能修改这个项目实现。
建议去找一下含有“输入法智能提示”、“拼音输入法候选项”关键词的项目。

authors: VaslD caozhx

repo: promeG/TinyPinyin


text:
Fixed typo

Coverage increased (+0.07%) to 98.282% when pulling d2a62d7419fe7c79e37692440ef817dd9de45897 on gquemener:patch-1 into 6a98854fc83e0188e5a1a9d4b1e0efac09cfd46f on prooph:master.

authors: coveralls gquemener

repo: prooph/service-bus


text:
add tests, small bugfix in exception handling

Coverage increased (+1.7%) to 100.0% when pulling aef9d68a584cc67c2bb54cc3af37969bf2366145 on tests into 3df548fc554ea230fe41dc32a61af2d1e27b2b14 on master.

authors: coveralls prolic


text:
Fixing tests for previous PR

Oh, I see you've already fixed this?

repo: psecio/iniscan


text:
Currator 4.1.2

merged

repo: pubnative/docker-images


text:
(PDB-3521) Migrate 4.2.3.x to openjdk8

Test FAILed

Test PASSed

CLA signed by all contributors.

authors: puppetcla puppetlabs-jenkins rbrw

repo: puppetlabs/puppetdb