text:
[BACKLOG-8492] Added MDI Support for Get Table Names

Build completed
:x: This pull request has errors. They will need to be addressed before it can be accepted. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands:

ant -Dtestreports.dir=bin/reports/unit-test -f engine/build.xml clean-all resolve jacoco && ant -Dtestreports.dir=bin/reports/integration-test -f engine/build.xml jacoco-integration checkstyle publish-local

Cleanup Commands:
rm -r ~/.ivy2/local || echo no publish local to remove
Changed files:
engine/src/org/pentaho/di/trans/steps/gettablenames/GetTableNamesMeta.java
engine/src/org/pentaho/di/trans/steps/gettablenames/messages/messages_en_US.properties
engine/test-src/org/pentaho/di/trans/steps/gettablenames/GetTableNamesMetaInjectionTest.java
Newly Broken Tests:
org.pentaho.di.trans.steps.monetdbbulkloader.MonetDBBulkLoaderMetaTest.testSerialization:
java.lang.RuntimeException
java.lang.RuntimeException: Error validating autoSchema
	at org.pentaho.di.trans.steps.loadsave.LoadSaveTester.validateLoadedMeta(LoadSaveTester.java:193)
	at org.pentaho.di.trans.steps.loadsave.LoadSaveTester.testXmlRoundTrip(LoadSaveTester.java:238)
	at org.pentaho.di.trans.steps.loadsave.LoadSaveTester.testSerialization(LoadSaveTester.java:199)
	at org.pentaho.di.trans.steps.monetdbbulkloader.MonetDBBulkLoaderMetaTest.testSerialization(MonetDBBulkLoaderMetaTest.java:133)
Caused by: org.pentah

Still Broken Tests:
org.pentaho.di.trans.steps.rest.RestIT.testNoContent:
java.lang.ArrayIndexOutOfBoundsException
java.lang.ArrayIndexOutOfBoundsException: 11264
	at org.objectweb.asm.ClassReader.readClass(Unknown Source)
	at org.objectweb.asm.ClassReader.accept(Unknown Source)
	at org.objectweb.asm.ClassReader.accept(Unknown Source)
	at com.sun.jersey.spi.scanning.AnnotationScannerListener.onProcess(AnnotationScannerListener.java:136)
	at com.sun.jersey.core.spi.scanning.JarFileScanner.scan(JarFileScanner.java:97)
	at com.sun.jersey.core.spi.scanning.JarFileScanner$1.f(JarFileScanner.java:74)
	at com.sun.j

org.pentaho.di.trans.steps.rest.RestIT.testResponseHeader:
java.lang.ArrayIndexOutOfBoundsException
java.lang.ArrayIndexOutOfBoundsException

Unit test coverage change:
org.pentaho.di.job.entries.evalfilesmetrics.JobEntryEvalFilesMetrics:

Branch Change : -.39%
Complexity Change : -.51%
Instruction Change : -.08%

org.pentaho.di.job.entries.evaluatetablecontent.JobEntryEvalTableContent:

Branch Change :  + 1.01%
Complexity Change :  + 1.06%
Instruction Change :  + .17%

org.pentaho.di.trans.step.StepMeta:

Instruction Change : -.19%
Line Change : -.27%

org.pentaho.di.trans.steps.checksum.CheckSumMeta:

Instruction Change :  + .12%

org.pentaho.di.trans.steps.gettablenames.GetTableNamesMeta:

Complexity Change :  + .53%
Instruction Change :  + .41%
Line Change :  + .73%
Method Change :  + .29%

org.pentaho.di.trans.steps.groupby.GroupByMeta:

Branch Change : -1.04%
Complexity Change : -1.05%

org.pentaho.di.trans.steps.ifnull.IfNullMeta:

Branch Change : -2.38%
Complexity Change : -1.64%
Instruction Change : -.17%

org.pentaho.di.trans.steps.insertupdate.InsertUpdateMeta:

Instruction Change :  + .05%

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

Instruction Change :  + .06%

org.pentaho.di.trans.steps.monetdbbulkloader.MonetDBBulkLoaderMeta:

Branch Change : -6.85%
Complexity Change : -4.38%
Instruction Change : -14.09%
Line Change : -11.53%
Method Change : -6.25%

org.pentaho.di.trans.steps.rowgenerator.RowGeneratorMeta:

Instruction Change : -.08%

org.pentaho.di.trans.steps.sortedmerge.SortedMergeMeta:

Instruction Change : -.18%

org.pentaho.di.trans.steps.systemdata.SystemDataMeta:

Branch Change : -2.04%
Complexity Change : -2.04%
Instruction Change : -.32%
Line Change : -.81%

org.pentaho.di.trans.steps.textfileinput.TextFileInputMeta:

Branch Change : -.53%

Checkstyle violations:
See: https://github.com/pentaho/pentaho-coding-standards
File /home/buildguy/project/engine/src/org/pentaho/di/trans/steps/gettablenames/GetTableNamesMeta.java

Message: ',' is preceded with whitespace. at Line: 60 Column: 90
Message: ',' is preceded with whitespace. at Line: 60 Column: 103
Message: Line has trailing spaces at Line: 66
Message: Line has trailing spaces at Line: 69
Message: Line has trailing spaces at Line: 73
Message: Line has trailing spaces at Line: 76
Message: Line has trailing spaces at Line: 79
Message: Line has trailing spaces at Line: 85
Message: Line has trailing spaces at Line: 88
Message: Line has trailing spaces at Line: 91
Message: Line has trailing spaces at Line: 94
Message: Line has trailing spaces at Line: 97
Message: Line has trailing spaces at Line: 100
Message: Line has trailing spaces at Line: 103
Message: Line has trailing spaces at Line: 106
Message: Line has trailing spaces at Line: 111
Message: File contains tab characters (this is the first instance). at Line: 287 Column: 1
Message: Line has trailing spaces at Line: 289

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:

ant -Dtestreports.dir=bin/reports/unit-test -f engine/build.xml clean-all resolve jacoco && ant -Dtestreports.dir=bin/reports/integration-test -f engine/build.xml jacoco-integration checkstyle publish-local

Cleanup Commands:
rm -r ~/.ivy2/local || echo no publish local to remove
Changed files:
engine/src/org/pentaho/di/trans/steps/gettablenames/GetTableNamesMeta.java
engine/src/org/pentaho/di/trans/steps/gettablenames/messages/messages_en_US.properties
engine/src/org/pentaho/di/trans/steps/getvariable/messages/messages_en_US.properties
engine/test-src/org/pentaho/di/trans/steps/gettablenames/GetTableNamesMetaInjectionTest.java
Newly Fixed Tests:
org.pentaho.di.trans.steps.monetdbbulkloader.MonetDBBulkLoaderMetaTest.testSerialization:
java.lang.RuntimeException
java.lang.RuntimeException: Error validating autoSchema
	at org.pentaho.di.trans.steps.loadsave.LoadSaveTester.validateLoadedMeta(LoadSaveTester.java:193)
	at org.pentaho.di.trans.steps.loadsave.LoadSaveTester.testXmlRoundTrip(LoadSaveTester.java:238)
	at org.pentaho.di.trans.steps.loadsave.LoadSaveTester.testSerialization(LoadSaveTester.java:199)
	at org.pentaho.di.trans.steps.monetdbbulkloader.MonetDBBulkLoaderMetaTest.testSerialization(MonetDBBulkLoaderMetaTest.java:133)
Caused by: org.pentah

Still Broken Tests:
org.pentaho.di.trans.steps.rest.RestIT.testNoContent:
java.lang.ArrayIndexOutOfBoundsException
java.lang.ArrayIndexOutOfBoundsException: 11264
	at org.objectweb.asm.ClassReader.readClass(Unknown Source)
	at org.objectweb.asm.ClassReader.accept(Unknown Source)
	at org.objectweb.asm.ClassReader.accept(Unknown Source)
	at com.sun.jersey.spi.scanning.AnnotationScannerListener.onProcess(AnnotationScannerListener.java:136)
	at com.sun.jersey.core.spi.scanning.JarFileScanner.scan(JarFileScanner.java:97)
	at com.sun.jersey.core.spi.scanning.JarFileScanner$1.f(JarFileScanner.java:74)
	at com.sun.j

org.pentaho.di.trans.steps.rest.RestIT.testResponseHeader:
java.lang.ArrayIndexOutOfBoundsException
java.lang.ArrayIndexOutOfBoundsException

Unit test coverage change:
org.pentaho.di.job.entries.sql.JobEntrySQL:

Instruction Change : -3.28%
Line Change : -3.87%

org.pentaho.di.repository.kdr.KettleDatabaseRepositoryCreationHelper:

Branch Change : -1.64%
Complexity Change : -.69%
Instruction Change : -2.43%
Line Change : -3.70%

org.pentaho.di.trans.steps.checksum.CheckSumMeta:

Instruction Change : -.12%

org.pentaho.di.trans.steps.datagrid.DataGridMeta:

Instruction Change :  + .09%

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

Instruction Change : -.11%

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

Branch Change : -.66%
Complexity Change : -.41%
Instruction Change : -.50%
Line Change : -.24%

org.pentaho.di.trans.steps.gettablenames.GetTableNamesMeta:

Complexity Change :  + .53%
Instruction Change :  + .41%
Line Change :  + .73%
Method Change :  + .29%

org.pentaho.di.trans.steps.groupby.GroupByMeta:

Branch Change : -1.04%
Complexity Change : -1.05%

org.pentaho.di.trans.steps.ifnull.IfNullMeta:

Instruction Change :  + .08%

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

Instruction Change : -.06%

org.pentaho.di.trans.steps.memgroupby.MemoryGroupByMeta:

Branch Change :  + 2.50%
Complexity Change :  + 2.82%

org.pentaho.di.trans.steps.monetdbbulkloader.MonetDBBulkLoaderMeta:

