text:
Log when all versions are ignored

Ehh not sure about this approach, it looks like call these version methods a lot so end up with duplicate logs:
=== numpy (1.11.0)
 => checking for updates 1/1
I, [2021-05-05T17:38:20.414118 #98032]  INFO -- : All versions for numpy were ignored
 => latest available version is
I, [2021-05-05T17:38:21.329798 #98032]  INFO -- : All versions for numpy were ignored
 => latest allowed version is 1.11.0
I, [2021-05-05T17:38:21.438065 #98032]  INFO -- : All versions for numpy were ignored
I, [2021-05-05T17:38:21.553390 #98032]  INFO -- : All versions for numpy were ignored
I, [2021-05-05T17:38:21.695491 #98032]  INFO -- : All versions for numpy were ignored
I, [2021-05-05T17:38:21.811033 #98032]  INFO -- : All versions for numpy were ignored
 => requirements to unlock: update_not_possible
 => requirements update strategy: bump_versions
    (no update possible 🙅♀)

Also for some package managers (python) the latest_version becomes nil if we filter out lower versions 🤦♂

repo: dependabot/dependabot-core


text:
Empty state for no posts

Resolved by #178

source: gharchive/issue

repo: devonzuegel/clarity

license: mit

license_type: permissive

license_source: bigquery


text:
New/buy crypto page

@Alexintosh this should work now

source: gharchive/pull-request

repo: dexlab-io/DexWallet-website

license: MIT

license_source: github-api


text:
Add optional read timeout for requests

Thanks!

authors: aromaticum dfm


text:
Adaptive Icons for Android 8

Just a note that Android Studio can help generate the new adaptive icons (File / New / Image Asset).

Please add support for icon shape
https://developer.android.com/guide/practices/ui_guidelines/icon_design_adaptive.html
https://material.io/guidelines/style/icons.html

you should fix it to upload new version to google play
https://www.xda-developers.com/google-play-app-updates-must-target-android-8-0/

Today I've started to roll out the new version (with adaptive icons). Unfortunately I had to stop it because someone reported an issue. I'll check that and release a hotfix if necessary.

Closing this since it has finally been implemented.

authors: c-e-p-x-u-o dmfs lemonboston

repo: dmfs/opentasks


text:
updated readme

Pull Request Test Coverage Report for Build 3

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 50.0%

Totals

Change from base Build 2:
0.0%

Covered Lines:
2

Relevant Lines:
4

💛  - Coveralls

authors: Davide-Gheri coveralls

repo: docomodigital/pdor-typescript-library


text:
Infer IsDelete and IsCreate from Before/After fields

Codecov Report

Merging #63 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #63      +/-   ##
==========================================
+ Coverage   98.36%   98.36%   +<.01%
==========================================
  Files          44       44
  Lines        2502     2505       +3
==========================================
+ Hits         2461     2464       +3
  Misses         36       36
  Partials        5        5

Flag
Coverage Δ

#dot
98.36% <100%> (ø)
:arrow_up:

#dot_changes
100% <100%> (ø)
:arrow_up:

#dot_conv
100% <ø> (ø)
:arrow_up:

#dot_encoding
90.05% <ø> (ø)
:arrow_up:

#dot_encoding_counter
74.41% <ø> (ø)
:arrow_up:

#dot_encoding_richtext
100% <ø> (ø)
:arrow_up:

#dot_encoding_set
100% <ø> (ø)
:arrow_up:

#dot_encoding_sparse
100% <ø> (ø)
:arrow_up:

#dot_fold
100% <ø> (ø)
:arrow_up:

#dot_ops
100% <ø> (ø)
:arrow_up:

#dot_refs
100% <ø> (ø)
:arrow_up:

#dot_test_seqtest
100% <ø> (ø)
:arrow_up:

#dot_x_rt
100% <ø> (ø)
:arrow_up:

#dot_x_types
100% <100%> (ø)
:arrow_up:

Impacted Files
Coverage Δ

x/types/array.go
100% <100%> (ø)
:arrow_up:

x/types/map.go
100% <100%> (ø)
:arrow_up:

x/types/string.go
100% <100%> (ø)
:arrow_up:

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

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

x/types/counter.go
100% <100%> (ø)
:arrow_up:

changes/atomic.go
100% <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 a004c63...b022ca8. Read the comment docs.

authors: codecov-io rameshvk

repo: dotchain/dot


text:
fixed metadata

Previous build failed; closing and reopening to begin new build.

I've actually noticed that we had a few more fields to fix @rpetrusha.

authors: mairaw rpetrusha

repo: dotnet/docs


text:
size page better for mobile / etc

Basically solved by #14.

repo: dougalsutherland/chips-with-friends


text:
Need IndexOf to safely interact with nodeList

Fixed with #3

repo: drognisep/fynetree


text:
[DROOLS-1261] browse trait class hierarchy when creating a mixin

Merged by https://github.com/droolsjbpm/drools/commit/e7a8aaac83ea0e1549d3ebcfa62ad18ea50fa528

repo: droolsjbpm/drools


text:
[6.5.x] RHBPMS-4547 - Objects in a List is not correctly unmarshalled by JSONMarshaller

Jenkins retest this

Jenkins retest this

Jenkins retest this

repo: droolsjbpm/droolsjbpm-integration


text:
Add "Mandatory Features" page

I reworked this a bit so closing the PR but if have further updates after the changes don't hesitate to send a PR.

authors: GDaw sylus

repo: drupalwxt/website


text:
Add ups.com

Thanks. I don’t think 7-26 characters is one I’ve seen before.

@duffn I just now noticed there is an extra + on line 1099. Which breaks the formatting of the table of contents. Do you want me to create a separate PR to fix this?

@Yannickvdv Please, that would be great.

@duffn https://github.com/dumb-password-rules/dumb-password-rules/pull/248

authors: Yannickvdv duffn

repo: dumb-password-rules/dumb-password-rules


text:
Tree

done

repo: eBaoTech/parrot


text:
Colombia document should not be mandatory, leave it to pay

The problem is, if we leave it to pay the error will be

repo: ebanx/woocommerce-gateway-ebanx


text:
Add some stuff for SVG icons

Can't we just inline the svg icons we use? Font files are large D:

authors: btkostner danrabbit

repo: elementary/appcenter-dashboard


text:
Small bunch of fixes

Probably just me, but the levelHigher levelLower methods still need better names.  levelHigher(3) is true for 3 ... which is not higher but equal, etc.  I get the intent, but since it is for readability, perhaps a better name still ?

Yeah... I had similar thoughts, but I was not able to find anything that wasn't levelHigherOrEqualThan that seems a bit of the longish side... :confounded:

maybe minLevel and maxLevel ... not sure those make sense either

Could make sense, I'll give it a try!
BTW: do not merge this branch until #2673 is verified and resolved, I would avoid changing the cache two times when it's broken.

Caught in testing hell? Ima fix that up for ya...
https://www.youtube.com/watch?v=k9Ylu0YywoA

Something very like that. LOL

Looks like some shock therapy https://www.youtube.com/watch?v=nG2DwL3MCT4

Such a long time I didn't see this episode!! LOL

maybe minLevel and maxLevel ... not sure those make sense either

Let me see if this makes sense:
function minLevel($val) {...}

I would expect the function returns true if the current level is at least $val:
$cache->setLevel(2);
$cache->minLevel(1); // <= true
$cache->minLevel(3); // <= false

And the opposite:
function maxLevel($val) {...}

should give:
$cache->setLevel(2);
$cache->maxLevel(1); // <= false
$cache->minLevel(3); // <= true

What do you think?

seems more readable to me.

I'm spending a lot of time trying to understand what I should put (if one or the other) in each case.
And if I'm having difficulties it means others will have as well... :-\
What would you think of something that reminds version_compare?
function levelCompare($val, $operator = '')
{
	if (empty($operator))
	{
		return version_compare($this->level, (int) $val);
	}
	else
	{
		return version_compare($this->level, (int) $val, $operator);
	}
}

That would work as well ... do we even need anything other than min level anyway?

In theory we shouldn't need it, but I'm confused right now... lol
But... if we need just one, I would then use a levelLowerThan and make it a strict comparison. I feel it's the most readable and is not that long.

repo: elkarte/Elkarte


text:
check for the default username / password any require this be changed before allowing anything else to work

this could take place in the #5

repo: elmsln/haxcms


text:
How can i use the http api without username and password?

It is recommended that your provide a more detailed description

You cannot.

authors: emqplus fanyebo huangdan


text:
The value of SSL connection strict mode should be true by default

Fixed on https://github.com/emqx/MQTTX/releases/tag/v1.5.0

Fixed on https://github.com/emqx/MQTTX/releases/tag/v1.5.0

repo: emqx/MQTTX


text:
How can I test the pre-trained model with less images? Where do I have to change the input?

have you sort this problem?

authors: maxlchina pranavteddu

repo: endernewton/tf-faster-rcnn


text:
Update latest versions of Python 3.8 and 3.9

Thank you for your first PR and the support of the project. I have verified your changes on the website and everything is correct.

Thank you for your first PR and the support of the project. I have verified your changes on the website and everything is correct.

authors: BiNZGi CrafterKolyan


text:
add missing Arabic locales for draft-07

Coverage remained the same at 90.347% when pulling f9db4b922501a96cf24de6fa1c13cdc1d647f0a1 on Mahmoud-Mardeni:master into d579b4fef5b3571af644a7a657d8d6cec1497cc3 on epoberezkin:master.

authors: Mahmoud-Mardeni coveralls


text:
catch up to NMA

done via command line

repo: erlichlab/course-content


text:
jsonrpc tests

@heikoheiko we can use https://github.com/ethereum/rpc-tests
git clone https://github.com/ethereum/rpc-tests
cd rpc-tests
git submodule --init

rm -rf /tmp/rpctests ; pyethapp -d /tmp/rpctests -l :info,eth.chainservice:debug,jsonrpc:debug -c jsonrpc.listen_port=8081 -c p2p.max_peers=0 -c p2p.min_peers=0 blocktest lib/tests/BlockchainTests/bcRPC_API_Test.json RPC_API_Test & sleep 60 && make test

Current stats (after merging #98):

529 passing (5s)
34 failing

One of the fails is due to pyethapp not creating a default account. (1) eth_accounts IPC should return an array with accounts:)

  1) eth_accounts PYTHON should return an array with accounts:
  2) eth_call PYTHON calling getBool (0x12a7b914) should return "0x", when using a block (0) where no contract is deployed:
  3) eth_call PYTHON calling getBytes32 (0x1f903037) should return "0x", when using a block (0) where no contract is deployed:
  4) eth_call PYTHON calling getUint8 (0x343a875d) should return "0x", when using a block (0) where no contract is deployed:
  5) eth_call PYTHON calling getAddress (0x38cc4831) should return "0x", when using a block (0) where no contract is deployed:
  6) eth_call PYTHON calling getInt8 (0x57cb2fc4) should return "0x", when using a block (0) where no contract is deployed:
  7) eth_call PYTHON calling getUint256 (0x68895979) should return "0x", when using a block (0) where no contract is deployed:
  8) eth_call PYTHON calling getInt256 (0xf5b53e17) should return "0x", when using a block (0) where no contract is deployed:
  9) eth_getFilterLogs PYTHON should return a list of logs, when asking without defining an address and using toBlock "latest":
  10) eth_getFilterLogs PYTHON should return a list of logs, when asking without defining an address and using toBlock "pending":
  11) eth_getFilterLogs PYTHON should return a list of logs, when filtering with defining an address and using toBlock "latest":
  12) eth_getFilterLogs PYTHON should return a list of logs, when filtering with defining an address and using toBlock "pending":
  13) eth_getFilterLogs PYTHON should return a list of logs, when filtering by topic "0x0000000000000000000000000000000000000000000000000000000000000001":
  14) eth_getFilterLogs PYTHON should return a list of anonymous logs, when filtering by topic "0xffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffff":
  15) eth_getFilterLogs PYTHON should return a list of logs, when filtering by topic "0xffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffff":
  16) eth_getTransactionByHash PYTHON should return a transaction with the proper structure:
  17) eth_getTransactionByHash PYTHON should return a transaction with the proper structure:
  18) eth_getTransactionByBlockNumberAndIndex PYTHON should return transactions for the pending block when using "pending" and sending transactions before:
  19) eth_getTransactionReceipt PYTHON should return a transaction receipt with the proper structure:
  20) eth_getTransactionReceipt PYTHON should return a transaction receipt with the proper structure:
  21) eth_getUncleByBlockNumberAndIndex PYTHON should return a no uncles when using "pending" (as there are none):
  22) eth_getUncleCountByBlockNumber PYTHON should return 0 as a hexstring when "pending":
  23) eth_uninstallFilter PYTHON should return a boolean when uninstalling a block filter:
  24) shh_hasIdentity PYTHON should return TRUE if the identity already exists:
  25) shh_hasIdentity PYTHON should return FALSE if the identity doesn't exists:
  26) shh_hasIdentity PYTHON should return an error when no parameter is passed:
  27) shh_newFilter PYTHON should return a number as hexstring when all options are passed:
  28) shh_newFilter PYTHON should return a number as hexstring when a few options are passed:
  29) shh_newFilter PYTHON should return an error when a wrong parameter is passed:
  30) shh_newFilter PYTHON should return an error when no parameter is passed:
  31) shh_newIdentity PYTHON should return an identity:
  32) shh_uninstallFilter PYTHON should return a boolean when uninstalling a options filter:
  33) shh_uninstallFilter PYTHON should return an error when no parameter is passed:
  34) shh_version PYTHON should return a version number as HEX string:

