text:
Fixing flickery test

Changes Unknown when pulling 7ae1343ee5b3dc1e50d911eb12d75f08de435fb3 on lyncodev:master into ** on jtwig:master**.

authors: coveralls lyncodev

repo: jtwig/jtwig-core


text:
Check for a running session before looking for kernel specs

Fixed by https://github.com/jupyter/jupyter-js-plugins/pull/151

source: gharchive/issue

repo: jupyter/jupyter-js-plugins

license_type: permissive

license_source: bigquery


text:
Proper username resolving

А зачем эта правка? Что она дает?
И если она нужна, то лучше так
self.user = get_object_or_404(User, **{username_field: username})

Я просто скопировал откуда-то выше. Т.е. юзкейс не мной выдуман. Это так сейчас принципиально?)

То есть, ясно, что туда можно и не через кварги скормить класс User) что попалось под руку, короче)

О, да! Это случилось после перхода на email в качестве логина.

Женя, пересоздай плиз ревью с этой веткой
https://github.com/just-work/pybbm/tree/0.16.1-enot

source: gharchive/pull-request

authors: Atorich anatoliy-larin ttyS15

repo: just-work/pybbm


text:
how to run? Could you make available to us only invite to our personal servers?

For sure! I'm still developing one more feature (Habit Tracker + Integration of all Personal Development Tasks), but the other functions work and have been tested, I'd be honored to have the bot in your server, here's the link to invite PD Bot:

(The admin priviledges are to see all channels, be able to post anywhere - user permissions considered, and manage messages for a clean UI)

Please send any bugs or issues back on here - it would help tremendously with development!

I really want to use PD bot but PD is not replying to ?help or any command. There is something I must do first?

No it should work right off the bat, sorry about that it's likely a permissions issue - I've updated the invite link! (Kick PD Bot first, then reinvite)

Let me know if it's working or not! It should be working now

Working fine now! I'm testing rn

starting with journal commands

Awesome, thanks! Let me know if you have any questions, comments or suggestions

I'm so amazed of how awesome this bot is! You have put a lot of work on it! I have some questions! How many daily journals does the bot can hold until I ask it to post? And it is possible to set a channel to make PD bot send to this channel instead of asking me every time?
image

 I'm so glad I'm using this ahhhh =)

I'm so glad that I'm using this ahhh =)

repo: justincmendes/pd-bot

license: MIT

license_source: github-api


text:
Borderlands

This URL is preferred: https://www.epicgames.com/store/purchase?namespace=85189f7cf7a64f86aa6aa91d81d36c08&showNavigation=true&highlightColor=0078f2&offers=a61c6602b5784d36801d0cab73e02337

Thanks

authors: juzjus10 tcmaps xiurobert


text:
不能用brew安装吗，我用超级用户安装还是不行sudo gem install codeobscure，报错：ERROR:  While executing gem ... (Gem::FilePermissionError)     You don't have write permissions for the /usr/bin directory.

苹果系统升级之后，终端好多命令得重新安装，坑的一笔

这是gem包，必须gem安装！

sudo gem install -n /usr/local/bin codeobscure
改成这个就行了

authors: kaich makehapi

repo: kaich/codeobscure


text:
Adding scripts for the transformation of ontology files

What is the purpose of the core directory here?  Do you have other conversions that you expect will be using that code?  So far we have been including common "library" type code in the lib/ area.  If this is meant to be reused, it should reside there and not scripts/core/.
Can you add exception handling to your code?
Do you have tests and example input data?

I've added ontology.t and Toy_Ontology.* test data, and also an eval statement for the system() command in the transform scripts.
I did initially look in lib, but it seemed like it was specifically for the transform handling code itself, but I moved it to the root lib directory. @levinas intends to address how it should be integrated properly but won't have time before the demo.

@samseaver you should take a look at @cshenry code for FBA that uses try/catch instead of eval.  I'm concerned about "hidden" error conditions when code runs and we don't properly catch errors and report them out.  This is not only critical information for users, but also for debugging as a developer.

IT’s real simple Sam.. literally almost replacing the “eval” with a “try”...
--
Chris Henry
Sent with Airmail
On April 13, 2016 at 6:05:51 PM, Matt Henderson (notifications@github.com) wrote:
@samseaver you should take a look at @cshenry code for FBA that uses try/catch instead of eval. I'm concerned about "hidden" error conditions when code runs and we don't properly catch errors and report them out. This is not only critical information for users, but also for debugging as a developer.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub

Done

I guess I'm wondering why we need to convert from .obo to JSON at all?  Does it provide a significant advantage?  Do we expect that these .obo files would be quite small?  Would you always have a 1:1 association between JSON and .obo?
What about just keeping the .obo files in Shock and maintaining handles from the JSON?  Then the JSON would be high level summary information, possibly some lookup info for the file contents.
The uploaders and downloaders also then become simpler.
I didn't literally mean lib/ for the obo.pl file, sorry for not being more clear.  The perl library code is under lib/Bio/KBase/Transform.  I suggested there because I assumed you would want to import it as a module, and this path is already available.  I do think that if code was written to be specifically linked to a particular set of uploads or downloads, that should live together, although there was always an assumption in there that everything was independent.  There are efforts to migrate plugins into modules created using the SDK.  So I would say try to keep your code as self contained as possible to make that transition easy.  If you need to duplicate obo.pl between an upload and download dir, it is ugly, but practical for now.  So I would say just move obo.pl to be local to the rest of your upload code here.

Matt, what you propose seems like a significant departure from the data-modeling philosophy of this project. I know something along these lines is being discussed, but I didn't think any decisions had been made. I'm concerned that going this route would preclude the use of things like workspace subsetting to drive the population of dropdown in input forms etc. Ontology is an important type. It's one I feel we should handle rigorously and not just drop a handle to a file in the workspace. Also, we would have to port parsing code everywhere that the object is used, which seems bad.
I understand that data-API will fix this issue, but until we have a rigorous data-API in place, I think we should continue with JSON typed objects, as this serves as a proxy for data-API by keeping data in a convenient pre-parsed form.
I also think the OBO format is not perfect. For example, it does not directly support a lineage or hierarchical organization for ontology terms, which is very important. While it is true that our first prototype spec is very close to a direct translation of OBO, I expect that over the course of our sprints, we will expand and tweak this spec to better handle our use cases, and while these will likely always be a 1-1 correspondence between obo and the Ontology dictionary, I expect the dictionary to be a richer object.

I disagree that this is a significant departure because there has been no actual requirement for how to model data in KBase, only that it must have a representation as a KBase Type, which in some cases means a thin wrapper around a file (many cases of this) and in some cases all the data is in JSON docs (also many cases of this).  So I would say either approach has been accepted.  It has been relatively free form up to this point, since once you register a workspace module you can basically do whatever and there has been no actual formal review process, it has been ad hoc and inconsistently applied.
I am advising you based on our experience with GenomeAnnotation that going down this road of trying to store everything in JSON docs via the Workspace will later require a significant refactor or complete replacement.  JSON itself is not a bad thing, but it does have drawbacks, just like any other format, so we should be careful not to treat it as a one-size-fits-all approach to data.
Whether or not OBO is ideal, you will have to parse something no matter what.  There are other possible transformations of the data here.  It looks like a BigTable type storage might be nice to have here, but you could also just use tabular flat files on disk to get some of those benefits.  This would be easier overall to parse, subset, and index.  I would strongly recommend that your team at least consider this as an option.  The JSON should not be empty, but it also does not need to literally contain the primary data.  You should leverage the JSON for what it helps you with the most, mainly summarizing and visualizing.
If you really want to pursue the JSON only route, I'm not going to stop you in doing that.  I am just strongly advising against it.

At this point in time, we've primarily concerned with being able to demonstrate work done by the Ontology team in the narrative-ci. We plan on making changes following feedback, but not before next Tuesday given that the test scripts worked, can you merge this PR.

I see what you’re saying, but I just don’t see how we can make a landing page if we store the obo file directly in shock. Are we going to write JS code that pulls down the file and parses it in the browser? This is exactly what I meant by either needing a fully enabled data-API, or storing all the data needed to load the website in JSON.
And again, I think we will want to add additional relationships to the JSON that aren’t in the obo. So I see what you’re saying, but right now I think 100% JSON is the right way to go…  We may change our mind as we get deeper into this… and that’s fine. None of this is final… it’s just version 1.0. We have two more sprints coming on this topic.
On April 14, 2016 at 12:33:43 PM, Matt Henderson (notifications@github.com) wrote:
I disagree that this is a significant departure because there has been no actual requirement for how to model data in KBase, only that it must have a representation as a KBase Type, which in some cases means a thin wrapper around a file (many cases of this) and in some cases all the data is in JSON docs (also many cases of this). So I would say either approach has been accepted. It has been relatively free form up to this point, since once you register a workspace module you can basically do whatever and there has been no actual formal review process, it has been ad hoc and inconsistently applied.
I am advising you based on our experience with GenomeAnnotation that going down this road of trying to store everything in JSON docs via the Workspace will later require a significant refactor or complete replacement. JSON itself is not a bad thing, but it does have drawbacks, just like any other format, so we should be careful not to treat it as a one-size-fits-all approach to data.
Whether or not OBO is ideal, you will have to parse something no matter what. There are other possible transformations of the data here. It looks like a BigTable type storage might be nice to have here, but you could also just use tabular flat files on disk to get some of those benefits. This would be easier overall to parse, subset, and index. I would strongly recommend that your team at least consider this as an option. The JSON should not be empty, but it also does not need to literally contain the primary data. You should leverage the JSON for what it helps you with the most, mainly summarizing and visualizing.
If you really want to pursue the JSON only route, I'm not going to stop you in doing that. I am just strongly advising against it.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub

@samseaver the only issue currently is the location of obo.pl, which I think should be easy to correct.
The common area for perl code would be under lib/Bio/KBase/Transform/.  Since you are calling this using a system call from the upload and download code, changing the path of this may cause problems for you.
I'm not actually trying to block your sprint activity.  The problem is you are attempting to do something that is not supported by the transform plugin model.  As soon as the deployment happens, you will lose the relative path to obo.pl and your execution will no longer function.
If you treat obo.pl as a library, you can place it under lib/Bio/KBase/Transform/ and you can use a standard perl import to bring that code in and call the functions you need.  If you treat obo.pl as a command line tool, that is currently not supported, not because you can't use command line tools, but because you are organizing and executing your code in a way that transform does not support.
I'm trying to save you some extra debugging work and frustration since I can see the problem already.

Oh, I get it, the problem is the relative path. OK, given that @levinas is updating the code regularly in an external repository, I'll add copies of it into both the upload and download folders, and after the demo, we'll write a proper perl module.

authors: cshenry mlhenderson samseaver

repo: kbase/transform

license: mit


text:
add error_reporting(E_ALL); to bootstrap.php

fixed by 3ec466cb486633aa2d5e6ec67db2474d5e61e779

repo: keboola/docker-bundle


text:
我们为什么不问问神奇海螺！？

海螺：那我可去你妈的

authors: CMinHong difey

repo: kelseyhightower/nocode


text:
Fix modification of vault variables in [] access. #51

Codecov Report

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

@@            Coverage Diff            @@
##             master      #52   +/-   ##
=========================================
  Coverage          ?   99.56%
=========================================
  Files             ?       20
  Lines             ?      915
  Branches          ?      128
=========================================
  Hits              ?      911
  Misses            ?        3
  Partials          ?        1

Impacted Files
Coverage Δ

src/konfetti/vault/core.py
100% <100%> (ø)

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 fd97b28...daabae9. Read the comment docs.

repo: kiwicom/konfetti


text:
mv backtrace.rs backtrace/mod.rs

Seems good to keep this consistent between knurling projects (defmt also uses mod.rs)
bors r+

authors: Urhengulas jonas-schievink

repo: knurling-rs/probe-run


text:
Grammar in node join wait msg

LGTM

authors: jakolehm kke

repo: kontena/kontena-plugin-vagrant


text:
Fix binary images sorting

Haha oh damn!

authors: BamX kstenerud