Branch Change :  + 6.85%
Complexity Change :  + 4.38%
Instruction Change :  + 14.09%
Line Change :  + 11.53%
Method Change :  + 6.25%

org.pentaho.di.trans.steps.rssinput.RssInputField:

Branch Change : -2.27%
Complexity Change : -1.54%
Instruction Change : -.29%

org.pentaho.di.trans.steps.setvalueconstant.SetValueConstantMeta:

Instruction Change :  + .14%

org.pentaho.di.trans.steps.singlethreader.SingleThreaderMeta:

Instruction Change : -.08%

org.pentaho.di.trans.steps.symmetriccrypto.symmetriccryptotrans.SymmetricCryptoTransMeta:

Branch Change : -3.33%
Complexity Change : -1.85%
Instruction Change : -.28%

org.pentaho.di.trans.steps.synchronizeaftermerge.SynchronizeAfterMergeMeta:

Instruction Change :  + .04%

org.pentaho.di.trans.steps.systemdata.SystemDataMeta:

Branch Change : -2.04%
Complexity Change : -2.04%
Instruction Change : -.32%
Line Change : -.81%

org.pentaho.di.trans.steps.tableoutput.TableOutputMeta:

Branch Change :  + .91%
Complexity Change :  + .82%

org.pentaho.di.trans.steps.uniquerows.UniqueRowsMeta:

Instruction Change :  + .18%


text:
Add AppVeyor badge

@saurvs I see what you meant when you said we should allow CI failures on nightly. Let's enable that for AppVeyor only for now in this PR itself.

source: gharchive/pull-request

authors: saurvs suyash

repo: performancecopilot/hornet

license_type: permissive

license_source: bigquery


text:
Module :: Mercury Service Implementation - FindBugs fixes

Bug
Category
Details
Line
Priority

org.apache.river.mercury.ServiceRegistration defines compareTo(Object) and uses Object.equals()
BAD_PRACTICE
EQ_COMPARETO_USE_OBJECT_EQUALS
282-292
Medium

Initially this seemed like a harmless bug, however mutable ServiceRegistration instances are used as keys in TreeMap and expiration is part of mutable state and compareTo comparisons.
Solution:
Implement equals and hashCode methods, compliant with the Comparable interface.
Implement Cloneable
Implement new method renew, which returns a copy of ServiceRegistration with a new expiration, without modifying the original.
Update uses of ServiceRegistration, to use the new method and replace instances in the TreeMap with new copies instead of mutating the existing.

source: gharchive/issue

repo: pfirmstone/JGDMS


text:
base: reduce systemd timeouts in /etc/systemd/systemd.conf

reduced to 10s


text:
CI for Node

● Game › Game › Starts the game with a random player
expect(received).toBe(expected) // Object.is equality

Expected: "Nate"
Received: "Bananas"

license: MIT

license_source: github-api


text:
Check device and manifest

👎

repo: phonegap-build/PushPlugin

license: mit


text:
add drag-drop and mimedata modules

Changes from phosphor-dragdrop:

DropAction and DropActions enums are now the DropAction and SupportedActions string literal type aliases.
IDragOptions -> Drag.IOptions
MimeData is in a separate module: core/mimedata.ts so it can be reused for other things/libraries

Coverage remained the same at 97.926% when pulling fcb47d56e78012d4a7abb979d9a0108a9cca4a5e on feature-dragdrop into f0b44a0a0deb43259f57f67229d3cee27b1b1ec1 on master.

LGTM.

I'm pretty sure I asked in the original module, but since I don't remember the answer, perhaps it's better to ask here so that there will be a record of it. What's the reasoning again for having two arrays in MimeData, one for keys and one for values, instead of having a simple object: { [key: string]: any }?

@afshin so that the return value of types() is always order of call to setData().

The de facto standard for Object.keys() is order of insertion, but the real standard actually specifies it as undefined order. Not sure if order of insertion is maintained after deletes though.

In V8, it looks like only initial order of insertion is maintained:
> var a = {}
var a = {}
undefined
> a['1'] = 1
1
> a['2'] = 2
2
> a['3'] = 3
3
> Object.keys(a)
[ '1', '2', '3' ]
> delete a['2']
true
> Object.keys(a)
[ '1', '3' ]
> a['2'] = 2
2
> Object.keys(a)
[ '1', '2', '3' ]

Awesome, thanks for the explanation.
👍

If it ends up being a performance problem, we can change the internal data structure to something with better lookup characteristics. I doubt that will happen though, since the number of mime types used will normally only be a handful.

authors: afshin blink1073 coveralls sccolbert

repo: phosphorjs/phosphor


text:
Create master

https://www.pinterest.co.uk/

https://www.pinterest.com/pin/537828380480333120/

repo: pinterest/pinterest-api-demo


text:
A few more projects

Thanks! :)

authors: keyphact leveled

repo: pkmngo-odi/awesome-pokemongo


text:
Plug Volto's real store to storybook

@tiberiuichim thanks a lot for looking into this!

authors: sneridagh tiberiuichim

repo: plone/volto


text:
Fixed environment variable typo, DASHBOARDSLY not DASHBOARDLY

Thanks!

authors: chrstphe scjody


text:
Depends on yaml-cpp

typo

done

repo: poacpm/poac


text:
FC013 Use file_cache_path

@chrishenry Any chance you wanna rebase this?

Closing in favor of https://github.com/poise/rubocop-chef/pull/10

repo: poise/rubocop-chef


text:
WIP: use rococo-v1 dependencies [PONC-134]

Pull Request Test Coverage Report for Build 856791482

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

Totals

Change from base Build 832366680:
0.0%

Covered Lines:
73

Relevant Lines:
79

💛  - Coveralls

authors: coveralls vitvakatu

repo: pontem-network/sp-move


text:
Broke stuff, reverting

/hsurhg

repo: ppy/osu-framework


text:
test file

Where is the rest of the test framework?

authors: dbreedt retrorabbitWD

repo: praekelt/gem-survey-tool


text:
IoGT 105: Repeat polls headline on footer menu

@saeedmb8 @nathanbegbie
Please review

@saeedmb8 :+1:

@Mitso happy to merge this 👍  🍰


text:
Related article basic elements update

@saeedmb8 please review

👍

repo: praekelt/molo


text:
Update maintainers

Welcome to the team! 🎆

Thank you!

authors: LilSpazJoekp bboe

repo: praw-dev/praw


text:
Added docs for undertone USP support

This is merged into 3.1 but needs a separate code PR for 2.44.x

authors: bretg omerko


text:
Implement condition on measure-zero observation (i.e., using logpdf)

Blocked by https://github.com/probcomp/sppl/issues/78

repo: probcomp/sppl


text:
Swagger Feature

This should be two different pull request/issues.  The one for core should on exposing application properties (name,version,description) to the triggers.   The second one should be a pull request to add a swagger activity to contrib or possibly its own repository depending on the amount of dependencies it brings in.  We want to limit contrib third-party dependencies to some extent.

Removed swagger activity from core and will move it to contrib as a seperate request.
Current core branch would be for  application properties (name,version,description) to the triggers.

authors: AkshayGadikar fm-tibco

repo: project-flogo/core


text:
Add --kube-plugin-version to node docs

Are there any unit tests that need to be added for these changes?

The changes have already been merged and tested as of v0.9.0 - these are just the corresponding docs changes.

Gotcha.  LGTM

authors: TrimBiggs caseydavenport

repo: projectcalico/calico-docker


text:
[WIP] bugfix: service key register and graceful unregister

related pr: https://github.com/projecteru2/core/pull/326

related pr: https://github.com/projecteru2/core/pull/326


text:
Honor execludeList in cleanup of old webjobs. Also fix a typo

actually now that I think of it, this is not sufficient. Primary will still clean up secondary webjobs it doesn't know about

repo: projectkudu/kudu


text:
Tollerate server not sending RECEIPT frame on DISCONNECT

Out of Date

repo: projectodd/stilts


text:
Keep consistency with the toString template

FppTest\LoadTemplateTest::it_loads_body_template_for_base_enum_class
Failed asserting that two strings are identical.
--- Expected
+++ Actual
@@ @@
{
return $this->name;
}
+

public function toString(): string
{

   return $this->name;

}

}
'
/home/travis/build/prolic/fpp/tests/LoadTemplateTest.php:212
FAILURES!
Tests: 280, Assertions: 488, Failures: 1, Incomplete: 1.
On Tue, Oct 2, 2018, 07:29 Matthias Breddin notifications@github.com
wrote:

You can view, comment on, or merge this pull request online at:
https://github.com/prolic/fpp/pull/87
Commit Summary

Keep consistency with the toString template

File Changes

M src/templates/enum.template
https://github.com/prolic/fpp/pull/87/files#diff-0 (7)

Patch Links:

https://github.com/prolic/fpp/pull/87.patch
https://github.com/prolic/fpp/pull/87.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/prolic/fpp/pull/87, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAYEvDrZQs5AojHaatqMtjxIpoLWqF0yks5ugqVigaJpZM4XDERk
.

closed -> new pr: https://github.com/prolic/fpp/pull/88

authors: lunetics prolic

repo: prolic/fpp


text:
factored out apos properties for easy overrides and super pattern

Reviewed with Bob.

repo: punkave/apostrophe


text:
(FACT-2788) Remove semaphore from all resolvers

Jenkins please test this on all

Why?

CLA signed by all contributors.

Good question, and it's not answered by
https://tickets.puppetlabs.com/browse/FACT-2788 either

