text:
fix: bug with unable to tap on first found element

Changes are small and I well tested it with unit testing and with hermione tests, so I will merge it without review.

repo: gemini-testing/hermione-safari-commands


text:
updates to line up with molecular event

Taking early as @goodb pinky promises it's fine.

source: gharchive/pull-request

authors: goodb kltm

repo: geneontology/minerva

license_type: permissive

license_source: bigquery


text:
いいところ診断にやさしさを追加したい

対応します

source: gharchive/issue

license: MIT

license_source: github-api


text:
Changed case on GoCD to preferred format

Thanks a lot for this PR!

authors: kmugrage mahendrakariya

repo: getgauge/gauge


text:
feat: Upgrade to sentry-java 5.0.0

J

authors: AbhiPrasad HangRathtana

repo: getsentry/sentry-spark


text:
Release 0.10.0

Coverage increased (+0.3%) to 74.383% when pulling 0a92a66f60ef2dd8fdcc43b40472d4dba94f8df5 on philipc:release into 42990e42e64ad97dec7bb1ae4dd93bff2221eab2 on gimli-rs:master.

authors: coveralls philipc


text:
first

Test

repo: githubschool/github-games

license: mit


text:
Change title from Erich Gamma to "VS Code Dev Lead @Microsoft"

Fixed by https://github.com/gitpod-io/devxconf.org/pull/51/commits/73300507746651e0a580030e1b406f9df08fec27 will be live once that Pr is merged.


text:
Flag error strings that end with a newline

Sending this PR as a proposal with attached code for a new check.

@dsnet ping

Maybe, I'm missing something (I'm not as familiar with GitHub's PR system), but I still see the same code as prior to all the comments. Was there changes made?

@dsnet IIRC the ping wasn't because of new code, but because you asked for stats which I have now provided, based on which you wanted to make a decision.

PTAL

Merged as 206c0f020eba0f7fbcfbc467a5eb808037df2ed6.

authors: dominikh dsnet

repo: golang/lint


text:
Make kSqrtTwoPi const

@wally-the-cartographer merge

Merge requested by authorized user kdaun. Merge queue now has a length of 2.

authors: kdaun wally-the-cartographer

repo: googlecartographer/cartographer


text:
feat: add alisls writer

Test failed

Test failed

Test failed

Without Ali cloud sls AK configured, the test will be executed fail, I remove this test case later.

Test failed

Without Ali cloud sls AK configured, the test will be executed fail, I remove this test case later.

authors: flycash sevennt

repo: gotomicro/ego


text:
Add toolchain support to release notes

@bot-gradle test QF

OK, I've already triggered QuickFeedback build for you.

@pioterj I addressed your comments and also moved the demo to be right before the documentation links, after all the text. I could not find a good place for it earlier without breaking the flow of the explanation.
Let me know what you think of the content now.

@ljacomet Could you give me a link to the rendered version?

Changes included in #14489

authors: bmuskalla bot-gradle ljacomet pioterj

repo: gradle/gradle


text:
How to send files from front end to backend ?

The  https://github.com/graphql-java-kickstart/graphql-java-servlet project supports https://github.com/jaydenseric/graphql-multipart-request-spec

Sample application demonstrating it: https://github.com/graphql-java-kickstart/samples/tree/master/file-upload

authors: Dluquem oliemansm osi


text:
massage the About pages a bit more

Ready to go.


text:
Mine

Goddammnit github


text:
fix VEGUI test to work with latest storage test branch

Commit 114723ece5bcd4e8811eb1d32d55fde85e2319d9 fixes the issue.

authors: bstabler goreaditya

repo: gregorbj/VisionEval


text:
Upgrade to Stylus 0.54

Hey, that was perfectly quick :-D

:)

Hm thought it was green, any idea what happened to this option:
Running "nodeunit:tests" (nodeunit) task
Testing stylus_test.js....F.........
>> stylus - embedurl
>> Message: `embedurlOpts` limit should prevent Data URI embed
>> Error: '.normal{background-image:url("test.png")}.base64not{background-image:url("data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAAAoAAAAKCAYAAACNMs+9AAAATUlEQVQYV5WQAQoAIAgD7f+PrhZM5ioiQSS81laLiD77Wc1AnFlFQEGFNpigToVSyEFA/sqycFLyYAXkJUws6DcDfYdxG1S9KipQ/ngAy3ccAe3ha2EAAAAASUVORK5CYII=")}' == '.normal{background-image:url("test.png")}.base64not{background-image:url("test.png")}'
>> at Object.exports.stylus.embedurl (test/stylus_test.js:69:10)

?

cc @thybzi

Strange thing, I'll try to investigate

For now, seems that limit option set value is somehow overridden to false.
Continuing investigation.

Gotcha! Here is it:
https://github.com/stylus/stylus/commit/fb296c59f4639733e654317d4426807418e6b83f#diff-2484967dbaf30d736500a330414ff5d1R220
Now trying to find out why that false value was ever needed here, and how to avoid it

Oh Snap! :snake:

Now I'm sure that it's just naming conflict with their new bif called embedurl
I think we should rename our tests function having the same name.
Should it be embedurllimit or whatever?

@thybzi sure, sounds good!

@vladikoff No, my bad. This function name is used not only in "size-limited" cases.
Preparing pull request with rename embedurl (defined in grunt-contrib-stylus) it to data-uri().

Fixed here https://github.com/gruntjs/grunt-contrib-stylus/pull/151
Please accept, and we would be able to upgrade to stylus 0.54.0

authors: thybzi vladikoff

repo: gruntjs/grunt-contrib-stylus


text:
it broken

Thank you

authors: gsckoco itsdrawn

repo: gsckoco/lma


text:
Test on PHP 7.1

:)

repo: guzzle/guzzle


text:
Jenkins server is down.

The Jenkins server has been moved over to https://jenkins-gvl.rhcloud.com/

May want to update the readme on the git page and the spigot page.

I hadn't realised there were links for it there. I've update them on all three websites.

Jenkins go down again? :(

If it says it's down for maintenance or such usually reloading the page a couple time forces it to wake up again. Sometimes you have to close the tab and open another to force reload and not pull the page from the cache.

Quite odd. It was down entirely, unreachable. But seems to be back up now, so all is good. :)

license: unlicense


text:
update query result displayed in tableview_clients_query after clicking overwrite or delete

done


text:
implemented email-only newsletter formular

@all-contributors please add @dsumer  for code

@all-contributors please add @dsumer  for code

authors: dsumer hauptrolle

repo: hauptrolle/chakra-templates


text:
ref(go.mod): kubernetes 1.16.2

Solves #6783 for v3

repo: helm/helm


text:
[06 - R&M reference] Igor Kholopov,493

Принято

authors: IKholopov pkuderov

repo: hisubbotin/net-study


text:
how

I guess this isn't any issue?

authors: hjalmers sahilkatia

repo: hjalmers/angular-generic-table


text:
[DOR-50] Upload your documents (Importer/Exporter)

Again something you're most likely aware of but will need a rebase as Sam's changes were merged this morning and there are some conflicts

Oh sorry forgot to pop above but the tests won't run for me after pulling the newest commits (they did spit out some errors previously (before most recent commits) but once they ran they all passed). Assuming this is more to do with needing a rebase or conflicts than anything major though not totally sure

repo: hmrc/trader-services-route-one-frontend


text:
more mac OS app style Icon

What’s different about this?

authors: BobbySlope mkellsy

repo: hoobs-org/desktop


text:
add Slack token verification

https://github.com/howdyai/botkit/issues/967

https://github.com/howdyai/botkit/issues/966

I am bumping this up. Let me know if this approach is not favoured.

@shishirsharma thanks for the reminder we'll try to process this one soon.

authors: benbrown shishirsharma

repo: howdyai/botkit


text:
Upgrade to jetty-9.4.39

FWIW, after upgrading to http4s 0.21.21, we ran into https://github.com/eclipse/jetty.project/issues/6121 when using jetty client, had to downgrade to 0.21.20

Thanks for the report.  I guess let's see if they can fix this in the 9.4 series.  I'll keep an eye on the other ticket.

authors: anilkumarmyla rossabaker


text:
Appstore 下架了？

cuImage1.4.zip

好同志！

authors: hulizhen searich

repo: hulizhen/cuImage


text:
Update to API change in hpp-corbaserver.

@ymontmarin The way we read and write roadmaps has been updated. This may impact you.
The file format has completely changed (we switched to boost serialization) and there is no built-in way of converting old roadmap files so those files are lost.

@ymontmarin The way we read and write roadmaps has been updated. This may impact you.
The file format has completely changed (we switched to boost serialization) and there is no built-in way of converting old roadmap files so those files are lost.

@jmirabel Thank you for letting me know !

@jmirabel Thank you for letting me know !

authors: jmirabel ymontmarin

repo: humanoid-path-planner/hpp-gui


text:
Possible to construct X509Certificate2 with private key?

Can you elaborate what you're trying to do?

With net core standard library, a X509Certificate2 with private key can only be constructed from a pfx cert file. My question is that, does pem-utils provides APIs to constructe a X509Certificate2 from pem files.  Please let me know if that does not make sense.

repo: huysentruitw/pem-utils


text:
Add <User/Guild/Role/Channel>.createdAt

Forgot to do docs.

Hi, can you store the date as a Unix timestamp _createdAt, and make a getter that returns it as a new Date? It should reduce memory usage.

Or alternatively, remove _createdAt altogether and make it a getter

authors: Hyper-Coder hydrabolt


text:
Corda Support for arbitrary function calls in case of data transfer

@sanvenDev What was this issue about?

In Fabric, we have the option to invoke an arbitrary chaincode function when writing an external view to the ledger. But in Corda, this is hardcoded. We need to fix that.
Also, support an array of view requests corresponding to a single view writing (locally) in Corda, just as we do in Fabric.

authors: VRamakrishna sanvenDev


text:
Add possibility to serve spec

lgtm

authors: hypnoglow tetafro


text:
Dev

merge

merge


text:
Constrain organism set

Done. Now limited to the three IBISBA ones

repo: ibisba/hub


text:
Development

Changes Unknown when pulling d73c354cf30ba3bd02df7b33e7f0be6b83620dec on development into ** on master**.

authors: cirilla coveralls


text:
Loosen URL dependency version

Thanks for the PR! You're right, semver allows this and it should eliminate multiples of url where none would be actually necessary. I'll publish the updated crate soon.


text:
More integration testing

Test-code only, self-merging.

repo: influxdb/influxdb


text:
[DO NOT MERGE] Version 7.1.0

@erikd Can you please leave the commit 3cf868ec8c06265e4b670ac737af640d716e5ef7 separate and not rewrite Git history? For now, I'm depending on that commit and once you have things ready we can sync up and I can target your merge commit.

If no further issues are found and this does indeed become the 7.1.0 release, this commit will remain. If an issue is found and it needs to be corrected, this commit will be replaced by the fix commit.

If no further issues are found and this does indeed become the 7.1.0 release, this commit will remain. If an issue is found and it needs to be corrected, this commit will be replaced by the fix commit.