repo: kstenerud/KSCrash


text:
Test KubeArmor on Azure (AKS)

@nam-jaehyun @nyrahul taking a look at this as discussed in the last community call.
Also, is https://github.com/kubearmor/KubeArmor/issues/58 a good reference on how we generally test compatibility with managed clusters?

authors: DelusionalOptimist nam-jaehyun

repo: kubearmor/KubeArmor


text:
Minor tweaks to the readme

/assign @rjbez17

/lgtm

repo: kubernetes-sigs/hierarchical-namespaces


text:
add zh trans content/zh/docs/reference/setup-tools/kubeadm/generated/…

/assign @chenopis

@tengqm  已经修改

/lgtm
/approve

authors: tanjunchen tengqm

repo: kubernetes/website


text:
Shall we rename repo to "dansuma-frontend"?

@grigio opinions?

BTW @lucabelluccini agreed to rename kuruko/backend to kuruko/dansuma-backend

Ok

Thanks @grigio
I have just updated https://github.com/kuruho/kuruho-pipelines accordingly.
See https://github.com/kuruho/kuruho-pipelines/commit/033e943503e8f5504ba665ada4d874806944139c

authors: gmacario grigio

repo: kuruho/frontend


text:
feat: sync 20170220 and Add missing config entry

喵喵哥真牛逼

authors: milkmeowo summerblue

repo: laravel-china/laravel-docs


text:
[5.2] Fixed docblock in ThrottleRequests middleware

I should have sent this PR to the 5.2 branch.

repo: laravel/framework


text:
[6.x] Use ubuntu-18.04

What's breaking on 20.04?

authors: GrahamCampbell driesvints


text:
Rewards rework

Closing this for a new PR based on the master branch.

repo: lbryio/lbry-android


text:
build out api client to associate courses to a blueprint

Alright, just bumped the version. I also added a few test courses on the test instance. Slacking those id's over to ya.

Verified, looks good!

Thanks!

authors: lcary wgwz

repo: lcary/canvas-lms-tools


text:
Planet option on "push buildings up" task

Additionally, options for building type and level limit have been suggested.

repo: le-serf/le-serf


text:
Swift version

Not sure what you mean by "Swift 4" as an issue. If there is still a problem please add a description.


text:
怎么设置当前可播放视频的时常，比如一分钟的视频，只给提供前十秒的试观看，还有就是怎么播放mp3音频文件

继承JCVideoPlayerStandard复写setProgressAndText()函数，取得当前的播放时间
mp3直接怼url就好了

authors: lipangit liuchenyang

repo: lipangit/JieCaoVideoPlayer


text:
Add Auto Lint

Hey @steviebeenz,
Your changes look good to me!
View build log
TravisBuddy Request Identifier: 468f58f0-c517-11e9-8712-75d78f9b457f

Hey @steviebeenz,
Your changes look good to me!
View build log
TravisBuddy Request Identifier: 4c532d70-c517-11e9-8712-75d78f9b457f

Please provide a description to understand easily what this PR does.

Fully lints, and formats HTML, CSS and JS.
See the Pocket Lint github plugin for more info.

Closing this PR since project had been migrated to Vue Nuxt  #51

authors: TravisBuddy liyasthomas steviebeenz

repo: liyasthomas/postwoman


text:
Figure something out for the  tags in portfolio detail page

Please describe the issue further with code examples in the issue description.
It's not clear what the issue is without some context.

authors: lottebijlsma thibmaek


text:
MS26 LSST+VISTA Forced Photometry Module ready to run on LSST Commissioning and Science Verification Observations

12/AUG/22

@raphaelshirley and @tms-epcc discussed the need for a change request for this Milestone due to its reliance on commissioning and science verification observations

17/OCT/22

Change request C050 submitted to 08/NOV/22 Exec Group meeting. This request is to delete this milestone.

09/NOV/22

Change request C050 approved at 08/NOV/22 Exec group meeting.
This change request removed this milestone due to lack of commissioning and science verification observations during Phase B.

repo: lsst-uk/lsst-ir-fusion


text:
How to delete record?

here 66b96e0ec64eee863b46fcda03a47bf548abf7d3

repo: luads/php-xbase


text:
Topic 4

Good work! But Consider commit diferent exercices in diferent folders, is more easy to read

repo: lucast-globant/java-bootcamp-bahia


text:
Fixes capitalization typo in StackExchangeResultset

Thanks a lot for spotting this!

authors: lucjon suever

repo: lucjon/Py-StackExchange


text:
no need to import cd, it isn't used in code

sorry, need to read the docs before i start doing PRs here... (code not working without that)

oops, it is working, actually.  :-\

good catch Zev, no need to import cd in this script

authors: makaimc zevav

repo: makaimc/fsp-deployment-guide


text:
updated the changelog

LGTM, thanks!

authors: dimosped eeaston

repo: manahl/pytest-plugins


text:
Add note about multiple authors to Metadata

CI failed. How do I fix this?
/Users/travis/.travis/job_stages: line 166: shell_session_update: command not found

repo: matryer/bitbar


text:
solve memory leak

Thanks! I merged this from the command line.

authors: fightingyy oschwald

repo: maxmind/libmaxminddb


text:
Add Delta NNN40 to releases and update ARM MPS2 target support

Manually merged squashed.

repo: mbedmicro/mbed


text:
Explain when client-side constants in offline-first apps get updated

 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.

Needs team review, will remove the draft flag once we're done reviewing it!

We should not mention PWA here

Done ✔

@ConnorLand I think this is for your component, please take a look.

@ConnorLand I think this is for your component, please take a look.

@ConnorLand I think this is for your component, please take a look.

Thank you. Resolving conflict and merging now.

@ConnorLand I think this is for your component, please take a look.

Thank you. Resolving conflict and merging now.

This PR was meant for the Constants doc in RefGuide8, and successfully did so before the Mx9 migration. Therefore I've taken the text from Osama and updated the RefGuide8 doc myself.
I'll shut this PR down now, because it targets the Mx9 doc mistakenly.

This PR was meant for the Constants doc in RefGuide8, and successfully did so before the Mx9 migration. Therefore I've taken the text from Osama and updated the RefGuide8 doc myself.
I'll shut this PR down now, because it targets the Mx9 doc mistakenly.

authors: Adam-Dupaski CLAassistant ConnorLand omnajjar

repo: mendix/docs


text:
update FAQ

good job!

Could English translation of this article be added?

Thanks for your comment. The translation of this article is in progress.

repo: mengshukeji/Luckysheet


text:
Ss/auth token and vips

@sascala:  One minor comment.  Otherwise, I'm happy to merge it.

authors: gabrielhartmann sascala

repo: mesosphere/dcos-kafka-service


text:
Chore/refactor to new ejsonable mixin

@DominikGuzei: Please review and merge if OK

repo: meteor-space/event-sourcing


text:
Correcting the comment on encoders.cpp and encoders.cs

 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.:x: pemamina sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

authors: msftclas pemamina

repo: microsoft/SEAL


text:
SqlCmdVar editing in Deploy UI

Dropping to Pri 2 because Insider experience will use profiles loaded from file instead of UX for changing sqlcmdvars at deployment time

Tracked in ADO

repo: microsoft/azuredatastudio


text:
Automatically adapt the type of button in structured lg

Pull Request Test Coverage Report for Build 84829

0 of 0   changed or added relevant lines in 0 files are covered.
721 unchanged lines in 85 files lost coverage.
Overall coverage increased (+0.1%) to 77.411%

Files with Coverage Reduction
New Missed Lines
%

/libraries/Microsoft.Bot.Builder.AI.LUIS/LuisApplication.cs
1
95.45%

/libraries/Microsoft.Bot.Builder.AI.LUIS/V3/LuisApplication.cs
1
96.43%

/libraries/Microsoft.Bot.Builder.AI.LUIS/V3/LuisUtil.cs
1
96.5%

/libraries/Microsoft.Bot.Builder/BotStateSet.cs
1
85.0%

/libraries/Microsoft.Bot.Builder.Dialogs/Debugging/SourceRange.cs
1
46.15%

/libraries/Microsoft.Bot.Builder.Dialogs/Memory/Scopes/MemoryScope.cs
1
88.37%

/libraries/Microsoft.Bot.Builder.Dialogs/PersistedStateKeys.cs
1
60.0%

/libraries/Microsoft.Bot.Builder.Dialogs/Prompts/PromptValidatorContext.cs
1
82.35%

/libraries/Microsoft.Bot.Builder/FileTranscriptLogger.cs
1
97.34%

/libraries/Microsoft.Bot.Builder/Inspection/InspectionSession.cs
1
76.47%

Totals

Change from base Build 84699:
0.1%

Covered Lines:
5658

Relevant Lines:
7309

💛  - Coveralls

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

authors: Danieladu coveralls fuselabs

repo: microsoft/botbuilder-dotnet


text:
Catch docgen feature up to main.

The test failures look to be happening only in IQ# and don't seem to be related to this PR. @bettinaheim, would you be good with merging this to the feature branch so that we can start e2e builds there and get that ready for merging to main?

For more context, the IQ# failure appears to be related to the recent cross-repo change between qsharp-compiler and iqsharp, and this branch somehow leading to an inconsistent state across the two repos in the E2E build. I'd recommend we merge this as-is to get the feature branch up-to-date, and then rely on the E2E build when merging the feature branch back to main.

Merging this PR with failing e2e builds as per @rmshaffer's diagnosis above and with @bettinaheim's signoff.

authors: cgranade rmshaffer

repo: microsoft/qsharp-compiler


text:
Adding support for queries in signing

LGTM

repo: minio/objectstorage-java


text:
Added sentry integration for error notifications.

:+1:

authors: aldavidson andyhd

repo: ministryofjustice/postcodeinfo


text:
fr-translation

Merci beaucoup ! J'ai remplacé "routing table" par "table de routage". Je mettrai à jour dés que possible.

Merveilleux !
Il y a peut-être d'autres anglissismes que j'ai laissés et qu'il convient de remplacer ?
Merci infiniment !
----- Mail original -----
De: "mintoo" notifications@github.com
À: "mintoo/checkio-task-summarization" checkio-task-summarization@noreply.github.com
Cc: "BibichePython" bibiche.python@laposte.net
Envoyé: Mercredi 9 Mars 2016 18:41:37
Objet: Re: [checkio-task-summarization] fr-translation (#3)
Merci beaucoup ! J'ai remplacé "routing table" par "table de routage". Je mettrai à jour dés que possible.
—
Reply to this email directly or view it on GitHub .

Non, je pense que c'est très bien comme ça :) Je ne crois pas qu'il y ait un équivalent de "summary route" en français.

Je viens de mettre à jour: la version française est disponible.
Merci !

authors: BibichePython mintoo

repo: mintoo/checkio-task-summarization


text:
GLT-3004: updated setup instructions for v4 scanner software

@avarsava: all fixed

repo: miso-lims/visionmate-client


text:
Support BaseQuery methods in db.session.query

ping?

@immunda this one seems trivial to me for addition

This would close #232 as well.

This doesn't fully solve the issue, a more far reaching solution proposed in #319

authors: RonnyPfannschmidt davidism immunda

repo: mitsuhiko/flask-sqlalchemy


text:
Fix missing " in docs/tricks.rst

Finally accepted, thanks!

authors: hszcg untitaker


text:
Add docstrings to public API. Make non-API members private.

UnaryFunctionFlow, FunctionWithStateFlow -> either make private or add docs (or both)
MapWithState -> add docs

authors: gtopper talIguaz

repo: mlrun/storey


text:
Add more translations for de

Thanks!

authors: mmistakes tmaier

repo: mmistakes/minimal-mistakes


text:
Compare Ray and Enzyme cloud API

Doc is in wiki

repo: modin-project/modin


text:
fix the version iri on mondo.owl

Its actually correct - not sure why OLS is showing the wrong one.

authors: matentzn nicolevasilevsky

repo: monarch-initiative/mondo


text:
a little fix :)

is this really open for anyone to edit ?!

repo: mongomock/mongomock

license: ISC