Until #2050 we were creating a thread for each fact. As multiple facts could use the same resolver we needed to ensure that there is no concurrency inside a resolver. This was achieved with a Mutex in each resolver. One of the disadvantages of this approach was that we created many threads and the performance gain was inconsistent (one some platforms the parallel version was faster and on others the sequential one was faster). A step was made to improve the multi threaded implementation with https://github.com/puppetlabs/facter/compare/main...IrimieBogdan:FACT-2753-parallel. This new implementation used a thread pool from concurrent-ruby. The thread pool implementation was the slowest of them all.
With MRI ruby we can obtain concurrency but not parallelism, and it seams that concurrency alone does not offer a consistent performance advantage over the sequential approach.
Taking this into account we decided to resolve facts sequentially for the time being and to experiment with parallelism on https://github.com/IrimieBogdan/facter/tree/FACT-2753-parallel

that would've made for a pretty good commit message / PR description

@igalic agreed, will try to add a description for further PRs

authors: IrimieBogdan MikaelSmith igalic puppetcla

repo: puppetlabs/facter


text:
(PUP-9448) Rephrase NOT_ABSOLUTE_TOP_LEVEL to not use a/an

CLA signed by all contributors.

include in single PR with multiple commits

authors: ehom puppetcla

repo: puppetlabs/puppet


text:
(CPR-718) Add resiliency for consul ipaddress checks

CLA signed by all contributors.

authors: puppetcla underscorgan

repo: puppetlabs/puppetdb


text:
Add chai-enzyme for testing components

Already fixed by @busypeoples 😄

repo: pure-ui/styleguide


text:
Fix the weird caching issues we saw at ISTS

I think we fixed the caching in ajax in https://github.com/pwnbus/scoring_engine/pull/257 and https://github.com/pwnbus/scoring_engine/pull/256.


text:
Device measurementcomputing

Don't merge now I need some extra work.

I am back on this very old PR.
I would like to merge it soon.
If someone is courageous enougth to have a look...

repo: pyacq/pyacq


text:
Spark - QScriptCore.Reduce + important SKI change

Current coverage is 65.63% (diff: 0.00%)

Merging #1490 into master will decrease coverage by 0.17%

@@             master      #1490   diff @@
==========================================
  Files           308        308
  Lines         14281      14326    +45
  Methods       13931      13973    +42
  Messages          0          0
  Branches        209        212     +3
==========================================
+ Hits           9398       9403     +5
- Misses         4883       4923    +40
  Partials          0          0

Powered by Codecov. Last update cfd454d...c3b9731

I've fixed stuff from review, I don't get the comment regarding maybeTransformers and reducersFuncs.

This looks fine. Code’s not currently compiling, though.

sigh was just about to merge this, but conflicts.

Missing a changelog (add one with [skip ci])

authors: codecov-io rabbitonweb sellout

repo: quasar-analytics/quasar


text:
Tests added for packet registeration for serialization

Thanks! This finally prevents that someone forgets to add the packet to the serializer.

repo: quasar/QuasarRAT


text:
spark command support in java sdk

Please fix the merge conflicts.

@Sathya12 - Can you look at this ?

Squash merged: https://github.com/qubole/qds-sdk-java/commit/54d6261591bbe6cf6f044e560180a7bbf387e7df

authors: PraveenSeluka Sathya12 mindprince

repo: qubole/qds-sdk-java


text:
Foreign schema refs

Everything looks good to me 👍

authors: dvdsgl schani

repo: quicktype/quicktype


text:
Update travis Node.js version & remove io.js

Looks like there are issues with contextify. Unfortunately, @brianmcd hasn't update the project in a while and it's not been working with iojs 3.0 either. Anyone know of any alternatives?

We can wait for Lodash 4 (#611) and then we won't need that test relying on contextify.

@demohi Can you merge in master to your branch? It should now run correctly under 4.0.0. I've tested locally and want to double-check against Travis.

@timdorr    I merged tha.

Looks like there's an upstream fix in webpack-dev-server (and socket.io) that isn't been released yet. It should fix all those npm build errors in Travis. But it runs, so it's good for now.

authors: demohi gaearon timdorr

repo: rackt/redux


text:
When I select a snippet and press Enter nothing happens

Do you have vsnip set up ? Here some examples from the wiki.
" NOTE: You can use other key to expand snippet.

" Expand
imap <expr> <C-j>   vsnip#expandable()  ? '<Plug>(vsnip-expand)'         : '<C-j>'
smap <expr> <C-j>   vsnip#expandable()  ? '<Plug>(vsnip-expand)'         : '<C-j>'

" Expand or jump
imap <expr> <C-l>   vsnip#available(1)  ? '<Plug>(vsnip-expand-or-jump)' : '<C-l>'
smap <expr> <C-l>   vsnip#available(1)  ? '<Plug>(vsnip-expand-or-jump)' : '<C-l>'

" Jump forward or backward
imap <expr> <Tab>   vsnip#jumpable(1)   ? '<Plug>(vsnip-jump-next)'      : '<Tab>'
smap <expr> <Tab>   vsnip#jumpable(1)   ? '<Plug>(vsnip-jump-next)'      : '<Tab>'
imap <expr> <S-Tab> vsnip#jumpable(-1)  ? '<Plug>(vsnip-jump-prev)'      : '<S-Tab>'
smap <expr> <S-Tab> vsnip#jumpable(-1)  ? '<Plug>(vsnip-jump-prev)'      : '<S-Tab>'

This repo is just a snippet collection and doesn't provide anything to expand, confirm or jump snippets. That's left to whatever plugins you use and if they are compatible with the snippets format that vscode uses. Because of that I'm gonna close this. However you can still comment and I will help you. I recommend using nvim-compe with vsnip if you aren't already.

repo: rafamadriz/friendly-snippets


text:
WIP

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@654fee7). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master       #6   +/-   ##
=========================================
  Coverage          ?   10.42%
=========================================
  Files             ?       11
  Lines             ?     1650
  Branches          ?        0
=========================================
  Hits              ?      172
  Misses            ?     1478
  Partials          ?        0

Flag
Coverage Δ

#unittests
10.42% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out 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 654fee7...39fa996. Read the comment docs.


text:
Support vmTemplate and vmTemplateRevision

Updated pr, will address namespaceStore after getting more info

updated.

repo: rancher/harvester


text:
Update slirp4netns to v1.0.1

Do we want to close this out in favor of #24?

Closing since #24 was merged

authors: brandond erikwilson


text:
Remove rdkit/Chem/fmcs

Duplicate with #1344

repo: rdkit/rdkit


text:
WIP: release beta 3.0.0

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

v3.0.0
GitHub release

Your semantic-release bot :package::rocket:

repo: react-page/react-page


text:
JBDS-4473 Update CDK to 3.1.0.GA build

Codecov Report

Merging #819 into master will decrease coverage by 0.13%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #819      +/-   ##
==========================================
- Coverage   96.51%   96.38%   -0.14%
==========================================
  Files          36       36
  Lines        3212     3399     +187
  Branches      267      480     +213
==========================================
+ Hits         3100     3276     +176
- Misses        112      114       +2
- Partials        0        9       +9

Impacted Files
Coverage Δ

browser/model/helpers/downloader.js
96.94% <0%> (-1.52%)
:arrow_down:

browser/model/jbosseap.js
98.8% <0%> (-1.2%)
:arrow_down:

browser/pages/about/controller.js
21.05% <0%> (-1.17%)
:arrow_down:

browser/directives/componentPanel.js
14.28% <0%> (-1.1%)
:arrow_down:

browser/pages/confirm/controller.js
89.79% <0%> (-0.97%)
:arrow_down:

browser/services/data.js
97.03% <0%> (-0.92%)
:arrow_down:

browser/model/jbossfuse.js
99.21% <0%> (-0.79%)
:arrow_down:

browser/model/cdk.js
99.22% <0%> (-0.78%)
:arrow_down:

browser/model/jdk-install.js
97.29% <0%> (-0.45%)
:arrow_down:

browser/model/devstudio.js
89.13% <0%> (-0.41%)
:arrow_down:

... and 22 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 a949e03...ac6b00c. Read the comment docs.

authors: codecov-io dgolovin


text:
import kots 1.13.3

supersedes https://github.com/replicatedhq/kotsadm/pull/1187

repo: replicatedhq/kotsadm


text:
Disable by default cached adapters

Temporary fix until we find no workaround for the memory limit issue.

Temporary fix until we find no workaround for the memory limit issue.

repo: repman-io/repman


text:
conversation notifications in the header and postbox doesn't get incremented

closed. db702e42

repo: researchstudio-sat/webofneeds


text:
Retire Systemd base images

SGTM

repo: resin-io-library/base-images


text:
run-resinhup.sh: only umount the boot partition if it's mounted on /boot

@agherzan @floion

Can you also add a CHANGELOG entry please?

authors: floion michal-mazurek

repo: resin-os/meta-resin


text:
Use only the initramfs bundled in the kernel

@agherzan I added the meta-resin submodule update to this PR.
Please re-review

repo: resin-os/resin-up-board


text:
How to integrate reql in Actix web?

Currently, reql is designed in a way that makes it very easy to integrate with any framework. The only object that needs to be passed around is the Connection. However, it doesn't carry the connection pool, itself. It only carries its ID, so it's very small. Because of that it implements Copy.
In Actix, like in any other framework, you should initiliase the connection in your main function to get the connection object. You can then copy the object around as app state, using App::with_state. See this example.

@rushmorem got it!

@rushmorem can i use everywhere a new connection?
                let r = Client::new();
                let conn = r.connect(Config::default()).unwrap();

authors: rushmorem sackery

repo: rethinkdb-rs/reql


text:
added vgg16 example EVS 2018

Few additional changes requested:

add a link to this example in top-level README.md
add a link to original khronos NNEF-Tools repo -- and mention that this change will be moved into khronos repo eventually

Thx

authors: lcskrishna rgiduthuri


text:
PSR4

Should be closed. @leogr do you agree?

repo: ripaclub/sphinxsearch


text:
Added Basic Android RE-1 - Reverse

mu goodness

repo: rishitsaiya/HelloWorld


text:
Dashboard