context?

context?

That's how I understood:

jsonrpc tests #29

How To...run the jsonrpc-tests? https://github.com/ethereum/pyethapp/issues/29#issuecomment-184234128

Are these tests integrated with the 'coveralls' check?

After fixing most of the issues, here is the current picture:

eth_accounts PYTHON should return an array with accounts:
eth_getFilterLogs PYTHON should return a list of logs, when asking without defining an address and using toBlock "latest":
eth_getFilterLogs PYTHON should return a list of logs, when asking without defining an address and using toBlock "pending":
eth_getFilterLogs PYTHON should return a list of logs, when filtering with defining an address and using toBlock "latest":
eth_getFilterLogs PYTHON should return a list of logs, when filtering with defining an address and using toBlock "pending":
eth_getFilterLogs PYTHON should return a list of logs, when filtering by topic "0x0000000000000000000000000000000000000000000000000000000000000001":
eth_getFilterLogs PYTHON should return a list of anonymous logs, when filtering by topic "0xffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffff":
eth_getFilterLogs PYTHON should return a list of logs, when filtering by topic "0xffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffff":
eth_getFilterLogs PYTHON should return a list of logs, when asking without defining an address and using toBlock "latest":
shh_hasIdentity PYTHON should return TRUE if the identity already exists:
shh_hasIdentity PYTHON should return FALSE if the identity doesn't exists:
shh_hasIdentity PYTHON should return an error when no parameter is passed:
shh_newFilter PYTHON should return a number as hexstring when all options are passed:
shh_newFilter PYTHON should return a number as hexstring when a few options are passed:
shh_newFilter PYTHON should return an error when a wrong parameter is passed:
shh_newFilter PYTHON should return an error when no parameter is passed:
shh_newIdentity PYTHON should return an identity:
shh_uninstallFilter PYTHON should return a boolean when uninstalling a options filter:
shh_uninstallFilter PYTHON should return an error when no parameter is passed:
shh_version PYTHON should return a version number as HEX string:

Where:
eth_accounts as @konradkonrad says can be ignored as not matching actual requirements;
eth_getFilterLogs should go away after resolving the data issue in tests https://github.com/ethereum/tests/issues/112
shh_* errors are caused by the missing Whisper protocol. In order to implement it a clear commitment by the project/foundation is required to invest in this.

Also this PR should be pulled https://github.com/ethereum/rpc-tests/pull/9

authors: RomanZacharia heikoheiko konradkonrad

repo: ethereum/pyethapp


text:
feat: parallel download proof of concept

I suggest we close this

authors: agazso vojtechsimetka

repo: ethersphere/gateway


text:
Make all tests executable

Nice! I love all the improvements you've been making to this track in the past few days.

authors: kytrinyx mienaikage


text:
Update package.json

Hi @EasonWang01, thanks for this pull request! Can you split out the changes into separate commits like other dependency updates and update the History.md file with the description of the non-devDependencies updates?

I'm going to close this due to lack of action on the outstanding request. Please feel free to make a new pull request with the above changes any time, though!

repo: expressjs/express


text:
Add doc about the new wildfly-swarm-health-check

[merge]


text:
[csharp] fix build, correctly size UIScrollViews

👍

👍

authors: Shaddix rmarinho splhack

repo: facebook/yoga


text:
Correct google-style indentation

LGTM, but we need to wait for CLA.

The change is in main.

authors: bethcutler cgrushko

repo: facebookincubator/ktfmt


text:
Account expiration feature

Looks good.

repo: fedspendingtransparency/data-act-broker


text:
Add Rosetta license, make paths relative

@copiesofcopies Can you take a quick look at this?

Thanks @maoo @copiesofcopies @gs-bracej @mcleo-d

authors: brooklynrob gs-bracej

repo: finos/alloy


text:
add parameter for printing only unique results

Satisfies #167

i wonder if perhaps this should be the default mode, and a "more advanced mode" (hardcore mode?) that prints duplicates with their different contexts? cc: @mr-tz thoughts?

Yes, I think enabling it by default makes sense for most users.

Awesome, thanks. We can expand the advanced behavior based on this now.

authors: mr-tz pilate williballenthin

repo: fireeye/flare-floss


text:
fix: check for project id before running jobs

LGTM!

authors: houqp narenst

repo: floydhub/floyd-cli


text:
fix typo.

Thanks for the contribution. However, this is correct as-is. I am going to close this.

authors: goderbauer jhwsx

repo: flutter/flutter


text:
Dialog edit.

Fixed in another change.

authors: fogleman shaggytwodope

repo: fogleman/nes


text:
Added werewolf

Thanks!

repo: fpco/stackage


text:
Add Fedora build deps to README

ping

repo: freeotp/jelling-linux


text:
Adds option to set AppName in APIClient

I'm wondering if we should do something involving the client_id, that way we don't rely on the 3rd-party to have to set an app name, as they need to set a client_id.
I was thinking about this for the python sdk and maybe we could standardize on something:

FreshBooks node sdk/{VERSION} client_id {client_id}
FreshBooks python sdk/{VERSION} client_id {client_id}

Thoughts?

I'm wondering if we should do something involving the client_id, that way we don't rely on the 3rd-party to have to set an app name, as they need to set a client_id.
I was thinking about this for the python sdk and maybe we could standardize on something:

FreshBooks node sdk/{VERSION} client_id {client_id}
FreshBooks python sdk/{VERSION} client_id {client_id}

Thoughts?

That does make more sense. More questions though, should we make client_id a mandatory field? If they don't pass the client_id should we throw an error or set a default value and return a warning or something?

I was debating that too.
While you can technically use the sdk with just a bearer token you got from somewhere, I don't think it's a common use-case outside of basic testing of the API, so I think it's safe to require a client_id.

authors: amcintosh ngalandefb

repo: freshbooks/api-nodejs-sdk


text:
Walkmodb

I adapted the PR #1 from @rpau and added walkmod to the project, thanks!

repo: ftomassetti/java-symbol-solver


text:
changes to quiz

Score: 2/5
Don't fret. Looks like you gave this an honest attempt. There will be plenty more jQuery practice on the way

authors: MaxBlaushild csadusky


text:
Add a short description of how to use the jupyter notebooks with Galaxy

I could complete this task - however, would it be faster to just link these instructions? https://galaxyproject.github.io/training-material/topics/galaxy-ui/tutorials/galaxy-intro-jupyter/tutorial.html#open-a-notebook
LMK and I'll proceed if needed

@nekrut do we use IEs or ITs?
@lynnlangit thanks a lot. On EU we are using the newer Interactive Tools, they behave the same but are tools and can be started like tools. E.g. from live.usegalaxy.eu

Unfortunately we are still using IEs -> @jmchilton and @natefoo are prepping for proper IT release on main, but it will take a month or so

Is my PR sufficient here? If not, I can write a one-paragraph explanation.  LMK

@lynnlangit I think so. If I get some time I would like to add some more information about the Interactive Tools. But I don't think this is crucial atm.
Thanks Lynn!
P.S. Do you have any other feedback for us?

@bgruening - I employ a number of graduate students, would it be useful to have them test these workflows to make sure they can replicate the workflows?  If so, we can do that and provide feedback on any small documentation changes or possible omissions/errors in the workflows - LMK

Oh my gosh, this sounds terrific! @lynnlangit please do so! Any feedback is welcome and if you students learn something from this even better. This sounds awesome!
I actually wanted to write some training material for those workflows. But we haven't got time so far. If someone of your students is motivated to help writing some training material for it, let me know. There is also training to write training material. Sorry, don't want to push work to you, just wanted to point out the training material and that this would probably also enable citizen science in some way.
Thanks a lot @lynnlangit!

@bgruening ok - if my team tests and tries to write training, where should we put it?  do you want to make a folder on this repo and we can just PR to that location?  LMK

Thanks, I lost my laptop, but should be back again, let me know when you have any problems!

@lynnlangit can we help in any way? We got many requests for training material. We also created a Docker container with everything included: https://covid19.galaxyproject.org/deploy/#docker

@lynnlangit I will set up the backbone for these tutorials on the Galaxy Training Network, it will than be easy to contribute to them
.

ping @wm75 ...

authors: bedroesb bgruening lynnlangit nekrut wm75


text:
Add biopython dependency

To preinstall NumPy, you can use a prebuild command just as is done with bx-python:
https://github.com/galaxyproject/starforge/blob/master/wheels/build/wheels.yml#L6

error: invalid command 'bdist_wheel' -  Duck-Duck-Go tells me that I should install wheels at first?