Yes, but could you just temporarily add a new commit and squash them when it's finalized?

authors: erikd ksaric

repo: input-output-hk/cardano-db-sync


text:
Make backups sensitive to cluster name, not only to datacenter

https://github.com/instaclustr/cassandra-backup/commit/cdba9424c3134316cd98645235de8b74aba06c00

repo: instaclustr/cassandra-backup


text:
adding bunsen POST search to Ionic library

This checks out in Rosita, I vote that it's ready to merge

repo: ion-channel/ionic


text:
Anlegen eines neuen Workspaces (superadmin) scheitert

Kann ich nicht reproduzieren, weder lokal noch auf OCBA..?

offenbar gelöst

authors: mechtelm paflov

repo: iqb-berlin/testcenter-iqb-php


text:
更新后 热更新不能用了,改了代码也不会自动刷新页面

更新到iview-admin最新版吗？你是修改 了什么吗

repo: iview/iview-admin


text:
Travis: Add Node.js 10 and remove the obsolete sudo.

Sorry for not looking at it earlier. We can probably follow the same story as in clean-css: ditch Node.js 4 and include Node.js 12.

@jakubpawlowicz just squash it when merging :)

:+1:

authors: XhmikosR jakubpawlowicz

repo: jakubpawlowicz/clean-css-cli


text:
create a lot more e2e functional tests

at least 1 e2e positive tc

repo: jameslinjl/SHPJavaGrader


text:
Configurable USER AGENT and permanent auth support

Has this code ever worked? I am asking because I am looking at it and I noticed that you assume that what runCurl() returns is an array while in reality it return objects. It puzzles me somewhat.

authors: creesch redditbotpt

repo: jcleblanc/reddit-php-sdk


text:
Modify architecture

done

repo: jdi-testing/jdi-dark


text:
Add Epic Games to sites.json

Someone hasn't enjoyed playing "Fork knife" hahaha
jk, thanks for the contribution!

authors: Nightfirecat tupaschoal

repo: jdm-contrib/jdm


text:
Correct all arrow alignments using puppet-lint

Doh! I just opened a PR for this too.  I will merge this when travis finishes.

authors: jhoblitt vStone

repo: jenkinsci/puppet-jenkins


text:
运行python proxyPool.py schedule错误，提示python: can't open file 'proxyPool.py': [Errno 2] No such file or directory

具体日志贴上来

具体日志贴上来

就是提示python: can't open file 'proxyPool.py': [Errno 2] No such file or directory

要么代码版本不对，要么运行位置不对，提示很明显了找不到文件，要到cli目录下

莫名其妙的好了…不知道为啥
发自我的iPhone
------------------ 原始邮件 ------------------
发件人: J_hao104 <notifications@github.com>
发送时间: 2019年8月29日 19:33
收件人: jhao104/proxy_pool <proxy_pool@noreply.github.com>
抄送: Subscribed <subscribed@noreply.github.com>
主题: 回复：[jhao104/proxy_pool] 运行python proxyPool.py schedule错误，提示python: can't open file 'proxyPool.py': [Errno 2] No such file or directory (#342)
要么代码版本不对，要么运行位置不对，提示很明显了找不到文件，要到cli目录下
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

要么代码版本不对，要么运行位置不对，提示很明显了找不到文件，要到cli目录下
没有cli目录啊

要么代码版本不对，要么运行位置不对，提示很明显了找不到文件，要到cli目录下

没有cli目录啊

要么代码版本不对，要么运行位置不对，提示很明显了找不到文件，要到cli目录下

没有cli目录啊

我知道了，我在relases里面直接下载的没有cli目录

authors: banzuan jhao104 zhuhuadeaa


text:
[MRG] Supervised base class changed

Pull Request Test Coverage Report for Build 732

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

Totals

Change from base Build 727:
0.004%

Covered Lines:
1874

Relevant Lines:
1960

💛  - Coveralls


text:
Fix minor typos

Thanks!

authors: aidos jhollingworth

repo: jhollingworth/marty


text:
Store Heroku name in file

Done


text:
safe -k Flag for Skipping SSL validation

v0.0.16 should have this feature.

repo: jhunt/safe


text:
Custom ffmpeg command line

Fixed in #227


text:
improved docs

thanks!

authors: joshmarinacci virendrakini

repo: joshmarinacci/css-tables-demo


text:
Update index.html

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

Needs approval from an approver in each of these files:

OWNERS

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

:star: PR built and available in a preview jstrachan-nodey532-pr-9 here


text:
CHANGELOG.md - update for 0.6.0 release

cc: @dmajda

+1

authors: bkutil dmajda

repo: juttle/juttle-influx-adapter


text:
Allow #Proxy to Abort #Open Entirely

Middleware #10 changes behavior to allow proxies to pass through to next proxy, or discontinue if route or parameters are modified.

repo: jwalapr/Routing


text:
Create deploy task to work with kustomize

Will be shipped in the gitops release

Created under experimental-pipelines

repo: kabanero-io/kabanero-pipelines


text:
Modified to run in jetty

Can we merge? @msneddon


text:
Manage Team from Backend

Current iteration seems to be enough for now - may revisit if it proofs not to be

repo: kealsera/rikki


text:
Can't work with HDRP?

It does work on HDRP.

I'm closing this issue now. Please feel free to reopen it for further problems.

Once Project Settings>Graphics>SRP Settings is none, both Spout and Spyhon work well, but if SRP Settings is "HDRenderPipelineAsset", either can't work (nothing shows in TouchDesigner's syphonspoutin.

Did you try the render texture mode? As noted on README, the camera capture mode is not supported on SRP (URP/HDRP).

Thanks, but still can't work with Spyhon, and have found there is the same question on KlakSpyhon page. Looking for any solution for Spyhon.

First of all, don't mix up Spout and Syphon. They're completely different technologies.
Please make it clear if you're trying to use Spout on Windows, or Syphon on Mac.

authors: Tongzhou-Yu keijiro

repo: keijiro/KlakSpout

license: Unlicense


text:
git harvest args wrong

Close this since the harvest command has been removed.

authors: kennethreitz weakish

repo: kennethreitz/legit


text:
Is remove-destination alright with non-function-like configuration or destination?

This isn't really relevant anymore.

repo: kframework/kore


text:
Clean up the provision script

The main reason is that the non-binary deployment never works, so we shouldn't con people in thinking it does.

jenkins retest this

repo: kiegroup/optashift-employee-rostering


text:
change header sizes on xs breakpoint only.  update docs.

Deploy preview for vibrant-yonath-a27de1 ready!
Built with commit 3570298ba63a06be7799855fc9277302770db45a
https://deploy-preview-8--vibrant-yonath-a27de1.netlify.com

repo: kimfucious/sakura


text:
implementation request: Lykke

+1

This exchange was implemented

authors: earce makarid npomfret

repo: knowm/XChange


text:
Attempt 2 at returning a serializable config

Tested this with osqueryd TLS plugins.

repo: kolide/fleet


text:
Apply sonar

/it oracledb-it

/it google-grpc-it

/it google-grpc-it

/it oracledb-it

/it oracledb-it

/it

/it gson-it

/it gson-it

/it gson-it

/it gson-it

/it gson-it

/it gson-it

/it gson-it

/it gson-it

/it gson-it

/it

/it

repo: koo-taejin/pinpoint


text:
Add save button in editor page

Add styled extension main page


text:
Fix a type in 10_associateBy.md

Thanks :)

authors: AlexanderPrendota tatsuya-fujisaki


text:
Show usage of after_configuration_change

Thanks, @ramontayag! 👍

authors: manuelmeurer ramontayag


text:
Update proxy.md

@chaneylc thanks! Merging

authors: andreyaksenov chaneylc

repo: ktorio/ktor-documentation


text:
Added new HUD styles BlurLight, BlurExtraLight and BlurDark

Could you please make new pod release? Thanks!

repo: kubatru/JTProgressHUD


text:
Add node to docs about kubelet deployment type changes

@bogdando

repo: kubernetes-incubator/kubespray


text:
remove redundancy code

ci check this

authors: chenhonggc riverzhang


text:
IGNORE: Another test issue

Update

Update

[MILESTONENOTIFIER] Milestone Issue Needs Attention
@marun @kubernetes/sig-cli-bugs
Action Required: This issue has not been updated since Oct 5. Please provide an update.
Note: This issue is marked as priority/critical-urgent, and must be updated every 0 days during code slush.
Example update:
ACK.  In progress
ETA: DD/MM/YYYY
Risks: Complicated fix required

Issue Labels

sig/cli: Issue will be escalated to these SIGs if needed.
priority/critical-urgent: Never automatically move out of a release milestone; continually escalate to contributor and SIG through all available channels.
kind/bug: Fixes a bug discovered during the current release.

Help

Additional instructions
Commands for setting labels

[MILESTONENOTIFIER] Milestone Labels Incomplete
@marun
Action required: This issue requires label changes. If the required changes are not made within 0 days, the issue will be moved out of the v1.8 milestone.
priority: Must specify exactly one of priority/critical-urgent, priority/important-longterm or priority/important-soon.

Help

Additional instructions
Commands for setting labels

[MILESTONENOTIFIER] Milestone Issue Needs Attention
@marun @kubernetes/sig-cli-bugs
Action required: During code slush, issues in the milestone should be in progress.
If this issue is not being actively worked on, please remove it from the milestone.
If it is being worked on, please add the status/in-progress label so it can be tracked with other in-flight issues.
Note: If this issue is not resolved or labeled as priority/critical-urgent by the time heck freezes over it will be moved out of the v1.8 milestone.

Issue Labels

sig/cli: Issue will be escalated to these SIGs if needed.
priority/important-soon: Escalate to the issue owners and SIG owner; move out of milestone after several unsuccessful escalation attempts.
kind/bug: Fixes a bug discovered during the current release.

Help

Additional instructions
Commands for setting labels

[MILESTONENOTIFIER] Milestone Issue Needs Attention
@marun @kubernetes/sig-cli-bugs
Action required: During code slush, issues in the milestone should be in progress.
If this issue is not being actively worked on, please remove it from the milestone.
If it is being worked on, please add the status/in-progress label so it can be tracked with other in-flight issues.
Note: If this issue is not resolved or labeled as priority/critical-urgent by the time heck freezes over it will be moved out of the v1.8 milestone.

Issue Labels

sig/cli: Issue will be escalated to these SIGs if needed.
priority/important-soon: Escalate to the issue owners and SIG owner; move out of milestone after several unsuccessful escalation attempts.
kind/bug: Fixes a bug discovered during the current release.

Help

Additional instructions
Commands for setting labels

[MILESTONENOTIFIER] Milestone Issue Needs Attention
@marun @kubernetes/sig-cli-bugs
Action required: During code slush, issues in the milestone should be in progress.
If this issue is not being actively worked on, please remove it from the milestone.
If it is being worked on, please add the status/in-progress label so it can be tracked with other in-flight issues.
Note: If this issue is not resolved or labeled as priority/critical-urgent by the time heck freezes over it will be moved out of the v1.8 milestone.

Issue Labels