Coverage increased (+0.05%) to 96.321% when pulling bcc32d09cc53a438e61d932522b9c85279ec3250 on dashboard into 390ce524ddaa3e24792d53217f43470cc1d5f6ad on master.

Coverage increased (+0.05%) to 96.321% when pulling 3dbbf8cce309b232d7e7f35120b202c4e5573d11 on dashboard into 390ce524ddaa3e24792d53217f43470cc1d5f6ad on master.

authors: coveralls e-i-n-s

repo: rocketvoip/rocketvoip-frontend


text:
Bump to vue 2?

Done!


text:
added https support

Coverage increased (+0.009%) to 96.438% when pulling 15c673b1df81732a90f56d4df87bc93de3af5640 on a5i:master into 75c5ca331789a9b4c1f8747b6b54f88bc650bd5a on roistat:master.

Также прошу написать тест на NewConn

Coverage increased (+0.009%) to 96.438% when pulling e7959d92f0b9246edec745b5ef2bb8275cca26c6 on a5i:master into 75c5ca331789a9b4c1f8747b6b54f88bc650bd5a on roistat:master.

repo: roistat/go-clickhouse


text:
Use dry-initializer v2.0

@solnic this can go to master

repo: rom-rb/rom


text:
[WIP] Redfish Interoperability Profile for Computer Systems

Incorrect branch

repo: rpioso/sushy


text:
update example context.read

Nice catch!


text:
Trying out CLA

Funny enough - Sean got called out 😛  https://github.com/rstudio/rstudio-docker-products/pull/88#issuecomment-868821738

authors: colearendt trestletech

repo: rstudio/rstudio-docker-products


text:
Feature3 - add lessons and terms

All done with my review. Good job! 🚀

authors: jmkoni osondoar

repo: rubyforgood/justice


text:
Use Member#uid as BitGo's destination address label

You can use the parameter options in def create_withdrawal!(issuer, recipient, amount, options = {}) to pass member UID.

Closed by #1318.

authors: yivo yssavchuk

repo: rubykube/peatio


text:
FIxes #15

Thanks!

repo: rust-osdev/cargo-xbuild


text:
Adding Duke University

I am closing this, but I will add Duke to the list. The merge does not want to go smoothly, so I just manually updated. Thank you for the update @phpforfree

authors: phpforfree ryanswanstrom


text:
SAK-40618 - Elfinder can't manage PermissionExceptions