error: invalid command 'bdist_wheel' can be fixed by inserting setuptools with:
insert_setuptools: true

Under the biopython entry in wheels.yml.

@bgruening here you are!: https://depot.galaxyproject.org/starforge/wheels/build-128/

Ah, this is good to know. The error message was a little bit misleading. Thanks for fixing!

These wheels have been copied to wheels.galaxyproject.org

authors: bgruening natefoo

repo: galaxyproject/starforge


text:
add yarn.lock

Lock files are a pain to keep updated. We ended up adding the package-lock.json because NPM is really cute and deletes everything in node_modules if that doesn't exist.
At some point we'll setup a bot to automatically update the lock files at which point this would be doable but in the meantime, let's not add a yarn.lock.

authors: KyleAMathews ekoeryanto

repo: gatsbyjs/gatsby-starter-hello-world


text:
Enable the TinkerPop implementation to connect to a Gaffer rest api

I would recommend binding it via the Java API. That is, implement the following interfaces in TinkerPop:
Graph: https://github.com/apache/tinkerpop/blob/master/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/Graph.java
Vertex: https://github.com/apache/tinkerpop/blob/master/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/Vertex.java
VertexProperty: https://github.com/apache/tinkerpop/blob/master/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/VertexProperty.java
Edge: https://github.com/apache/tinkerpop/blob/master/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/Edge.java
Property: https://github.com/apache/tinkerpop/blob/master/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/Property.java
That is about 90% of the effort required. After that, to make things solid:

Connect it to the StructureStandardSuite and ProcessStandardSuite. You can see how this is done via TinkerGraph: https://github.com/apache/tinkerpop/blob/master/tinkergraph-gremlin/src/test/java/org/apache/tinkerpop/gremlin/tinkergraph/structure/TinkerGraphStructureStandardTest.java and https://github.com/apache/tinkerpop/blob/master/tinkergraph-gremlin/src/test/java/org/apache/tinkerpop/gremlin/tinkergraph/process/TinkerGraphProcessStandardTest.java