sig/cli: Issue will be escalated to these SIGs if needed.
priority/important-soon: Escalate to the issue owners and SIG owner; move out of milestone after several unsuccessful escalation attempts.
kind/bug: Fixes a bug discovered during the current release.

Help

Additional instructions
Commands for setting labels

[MILESTONENOTIFIER] Milestone Issue Current
@marun
Note: If this issue is not resolved or labeled as priority/critical-urgent by the time heck freezes over it will be moved out of the v1.8 milestone.

Issue Labels

sig/cli: Issue will be escalated to these SIGs if needed.
priority/important-soon: Escalate to the issue owners and SIG owner; move out of milestone after several unsuccessful escalation attempts.
kind/bug: Fixes a bug discovered during the current release.

Help

Additional instructions
Commands for setting labels

repo: kubernetes/kubectl


text:
Added short code around include statement

lgtm
/assign @Bradamant3

lgtm
/assign @Bradamant3

/lgtm
/approve

repo: kubernetes/website


text:
Update readme

Ugh, need to take out the other commit first.

Yeh,

I've sorted it anyway now.

Yeh but pr closed? :p I'm confused

Your PR contained extra changes so I just made the tiny fix myself to the readme.

K

authors: GrahamCampbell hannesvdvreken

repo: laravel/framework


text:
Added non logging constructor

Codecov Report

Merging #266 into master will increase coverage by 0.02%.
The diff coverage is 40.00%.

@@            Coverage Diff             @@
##           master     #266      +/-   ##
==========================================
+ Coverage   62.17%   62.20%   +0.02%
==========================================
  Files          38       38
  Lines        1285     1286       +1
  Branches        7        8       +1
==========================================
+ Hits          799      800       +1
  Misses        486      486

Impacted Files
Coverage Δ

cli/src/main/scala/laserdisc/cli/CLI.scala
0.00% <0.00%> (ø)

fs2/src/main/scala/laserdisc/fs2/RedisClient.scala
64.91% <66.66%> (+0.62%)
: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 2695e36...444aa11. Read the comment docs.

I love it

Yes!

authors: barambani barryoneill codecov-commenter

repo: laserdisc-io/laserdisc


text:
Make sure each running app on DCOS has a unique id

What specifically needs to be done here?

A running app in marathon has an id like:

lasp_client_$TIMESTAMP
lasp_server_$TIMESTAMP

This $TIMESTAMP comes from the DCOS Runner.
If we have multiple runners, we might have collisions.

Can't you just drop $MESOS_TASK_ID in there to make it unique?

I guess that will be different between the server and client, because each is a different task.  Can't you use the TASK_ID of the runner itself?

Yes. If we always use the DCOS Runner, that's okay.
If we try to start the tasks locally, that won't be available.
So, if, for example, transmission.sh detects that TASK_ID is not defined, it just creates something random?

In sprinter, the TASK_ID from the runner won't be available right?
So we need another environment variable on Lasp for this.

Can we close this or is this something that still needs to be done?  We seem to be getting on fine without it.

I'm closing it then.
I did this 3cb8e866a79b6cc4df87f5552ee536cbcc95edc4 to reduce the probability of a collision.

authors: cmeiklejohn vitorenesduarte

repo: lasp-lang/lasp


text:
Feature bug reporter

Moving this PR to track the version of feature-BugReporter on the leapmotion-examples repo and not @yuwilbur 's fork.

authors: RandomOutput yuwilbur


text:
fixed some spelling and word choices

Thanks.

authors: OMGasm lemire

repo: lemire/simdjson


text:
CI for Node

Starts the game with a random player

repo: leoEspin/github-actions-for-ci


text:
add codecov.yml file

not merging until I see a comment from codecov

perhaps it won't comment until the file is in master, so I'll merge this and open a new PR to test

repo: liatrio/github-azure-demo


text:
Updating README.md to include instructions on validating and releasing

@eloots The PDF has been updated.


text:
Make new tests

test_tilemap.py fails on my laptop, but works on @kcsaff's.
This reason is compounded:

Our installation methods may be different, as in I may only be using the pip install bitbucket verison, while @kcsaff's may be using the brew installed version, or vice versa. The underlying implications being SDL libraries installed on one machine that differ from the other.
pygame.init() is never called, which covers for these discrepancies, wheras, when pygame.init() isn't called, it falls back to more rudimentary SDL usage?

BREAKING:
I think this will work:

The other problem you will have once Pygame is working is that images will be corrupted due to a bug/incompatibility with the latest version of SDL_image. You will have to downgrade SDL_image to 1.2.10 as described here:

https://bitbucket.org/pygame/pygame/issues/284/max-osx-el-capitan-using-the-deprecated

Coverage remained the same at 91.083% when pulling 33ccdfab148f4326e8bd1e635d860e852a4ddbe4 on make-new-tests into 0944e469618e2182e870ca76620e870cb7008145 on master.

The pygame.image.save statements should be removed from test tilemap.
Is it safe to say we can remove new_tests and old_tests?

Coverage remained the same at 91.083% when pulling 1166e78a3f549a66519e28becb8da72a5579bdd8 on make-new-tests into 0944e469618e2182e870ca76620e870cb7008145 on master.

Exploring a possible issue with camera behavior: not centering as expected (see Slack).

retest this please

retest this please

Coverage decreased (-0.03%) to 91.054% when pulling 6c8ce0431420e9223120a6edbedec03415003ffa on make-new-tests into 0944e469618e2182e870ca76620e870cb7008145 on master.

repo: lillian-gardenia-seabreeze/sappho


text:
Altering mapper and segment creation to be compatible with PinotBuildAndPushJob

Codecov Report

Merging #1997 into master will increase coverage by 0.07%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1997      +/-   ##
==========================================
+ Coverage   69.71%   69.79%   +0.07%
==========================================
  Files         829      829
  Lines       40013    40013
  Branches     5383     5383
==========================================
+ Hits        27896    27927      +31
+ Misses      10262    10237      -25
+ Partials     1855     1849       -6

Impacted Files
Coverage Δ

...e/impl/dictionary/LongOnHeapMutableDictionary.java
88.88% <0%> (-6.67%)
:arrow_down:

...mpl/dictionary/DoubleOffHeapMutableDictionary.java
72.72% <0%> (-5.46%)
:arrow_down:

.../impl/dictionary/LongOffHeapMutableDictionary.java
87.27% <0%> (-3.64%)
:arrow_down:

...e/operator/dociditerators/MVScanDocIdIterator.java
64.17% <0%> (-2.99%)
:arrow_down:

...om/linkedin/pinot/transport/netty/NettyServer.java
81.05% <0%> (-2.11%)
:arrow_down:

...impl/dictionary/FloatOffHeapMutableDictionary.java
85.45% <0%> (-1.82%)
:arrow_down:

.../com/linkedin/pinot/common/data/TimeFieldSpec.java
97.36% <0%> (-1.32%)
:arrow_down:

.../pinot/core/common/datatable/DataTableBuilder.java
91.06% <0%> (-1.12%)
:arrow_down:

...data/manager/offline/AbstractTableDataManager.java
91.5% <0%> (-0.95%)
:arrow_down:

...com/linkedin/pinot/core/common/DataBlockCache.java
76.51% <0%> (+0.67%)
:arrow_up:

... and 18 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 6d2b603...9cb2a33. Read the comment docs.

authors: codecov-io jenniferdai

repo: linkedin/pinot


text:
Emitting events when updating schema

Codecov Report

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

@@            Coverage Diff            @@
##             master    #2073   +/-   ##
=========================================
  Coverage          ?   69.37%
=========================================
  Files             ?      830
  Lines             ?    39707
  Branches          ?     5381
=========================================
  Hits              ?    27547
  Misses            ?    10313
  Partials          ?     1847

Impacted Files
Coverage Δ

...i/resources/PinotSegmentUploadRestletResource.java
50.51% <ø> (ø)

.../com/linkedin/pinot/controller/ControllerConf.java
43.96% <0%> (ø)

...data/manager/offline/AbstractTableDataManager.java
91.58% <100%> (ø)

...ller/api/resources/PinotSchemaRestletResource.java
44.44% <100%> (ø)

...m/linkedin/pinot/controller/ControllerStarter.java
59.37% <100%> (ø)

...i/events/DefaultMetadataChangeNotifierFactory.java
100% <100%> (ø)

...ller/api/events/DefaultMetadataChangeNotifier.java
66.66% <66.66%> (ø)

...ller/api/events/MetadataChangeNotifierFactory.java
75% <75%> (ø)

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 90d2df9...0897a59. Read the comment docs.


text:
Adding init for segment upload

Codecov Report

Merging #3096 into master will decrease coverage by <.01%.
The diff coverage is 50%.

@@             Coverage Diff              @@
##             master    #3096      +/-   ##
============================================
- Coverage     70.24%   70.24%   -0.01%
  Complexity        4        4
============================================
  Files           936      936
  Lines         43520    43522       +2
  Branches       5901     5901
============================================
- Hits          30572    30571       -1
  Misses        10896    10896
- Partials       2052     2055       +3

Impacted Files
Coverage Δ
Complexity Δ

...va/com/linkedin/pinot/filesystem/AzurePinotFS.java
23.4% <0%> (-0.51%)
4 <0> (ø)

...i/resources/PinotSegmentUploadRestletResource.java
57.63% <100%> (+0.14%)
0 <0> (ø)
:arrow_down:

...lix/EmptyBrokerOnlineOfflineStateModelFactory.java
86.66% <0%> (-13.34%)
0% <0%> (ø)

...ore/query/scheduler/resources/ResourceManager.java
87.09% <0%> (-9.68%)
0% <0%> (ø)

.../impl/dictionary/LongOffHeapMutableDictionary.java
83.63% <0%> (-9.1%)
0% <0%> (ø)

...mpl/dictionary/DoubleOffHeapMutableDictionary.java
72.72% <0%> (-9.1%)
0% <0%> (ø)

...e/impl/dictionary/LongOnHeapMutableDictionary.java
82.22% <0%> (-6.67%)
0% <0%> (ø)

.../pinot/core/query/scheduler/PriorityScheduler.java
78.08% <0%> (-5.48%)
0% <0%> (ø)

...om/linkedin/pinot/transport/netty/NettyServer.java
80% <0%> (-4%)
0% <0%> (ø)

...linkedin/pinot/common/metrics/AbstractMetrics.java
77.03% <0%> (-1.49%)
0% <0%> (ø)

... and 14 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 b67e5e8...9a8e297. Read the comment docs.


text:
Fix crashes on firebase

Will be nice to get a log for the if{} else{}

Done!

Ok. I already approved.

authors: kcw-grunt mosadialiou

repo: litecoin-foundation/loafwallet-android


text:
Screenshot specs

👍


text:
preact support

Also, what is your Twitter&Telegram accounts to mention you in releae tweet?

Also, what is your Twitter&Telegram accounts to mention you in release tweet?

Twitter: @12Future12
Telegram: @Merciful

Is it OK if I will release it a few days later? You do not pan to use it soon, don't you?

To have full Preact support we need to port this fles too https://github.com/logux/client/tree/main/react