I didn't realize earlier, should switch the logging to SLF4J as we don't use apache commons logging anymore. (We have a slf4j bridge for commons logging for libraries we can't change)

@ern and @bjones86 , @mpellicer has kindly done several commits that address your requested changes. Could you review them, please? Thanks everyone.

@bjones86 I've done the changes. Please, could you review again?

authors: danielmerino ern

repo: sakaiproject/sakai


text:
add persisten volumes to ldap chart

Fine with me.
My concern is that we don't build this repo anymore.  @viroos you may want to split this out as its own thing, I don't know if we are going to.

authors: coffeepac viroos


text:
remove andThen from as helper example

Coverage remained the same at 98.763% when pulling 59ff7a76bea3f01d88d417c476d80c4a98e68e76 on ro0gr:no-andthen-in-as into aaec068aa9a8c4faba810af04d8644ea968c5669 on san650:master.


text:
Clear cookies with path SASLogon and serverUrl domain before login

Cookies that are related to SASLogon are HttOnly which means we can't remove them programatically.

repo: sasjs/adapter


text:
Implement a schemadeploy command

とりあえず書いてみた。テストは多少ある

schemadeploy と git-schemalex と挙動変わってるので、別ツールとして提供して行くって感じの方がいいですかねぇ。
前者は、mysql を見て、指定した ( git の commit でも可）ものをデプロイする
となっていて、
後者は、mysql に入っている git の version から git 経由でスキーマを見て、一応 HEAD のスキーマがデプロイされる
となっているので。
利用者から見た挙動は変わらないはずですけど、前者で HEAD を指定した場合、git rev-parse HEAD で HEAD の commit, 後者はファイルの最新の変更の commit になっているので、必ずしも HEAD の commit ではないというところが違いますね。（db.version == deploy.version にはならないけど、結局 diff は出ないから）
git-schemalex は gitddl をそのまま持ってきているので、どちらが良い悪いという話ではないです。

前者は、mysql を見て、指定した ( git の commit でも可）ものをデプロイする

あれ、なんか違ってるのか。テストが通ったのでこれでいいかと思ってました！テストがあれば…！ :D

https://github.com/schemalex/git-schemalex/blob/master/gitschemalex_test.go 今のやつもこれから持ってきたと思うんですけど、ざっくりしかないですからねぇ

@soh335 https://github.com/schemalex/schemalex/pull/52#issuecomment-365171436
これの件は挙動が変わるとgit-schemalexをすでに使ってる場所では導入されなくなっちゃいそうですかね？

ちょっとまってくださいね。
https://github.com/winebarrel/ridgepole これも似たような雰囲気を感じますが、これは mysql 直接と比較してるのかな。

ridgepole は dsl 同士の比較っぽいですね。
それはさておき、聞いてきたところによると
mysql が吐くスキーマとこちらが手書きしたスキーマだと意図しない差分が出る可能性があるので（シノニム系の統一とかが大変）、あくまでも mysql に入っているバージョンからその時の手書きしたスキーマを取得し、比較したほうが良いのではないかという話がありました。
それはその通りだと思いました。
mysql の吐く現在のスキーマと差分が適用できるので便利そうという話もありましたが、これは schemadiff mysql /path/to/file で出したものを適用すれば良いのではないかという話をしました。
この話をして考えたことなのですが gitddl の挙動にするなら

適用する diff を表示するモード（ドライラン的な）
バージョンを記録するテーブルの名前が変えられる

あたりも必要かなぁと思いました。

repo: schemalex/schemalex


text:
Add option --webdewey and guess output format from filename

I also extended the README in branch https://github.com/gbv/mc2skos/tree/readme but not mentioning option --webdewey yet.

Codecov Report

Merging #33 into master will decrease coverage by 0.76%.
The diff coverage is 33.33%.

@@            Coverage Diff             @@
##           master      #33      +/-   ##
==========================================
- Coverage   79.09%   78.32%   -0.77%
==========================================
  Files           5        5
  Lines         684      692       +8
==========================================
+ Hits          541      542       +1
- Misses        143      150       +7

Impacted Files
Coverage Δ

mc2skos/mc2skos.py
53.23% <33.33%> (-1.69%)
: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 e8e8340...523cf91. Read the comment docs.

Thanks! I can also document the --webdewey option.

Btw. the --notes flags is mostly included because I'm not allowed to publish that for WebDewey. But for a general tool it probably makes more sense to include notes by default and have a --exclude-notes flag for the WebDewey case.

Thanks! I'd prefer --no-notes instead of --exclude_notes but nevermind.

authors: codecov-io danmichaelo nichtich

license: unlicense


text:
changed carrot to caret

Good catch! Thank you.

authors: jeromecholewa seankross

repo: seankross/the-unix-workbench


text:
Begin pulling out common code to a crate::core module, starting with request.

A problem this PR creates which Im not sure what the best approach to is, is that it moves the documentation to a common core module which is private. Which means that the useful examples on how to use the methods on RequestBuilder are hidden.
It is possible to document private items, but I dont know how to selectively document only some private items and generating docs for all private items seems quite bad.
Alternatively we could make core::request::* public which... is maybe fine? Its not supposed to be used directly since users wont have a client to sent requests using their own generic requests with but maybe its fine?

This is the first in a series of cleanup PRs. Other changes I would like to make described in #1004

With regards to the docs, perhaps a different way could work... With a macro to generate the impls instead of the generic versions in core. Then a core module also don't need to be exported...

Could you give thoughts on #1004? This is in preparation for that, after which there will be no need for type aliases at all, and the one in core (without Client generics) would be the one true one. Having separate implementations in each client type folder seems incorrect and I think having a public core module is a good thing.

Per online discussion, this will go into the next version. Ive updated the PR to point to the new version branch

Is there still interest in getting this merged? I've been looking at this subject recently and if this branch needs rebasing onto the current master branch or something I could maybe help with this.

I believe the goal of this PR is probably still desired. Basically, if we can reduce the amount of duplicated code in reqwest, that'd be a huge win! As for actually making the two APIs use the exact same types, I believe that would have some problems at first:

At the very least, it's technically a breaking change to make 2 distinct types the same type (some hypothetical impl MyTrait for reqwest::Request {} and impl MyTrait for reqwest::blocking::Request {} would break, for instance).
Also, some methods assume an async context where others assuming it can block.

So, maybe they can't easily be the same type (though maybe a 1.0 release could do that, would need to be better explored), but perhaps a macro or something that allows the methods to be defined only once?

authors: 707090 jstasiak seanmonstar

repo: seanmonstar/reqwest


text:
Expose h2 connection parameters.

Looks like config.http_builder doesn't exist... Was this meant to be against the 0.9.x branch?

Hrm, I rebased my 0.9 branch and I assumed the same thing would work on master, but it looks like the http_builder part from #606 didn't land on master ...

Hopefully that force-push will do it.

Thanks!

authors: khuey seanmonstar


text:
(try to) Improve error message upon invalid spec file

@charlotte12l : this one improves the error message upon invalid spec,as discussed last monday.

repo: serge-sans-paille/pythran


text:
fix: deploy by bucket and object bug

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

authors: slsplus yugasun

repo: serverless-components/tencent-layer


text:
Save data and current page in EndlessListFragment to instance state

Changes Unknown when pulling 54a2769c67575eca6153272930106ae9b834fe55 on enhancement/config_change into ** on master**.

authors: coveralls setiawanp

repo: setiawanp/sunny-reader


text:
fix benchmarks and add convert benchmark

probably broken only in github actions

removed from github actions

repo: shenek/streamson


text:
LinalgRefactor - clean up macro

Much cleaner! Ready to merge when travis is green

authors: OXPHOS karlnapf

repo: shogun-toolbox/shogun


text:
file icon disappeared in file tab when use material-ui

i have found the configuration.

repo: silvestreh/atom-material-ui


text:
support phpspec 5

Coverage remained the same at 100.0% when pulling 54f02f5761013c320eb8aa476a4c36be95466805 on samsonasik:phpspec5 into f710176d6a862d7ebca0c08cb56f3dd45bf4cf67 on sitrunlab:master.

authors: coveralls samsonasik

repo: sitrunlab/learn-phpspec


text:
Fix buttons

Codecov Report

Merging #43 (7cba6b0) into master (8e771bd) will decrease coverage by 0.05%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master      #43      +/-   ##
==========================================
- Coverage   21.32%   21.26%   -0.06%
==========================================
  Files          58       58
  Lines       10497    10523      +26
  Branches       98       98
==========================================
  Hits         2238     2238
- Misses       8161     8187      +26
  Partials       98       98

Impacted Files
Coverage Δ

src/clj/skylobby/fx/battle.clj
2.53% <0.00%> (-0.01%)
:arrow_down:

src/clj/skylobby/fx/map_sync.clj
1.73% <0.00%> (-0.49%)
:arrow_down:

src/clj/skylobby/fx/mod_sync.clj
2.36% <0.00%> (ø)

src/clj/spring_lobby/main.clj
14.92% <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 8e771bd...7cba6b0. Read the comment docs.

authors: codecov-commenter skynet-gh

repo: skynet-gh/skylobby


text:
EmrEtlRunner: retrieve the correct latest run ID during archive_shredded step

Capitalized ID for consistency with existing CHANGELOG entries

This is a bug right? Please always flag bugs with a description, impact and of course the label. We don't bury bugs.

authors: BenFradet alexanderdean

repo: snowplow/snowplow


text:
R97 EmrEtlRunner

@BenFradet the target branch should be the release/r9x-knossos branch right?

You can always cherry-pick them.
Anyway, I'll add #3529 first, and then change the base branch.

@chuwy updated 👍

@knservis you can cherry-pick:

EmrEtlRunner: uncompress gzipped raw files when copying to HDFS (closes #3525)
EmrEtlRunner: bump to 0.30.0 (closes #3526)
EmrEtlRunner: add ability to skip RDB Loader consistency check (closes #3529)

and close 👍

@BenFradet chery-picked to release branch.

authors: BenFradet knservis


text:
preserve trailing commas wip

On master ;)

authors: qoelet sol

repo: sol/hpack


text:
Updating gh-pages example

Would need the original source code of gh-pages to update the deps.

@bogas04 you can get that by checking out gh-pages branch of the repository.

Hi, I think it only has bundle.js file.

Yeah I just saw , will update it with unbundled files also.


text:
Do release with #15, plz

Fixed https://github.com/souporserious/react-measure/commit/1e59aac0b1a859c673ceb6aee7085eb58ee059ed

authors: avdeev souporserious

repo: souporserious/react-measure


text:
fix(entrypoint): bash error when verify is not set yes or no

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

repo: splunk/splunk-connect-for-syslog


text:
maven-compiler-plugin 3.7.0

@davidxia  the Travis build failed, but I don't think the failures are related to my change.

@sullis Thanks for your PR. Just curious what you need from the upgrade. If you rebase on master, tests should pass.

@sullis can you mention something about why this is upgraded?

obsolete.

Codecov Report

Merging #1040 into master will decrease coverage by 0.06%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #1040      +/-   ##
============================================
- Coverage     67.37%   67.31%   -0.07%
+ Complexity      774      773       -1
============================================
  Files           176      176
  Lines          3234     3234
  Branches        368      368
============================================
- Hits           2179     2177       -2
- Misses          898      899       +1
- Partials        157      158       +1

authors: codecov-io davidxia mattnworb sullis

repo: spotify/docker-client


text:
more prefix additions

rebased and merged.

authors: markpollack sobychacko


text:
introduce ts-lint

3156cbb69d599622cccbfc825309bf0cf893015d


text:
请教下，labelme标注的xml如何转为voc或者coco呢

建议先转成csv，再转过去

authors: Flyooofly spytensor


text:
[STK-256] Go build and release

@faizanahmad055 Image is available for testing. docker pull stakater/chowkidar:PR-25-1

@faizanahmad055 Yikes! You better fix it before anyone else finds out! Build 4 has Failed!

@faizanahmad055 Yikes! You better fix it before anyone else finds out! Build 5 has Failed!

@faizanahmad055 Yikes! You better fix it before anyone else finds out! Build 6 has Failed!

@faizanahmad055 Yikes! You better fix it before anyone else finds out! Build 7 has Failed!

@faizanahmad055 Yikes! You better fix it before anyone else finds out! Build 8 has Failed!

@faizanahmad055 Yikes! You better fix it before anyone else finds out! Build 9 has Failed!

@faizanahmad055 Image is available for testing. docker pull stakater/chowkidar:PR-25-10

@faizanahmad055 Yikes! You better fix it before anyone else finds out! Build 12 has Failed!

@faizanahmad055 Image is available for testing. docker pull stakater/chowkidar:PR-25-13

@faizanahmad055 Image is available for testing. docker pull stakater/chowkidar:PR-25-14

@faizanahmad055 Image is available for testing. docker pull stakater/chowkidar:PR-25-15

repo: stakater/Chowkidar


text:
Fix unit tests for Windows

Something is wrong with the build process for Windows.

This is fixed now.

repo: stan-dev/math


text:
Bug fix for existential subquery in legacy syntax

Pull Request Test Coverage Report for Build 1271

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.08%) to 77.307%

Totals

Change from base Build 1270:
0.08%

Covered Lines:
10012

Relevant Lines:
12450

💛  - Coveralls

authors: coveralls rayslxu

repo: stanford-oval/thingtalk


text:
Add additional examples to REST API for collections, tuples, and UDTs

PR #64

PR #64

Shall have to revisit the queries for tuple and udt after stargate bug fix is completed.
Awaiting resolution of: https://github.com/stargate/stargate/issues/639

https://github.com/stargate/stargate/pull/895

Add this example (it's in the postman collection) - works after index is created:
curl -s -G -L "http://localhost:8082/v2/keyspaces/users_keyspace/users" \

-H  "X-Cassandra-Token: $AUTH_TOKEN"
-H  "Content-Type: application/json"
--data-urlencode 'where={
"firstname": {"$eq": "Janesha"},
"lastname": {"$eq": "Doesha"},
"current_country": { "$eq":  [ "France", "2016-01-01", "2020-02-02" ] }
}'

repo: stargate/docs


text:
Some small tweaks to create-and-direct-fund.ts

I think this is pretty stale now, maybe we should just close it?

authors: lalexgap snario

repo: statechannels/monorepo


text:
fix(shared): throw an InvalidRequestException when request payload is not a valid JSON document #188

I've not looked at all the code details, but one thing worries me: the main code changed are in JsonLdUtils but the tests are made at the handler level (via MockkedHandler and some fake "json" routes).
You should first test the new functions in JsonLdUtils.
You can add tests at the handler level, for instance by mocking JsonLdUtils and throwing JsonXXXExceptions. Then you can test you get the correct error response. But I don't see the point of adding those fake routes?

I added the fake routes in shared module since the fix is a cross concern.
I can either:

Add tests at the handler level of entity-service, by mocking service functions to throw the expected exceptions, but this should be done in other services as well since they are concerned by the fix.
Keep the tests at the mocked handler of shared module, throw exceptions at the handler level without using JsonLdUtils functions.

I'll add tests for the new JsonLdUtils functions

authors: HoucemKacem bobeal

repo: stellio-hub/stellio-context-broker


text:
Adjustments to tests for https://github.com/stleary/JSON-java/pull/337

Thanks, will merge when https://github.com/stleary/JSON-java/pull/337 is merged.

this one is for 337, not 336.

Sorry, just assumed this was the fix for the broken increment test in 336 (branch johnjaylward-fixSpelling), but I did not actually execute the test to make sure.

#70 fixes the increment tests

Thanks, I think the comments are corrected now

I believe these tests cover what we need.

OK to merge after stleary/JSON-java#337

authors: johnjaylward stleary

repo: stleary/JSON-Java-unit-test


text:
testing a travis build step to verify CSS

DAMN IT! I'm sorry but I just don't know how to get travis/bash working the way I need to in this. Essentially the whole issue is caused by clean-css-promise deciding to override its base implementations decisions https://github.com/shinnn/clean-css-promise/blob/master/README.md#class-cleancsspromiseoptions
clean-css dangerously ignores these errors but clean-css-promise doesn't that means that the build process doesn't fail on our side but the downstream minification breaks on the consuming apps.
I have a right mind to open an issue / PR on clean-css-promise and bring it up there but I'm kinda maxed out at the moment.

Closing this until I can come up with a better solution 👍

repo: stonecircle/ember-cli-notifications


text:
Hsl

Current coverage is 100% (diff: 100%)

Merging #70 into master will not change coverage

@@           master   #70   diff @@
===================================
  Files          34    36     +2
  Lines         152   190    +38
  Methods         0     0
  Messages        0     0
  Branches        0     0
===================================
+ Hits          152   190    +38
  Misses          0     0
  Partials        0     0

Powered by Codecov. Last update 6257c31...83a4a33

authors: codecov-io nikgraf

repo: styled-components/polished


text:
Update copyright notice to 2019

Thanks!

repo: stympy/faker


text:
Libreswan 4

Thank you @skitt

This needs https://github.com/submariner-io/submariner/pull/747 to be fixed.

This needs #747 to be fixed.

What do you mean?, Isn't it ok making it part of this PR or the switching commit?

This needs #747 to be fixed.

What do you mean?, Isn't it ok making it part of this PR or the switching commit?

I mean this: https://github.com/submariner-io/submariner/pull/747#pullrequestreview-518490763

authors: mangelajo skitt

repo: submariner-io/submariner


text:
Fix marking of active heading in TOC

Fixed.

repo: substance/substance


text:
[FEATURE] Adapted test data for advanced tests

else ack

authors: notsure turbo-ele

repo: sulu-io/SuluSalesOrderBundle


text:
config in js to allow integration with webpack

closing and making new PR with keeping yml

repo: svg/svgo


text:
Added provisioning of Python 3.6 for runway & SAIO

I dunno... Maybe:

Travis CI (ProxyFS/.travis.yml)
CI (@gerardgine knows what, if anything, additional)
runway (ProxyFS/cookbooks...)
SAIO (ProxyFS/saio)
Native MacOS (although here only Go code is built/tested)
Native Linux devs (perhaps @charmster could answer)
SwiftDebBuilder (it actually disables most of the testing... just builds the .rpm)

repo: swiftstack/ProxyFS


text:
Add route speed functions

Can you rebase this with master?

Rebased

Addressed Ed's comments and rebased again.

Is the build failure unrelated?

@malban It looks like we have some flaky tests in swri_transform_util. I'll rerun the failed builds.

I'm tracking the flaky test in #468, if @pjreed or @malban is looking for a challenge.

authors: elliotjo evenator malban


text:
Update README.rst

Wait a minut, why change you the mixin order? Why after?

authors: SalahAdDin bashu

repo: takeflight/wagtail-metadata


text:
feat(useuserdata): added mocking for hook

: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: lucasoeth tsboter

repo: technologiestiftung/stadtpuls-frontend


text:
Enabling JavaScript accessible events - navigation with other keys for grid

Closing as focus is now switched towards basic content accessibility instead of building rich widgets.

repo: techvision/waiable


text:
Add BuiltinOperatorName message in debug log

Can you provide a sample of what the printed statement looks like (before and after)?

Can you provide a sample of what the printed statement looks like (before and after)?

Can you provide a sample of what the printed statement looks like (before and after)?

before :
Node   0 Operator Builtin Code  36
  Inputs: 20 23
  Outputs: 19
Node   1 Operator Builtin Code  22
  Inputs: 19 3
  Outputs: 2
Node   2 Operator Builtin Code   3
  Inputs: 2 4 1
  Outputs: 0
Node   3 Operator Builtin Code  39
  Inputs: 0 11
  Outputs: 10
Node   4 Operator Builtin Code  22
  Inputs: 10 6
  Outputs: 5
Node   5 Operator Builtin Code  48
  Inputs: 5 8
  Outputs: 7 9
....

after :
Node   0 Operator Builtin Code  36 GATHER
  Inputs: 20 23
  Outputs: 19
Node   1 Operator Builtin Code  22 RESHAPE
  Inputs: 19 3
  Outputs: 2
Node   2 Operator Builtin Code   3 CONV_2D
  Inputs: 2 4 1
  Outputs: 0
Node   3 Operator Builtin Code  39 TRANSPOSE
  Inputs: 0 11
  Outputs: 10
Node   4 Operator Builtin Code  22 RESHAPE
  Inputs: 10 6
  Outputs: 5
Node   5 Operator Builtin Code  48 TOPK_V2
  Inputs: 5 8
  Outputs: 7 9
...

authors: Mycaster alanchiao

repo: tensorflow/tensorflow


text:
Show a source tab on the schema and (document) schemas pages as well as the document page

This should really be jump into the source at a given path, which I need for other things.

repo: tetherless-world/kairos-schema-tools


text:
update README.md

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@a1d1aac). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             master   #187   +/-   ##
=======================================
  Coverage          ?   100%
=======================================
  Files             ?      4
  Lines             ?     45
  Branches          ?      5
=======================================
  Hits              ?     45
  Misses            ?      0
  Partials          ?      0

Continue to review full report at Codecov.

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

authors: codecov-io theDavidBarton


text:
typo 'the The'

Coverage remained the same at 90.802% when pulling dc5e2a8352fce2c9656bbfcdbb2744dee6d8b295 on rdworth:patch-1 into 8a047005788aaae89bc0d2d93b15e6899e7e2ddd on theory:master.

authors: coveralls rdworth

repo: theory/sqitch


text:
Add assertfatal

LGTM :)

By the way, I don't think assert.EqualAndFatal has any ambiguity.
For example, Node.js assert.equal(actual, expected[, message]) https://nodejs.org/api/assert.html#assert_assert_equal_actual_expected_message ,
assert equal mean if not equal then fatal, not if equal then fatal.
For Ruby, https://ruby-doc.org/stdlib-2.1.1/libdoc/test/unit/rdoc/Test/Unit/Assertions.html#method-i-assert_equal

In my understanding, function in assert package is to assert the actual and the expected meet some conditions just as the function name, like equal, notEqual, bigAndRed. So assert.EqualAndFatal really confused me, but to think it as assert A and B are equal, if they are, then fatal.

For "assert a and b are equal", mean I expect a and b are equal, so if they are not equal then fatal, it's true in any language,

Let's follow other famous packages names.
Like Linux commands that -r -f -d -l, you only need to study ones and guss what them meaning.

Let's follow other famous packages names.
Like Linux commands that -r -f -d -l, you only need to study ones and guss what them meaning.

But in assert package, you don't need to guess, it's clear.

assert.IfEqualThenFatal() is more confused, I guess you like if.IfEqualThenFatal().

@tnclong the new package assertfatal is ok?

if

if is a keyword.

the new package assertfatal is ok?

You mean we have both of assertfatal and assert packages ?
Do you think it will worse than current only have one package?

authors: lifenod tnclong xuxinx

repo: theplant/testingutils


text:
[app] vscode latestversion doesn't work -> can't be installed

done

repo: thetillhoff/eac


text:
VS 2015 Support

Thanks @alex-doe, I've merged your PR. I will soon publish a new release.

authors: alex-doe thoemmi

repo: thoemmi/Solutionizer


text:
fix unavailable "GitHub Top 10 Best Practices"

@tiimgreen maybe the link doesn't have to be removed, but replaced with https://www.datree.io/resources/github-best-practices or https://try.datree.io/github-best-practices-guide

I'm happy to merge if you add the datree link.

I readded the updated link

authors: djbrown tiimgreen

repo: tiimgreen/github-cheat-sheet


text:
Nbval

Codecov Report

Merging #17 into master will decrease coverage by 0.29%.
The diff coverage is 14.28%.

@@            Coverage Diff            @@
##           master      #17     +/-   ##
=========================================
- Coverage   23.41%   23.12%   -0.3%
=========================================
  Files           5        5
  Lines         316      320      +4
=========================================
  Hits           74       74
- Misses        242      246      +4

Impacted Files
Coverage Δ

jupyterlab_celltests/tests.py
15.45% <14.28%> (-0.59%)
: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 3fc5b76...23b021c. Read the comment docs.

authors: codecov-io timkpaine


text:
added @mention feature

It works only group topics like Whatsapp

Overall this PR seems to be dependent on https://github.com/tinode/tindroid/pull/73
Please follow these rules:
One feature - one PR.
Not two features - one PR. Not one feature - two PRs.
One feature - one PR.
Thanks!

authors: Kuvonchbeky or-else

repo: tinode/tindroid


text:
Use PerformanceTestTools

Codecov Report

Merging #210 into master will decrease coverage by 5%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master    #210      +/-   ##
=========================================
- Coverage   92.41%   87.4%   -5.01%
=========================================
  Files          19      19
  Lines        1279    1199      -80
=========================================
- Hits         1182    1048     -134
- Misses         97     151      +54

Impacted Files
Coverage Δ

src/interop/dataframes.jl
0% <0%> (-100%)
:arrow_down:

src/basics.jl
35% <0%> (-50.72%)
:arrow_down:

src/core.jl
72.43% <0%> (-19.03%)
:arrow_down:

src/simd.jl
78.12% <0%> (-18.85%)
:arrow_down:

src/reduce.jl
79.74% <0%> (-13.03%)
:arrow_down:

src/lister.jl
80.48% <0%> (-7.32%)
:arrow_down:

src/unordered.jl
93.02% <0%> (-3.14%)
:arrow_down:

src/comprehensions.jl
81.81% <0%> (-0.54%)
:arrow_down:

src/library.jl
93.75% <0%> (-0.5%)
:arrow_down:

src/show.jl
94.87% <0%> (-0.17%)
:arrow_down:

... and 5 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 451e911...172a462. Read the comment docs.

authors: codecov-io tkf


text:
Add Settings

LGTM.

authors: juvester mikkomaa

repo: tmc-cli/tmc-cli


text:
Add the excellent UWP Community Toolkit

I don't know how I managed to forget to add this already... thanks for the PR! :+1: :)

authors: jonstodle tomzorz

repo: tomzorz/awesome-uwp


text:
Templated naming and doc fixes

Note I haven't run through the entire workshop again after templating the code


text:
~painfully~ hopefully got rid of all instances of ActiveAdmin. Yeesh

Gets rid of #11 . Closed.


text:
Intermediate workshop on GNU awk

Here's the awk lesson I was talking about - feel free to use, adapt, or write your own!

here is a list of one line awk commands that can be useful (picked from github)
http://coderscrowd.com/app/public/codes/view/163

authors: BillMills brunogrande radaniba

repo: ttimbers/studyGroup


text:
support lambda functions in MetricExtractor

This has a few limitations, e.g. that lambdas requiring modules other than numpy and scipy (i.e. those imported in the misc extractors module) cannot be evaluated.

Coverage decreased (-4.8%) to 72.7% when pulling ed5d026e31e35895a8e71c497729c618a65df43d on rbroc:restore_lambda_metricext into eb8b855a31f02c8deaa558d4b257061487ec389e on tyarkoni:master.

My main concern here now that I've thought about it some more, is that it's a bit dangerous to allow arbitrary evaluation of strings. I'll look into it.

We're already allowing callables to be passed, so I don't think eval() introduces any new security risks. Anything malicious you could do in a string you could also do inside a callable. I think it's the user's responsibility not to run pliers in an insecure environment.

Allright lets do it then!

authors: adelavega coveralls rbroc tyarkoni

repo: tyarkoni/pliers


text:
-threaded option should turn on threading in the RTS

Done in https://github.com/typelead/eta/commit/025bd875664a2ee44d0a94ab5149ed2b0ace4855

repo: typelead/eta


text:
Fix readme

What needs fixing?

repo: types/npm-bcrypt


text:
Add fontPens to extras

@anthrotype like so?

I think so. Can you do
pip install -e .[pens]
and get fontPens as well as defcon and the other deps?

@anthrotype yes, that works!

(this pip install defcon[pens] extras should probably also be documented somewhere I guess)

@anthrotype changed setup.py and added to the documentation.

@anthrotype can you merge this, I don't have commit rights to defcon. Thank you!

I don't have commit rights to defcon

ping @typesupply ?

authors: anthrotype benkiel

repo: typesupply/defcon


text:
Fix merge conflict between PRs #300 and #295 (last two commits)

Just going to go ahead and merge this fix myself as soon as CI is green, since NullAway master actually ended up red (both commits passed CI and merged cleanly, but CI didn't pass after applying both together and GitHub doesn't check that for merging... basically, GitHub needs SubmitQueue ;) )

repo: uber/NullAway


text:
Links in readme were broken

👍

authors: Apercu mcnuttandrew

repo: uber/react-vis


text:
upload modifications in test redoCancellation

Pull Request Test Coverage Report for Build 87

30 of 33 (90.91%)  changed or added relevant lines in 8 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.6%) to 98.089%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