Add a TraversalStrategy to use index lookups. I assume you have fast lookups for stuff like "get me all vertices with name 'marko'." If so, then you will need something like TinkerGraphStepStrategy (https://github.com/apache/tinkerpop/blob/master/tinkergraph-gremlin/src/main/java/org/apache/tinkerpop/gremlin/tinkergraph/process/traversal/strategy/optimization/TinkerGraphStepStrategy.java) where your GafferGraphStep implementation will use the HasContainers accordingly to do index lookups -- e.g. g.V().has('name','marko').

That is really all there is to it. I suspect a solid 1 week effort and you will then be able to use Gremlin to query your graph.
http://tinkerpop.apache.org/gremlin.html
Hope that helps.

Thanks again for your help. As you saw we have a very basic implementation over in the Gaffer Tools Repository.
The current implementation was written just to investigate if and how it might be done and is still very much a prototype. Unfortunately we just haven't had chance to tidy it up and implement the extra bits. As you mentioned, we do really need to make use of TraversalStrategies to optimise it for Gaffer.
I'll close this issue and reopen a similar one in the gaffer tools repo with your ideas.

repo: gchq/Gaffer


text:
Singularity runner step-specific container engine configuration

thanks Anders. Looks like you are working from your fork's master branch. That has the problem of having to synchronize the changes in the upstream's master with yours. To avoid this, you can create a new branch out of the upstream master. For example:
git remote update

git checkout -b my-feature origin/master

The above places you in a my-feature branch that then you can push to your remote and create a PR from that.

thanks Anders. Looks like you are working from your fork's master branch. That has the problem of having to synchronize the changes in the upstream's master with yours. To avoid this, you can create a new branch out of the upstream master. For example:
git remote update

git checkout -b my-feature origin/master

The above places you in a my-feature branch that then you can push to your remote and create a PR from that.

Thanks for pointing that out 🙂. I'll create a new branch and open a draft there

authors: Jswig ivotron

repo: getpopper/popper


text:
Add Debug impl to the Backbuffer

bors r+

authors: kvark omni-viral

repo: gfx-rs/gfx


text:
Merged at 20170614

Merged at 20160614

repo: ghostlyman/pyenv


text:
Updated Package.json for missing dev dependencies

+1

authors: GregM archanks

repo: gilt/gilt-module-boilerplate


text:
Change color to blue

Don't actually use this, I think it breaks things.


text:
Implements Issue #157- Add support for foreign key constraints

I've opened a disucssion about how we approach the different PR's here: https://groups.google.com/forum/#!topic/gorp-dev/rEsu0efgUfw

Hey @rickb777,
Thanks for this PR!
The last commit (423a897) moves a lot of code from gorp.go to schema.go. I agree with the concept of splitting the code into multiple smaller files, but right now it will probably break a lot of open PR's. Can you undo the moving of code and squash/overwrite that commit? We've also deprecated the develop branch, but you should be able to rebase on master without any problems.
We will split the code in desperate files once the open PR's are closed.

Hi @rickb777
Any chance you can work a bit more on this so we can merge it into master?

This will need some work and manual merging before it can be added. Could be done in v2-phase3 (see #270).

repo: go-gorp/gorp


text:
Lock table detection for YugabyteDB

Thanks!

authors: bithavoc vmihailenco

repo: go-pg/migrations


text:
fix RelURL and AbsURL when path starts with language

 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.

@bep

What is this trying to fix? Can you give some "failing examples"?

Example:
language: de
url: deposit
Expected: de/deposit
Actual: deposit
There are test in the pull request.

So for a site baseURL set to https://example.com/hugo/ and the current language is en:
{{ "blog/" | relLangURL }} → "/hugo/en/blog/"

What is this trying to fix? Can you give some "failing examples"?

So, you meant that {{ "deposit" | relURL }} should be /de/deposit?
relURL does not take language into account, I don't see how it could. See
https://gohugo.io/functions/rellangurl/

https://gohugo.io/functions/rellangurl/
relLangURL and absLangURL take language into account
{{ "blog/" | absLangURL }} → "https://example.com/hugo/en/blog/"
{{ "blog/" | relLangURL }} → "/hugo/en/blog/"
blog/ becomes /hugo/en/blog/
the problem is that if the language is de and the parameter (blog/ from example above) starts with de (example deposit) hugo thinks it is already added the prefix, but it don't and the url becomes
/deposit instead of /de/deposit
Could you please, merge this Pull Request ?

@bep what's necessary to get this merged?
Gajim website is affected, see https://dev.gajim.org/gajim/website/issues/25

@spf13 Could you please tell me what I need to do this pull request to be merged ?

@anthonyfok Could you please tell me what I need to do this pull request to be merged ?

What is this trying to fix? Can you give some "failing examples"?

So, you meant that {{ "deposit" | relURL }} should be /de/deposit?
relURL does not take language into account, I don't see how it could. See
https://gohugo.io/functions/rellangurl/

https://gohugo.io/functions/rellangurl/
relLangURL and absLangURL take language into account
{{ "blog/" | absLangURL }} → "https://example.com/hugo/en/blog/"
{{ "blog/" | relLangURL }} → "/hugo/en/blog/"
blog/ becomes /hugo/en/blog/
the problem is that if the language is de and the parameter (blog/ from example above) starts with de (example deposit) hugo thinks it is already added the prefix, but it don't and the url becomes
/deposit instead of /de/deposit
Could you please, merge this Pull Request ?

@bep
I provided the example from the documentation that relLangURL and absLangURL add the language to the url.
But there is a bug when the url the the example "blog" in my example "deposit" starts with language example "de". "deposit" starts with "de" and the code skips to add "/de/deposit", but becomes "/deposit".
For english it works correctly "/en/deposit".
Please, tell me what is needed to merge this ?
Or if you do not think that this is correct please let me know so I can make changes to merge request.

Hi, I visit again because this bug still affects us when trying to deploy content with i18n on gajim.org. I just tested with vanilla Hugo 0.64.1, and the bug is still present, see following example:

How it should be: website.tld/de/development
How it actually is: website.tld/development (thus falling back to original EN content, instead of DE)

However, manually applying the patch from this MR fixes the bug (tested using Hugo 0.64.1).

Experiencing the same problem with URL /data/ on language da (Danish).
@bep what is needed to get this put into a release?
Is there something we can do?

Yes, this is still a problem which needs to be fixed. I just tested this with Hugo 0.73. This pull request provides a working fix for the problem. I manually patch hugo every time I update it on the server.
I understand using StaleBot might be a solution to reduce the count of open issues. Managing an issue tracker is time consuming and needs a lot of maintenance from my experience. But you really shouldn’t apply StaleBot on pull requests. After all, people invested their time to fix/add something they found are worth spending time on.

@bep
Please, what should be done to merge this pull request?
I really like your product and I use it on every day.
I just want to contribute to make it a little better.
Please, cloud you merge this fix for a bug?

OK, sorry for the delay. I misunderstood this the last time I looked at it.

Thanks for merging this @bep
Is there any timeline on when this will be included in a release?

Is there any timeline on when this will be included in a release?

No, statistically I would have needed to do a patch release fixing a regression from 0.79, but that hasn't happened... If you need this right away, it's fairly easy to build from source.

... that said, the release frequency is pretty high ...

authors: CLAassistant Steiniche a-andre bep ivan-meridianbanc-com wurstsalat3000

repo: gohugoio/hugo


text:
Add support for "hmac-sha2-512-etm@openssh.com"

@googlebot I signed it!

@FiloSottile Is there anything we can do to get this pull request merged? As I mentioned in https://github.com/golang/crypto/pull/127#issuecomment-1247053815, we've been running a fork of this repository in production, and I'm trying to ensure that all the required upstream changes land soon. Thanks!

repo: golang/crypto


text:
Add Travis CI configuration

Gah! It works fine on my box :-(
Aha! was missing an "-ordered" flag. So contents of a given directory isn't
deterministic.
Expect a PR very soon
Mark
On Wed, Mar 8, 2017 at 3:12 PM, Lei Zhang notifications@github.com wrote:

@antiagainst commented on this pull request.
In .travis.yml
https://github.com/google/findutils/pull/2#discussion_r104938046:

@@ -0,0 +1,20 @@
+language: rust

+sudo: false
+os:

linux

osx

Oh, it also fails on Linux
https://travis-ci.org/google/findutils/builds/209000807...
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/google/findutils/pull/2#discussion_r104938046, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ALbIF6tsKmeeMNRdculfaiuX96jn9uXJks5rjsVmgaJpZM4MW4uH
.

I somehow managed to pull in the wrong direction so the fix is now in,
without any approvals :-(
Can you update your local copy and check that the tests now pass on other
machines?
thanks
Mark
On Wed, Mar 8, 2017 at 3:20 PM, Mark Charsley mcharsley@google.com wrote:

Gah! It works fine on my box :-(
Aha! was missing an "-ordered" flag. So contents of a given directory
isn't deterministic.
Expect a PR very soon
Mark
On Wed, Mar 8, 2017 at 3:12 PM, Lei Zhang notifications@github.com
wrote:

@antiagainst commented on this pull request.
In .travis.yml
https://github.com/google/findutils/pull/2#discussion_r104938046:

@@ -0,0 +1,20 @@
+language: rust

+sudo: false
+os:

linux

osx

Oh, it also fails on Linux
https://travis-ci.org/google/findutils/builds/209000807...
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/google/findutils/pull/2#discussion_r104938046, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ALbIF6tsKmeeMNRdculfaiuX96jn9uXJks5rjsVmgaJpZM4MW4uH
.

Rebased and pushed. Should trigger a new build job now.
The test still fails on my mac.

Latest travis build is green:
https://travis-ci.org/google/findutils/builds/209483094, which includes
some runs on osx. Is find_main_depth_first still failing on your mac? Can
you check that it contains the sorted flag:
let rc = find_main(&["find", "./test_data/simple", "-sorted", "-depth"],
&deps);
thanks
Mark
On Thu, Mar 9, 2017 at 7:36 PM, Lei Zhang notifications@github.com wrote:

Rebased and pushed. Should trigger a new build job now.
The test still fails on my mac.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/google/findutils/pull/2#issuecomment-285457449, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ALbIF8RWZrbk0QgwAt-NSPdRQye7mK-8ks5rkFTYgaJpZM4MW4uH
.

My fault. Somehow I failed to rebase this change against the current master.

Merged. ;-P

authors: antiagainst mcharsley

repo: google/findutils


text:
remove redundant line

Coverage decreased (-0.0008%) to 98.623% when pulling 6d53a2aa2744a3c144722e9b011ab35d2fa20518 on cleanup into af69fc5347e5df1bf556ae458d9df8b6d95041a1 on master.

authors: TheRoyalTnetennba coveralls

repo: google/google-auth-library-ruby


text:
NFC: Add _path to args in update_to_llvm_syncpoint.py

Consider also renaming the _commit args to _ref as well? Maybe leave _commit as an alias?

I would, but then I would want to replace all of the other instances of _commit referring to non-commit values in the code, and at that point I'm rewriting a lot of things.

repo: google/iree


text:
Minor dedupe xla/pxla tracing code.

Hey, are you planning to add this support in sharded_jit also?

Hey, are you planning to add this support in sharded_jit also?

How would I do that? Should I put in a pull request onto Skye's branch? Do you want me to do that before this gets merged?

The sharded_jit is already in the interpreters directory. Let's ask @skye if she thinks there is value in adding this improvement there, or she will add it herself in the final version.

The sharded_jit is already in the interpreters directory. Let's ask @skye if she thinks there is value in adding this improvement there, or she will add it herself in the final version.

Ah. I didn't realise that was already in. I've added the change there too.

repo: google/jax


text:
Add grpc streaming tests to pubsub and speech

Do these tests run without failure?

@landrito Yes they do

authors: landrito shinfan

repo: googleapis/api-client-staging


text:
Is there any plan to provide an interface to C?

No. We have no plans to produce C libraries at this time.

Thanks for the answer.

repo: googleapis/google-cloud-cpp


text:
feat: stub out initial samples

Codecov Report

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

@@            Coverage Diff            @@
##             master       #2   +/-   ##
=========================================
  Coverage          ?   96.12%
=========================================
  Files             ?        5
  Lines             ?     1549
  Branches          ?       68
=========================================
  Hits              ?     1489
  Misses            ?       59
  Partials          ?        1

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 7f02139...11b251d. Read the comment docs.

authors: codecov-commenter sofisl


text:
Update appsscript manifest to be more consistent

@googlebot - can you try again.

repo: googledatastudio/community-connectors


text:
Call out what the samples are on the landing README.md

README has been updated


text:
build/Makefile: update ngfx-install rule

updated ngfx build: ported gnu makefile for building 3rd party libs to ninja
this is the corresponding change


text:
[WIP] Refactor

done with https://github.com/gr2m/editable-table/releases/tag/v1.0.0


text:
KeyError: 'QG_RCNN is not in the detector registry'

This repo does not support model outside mmdetection.

authors: grimoire jxuepeng

repo: grimoire/mmdetection-to-tensorrt


text:
Fazer deploy da aplicação usando GitHub Pages

Fechando o PR pq estamos utilizando o netlify

authors: laionazeredo shebangbash

repo: grio-project/grio-landing-page


text:
Coverage

Hello @gustavooquinteiro! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file sample/Gantt.py:

Line 332:53: W291 trailing whitespace

In the file sample/InterFace.py:

Line 132:1: W293 blank line contains whitespace
Line 177:80: E501 line too long (115 > 79 characters)
Line 190:80: E501 line too long (85 > 79 characters)
Line 206:34: E703 statement ends with a semicolon
Line 216:80: E501 line too long (90 > 79 characters)

repo: gustavooquinteiro/process-escalonator


text:
Add Android O support, bump support libraries to v26.0.0

Hi. Thanks for the great contribution.
I'll merge it into the feature/support_libraries_v26 branch instead of the master for this time.


text:
Add receipt printer functionality

Added! Printer working on Windows!

PHP + Windows 😢

Puc fer que funcioni també en Linux, però fa mandra. :)

repo: hackupc/baggage


text:
remove default color

Codecov Report

Merging #44 (80a5076) into main (28406d5) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #44   +/-   ##
=======================================
  Coverage   83.97%   83.97%
=======================================
  Files          15       15
  Lines         855      855
=======================================
  Hits          718      718
  Misses        137      137

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 28406d5...80a5076. Read the comment docs.


text:
Fix maintainer typo

Ty

authors: halkeye nfg

repo: halkeye/language-versions


text:
Test

Thank you for your submission! We require that all contributors sign our Contributor License Agreement ("CLA") before we can accept the contribution. Read and sign the agreement
Learn more about why HashiCorp requires a CLA and what the CLA includes
Have you signed the CLA already but the status is still pending? Recheck it.

repo: hashicorp/hashicat-aws


text:
278

下次可以考虑从你的develop 分支pull request origin 的develop分支哈，有问题我们都尽量在develop分支解决，当然你的一般都没有问题

authors: henrytien yhwhu

repo: henrytien/leetcode-lintcode


text: ToggleFlyButton: Make local text entity not grabbable

repo: highfidelity/hifi-content


text:
Check for updates once a day, add option to disable update check

Codecov Report

Merging #15 (cca3056) into master (45b9976) will increase coverage by 0.52%.
The diff coverage is 99.37%.

@@            Coverage Diff             @@
##           master      #15      +/-   ##
==========================================
+ Coverage   92.45%   92.97%   +0.52%
==========================================
  Files          32       34       +2
  Lines        1458     1609     +151
==========================================
+ Hits         1348     1496     +148
- Misses        110      113       +3

Impacted Files
Coverage Δ

src/main.rs
84.78% <95.00%> (-0.52%)
:arrow_down:

src/update/baca_release.rs
89.28% <100.00%> (-0.37%)
:arrow_down:

src/update/time_provider.rs
100.00% <100.00%> (ø)

src/update/update_check_timestamp.rs
100.00% <100.00%> (ø)

src/workspace/mod.rs
90.06% <100.00%> (+3.39%)
:arrow_up:

src/update/github_releases.rs
88.73% <0.00%> (-4.23%)
:arrow_down:

src/error.rs
33.89% <0.00%> (-1.70%)
: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 45b9976...cca3056. Read the comment docs.

authors: codecov-commenter hjaremko

repo: hjaremko/baca-cli


text:
Split out git-independent formatter (called omnifmt)

hjwylde/omnifmt

repo: hjwylde/git-fmt


text:
Cb 8522 audit api

thanks @akanto

authors: akanto doktoric

repo: hortonworks/cloudbreak-deployer


text:
unit tests for backends

Fixed since cbabd7072c13e1312114961a7a9a090cb3423b05.

repo: hydrargyrum/vignette

license: WTFPL


text:
Turkish translation for validation page #299

Codecov Report

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

@@          Coverage Diff           @@
##             dev     #375   +/-   ##
======================================
  Coverage       ?   81.68%
  Complexity     ?      313
======================================
  Files          ?      105
  Lines          ?      988
  Branches       ?        0
======================================
  Hits           ?      807
  Misses         ?      181
  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 f1dd4ac...fa3a958. Read the comment docs.

authors: codecov-io ermanyilmaz

repo: iluminar/goodwork


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

Looks fine. Nothing major so feel free to self-merge this when checks have ran (you should have the access rights to do so).

Note: -Z minimal-version being broken does not look like your fault to me, although it is a very interesting crash.

repo: image-rs/image-png


text:
update dialer's writer to use the new :writeinto_iovec(iov) interface

I actually do have this, I just added the wrong file to the commit. Oops, fixing now. :/

Sweet!

This makes sense right? https://github.com/imgix/levee/pull/22/commits/85c2d85a4e6d465f2ef9582fe4112733aadb5147

Yep, it does. This already looks like a pattern we might be able to capture. W:send takes a ... .. it might be able to effectively translate to this.  85c2d85 seems good though until we do this sort of thing a few more times.

Ah, yeah. I was thinking Message could capture a full array as a general purpose iovec writer, but that would be nicer. I suppose I could have done this with the nested Iovec too. Good for now though.

Yeah, it's nice to let things stew a bit, before committing to it.

Fixed by #22

authors: cablehead kalamay

repo: imgix/levee


text:
AddResourceDropdown should use the Link tag provided by react-router for redirecting the user to overlays.

Unfortunately, just inserting a Link component isn't as straightforward as it seems.  The issue then becomes, that the anchor (<a/>) tag that Link renders does not occupy the entire width and height of its parent div making part of the dropdown list item  unclickable:

Solving this problem requires that clockface Dropdown.Item take custom styles, class names, or both.  Since, all this story is doing is saving us the trouble of passing a couple of callbacks that fire router.push I say this story is fine to hold off until this occurs in clockface.

repo: influxdata/influxdb


text:
Pattern synonyms for degenerate HFC instances

bors merge

GitHub issues. This is now merged.

repo: input-output-hk/ouroboros-network


text:
chore(deps): Update dependencies to latest

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

repo: inpyjamas/scripts


text:
Add drone ci

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

repo: insuusvenerati/treats-portfolio


text:
Use atomicwrites if available

[`os.replace'](https://docs.python.org/3/library/os.html#os.replace) seems to do exactly what `atomic_replace` does (and the doc says it is atomic). I'd be happy to do `os.rename`->`os.replace`; silent overwriting seems like the write thing to do, since that is what `open(, mode='wb')` does.

Yeah, it looks like os.replace does the same thing:
https://github.com/untitaker/python-atomicwrites/issues/25#issuecomment-358050598
https://github.com/python/cpython/blob/7514f4f6254f4a2d13ea8e5632a8e5f22b637e0b/Modules/posixmodule.c#L4319
It might be worthwhile to use that then instad of os.rename.
No great urgency but I'll try to find time to update the PR...

authors: dhirschfeld martindurant


text:
tests: programatically get/create valid session for account for tests

This can be closed. I've found no simple or clean way to programatically create a valid session without actually performing the login routine.
#49 added testutils.py:login_client_via_view which, after having logged the user in, causes a valid session to be available via flask.session.

Example of how to obtain a valid session for a test client:
from invenio_accounts import testutils, InvenioAccounts
from invenio_accounts.views import blueprint

def this_is_a_test(app):
  """And in this test we test some stuff."""
  InvenioAccounts(app)
  app.register_blueprint(blueprint)
  with app.test_client() as client:
    user = testutils.create_test_user()
    testutils.login_user_via_view(client, user=user)
    # Or testutils.login_user_via_view(client, user.email, user.password_plaintext)

repo: inveniosoftware/invenio-accounts


text:
Use SHA2.256 as default, add support for BLAKE2s, add spec for SHA1, Doc Updates

Coverage increased (+0.3%) to 82.155% when pulling cd2ef8ebe038045a4be8a8f05542fe1e0a6ca72c on grempe:safer into 02d52ef7b04c939d772e32cfa4df7a6a21b087eb on ioquatix:master.

Thanks for the merge!

authors: coveralls grempe

repo: ioquatix/fingerprint


text:
Fix notebooks

Coverage remained the same at 90.159% when pulling 18d7d6c15e0c6f14504aef79c15ec2ba9c51d275 on develop into 7aef8a34d82911635ebc8893828bc5de1a49a565 on master.

authors: FavioVazquez coveralls

repo: ironmussa/Optimus


text:
enable pilot/model linter test

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: hzxuzhonghu
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: costinm
If they are not already assigned, you can assign the PR to them by writing /assign @costinm 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:

bin/OWNERS
pilot/OWNERS [hzxuzhonghu]

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

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: hzxuzhonghu, 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:

bin/OWNERS [rshriram]
pilot/OWNERS [hzxuzhonghu,rshriram]

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

authors: hzxuzhonghu istio-testing

repo: istio/istio


text:
Use getter methods of proto message on FromProto functions

Pull Request Test Coverage Report for Build 369167673

4 of 6 (66.67%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.007%) to 88.859%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

gqlgenplugin/protomodelgen/plugin.go
4
6
66.67%

Totals

Change from base Build 363856416:
0.007%

Covered Lines:
1332

Relevant Lines:
1499

💛  - Coveralls


text:
Adds composer.json

Thanks!

authors: aschrammel jahvi

repo: jahvi/FileUploadAttribute


text:
Fix typo in Aside: Big-O notation

Sorry my bad! Little sleep can do this to people :(

repo: jakevdp/PythonDataScienceHandbook


text:
fixed typo while rendering partial repositories/download_revision

Thank you!

authors: n-rodriguez perltwiesekt


text:
Update for use with Elm 0.13

Cool. Thanks!

I think you accidentally haven't actually merged my changes.

Not sure how that happened. very strange. Thanks again.

Now that elm-audio works with Elm 0.13, do you have plans to add it to the global Elm package repository at http://package.elm-lang.org/packages ?

One of my main reasons for not doing this is that it has a Native module which needs to go through an approval process. Unfortunately, at this time I don't think I would have time to go through any revisions that might be required. Would you be interested in being a collaborator on this project?
Also, do you mean Elm 0.14?

Yeah, I meant Elm 0.14.
Didn't know about the Native restriction, that's a bummer I guess. TBH I only plan to use the whole of Elm for single project that I will only be able to reveal in a week and then promptly abandon,
The fact that there's a Ludum Dare mini-challenge next week is not a coincidence ;)

authors: gergoerdi jcollard

repo: jcollard/elm-audio


text:
Add proper English player name handling to the credits

Possibly relevant function: 13538. Seems to be called by another function (F4BC) that pieces stuff (tilemaps?) together. It's also the only other thing in the ROM I found that references the English player name besides naming screen related stuff.

authors: Lorenzooone PhoenixBound


text:
Note case sentivity in help

Thank you for a pull request! Please check this document for how the Jenkins project handles pull requests

:+1:

Anything else needed before this can be committed? Thanks

It would make sense to replace "authentication backend" to "security realm" just to reference common Jenkins terms. You could also add a hyperlink to https://wiki.jenkins-ci.org/display/JENKINS/Securing+Jenkins in order to clarify it

I don't think this is correct. Is there a security realm this does not apply to?

@oleg-nenashev Updated text per your comments
@daniel-beck I'm not sure if it does or doesn't apply to all, but the mention of "case sensitivity" likely will help some others out

Is this still applicable given #9?

I would like to deploy an updated plugin and check ...

authors: daniel-beck gliptak jenkinsadmin jglick oleg-nenashev

repo: jenkinsci/matrix-auth-plugin


text:
quora上显示不正常

duplicate of #57

duplicate of #46

repo: jinntrance/shanbay-crx


text:
Version 0.8.4: use experimentalUpdater for buildSrc consumers

Misses supports for alias to plugin versions. Also, I found a way to test locally with buildSrc, so I'll make a new PR later, possibly with other improvements.

repo: jmfayard/buildSrcVersions


text:
Make footer stick to bottom regardless of page height

Disregarding this, footer will float for short pages but is not a huge issue as primary use case for this website is mobile (and the css is like a mix between a bird's nest and a bear trap)

repo: jmwerner/recipes


text:
Fix ComparisonFailure on Windows

I'd like JMXTrans was also built on a Windows CI (Appveyor for example), in order to check that it builds and runs on Windows.

@gquintana feel free to put Appveyor in place. And ping me if you need some help...

authors: gehel gquintana

repo: jmxtrans/jmxtrans


text:
Restyled by prettier-markdown

Merged

repo: johnsonjh/gfpsgo


text:
Understand cloudtrail and cloudwatch logs bill

turned of read only events
temp deactivated tail in member accounts, see #266

repo: jonny-rimek/wowmate


text:
Renamed Class1.cs to ExtensionMethods.cs (use class name)

Thank you!

authors: birbilis jphellemons

repo: jphellemons/CommandAsSql


text:
bug: fixed issue to be able to remove the tmp folder when it has content

Hey @jprichardson can you check my pull request?

hey @jprichardson any luck checking my pull request?

fs.remove should do this - it's from fs-extra.
I have no intention of maintaining this package anymore.

got it, maybe just update the fs-extra package should fix it, still use it on one of my packages.

Added a new commit, let me know if is possible for you to add a new version to NPM, thanks.

repo: jprichardson/node-github-download


text:
Show user name in message instead of user id,

OK

repo: jsonkuan/chatApp


text:
upload eecology movies to vimeo

@lodekulik would you please upload the eecology movies to vimeo? You can find them under ``/demos/eecology/`

Both videos can be found here: https://vimeo.com/channels/1023710

authors: jspaaks lodekulik

repo: jspaaks/collab-demos


text:
Adds two different kinds of table visualization + docs for non devs

I'm having an issue with travis and I have no idea why:
CMake Error at CMakeLists.txt:65 (find_package):

  By not providing "Findtabulate.cmake" in CMAKE_MODULE_PATH this project has

  asked CMake to find a package configuration file provided by "tabulate",

  but CMake did not find one.

  Could not find a package configuration file provided by "tabulate" with any

  of the following names:

    tabulateConfig.cmake

    tabulate-config.cmake

  Add the installation prefix of "tabulate" to CMAKE_PREFIX_PATH or set

  "tabulate_DIR" to a directory containing one of the above files.  If

  "tabulate" provides a separate development package or SDK, be sure it has

  been installed.

It works on my Linux. Will try on the Mac tomorrow.
Even though, in my Linux I get some Warnings on Cmake step:
CMake Warning at CMakeLists.txt:108 (add_executable):
  Cannot generate a safe runtime search path for target xsqlite because files
  in some directories may conflict with libraries in implicit directories:

    runtime library [libcrypto.so.1.1] in /usr/lib may be hidden by files in:
      /home/mariana/.conda/envs/xeus-sqlite/lib
    runtime library [libuuid.so.1] in /usr/lib may be hidden by files in:
      /home/mariana/.conda/envs/xeus-sqlite/lib
    runtime library [libsqlite3.so.0] in /usr/lib may be hidden by files in:
      /home/mariana/.conda/envs/xeus-sqlite/lib

  Some of these libraries may not be found correctly.

CMake Warning at CMakeLists.txt:104 (add_library):
  Cannot generate a safe runtime search path for target xeus-sqlite because
  files in some directories may conflict with libraries in implicit
  directories:

    runtime library [libsqlite3.so.0] in /usr/lib may be hidden by files in:
      /home/mariana/.conda/envs/xeus-sqlite/lib
    runtime library [libcrypto.so.1.1] in /usr/lib may be hidden by files in:
      /home/mariana/.conda/envs/xeus-sqlite/lib
    runtime library [libuuid.so.1] in /usr/lib may be hidden by files in:
      /home/mariana/.conda/envs/xeus-sqlite/lib

  Some of these libraries may not be found correctly.

I wonder if it does not find cpp-tabulate because it's installed under lib64. Maybe you should add the cmake option -D CMAKE_INSTALL_LIBDIR=lib in the cpp-tabulate-feedstock

Oh, thanks for the input. I'll try that. :)

Still the same issue, but when cpp-tabulate gets installed it's the build 0 that comes, not the build 1. Hopefully this will be fixed if you try to re-trigger the CI later today.

authors: marimeireles martinRenou

repo: jupyter-xeus/xeus-sqlite


text:
Docs site update

Codecov Report

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

@@          Coverage Diff          @@
##           master   #190   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           5      5
  Lines         203    203
=====================================
  Hits          203    203

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 462283d...cbe2403. Read the comment docs.

authors: codecov-io jxnblk

repo: jxnblk/styled-system


text:
numpy integration tests

Codecov Report

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

@@           Coverage Diff           @@
##           master      #68   +/-   ##
=======================================
  Coverage   57.57%   57.57%
=======================================
  Files          13       13
  Lines         198      198
=======================================
  Hits          114      114
  Misses         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 3c2d002...42ce701. Read the comment docs.

authors: atilaneves codecov-io

repo: kaleidicassociates/autowrap


text:
Add snowflake dialect

Coverage increased (+0.008%) to 97.738% when pulling 2b2926605a3eab34d577a86a1742ca47fe6ec3cf on RobinPapke:master into 1b089892a0594fb20079c563946d6fceda1ec350 on kayak:master.

authors: RobinPapke coveralls

repo: kayak/pypika


text:
Issue 329

Besides doc additions, this is largely just a 2 line bug fix
@homu r+

:pushpin: Commit 62679d8 has been approved by kbknapp

:hourglass: Testing commit 62679d8 with merge e047ab4...

:sunny: Test successful - status

authors: homu kbknapp

repo: kbknapp/clap-rs


text:
How can I access to pulseUI if I have become a sponsor?

Hi, @gre4ixin. It's a manual process which is part of the reason I'm making it open to download soon. It is distributed as XCFrameworks and can be installed either manually or using Swift Package Manager. I'll provide access soon. Thanks for joining!

@kean I'm sorry, my first sponsored subscription, didn't know how it worked)

My bad, I should make it clear in the README.

I sent access. Please keep in mind this project is in early stages. The distribution might need some improvement.

@kean Okay)

repo: kean/Pulse


text:
Can an anymatch set be used in "sources" ?

It looks like I'm not able to get the simplest use case described in the README to work. Not sure if I'm doing anything wrong. Here's a barebones repo that can reproduce my issue : https://github.com/emilsoman/brunch-concat-test .

What are you trying to achieve here? Have you tried the default method?

I'm trying to concat some arbitrary text files, not really JS, CSS or HTML.
On 3 May 2015 at 03:08, Kenneth Kan notifications@github.com wrote:

What are you trying to achieve here? Have you tried the default method
https://github.com/brunch/brunch/blob/master/docs/config.md#files?
—
Reply to this email directly or view it on GitHub
https://github.com/kenhkan/concat-brunch/issues/2#issuecomment-98407121.

authors: emilsoman kenhkan

repo: kenhkan/concat-brunch


text:
streakランキングをV3 APIを用いた実装に差し替え

なんか良さそうなのでとりあえずマージしちゃいますね

authors: fukatani kenkoooo

repo: kenkoooo/AtCoderProblems


text:
$ pipenv install --skip-lock

this is written and pushed to master, but a test is failing

repo: kennethreitz/pipenv


text:
Made Lat and Long optional

Also change in interface


text:
fixes #84

Thanks! 😄

repo: keroxp/deno-redis


text:
another hidden flake

cc @maxtaco. Tried for a bit to figure this out but couldn't. Will take another look in a few days

(This fails at the last AddMember with a serverside generation assertion error - expected 2, got 1)

@heronhaye out of curiosity, why were you making this test?  In the past, it might suffice to add errors to the list of those that are auto-retried, if this is a race that can happen with CLKRs requiring asynchronous team reloads.

@maxtaco This is another one of the flakes that was triggered in my metadata update systest. Should I add a case to RetryIfPossible?

Which error & error code is it?

"generation assertion failed processing team secret keys", sc.TEAM_BAD_GENERATION

Sure, I'd try to just add another retry condition....

@maxtaco ready for another look.

ok, I'm just merging this. CI constantly flakes on a chat test and mysterious grey orb.

authors: heronhaye maxtaco

repo: keybase/client


text:
Create team signed invites

The commits d6b5729fd7a8...fe887308373c introduce visual changes on linux. :mag_right: 1 changed

Changed: Markdown-emoji (before) (after)

authors: keybase-ci-visdiff patrickxb


text:
comment debug info

👍

authors: MengLiuPurdue kfoynt

repo: kfoynt/LocalGraphClustering


text:
TASK: upgrade to Neos 3.x compatibility

Please squash it instead of just merging.


text:
Operator Mono Book and Light fonts support

@yamenarahman Thanks for the pull request. Although technically this does add the ligatures to the Book and Light weights, since you are using the existing Medium weight ligatures, they probably stand out too much against the lighter weight characters.
I've made some enhancements to the extraction process so it will take less time to create the ligatures for each weight. These ligatures will look right at home with the rest of the font. I should have these done soon.
Thanks again for taking the time to contribute. It's been awesome the feedback that I've gotten.

I've used the existed ligatures as it's not making that huge difference. Here is Book font with Medium ligatures.

Yeah, it's not as bad as I thought it would be. But here are some ligatures for the Book weight.

@kiliman @yamenarahman Thank you both for the work you've put into this. I've been thinking, for some time, that it'd be wonderful to just include the Fira Code ligatures in the Operator Mono font. I haven't had time to grok everything font, so I appreciate the work you've both been putting into this. (It's silly how much impact a font can have, isn't it?)

It's so funny that I returned back to sublime after yesterday's release, sublime doesn't support font ligatures yet but I've to return because I use Laravel and its blade template engine which vscode really sucks in supporting it.

I've pushed Operator Mono SSm Book and Medium weights (including italics).
Definitely try those and let me know what you think.

I don't have the ScreenSmart version of Operator Mono. Any chance we will get ligatures for the regular version?

Yes, I plan on doing the other versions as well. It takes about 30 minutes now to do each weight.

Waiting patiently for Operator Mono Book, any time frame for when we can expect it?

I would love to see light font support, is this PR dead?

Yes. This is an old PR. It simply merged the Medium weight ligatures into the Book and Light weights.
I have finished true Book weight ligatures. I still need to do the Light weight.
Sorry for the delay.

+1 for Operator Mono Light/Light Italic support. Willing to donate hard $$$ if necessary!

authors: carlevison drod3763 jdelman johnemau kiliman storskegg yamenarahman

repo: kiliman/operator-mono-lig


text:
#62 に対するテスト

#62

repo: kmyk/online-judge-tools


text:
add some UT cases for service measurement

/assign @zhanggbj

/lgtm
@gyohuangxin Thanks for improving the UT coverage!

authors: gyohuangxin zhanggbj

repo: knative-sandbox/kperf


text:
画面 (GUI) の実装

基本的な画面は完成したので Epicとしては閉じることとする

repo: koooyooo/soi-go


text:
In the beginning was the BinWord.

Pull Request Test Coverage Report for Build 26

259 of 259 (100.0%) changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+100.0%) to 100.0%

Totals

Change from base Build 24:
100.0%

Covered Lines:
259

Relevant Lines:
259

💛  - Coveralls

authors: coveralls koriakin

repo: koriakin/binflakes


text:
Issue 3

duplicated

repo: krizald/hn-ui-react


text:
[stable/docker-registry]: fix typo in README

/ok-to-test

/lgtm

/assign

authors: jonboulle unguiculus

repo: kubernetes/charts


text:
Updated .ts bindings file + added .js bindings files

Updated previous ### Changes by fixing a small typo, wanted to notify since I'm not sure if it will break anything
Perhaps I should organize the .js file and .ts files into separate folders, but I taught I would get my current organization system reviewed

repo: kurtosis-tech/kurtosis-client


text:
fix template path

ugh, dup of #158

repo: labstack/echox


text:
[WIP] Rehaul v2

This will need one/two follow-ups but I think we better start merging back in

🎉

@barambani not sure why snyk now reports again two issues from the aws sdk that previously did not report… anyhow… happy to merge?

I'm not sure why Snyk didn't fail on this week's PRs. It could be a glitch. I actually was expecting it to keep failing as we still depend on an affected version. Anyway let's merge, yes.

authors: barambani sirocchj

repo: laserdisc-io/tamer


text:
meaningful error messages, improve cli, remove unused search from daemon

This needs to be broken apart into separate commits.  I'd suggest doing
$ git reset HEAD~1

Which will set you back one commit:
$ git status
On branch error-messages
Your branch is behind 'origin/error-messages' by 1 commit, and can be fast-forwarded.
  (use "git pull" to update your local branch)
Changes not staged for commit:
  (use "git add/rm <file>..." to update what will be committed)
  (use "git checkout -- <file>..." to discard changes in working directory)

	modified:   lbrynet/conf.py
	modified:   lbrynet/dht/protocol.py
	modified:   lbrynet/lbrynet_daemon/Daemon.py
	modified:   lbrynet/lbrynet_daemon/DaemonCLI.py
	deleted:    lbrynet/lbrynet_daemon/Lighthouse.py
	modified:   lbrynet/lbrynet_daemon/auth/server.py

From here, you can do git add -p and selectively add and commit each piece. There are probably 3-5 separate commits in here.
The commit that puts reactor.iterate() back in place should include a commit message explaining the error that happens without that line of code so that when we go to look at it in the future we'll have better context for why its there.
After making all of the commits, do a $ git push -f to force the push and overwrite the original commit.

Regarding

Its not a big deal, but generally I'm a fan of rebasing over merge commits for bringing in changes from master.  Its definitely cleaner and as long as nobody else has started working off of your feature branch its safe.

@jackrobison @jobevers whats the next step here?

There is one outstanding comment to be addressed, after that it can be merged in

authors: jackrobison jobevers lyoshenka

repo: lbryio/lbry


text:
setup codecov

Codecov Report

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

@@            Coverage Diff            @@
##             master      #13   +/-   ##
=========================================
  Coverage          ?   75.74%
=========================================
  Files             ?       15
  Lines             ?      602
  Branches          ?        0
=========================================
  Hits              ?      456
  Misses            ?      125
  Partials          ?       21

Flag
Coverage Δ

#unittests
75.74% <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 7ba73c8...a18f742. Read the comment docs.

authors: codecov-io leberKleber

repo: leberKleber/simple-jwt-provider


text:
Add SMTexas.org

When/will this be accepted? @leereilly I want to know if I should go through with a manuall application to things I need.

Does this school provide post-secondary education? I'm only partially familiar with the US school system.
See the following lines from the README.

Swot is a community-driven or crowdsourced library for verifying that domain names and email addresses are tied to a legitimate university of college - more specifically, an academic institution providing higher education in tertiary, quaternary or any other kind of post-secondary education in any country in the world.

if your institution does not provide any of the above services it cannot be included in swot.
Also:
@20zinnm, Lee Reilly doesn't frequent this repository much, and he tends to lead the maintenance of the project to the contributors. Pull requests are taking longer to accept nowadays due to reduced time spent by the contributors, and the fact that Lee Reilly would need to be the one add new contributors if the existing contributors wanted some more help.

@ReidWeb Thanks for the reply. St. Mark's is the 26'th best private school in the nation, but it does not offer post-secondary education. Thanks for the clarification. I didn't notice that in the readme. While St. Mark's offers a colledge-like atmosphere and very good references, it is not a university.

repo: leereilly/swot


text:
Fix vet error

Codecov Report

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

@@            Coverage Diff             @@
##             develop     #136   +/-   ##
==========================================
  Coverage           ?   59.68%
==========================================
  Files              ?        7
  Lines              ?      516
  Branches           ?        0
==========================================
  Hits               ?      308
  Misses             ?      157
  Partials           ?       51

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 c64bbdb...296e83b. Read the comment docs.

authors: apoorvam codecov-io

repo: leopardslab/dunner


text:
Winuap adopt

Did you intend to close this?
On Thursday, November 26, 2015, Max Lazarev notifications@github.com
wrote:

Closed #631 https://github.com/libuv/libuv/pull/631.
—
Reply to this email directly or view it on GitHub
https://github.com/libuv/libuv/pull/631#event-475674894.

--
/Saúl
bettercallsaghul.com

yes, I created this pull request by mistake.
I tried to make libuv work on window 10 for universal app platform and turned off all features but network. Also I used some tricks to access API hidden by microsoft for uwp.
Pull request requires further investigations, though I tested network on desktop, ms surface and nokia lumia 735 phone in developer mode, and it seems to be working

authors: kkdaemon saghul

repo: libuv/libuv


text:
Fixes #219; serve requested resource if missing mime type

Ready for merge.

Tested it out, fixes the issue. :+1: from me.

authors: deiu dmitrizagidulin

repo: linkeddata/ldnode


text:
Made event submission synchronous to avoid missed events.

@pcadabam can you review?

LGTM

authors: ibuenros pcadabam

repo: linkedin/gobblin


text:
Added OpenVPN

Hi,
Thanks for your contribution!  The project was added into the linuxfoundation:develop branch.

repo: linuxfoundation/cii-census


text:
Updated version for release of version 0.18.1

 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 oscarbatori

repo: liquidata-inc/dolt


text:
Thumbnails

your tests are failing:
https://jenkins.liquiddemo.org/liquidinvestigations/hoover-snoop2/1835/3/7

ERROR    snoop.data.tasks:tasks.py:376 #4175 thumbnails.get_thumbnail(['c85700df5fc73c306b6367e5c6a66d69cb3bc275fc370eb011510d50ad47f95e']) [error] failed: TypeError("unsupported operand type(s) for +: 'NoneType' and 'str'") [0.000 s]
--

You have to:

make everything work correctly even if the thumbnail service is not there

if you don't have the service, just don't crate the extra tasks!

configure the env with thumbnail service when testing

there's another settings file in the testsuite

Fix the tests and then i'll review.

authors: gabriel-v k-jell


text:
自定义的头部module可以动态设置样式吗

当成普通的 React 组件处理就好，动态改变样式应该不成问题

当成普通的 React 组件处理就好，动态改变样式应该不成问题

我需要在FaltList滚动时改变titleItem的module的一些样式，但是顶部module只会渲染一次，无法动态改变。

你好，是可以的。

在 TitileItemModule 的 onComponentDidMount 生命周期函数，或构造函数中，调用 this.props.navigator.setPrames 设置一个回调函数来监听 FlatList 的滚动

在处理 FlatList 滚动时，通过 this.props.navigator.state 来获取第一步设置的函数（注意非空判断），调用它即可。

你好，是可以的。

在 TitileItemModule 的 onComponentDidMount 生命周期函数，或构造函数中，调用 this.props.navigator.setParams 设置一个回调函数来监听 FlatList 的滚动
在处理 FlatList 滚动时，通过 this.props.navigator.state.params 来获取第一步设置的函数（注意非空判断），调用它即可。

好的，感谢，我去试下😂

authors: YangfanY listenzz

repo: listenzz/react-native-navigation-hybrid


text:
Update README.md

@all-contributors please add @lleontop for bug

repo: lleontop/PySnooper


text:
Add complete refund notify and adjust composer.json file.

Codecov Report

Merging #162 into master will increase coverage by 0.72%.
The diff coverage is 88.37%.

@@            Coverage Diff             @@
##             master   #162      +/-   ##
==========================================
+ Coverage     57.27%    58%   +0.72%
- Complexity      688    703      +15
==========================================
  Files            64     66       +2
  Lines          1793   1836      +43
==========================================
+ Hits           1027   1065      +38
- Misses          766    771       +5

Impacted Files
Coverage Δ
Complexity Δ

src/Requests/AbstractAopRequest.php
72.89% <ø> (ø)
67 <0> (ø)
:arrow_down:

src/AbstractAopGateway.php
67.36% <100%> (+0.7%)
44 <1> (+1)
:arrow_up:

src/Requests/AopCompleteRefundRequest.php
100% <100%> (ø)
6 <6> (?)

src/Responses/AopCompleteRefundResponse.php
66.66% <66.66%> (ø)
8 <8> (?)

Continue to review full report at Codecov.

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

Great!

@gdcfivepercent Waiting for author merge it.

这是全额退款接口吗

@gdcfivepercent 根据 trade_status 判断，全额为 TRADE_CLOSED，部分为 TRADE_SUCCESS

authors: codecov-io gdcfivepercent maxsky

repo: lokielse/omnipay-alipay


text:
Added new hasher class with corresponding hash & verify methods

tests/HasherTest.php:42

May as well use the variable here so if it's changed the test will still pass

authors: MitchellMacpherson sjdaws

repo: loyaltycorp/utils


text:
ios Delay start fixed

For what it's worth...
This PR fixes audio delay on iOS as confirmed by OP of this stackoverflow question.

authors: PonnamKarthik om-ha

repo: luanpotter/audioplayers


text:
Create CoreTemp

Missing .json file extension

repo: lukesampson/scoop-extras

license: Unlicense


text:
Adding Jetbrain's PHPStorm IDE

It's recommended to install [url=https://github.com/lukesampson/scoop/blob/master/bucket/jetbrains-toolbox.json]jetbrains-toolbox[/url]


text:
RegRead only fails on invalid regs, right? might be good to use panic instead of error.

nope because repls

repo: lunixbochs/usercorn


text:
redis: hashing connection pool implementation

@lyft/network-team @htuch

P.S., this change is smaller than it looks. Most of the changes are just fixing tests to use a moved mock.

repo: lyft/envoy


text:
Mentioned native way to do this

@sanemat Maybe a quick merge here just to make sure people are redirected accordingly with the new versions ? :)


text:
Horizontal and vertical spacings between blocks are slightly different

Seems to be fixed!


text:
Address writes counted toward consistency but then dropped when restarting during bootstrapping

Grabbing this, was thinking about it during the write quorum stuff.

Also need to consider data streamed from peers for a new shard that isn't persisted to commit log nor to disk at the end of the bootstrap cycle.
We need to have a mechanism to snapshot the current two hour window to disk too for checkpointing purposes.

cc @prateek
This could probably done w/ incremental versioned flushes to incrementally flush out the data for the most recent 2 hour block and mark the data immutable once we are past the 2hr10min mark.

authors: ben-lerner robskillington xichen2020


text:
[WIP] Ruby-related improvements

Meant to send PR to sspinc:master.

repo: maccman/abba


text:
Fix error when no swap are used

Coverage remained the same when pulling cb8dfb19bf480d3c89743808d5be7e7ad962765a on hiroaki0404:spike-freebsd-swap into 89fd930c84f30b2822b72ef7079e49b855be2473 on mackerelio:master.

repo: mackerelio/mackerel-agent


text:
Update to 1.9/1.10/1.11?

Oh how I'd love to. Unfortunately, the hardest part is keeping the protocol definition up to date, which is why I used fastmc. That hasn't been updated since 1.8 and I haven't seen a fully implemented alternative.

Couldn't you try mineflayer? It seems quite similar

Offtopic, but thanks to @ali6836 I realized that fastmc was actually used by anyone else other than myself. Seems like a really neat project.

@magahet I believe this is fairly similar to fastmc https://github.com/ammaraskar/pyCraft/tree/master/minecraft

I did look at pycraft once. However, they implement only a fraction of the full set of packets. So it's good for making chat bots, but not enough to do everything peon does.
I've plugged away a little at adding 1.11.2 packets to fastmc, but it would take a fair amount of work to get them all.

@magahet (This is a quote from dividuum) "Updating would require you to include all changes made since 1.8 according to the protocol documentation at http://wiki.vg/Protocol_History. It would probably also require that your write custom decoders for new primitives (maybe? I didn't check to see if there is anything new). It's quite fiddly and no easy task."
Just to help you if you do try and add the packets :)