text:
docs: link to app specific docs.moov.io page

Codecov Report

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

@@           Coverage Diff           @@
##           master     #570   +/-   ##
=======================================
  Coverage   90.46%   90.46%
=======================================
  Files          64       64
  Lines        5086     5086
=======================================
  Hits         4601     4601
  Misses        351      351
  Partials      134      134

authors: adamdecaf codecov-io

repo: moov-io/ach


text:
motionblur: using MeshTools::compile for configuring the mesh

Merged as 092fb8aed5b3ad8c0e03410659467786baf24589. Thanks a lot!

authors: codewing mosra

repo: mosra/magnum-examples

license: unlicense


text:
結果がみづらい

・コマンドの置き換え結果
・実行結果を強調して表示する

repo: motsat/copipe


text:
[bug 1131391] Update pep8 to 1.6.1

Looks good!

7545259 [bug 1131391] update to pep8 1.6.1

repo: mozilla/fjord


text:
[Fix bug 1201027] Simplify Profile status UI

Coverage remained the same at 92.534% when pulling 32861940a8fd2abf018bfbd7a5002fc55f19725e on comzeradd:profile-status into c9245d46e0d75aec178be6107c1d5c93756456de on mozilla:master.

@johngian r?

@comzeradd please provide screenshot

I moved the vouch status on the left corner along with the rest of the profile details. For the unvouched user I kept the alert, as we discussed, becuase it's important for people to notice. The text also gets displayed on a tooltip if the user hovers the "pending profile" information.
I'd also removed the "Mozillian Profile" notice on top, which offers nothing. I kept the "Your Profile" notice when viewing yourself.

Coverage remained the same at 92.534% when pulling 28c344bf28890684ddba2b3181f97da774a1c890 on comzeradd:profile-status into ce955fb6c7cae6695934d25888d1e4a5e10de042 on mozilla:master.

PR updated with the new specs as laid out on the bug.

Coverage remained the same at 92.534% when pulling 8b432880e33fe4395f8e63458e88427d3c35d0ee on comzeradd:profile-status into ce955fb6c7cae6695934d25888d1e4a5e10de042 on mozilla:master.

Rebased to master

Coverage remained the same at 92.576% when pulling 18aaaa853089e756249ced65755877c1d0a780b7 on comzeradd:profile-status into 564ad3cad7dd6ba62ed46aa2fca0520c5d33edee on mozilla:master.

Coverage remained the same at 93.124% when pulling 429bb06bd1556af135f5e54ff1586c77bc94ff0f on comzeradd:profile-status into 44015ece7c61024643d233c8bbeddf096befdbef on mozilla:master.

Coverage remained the same at 93.231% when pulling 475d2135e4eefcf8638acccffb353b9a2bcee74e on comzeradd:profile-status into 1bcd32456fb6c98a675c14e41b9382b711746b06 on mozilla:master.

Coverage remained the same at 92.21% when pulling c493dc8c13f02758f5216007389165695a5ae69d on comzeradd:profile-status into 0abe0fa1bdfb506bde733138734178a0a9668456 on mozilla:master.

Coverage remained the same at 89.023% when pulling 7c5266411a1e2a8e8ca79b1af6b6e5ccf517ad38 on comzeradd:profile-status into 50e11c58fb416a34c06ddf4a188618b677e03a04 on mozilla:master.

authors: comzeradd coveralls ppapadeas

repo: mozilla/mozillians


text:
Cache bugzilla queries.

Filenames collide:
…
Caching data for: "https://bugzilla.mozilla.org/rest/bug?id=1208038" in "dist/cache/httpsbugzillamozillaorgrestbugid12.json"

Caching data for: "https://bugzilla.mozilla.org/rest/bug?id=1202580" in "dist/cache/httpsbugzillamozillaorgrestbugid12.json"
…

Would it make sense to have the various caches in-memory, as most development is done during watch (except changes in engine/, sorry)?

Would it make sense to have the various caches in-memory, as most development is done during watch (except changes in engine/, sorry)?

We could, I personally don't run watch much though.

authors: brendandahl digitarald

repo: mozilla/platatus


text:
Doesn't work with #ifdefs marks the scope as global after

Hi @alfaiyma, would you mind providing a simple code sample that demonstrate the issue?

Sorry for the late response but any code within an ifdef block in c reverts
to [global]
I.e
Void fun() {
int i;
#ifdef 1
i=1
#endif
}
On Apr 25, 2016 11:13 PM, "KS Chan" notifications@github.com wrote:

Hi @alfaiyma https://github.com/alfaiyma, would you mind providing a
simple code sample that demonstrate the issue?
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/mrkschan/ctags-status/issues/35#issuecomment-214594512

same problem here, should ignore pre-compile directives (#)

@pbraga @alfaiyma which ctags-status version are you using? And, what is the file extension of the file? .c or other file extension?
I cannot reproduce on ctags-status v1.3.5

I am using ctags-status v1.3.5 as well, and the files are .c and .h files.
It could be due to indentation as the pre compile directives do not have any in the project i am working on. So mostly like your example but with #ifdef and #endif at the same level the function.
I have not been able to properly test this.

Confirmed with the one below.
int f3() {
    int i;
#ifdef 1
    i=1
#endif
    return 0;
}

Hey, try v1.3.6.

Hi, I have just tested with new version and it does work :)
Thank you

authors: alfaiyma mrkschan pbraga

repo: mrkschan/ctags-status


text:
Add .env file, add scripts to Pipfile, change default Mongo port

@AlmightyYakob can we delete this branch?

@AlmightyYakob can we delete this branch?

Sure

authors: AlmightyYakob waxlamp

repo: multinet-app/multinet-girder


text:
Implement AnyMatcher

Closing in favour of #50

repo: myelin-ai/mockiato


text:
Develop

Pull Request Test Coverage Report for Build 477

167 of 169 (98.82%)  changed or added relevant lines in 17 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.04%) to 98.539%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/app/features/library/library.component.ts
7
8
87.5%

src/app/features/tracker/services/timer.service.ts
1
2
50.0%

Totals

Change from base Build 447:
0.04%

Covered Lines:
1696

Relevant Lines:
1709

💛  - Coveralls

authors: coveralls mzrimsek

repo: mzrimsek/track-my-playtime


text:
Fix url field in setup.py

Doh! Thanks.

authors: bnewbold djc

repo: nanomsg/nnpy


text:
Onestep install with pip

@leipert do you want to create a PR? Otherwise I would try to submit a PR for this tonight.

@miku I talked with @nareike about it. He want's to implement it. But I think if you create a PR, it would make him happy as well :+1:

@leipert, very nice. I added some packaging support in #8. I would even offer to maintain a pypi package, if @nareike is interested.
Btw, just did a tour of http://leipert.github.io/vsb/ – and wow, you have a great intuition for user interfaces.

authors: leipert miku

repo: nareike/adhs


text:
Add ffid metadata fields to existing API endpoint

Just noticed that there are a couple of tests failing on Jenkins.

Just noticed that there are a couple of tests failing on Jenkins.

Yes, I'm looking into what went wrong. I merged another branch (v1 of this one) that I accidentally committed to, into this one and I think it seems as though I broke something in another test.

authors: suzannehamilton techncl

repo: nationalarchives/tdr-consignment-api


text:
Automated Testing

https://github.com/cypress-io/cypress-electron-plugin

repo: neatfx/tiny-evt


text:
Add Python 2.7 as language for osx platform on Travis

This is not working.

repo: neuropoly/spinalcordtoolbox


text:
Add docs for NF edge release installation

Thanks @maxulysse !


text:
docs: add install via CDN guide

Can one of the admins verify this patch?

ok to test

authors: kyuwoo-choi seonim-ryu


text:
:ship: use normal user instead of root

@adamgraves-choices - can you rebase this which should hopefully address the broken build.

:rocket: deployed to http://connecting-to-services-pr-265.dev.c2s.nhschoices.net

:rocket: deployed to http://connecting-to-services-pr-265.dev.c2s.nhschoices.net

@adamgraves-choices The branch needs updating please. There are changes made to 'dockerise' this repo with docker-compose files but they are sat in PR #271 atm. Probably not worth making those changes in this PR but when #271 gets merged there will need to be some tweaks to the docker-compose files. And the removal of prepublish from package.json will also need to be done /cc @ajthomascouk

:rocket: deployed to http://connecting-to-services-pr-265.dev.c2s.nhschoices.net

Closing this as I've rebased and created a new branch/PR #274
Issues have been created for the additional notes I've made in the comments here.
@adamgraves-choices Can you review #274 please?

repo: nhsuk/connecting-to-services


text:
Added persistent_enums

@dom96, it looks like there was some connection problem during the test:
E: grp: unable to fetch repository https://github.com/achesak/nim-grp Call to 'readLine' timed out.

Could you restart them please?

Guess the tester needs to be less strict with timeouts.

repo: nim-lang/packages


text:
Falco on GKE

/lgtm

repo: nlamirault/portefaix


text:
L2SqrI: add fallback if the dimension is not a multiple of 4

Thank you @fabiencastan! I'll check the PR during the weekend!

Thanks! I've already forgot that dims with d%4!=0 where not supported at all!

authors: fabiencastan yurymalkov

repo: nmslib/hnswlib


text:
Same problem : Timeout (2000) waiting for 127.0.0.1:9229 to be free

please check: https://github.com/nodejs/node-inspect/issues/48#issuecomment-520246415

authors: linqFR yilmazdurmaz

repo: nodejs/node-inspect


text:
Updated circleci

Codecov Report

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

@@         Coverage Diff          @@
##           master   #10   +/-   ##
====================================
  Coverage       0%    0%
====================================
  Files           6     6
  Lines         257   257
====================================
  Misses        257   257

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 89838ce...576394c. Read the comment docs.

authors: BrettRToomey codecov-io

repo: nodes-vapor/meta


text:
docs: Added a note about rhoaster

Pull Request Test Coverage Report for Build 562

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.774%

Totals

Change from base Build 561:
0.0%

Covered Lines:
23

Relevant Lines:
23

💛  - Coveralls

Pull Request Test Coverage Report for Build 562

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.774%

Totals

Change from base Build 561:
0.0%

Covered Lines:
23

Relevant Lines:
23

💛  - Coveralls

authors: coveralls helio-frota

repo: nodeshift-starters/nodejs-health-check


text:
Have separate disposable for each project

Looks good. I'll merge and hotfix this when CI is done

OK, thanks!

Thanks for finding and fixing this!

authors: noseglid orgkhnargh

repo: noseglid/atom-build


text:
feat(azure-servicebus): support queues

Great work. The only thing missing is an update of Readme.md for the new feature.

Lacks tests 👎

@agevelt Feel free to add tests! :)

authors: agevelt oyvindholmstad tobiasgwaaler

repo: nrkno/odashboard


text:
Support for reference counted items in nopython lists

I believe this is an old (almost ancient) issue that can now be closed. With the advent of the  numba.typed.List we can store reference  counted types such as Numpy arrays in it. I think this is what this feature was asking for. I will close this now, as I believe the functionality requested exists.

authors: esc seibert

repo: numba/numba


text:
User can use Pages format for printing

Done. 3a45841
However, for now, user must switch to Pages mode manually before printing > #207

authors: amoshydra tharain


text:
Support 1.12, remove 1.10

Closing for #114.

repo: nytimes/drone-gke


text:
Add activity.published

Followed up and merged

authors: dooremont nicolaasmatthijs


text:
Mobile Material Enhancement

UnlightMaterial

[ ] AlphaMode.Cutoff
[ ] ambient logic

BinnPhongMaterial

[ ] AlphaMode.Cutoff
[ ] normalTexture
[ ] abstract normal plugin-in OES_standard_derivatives
[ ] occlusionTexture
[ ] opacity

shaderLib Clean Up

UnlitMaterial

[x] AlphaMode.Cutoff
[ ] ambient logic

BinnPhongMaterial

[x] AlphaMode.Cutoff
[ ] normalTexture
[ ] abstract normal plugin-in OES_standard_derivatives
[ ] occlusionTexture
[ ] opacity