repo: logux/state


text:
Fix small typo

Thanks @GuiSim !

authors: GuiSim segalziv

repo: logzio/logzio-docker


text:
Documentation update

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

repo: lokesh-coder/toppy


text:
Add loklak API support for the plugin

duplicate!

repo: loklak/wp-juiz-last-tweet-widget


text:
add dhcp for archive lhn subnet + add comcam-arctl01 lhn interface

I may push some more commits into this PR or split it into two separate PRs.  I'm adding WIP to the title to indicate I've changed plans and this PR isn't yet ready to merge.

This PR is ready for review / merge as-is.  I had hoped to include some tcp/ip tuning as part of this PR but the current state of the LHN makes it impossible to do practical testing.

repo: lsst-it/lsst-itconf


text:
Ventoy: add version 1.0.08

Duplicate of #3961

Forgot searching PR🤣Why not merge it?

I merge PRs, approved by me after 7 days. It is enough time for other collaborators to see and potentially merge it instead of me.

Got it.

repo: lukesampson/scoop-extras


text:
Make lvgl tick more accurate.

Hi @embeddedt,
I am still using v6.1, so I am not sure this PR is suitable for latest version. I can rebase this to master branch if you think the latest code needs this PR.
(delta - one_ms) is the error we missed before, I add 1ms to lv_tick_inc() when the sum of error exceeds 1ms.

I am still using v6.1

Any plans to move to v7 (or v8 when it's ready)?
We don't really update the v6 branches any more with new features and fixes, so you are probably missing a few things.

Hi @amirgon,
We may move to v8 later, but no plan for v7.
I have rebase this PR to master #130 .

Closing in favor of #130

authors: amirgon rockindy


text:
Added Glyphfriend under Community Extensions

This is a fork. You should send the PR to https://github.com/Microsoft/extendvs

Sorry Mads, totally didn't realize it was a fork.

authors: Rionmonster madskristensen

repo: madskristensen/extendvs


text:
Updated AppVeyor script.

r? @dirvine
(maidsafe_highfive has picked a reviewer for you, use r? to override)

r? @rossmuir


text:
MAISTRA-1747: Update base tag for 1.1.7

merging manually

repo: maistra/istio-operator


text:
Interpret empty namespace list as []string{metav1.NamespaceAll}

I think this makes sense to default to metav1.NamespaceAll, but personally I'd rather keep SetNamespaces() as is, and instead change the defaults in NewSharedInformerFactoryWithOptions():
factory := &multiNamespaceInformerFactory{
    client:       client,
    scheme:       scheme.Scheme,
    resyncPeriod: resync,
    informers:    make(informersCache),
    customResync: make(map[schema.GroupVersionResource]time.Duration),
    namespaces:   sets.NewSet(metav1.NamespaceAll),
}

Does that work for you?

The new factories default to all namespaces as of #10.

The new factories default to all namespaces as of #10.

authors: bison dgn

repo: maistra/xns-informer


text:
Topleft, Topright, Bottomleft, Bottomright options

Cool, this is a great idea! I'll get it merged in.


text:
Background for project and job type settings is not white

Fixed.


text:
Add voice sending for GUI example

Closed?

open another.
19 июля 2016, 17:27:53, от "Martin Evans" < notifications@github.com >:
Closed? —
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub , or mute the thread .

authors: IGamemaker martindevans

repo: martindevans/MumbleSharp


text:
Add ability to delete a project

Depends on #1017

authors: Terry-Weymouth gtarcea


text:
Implement format

Coverage remained the same at 100.0% when pulling 42e66f1e2a712e13d0e03273aed01860e5053598 on vishaldeepak:implement-format into 6b12ba65b59b6068be5613ec66a01a693b0f9d8f on mathewdgardner:master.

authors: coveralls vishaldeepak

repo: mathewdgardner/momento


text:
Update config-settings.rst

Used this as a training test.

repo: mattermost/docs


text:
Add MtM MtConnect04S development board support

/morph test

I've accepted the Contributor Agreement on Sat 03 Jun 2017.

/morph test

@c1728p9 I've updated the model name and the info.py, I did not find the definition of PROJECT_RELEASE_INFO and SUPPORTED_CONFIGURATIONS. Our board uses lpc11u35 as programming adapter and the main MCU is nrf51822. Could you help to verify if my information is correct please.

/morph test

@c1728p9 when it will be merged to base?

Hi @shenmengkai, can you rebase this on master. It has merge conflicts.

@c1728p9 done.

/morph test

DAPLINK test looks like it failed : http://aus-raas-linux-00:8080/job/daplink_test_matrix/42/

@c1728p9 @studavekar what is the failure? The link is not reachable. how do I proceed?

Hey @c1728p9 looks like skipped mac specific test failed for k20dx_frdmk64f_if.. All Linux and Windows tests passed..
12:10:33 SubTest: test_fs_contents
12:10:33 File /Volumes/MICROBIT/MICROBIT.HTM valid
12:10:33 File /Volumes/MICROBIT/DETAILS.TXT valid
12:10:33 Skipping non file item /Volumes/MICROBIT/.fseventsd
12:10:33 SubTest: test_details_txt
12:10:33 SubTest: parse_kvp_file
12:10:33 SubTest: test_details_txt
12:10:33 SubTest: parse_kvp_file
12:10:33 <test_info.TestInfo object at 0x10f242850>
12:10:36 Data matches
12:10:36 SubTest: Load good file to restore state
12:10:36 Skipping USB test on this platform
12:10:36
12:10:36 Test: k20dx_frdmk64f_if FRDM-K64F: Failure
12:10:36   Test: daplink_test: Failure
12:10:36     Test: bin interface filetype test: Failure
12:10:36       Test: Extra Files: Failure
12:10:36         Failure: Data does not match

/morph test

Hi @c1728p9 @cederom @studavekar, I wonder if it is ready to merge? or anything needs me to improve?

@shenmengkai only thing pending is a green flag from CI, test on MAC were failing we will re-trigger.

/morph test

@studavekar @c1728p9 please update CI host to aus-raas-linux-00.austin.arm.com :-)

looks good to merge we have all green

Thank @c1728p9 @studavekar and @cederom, for all your help.

authors: c1728p9 cederom shenmengkai studavekar

repo: mbedmicro/DAPLink


text:
Website body missing

resolved


text:
Add logging to windows application event log anytime we ban an IP

Already has logging


text:
Add Jebathoatta Jeyageethanggal Vol 33 songs

Unnatharae Um
Nambathaka -
Uthari Thallu
Kutram Neenge -
En Maganae -
Nambi Vantha -
Thaayin Madiyil -
Sangaripaen -
Payamillai Payamillaiyae -
Kondaduvom -

repo: mcruncher/worshipsongs-db-dev


text:
linux

Coverage decreased (-0.004%) to 97.942% when pulling dec0bde8d7bca4127ac53e9fa235ddf4e5a8a83e on linux into 764471953cbf5a95115164a754ef10e5f3f7d846 on master.

authors: coveralls mdreizin


text:
Use latest nominatim repository and version

Not tested yet. Will reopen the request when I tested building.

repo: mediagis/nominatim-docker


text:
When data is too laggy, the sliding process will be very stuck.

Sorry, please allow me to advertise for my open source library! ~
I think this library react-native-reanimated-carousel will solve your problem. It is a high performance and very simple component, complete with React-Native reanimated 2

repo: meliorence/react-native-snap-carousel


text:
Broken Mesos details link in debug

duplicate https://github.com/mesosphere/marathon/issues/3447

repo: mesosphere/marathon


text:
chore(devDeps): update to protractor v5

Hey @Shyam-Chen thanks for the PR! The Travis build is failing. Do you think you can fix it?

I think it needs Java 8.

authors: Shyam-Chen mgechev

repo: mgechev/angular-seed


text:
implement lineSplit

@ivan-m From an no longer visible comment you made:

I think having \r\n might break null check you have there

I cannot see how this could happen. The lineSplit function should satisfy this law:
Ɐ n. concat ∘ lineSplit n ≅ id

If you can find an input ByteString Identity () that makes this not hold, then lineSplit has a mistake for sure.

What happens if you have something like:
1
2
3

4
5
6

With two blank lines in the middle.  Is an empty ByteString produced to represent it?

@ivan-m I've added some tests to verify the laws that lineSplit should satisfy. I don't totally understand your question

Is an empty ByteString produced to represent it?

But, here is what happens if your run this in GHCi with OverloadedStrings on:
>>> lineSplit 1 "1\n2\n3\n\n\n4\n5\n6\n"
Step Chunk "1\n" (Empty (Step Chunk "2\n" (Empty (Step Chunk "3\n" (Empty (Step Chunk "\n" (Empty (Step Chunk "\n" (Empty (Step Chunk "4\n" (Empty (Step Chunk "5\n" (Empty (Step Chunk "6\n" (Empty (Return ()))))))))))))))))
>>> lineSplit 2 "1\n2\n3\n\n\n4\n5\n6\n"
Step Chunk "1\n2\n" (Empty (Step Chunk "3\n\n" (Empty (Step Chunk "\n4\n" (Empty (Step Chunk "5\n6\n" (Empty (Return ()))))))))
>>> lineSplit 3 "1\n2\n3\n\n\n4\n5\n6\n"
Step Chunk "1\n2\n3\n" (Empty (Step Chunk "\n\n4\n" (Empty (Step Chunk "5\n6\n" (Empty (Return ()))))))

We never produce an empty ByteString. We make a ByteString that just has the newline in it (when the split size is 1). But this is this expected behavior, and it satisfies the law given earlier.

This has been merged into the new repo.

authors: andrewthad ivan-m

repo: michaelt/streaming-bytestring


text:
Quantum game notebook expansions

Also, it looks like there is some kind of validation error - if it persists after you move the code to the notebook, you should be able to reproduce it locally using https://github.com/microsoft/QuantumKatas/blob/master/scripts/validate-notebooks.ps1 script, then running Check.ipynb notebook it will generate and looking at which cell fails. The CI report doesn't have the actual validation error propagate to the log.

authors: rslstein tcNickolas

repo: microsoft/QuantumKatas


text:
fix stylecop build break

:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.AI.Luis.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.AI.QnA.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.ApplicationInsights.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.Azure.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.Dialogs.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.TemplateManager.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Configuration.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Connector.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Schema.dll compared against version 4.3.1

authors: fuselabs johnataylor

repo: microsoft/botbuilder-dotnet


text:
Adding more documentation

LGTM

repo: minio/objectstorage-java


text:
Support service settings without models

@bquinn do you think this covers the features you wanted?

@ushkarev yes I've tested it and it's great, that solves my problem better than I did! Thanks very much :-)

authors: bquinn ushkarev

repo: ministryofjustice/django-govuk-template


text:
add start/end time to teardowns

fixed in https://github.com/mlab-lattice/lattice/pull/223

repo: mlab-lattice/lattice


text:
Improve blurring performance

WIP

repo: mob-sakai/UIEffect


text:
Sound files streaming

Sound streaming is implemented, However the file being streamed is never closed.

repo: mofr/Diablerie