repository/IngSoft2-Model/Repeat.class.st
0
3
0.0%

Totals

Change from base Build 85:
-0.6%

Covered Lines:
462

Relevant Lines:
471

💛  - Coveralls

Pull Request Test Coverage Report for Build 87

30 of 33 (90.91%)  changed or added relevant lines in 8 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.6%) to 98.089%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

repository/IngSoft2-Model/Repeat.class.st
0
3
0.0%

Totals

Change from base Build 85:
-0.6%

Covered Lines:
462

Relevant Lines:
471

💛  - Coveralls


text:
Entrega tarea 5

Muy bien pensado jugar con la carga de J para ver cómo su efecto igualaba o superaba en importancia a la línea. Es genial que se den el tiempo de investigar con los juguetes que crean.
Perfecto el pep8.
Las implementaciones posibles para mejorar el sistema están muy bien explicadas y serían efectivamente buenas ampliaciones del problema.
Las figuras explican muy bien el texto que acompañan. Muy buen informe.

authors: JoseGuillermoAraya Maquox


text:
Release

Codecov Report

Merging #144 into master will decrease coverage by -0.87%.
The diff coverage is 83.69%.

@@            Coverage Diff            @@
##           master    #144      +/-   ##
=========================================
- Coverage   94.37%   93.5%   -0.87%
=========================================
  Files          31      33       +2
  Lines        1048    1140      +92
  Branches       45      55      +10