shaderLib Clean Up

[x] clipplane

repo: oasis-engine/engine


text:
ES3 codestyle & package.json

What's the point of replacing const/let to var since all modern browsers (and mobile/embedded with strict mode) supports them?

Обсуждение с коллегами дало такие результаты:

const вообще в некоторых браузерах в реальности остается writable
с let в некоторых хромах есть проблемы

Также, говорят, что есть проблемы в сафари.
Ну и это же публичное SDK - не все пользуются последними версиями браузеров, надо поддерживать большую аудиторию.
К тому же, профита именно от использования const/let в коде я не вижу. В остальных ситуациях используется var и получается мешанина.

authors: Anonym-tsk lintax

repo: odnoklassniki/ok-js-sdk


text:
Use Native::Resouces instead of LibraryMake

I recalled that I had had the same issue Type check failed in binding %RESOURCES and had gave up using Native::Resources :)
In fact, https://github.com/skaji/perl6-librarymake-example does not use Native::Resources.
So you have 2 choices:

raise an issue to Native::Resources, or even pull request
give up using Native::Resources and use sub library:diff --git lib/Algorithm/KdTree.pm6 lib/Algorithm/KdTree.pm6
index aa23162..efa9bf8 100644
--- lib/Algorithm/KdTree.pm6
+++ lib/Algorithm/KdTree.pm6
@@ -2,16 +2,21 @@ use v6;
 unit class Algorithm::KdTree is export is repr('CStruct');

 use NativeCall;
-use Native::Resources;
 use NativeHelpers::Array;
+use LibraryMake;
 use Algorithm::KdTree::Response;

-my sub kd_insert(Algorithm::KdTree, CArray[num64], Pointer) returns int32 is native(resource-lib('kdtree', :%?RESOURCES)) { * }
-my sub kd_nearest(Algorithm::KdTree, CArray[num64]) returns Algorithm::KdTree::Response is native(resource-lib('kdtree', :%?RESOURCES)) { * }
-my sub kd_nearest_range(Algorithm::KdTree, CArray[num64], num64) returns Algorithm::KdTree::Response is native(resource-lib('kdtree', :%?RESOURCES)) { * }
-my sub kd_create(int32) returns Algorithm::KdTree is native(resource-lib('kdtree', :%?RESOURCES)) { * }
-my sub kd_free(Algorithm::KdTree) is native(resource-lib('kdtree', :%?RESOURCES)) { * }
-my sub kd_clear(Algorithm::KdTree) is native(resource-lib('kdtree', :%?RESOURCES)) { * }
+my sub library {
+    my $so = get-vars('')<SO>;
+    return ~%?RESOURCES{"lib/libkdtree$so"};
+}
+
+my sub kd_insert(Algorithm::KdTree, CArray[num64], Pointer) returns int32 is native(&library) { * }
+my sub kd_nearest(Algorithm::KdTree, CArray[num64]) returns Algorithm::KdTree::Response is native(&library) { * }
+my sub kd_nearest_range(Algorithm::KdTree, CArray[num64], num64) returns Algorithm::KdTree::Response is native(&library) { * }
+my sub kd_create(int32) returns Algorithm::KdTree is native(&library) { * }
+my sub kd_free(Algorithm::KdTree) is native(&library) { * }
+my sub kd_clear(Algorithm::KdTree) is native(&library) { * }
 my int32 $c-dimension;

 method insert(@array) returns Int {

@skaji Thank you for sharing your experience !
It looks that using sub library is the rational way to solve this problem.

authors: okaoka skaji


text:
adds hideIfEmpty prop, small props refactoring

Please remove package-lock.json due to yarn.lock is used. Update README.md with remark after table row compare to let users knows about cool feature. Also may be add test that container exists if no hideIfEmpty is specified to be sure with backward compatibility.

Could you give me direction on how to make tests? I don't understand how you organize files.

The file index.spec-set, where you added the test is like reusable library, it contain set of tests, which applied to different implementations. Also there some additional tests which do not uses that spec set. You added test in correct place.

@oklas yeah, but you have several TestApp that is not imported explicitly. And I need new TestApp with empty breadcrumbs. So where should I put it? Or I need  add props to some of your TestApp's?

yes, add props to some of TestApp's

Sorry, waste too much time on a such easy thing, even I don't really necessary to fix it.
If you want you can use my code to make change, but it would be easier to make it by yourself.
You need to make some refactoring and clean up the code I think.

Thank you. Please update commit by removing .idea folder. I will look at this.

Oh, do nothing, commit f6efe1567304ddc1b5c0bb03a99cf586a4989727 still available and looks good, i will continue from it.

Hi @FriOne. System does not recognize your commits as maded by you. Emails mismatch. The commits which you provided authored with different email than your acount. Could you please reassign your email in your commits in branch hide-if-empty. Or may be chage your account email will help. You can see that it is ok when your photo will be on your commits. You can just push in your fork repo. Ready to release. Looking forward your activity about authoring.

authors: FriOne oklas

repo: oklas/react-breadcrumbs-dynamic


text:
Transform services

Hi Luke
Your branch is messed up here - not sure why you have commits from tim in here. Can you see if you can sort out the branch history?
David

authors: dstuebe lukecampbell

repo: ooici/coi-services


text:
Update AWS NFS4 unsupported features URL

Erros seem to be unrelated to my change

repo: open-guides/og-aws


text:
Correction of spelling and labels homogenization

:+1:

authors: KevinArtus betd-rcarles


text:
Let Tom go ..

bors merge

bors retry

repo: openebs/Mayastor


text:
We only plan on releasing XRootD 5 (3.5 upcoming only) for EL8

Are there any corresponding changes for the prerelease parameters?

Yeah but there are a few other updates that need to go in so I figured I'd do that in a separate PR

authors: brianhlin matyasselmeci

repo: opensciencegrid/vm-test-runs


text:
AGM#7486-Add plainTextBody new output to GetMailMessage operation.

merged

authors: adinatusa ioanvranauhp

repo: openscore/score-actions


text:
Add: Largest Time for Given Digits

Codecov Report

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

@@          Coverage Diff          @@
##           master   #351   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files         126    127    +1
  Lines        1304   1315   +11
=====================================
+ Hits         1304   1315   +11

Impacted Files
Coverage Δ

...-for-given-digits/largest_time_for_given_digits.go
100% <100%> (ø)

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 023d008...1af1aaf. Read the comment docs.

authors: codecov-io openset

repo: openset/leetcode


text:
Target profile page rewrite: Create Safety data summary and detail widget

Related to #609. @LucaFumis please assign to @mirandaio once you have a working version in the angular app

Closing in favor of this https://github.com/opentargets/platform/issues/633

authors: ElaineMcA d0choa mirandaio

repo: opentargets/platform


text:
[travis] fix Cert_6_3_02_NetworkDataUpdate of posix-ncp check

I added another go() for parent updating its child address table. I results pretty stable. @librasungirl

@librasungirl, I'm not exactly sure.  Given that this issue occurs only in the pyspinel tests, I suspect it has something to do with the process split between ot-ncp and pyspinel.  We'll have to think of how to ensure the two remain in sync.  But, for now, I'll go ahead and merge this PR to unblock the rest.

authors: bukepo jwhui

repo: openthread/openthread


text:
Translation cn

@robekeane Just checking in here - I see the PR was opened but then closed?

@dzello I have translated the introduction, catalogue, and part of chapter I, as the main content is not yet completed, so I don't know if it is appropriate to submit the PR.

@robekeane Got it. It's okay with me if you want to do it incrementally with multiple PRs or all in 1 big one when the whole thing is ready. However you prefer.

authors: dzello robekeane

repo: orbit-love/orbit-model


text:
Test PHP 8.0

:+1:

authors: nauxliu overtrue

repo: overtrue/wechat


text:
Python 3.7 support

Travis CI issue https://github.com/travis-ci/travis-ci/issues/9815


text:
Replace the contextualCallbackUrl by a specific resolver

Will merge tomorrow unless there is a concern


text:
New pagination / iteration model.

One fairly large change requested, I can't submit and official review on this though because it's technically my PR.

Few more minor requests.

All LGTM, again, can't officially approve this since I opened it.

repo: pachyderm/pachyderm


text:
more

whoops, accidentally created this PR. Closing!


text:
Fix release run-tox uses.

The release failure this fixes is here: https://github.com/pantsbuild/lambdex/runs/2453608621?check_suite_focus=true

repo: pantsbuild/lambdex


text:
AppX Packages provider

Some question on this.
This provider could use the powershell provider and in purely in Windows PowerShell flavor since AppX module is not compatible with PowerShell 7.
I don't know of any C# library for accessing appx info.
Since this would most like map to powershell.exe Add-AppxPackage and Get-AppxPackage for getting the state.
This would indeed be useful to ensure WinGet appx is installed.
Name              : Microsoft.Winget.Source
Publisher         : CN=Microsoft Corporation, O=Microsoft Corporation, L=Redmond, S=Washington, C=US
Architecture      : Neutral
ResourceId        :
Version           : 2021.916.2041.763
PackageFullName   : Microsoft.Winget.Source_2021.916.2041.763_neutral__8wekyb3d8bbwe
InstallLocation   : C:\Program Files\WindowsApps\Microsoft.Winget.Source_2021.916.2041.763_neutral__8wekyb3d8bbwe
IsFramework       : False
PackageFamilyName : Microsoft.Winget.Source_8wekyb3d8bbwe
PublisherId       : 8wekyb3d8bbwe
IsResourcePackage : False
IsBundle          : False
IsDevelopmentMode : False
NonRemovable      : False
IsPartiallyStaged : False
SignatureKind     : Store
Status            : Ok

There is the Package deployment API and the Package query API which I think would be the preferable approach for this.
As an initial approach, I see nothing wrong with shelling out to PowerShell though.

Seems simple enough :) https://github.com/microsoft/Windows-classic-samples/blob/main/Samples/PackageManagerAddPackage/cs/AddPackageSample.cs

authors: jetersen patriksvensson

repo: patriksvensson/cupboard


text:
Update for Chalk

Images also go off screen a bit so it might be good to size images appropriately.

Added new resources

@C-D-Lewis Looks good.. :+1:

authors: C-D-Lewis MRoyPebble

repo: pebble-examples/big-time


text:
A simple typo

Thanks!

authors: merhawifissehaye mitar

repo: peerlibrary/meteor-blaze-components


text:
Agregado cambio de contraseña para la vista de usuarios.

Olvidada prueba unitaria.

repo: periface/AbpCinotamZero-SmartAdmin


text:
Add jsbeautifier support

going to migrate to eslint

repo: pierceray/baseproject


text:
add pipeline in saveas

LGTM

authors: googollee longbai

repo: pili-engineering/pili-sdk-python


text:
[DNM] pd-client: get cluster ID from PD

LGTM

PTAL @disksing

LGTM.

LGTM

authors: disksing huachaohuang siddontang

repo: pingcap/pd


text:
*: make test_scheduler_leader_change_twice more stable.

LGTM

PTAL @BusyJay

LGTM

authors: disksing ngaut siddontang

repo: pingcap/tikv


text:
Move conversion dialog to pipLifeCommons

complete

authors: KateNegrienko seroukhov

repo: pip-webui/pip-webui-controls


text:
fix onSuccess callback dependencies array firing

this fixed PlaidLinkOnSuccess not being able to be imported for me.

@skylarmb for me onSuccess was the only import not working.

author did not clean up PR to discard irrelevant changes, closing

repo: plaid/react-plaid-link


text:
Update README with local environment warning

@ydeshayes Why was this change made in the first place?
Wasn't the previous version returning null perfectly fine?

authors: muffe neyric

repo: platformsh/platformsh-nodejs-helper


text:
Fix/auhorized decorator koa

@NoNameProvided
Looks like you've merged this PR with rebase. It's a bad idea, you should always make a merge commit to keep track of PR and changes on branches.

At work we prefer clean history, but it's totally fine for me to do merge commits. I wont rebase in the future.

That's why rebase on github looks bad 😜

repo: pleerock/routing-controllers


text:
Release v0.0.2