authors: ali6836 dividuum magahet

repo: magahet/peon


text:
Disable logging in npm release

Version 2.0.13 has a configurable and conditional logging. The only logged messages would be the exceptions.

authors: grozdanov kpiascik


text:
Remove PhishFort

:rocket: PR was released in @magic-sdk/provider@3.0.1, @magic-sdk/react-native@3.0.1, @magic-sdk/types@2.0.1, magic-sdk@3.0.1 :rocket:

repo: magiclabs/magic-js


text:
plugin: remove transmutation endpoint

Codecov Report

Merging #303 (c040ca5) into master (e9ef752) will decrease coverage by 0.04%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #303      +/-   ##
==========================================
- Coverage   84.33%   84.29%   -0.05%
==========================================
  Files          55       55
  Lines        4335     4335
==========================================
- Hits         3656     3654       -2
- Misses        679      681       +2

Impacted Files
Coverage Δ

quetz/testing/fixtures.py
97.70% <0.00%> (-1.53%)
: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 e9ef752...c040ca5. Read the comment docs.

Codecov Report

Merging #303 (c040ca5) into master (e9ef752) will decrease coverage by 0.04%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #303      +/-   ##
==========================================
- Coverage   84.33%   84.29%   -0.05%
==========================================
  Files          55       55
  Lines        4335     4335