=========================================
+ Hits          989    1066      +77
- Misses         49      62      +13
- Partials       10      12       +2

Impacted Files
Coverage Δ

datahub/core/fields.py
100% <100%> (ø)

datahub/company/views.py
100% <100%> (ø)
:white_check_mark:

datahub/core/utils.py
81.57% <40%> (-6.3%)
:x:

datahub/company/management/commands/sync_ch.py
84% <84%> (ø)

Continue to review full report at Codecov.

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

authors: canni codecov-io

repo: uktrade/data-hub-leeloo


text:
Update Vocalink dataset to v5.90 (effective 3 February 2020)

Deprecated by https://github.com/uphold/uk-modulus-checking/pull/23.
Sorry for the delay @raybotha and thanks for the contribution.

authors: franciscocardoso raybotha

repo: uphold/uk-modulus-checking


text:
Add fingerprint for ~leb

This can probably be merged despite CI, since just adding a newline to %ames(or presumably any other vane) hits the same issue.

Thanks!

authors: famousj ohAitch

repo: urbit/arvo


text:
Add Node 7 tests

Coverage remained the same at 93.243% when pulling be1f3890a42674d238abe3ef2e250838d49a690c on alippai:patch-3 into 5014cad15cba26db3bd7c4ae395ebaad32955978 on urish:master.

Coverage remained the same at 93.243% when pulling be1f3890a42674d238abe3ef2e250838d49a690c on alippai:patch-3 into 5014cad15cba26db3bd7c4ae395ebaad32955978 on urish:master.

Coverage remained the same at 93.243% when pulling be1f3890a42674d238abe3ef2e250838d49a690c on alippai:patch-3 into 5014cad15cba26db3bd7c4ae395ebaad32955978 on urish:master.

Thanks!

authors: alippai coveralls urish

repo: urish/angular-moment


text:
add APRS message encoder

This has been done.

authors: stcarlso usaguerrilla

repo: usaguerrilla/hab


text:
Template

Let's use the template branch for this evaluation.
We'll merge template branch into main after the evaluation.

wrong branch

authors: kyao liangmuxin


text:
Fix typo in 09.functional-components.js

@davidfurlong Thank you :)

authors: davidfurlong vasanthk

repo: vasanthk/react-bits


text:
return total pages

i don't see the reason why we need it

developer can calculate total amount of pages without react-js-pagination help

authors: anderpo vayser

repo: vayser/react-js-pagination


text:
Include DLite