Thank you!

repo: plotly/dash-cytoscape


text:
What is the .xctestrun file for?

It contains necessary information to execute the tests. For each test target it contains an entry with the path to the test host, some environment variables, command line arguments, ...
This file is not only used by pxctest, but also by xcodebuild when you use test-without-building.
You can find an example here: https://github.com/plu/pxctest/blob/master/PXCTestKitTests/Fixtures/Sample_iphonesimulator10.3-x86_64.xctestrun

authors: amesandy plu

repo: plu/pxctest


text:
Fix typo in README

Dupe of #1

authors: mignev pzi


text:
use full path for GA tracking

👍

repo: praekelt/molo


text:
Send a friend request function

Hello Orkan,
Could you rename the method to sendBuddyRequest ?
Thank's

Done!

Thank's !

authors: baoluo orkhanalizade

repo: processone/xmpp-messenger-ios


text:
Added CI with Jenkins

retest this please

retest this please

@metaswitchbot retest this please

@metaswitchbot retest this please

test this please

test this please

test this please

test this please

test this please

test this please

test this please

test this please

repo: projectcalico/calico-mesos


text:
AdminSetCreateService should not duplicate knowledge in the ASBehavior

@mjgiarlo I pushed up my suggested change to this branch

authors: jeremyf mjgiarlo

repo: projecthydra-labs/hyrax


text:
Allow setting of kubeRbacProxyMainResources in kube-state-metrics

kube-rbac-proxy resources can be set by editing its own library, isn't that enough?
https://github.com/prometheus-operator/kube-prometheus/blob/ae12388b33954fe228423dc2898719bdec2c0006/jsonnet/kube-prometheus/components/kube-rbac-proxy.libsonnet#L1-L39

I tried setting it like this:
local kp =   (import 'kube-prometheus/main.libsonnet') +   // Uncomment the following imports to enable its patches   // (import 'kube-prometheus/addons/anti-affinity.libsonnet') +   // (import 'kube-prometheus/addons/managed-cluster.libsonnet') +   // (import 'kube-prometheus/addons/node-ports.libsonnet') +   // (import 'kube-prometheus/addons/static-etcd.libsonnet') +   // (import 'kube-prometheus/addons/custom-metrics.libsonnet') +   // (import 'kube-prometheus/addons/external-metrics.libsonnet') +   {     values+:: {       common+: {         namespace: 'monitoring',       },       kubeRbacProxy+: {         resources+: {           limits+: { cpu: '400m' },         },       },     },   };
and
local kp =   (import 'kube-prometheus/main.libsonnet') +   // Uncomment the following imports to enable its patches   // (import 'kube-prometheus/addons/anti-affinity.libsonnet') +   // (import 'kube-prometheus/addons/managed-cluster.libsonnet') +   // (import 'kube-prometheus/addons/node-ports.libsonnet') +   // (import 'kube-prometheus/addons/static-etcd.libsonnet') +   // (import 'kube-prometheus/addons/custom-metrics.libsonnet') +   // (import 'kube-prometheus/addons/external-metrics.libsonnet') +   {     values+:: {       common+: {         namespace: 'monitoring',       },     },     kubeRbacProxy+: {         resources+: {           limits+: { cpu: '400m' },         },       },   };
both didn't work.
I think maybe because the arguments are overriden by the components creating the kube-rbac-proxy
https://github.com/prometheus-operator/kube-prometheus/blob/ae12388b33954fe228423dc2898719bdec2c0006/jsonnet/kube-prometheus/components/kube-state-metrics.libsonnet#L95

authors: ArthurSens Luis-TT

repo: prometheus-operator/kube-prometheus


text:
Do not output chunks that will be outside of the current max head time in future

Thanks for the PR.
does this fix a bug and what are the steps to reproduce it?
we should also add a test if it fixes a bug.

OK, i modified the TestDB_Snapshot to show what i said.
func TestDB_Snapshot(t *testing.T) {
db, close := openTestDB(t, nil)
defer close()
mint := int64(1414141414000)

app := db.Appender()

for i := 0; i < 1000; i++ {
	_, err := app.Add(labels.FromStrings("foo", "bar"+strconv.Itoa(i)), mint+int64(i), 1.0)
	testutil.Ok(t, err)
}

testutil.Ok(t, app.Commit())
testutil.Ok(t, app.Rollback())

// append data
go func() {
	app2 := db.Appender()
	for i := 1001; i < 100000; i++ {
		_, err := app2.Add(labels.FromStrings("foo", "bar"+strconv.Itoa(i)), mint+int64(i), 1.0)
		testutil.Ok(t, err)

		if i%1000 ==0{
			testutil.Ok(t, app2.Commit())
			testutil.Ok(t, app2.Rollback())
		}
	}

}()

// create snapshot
snap, err := ioutil.TempDir("", "snap")
testutil.Ok(t, err)

defer os.RemoveAll(snap)
testutil.Ok(t, db.Snapshot(snap, true))
testutil.Ok(t, db.Close())

// reopen DB from snapshot
db, err = Open(snap, nil, nil, nil)
testutil.Ok(t, err)
defer db.Close()

querier, err := db.Querier(mint, mint+1000)
testutil.Ok(t, err)
defer querier.Close()

// sum values
seriesSet, err := querier.Select(labels.NewPrefixMatcher("foo", "bar"))
testutil.Ok(t, err)

sum := 0.0
for seriesSet.Next() {
	series := seriesSet.At().Iterator()
	for series.Next() {
		_, v := series.At()
		sum += v
	}
	testutil.Ok(t, series.Err())
}
testutil.Ok(t, seriesSet.Err())
testutil.Equals(t, sum, 1000.0)

}
db_test.go:406: unexpected error: snapshot head block: write compaction: add series: symbol entry for "bar3338" does not exist
--- FAIL: TestDB_Snapshot (0.09s)
FAIL
after using &rangeHead, the error disappeared.

You also can use this version TestDB_Snapshot for test.
func TestDB_Snapshot(t *testing.T) {
	db, close := openTestDB(t, nil)
	defer close()

	mint := int64(1414141414000)

	app := db.Appender()

	for i := 0; i < 1000; i++ {
		_, err := app.Add(labels.FromStrings("foo", "bar"+strconv.Itoa(i)), mint+int64(i), 1.0)
		testutil.Ok(t, err)
	}

	testutil.Ok(t, app.Commit())
	testutil.Ok(t, app.Rollback())

	// append data
	go func() {
		app2 := db.Appender()
		for i := 1001; i < 3000; i++ {
			_, err := app2.Add(labels.FromStrings("foo", "bar"+strconv.Itoa(i)), mint+int64(i), 1.0)
			testutil.Ok(t, err)

			if i%1000 ==0{
				testutil.Ok(t, app2.Commit())
				testutil.Ok(t, app2.Rollback())
			}
		}

	}()

	// create snapshot
	snap, err := ioutil.TempDir("", "snap")
	testutil.Ok(t, err)

	defer os.RemoveAll(snap)
	testutil.Ok(t, db.Snapshot(snap, true))
	testutil.Ok(t, db.Close())

	// reopen DB from snapshot
	db, err = Open(snap, nil, nil, nil)

	testutil.Ok(t, err)
	defer db.Close()

	snapShotMaxTime := db.blocks[0].meta.MaxTime
	querier, err := db.Querier(snapShotMaxTime, snapShotMaxTime+500)
	testutil.Ok(t, err)
	defer querier.Close()

	// sum values
	seriesSet, err := querier.Select(labels.NewPrefixMatcher("foo", "bar"))
	testutil.Ok(t, err)

	sum := 0.0
	for seriesSet.Next() {
		series := seriesSet.At().Iterator()
		for series.Next() {
			_, v := series.At()
			sum += v
		}
		testutil.Ok(t, series.Err())
	}
	testutil.Ok(t, seriesSet.Err())
	testutil.Equals(t, sum, 0)
}

There is already #339 that addresses the issue the head initialisation so lets focus this one on fixing the Snapshoting.

@krasi-georgiev I saw. I think there are two problems, one is about output head to chunks, the other is about the head initialisation.

yes, and the other one is already addresses in the PR I linked.

yes, the problem about the head initialisation is already addressed in the PR.

Why did you close it? we still need the other changes. I meant just to remove the Head code changes.

authors: krasi-georgiev work-chausat

repo: prometheus/tsdb


text:
Artifact mount point doesn't work with udocker on remote HTC/HPC systems

For udocker, fixed in https://github.com/prominence-eosc/prominence/commit/4c54e1ee9c812fb0d2ed3fcf1d48e58959e56a5a

Fixed for Singularity in https://github.com/prominence-eosc/prominence/commit/dad85f25824ec51d5137c2492db9965c4db41b50

repo: prominence-eosc/prominence


text:
add validation that all files are remote

Coverage decreased (-0.1%) to 92.244% when pulling 3f24433b5a0b62dac086d9e8629a39dd1c33ab9d on validate into f8d208bd2fcc682537f1bfe161828d6af8c08b6e on develop.

repo: psu-stewardship/scholarsphere


text:
Use the latest version of Ruby 2.1 for testing

:watch: on Travis

:watch: :watch: :watch:

authors: jcoyne mjgiarlo


text:
Update metadata.json

LGTM.
Impartial on the question of version bump now or later.

Proposal: Merge as is. Postpone decision on version bump until the next feature release is being planned.

okay! thanks @nibalizer  / @ffrank.

authors: ffrank igalic nibalizer

repo: puppet-community/puppet-corosync


text:
DO NOT MERGE. PN implementation. For review only

CLA signed by all contributors.