==========================================
- Hits         3656     3654       -2
- Misses        679      681       +2

Impacted Files
Coverage Δ

quetz/testing/fixtures.py
97.70% <0.00%> (-1.53%)
: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 e9ef752...c040ca5. Read the comment docs.

authors: btel codecov-io

repo: mamba-org/quetz


text:
RELEASE

Pull Request Test Coverage Report for Build 17

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

Totals

Change from base Build 16:
1.4%

Covered Lines:
54

Relevant Lines:
70

💛  - Coveralls

authors: aljesusg coveralls

repo: managef/api


text:
Create INSTALL.md

I personally prefer seeing things in one place, at least the important things.  When people have to click around, they tend to miss things.  So I personally would prefer to see the install information on the main page.
What happened to the order-by comments?

No it's too long to be in README.
Regarding the Order-by comments, we need to move it to a different place - it's too verbose and needs to be outside of the main README. I'm trying to figure out the right place for it. I'm thinking of a docs directory and put it in that.

We should also document what parameters to change in Drill in order to run the test framework.  Things like Direct Memory.  We can't ask Abhishek to tune each person's setup.  Khurram and i both ran into problems with not allocating enough Direct Memory.

The order-by documentation is verbose because testing order-by is complicated.  If not done properly, people will think they are validating order-by tests when they are not.  Especially for new QA testers.  I'm not saying it should be in the README, but if not put in a visible place, it may be ignored.  Testing SQL is not easy, and I did not realize what I was missing with order-by testing until I dug into it.