text:
blog: v0.17.1.5 gui release notes

Hashes ok.
Title needs fixing, it says 0.17.1.4

Updated title

Updated title

Thanks! Hashes still ok :)

authors: binaryFate selsta

repo: monero-project/monero-site


text:
Add Ruby syntax highlighting to "Multiple Trackers" example in README

Coverage remained the same at 99.417% when pulling dd8ff9a9f5a4f5ee806d2373d7985b023b554356 on hoffm:patch-1 into 6616cea5da9aff80cbae34f1060f328318ed07ac on mongoid:master.

Coverage remained the same at 99.417% when pulling dd8ff9a9f5a4f5ee806d2373d7985b023b554356 on hoffm:patch-1 into 6616cea5da9aff80cbae34f1060f328318ed07ac on mongoid:master.

👍merged
If you're looking for ways to contribute, #238 is probably doable, starting with travis CI for mongoid 7.

@dblock Thanks! I haven't used this library yet—came upon it doing some research—but if I do I'll check back for ways to contribute :)

authors: coveralls dblock hoffm

repo: mongoid/mongoid-history


text:
Implement Email sending solution using SMTP

For now, we can use Travis. It will send the emails when the tests are complete

authors: deepesh-kn puneet-khushwani-eth

repo: mosaicdao/facilitator


text:
fix empty namespace

closing because of this https://github.com/moszeed/easysoap/issues/19
Thanks you for your work!

authors: IAlexandr moszeed

repo: moszeed/easysoap


text:
Added height to the featured status

@dinnu93: Can you elaborate on what this fixes?

In the home page https://platatus.herokuapp.com/ the icons on the right seem to be chopped off in the bottom part, so gave a little bit of height to extend it to make it look good.

Thank you! This is indeed reoccurring feedback that I am trying to address in #180 . The commits should come in today with the new status design.
I am closing this for now but we can revisit when the new layout it up.

Thanks for the quick reply I'll search for other issues and contribute.

repo: mozilla/platatus


text:
Correct Shared Memory spec link

Updated to use https link.


text:
xiiau easter egg

fixed with commit f7b3fd57d10a88da54fc4bcc5a377659e354410b

repo: muddasheep/hqtrilogy


text:
Fix for undefined constant assumed by PHP, giving error in newer vers…

I see. WP_REPOSIDGET_HOMEPAGE is not defined, but fortunately it would be interpreted as 'WP_REPOSIDGET_HOMEPAGE' in PHP 5.x. I guess now the rules becomes strict and won't allow it anymore.

I see. WP_REPOSIDGET_HOMEPAGE is not defined, but fortunately it would be interpreted as 'WP_REPOSIDGET_HOMEPAGE' in PHP 5.x. I guess now the rules becomes strict and won't allow it anymore.

I see. WP_REPOSIDGET_HOMEPAGE is not defined, but fortunately it would be interpreted as 'WP_REPOSIDGET_HOMEPAGE' in PHP 5.x. I guess now the rules becomes strict and won't allow it anymore.

Exactly. Thank you for the merge!

I see. WP_REPOSIDGET_HOMEPAGE is not defined, but fortunately it would be interpreted as 'WP_REPOSIDGET_HOMEPAGE' in PHP 5.x. I guess now the rules becomes strict and won't allow it anymore.

Exactly. Thank you for the merge!


text:
Typo / broken link

Thanks @dfahlander for this. Not sure how I've missed it :)


text:
Blender2.8への移行

2018/12/30 Blender2.8βへの対応着手

プロジェクトを作成したので，本Issueは閉じる


text:
Bump version and release notes v4.7.3

I'm starting up a node with latest master build.

authors: aguycalled mxaddict

repo: navcoin/navcoin-core