I deliberately left out the AST to PN transformer tests because once the pspec is functional, each and every transformation will be thoroughly tested by the parser.pspec. It felt somewhat redundant to implement very similar tests in Ruby (it's 70 transformations and hence > 70 tests that needs to be duplicated).

What is PN? All of the commit messages are empty.

@josh, PN is short for Puppet Extended S-Expression Notation and it's Clojure like string representation is what that  the Puppet Specification Language, pspec, is using when validating a generated AST. I'm currently using it when validating the  Go-parser and the parser in pcore-java.
This PR is about adding PN and parts of pspec to Ruby so that we can validate all Puppet Language and EPP parsers using the same test suites. It's work in progress and I put it up here to get a preliminary review of the source code. Once the final PR arrives, it will have proper commit comments, ticket affinity, and API documentation.

@joshcooper PN is a formal replacement of the informal AST dump format we already have in the code base (used in puppet parser dump and in unit tests of the language). We want to make that API, and thomas has improved and fixed issues with the informal existing format.

superseded by #6673

authors: hlindberg joshcooper puppetcla thallgren

repo: puppetlabs/puppet


text:
Fix Unexpected token '.' error on Safari.

It's a shame my fix didn't actually fix the issue, even though the test case I set up passed... something to figure out another time.

authors: FlyingSixtySix Mikarific

repo: pxlsspace/Pxls


text:
Vendorizer licenses

blocked by #675

Coverage decreased (-0.3%) to 79.271% when pulling 38408d58507b003bcb849050128b84a6312e701d on arcivanov:vendorizer_licenses into 2392e90d1525e0d9764b809233ca1c25ccc22cd0 on pybuilder:master.

authors: arcivanov coveralls

repo: pybuilder/pybuilder


text:
Add clang-format to CircleCI

Codecov Report

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

@@          Coverage Diff           @@
##           master   #2057   +/-   ##
======================================
  Coverage    0.48%   0.48%
======================================
  Files          92      92
  Lines        7411    7411
  Branches     1128    1128
======================================
  Hits           36      36
  Misses       7362    7362
  Partials       13      13

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 0777b51...6bc4b5e. Read the comment docs.

authors: codecov-io fmassa

repo: pytorch/vision


text:
Update to cover Simon's Algorithm

@sebgrijalva Do it on the same PR.

authors: sebgrijalva vtomole

repo: quantumlib/Cirq


text:
Added code syntax highlighting to Readme

Thanks

Thanks for accepting.

authors: ragnraok ravidsrk

repo: ragnraok/RxCamera


text:
最新代码好像没有传到rubygems.org

@huacnlee 麻烦抽空提交到 rubygems 吧。Redis 的相关代码改了，但一直没提交过去。

repo: rails-engine/status-page


text:
Validate Azure volume plugin for K8s 1.4.

Here are my steps:

Register an azure virtual machine with kubernetes cluster on Rancher.
Attach a new disk to the azure virtual machine of size 1GB.
Label the azure virtual machine on kubernetes with label (provider=azure), to be used later with node selector to make sure the pod will get created on the azure machine.

kubectl label nodes testvol-k8s provider=azure

Run the pod, which should mount the disk on azure machine:

apiVersion: v1
kind: Pod
metadata:
 name: azure
spec:
 containers:
  - image: kubernetes/pause
    name: azure
    volumeMounts:
      - name: azure
        mountPath: /mnt/azure
 nodeSelector:
   provider: azure
 volumes:
      - name: azure
        azureDisk:
          diskName: newvol-test
          diskURI: xxxxxxxxxxxxxxxxxx

Results
$  kubectl describe pods
Name:		azure
Namespace:	default
Node:		testvol-k8s/13.93.200.151
Start Time:	Thu, 20 Oct 2016 22:42:10 +0200
Labels:		<none>
Status:		Pending
IP:
Controllers:	<none>
Containers:
  azure:
    Container ID:
    Image:		kubernetes/pause
    Image ID:
    Port:
    State:		Waiting
      Reason:		ContainerCreating
    Ready:		False
    Restart Count:	0
    Volume Mounts:
      /mnt/azure from azure (rw)
      /var/run/secrets/kubernetes.io/serviceaccount from default-token-zs7xu (ro)
    Environment Variables:	<none>
Conditions:
  Type		Status
  Initialized 	True
  Ready 	False
  PodScheduled 	True
Volumes:
  azure:
    Type:		AzureDisk (an Azure Data Disk mount on the host and bind mount to the pod)
    DiskName:		newvol-test
    DiskURI:		https://k8sdiag342.blob.core.windows.net/bootdiagnostics-testvolk8-4718c42b-863d-4bc7-a77c-e1717c00337f/newvol-test.vhd
    FSType:		ext4
    CachingMode:	None
    ReadOnly:		false
  default-token-zs7xu:
    Type:	Secret (a volume populated by a Secret)
    SecretName:	default-token-zs7xu
QoS Class:	BestEffort
Tolerations:	<none>
Events:
  FirstSeen	LastSeen	Count	From			SubobjectPath	Type		Reason		Message
  ---------	--------	-----	----			-------------	--------	------		-------
  20s		20s		1	{default-scheduler }			Normal		Scheduled	Successfully assigned azure to testvol-k8s
  20s		4s		6	{kubelet testvol-k8s}			Warning		FailedMount	MountVolume.SetUp failed for volume "kubernetes.io/azure-disk/ac82b7bf-9705-11e6-8614-025d5b72cb19-azure" (spec.Name: "azure") pod "ac82b7bf-9705-11e6-8614-025d5b72cb19" (UID: "ac82b7bf-9705-11e6-8614-025d5b72cb19") with: mount failed: exit status 32
Mounting arguments: /var/lib/kubelet/plugins/kubernetes.io/azure-disk/mounts/newvol-test /var/lib/kubelet/pods/ac82b7bf-9705-11e6-8614-025d5b72cb19/volumes/kubernetes.io~azure-disk/azure  [bind]
Output: mount: special device /var/lib/kubelet/plugins/kubernetes.io/azure-disk/mounts/newvol-test does not exist

@wlan0 Can you please take a look ^

I encountered similar issue for 1.5.1... Any workaround?

authors: galal-hussein sangeethah sheerun

repo: rancher/rancher


text:
[fixes #48] Prod run fixes

See issue #48
@bennett000 @neilff

Actually leaning more towards something like this for serving the client from node: https://github.com/SethDavenport/sethdavenport.com/blob/master/server/serve-client.js

in prod mode, serves statically from dist
in dev move, serves from webpack's dev middleware (with watching)
in both modes, supports html5mode history routing

This is what I'm using on my website; if there's interest I can spin it into it's own npm package.

repo: rangle/react-redux-starter


text:
Make modal accessible

Coverage increased (+0.2%) to 94.152% when pulling d813eb37d151eb6b22c93f03b2bfcfa750d22cd7 on make-modal-accessible into 51cea8bb3c1cc1a528cbb392dbec96b0e6693f3e on master.

authors: coveralls rbrtsmith

repo: rbrtsmith/nebula-react


text:
Update examples.md

Thanks, but we'd like to keep this page focused on vanilla React examples.


text:
ExtEventLoop: No longer suppress all errors

The problem is half solved now :) One of the @ suppression was removed with https://github.com/reactphp/event-loop/commit/8fa040756c3a62de2e5e160c2a4d63595c0c658e

@mamciek May I ask you to rebase this on the new "0.4" branch so we get this in for the upcoming v0.4.3 release? :shipit:

sure: https://github.com/reactphp/event-loop/pull/80

Sure. Done.

Thanks! 👍

authors: clue jsor mamciek

repo: reactphp/event-loop


text:
Specify charset explicitly for Admin Endpoint

If we do specify I feel it would be better to restrict to US-ASCII.

This is changing. Drastically. It'll be the same format as ClusterMember.parse. Including the memberId.

I've pushed the format change. I agree with @mjpt777 , the format used should be US-ASCII.

Done

authors: JPWatson mjpt777 tmontgomery

repo: real-logic/aeron


text:
CPU spike after Slave failover if subscriptionMode=SLAVE

Fixed

repo: redisson/redisson


text:
Make it easier to write unit tests (mocking etc)

:+1:

authors: cbxp takahi-i

repo: redpen-cc/redpen


text:
Update for Redwood 0.1.0

Closed in #19

authors: cannikin peterp

repo: redwoodjs/example-invoice


text:
update package.json for support React 15.0.0

Thanks for the reminder!  There were a couple other small changes to be made, so I'm working off PR #461.

authors: LeonidasEsteban devinivy

repo: reflux/refluxjs


text:
propogate maxFormCOntentSize in executor server as well

LGTM

@yanyang82 Don't need to update the gradle version. Do it from teamcity when releasing the jar

authors: ajmssc einstein-mychen yanyang82

repo: relateiq/azkaban


text:
Fix broken links

links fixed!!!

authors: kamolhasan reyadussalahin


text:
return empty array when user donot have ReadForwardingNumbers permission

Coverage decreased (-0.03%) to 38.25% when pulling 7a60aec417a07612ddf2aea4e0e6e3f45ae684c1 on embbnux:bugfix/regression-bug into 3357c09ca928aa05c905faf85ac62a9de57bc415 on ringcentral:master.

authors: coveralls embbnux


text:
Convert to .net standard 2.0

Cool! I wrote this library when I used to work at ripple. Be warned that the protocol definitions probably need updating. There's almost certainly been some new fields added.
Does this work xplatform (linux/osx too) now?
What are you using the library for?

Hey, no specific usage just did it as I'm interested in Ripple and how it will play out long term. It will be x platform only exception being the binary used in one of the tests...

authors: dandcg sublimator

repo: ripple/ripple-dot-net

license: isc


text:
Rename to DataPanel

Codecov Report

Merging #11 (f814a05) into main (a503720) will decrease coverage by 1.16%.
The diff coverage is 61.53%.

@@            Coverage Diff             @@
##             main      #11      +/-   ##
==========================================
- Coverage   48.33%   47.16%   -1.17%
==========================================
  Files          36       36
  Lines        2036     1972      -64
  Branches      358      358
==========================================
- Hits          984      930      -54
+ Misses        974      964      -10
  Partials       78       78

Flag
Coverage Δ

unittests
47.16% <61.53%> (-1.17%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

mosaic/contrib/wilds/__init__.py
0.00% <0.00%> (ø)

mosaic/mixins/inspect_fn.py
67.34% <ø> (ø)

mosaic/mixins/visibility.py
65.62% <ø> (-1.05%)
:arrow_down:

mosaic/datapanel.py
64.82% <61.90%> (ø)

mosaic/__init__.py
100.00% <100.00%> (ø)

mosaic/mixins/mapping.py
73.77% <100.00%> (ø)

mosaic/tools/identifier.py
38.46% <0.00%> (-5.07%)
:arrow_down:

mosaic/cells/imagepath.py
47.82% <0.00%> (-4.18%)
:arrow_down:

mosaic/mixins/identifier.py
77.77% <0.00%> (-4.05%)
:arrow_down:

mosaic/cells/abstract.py
60.71% <0.00%> (-3.81%)
:arrow_down:

... 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 a503720...f814a05. Read the comment docs.

authors: codecov-commenter krandiash

repo: robustness-gym/mosaic


text:
Add tracker package

#12


text:
Broken demo link

Thanks for catching this!

authors: kabirvirji rrdelaney

repo: rrdelaney/material-resume


text:
Fix Implicit Conversion Error

You have error in your configs. Market ID is always string.

authors: Tach-Yon yivo

repo: rubykube/peatio-trading-ui


text:
Add AppVeyor configuration

I'll let you play with this and merge it as soon as AppVeyor is green (which might that a while, lol)
Also I take offense that you renamed Readme.md to README.md – it feels like you want to tell me PascalCase is no longer appropriate here! 😛

Oh oops sorry! I figured that was a mistake but I'll rename back!

Haha I don't really care

Am 04.05.2018 um 22:57 schrieb Alex Crichton notifications@github.com:
Oh oops sorry! I figured that was a mistake but I'll rename back!
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

authors: alexcrichton killercup

repo: rust-lang-nursery/rustfix


text:
Fix type for Android st_nlink

r? @Amanieu
(rust-highfive has picked a reviewer for you, use r? to override)

@JohnTitor This changes st_nlink from i32 to u32. Is this an acceptable breaking change?

Actually these are already unsigned, so this isn't a breaking change.
@bors r+

:pushpin: Commit 6055dcb8ab59903ed72df036e37eece3b23e6ce1 has been approved by Amanieu

:hourglass: Testing commit 6055dcb8ab59903ed72df036e37eece3b23e6ce1 with merge 56fb9a97f471a2c748ed4f8b3d7abefe74931800...

:broken_heart: Test failed - checks-actions

@bors retry

:hourglass: Testing commit 6055dcb8ab59903ed72df036e37eece3b23e6ce1 with merge 2261e1a63bf1d7a2e457bea756693305aad820fd...

:sunny: Test successful - checks-actions, checks-cirrus-freebsd-11, checks-cirrus-freebsd-12, checks-cirrus-freebsd-13
Approved by: Amanieu
Pushing 2261e1a63bf1d7a2e457bea756693305aad820fd to master...

authors: Amanieu bors rust-highfive tweksteen

repo: rust-lang/libc


text:
add signup endpoint & profile schema

:+1:

authors: geraldanekwe ryanosaur

repo: ryanosaur/tour-journal


text:
Homework1 Веселов Николай

Билд не проходит - поправь checkstyle

а

Треш какой-то еще попал в PR - *.iml и Bar - проверь почему так

authors: Al-p-i Veselov-Nikolay rybalkinsd

repo: rybalkinsd/atom


text:
2019-04-10：自定义控件优化方案

性能优化
1.使用工具进行排查，包括：Memory Monitor、Allocation Tracker、MAT、LeakCanary
2.避免在onDraw()和onLayout中创建对象，因为这两个方法在绘制过程中会被频繁的调用
3.使用较小的数据结构，如SparseArray。
4.对Bitmap进行压缩和复用。
绘制优化
1.保证每帧耗时操作不能超过16ms。由于Android中每秒传输(FPS)不能低于60fps，所以，需要保证每帧耗时操作不能超过16ms。方法：在onDraw()绘制前开始计时，在结束后计算绘制时间。从而计算onDraw()绘制的时间。
2.通过主线程Looper处理消息或事件的时间差来监控应用整体卡顿的情况，例如第三方工具：BlockcCanary。
3.避免绘制过度。例如，控件之间的重叠，空间本身与背景之间的重叠等。在手机的开发者选项中开启调试GPU过度绘制，选择显示过度绘制区域，查看手机屏幕是否有颜色过深的区域，则为过度绘制区域。
4.减少无意义的绘制，例如进度条更新显示，如果读取文件的数据量不超过总文件的1%，就无需更新绘制。
布局优化
1.减少布局的重叠和嵌套。如果自定义控件继承RelativeLayout，该布局的父布局也是RelativeLayout，则可以将子布局的根节点设置< merge>< /merge> 或者< viewStub> < /viewStub>
2.尽量使用相对布局LinearLayout和约束布局ContraintLayout

自定义View优化策略
1.为了加速你的view，对于频繁调用的方法，需要尽量减少不必要的代码。先从onDraw开始，需要特别注意不应该在这里做内存分配的事情，因为它会导致GC，从而导致卡顿。在初始化或者动画间隙期间做分配内存的动作。不要在动画正在执行的时候做内存分配的事情。
2.你还需要尽可能的减少onDraw被调用的次数，大多数时候导致onDraw都是因为调用了invalidate().因此请尽量减少调用invaildate()的次数。如果可能的话，尽量调用含有4个参数的invalidate()方法而不是没有参数的invalidate()。没有参数的invalidate会强制重绘整个view。
3.另外一个非常耗时的操作是请求layout。任何时候执行requestLayout()，会使得Android UI系统去遍历整个View的层级来计算出每一个view的大小。如果找到有冲突的值，它会需要重新计算好几次。另外需要尽量保持View的层级是扁平化的，这样对提高效率很有帮助。
如果你有一个复杂的UI，你应该考虑写一个自定义的ViewGroup来执行他的layout操作。与内置的view不同，自定义的view可以使得程序仅仅测量这一部分，这避免了遍历整个view的层级结构来计算大小。这个PieChart 例子展示了如何继承ViewGroup作为自定义view的一部分。PieChart 有子views，但是它从来不测量它们。而是根据他自身的layout法则，直接设置它们的大小。

repo: safier/Android-Interview-Collection


text:
Fix usage example and update packages

Looks good, thanks for the contribution 👍

authors: bestpika samt

repo: samt/node-wiegand


text:
Merge master-qa

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.2 out of 3 committers have signed the CLA.:white_check_mark: jerome-benoit:white_check_mark: LucasBrazi06:x: Jakob8You have signed the CLA already but the status is still pending? Let us recheck it.

repo: sap-labs-france/ev-dashboard


text:
Xcode 7 Beta 4 changes

Thanks!

repo: sascha/DrawerController


text:
Added SaveDAI interface

@dougiebuckets sent Scott an email to ask if inheriting is necessary. We could post a question on OZ forum too.

@kseniya292 Cool, cool. Posed the question here: https://forum.openzeppelin.com/t/inheriting-from-interfaces/2909


text:
add context-bound-relaxed helpers for write-only Tracked

@bjaglin Thanks for the contribution! Overall I think the idea is good. Since this is an area of sbt that's confusing, I think it would be good to put Scaladocs to document our intent while it's fresh in our minds.

@bjaglin I think it would be good to put Scaladocs to document our intent while it's fresh in our minds.

Indeed! I amended my second commit 6be1090 to use W as a suffix and add scaladoc everywhere.
I realized the scaladoc was the same for all overloads (CacheStore or File`), so I took the opportunity to update the examples in 33b09dc .

repo: sbt/sbt


text:
Saving options shouldn't stop current session

Fixed as a small part of 370bba470350eebde95abe101920c037c163f613.
Users will now be prompted to save changes when they edit their settings.

repo: schmich/marinara


text:
Add shutdown button to safely shutdown the raspberry pi

Pull request #32 adds a shutdown button to the settings page.


text:
请问可以从PB文件逆向为.proto文件吗

可以的，根据Descriptor和FieldDescriptor

authors: jasonpakio zhangguanzhang

repo: sean-lin/protoc-gen-lua


text:
Add sixpack-java to client list

The timeline on this PR is really something.

authors: sddamico zackkitzmiller

repo: seatgeek/sixpack


text:
Fixed travis build warnings (#268)

Coverage increased (+0.002%) to 99.215% when pulling 512efd3a5c99ab97f4ce2313208e30453cff4be3 on 268/fix_travis into 478477a5334ff772115d8fa1de5e64b33a296e69 on dev.

authors: coveralls secynic

repo: secynic/ipwhois


text:
Fixing two compile errors

r? erickt

@homu: r+

:pushpin: Commit db3afa3 has been approved by erickt

:hourglass: Testing commit db3afa3 with merge 2b6d63b...

:sunny: Test successful - status

authors: erickt homu

repo: serde-rs/json


text:
Minor doc formatting improvements for Number.

I agree with most of this except for `Number`. For example in is_i64, the point is not to represent the serde_json::Number as i64. The point is to represent the JSON number (or number, depending on your perspective) as i64. That is, serde_json::Number and i64 are both just representations of some underlying concept of a number. In is_i64 or as_i64, we are taking that underlying concept and turning it into a different representation. Your emphasis makes it seem like the i64 represents a serde_json::Number which represents a JSON number, which does not seem accurate.

Addressed in latest force push.

Addressed in latest force push.

Thanks!

authors: dtolnay frewsxcv


text:
fix: support all parameters for apigw

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

authors: slsplus yugasun

repo: serverless-components/tencent-nextjs


text:
fix(scf): support custom runtime

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


text:
make dist-deps should install development dependencies

This is outdated, we use Gulp since #20

repo: shamansir/rpd


text:
Update to node 10 in .travis.yml

closing until oracledb has released version 2.3 with node.js 10 support

repo: shellthor/restify-boilerplate


text:
[fix] application config : set config.action_controller attributes

Coverage increased (+9.08%) to 69.338% when pulling 09920ba8b8b492239d275f6309e177e0d8cc9046 on fix/config into 5a435fc14bef418444af57e0c6fccdb58909894c on master.

Coverage increased (+9.08%) to 69.338% when pulling 09920ba8b8b492239d275f6309e177e0d8cc9046 on fix/config into 5a435fc14bef418444af57e0c6fccdb58909894c on master.

Coverage increased (+9.08%) to 69.338% when pulling 09920ba8b8b492239d275f6309e177e0d8cc9046 on fix/config into 5a435fc14bef418444af57e0c6fccdb58909894c on master.

authors: coveralls itowtips

repo: shirasagi/shirasagi


text:
feat: Add ConditionRule

Fails

:no_entry_sign:
You have added a new rule. Please change the docs too.

authors: FroshyBot shyim

repo: shyim/danger-php


text:
Created Ch 5 Loops Exercise

:hamburger:


text:
Rewrite all controls that use regex where a simple attribute will do

@aaronlippold vague and doesn't make much sense to me. re-open with an example if you still think this is relevant.

authors: aaronlippold samcornwell


text:
Add cross-env

The description needs work.

Yeah, I struggled with it. How about:
Set environment variables in scripts with unix style and it still works on Windows.

I'm not a huge fan of that either... :-/

Set environment variables cross-platform.

You should also add it here: https://github.com/bcoe/awesome-cross-platform-nodejs

Expose unix style environment variables cross-platform

Or what Sindre said :)

Updated. Thanks!

authors: SamVerschueren kentcdodds sindresorhus

repo: sindresorhus/awesome-nodejs


text:
Collapse sections

Where are your tests?

repo: smartcitiesdata/smartcitiesdata


text:
Update DirectInterpreter.php

Care to explain the change?

Care to explain the change?

sorry this my first time to use github

repo: smuuf/primi


text:
add Hypothesis as an alternative commenting system

 You should sign our Contributor License Agreement in order to get your pull request merged.

I'm a bit confused about this addition. As I understand it, hypothesis is 100% on the client meaning that each visitor would need to install it. Yes it does work with a Hugo site, but it would work with any site and since there is no action for the Hugo user to take it seems odd to mention it.

It works like Disqus, the site can include the Hypothesis script and the user will see Hypothesis and be able to comment on the page without having to manually install it.

You should need morereason to close this PR.

authors: CLAassistant fiatjaf spf13


text:
Add command to save all pipelines for a given application

Roer has been EOL'd in favor of spin: https://github.com/spinnaker/spin

authors: robzienert sargun

repo: spinnaker/roer


text:
build: fix poetry-dynamic-versioning not found

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

authors: arys-splunk rfaircloth-splunk

repo: splunk/addonfactory-ta-library-python


text:
Use os.path.normpath() instead of bespoke code. Fixes #92

This PR is a continuation of #131.

Sorry for the PR mess :)

Still sweet :+1:

authors: Tarrasch ravwojdyla wouterdebie

repo: spotify/snakebite


text:
Add spring-cloud docs infrastructure

fixed via 556bc4a13953eafcf28d921825d3ff660de093f9


text:
chore(deps): bump dependencies

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

repo: spurtli/react-konfettikanone


text:
Remove silent and other aliases

LOL!

dammit, some commands changed - gonna close this and redo later

authors: potatoqualitee wsmelton

repo: sqlcollaborative/dbatools


text:
Handle RejectedExecutionException when enqueue fails

Fixed #4152

This seems like a good change, especially if people customise the dispatcher thread pool.
Please add a test.  You will also need to sign the CLA https://github.com/square/okhttp/wiki/Contributing

Also, can you add a test?

Done, I will write a test later on if you don't mind.

@yschimke Am I doing something wrong that causes checks to fail ? Thanks.

It comes from https://github.com/socketio/socket.io-client-java/issues/539

@dave-r12 I disagree, we default to an infinitely sized executor so don't see this ourselves.  But okhttp public APIs allow specifying a tuned dispatcher with a specific (constrained) ExecutorService. And it's a public part of that API that job submissions can fail with this specific RuntimeException.  So we should handle this gracefully, and for OkHttp thats returning an IOException.  OkHttp throwing runtime exceptions tends to be fatal for android apps.

But it's not an IOException. That would be lying to the user.

How do we handle other cases when a request can't be satisfied on the client side before connecting remotely? We throw a fake 503 in other cases, when cache rules make the request impossible to fulfill.
But really the awkward thing for a caller is that given a valid but non default OkHttp client, a valid enqueue call can throw a RuntimeException which otherwise generally that indicates a bug or misuse, and sometimes leaves OkHttp in a undefined state.

Have you come to any kind of a decision on this? Thanks.

I think it’s a programming error to shutdown an executor service before you’re done using it. The RejectedExecutionException might already be the right choice.

It's not necessarily a shutdown executor, it could be a bounded one.  And reasons why I think we should handle it better than we currently do.

We suggest callers reuse a singleton Client.  So a caller to enqueue may not know how the client is configured.
Dispatcher with custom ExecutorService is part of the public API.  This usecase seems like a valid reason why you would specify a custom executor, to avoid unbounded memory growth.
We fail in surprising ways that may not be exposed to callers.  Leaving the call, event listeners and the client instance itself in an inconsistent state.

package com.baulsupp.oksocial

import okhttp3.Call
import okhttp3.Callback
import okhttp3.Dispatcher
import okhttp3.OkHttpClient
import okhttp3.Protocol
import okhttp3.Request
import okhttp3.Response
import java.io.IOException
import java.util.concurrent.ArrayBlockingQueue
import java.util.concurrent.BlockingQueue
import java.util.concurrent.RejectedExecutionException
import java.util.concurrent.RejectedExecutionHandler
import java.util.concurrent.ThreadPoolExecutor
import java.util.concurrent.TimeUnit

fun main(args: Array<String>) {
  val queue: BlockingQueue<Runnable> = ArrayBlockingQueue(3)
  val rejectionHandler: RejectedExecutionHandler = ThreadPoolExecutor.AbortPolicy()
  val tp = ThreadPoolExecutor(1, 3, 5, TimeUnit.SECONDS, queue, rejectionHandler)

  val dispatcher = Dispatcher(tp)

  val client = OkHttpClient.Builder().dispatcher(dispatcher).protocols(listOf(Protocol.HTTP_1_1)).build()

  val delayRequest = Request.Builder().url("https://httpbin.org/delay/1").build()

  val callback = object : Callback {
    override fun onFailure(call: Call, e: IOException) {
      println("onFailure: ${e.message}")
    }

    override fun onResponse(call: Call, response: Response) {
      println(response.code())
    }
  }

  try {
    client.newCall(delayRequest).enqueue(callback)
    client.newCall(delayRequest).enqueue(callback)
    client.newCall(delayRequest).enqueue(callback)
    client.newCall(delayRequest).enqueue(callback)
    client.newCall(delayRequest).enqueue(callback)
    client.newCall(delayRequest).enqueue(callback)
    client.newCall(delayRequest).enqueue(callback)
  } catch (ree: RejectedExecutionException) {
    println("Rejected: ${ree.message}")
  }
}

n.b. Neither onFailure or Rejected is ever printed.
Exception in thread "pool-1-thread-1" java.util.concurrent.RejectedExecutionException: Task okhttp3.RealCall$AsyncCall@51c9d05e rejected from java.util.concurrent.ThreadPoolExecutor@23048844[Running, pool size = 3, active threads = 3, queued tasks = 2, completed tasks = 1]
	at java.util.concurrent.ThreadPoolExecutor$AbortPolicy.rejectedExecution(ThreadPoolExecutor.java:2063)
	at java.util.concurrent.ThreadPoolExecutor.reject(ThreadPoolExecutor.java:830)
	at java.util.concurrent.ThreadPoolExecutor.execute(ThreadPoolExecutor.java:1379)
	at okhttp3.Dispatcher.promoteCalls(Dispatcher.java:166)
	at okhttp3.Dispatcher.finished(Dispatcher.java:203)
	at okhttp3.Dispatcher.finished(Dispatcher.java:190)
	at okhttp3.RealCall$AsyncCall.execute(RealCall.java:164)
	at okhttp3.internal.NamedRunnable.run(NamedRunnable.java:32)
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
	at java.lang.Thread.run(Thread.java:748)
200
200
200
200
200
200

I'm going to prod this further along by writing an illustrative test case I think we should aim to make pass.

What about the JavaDocs on Dispatcher?

Each dispatcher uses an {@link ExecutorService} to run calls internally. If you supply your own executor, it should be able to run {@linkplain #getMaxRequests the configured maximum} number of calls concurrently.

@dave-r12 fair point, it's still awkward to fail internally without ability to detect.

D

Do you any concrete suggestions how to handle this?
You are saying the socket.io that uses okhttp should be responsible?

Bump. Will appreciate a response.

@mstrelex the way I'm reading this: socket io library is shutting down the executor then attempting to use it after shutdown. They shouldn't do that. Can we confirm that's the case?

I do not see socket.io specifying a custom dispatcher or shutting down the executor.

Obsoleted by https://github.com/square/okhttp/pull/4365

Thanks @mstrelex ! I started by adding tests to your PR, and it ended up being a completely nastier change. Though you got us going in the right direction.

authors: JakeWharton dave-r12 garun2536 mstrelex swankjesse yschimke

repo: square/okhttp


text:
Clear all, exiting app and returning fills with default values

Fixed 1.1.0

repo: sreich/android-count-the-days


text:
MIMIR-17 added change date and description

Fixed position of element and fixed lint error

repo: statisticsnorway/mimir


text:
Fix links to readme in changelog

Ah thanks!

authors: rkallensee stevenosloan

repo: stevenosloan/slack-notifier


text:
[Unrestricted] Extra API Bits

Needs to be tested to confirm the controller changes work

authors: codingupastorm rowandh

repo: stratisproject/StratisBitcoinFullNode


text:
Implemented build, forwardSearch custom messages

You may want to add some checks somewhere to differentiate ST3 and ST4, because this code won't work on ST3.

According to the README the whole plugin is meant to be used on ST4 only. Would you add some checks anyway? If yes, should I simply return and do nothing?

Actually, TexLab does this already. All of that! ;) You can set the pdf
viewer in "settings".
I use a custom build system to invoke the TextCommand with my BuildSysten
keybinding.
Raoul Wols notifications@github.com schrieb am So., 28. Feb. 2021, 21:42:

@rwols commented on this pull request.
In plugin.py
https://github.com/sublimelsp/LSP-TexLab/pull/1#discussion_r584357352:

   if window is None:

       return

   if status == 0:

       pass  # success

   elif status == 1:

       window.status_message(PLUGIN_NAME + ": Previewer exited with errors")

   elif status == 2:

       window.status_message(PLUGIN_NAME + ": Previewer failed to start or crashed")

   elif status == 3:

       window.status_message(PLUGIN_NAME + ": Previewer is not configured")

def on_error_async(self, error):

   pass

+class LspTexlabBuildCommand(LspTextCommand):
I have a feeling we could do a lot more with this build command. Things
like

Recognize the %!TEX root =  directive to build a "root"
file instead of the current file (from LaTeXTools)
Maybe read view.settings().get("TEXroot") and use that as the root
file
Open the built pdf with a pdf viewer perhaps

https://latextools.readthedocs.io/en/latest/features/#multi-file-documents
But maybe that's all for another pull request :)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/sublimelsp/LSP-TexLab/pull/1#pullrequestreview-600303494,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AB6VULAVFQXDRB4XJTCRDZLTBKTC7ANCNFSM4YLBRWSA
.

Given my limited knowledge about LSP and I don't actually use LSP-TexLab, feel free to merge this when it's ready. And probably make a new release since the server binary is not updated very often so there is nothing to wait for.

authors: LDAP jfcherng

repo: sublimelsp/LSP-TexLab


text:
fixed typos

ack

authors: notsure turbo-ele

repo: sulu-io/SuluProductBundle


text:
合并冲突解决

已合并

authors: suwadaimyojin wrzrmzx

repo: suwadaimyojin/patchyvideo-vue


text:
fix typo on README

Looks good -- thanks!

authors: lerrua svanoort

repo: svanoort/pyresttest


text:
APP-1140 fix

Can one of the admins verify this patch?

Codecov Report

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

@@          Coverage Diff           @@
##             dev      #47   +/-   ##
======================================
  Coverage       ?   90.19%
======================================
  Files          ?        7
  Lines          ?       51
  Branches       ?        5
======================================
  Hits           ?       46
  Misses         ?        4
  Partials       ?        1

Continue to review full report at Codecov.

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

repo: symphonyoss/App-Integrations-Universal


text:
Hard link identical files from other containers

In our project with this feature enabled we have next results:
$ du -d 1 -h .vagga/.lnk/.roots
1019M   .vagga/.lnk/.roots/uaprom-static.c1e74563
237M    .vagga/.lnk/.roots/postgresql-base.6636d23a
632M    .vagga/.lnk/.roots/uaprom-test.e5a1a5b3
69M     .vagga/.lnk/.roots/flake8.a3844152
192M    .vagga/.lnk/.roots/elasticsearch.c3b5c1ed
234M    .vagga/.lnk/.roots/mongodb.8530a64c
436M    .vagga/.lnk/.roots/uaprom-tools.451a4eed
1,8M    .vagga/.lnk/.roots/docker-parser-test.3586b97a
26M     .vagga/.lnk/.roots/uaprom.a3269008
1,7M    .vagga/.lnk/.roots/docker-parser.767e8252
2,8G    .vagga/.lnk/.roots

$ du -d 1 -h -l .vagga/.lnk/.roots
1023M   .vagga/.lnk/.roots/uaprom-static.c1e74563
246M    .vagga/.lnk/.roots/postgresql-base.6636d23a
949M    .vagga/.lnk/.roots/uaprom-test.e5a1a5b3
302M    .vagga/.lnk/.roots/flake8.a3844152
314M    .vagga/.lnk/.roots/elasticsearch.c3b5c1ed
308M    .vagga/.lnk/.roots/mongodb.8530a64c
726M    .vagga/.lnk/.roots/uaprom-tools.451a4eed
59M     .vagga/.lnk/.roots/docker-parser-test.3586b97a
842M    .vagga/.lnk/.roots/uaprom.a3269008
59M     .vagga/.lnk/.roots/docker-parser.767e8252
4,8G    .vagga/.lnk/.roots

Okay, looks fine at a glance. Let's merge it in, and see how it plays out!

authors: anti-social tailhook

repo: tailhook/vagga


text:
There is no splitting of json dump output file into multiple files.

Are there any plans to enhance file.js module to take into account dump-file-splitting for ease of use of output file?

That sounds like an excellent pull request! Please submit it!
On Thu, Nov 26, 2015 at 2:19 PM maskofG notifications@github.com wrote:

Are there any plans to enhance file.js module to take into account
dump-file-splitting for ease of use of output file?
—
Reply to this email directly or view it on GitHub
https://github.com/taskrabbit/elasticsearch-dump/issues/137#issuecomment-159999437
.

--
Evan Tahler | TaskRabbit | Director of Technology
D: 415.795.1760

closing due to inactivity.

Sorry I am a bit occupied with my work... I may start working on this from
27th December..
On Dec 22, 2015 11:02 PM, "Evan Tahler" notifications@github.com wrote:

Closed #137 https://github.com/taskrabbit/elasticsearch-dump/issues/137.
—
Reply to this email directly or view it on GitHub
https://github.com/taskrabbit/elasticsearch-dump/issues/137#event-498731572
.

authors: evantahler maskofG

repo: taskrabbit/elasticsearch-dump


text:
script for demo - and content

I sent out an email with the doc/script. please review

authors: hinasakazaki wikiwikisf

repo: teamVCH/sway


text:
updated cdns

cdns updated


text:
Fix uglify plugin name

Merged as 1f2c50821429d1ad6c977d4fb8feb8fda7908a6d.

authors: ilkome terinjokes

repo: terinjokes/gulp-uglify


text:
Add missing override specifiers

virtual StaticShape OutputShape(const StaticShape& input_shape) const override;

virtual is redundant here.

@amitdo I adapted the commit accordingly.

authors: UniQP amitdo

repo: tesseract-ocr/tesseract


text:
Improve man page for tesseract  and add Makefile rule for PDF

Please also check for new info in --help-extra.
You can add the new psms.
 11    Sparse text. Find as much text as possible in no particular order.
 12    Sparse text with OSD.
 13    Raw line. Treat the image as a single text line,
       bypassing hacks that are Tesseract-specific.

--dpi N is also still missing.

You can also consider whether some of the info in the following pages also needs to be added to man page.
https://github.com/tesseract-ocr/tesseract/wiki/Command-Line-Usage
https://github.com/tesseract-ocr/tesseract/wiki/FAQ
Examples of what could be added (IMO)...
https://github.com/tesseract-ocr/tesseract/wiki/FAQ#what-page-separators-are-used-in-txt-output-by-tesseract-4-0-0
Info about pdf with text layer only
https://github.com/tesseract-ocr/tesseract/wiki/Command-Line-Usage#integrate-original-image-file-and-detected-text-into-pdf
https://github.com/tesseract-ocr/tesseract/wiki/Command-Line-Usage#ocr-single-page-of-a-multi-page-tiff

2 = Tesseract + LSTM.
How about adding that this is only available in the traineddata files from tessdata repo - though not all of them.

I added two commits for the missing --psm and --dpi options. See old and new man pages.

@Shreeshrii, thank you for your suggestions. I am currently a little bit short of time, as I am just preparing a Tesseract workshop for next Monday.

See old and new man pages.

The new format looks better. Thank you.
You could move the following lower on the page:
--user-words FILE
Specify the location of user words file.

--user-patterns FILE
Specify the location of user patterns file.

Those options are now sorted alphabetically.

I am just preparing a Tesseract workshop for next Monday.

@stweil 👍

authors: Shreeshrii stweil


text:
setup process to package and publish to npm

I think I’d like to shorten the name before this is published. testcontainers-js?
Also, please let’s review it as a whole before publication.

authors: rnorth swissarmykirpan


text:
Add tests for pulling stats

this is hard to do because we don't have a test GA environment -- but we probably ought to have a test environment -- but that's hard and weird

authors: brianboyer hampelm

repo: thecarebot/carebot