I agree - I wrote the initial doc with this in mind. But I've come to realize that we can do better. I'm not sure where you got the message that i'm removing stuff to be unhelpful.
I'm thinking of keeping the primary README as an overview page. I think i'll move back some of the stuff back here - such as the usage details. We need to update the parameters as well (it's partly outdated).
Now things like Direct memory configuration is outside the purview of this repository. It requires basic understanding of Drill administration - we could add a note about it in a requirements section and point to Apache Drill docs.

Regarding the ORDER-BY documentation, I think it's a good thing to have details. My issue is with where to keep the documentation and to rewrite parts of it to be more concise and with better formatting.

Take a look at https://github.com/agirish/drill-test-framework/tree/install (work-in-progress) to see how this will look like once merged.

Folks, please take a look to see if any of you have other concerns. If not we can get this in soon.

Looks good.
I noticed a couple of typos.
In the Execute Tests section, the explanation for the -n option should have "20" to match the example.
On the Contributing page, the title "Structure of test definiton files" needs an "i" in definition.

The order by section looks much better now with proper tables.  Thanks for doing this.

repo: mapr/drill-test-framework


text:
Support for RTL languages

You can use my package, it supported RTL
https://pub.dev/packages/marquee_widget
https://github.com/yousifk/marquee_widget

authors: marcelgarus yousifk

repo: marcelgarus/marquee


text:
Update README clarifying Grafana instructions

rebased and merged as https://github.com/markfisher/sk8s/commit/9ddc63c9cc89dccec48d59c7fbf0acdbc5c227f2

authors: markfisher trisberg


text:
MAQL DML support

Coverage increased (+0.21%) to 81.14% when pulling 385db62528bb432b9b43e6ba4e7b8625715a731c on sadam21:dml into 267e80498bebca0fc9b38230cd7526c4e4644235 on martiner:master.

Coverage increased (+0.21%) to 81.14% when pulling 49162d2b62ba08a885f8d8421699e138748a96e2 on sadam21:dml into a12363c3f73fbe6046859e0b073b1b749042a12e on martiner:master.

looks terrific. one more think: can you please write a simple acceptance test in ShowcaseAT?

[ ] update readme

Readme updated, AT added.

Coverage increased (+0.22%) to 79.7% when pulling 897ee7a9bb19d7dafcedfd48cdfa0f35dcc23f8f on sadam21:dml into f1fa84bdad6253530108a4069ec9329bf87afcc4 on martiner:master.

authors: coveralls martiner sadam21

repo: martiner/gooddata-java


text:
[All] Add links to CII Best Practices (v, partly)

Added in 9f5e925d0d4a803bc68a4587fa6b8017247025f1

repo: martinmoene/catch-lest-other-comparison


text:
Move C++98 support to a branch named 'legacy'

How would you explain the issue?

Hi Martin, well probably like this:
It is not feasible to go bellow C++ version that is more than 8 years old. For good people in a real "need" for C++98, let them congregate around (and fork from) the  "legacy" branch.
I hope onlookers do realise C++17 and C++98 are two different languages.
Important to notice: the suggested branch is not to be called "deprecated" it is to be called "legacy".  Contributors who need this version are welcome to contribute.
Something along these lines?

@DBJDBJ: I believe Martin's question is, why should this be done? It sounds like (A) you've asked for some work to be done in the repo; (B) you've just now given a reasonable explanation as to why this work would not be very harmful to most users; but you haven't given any thought to (C) whether this work would be helpful to anybody.

@Quuxplusone ... feasibility Arthur :) Ie the amount of effort to make something that is useful and already exists in C++11 onwards or (god forbid, only) in the standard C++ onwards. So this work (which I have not specifically asked for) would be helpful to the maintainers of the project. First.