text:
[#7036] Supports function to get the information of the active agent

Codecov Report

Merging #7037 into master will increase coverage by 0.01%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master    #7037      +/-   ##
==========================================
+ Coverage   38.92%   38.93%   +0.01%
==========================================
  Files        3239     3239
  Lines       94478    94504      +26
  Branches    11852    11856       +4
==========================================
+ Hits        36777    36799      +22
- Misses      54747    54752       +5
+ Partials     2954     2953       -1

Impacted Files
Coverage Δ

...pinpoint/web/controller/ApplicationController.java
0.00% <0.00%> (ø)

...orp/pinpoint/web/service/AgentInfoServiceImpl.java
0.00% <0.00%> (ø)

...pc/server/LoggingServerMessageListenerFactory.java
90.00% <0.00%> (-10.00%)
:arrow_down:

...cluster/zookeeper/ZookeeperClusterDataManager.java
54.83% <0.00%> (+1.93%)
:arrow_up:

...m/navercorp/pinpoint/rpc/client/ConnectFuture.java
85.00% <0.00%> (+5.00%)
:arrow_up:

.../pinpoint/rpc/client/PinpointClientHandshaker.java
66.42% <0.00%> (+6.42%)
:arrow_up:

...rp/pinpoint/rpc/control/ControlMessageDecoder.java
78.57% <0.00%> (+14.28%)
:arrow_up:

...p/pinpoint/rpc/client/WriteFailFutureListener.java
75.00% <0.00%> (+18.75%)
: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 8d94db4...903d650. Read the comment docs.

authors: codecov-commenter koo-taejin

repo: naver/pinpoint


text:
clang/gcc maker should search for compile_commands.json

I've read through the clang manuals to try to implement this myself, but it seems that there is no command line flag that enables usage of the compile_commands.json file. If that is the case, then one would need to write a wrapper script that is included in Neomake and uses the libclang API to load the compile_commands.json from there.

Or alternatively, Neomake can parse the compile_commands.json file (using libclang, for example), and append the flags onto compiler commandline.

I just found that apparently there is a tool called 'clang-check' that can be used to check the syntax and automatically finds the 'compile_commands.json'.

@milch Maybe we can use 'clang-check' by default?

If clang-check is installed it should work. The only issue I found is that clang-check seems to have issues when modules are enabled (at least on the code base I was testing it on). It works, however, if I invoke it using clang-check -extra-arg "-fno-modules" ./src/my_source_file.cpp.

Also, I think the only change required to make this work would be to add
    if executable('clang-check')
        let maker.exe = 'clang-check'
        let maker.args = ['-extra-arg', '-fno-modules']
    endif

at the end of the neomake#makers#ft#cpp#clang() function. For me this picks up the compile_commands.json correctly.

authors: milch yshui

repo: neomake/neomake


text:
Supports Min/Max Range of ROW IDs per index.

Wait up on this.  There is a boundary condition that is failing.

Fixed the issues.

repo: ngageoint/geowave


text:
change iojs-ko to nodejs-ko

:+1: 입니다.
다만, 저장소명 변경을 먼저하시고 머지 하셔야 합니다.
저번에 iojs에서 nodejs로 변경했던 경험상 따로 디플로이는 안해주더라구요.

네네 미리 준비만 해 놓은 겁니다. 이름 바꾸면 머지하려구요.
그리고 전에 경험상 이름 바꾸면 gitter 채팅방도 튕겼던 것 같아서 이부분도 공지하고 바꿔야 할 것 같습니다.

:ok_woman:

일단 다 봤습니다. Gitter 주소는 저장소 주소를 바꾸면 알아서 바뀌는 건가요?

@yous 오래돼서 헷갈리는데 gitter가 저장소 기반이라 이름 바꾸면 전에 방이 폭파되고 새로운 방이 생겼던것 같습니다.

참고삼아 적어놓습니다. 이 PR은 저장소 변경용 PR이므로 준비가 끝날때까지는 merge되면 안됩니다.

authors: marocchino outsideris yous

repo: nodejs/iojs-ko


text:
Greenkeeper/coveralls 3.0.6

Pull Request Test Coverage Report for Build 888

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

Totals

Change from base Build 234:
0.0%

Covered Lines:
628

Relevant Lines:
653

💛  - Coveralls

authors: coveralls lholmquist

repo: nodeshift/nodeshift


text:
Materialize css redesign

There is still a bug. When picking a date it doesn't set it correctly and it ends up being null or previous value.

authors: jaolejnik mKomorek


text:
Add serverbot.conf validation of parameters

Added in https://github.com/nozel-org/freebsd-serverbot/commit/e839a0a4582919ea0a8547618a4097426083b911.

repo: nozel-org/freebsd-serverbot


text:
There is no package_data, and the package is zip_safe.

Thanks again. :fire:

Coverage remained the same at 67.47% when pulling ba926eadff06265e5d68eb6e2929c17a5f7be2d6 on graingert:patch-1 into 8b716f54c7085f0042d0b1aaa06a7f3ae55e3969 on numberly:master.

Coverage remained the same at 67.47% when pulling ba926eadff06265e5d68eb6e2929c17a5f7be2d6 on graingert:patch-1 into 8b716f54c7085f0042d0b1aaa06a7f3ae55e3969 on numberly:master.

authors: coveralls graingert ramnes

repo: numberly/lazify


text:
feat(snippets) added new snippet - inner form group as a form control

Hey Liron, thanks a lot for the PR!
I made a quick pass, moved things around, made the demo worked and reworded it a bit, let me know if this makes sense.
Also would be curious what other folks think.

@NothingEverHappens seems ok, thanks!

Made another pass, I'm waiting on the form to start supporting editing to make sure the demo works, will merge after.

authors: LironHazan NothingEverHappens


text:
feat(web): editor-focused config changes

This is pretty neat, had no idea you could do this inside of Visual Studio Code 👍🏽

@heybereket i was going to add more editor things and you merged. i spent 10 mins trying to see why i couldn't push.....
scammmm

repo: oasis-sh/oasis


text:
Network.Wai.Middleware.Prometheus module is missing from the repo

Added!

authors: ocharles qrilka

repo: ocharles/prometheus-effect


text:
Change URL for status on PRs

Thank you!

repo: ocurrent/current-bench


text:
Updates to helm charts to better support local & cloud

A couple of minor changes

<! xml marker in yaml comment
external IGC configurable port
IGC auth base64 string moved out of configmap into values
default to no debug (egeria policy -- I override it locally)
a few lint issues noticed whilst debugging (Note: there are more.....)
default IGC to disabled (most users won't have access to the image)
comment on how to rebuild docker images

Introduced new _kafkaauth.tpl. This is prefixed _ so that helm does not interpret as a k8s resource. Instead it provides a few common templates for external kafka. At this time these are supporting IBM IKS, but a conditional could be added here to support other environments
Changed external kafka values to improve structuring of external config. I did consider adding individual parms, or a custom map to allow for any extra kafka parms, but this makes configuration for known services more complex. Skipped for now, but in future we may want provider: custom & allow for any custom values
modified jobs/deployments to include the k8secret (from template above), and make use of the new kafka fragment when configuring eventbus. To reduce risk and improve readability the old parms were left in-situ - new code used for external kafka only
*Fixed up a few template parsing issues (based on parms)
Removed basic kafka monitor

Note that the secret is assumed to contain two values
username
password
to create this you can use
echo -n 'token' > username
echo -n '<APIKEY>' > password
kubectl create secret generic auth-vdc-kafka --from-file=./username --from-file=./password

For IBM Event Streams you can use user/password (cloud) or token/key (cloud and on-prem) - other providers are expected to work similarly.

repo: odpi/egeria


text:
Bug 1782793: Update library-go to fix installer state controller

/lgtm

authors: mfojtik sttts


text:
staticpod: switch controllers to use context instead of stopCh

/lgtm
/approve
Happy bumping!

repo: openshift/library-go


text:
initial import of remote healing tool

@joelsmith : Script has changed a bit since last time we spoke about this. General flow is still going to be a bunch of 'if(trigger): action' statements. Go ahead and give this a security review.

The key that will be used to run this script must be configured with a command="/path/to/the/script" option. With that force command in place, arguments will not be passed to the script as parameters. Instead, no matter what command (+ args) the user tries to run with the key, sshd will run the script (with no args). Because of this, the script should not parse arguments (that aren't there) but should instead look at and safely parse the SSH_ORIGINAL_COMMAND environment variable.
See https://github.com/openshift/puppet-openshift-online/blob/master/modules/oo_openshift/files/node/utils/verify_rhc-move_rsync_command for an example of such a script.

Sorry, there's another option, and perhaps it's what you had in mind. You can keep the script as is, but have a validation script wrapper that runs as the force command. It could do the validation and parsing of SSH_ORIGINAL_COMMAND and then invoke this script as written. That has the advantages of making this script easy to use for a human. It has the disadvantage of having to keep another script in sync.
If you want to keep this script easy to use for humans and have it do the SSH_ORIGINAL_COMMAND validation, perhaps the force command could be "path/to/script --ssh-validate", and if the script sees that the ssh-validate option is present, then it parses SSH_ORIGINAL_COMMAND safely and sets the arg options as if they had been passed in on the command line.

authors: joelddiaz joelsmith

repo: openshift/openshift-tools


text:
util/opensslwrap.sh: adjust to define OPENSSL_MODULES as well

@mattcaswell, didn't you ask me about a script like this a while ago?  It was there all along!

Merged.
d5fdb6a695b457cf1eea8d7b638ffde029231080 util/opensslwrap.sh: adjust to define OPENSSL_MODULES as well

repo: openssl/openssl


text:
[thread-cert] refactor case 7.1.2 and 7.1.4 using pktverify

@FiveDimensions , please rebase.

authors: FiveDimensions jwhui

repo: openthread/openthread


text:
docs(spelling) - fixed a spelling mistake in go.mod file

@Piyushhbhutoria thank you for the submission. I will request you to sign our CLA mentioned in https://github.com/optimizely/go-sdk/blob/master/CONTRIBUTING.md.

Thank you for pointing this out. Can you help me re-run the pipeline?

@Piyushhbhutoria we will re-run the pipeline and merge the change. Thanks again for contributing.

authors: Piyushhbhutoria aliabbasrizvi

repo: optimizely/go-sdk


text:
A help message was inverted for the cors plugin

Cheers!

authors: marckhouzam pantsel

repo: pantsel/konga


text:
Add osx to travis config

Waiting for this issue on libui repo

Test are now passing on Linux & OSX  :fireworks:
Waiting to manually test it on OSX & that merge

repo: parro-it/libui-node


text:
Scala 2.13 compatible

Is this project still actively being worked on?

Working on a fork here: https://github.com/razem-io/scala-influxdb-client
Support for 2.13.1 is already working.
Currently trying to publish the artifacts to sonatype. I will report here as soon as the artifacts can be used.

Artifacts are now published with support for 2.13. Further instructions: https://github.com/razem-io/scala-influxdb-client

authors: mdulac-photobox razem-io zebaslam

repo: paulgoldbaum/scala-influxdb-client


text:
[BACKLOG-31069] Propagate embedded meta store named connections

:white_check_mark: Build finished in 19m 33s
Build command:
mvn clean verify -B -e -Daudit -amd -pl \
core,engine,plugins/connections/ui,plugins/file-open-save-new/core,plugins/pur/core

:ok_hand: All tests passed!
Tests run: 4690, Failures: 0, Skipped: 7    Test Results

:information_source:  This is an automatic message

:white_check_mark: Build finished in 19m 58s
Build command:
mvn clean verify -B -e -Daudit -amd -pl \
core,engine,plugins/connections/ui,plugins/file-open-save-new/core,plugins/pur/core

:ok_hand: All tests passed!
Tests run: 4690, Failures: 0, Skipped: 7    Test Results

:information_source:  This is an automatic message

authors: bmorrise buildguy

repo: pentaho/pentaho-kettle


text:
Add readme entry on how to build or supply gradle configuration

Added a short description on how to build: https://github.com/peq/wurst4vscode#developer-information

authors: Frotty peq


text:
Remove dupe consecutive words

Nice catch catch!

authors: antquinonez samcv


text:
Video card has broken on HERB2

This seems to be fixed.


text:
For each

All of the challenges pass!

authors: Alwynblake peterjast


text:
ENH: Add Instrument.cuda() (close #145)

Codecov Report

Merging #151 (c093266) into develop (8471dd2) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##           develop      #151   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           40        40
  Lines         1080      1082    +2
=========================================
+ Hits          1080      1082    +2

Impacted Files
Coverage Δ

pfhedge/instruments/base.py
100.00% <100.00%> (ø)

pfhedge/stochastic/heston.py
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 8471dd2...c093266. Read the comment docs.

authors: codecov-commenter simaki

repo: pfnet-research/pfhedge


text:
dhcpd ipv4 - fix gui config to accept multiple subnets and/or answer requests from relay servers

Can it be reviewed until merge conflicts issues. It's the third time I rewrite this.

Leave it as is for now, I'm going to review it and let you know. After it's reviewed you rebase your fork and we get it merged.

Hi, we tryed your files today.
But the dhcpd.conf which was created did not load.
All networks are as pools under one subnet, in the old version you crate a subnet per pool.
Maybe you can take a look.
Regard niko

rbgarga, how about to bring this pull request into the master ?

Any progress on this issue?

Hey, how does it look? Don't tell me, the finished work will go to waste

After discuss privately with @marcelloc he agreed to submit a fresh patch, tested on 2.3, and I'll review it as soon as it lands here

Was this feature ever merged?

We're in a position where we are trying to configure DHCP as described above, and we were slightly surprised that it is not presently possible! Can someone post a link to the fresh PR where we can track the status?

Found myself needing this yesterday, I assume it is still not supported? Found a dirty workaround in the meantime (I haven't tested it yet) https://happy-coder.com/2014/06/27/pfsense-custom-dhcpd-configuration/

Any progress on this feature?

I need this too. Thank you for the happy coder workaround, but it's going to be very careful upgrading :-(

Need this too. Would be greate to see this feature in the near future.

+1 for using this

+1 for this basic functionality...

+1 for this basic functionality...

+1 also

+1 what happened to this request?

+1  official merge would be very much helpful it can really simplify messy setup example in my case, I have more than 25 vlans, creating 25+ vlan interfaces on pfsense and enabling DHCP on each interface is really tedious job practically, what if there are many more vlans.
From last few years I have been waiting for this feature as everything is relayed through  switch as a DHCP relay to single DHCP server in my case.
The day it gets merged I will migrate from ISC DHCP to pfsense.
Any updates on feature implementation ?

authors: Akshay-Hegde andrewkrug jerrylindahl jzapater koenr marcelloc nikor30 nveeser pedymaster rbgarga ross-w ssiemens-hm toboshii tofazzz vc zbzoch

repo: pfsense/pfsense


text:
Add GTS-21 as partial agonist

BEL Document Example for GTS-21 - WORK IN PROGRESS
`
SET DOCUMENT Copyright   = "Copyright © 2018 Fraunhofer Institute SCAI, All rights reserved."
SET DOCUMENT Licenses    = "Private Ownership to Fraunhofer Institute SCAI"
SET DOCUMENT ContactInfo = "simon.mueller@scai.fraunhofer.de"
SET DOCUMENT Name        = "GTS-21 as partial agonist at neural nicotinic acetylcholine receptors"
SET DOCUMENT Version     = "1.0.0"
SET DOCUMENT Authors     = "Simon Müller"
SET DOCUMENT Description = "Agonist activity at human recombinant alpha7 nAChR expressed in rat GH3 cells by calcium influx assay"
Namespaces and Annotations left out for easier readability, can be looked up in template
SET AssayType = "F"
SET MeasurementType = "EC50"
SET MeasurementUnits = "nM"
SET MeasurementRelation = "="
SET MeasurementValue = "1995.26"
SET TargetType = "single protein"
SET Species = "9606"
SET TargetOrganism = "?"
a(CHEMBL:CHEMBL134713) =>
`

authors: AgenoDrei cthoyt

repo: pharmacome/curation


text:
add upsert & deduplicate test

/run-all-tests

repo: pingcap/tispark


text:
chore: Release v0.2.0

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

repo: pinterest/graphql-lint-rules


text:
Add default_noreply property to PooledClient

Thanks!

authors: cgordon tjwudi

repo: pinterest/pymemcache


text:
Update field descriptions for rabbitmq-server.ssl.*

Thank you for the contribution, @michaelklishin

authors: michaelklishin svett

repo: pivotal-cf/cf-rabbitmq-release


text:
As podcaster I want to create and edit transcripts manually

Workflow:

Upload file to server (.vtt)
-> Parse file on server
<- Response from server
Display Transcripts with Chapters, Speaker and Transcript
Allow User to fix typos
Change the name of a speaker

Similar to #46 the user uploads a .vttfile and see's a progressbar.
Design idea 1:

Design idea 2:

Fix typos or whatever in the transcripts content

Change the speakers name

I like the design version 2 a bit more, because it is more compact.
Questions (cc  @eteubert, @monkeydom):

Do we only need to change the name of a speaker (which changes every name in the transcript of this speaker) or do we also need do change the name of a single sentence?
Do we want to connect the speakers to a contributor, so we could for example show their avatars or a small preview with more infos to the speaker while hovering over a speaker's name?
Do we really have information/data about the chapters in the .vtt context?

Renaming is not a desired feature, only assigning of already existing contributors (and a add-new sub-workflow in case the contributor does not exist yet).
Assigning only makes sense for the whole transcript, not single sentences.
Chapter information is not part of the .vtt but can be inferred from the timestamps.

Renaming is not a desired feature, only assigning of already existing contributors (and a add-new sub-workflow in case the contributor does not exist yet).

Ok, i will change rename to assign and from input to drop down menu.

Refactor Store:

transcript needs to be part of an audio instead of an episode

authors: Fischaela Letty eteubert

repo: podlove/radiator-cms


text:
update link to travis in readme badge

Please note that the travis unit tests are failing due to https://github.com/postmanlabs/httpbin/issues/554 please accept the PR https://github.com/postmanlabs/httpbin/pull/555 from @simonkowallik to resolve the issue.

repo: postmanlabs/httpbin

license: isc


text:
Cost calculator per hour/day/month for custom configuration

Can you provide more detail here? What do you mean by "custom configuration"? AWS already has a really powerful calculator; http://calculator.s3.amazonaws.com

authors: naterex powdahound


text:
Support PostgreSQL,Refactor jdbc synchronizer

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 8

Complexity increasing per file
==============================
- sequence-core/src/test/java/com/power4j/kit/seq/persistent/provider/PostgreSqlSynchronizerTest.java  2
- sequence-core/src/main/java/com/power4j/kit/seq/persistent/provider/PostgreSqlSynchronizer.java  3
- sequence-core/src/main/java/com/power4j/kit/seq/persistent/provider/AbstractSqlStatementProvider.java  1
- sequence-core/src/test/java/com/power4j/kit/seq/persistent/PostgreSqlSeqHolderTest.java  2

Complexity decreasing per file
==============================
+ sequence-core/src/test/java/com/power4j/kit/seq/persistent/MySqlSeqHolderTest.java  -1
+ sequence-core/src/main/java/com/power4j/kit/seq/persistent/provider/MySqlSynchronizer.java  -1

See the complete overview on Codacy


text:
Update to Swift 2.0

@beltex Hi beltex, I totally agree and I'm seeking volunteers to help be bring Pythonic.swift from 1.2 to current Swift :-)

I'll take a stab at it!

Should I just remove the custom Set class?

The project is now compiling under Swift 2.1. Clean-ups and improvements are welcome! :-)

authors: beltex harlanhaskins practicalswift


text:
Add initial metric

@imsickofmaps Please review.

Minor docs issues then 👍

authors: gsvr imsickofmaps

repo: praekelt/seed-message-sender


text:
Consider removing $disconnect() from examples as its only required in very specific cases

(Or at least clarify)

repo: prisma/docs


text:
Fix typo in docs

 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.

The current version is the correct markdown syntax. The link does not work because it's inside <InfoBox>

authors: CLAassistant jpJuni0r marktani

repo: prisma/prisma


text:
fix: generator engine versioning

I am still having the issue with the 2.9.0 version.
TypeError: Cannot read property 'type' of undefined

Happening in 2.19.0 either. 2.18.0 and 2.20.0 is fine.

authors: geosigno hyochan timsuchanek


text:
Patch 1

Let me know if you're ready for the PR to be merged or if you're still doing review/refinement on the text. Thanks for writing and submitting this guide!

Thanks Tim for your excellent suggestions.  I will execute them ASAP, probably later today.
Tom Bartlett
2otherjobs

On Aug 3, 2019, at 12:42 PM, Tim Moore notifications@github.com wrote:
Let me know if you're ready for the PR to be merged or if you're still doing review/refinement on the text. Thanks for writing and submitting this guide!
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub https://github.com/project-trident/guides/pull/3?email_source=notifications&email_token=AC7KBT5LWWLBRX75523JZGLQCW7PZA5CNFSM4IJDAFW2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD3PSYOA#issuecomment-517942328, or mute the thread https://github.com/notifications/unsubscribe-auth/AC7KBTYSB4R4FL76INN4PQLQCW7PZANCNFSM4IJDAFWQ.

I just finished the next re-edit.  Check it out at your convenience.
Tom Bartlett
2otherjobs

On Aug 3, 2019, at 12:42 PM, Tim Moore notifications@github.com wrote:
Let me know if you're ready for the PR to be merged or if you're still doing review/refinement on the text. Thanks for writing and submitting this guide!
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub https://github.com/project-trident/guides/pull/3?email_source=notifications&email_token=AC7KBT5LWWLBRX75523JZGLQCW7PZA5CNFSM4IJDAFW2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD3PSYOA#issuecomment-517942328, or mute the thread https://github.com/notifications/unsubscribe-auth/AC7KBTYSB4R4FL76INN4PQLQCW7PZANCNFSM4IJDAFWQ.

repo: project-trident/guides


text:
conf: return value in get_pdc_secret

Good catch, thanks!

authors: TomasTomecek bkabrda

repo: projectatomic/osbs-client


text:
Remove dev-focussed docs from 1.5 release and update datamodel.

Not sure I understand the reasoning from removing developer focused docs entirely, especially from an earlier release.
The tier stuff should go, but otherwise changes to previous releases should just be bugfix-y.

Closing this issue.   Unfortunately I didn't notice this PR and instead raised my own (which has since been merged).  We may need to re-instate the etcd doc if we find that necessary.

authors: caseydavenport fasaxc robbrockbank

repo: projectcalico/calico


text:
Document EC2 service discovery

Fixed by https://github.com/prometheus/docs/pull/230

authors: fabxc juliusv

repo: prometheus/docs


text:
How to use Custom Icon?

Thanks @lyseiha for raising the query.
Please refer below README reference for the same:

Usage
Icons

Please let me know incase you face any issues.
Thanks
</ Pranav >

authors: lyseiha prscX

repo: prscX/react-native-notification-banner


text:
hehe finished?

noice

authors: morphhyy punkholic


text:
cmake3 and cmake are required by showoff

Whoever reviews this, can you please "squash and merge"?

repo: puppetlabs/pltraining-bootstrap


text:
Merge 5.2.x into master

@puppetlabs-jenkins retest this please

CLA signed by all contributors.

authors: puppetcla rbrw

repo: puppetlabs/puppetdb


text:
Change to process *.scss instead of only project.scss.

@omin, thanks! I should ask, have you tested this?

Yep. Well, it depends on what the author was trying to achieve. If he/she was trying to concatenate to a single file, we need another solution.
Sames goes for processing javascript files with gulp btw.

authors: omin webyneter

repo: pydanny/cookiecutter-django


text:
Fix coveralls token

@pariterre This PR is ready to be merged :+1:

repo: pyomeca/pyomeca


text:
Remove copied Ramp block

The Ramp block is not the same but an adapted version of the one from the Modelica Standard Library, in which the startTime is not a parameter but a varying input. This is done in order to set it to a variable and not a constant value (for example as done in the model SC_opening_closing).

authors: beutlich queraltab

repo: queraltab/Greenhouses-Library


text:
Legacy ingress

Awesome :clap:

authors: pts-kevinqiu rajiteh

repo: rajiteh/external-dns


text:
Add tests for multiclusterapp actions

LGTM

authors: dramich mrajashree

repo: rancher/rancher


text:
Login

completed with login Auth


text:
remove setuptools pin upstream stated fix in 19.6.2

build timeout?
21:41:54 Build timed out (after 20 minutes). Marking the build as aborted.

lesigh, another build timeout
21:41:54 Build timed out (after 20 minutes). Marking the build as aborted.

and the other build timed out too
21:58:00 Build timed out (after 20 minutes). Marking the build as aborted.

recheck

(it did get to maas though, so I think we are fine)
21:38:00 + run_ansible verify-maas.yml

:+1:  fix released in setuptools upstream. This was only a temporary fix, so happy to remove it now.

recheck

:+1:

authors: d34dh0r53 mancdaz prometheanfire

repo: rcbops/rpc-openstack


text:
[Request] add support for js image assets

can you provide some more info?


text:
Added "When ReactPHP meet Symfony" article

ping!

repo: reactphp/react


text:
Certain (older) PHP versions ignore SSL option verify_peer=true

This issue is covered by our test suite. In other words, if you can run the test suite without any errors, then your installation is not affected.
This test DOES work correctly on ALL PHP versions tested via Travis (and everything I've tested locally on Ubuntu Linux).
This test does NOT work on PHP 5.5.30 on Mac, but DOES work with PHP 7.? on Mac.
As such, I'm not sure there's much to be done here. Also, PHP 5.5 is no longer officially supported by Zend, so you're recommended to upgrade anyway.
As such, I'm closing this for now and we can reconsider this if it affects other (supported) versions :+1:

repo: reactphp/socket-client


text:
Cqlmapper support

:haircut_man:

repo: reddit/baseplate


text:
Create GitHub build.yml workflow

@bowlofeggs Sure does!

@bowlofeggs Heh, that makes two of us! This is actually my very first dive into GitHub Actions.

repo: repoSpanner/repoSpanner


text:
Write publishing in ReScience C page

Not Applicable anymore. Note included in resources


text:
Replace wmic with custom vbscript in Windows

Can you rename the shell_scripts directory to just scripts?

authors: jviotti mkorakakis

repo: resin-io/drivelist


text:
Update resin-supervisor to v1.12.2

The update is merged.

authors: agherzan pcarranzav

repo: resin-os/meta-resin


text:
Remove efivar and efibootmgr redundant recipes

retest

authors: agherzan floion

repo: resin-os/resin-intel


text:
[RESTEASY-2294] Update to MP REST Client 1.3.3

@asoldano once this is merged, I think we're good for a 4.x release from Quarkus side

repo: resteasy/Resteasy


text:
solution and test 5

تست مسئله 5 را ننوشته بودم
.خیلی ممنونم، لطف کردید

خواهش میکنم

authors: FSZolfaghari reza-arjmandi

repo: reza-arjmandi/rpi-course


text:
Fixed cyclic import with Django 1.9

Thanks!

authors: Entropius rfk

repo: rfk/django-supervisor


text:
add awesome to example

:star: PR built and available in a preview environment ripcitysoftware-spring-boot-http-gradle-pr-1 here

:star: PR built and available in a preview environment ripcitysoftware-spring-boot-http-gradle-pr-1 here

repo: ripcitysoftware/spring-boot-http-gradle


text:
Add gcpa alias for git commit patch

Same as gapa; gc.

authors: mcornella xavhan

repo: robbyrussell/oh-my-zsh


text:
issue160: refactor Gson usage

Any comments?

repo: ronmamo/reflections

license: WTFPL


text:
Support console_scripts in entry_points

This will be supported in #798

repo: ros/catkin


text:
OpenX currency fix

These older PRs address the same issue but they've fallen to the second page. Merging any of them would clear out three PRs from the backlog :)
https://github.com/rubicon-project/prebid-server-java/pull/663
https://github.com/rubicon-project/prebid-server-java/pull/648

Coverred by #648
Closing.

authors: apaliy laurb9 rpanchyk

repo: rubicon-project/prebid-server-java


text:
Boost --ignore-site-config

Testing:

https://ci.appveyor.com/project/ingenue/hunter/build/1.0.2149
https://travis-ci.org/ingenue/hunter/builds/264225868?utm_source=github_status&utm_medium=notification

Uploading:

https://ci.appveyor.com/project/ingenue/hunter/build/1.0.2154
https://travis-ci.org/ingenue/hunter/builds/264330633?utm_source=github_status&utm_medium=notification

Released https://github.com/ruslo/hunter/releases/tag/v0.19.69

repo: ruslo/hunter


text:
weaken requirements of AstDef

bors r+
🤖

What are the advantages of this? Does this somehow improve performance?

In general it’s always a good idea to use the smallest possible interface. More specifically, this commit is the first step towards teasing ra_hir apart into several crates.

repo: rust-analyzer/rust-analyzer


text:
Fixed link to the list of supported platforms

@bors r+

:pushpin: Commit bc16b7d has been approved by brson

Thank you!

:hourglass: Testing commit bc16b7d with merge d712458...

:sunny: Test successful - status-appveyor, status-travis
Approved by: brson
Pushing d712458eef20eaa7e62ae0c7d848f0d362f6ec12 to master...

authors: bors brson mp4096


text:
Do not grow the container if an insertion is on a tombstone.

Can you provide benchmark results for this change?

Changes are in the noise. Unsurprisingly since the benchmarks don't test insert/erase workloads. The insert/erase inserts a bunch and removes a bunch so the difference in the code doesn't matter. It does verify it does not make them slower.
Before:
test insert_erase_i32 ... bench:      24,370 ns/iter (+/- 4,587)
test insert_erase_i64 ... bench:      24,403 ns/iter (+/- 1,900)
test insert_i32       ... bench:      19,143 ns/iter (+/- 2,207)
test insert_i64       ... bench:      20,110 ns/iter (+/- 1,531)
test iter_i32         ... bench:       1,643 ns/iter (+/- 304)
test iter_i64         ... bench:       1,640 ns/iter (+/- 54)
test lookup_fail_i32  ... bench:       1,759 ns/iter (+/- 562)
test lookup_fail_i64  ... bench:       1,912 ns/iter (+/- 57)
test lookup_i32       ... bench:       2,447 ns/iter (+/- 174)
test lookup_i64       ... bench:       2,358 ns/iter (+/- 74)

After
test insert_erase_i32 ... bench:      24,310 ns/iter (+/- 1,222)
test insert_erase_i64 ... bench:      25,604 ns/iter (+/- 15,702)
test insert_i32       ... bench:      19,896 ns/iter (+/- 13,620)
test insert_i64       ... bench:      19,917 ns/iter (+/- 816)
test iter_i32         ... bench:       1,655 ns/iter (+/- 131)
test iter_i64         ... bench:       1,649 ns/iter (+/- 153)
test lookup_fail_i32  ... bench:       1,749 ns/iter (+/- 104)
test lookup_fail_i64  ... bench:       1,995 ns/iter (+/- 796)
test lookup_i32       ... bench:       2,439 ns/iter (+/- 126)
test lookup_i64       ... bench:       2,404 ns/iter (+/- 650)

Any idea why insert_no_grow is now detected as dead even if it is used by rustc_entry.rs? Ideas to fix this?

Perhaps we can run some kind of real-life benchmark? Rustc maybe? Any instructions on how to do that?

You can try using https://github.com/rust-lang-nursery/rustc-perf to see the impact of the change on a wide range of crates (including rustc). You will need to change src/libstd/Cargo.toml to point to your modified hashbrown instead of the one from crates.io, and then build a custom rustc toolchain using that libstd.

CI is failing because insert_no_grow is an unused function. Can you mark it with #[cfg(feature = "rustc-dep-of-std")] since it is only used by rustc_entry?

Closing in favor of #106.

authors: Amanieu alkis

repo: rust-lang/hashbrown


text:
Can I do l10n for the website and documentation?

The site might be okay, but the documentation needs to be in some kind of finished state before we commit to doing a ton of i18n work. It's hard enough to keep it up to date in just English!

@steveklabnik So I plan to do i18n for the site.
Is there any recommended way or tool to do this? (Or best practise?)

@alexcrichton @steveklabnik @brson
So I plan to do i18n for the site.
Is there any recommended way or tool to do this? (Or best practise?)

Closed.
Move to rust-lang/rust#21129

authors: caizixian steveklabnik

repo: rust-lang/rust-www


text:
Remove reference to nightly Rust for now stabilised feature.

👍

👍

authors: alexcrichton teymour-aldridge

repo: rustwasm/wasm-bindgen


text:
update docs to match parsed color format

huh weird it works fine now - I agree the # is nice

repo: rvaiya/warp


text:
SAK-30461 Add bullhorns to portal

Update: I've removed a chunk of the user switching. I had to add an extra method to profile2 to achieve this. I had to keep a user switch for the assignments calls, just while I get the assignment as there are calls to getCurrentUser in there. Basically, security advice is not enough to access the data when running in batch mode, down to the varying designs of the service security models. They are not consistent. So, it's better, but not ideal.

Just out of curiosity, are there not supplemental methods provided by Assignments to pass in a userId as opposed to the methods that call getCurrentUser()?

Not that I could see. I'll have another look. There are methods that take a User instance, but I couldn't fathom what they did. If they work, then cool.

No, basically. getAssignment and getSubmission (the user one) depend on a user being set. So, the cludge needs to remain until assignments is refactored.

So, can we get this in?

@buckett Regarding the thread pool stuff - if the db goes down do we actually care if tons of blocked threads get created? Does it matter enough to put in extra architecture to handle it? Surely if the DB goes down then we have  a dead system which will just be cycled? Genuine question as I had considered this stuff and thought the extra complexity not worthwhile.

Okay - I didn't realise that. Just a salt on the end.
On 10 February 2017 at 11:55, Matthew Buckett notifications@github.com
wrote:

@buckett commented on this pull request.
In portal/portal-render-engine-impl/pack/src/webapp/vm/
morpheus/includeStandardHead.vm
https://github.com/sakaiproject/sakai/pull/3822:

@@ -19,6 +19,10 @@

#if ($useBullhornAlerts)

   <script src="/profile2-tool/javascript/profile2-eb.js"></script>

No, but portalCDNQuery is a way of invalidating JS caching on the client.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/sakaiproject/sakai/pull/3822, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAINksWx_Zc2XUxyHTCDhYX2zO-mitRIks5rbFA2gaJpZM4LxHmS
.

I've implemented all the changes suggested, except the worker thread stuff. I'm still not sure how important it is to put in complexity to handle the case of the db being pulled from under Sakai. That is surely an unrecoverable situation anyway? I may well be missing the point.

Each thread consumes memory so you'll end up killing the JVM and even if you don't having 1000 threads all trying to process events when the DB comes back is not what you want. You really want the users to be getting served again, and if you do survive all the threads starting back up may well stave the DB from anything else.

@buckett But aren't you just going to have the same thing with a big piled up stack of events and the thread pool chugging through them? Assuming the db comes back up and Sakai can recover, which I've never known to happen (at least not after a significant DB outage), you'll still end up with a long period of DB utilisation while the pool catches up. Shallower but longer. That scenario still seems sketchy to me. Have you ever had the case where a DB goes down for a significant period (> 5 minutes, for example) and you've just let Sakai recover without a reboot? If we want to really cover that case then we need to cache the events in an intermediate table, too, so we can get them into the bullhorns on recovery. Maybe that's what should happen - maybe I should minimise the event thread hogging by dumping the wanted events into a table with some minimal, non service involving,  metadata and use that as my stack for the processing thread to pull off and build the bulhorn table alerts with. But, then I'd need, based on your suggestion, a worker pool doing the inserts into the intermediate table and a worker pool massaging the intermediate events into the bullhorns tables proper, the more time intensive stuff. So, a significant chunk of architecture is unfurling from this concern about DB outages, which is fine if it is justified based on experience. In my production experience, which is hugely less than yours Matthew, it isn't and that's why I just haven't bothered.

@ern What do you think about the threading issues @buckett has raised? I know you've raised similar concerns. I'm not dismissing Matthew's concerns at all - just trying to understand them. What sayest thou?

@ern The persistence is dead simple so it can be easily migrated down the line.

Yeah the biggest problem with it not being hibernate is we don't have a good pattern for future release automatic non-hibernate migrations.
These aren't that big a deal if changes only happen in major releases but are if features need to get added to minors.
The ideal though is that all migrations would be automated, but this is a roadmap dream.

authors: adrianfish bjones86 buckett jonespm

repo: sakaiproject/sakai


text:
Point Updates & Various Changes to Component Solve Factory

Closed by b1cf661cd26c9ef401fd8211b4dce4e6520b497c


text:
Scala version updated to 2.12.8

Thank you greatly.

authors: amirsh vjovanov

repo: scala-records/scala-records


text:
Replace TreeSyntax with scalacenter/scala-syntax

To expand with details.
The current TreeSyntax pretty-printer in Scalameta (Tree.syntax and Tree.toString) has several problems

It's unable to print comments
It has several roundtrip bugs where the printed output does not parse back into the same syntax tree
It is not able to wrap long lines by a configurable column width

While the first two of those these issues are possible to fix in the current printer, I believe it would require a new architecture to support line wrapping.
There already exists a tree printer in the repo https://github.com/scalacenter/scala-syntax that addresses all of these problems.

The printer has been rigorously tested against a large corpus of open source code. It handles tricky cases such as infix operators with tuple arguments, term.select when the qualifier is a unary application on a numeric literal
The printer supports line wrapping thanks to paiges https://github.com/typelevel/paiges
The printer can automatically attaches comment tokens to existing tree nodes
The printer supports programmatically attaching leading/trailing comments to tree nodes (both existing and synthetic ones)

The printer has open issues:

some comments in the original source get lost, our property tests that ~70% of comments are preserved. Getting it up to 100% is about grinding through the remaining test cases, finding minimal reproductions, fix those bugs and repeat.
the output is not always pretty, priority until now has been correctness over prettiness. Fixing this is usually fun since the paiges API is nice and we have a large test suite to prevent regressions
the printer depends on a fork of paiges because it relies on this commit here https://github.com/olafurpg/paiges/commit/114ec05b4a3099906c9159ccd1357f3b772b4f1d that is a breaking change in the paiges public API. Fixing this either involves contributing it upstream or publishing the fork under a shaded namespace.

When we do this, we should also add a structure pretty-printer.
I made a proof of concept here: https://github.com/masseguillaume/scalameta-structure
I have been using it for several weeks now.

@MasseGuillaume structure is unrelated to this PR, please open a separate ticket to discuss structure if you want.
There is an unmerged WIP branch for paiges that seems to avoid the fork that we currently have https://github.com/typelevel/paiges/issues/76#issuecomment-409223103 🎉

I took a stab at moving scala-syntax to this repo in https://github.com/olafurpg/scalameta/compare/pretty?expand=1

scala-syntax unit tests pass
slow property tests pass, but run ~3-5x slower than in scala-syntax repo (with more frequent OOM errors)
I haven't run scalameta tests yet

I'm inclined to put this change on hold because

I estimate completing the upgrade requires 2-3 full days of additional work. The code needs a more refactoring, and it still has critical bugs that need fixing.
it's not the top priority, I badly need to get scalafix v0.6 out as soon as possible
scala-syntax currently depends on a fork of paiges, but there's been progress lately on addressing those limitations so we may no longer need the fork in few months from now.
I'm less productive working on scala-syntax in the scalameta build than in a separate scala-syntax repo since IntelliJ doesn't understand the @ast macro annotations on scala.meta.Tree nodes

I'm less productive working on scala-syntax in the scalameta build than in a separate scala-syntax repo since IntelliJ doesn't understand the @ast macro annotations on scala.meta.Tree nodes

Can we replace the macro annotation with source generation?

@MasseGuillaume I estimate that would at least take several days to implement

authors: MasseGuillaume olafurpg xeno-by

repo: scalameta/scalameta


text:
chore: add yarn.lock file

Doesn't do well with github projects as dependencies as the commit hash is used during install. I will re-open later with proper lock file.