I was going to add it, then I saw you pull request....  (  ^ - ^ ')
Nice job and thanks for the contribution :smile:

authors: mfburnett veggiemonk

repo: veggiemonk/awesome-docker


text:
Add expired time factor.

    def _format_nebula_command(self, name, meta_port, ports, debug_log=True):
        param_format = "--meta_server_addrs={} --port={} --ws_http_port={} --ws_h2_port={} --heartbeat_interval_secs=1"

Is this code still working? If it is, it seems we don't need to change it. Just unify all heartbeat_interval_secs in ci conf.

    def _format_nebula_command(self, name, meta_port, ports, debug_log=True):
        param_format = "--meta_server_addrs={} --port={} --ws_http_port={} --ws_h2_port={} --heartbeat_interval_secs=1"

Is this code still working? If it is, it seems we don't need to change it. Just unify all heartbeat_interval_secs in ci conf.

We need to expire based on heartbeat time.

what is expired_time_factor mean? will using heartbeat_interval_secs * expired_time_factor to determine timeout ?

Yes.

heartbeat_interval_secs

The ci fail, probably because meta/storage heartbeat_interval_secs is not unified.

--heartbeat_interval_secs=1

The ci fail, probably because meta/storage heartbeat_interval_secs is not unified.
heartbeat_interval_secs * 2has been used quite a looooog time, since 1.0.

The heartbeat config in ic is same for meta/storage.
I see the origin expired value in meta

DEFINE_int32(expired_threshold_sec, 10 * 60,
             "Hosts will be expired in this time if no heartbeat received");

I think it's 600s now it's heartbeat*2 (2s in ci in fact).

what is expired_time_factor mean? will using heartbeat_interval_secs * expired_time_factor to determine timeout ?

Yes.

What is the benefit for this ？

what is expired_time_factor mean? will using heartbeat_interval_secs * expired_time_factor to determine timeout ?

Yes.

What is the benefit for this ？

Could fit the environment (e.g. If running on slow environment the user could set a big expired_time_factor to avoid mistake)

expired_threshold_sec

The root cause is heartbeat_interval_secs in meta is 1s, and storage's heartbeat_interval_secs is not 1s. And that will make hosts lost.

authors: Shylock-Hg critical27 darionyaphet

repo: vesoft-inc/nebula-common


text:
tooltip 可以通过js直接展示吗？

https://github.com/viserjs/viser/issues/368 试试这个

No description provided.

这里有个详细的
https://viserjs.gitee.io/demo.html#/viser/components/tooltip-customized

authors: huxiaoyun kinghoo5201 qianyingoo

repo: viserjs/viser


text:
What is the test case on the readme?

The test case 0 seems to be different in my computer (locations of humans is different).
What one did you use?

Hi Tessa,
That command is just a description of how to run the code and doesn't correspond to the navigation demo below.
And sorry, I don't remember the case number of the navigation demo. But if you generate all of them, you should be able to find it.
Changan

authors: ChanganVR tessavdheiden

repo: vita-epfl/CrowdNav


text:
Finish 1st iteration task plan on all estimated tasks wiki page.

Commit [336c822] (https://github.com/vladislavivanistsev/SuperCap/wiki/All-estimated-tasks/336c822ef2774d85fe548f904cec9f561a873fcd)
[Up-to-date page] (https://github.com/vladislavivanistsev/SuperCap/wiki/All-estimated-tasks)

Work includes: getting to know what is required and what is agreed upon in 1st iteration.
Estimated: 2 man-hours

repo: vladislavivanistsev/SuperCap


text:
feature / travis windows setup

Too unstable at the moment.

repo: vladjerca/FFMpegCore


text:
fix typo and refine message in content trust interceptor

Coverage decreased (-0.03%) to 56.51% when pulling a905d72c5981910c9da1caba6f3e543d46471050 on reasonerjt:comment-in-proxy into 3586090bf6536ab5eb8ba8acfb8011f4c7860bb0 on vmware:master.

authors: coveralls reasonerjt

repo: vmware/harbor


text:
Update Chrome 76 result

Hi @danyao
Is there something missing from the wpt.fyi dashboard for payment request that you would need to submit this manually?

Hi @NavidZ, I'm updating this manually because we need the implementation report in this directory to move the Payment Request API forward to CR.

authors: NavidZ danyao


text:
fix countLineBreaks that not count new lines properly

can you describe the problem?

See here http://jsfiddle.net/ec6jfy2b/4/
When enter pressed or shift-enter ("br" in html) - it not counted like valid char
Sometimes when you place enter in text - it counted one time only, even you put more enters there.
In the end of text "br"s not counted too.

Okay thanks i add, it but i think it will cause some problems


text:
get_k_data 接口只有2015年之后的数据

设置开始日期有全部的

authors: jimmysoa qinkevin

repo: waditu/tushare


text:
Fix search in People snippet. Fixes #205

Thanks!

authors: kaedroho paolodina

repo: wagtail/bakerydemo


text:
awless list listeners does not include target groups

@moshloop The listing of the listener will not include its targetgroups. It seems the AWS API sees listeners as an intermediary components sitting between loadbalancers and targetgroups and that there is no obvious API call to link the two directly.
But using the awless show command, you can have the full tree of relations for the 3 components starting from the load balancer. Run:
awless show name_of_my_loadb

(note: you can add the local flag -l allowing offline exploration of resources using the local storage previoulsy synced.)
In the lineage section, you will see the listeners attached to this loadb. In the applied on, section you will see the targetgroups it applies on.

I am ultimately trying to get the target group ARN from a loadbalancer but I
don't see a way of doing that  - the tree might display it, but I can't get just
the ARN
The AWS cli does return it: aws elbv2 describe-listeners --load-balancer-arn
{    "Listeners": [        {            "ListenerArn":
"arn:aws:elasticloadbalancing:eu-central-1:xxxb5adde7",
            "LoadBalancerArn":
"arn:aws:elasticloadbalancing:eu-central-1:xxxx2a",            "Port": 80,
            "Protocol": "HTTP",            "DefaultActions": [                {
                    "Type": "forward",                    "TargetGroupArn":
"arn:aws:elasticloadbalancing:eu-central-1:xxxf8b02"                }
            ]        },        {            "ListenerArn":
"arn:aws:elasticloadbalancing:eu-central-1:x59810",
            "LoadBalancerArn": "arn:aws:elasticloadbalancing:eu-central-1:3xa",
            "Port": 443,            "Protocol": "HTTPS",
            "Certificates": [                {
                    "CertificateArn": "arn:aws:acm:eu-central-1:xfc8c"
                }            ],            "SslPolicy":
"ELBSecurityPolicy-2016-08",            "DefaultActions": [                {
                    "Type": "forward",              "TargetGroupArn":
"arn:aws:elasticloadbalancing:eu-central-1:xxxx:targetgroup/xxxxx/xxxxx"
                }            ]        }    ]}
On Thu, Feb 1, 2018 3:22 PM, Simon Caplette notifications@github.com  wrote:
@moshloop  The listing of the listener will not include its targetgroups. It
seems the AWS API sees listeners as an intermediary components sitting between
loadbalancers and targetgroups and that there is no obvious API call to link the
two directly.
But using the awless show  command, you can have the full tree of relations for
the 3 components starting from the load balancer. Run:
awless show name_of_my_loadb
(note: you can add the local flag -l  allowing offline exploration of resources
using the local storage previoulsy synced.)
In the lineage section, you will see the listeners attached to this loadb. In
the applied on, section you will see the targetgroups it applies on.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

I will see how if I can implement a way to get this information in the listing and mimic what the aws-cli does.
In the meantime (and a bit more cumbersome) to get the targetgroup ARN from a loadbalancer you can do:

awless show my-loadbalancer-name
Look at the section Applied on and grab the name of any targetgroup you want in there.
Then with the name from above do awless show my-target-group-name -l

@moshloop Fixed on master.
I have added the TargetGroups column in the listing of listeners. I display the short ARN for this column since their could be multiple TargetGroup for a listener.
Let me know is that is ok and fixes the issue for you. Thanks.

@simcap awesome - thanks, will test and let you know if their are any issues

authors: moshloop simcap

repo: wallix/awless


text:
Added wanbridge article

✓

repo: wanchain/explore-wanchain


text:
UI Architecture

Decide on the UI architecture and document it in the WEP

What is the status on this one?

Any update on this?

@palemtnrider says we are close for this.  Much more description, which needs to be documented.

@jpellizzari and @palemtnrider can we close this out now that we are aligned on how to move forward with this?

authors: JamWils markramm

repo: weaveworks/weave-gitops


text:
Remove deferred providers (?)

Nope

repo: webignition/php-basil-parser


text:
V7

Coverage increased (+0.01%) to 97.506% when pulling 26e9d1ea9cce8318b7940771ee1efa94d3a993ee on v7 into 315bf4bee040aa243eaf3b2a418741d35444bccb on master.

authors: coveralls wheresrhys

repo: wheresrhys/fetch-mock


text:
dimensionality reduction script

it'd be nice to have a test too :)

Oh shoot, I'll get on that
On Thu, Dec 6, 2018, 4:14 PM jao <notifications@github.com wrote:

it'd be nice to have a test too :)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/whizzml/examples/pull/69#issuecomment-445077420, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAws18_HBINcU_8dtsSUkV3-ZTAS3P9Aks5u2bLqgaJpZM4ZHiH-
.

@cheesinglee one more thing before merging: we need entries for the script at the top level's metadata.json and readme.md

top-level info added

authors: cheesinglee jaor

repo: whizzml/examples


text:
SWARM-1299: If requested, resolve dependencies with Gradle non-transi…

Can one of the admins verify this patch?

Can one of the admins verify this patch?

ok to test

repo: wildfly-swarm/wildfly-swarm


text:
#3 - implemented pay method

Coverage remained the same at 100.0% when pulling 3f29ffd364a740283b39d70aebd0defbc5b64b4c on 03-pay into bbcea6c3825a247a9fa12c4a6f4c2bf2c98525cb on 03-paypal-init.

Coverage remained the same at 100.0% when pulling 0558f0696c0ee978a7f009016504f9e8b55c98ed on 03-pay into bbcea6c3825a247a9fa12c4a6f4c2bf2c98525cb on 03-paypal-init.

Coverage remained the same at 100.0% when pulling 42b6672e49e23156946d64d0734a13055d74bb52 on 03-pay into bbcea6c3825a247a9fa12c4a6f4c2bf2c98525cb on 03-paypal-init.

authors: coveralls horfic

repo: wirecard/paymentSDK-php


text: add logging to built/unbuilt filter

repo: wix/lerna-script


text:
Added scm of forked repo

Closed, will merge when needed


text:
Make shortcuts help modal searchable

Fixed in 4efd868e9208c3154f230824b98f67b05c02b239


text:
update to RTD

Codecov Report

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

@@           Coverage Diff           @@
##           master      #68   +/-   ##
=======================================
  Coverage   95.12%   95.12%
=======================================
  Files           2        2
  Lines         698      698
  Branches      121      121
=======================================
  Hits          664      664
  Misses         18       18
  Partials       16       16

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 c6aa839...ec761de. Read the comment docs.

authors: codecov-io roxyboy

repo: xgcm/xrft


text:
Added CORS, fixed docs

Thank you for submitting this pull request, however I do not see a valid CLA on file for you. Before we can merge this request please visit https://yahoocla.herokuapp.com/ and agree to the terms. Thanks! :smile:

authors: rcocetta yahoocla

repo: yahoo/mockaccino


text:
Support default currying

特にオモチャ以上の旨味がないのと、現在の仮想マシンの実装から離れすぎてるのと、別に面白いものを見つけたのでclose。

repo: yamacir-kit/meevax


text:
ZkClient should close when the session is opened.

LGTM, pending CI

thanks, merged to master

authors: xjzrc yaooqinn

repo: yaooqinn/kyuubi


text:
support other eslint configuration

[ ] support parserOptions

[ ] ecmaFeatures - jsx, globalReturn
[ ] ecmaVersion
[ ] sourceType

repo: yeonjuan/typescript-eslint-demo


text:
Create verify.yml

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

repo: yousifalraheem/rollup-plugin-summary


text:
lock the cache

LGTUs (see comments)

authors: qoelet tuleism

repo: zalora/tinc


text:
Bluetooth: controller: Controller-based privacy progress

@cvinayak @jhedberg commits updated

@jhedberg @cvinayak updated

repo: zephyrproject-rtos/zephyr


text:
Removed logback.xml hack without using test/resources

Codecov Report

Merging #287 into develop will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           develop      #287   +/-   ##
=========================================
  Coverage   99.778%   99.778%
=========================================
  Files           37        37
  Lines         3162      3162
  Branches         7         7
=========================================
  Hits          3155      3155
  Partials         7         7

authors: anthony-khong codecov-io

repo: zero-one-group/geni


text:
"Flip for reflection" does nothing

Seems to work well in the scroller, just needs to do the same in the preview

when the content is flipped, it should start at "the end", meaning the beginning of the content and scrolling should probably be reversed, too, so that when you use it in a mirror, it looks like it's the right side up

done, needs to be tested

authors: oittinen zilahir

repo: zilahir/teleprompter


text:
Use MAX_DIST with s->w_size instead of s->window_size in deflate_medium.

Similar types of checks found at:
https://github.com/zlib-ng/zlib-ng/blob/06173b6a49ae4a331758e009885e1aa1829cdd19/deflate_slow.c#L60
https://github.com/zlib-ng/zlib-ng/blob/06173b6a49ae4a331758e009885e1aa1829cdd19/deflate_medium.c#L218
https://github.com/zlib-ng/zlib-ng/blob/06173b6a49ae4a331758e009885e1aa1829cdd19/deflate_fast.c#L50

I'm not confident on this one.

repo: zlib-ng/zlib-ng

license: Zlib


text:
New PR to test SignCLA at 2018-08-09 16:37:14.994643574 -0400 EDT m=+5.140424498

Body of PR

corp-pass-staging: test succeeded

repo: google-test/signcla-probe-repo-dev