authors: DBJDBJ Quuxplusone martinmoene

repo: martinmoene/ring-span-lite


text:
ImportError: No module named 'libs.sepconv._ext'

This error derive from "./libs/sepconv/SeparableConvolution.py" ,its import part said "import libs.sepconv._ext as _ext"  .However,I didn't find  lib.sepconv._ext   either. Did I miss some setups? Could anyone tell me ? 🙏

Hi,I have the same question above.I wonder if you get the problem solved.

authors: ChillaxBoy Otianner

repo: martkartasev/sepconv


text:
Update issue templates

Codecov Report

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

@@           Coverage Diff           @@
##           master     #112   +/-   ##
=======================================
  Coverage   70.88%   70.88%
=======================================
  Files          26       26
  Lines        2195     2195
  Branches      241      241
=======================================
  Hits         1556     1556
  Misses        618      618
  Partials       21       21

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 6c7b8e2...eff013b. Read the comment docs.

authors: codecov-io oldfly

repo: marvin-ai/marvin-python-toolbox


text:
API: remove gap_freq estimation in estimators, prune by default

Codecov Report

Merging #132 into master will decrease coverage by 0.00%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #132      +/-   ##
==========================================
- Coverage   83.16%   83.16%   -0.01%
==========================================
  Files          11       11
  Lines         980      974       -6
  Branches      145      145
==========================================
- Hits          815      810       -5
+ Misses        116      115       -1
  Partials       49       49

Impacted Files
Coverage Δ

celer/dropin_sklearn.py
89.78% <100.00%> (+0.27%)
: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 bcaf068...f39d02f. Read the comment docs.

authors: codecov-commenter mathurinm

repo: mathurinm/celer


text:
Fix #2 by updating SDK version, build tools, etc

It builds fine for me after applying it.

i've added various questions, plus I suspect we'll be quite paranoid about upgrading to 23 and will test it a bunch before merging.

1 - i tested gradle-2.11 this week-end and it crashed a few times whereas it does not crash with oldest gradle versions.
2 - the build tools and the sdk version should be the same i.e. SDK 23 and builtools 23 (+ compat).
It is not common to use an old SDK with a recent buildtools (because of compat issues).

Remaining with an old version of software isn't a good idea, because eventually it will be unsupported. If we keep up with the current version, not only will we gain a bunch of performance improvements and bug fixes, but we'll be more agile on future changes.

I'm surprised that you've had issues with gradle 2.11 crashing. Google has developers use gradle directly, and they're using gradle 2.10/2.11 in their next build of Android Studio (the one with continuous deployment). If Google is recommending it, it should be a safe bet.

I've updated the builds tools and SDK version! I had to change notifications a little bit, but it still works (at least on an Android M device)

authors: ara4n mitchhentges pvagner ylecollen

repo: matrix-org/matrix-android-console


text:
Fix and include /directory api docs

Does this obviate the need for https://github.com/matrix-org/matrix-doc/pull/100 ?

Yes, probably. Shame about that.

LGTM!

authors: Kegsay illicitonion

repo: matrix-org/matrix-doc


text:
Scope and tasks

[ ] Specify tasks and scope

[ ] Orchestrator

[ ] Ask for a term to search
[ ] Ask for an prefix
[ ] Start robot of text
[ ] Start robot of images
[ ] Start robot of vídeo
[ ] Start robot of Youtube

repo: mayckondf/video-maker


text:
Added diagnostic messsages for rplidar error states

other than the above comments, lgtm


text:
consistency

simply #consistencyPolice
Synced KVS sends delete keys in array. No reason for local to be different.

repo: mediafly/mflyCommands


text:
Refactor Pipeline

The API definition has been moved to membraneframework/membrane_sfu#2


text:
Creation de pull request à la place de Mouhamed-nour

Si tu avais fait la pull request après avoir pushé ton code, j'aurai pu être notifié que tu avais fait quelque chose et j'aurai pu commenter ton code et te donner des indications pour faire mieux.
Là, ton code pour l'IA, malheureusement, a beaucoup d'imperfections qui, notamment, font que c'est très lent. Je n'ai plus de temps pour te donner des pistes d'amélioration, désolé.


text:
finalize arabic version

@ronaldtse there is an error that I don't get
what I've done was replacing all the ٪ with %
anything special about % with arabic that am not aware of?
worth mentioning that ٪ is the arabic version of %, it has a Arabic mapping, both look the exact same though, just a circle instead of a dot

The replacement is done in my code, and my code will differentiate between ASCII % and Arabic ٪. That's OK, I'll take care of it.

repo: metanorma/isodoc


text:
Fix content

Landed in https://github.com/metarhia/metarhia.com/commit/fb1d2c868b47aebd2703f1bb9d98ba407754484f


text:
Fix formatting directives in tests

Ah, thank you! :+1:

repo: mholt/caddy


text:
Create a archetype for new OSGI projects

No need.

repo: mhus/mhus-osgi-tools


text:
Trivial change.

The goal of this PR was to have the latest commit in the main branch that would not have " in the commit message. Since our CI is pretty backed up now, I'll close this PR to reduce the load, and will merge #540 instead to reach the same objective.

authors: anpaz tcNickolas

repo: microsoft/QuantumKatas


text:
br-ex로 기존의 host only 어댑터 내용을 덮어쓰고 이어준 부분은, 후에 설명키로

https://www.youtube.com/watch?v=eOlIB323c8s
이 동영상은 이미 전설임

repo: microstack/front


text:
Fix guest's DOCUMENT_ROOT (see: #6e20b10)

あっ、すでに修正済みですね

oops!


text:
Correct a spelling mistake.

Thanks!

authors: EdwardBetts mkaz

repo: mkaz/termgraph


text:
test

Made this by accident, please ignore.

Made this by accident, please ignore.

repo: moov-io/ach


text:
Feature/update example

yeah. we can do both of them.

repo: morpheusgraphql/morpheus-graphql


text:
add Gavin Schreiber profile

Thanks @GavinCS
Remember to tweet and come say hi if you havent already

authors: GavinCS mostertb


text:
Rebased 3d branch

@mourner any comments?

repo: mourner/simplify-js


text:
Agrego pruebas de batalla

Codecov Report

Merging #11 (bd55a70) into master (147f190) will increase coverage by 5.59%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master      #11      +/-   ##
============================================
+ Coverage     70.27%   75.86%   +5.59%
- Complexity       25       32       +7
============================================
  Files            12       12
  Lines            74       87      +13
  Branches          2        4       +2
============================================
+ Hits             52       66      +14
+ Misses           22       21       -1

Impacted Files
Coverage Δ

src/main/java/edu/fiuba/algo3/modelo/Batalla.java
100.00% <100.00%> (+100.00%)
:arrow_up:

src/main/java/edu/fiuba/algo3/modelo/Pais.java
100.00% <100.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 147f190...bd55a70. Read the comment docs.

authors: SantiagoVaccarelli codecov-commenter


text:
Add cycle log related APIs

woot~

authors: rdiankov ziyan

repo: mujin/mujincontrollerclientpy


text:
fix: renamed endpoint for sharing public key

@jessiematias Thanks a lot for the fix.  :)


text:
Fix the links pointing to the repository

Thanks!
I've modified the readme.md file instead. Doc. is generated from this file.

Thank you!

authors: ankon mathieucarbou

repo: mycila/license-maven-plugin


text:
fix receiver crushes on exit

Thank you for reporting!
In my first look, a crashing occured from destroying a thread while it's running.
If the crash you reporting was same to that, I think the fix will be this.
https://github.com/nariakiiwatani/ofxNDI/commit/abe9742af4d2b846e909b6ec690a155312f5fca9
If not, I need more information about your crash.
I couldn't reproduce any other crashing.

repo: nariakiiwatani/ofxNDI


text:
5.3.1 version

those two commits are already merged, please do an interactive rebase or remove those commits some other way

authors: locusf neochapay

repo: nemomobile/qtquickcontrols-nemo


text:
add speedrun leaderboard sorted by wallclock time

done

repo: nethackscoreboard-org/matter


text:
add nesta

Looks like this is a flat file CMS rather than a Static Site Generator, at least I don't see any option to simply output a static folder...

You're right, I made a mistake

authors: biilmann shark-oxi

repo: netlify/staticgen


text:
move AUDIT_INFO attachment to AttachmentConstants

 move AUDIT_INFO attachment to AttachmentConstants master

 move AUDIT_INFO attachment to AttachmentConstants


text:
Add Aggreagte Methods(Sum, Count, etc...)

duplicate of 105 issue.

repo: neuecc/UniRx


text:
Testing if jQueryUI has loaded

Thanks!

authors: Develle ninsuo

repo: ninsuo/symfony-collection


text:
Translate chapter 05 into Japanese.

18行目、思い出してください → 覚えておいてください
のほうがいいかなと思います。

@tako-black たぶんここは、chapter00やchapter01での説明を参照しているのだと思いますので、remember = 思い出せ、でいいかと。

@tai2 すみません確認不足でした！OKです！

repo: nodeschool-jp/how-to-npm

license: ISC