text:
CI for Node

Game
  ✕ Initializes with two players (3 ms)
  ✕ Starts the game with a random player (1 ms)
nextPlayer
  ✕ Sets the current player to be whoever it is not (1 ms)

repo: Nirodha-Github/github-actions-for-ci


text:
Gathered yaml files manually for Controllers, CRds and CRs

@kanwar-saad do we still need this issue ?

It seems we do not need this PR any longer.
Closing it.

source: gharchive/pull-request

repo: Nordix/airship-dev-tools

license_type: permissive

license_source: github-api


text:
Notify users when an operation times out

Actually fixed the message a few hours before you made this, just haven't restarted the bots yet for a core update.

source: gharchive/issue

repo: NotSoSuper/NotSoBot

license: mit

license_source: bigquery


text:
Regex filtering

a classic example of take it home and do it again.

repo: Novus-Ideam/novus-ideam

license: MIT


text:
Fixed #331: Listing/Unlisting packages is undiscoverable

Note that this file has messed up CRLF, and I did not fix it with this commit, so that my diff is just what I changed.

repo: NuGet/NuGetGallery


text:
[ReleasePrep][2018.04.25]RI of dev into master

Looks good right now but I'd like to get delete org in if possible!

authors: loic-sharma scottbommarito


text:
Update Dependency-Resolution.md

Docs Build status updates of commit bc51a26:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/consume-packages/Dependency-Resolution.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

fixes https://github.com/NuGet/docs.microsoft.com-nuget/issues/1542

authors: anangaur karann-msft


text:
master -> live

Docs Build status updates of commit b11f922:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/hosting-packages/Overview.md
:white_check_mark:Succeeded
View

docs/nuget-org/id-prefix-reservation.md
:white_check_mark:Succeeded
View

docs/reference/cli-reference/cli-ref-pack.md
:white_check_mark:Succeeded
View

docs/reference/errors-and-warnings/NU1605.md
:white_check_mark:Succeeded
View

docs/reference/msbuild-targets.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


text:
Oc 251 remove alliander from open source code

SonarQube analysis reported 17 issues

 17 major

Watch the comments in this conversation to review them.

All is well.

Review not possible due to large number (over 1000) of deleted amd added files.
(file moves were unfortunately not properly displayed in github, making file comparisons impossible)

repo: OSGP/Platform


text:
Excel data validation

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/excel/excel-add-ins-data-validation.md
:white_check_mark:Succeeded

docs/toc.yml
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

repo: OfficeDev/office-js-docs-pr


text:
Add 1.7+Mac

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/overview/office-add-in-availability.md
:white_check_mark:Succeeded

docs/reference/requirement-sets/outlook-api-requirement-sets.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


text:
Merge to live!

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/develop/create-sso-office-add-ins-aspnet.md
:white_check_mark:Succeeded

docs/excel/excel-add-ins-data-validation.md
:white_check_mark:Succeeded

docs/reference/manifest/officeapp.md
:white_check_mark:Succeeded

docs/reference/overview/excel-add-ins-reference-overview.md
:white_check_mark:Succeeded

docs/reference/overview/onenote-add-ins-javascript-reference.md
:white_check_mark:Succeeded

docs/reference/overview/word-add-ins-reference-overview.md
:white_check_mark:Succeeded

docs/testing/troubleshoot-manifest.md
:white_check_mark:Succeeded

docs/toc.yml
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


text:
Adiciona sistema Xbox aos links úteis de lógica

vc é top ❤

authors: ArthurFerrao thayannevls

repo: OpenDevUFCG/Tamburetei


text:
Fixed websocket and encoding errors

For the change in the password reset code: https://github.com/django/django/commit/c82893cb8c6b2a4a876965426c5a5bc4590e1583

repo: OpenSlides/OpenSlides


text:
JSON import check | show perm user groups | CuiHelper.AddUI json string

Looks great!  Thanks nogrod!

authors: Calytic Nogrod

repo: OxideMod/Oxide


text:
durandal-v1:Updates from code review

@bkueng - thank you for the feedback!


text:
[WIP] add overview content

新手入门、使用指南和进阶指南的guide，请各自放在对应的index文件，不需要额外新增文件

@shanyi15 还需要保留原来的目录内容吗？

http://paddlepaddle.org/documentation/docs/zh/1.0/beginners_guide/install/Start.html 关于“是否需要保留原来的目录内容”，这其实也是一个index，可以参考这样来写

新手入门

使用指南

进阶使用

repo: PaddlePaddle/FluidDoc


text:
7.7homework

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

authors: CLAassistant RobinWanglb

repo: PaddlePaddle/awesome-DeepLearning


text:
Code | Numeric Pattern 151

@aryashah2k kindly assign this one to me.

repo: PatternHouse/Java-PatternHouse


text:
EASTER EGGS

Duplicate of #44, closing.

authors: em-p nywillb

repo: PlanHubMe/PlanHub


text:
Literal args comma fix

From my tests, I'm still having issues with computed expressions that have only literal arguments.  In the example below, node.foobar returns undefined.
foobar: {
   computed: "computeFooBar('foo', 'bar')"
}

computeFooBar(foo, bar) {
   return foo + ' ' + bar;
}

authors: miztroh sjmiles

repo: Polymer/polymer


text:
Enabled restaurant demo test

Further investigation into #1339

repo: PolymerLabs/arcs


text:
Hausaufgabe_2_S0574551

Hier nochmal hübsch :)

repo: ProfWider/tdd-setup


text:
checks on added full stops in 10 lectures (batch3)

Nice job @AnjuJoon , thank you.

authors: AnjuJoon jstac

repo: QuantEcon/lecture-source-py


text:
Update README.md

thanks Scott!
+1

:+1:

:+1:

authors: benbp heckj scott-xu yyscamper

repo: RackHD/on-core


text:
SettingsLoader

@zebulasampedro another look?

repo: ResearchComputing/web-telemetry


text:
Can NetSerializer data be compressed?how is the data size compared to Json.Net + Gzip?

Can NetSerializer data be compressed
Sure you can compress data generated by NetSerializer.
If you want fast compression and decompression use LZ4.
how is the data size compared to Json.Net + Gzip
I didn't compare.

authors: QiuBuZai RevenantX

repo: RevenantX/LiteNetLib


text:
[IMPROVEMENT] Convert "Change password" activity to fragment.

can I Take this @divyanshub024?

@VANKINEENITAWRUN I'll be working on this.


text:
Add Abap Doc annotations to types

@huber-nicolas Would you also add the corresponding JSON schema file, please?

@huber-nicolas Would you also add the corresponding JSON schema file, please?

done. See the newest commit

@schneidermic0 you can start your re-review now. There is only the Problem with abaplint which claims that I do not use two types of zif_aff_types_v1 (although they are used in zif_oo_aff_intf_v1 )

false positives fixed, only the first file was found by the glob statement, https://github.com/SAP/abap-file-formats/pull/99/commits/ae26098fad103b66bfb3b270fb486ed5932d847a

authors: huber-nicolas larshp schneidermic0 wurzka

repo: SAP/abap-file-formats


text:
Fix destination environment variable value

 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.

@sacnl Do you have a mechanism/script to apply this change to all relevant branches?

Not a single script, but a sequence where I already did this. Will be uploaded soon.

authors: CLAassistant daniel-kurzynski sacnl


text:
Week 13 Dependency Updates

Will need to wait until CumulusCI is released, so that it matches the requirement of keyring==23.0.1

repo: SFDO-Tooling/MetaDeploy


text:
Add file import report to testrail

Coverage remained the same at 94.056% when pulling 597ca12a9332fd3366acae862bece80dfa758fd7 on dropdbs:master into 2a366ace0870149059c25515689d5002a6084158 on SSENSE:master.

authors: coveralls dropdbs

repo: SSENSE/node-cucumber-testrail-sync


text:
Feature: Add username to firebase data

Fixed in 71ef99d

repo: Sacret/githubify


text:
BRIDGE-636 Update tapping schema revision to 9

wow that's a lot of changes

authors: Erin-Mounts syoung-smallwisdom

repo: Sage-Bionetworks/mPower


text:
Vulkan: blending and stencil testing

The stencil buffers do not clear correctly in the GL renderer. Will fix that in a future PR.

@apjagdale @roshanch Please review

added blend flags to hashcode
separate namespace for vulkan render flags

looks good.

Planning to merge on Tuesday

@liaxim This crash has been fixed, will have a PR

authors: liaxim roshanch sushantojal

repo: Samsung/GearVRf


text:
Initial Content

Pull Requests

https://github.com/Senzing/docker-compose-demo/pull/2
https://github.com/Senzing/docker-compose-demo/pull/3
https://github.com/Senzing/docker-compose-demo/pull/4
https://github.com/Senzing/docker-compose-demo/pull/5
https://github.com/Senzing/docker-compose-demo/pull/6
https://github.com/Senzing/docker-compose-demo/pull/9

repo: Senzing/docker-compose-demo


text:
Merge sferik/twitter master

https://shuttlerock.atlassian.net/browse/STUDIO-1635

authors: avokhmin koki-miyazaki

repo: Shuttlerock/twitter


text:
ENH: upgrade CI for python packages to 5.2rc01

@SimonRit I see the artifact download error in Build, test, package / publish-python-packages-to-pypi (push) , but I do not see the source of this -- any pointers?

@SimonRit I see the artifact download error in Build, test, package / publish-python-packages-to-pypi (push) , but I do not see the source of this -- any pointers?

@SimonRit I see the artifact download error in Build, test, package / publish-python-packages-to-pypi (push) , but I do not see the source of this -- any pointers?

No. I'm guessing it's a github issue which wasn't there in the previous compilation. I only amended the commit and forced push to modify setup.py following InsightSoftwareConsortium/ITKModuleTemplate@ddf8381339ca92143b4e39ae6e30d252f58f6e7b. I could relaunch the compilation but I will not merge anyway because the produced packages don't work, see InsightSoftwareConsortium/ITK#2064. Any clue how to solve this issue? It's been there for a long time but I have no idea how to solve this...

@SimonRit I see the artifact download error in Build, test, package / publish-python-packages-to-pypi (push) , but I do not see the source of this -- any pointers?

No. I'm guessing it's a github issue which wasn't there in the previous compilation. I only amended the commit and forced push to modify setup.py following InsightSoftwareConsortium/ITKModuleTemplate@ddf8381339ca92143b4e39ae6e30d252f58f6e7b. I could relaunch the compilation but I will not merge anyway because the produced packages don't work, see InsightSoftwareConsortium/ITK#2064. Any clue how to solve this issue? It's been there for a long time but I have no idea how to solve this...

@SimonRit yes, addressed in https://github.com/InsightSoftwareConsortium/ITK/pull/2258

@SimonRit yes, addressed in https://github.com/InsightSoftwareConsortium/ITK/pull/2258

Superseded by #403.

authors: SimonRit thewtex

repo: SimonRit/RTK


text:
Generated queries objects with fragments not works

Hey @Sparow199 - I'm having this issue as well. Were you able to work around it using fragments?

Hi, have you tried with le latest version of the plugin? It's emmbed the
latest version of Apollo compiler. I think that it could be interesting to
try to generate the same queries with Gradle and compare the results.
On Tue, Jul 9, 2019, 22:24 Drew Horn notifications@github.com wrote:

Hey @Sparow199 https://github.com/Sparow199 - I'm having this issue as
well. Were you able to work around it using fragments?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Sparow199/apollo-client-maven-plugin/issues/3?email_source=notifications&email_token=AFJ7ZUTHYVONTHGM5HATFZTP6TXW7A5CNFSM4GZ54NG2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODZRNLEA#issuecomment-509793680,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFJ7ZUWLIKHAFA5YTRJ44FDP6TXW7ANCNFSM4GZ54NGQ
.

Hi @Sparow199 - I'm using 1.3.0 for this plugin and 1.0.0 for the apollo dependency per the docs in this repo. I believe those are the latest.

Yes, they are the latest, there is a bug, but it's not easy to debug. I
have to look deeply to Gradle plugin.
On Wed, Jul 10, 2019, 21:20 Drew Horn notifications@github.com wrote:

Hi @Sparow199 https://github.com/Sparow199 - I'm using 1.3.0 for this
plugin and 1.0.0 for the apollo dependency per the docs in this repo. I
believe those are the latest.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Sparow199/apollo-client-maven-plugin/issues/3?email_source=notifications&email_token=AFJ7ZURWVFYTRUGPLUCIYO3P6YZA3A5CNFSM4GZ54NG2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODZUPAIA#issuecomment-510193696,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFJ7ZUWLVY33VBMI5SCT263P6YZA3ANCNFSM4GZ54NGQ
.


text:
Added a new file swathy-rt

As of now , I created a branch named newbie , created a file and made a commit, and opened a pull request

created a branch, created a file and made a commit, and opened a pull request

created a branch, created a file and made a commit, and opened a pull request

repo: Swathy-rt/github-slideshow


text:
[EasyCodingStandard]: add readme section for vscode integration

Thanks :+1:
Could you use standard [Package] Format? It being split to changelog a according to it

See https://github.com/Symplify/ChangelogLinker/blob/master/README.md#a-dump-merges for merge

I've just removed the ":" after package name.
Perferct, thanks :+1:

authors: TomasVotruba azdanov

repo: Symplify/Symplify


text:
聊天的历史记录没了

是会有这个问题，因为加这个插件就相当于创建了一个新的app，所以聊天记录会没掉，如果想找回的话可以先卸载，执行 ./Other/Uninstall.sh

如果能合并就好了

同上，不知道可不可以合并聊天记录呢？

@Xunqf @XYScience 已修复

修复的原理是什么啊？

@XYScience 沙盒的位置改了，所以 hook 了带有 NSSearchPathForDirectoriesInDomains 的方法，将 NSDocumentDirectory 与 NSLibraryDirectory的路径重新指向原先的路径。不愧是鹅厂，方法封装的真好。

@TKkk-iOSer 还是不可以合并聊天记录啊。而且修改后，iCloud Drive里的文稿文件夹多里一个MMappedKV的文件夹啊～～

因为目录指向了之前的路径，所以期间的聊天记录就没了，会恢复到之前微信原版的聊天记录。另外 iCloud Drive 的那个文件夹是删掉之后又出现的么？

原版是指第一次安装的微信版本吗？删掉之后是又回出现，好像会根据聊天信息变化

@XYScience 就是恢复成未安装插件的样子， iCloud Drive 我这边是一直没有出现这个文件…无论有没有修复这个问题。去 iCloud Drive设置看下是否有选中什么东西。

authors: TKkk-iOSer XYScience Xunqf

repo: TKkk-iOSer/WeChatPlugin-MacOS


text:
create pool, unable to click on any of the left pane steps (optional), typo

@julienlim @sankarshanmukhopadhyay "Advanced Configuration" tab options depend on the some of the options selected in "Basic settings" tab. For example -

Minimum Replicas is only applicable for Standard/Replicated Pools. The Pool type is selected in "Basic settings" tab.
Minimum Replica value cannot be greater than Replica value. Again the Replica is being selected in "Basic settings" tab.

Hence we have disabled direct selection of left pane steps. User can proceed only by clicking "Next" button.

@Tendrl/qe Fixed. Please verify in tendrl-dashboard-1.2.1-04_17_2017_17_24_12.noarch.rpm

Able to click on the left pane steps now. @julienlim please have look and let us know so that we can close the issue

@nthomas-redhat @gnehapk I verified that the User is able to click the left pane now.

authors: gnehapk julienlim nthomas-redhat sankarshanmukhopadhyay

repo: Tendrl/dashboard


text:
:lipstick: (web) Improve UI

:tada: This PR is included in version 1.2.2-beta.2 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.2.3-beta.1 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

repo: TomokiMiyauci/registerable


text:
feat: add support for reanimated v2

👍 v2.1.0 with experiment reanimated v2 support is released. Closing the PR cause we want to keep that version on v2 instead master branch

👍 v2.1.0 with experiment reanimated v2 support is released. Closing the PR cause we want to keep that version on v2 instead master branch


text:
Add a 'skip this step' link to the forms

👍

authors: TheUKDave mannickutd

repo: UKTradeInvestment/enav-alpha


text:
Analyze Research Findings

See #383

repo: US-EPA-CAMD/easey-ui


text:
Fixed class -> id issues

I have come around to @thongsav-usgs  way of thinking that we should be using class in our backbone code where possible. This promotes reusability.

authors: isuftin mbucknell

repo: USGS-CIDA/sparrow-ui


text:
Add documentation

Some progress: Readme

repo: UnknownHero/location-by-ip-service

license: Unlicense


text:
[WIP] Near-duplicate detection using locality-sensitive hashing

Coverage increased (+0.5%) to 59.206% when pulling 50e249fb5d579cdc58eeeb383661b327a9602c10 on aecio/minhash into b785f13c9954d119f748930fec134c497255862f on master.

authors: aecio coveralls

repo: ViDA-NYU/ache


text:
test cases for install-apps handle invalid appsDirectory values

@cthigh From what i can tell the integration tests are not running on my not configured case. Can you spot anything off in the pom?

@dnwarnock All the current changes look good. For the notconfigured case, you are missing information in the pom for creating the liberty server and running the failsafe tests. I remember they were originally there when we went through and creates the liberty-assembly test project.  For the war packaging type, you still needed the plugin configuration for the liberty-maven-plugin executions and for the failsafe plugin.  We could delete these from the liberty-assembly project because they were automatically bound. I didn't catch that you removed them from the non configured case.

@cthigh g2g?

authors: cthigh dnwarnock


text: Add User Photo to Header

repo: WildAid/o-fish-web


text:
Add "Trigger Custom Behaviors" to Keyboard Shortcuts

Great tip @erwinmaza, thanks for submitting it!

nice! 🎉

authors: erwinmaza jessesquires lordcodes


text:
test ci

recheck

recheck

recheck

recheck

recheck

recheck

recheck

recheck

recheck

recheck

recheck

recheck

recheck

recheck

recheck

recheck

recheck

recheck

recheck

recheck

recheck

recheck

recheck

recheck

recheck

recheck

recheck

recheck

recheck

recheck

recheck

repo: ZhengZhenyu/kind


text:
下载的 Demo 无法运行

pod 一下 把库导进来 就可以了


text:
to add WACV 2021

Thanks!

repo: abhshkdz/ai-deadlines


text: Deploy/build nudging

authors: David-Rodden rodenmonte

repo: academia-edu/academia-hubot-scripts


text:
Repository doesn't have the last release tags

Perhaps I misunderstood the issue, but each time this package is published on NPM, the dist-tag is updated to the latest version. The "latest" dist-tag for gulp-eslint is currently set to the most recent version (0.12.0). Are you experiencing a problem with this tag?

I don't understand what you mean with dist-tag; my issue only referred about the repository.
Repository doesn't have the last tag version https://github.com/adametry/gulp-eslint/releases
It isn't a big deal, however sometimes I check the tags to know the last versions than to go to npm site or check package.json

Totally agree with @ifraixedes. I often check the latest version number of the package by opening the /releases page. I always create a new tag whenever I publish a new version to npm, but @adametry doesn't.

My mistake. I was thinking of npm tags (e.g., gulp-eslint@latest). I've tagged the commit for the v0.21.0 package.

Cool for me, then you can close if @shinnn is happy

@adametry, not v0.21.0 but v0.12.0 :(

@adametry good catch, I'm quite sleepy so far

haha. Fixed. Thanks.

authors: adametry ifraixedes shinnn

repo: adametry/gulp-eslint


text:
Authenticator config support

Great! Thanks! 👍

authors: JanisPlots jkroepke

repo: adorsys/keycloak-config-cli


text:
Upgraded Bootstrap

Thanks! 0.4.1-SNAPSHOT is now updated

authors: adrianhurt mkurz


text:
working with oculus touch

I do agree however that auto-detect-controllers should not be needed -- have updated with presumed changes but unlike prior version, have not been able to test this set

OK - working w/Touch, if you can confirm working w/Vive, we're good to go...

@dmarcos anything else needed here?  should be able to test drive changes at https://a-blast.herokuapp.com

confirmed working on Vive as well; @dmarcos @fernandojsg should be good to merge.

@dmarcos @fernandojsg anything holding this up?

@machenmusik just a question, the aframe-master is a custom build or an standard one?

console.log('A-Frame Version: 0.4.0 (Date 14-01-2017, Commit #a1e0116)');
https://github.com/aframevr/aframe/commit/a1e0116

ok cool :D so for me it's good to go, @dmarcos anything left?

(bump) OK to go?  @dmarcos

I can't seem to get the Vive controls to show up in the latest Firefox Nightly (54.0a1).

yeah Nightly change broke things, aframe 0.5.0 has fix, supposedly Nightly will revert change

Closed by https://github.com/aframevr/a-blast/pull/111

authors: HAZARDU5 fernandojsg machenmusik ngokevin

repo: aframevr/a-blast


text:
Manipulate NeutronL3HA as needed when using --tuskar as well

Will re-submit via gerrit against rdo-management/instack-undercloud

repo: agroup/instack-undercloud


text:
Add remote uri source option

Closing this because Lottie Android intentionally have networking built in because every app has a unique networking stack and I don't want to add any transitive deps to lottie.

authors: gpeal lelandrichardson

repo: airbnb/lottie-react-native


text:
UPD google-libraries with latest libphonenumbers

This has been updated since this was initially submitted, apologize for the delay!
https://github.com/albeebe/phoneformat.js/pull/59


text:
Add iconSize and fix various inconsistencies and unexpected behavior

@alex3165 :shipit:


text:
fix: zoom text

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

authors: CLAassistant noonnightstorm

repo: aliyun/react-visual-modeling


text:
Add elixir implementation reference

Current coverage is 82.85% (diff: 100%)

Merging #7 into master will not change coverage

@@             master         #7   diff @@
==========================================
  Files             2          2
  Lines            70         70
  Methods          20         20
  Messages          0          0
  Branches          8          8
==========================================
  Hits             58         58
  Misses           12         12
  Partials          0          0

Powered by Codecov. Last update e99c4c2...d879663

authors: codecov-io merongivian

repo: alizain/ulid


text:
Remove bold on current task page link

Thank you!

authors: Davidslv deborahchua

repo: alphagov/govuk-services-prototype


text:
Laravel 7 Support Please

https://github.com/anandsiddharth/laravel-paytm-wallet#laravel-7-changes

authors: anandsiddharth fullmad

repo: anandsiddharth/laravel-paytm-wallet


text:
ACCUMULO-4154 add fate dump command

closed because I put wrong issue# in commit message

repo: apache/accumulo


text:
AMQ-6506: Increase OSGi version range for package import javax.jms

ActiveMQ dont support JMS 2.0 yet so not sure if its good to increase that osgi range

Well, the same patch has already been done on activemq-osgi bundle. (AMQ-5574)

Ah okay lets make them the same.

Thanks for the PR it has been merged. Do you mind closing this?

authors: davsclaus graben

repo: apache/activemq


text:
2.x Fix performance issues in sending messages to Pulsar

Thanks merged on 2.x

authors: oscerd rshermanTHG

repo: apache/camel


text:
GROOVY-8514: NullPointerException in class MissingMethodException

Thanks

authors: denniskniep paulk-asert

repo: apache/groovy


text:
HDFS-14761. RBF: MountTableResolver cannot invalidate cache correctly

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
42
Docker mode activated.

_ Prechecks _

+1
dupname
0
No case conflicting files found.

+1
@author
0
The patch does not contain any @author tags.

+1
test4tests
0
The patch appears to include 1 new or modified test files.

_ trunk Compile Tests _

+1
mvninstall
1065
trunk passed

+1
compile
27
trunk passed

+1
checkstyle
30
trunk passed

+1
mvnsite
34
trunk passed

+1
shadedclient
689
branch has no errors when building and testing our client artifacts.

+1
javadoc
32
trunk passed

0
spotbugs
62
Used deprecated FindBugs config; considering switching to SpotBugs.

+1
findbugs
59
trunk passed

_ Patch Compile Tests _

+1
mvninstall
26
the patch passed

+1
compile
22
the patch passed

+1
javac
22
the patch passed

+1
checkstyle
13
the patch passed

+1
mvnsite
25
the patch passed

+1
whitespace
0
The patch has no whitespace issues.

+1
shadedclient
697
patch has no errors when building and testing our client artifacts.

+1
javadoc
32
the patch passed

+1
findbugs
71
the patch passed

_ Other Tests _

-1
unit
1353
hadoop-hdfs-rbf in the patch failed.

+1
asflicense
25
The patch does not generate ASF License warnings.

4352

Reason
Tests

Failed junit tests
hadoop.hdfs.server.federation.router.TestRouterWithSecureStartup

hadoop.hdfs.server.federation.security.TestRouterHttpDelegationToken

Subsystem
Report/Notes

Docker
Client=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/hadoop-multibranch/job/PR-1334/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/1334

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle

uname
Linux 2ac5d110f440 4.4.0-138-generic #164-Ubuntu SMP Tue Oct 2 17:16:02 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
personality/hadoop.sh

git revision
trunk / ee7c261

Default Java
1.8.0_212

unit
https://builds.apache.org/job/hadoop-multibranch/job/PR-1334/1/artifact/out/patch-unit-hadoop-hdfs-project_hadoop-hdfs-rbf.txt

Test Results
https://builds.apache.org/job/hadoop-multibranch/job/PR-1334/1/testReport/

Max. process+thread count
1604 (vs. ulimit of 5500)

modules
C: hadoop-hdfs-project/hadoop-hdfs-rbf U: hadoop-hdfs-project/hadoop-hdfs-rbf

Console output
https://builds.apache.org/job/hadoop-multibranch/job/PR-1334/1/console

versions
git=2.7.4 maven=3.3.9 findbugs=3.1.0-RC1

Powered by
Apache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

repo: apache/hadoop


text:
HBASE-25079 Upgrade Bootstrap to 3.3.7

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 28s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+0 :ok:
jshint
0m  0s
jshint was not available.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ master Compile Tests _

+0 :ok:
mvndep
0m 31s
Maven dependency ordering for branch

_ Patch Compile Tests _

+0 :ok:
mvndep
0m  8s
Maven dependency ordering for patch

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

_ Other Tests _

-1 :x:
asflicense
0m 21s
The patch generated 6 ASF License warnings.

2m 41s

Subsystem
Report/Notes

Docker
Client=19.03.13 Server=19.03.13 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2442/1/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/2442

Optional Tests
dupname asflicense jshint

uname
Linux 61c2447da7f5 4.15.0-60-generic #67-Ubuntu SMP Thu Aug 22 16:55:30 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 7e910a573f

asflicense
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2442/1/artifact/yetus-general-check/output/patch-asflicense-problems.txt

Max. process+thread count
48 (vs. ulimit of 30000)

modules
C: hbase-server hbase-thrift U: .

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2442/1/console

versions
git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f)

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

Will add RAT exclusions, back soon.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 26s
Docker mode activated.

-0 :warning:
yetus
0m  4s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ master Compile Tests _

+0 :ok:
mvndep
0m 22s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
4m  8s
master passed

-0 :warning:
javadoc
0m 40s
hbase-server in master failed.

-0 :warning:
javadoc
0m 51s
hbase-thrift in master failed.

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 16s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
4m 12s
the patch passed

-0 :warning:
javadoc
0m 42s
hbase-server in the patch failed.

-0 :warning:
javadoc
0m 53s
hbase-thrift in the patch failed.

_ Other Tests _

+1 :green_heart:
unit
134m 51s
hbase-server in the patch passed.

+1 :green_heart:
unit
3m 54s
hbase-thrift in the patch passed.

153m 26s

Subsystem
Report/Notes

Docker
Client=19.03.13 Server=19.03.13 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2442/1/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/2442

Optional Tests
javac javadoc unit

uname
Linux af228a11495d 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 7e910a573f

Default Java
2020-01-14

javadoc
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2442/1/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-server.txt

javadoc
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2442/1/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-thrift.txt

javadoc
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2442/1/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-server.txt

javadoc
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2442/1/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-thrift.txt

Test Results
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2442/1/testReport/

Max. process+thread count
4366 (vs. ulimit of 30000)

modules
C: hbase-server hbase-thrift U: .

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2442/1/console

versions
git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f)

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m  8s
Docker mode activated.

-0 :warning:
yetus
0m  3s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ master Compile Tests _

+0 :ok:
mvndep
0m 27s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
4m  4s
master passed

+1 :green_heart:
javadoc
1m 13s
master passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 13s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
3m 50s
the patch passed

+1 :green_heart:
javadoc
1m 11s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
204m 44s
hbase-server in the patch passed.

+1 :green_heart:
unit
5m 23s
hbase-thrift in the patch passed.

224m 18s

Subsystem
Report/Notes

Docker
Client=19.03.13 Server=19.03.13 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2442/1/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/2442

Optional Tests
javac javadoc unit

uname
Linux d081dcecb78b 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 7e910a573f

Default Java
1.8.0_232

Test Results
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2442/1/testReport/

Max. process+thread count
3610 (vs. ulimit of 30000)

modules
C: hbase-server hbase-thrift U: .

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2442/1/console

versions
git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f)

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

@ndimiduk FYI you were asking about this on list.

Update with RAT exclusion

Update fixes whitespace in pom

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 32s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+0 :ok:
jshint
0m  0s
jshint was not available.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ master Compile Tests _

+0 :ok:
mvndep
0m 41s
Maven dependency ordering for branch

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 15s
Maven dependency ordering for patch

+1 :green_heart:
whitespace
0m  1s
The patch has no whitespace issues.

+1 :green_heart:
xml
0m  1s
The patch has no ill-formed XML file.

+1 :green_heart:
hadoopcheck
13m 23s
Patch does not cause any errors with Hadoop 3.1.2 3.2.1.

_ Other Tests _

+1 :green_heart:
asflicense
0m 47s
The patch does not generate ASF License warnings.

24m  5s

Subsystem
Report/Notes

Docker
Client=19.03.13 Server=19.03.13 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2442/2/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/2442

Optional Tests
dupname asflicense jshint hadoopcheck xml

uname
Linux b7958dc8aa33 4.15.0-60-generic #67-Ubuntu SMP Thu Aug 22 16:55:30 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / dca0b593cf

Max. process+thread count
76 (vs. ulimit of 30000)

modules
C: . hbase-server hbase-thrift U: .

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2442/2/console

versions
git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f)

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 30s
Docker mode activated.

-0 :warning:
yetus
0m  4s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ master Compile Tests _

+0 :ok:
mvndep
0m 37s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
4m 12s
master passed

+1 :green_heart:
compile
2m 47s
master passed

+1 :green_heart:
shadedjars
6m 38s
branch has no errors when building our shaded downstream artifacts.

-0 :warning:
javadoc
0m 17s
root in master failed.

-0 :warning:
javadoc
0m 39s
hbase-server in master failed.

-0 :warning:
javadoc
0m 48s
hbase-thrift in master failed.

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 19s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
3m 57s
the patch passed

+1 :green_heart:
compile
2m 49s
the patch passed

+1 :green_heart:
javac
2m 49s
the patch passed

+1 :green_heart:
shadedjars
6m 36s
patch has no errors when building our shaded downstream artifacts.

-0 :warning:
javadoc
0m 39s
hbase-server in the patch failed.

-0 :warning:
javadoc
0m 49s
hbase-thrift in the patch failed.

-0 :warning:
javadoc
0m 16s
root in the patch failed.

_ Other Tests _

+1 :green_heart:
unit
167m 43s
root in the patch passed.

203m 19s

Subsystem
Report/Notes

Docker
Client=19.03.13 Server=19.03.13 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2442/2/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/2442

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 9f6bec2974be 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / dca0b593cf

Default Java
2020-01-14

javadoc
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2442/2/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-root.txt

javadoc
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2442/2/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-server.txt

javadoc
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2442/2/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-thrift.txt

javadoc
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2442/2/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-server.txt

javadoc
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2442/2/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-thrift.txt

javadoc
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2442/2/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-root.txt

Test Results
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2442/2/testReport/

Max. process+thread count
6864 (vs. ulimit of 30000)

modules
C: hbase-server hbase-thrift . U: .

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2442/2/console

versions
git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f)

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m 10s
Docker mode activated.

-0 :warning:
yetus
0m  3s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ master Compile Tests _

+0 :ok:
mvndep
0m 36s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
4m  1s
master passed

+1 :green_heart:
compile
2m 32s
master passed

+1 :green_heart:
shadedjars
7m  6s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
3m 36s
master passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 17s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
3m 55s
the patch passed

+1 :green_heart:
compile
2m 47s
the patch passed

+1 :green_heart:
javac
2m 47s
the patch passed

+1 :green_heart:
shadedjars
7m 31s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
3m 36s
the patch passed

_ Other Tests _

-1 :x:
unit
338m 12s
root in the patch failed.

377m 35s

Subsystem
Report/Notes

Docker
Client=19.03.13 Server=19.03.13 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2442/2/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/2442

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 4af601f6917e 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / dca0b593cf

Default Java
1.8.0_232

unit
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2442/2/artifact/yetus-jdk8-hadoop3-check/output/patch-unit-root.txt

Test Results
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2442/2/testReport/

Max. process+thread count
2642 (vs. ulimit of 30000)

modules
C: hbase-server hbase-thrift . U: .

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2442/2/console

versions
git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f)

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

Thank you @apurtell !

authors: Apache-HBase apurtell ndimiduk

repo: apache/hbase


text:
Ignite 481

IGNITE-481
Closed

authors: dspavlov ghost

repo: apache/ignite


text:
IGNITE-12433 checking column names for duplicates on create table statement

New TC run all (in progress):
https://ci.ignite.apache.org/buildConfiguration/IgniteTests24Java8_RunAll/5050660?mode=branches


text:
[KARAF-6953] Introduce autoRefresh property in etc/org.apache.karaf.features.cfg

Thanks!

Thanks!

I handle the deploy phase, I will do the same for the undeploy phase.

Nice one!  Looking forward to this.

authors: ahgittin jbonofre k-brooks

repo: apache/karaf


text:
KYLIN-3867 Enable JDBC to use key store & trust store for https connection

Can one of the admins verify this patch?

Pull Request Test Coverage Report for Build 4224

1 of 22 (4.55%)  changed or added relevant lines in 1 file are covered.
7 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.009%) to 26.934%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

jdbc/src/main/java/org/apache/kylin/jdbc/KylinClient.java
1
22
4.55%

Files with Coverage Reduction
New Missed Lines
%

core-cube/src/main/java/org/apache/kylin/cube/cuboid/TreeCuboidScheduler.java
2
68.46%

core-cube/src/main/java/org/apache/kylin/cube/inmemcubing/MemDiskStore.java
5
77.81%

Totals

Change from base Build 4216:
-0.009%

Covered Lines:
18955

Relevant Lines:
70375

💛  - Coveralls

authors: asfgit coveralls kyotoYaho

repo: apache/kylin


text:
PHOENIX-5259 Phoenix Salted Table Secondary Indexes Come Salted

Since the index not longer follows the salted base table, this might change the behavior of the index table. @gjacoby126 @ChinmaySKulkarni @kadirozde please take a look if this is expected behavior to remove this inheritance, thanks.
cc @dbwong

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m  8s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

-1 :x:
test4tests
0m  0s
The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ master Compile Tests _

+1 :green_heart:
mvninstall
15m 30s
master passed

+0
hbaserecompile
22m 38s
HBase recompiled.

+1 :green_heart:
compile
1m  2s
master passed

+1 :green_heart:
checkstyle
1m 42s
master passed

+1 :green_heart:
javadoc
0m 51s
master passed

+0 :ok:
spotbugs
3m  3s
phoenix-core in master has 959 extant spotbugs warnings.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
8m 10s
the patch passed

+0
hbaserecompile
18m 27s
HBase recompiled.

+1 :green_heart:
compile
1m  0s
the patch passed

+1 :green_heart:
javac
1m  0s
the patch passed

+1 :green_heart:
checkstyle
1m 46s
the patch passed

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
javadoc
0m 51s
the patch passed

+1 :green_heart:
spotbugs
3m 19s
the patch passed

_ Other Tests _

-1 :x:
unit
108m 47s
phoenix-core in the patch failed.

-1 :x:
asflicense
0m 45s
The patch generated 1 ASF License warnings.

165m 26s

Reason
Tests

Failed junit tests
phoenix.end2end.DeleteIT

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-589/1/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/phoenix/pull/589

Optional Tests
dupname asflicense javac javadoc unit spotbugs hbaserebuild hbaseanti checkstyle compile

uname
Linux 29b6fe693cd4 4.15.0-65-generic #74-Ubuntu SMP Tue Sep 17 17:06:04 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev/phoenix-personality.sh

git revision
master / 6ed3caf

Default Java
Private Build-1.8.0_242-8u242-b08-0ubuntu3~16.04-b08

unit
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-589/1/artifact/yetus-general-check/output/patch-unit-phoenix-core.txt

Test Results
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-589/1/testReport/

asflicense
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-589/1/artifact/yetus-general-check/output/patch-asflicense-problems.txt

Max. process+thread count
11060 (vs. ulimit of 30000)

modules
C: phoenix-core U: phoenix-core

Console output
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-589/1/console

versions
git=2.7.4 maven=3.3.9 spotbugs=4.1.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

authors: stoty yanxinyi

repo: apache/phoenix


text:
Fixed invalid annotations in docs.

@Zrnik In version ^0.7 (current development version) has been @ControllerPath annotation replaced by @Path annotation. The documentation for the stable version (0.6.0) you can be found here.

Yes, i got it like 10 minutes after i sent the PR.

maybe this will do it

repo: apitte/core


text:
Duplicate Syn and SynAck Bug Fix

Codecov Report

Merging #288 into master will decrease coverage by 0.07%.
The diff coverage is 50%.

@@            Coverage Diff             @@
##           master     #288      +/-   ##
==========================================
- Coverage   52.95%   52.87%   -0.08%
==========================================
  Files          40       40
  Lines        4406     4422      +16
==========================================
+ Hits         2333     2338       +5
- Misses       1878     1885       +7
- Partials      195      199       +4

Impacted Files
Coverage Δ

enforcer/datapath.go
68.96% <ø> (ø)
:arrow_up:

enforcer/connection.go
38.18% <100%> (+2.33%)
:arrow_up:

enforcer/datapath_tcp.go
61.57% <45.83%> (-1.23%)
:arrow_down:

Continue to review full report at Codecov.

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

Can we add a description with the bug we are trying to fix with this PR?

authors: codecov-io dstiliadis sibicramesh

repo: aporeto-inc/trireme


text:
CanImport: allow using string literal to indicate desired version

@swift-ci please smoke test

repo: apple/swift


text:
[test] Fix a couple of RUN directives

@swift-ci please smoke test

@swift-ci please smoke test macOS

@swift-ci please smoke test macOS

@swift-ci please smoke test macOS

@swift-ci please smoke test macOS

@swift-ci please smoke test macOS

@swift-ci please smoke test macOS

@swift-ci please smoke test macOS

@swift-ci please smoke test macOS

@swift-ci please smoke test macOS

@swift-ci please smoke test


text:
[SourceKit] Add a new request to translate markup text to its XML equivalence. rdar://30587403

@swift-ci please smoke test

@bitjammer mind taking a look at this patch?

@swift-ci please smoke test

This failed ASAN testing. I'm reverting in #8088. Please do not commit again without running ASAN first.

Build log here: https://ci.swift.org/job/oss-swift-incremental-ASAN-RA-osx/177/testReport/junit/Swift(macosx-x86_64)/SourceKit_ExtractComment/extract_comments_swift/

@nkcsgexi, LineList only has an ArrayRef of Lines, and Line only has a StringRef. You might want to pass an MC & into getLineListFromComment perhaps and hold onto it in the stack frame above.

oh, that makes sense.

authors: bitjammer gparker42 nkcsgexi


text:
Add test scenes for the standard surface shader

Hi!
Just to clarify. By "standard surface shader" is meant renderer/modeling/surfaceshader/physicalsurfaceshader.cpp?

I believe this ticket is about testing the "appleseed Standard Surface" OSL shader whose code can be found in src/appleseed.shaders/src/appleseed/as_standard_surface.osl.
The idea would be to add new test scenes in the test suite located in sandbox/tests/test scenes/.

authors: dictoon est77 lorentzo

repo: appleseedhq/appleseed


text:
fix typo

Thanks @zabio3 !

repo: appscode/guard


text:
Added navbar to different pages.

Good work, Thanks! @DeViL3998

repo: aquibbaig/club-website


text:
next attempt...

http://jenkins01.arangodb.biz:8080/view/PR/job/arangodb-matrix-pr/1248/

repo: arangodb/arangodb


text:
Ability to disable navigation menu's mobile mode

Obsolete since the menu now adapts according to the space it can get to display navigation items.

repo: archost/alicht


text:
Added user_feed.

Thanks much

authors: dblock dzucconi

repo: aredotna/arena-rb


text:
Add non-x86 support

@jameshloving It looks like this is failing the Travis tests with a compile error. Could you please resolve that for the PR?

Hi Armon,
Looking more into the error, it doesn't seem to be anything on my end:
https://travis-ci.org/armon/bloomd/builds/184398925 shows an error with pip
install, not the actual compilation. Please let me know if I'm wrong.
Thanks and happy New Year!
James Loving
jloving@mit.edu | jameshloving@gmail.com
On Tue, Jan 3, 2017 at 1:02 PM, Armon Dadgar notifications@github.com
wrote:

@jameshloving https://github.com/jameshloving It looks like this is
failing the Travis tests with a compile error. Could you please resolve
that for the PR?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/armon/libart/pull/29#issuecomment-270179568, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ARcrQPnnqx9i9el6c7g613zxePuBrCLwks5rOo1MgaJpZM4LOufy
.

@jameshloving Happy New Years to you as well! I fixed the pip issue with bloomd, but it looks like there is a double definition of cmp : https://travis-ci.org/armon/libart/jobs/184411772

Should be fixed!

Thanks!

authors: armon jameshloving

repo: armon/libart


text:
Update Moya.podspec for new major Alamofire version

Hey, awesome! Thanks :) The CI is failing because Travis. Gonna merge!

authors: ashfurrow vandyshev

repo: ashfurrow/Moya


text:
Update of Identity

@ardalis  I did a quick edit so double check my changes.

repo: aspnet/Docs


text:
Prototype shared AuthorizationPolicyEvaluator

@HaoK,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by .NET Foundation. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

@HaoK did you ever file an issue for this to do as part of 2.0?

Its on my todo list as part of AuthZ 2.0, which should happen in preview 2

https://github.com/aspnet/Security/issues/901 is the main one for AuthZ

Meta tracking issue for Auth 2.0 todo list for preview 2 https://github.com/aspnet/Security/issues/1190

Moved into https://github.com/aspnet/Security/pull/1193

authors: HaoK davidfowl dnfclas

repo: aspnet/Security


text:
$validators stadard and bank account solved

To merge this PR you need to adapt your code to the current codebase.
I suggest you to make one PR for each directive you want to include in the library.
Check the us module.

Any progress?

authors: assisrafael jonsonjonson

repo: assisrafael/angular-input-masks


text:
Store failed messages in Cassandra.

:+1:

authors: tfm wwajerowicz

repo: atlasapi/atlas-deer


text:
Fixed typo in ScopedResourceUpdater

Thanks!

authors: GeertvanHorrik zcbenz

repo: atom/rcedit


text:
EpicManager instances to scope chains

https://github.com/avcs06/EpicManager/commit/b1713b7a57b0f4d19dc53c957b38e2724c31d2b5


text:
nullsafety and rename unite to fold

I published a prerelease. Thank

I published a prerelease. Thank

Hi 👋. I published a stable release with your code. In the new version, I added the extension FutureEither for a more compact code.


text:
increment jackson version

Codecov Report

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

@@             Coverage Diff              @@
##             master      #62      +/-   ##
============================================
- Coverage     54.96%   54.92%   -0.04%
+ Complexity      559      557       -2
============================================
  Files           106      106
  Lines          2540     2540
  Branches        139      139
============================================
- Hits           1396     1395       -1
  Misses         1063     1063
- Partials         81       82       +1

Impacted Files
Coverage Δ
Complexity Δ

...a/com/axibase/tsd/client/PlainStreamingSender.java
44.11% <0%> (-0.74%)
15% <0%> (-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 f7c9e7e...78ed350. Read the comment docs.

authors: codecov-io unrealwork

repo: axibase/atsd-api-java


text:
changes in support of Java 8

Issue came from the old issue tracker https://code.google.com/p/reflectutils/issues/detail?id=19&can=1

Thanks Beth :-)

This is compiling for me, but there's 5 test failures after this. A few of them make sense, but a few of them don't. There is 1 failure in Java 7. Looking at these.

authors: azeckoski bkirschn jonespm

repo: azeckoski/reflectutils


text:
One-liner for initialsFromString function

Requested changes done! Thanks

repo: bachonk/InitialsImageView


text:
WildFly SwarmでJDBCデータソースを扱う

もう全然WildFly Swarm触ってない( ˙꒳˙ )


text:
fix asciiDoc links

Messages

:book:
:sparkles: Thanks for your contribution to Shields, @chris48s!

  Generated by :no_entry_sign: dangerJS against 065022901f0590346dab71818bfaa20b0f2020d5

repo: badges/shields


text:
Feature Request: 全屏时可以调整JSON text 框大小

非常需要这个，希望作者大大帮忙做下！
非常感谢作者大大提供这么好用的扩展

界面中输入暂时无法手动调整 新版已经将输入框调整到最大了

@baiy 感谢老哥！

authors: baiy liuhao2050 oknixus

repo: baiy/Ctool


text:
Support reading ADC values.

@Bucknalla we have users asking for this in JF already!

@phil-d-wilson alright 🙂 I'll take a look in the morning. As far as I can remember this is in place but just not exposed.

Did this turn out to be in place already?

Sorry @crbn60 - I missed your question somehow. No, this is not in place yet. I'm hoping if enough people badger @Bucknalla it can be done soon, though. :)

authors: Bucknalla crbn60 phil-d-wilson

repo: balenablocks/finabler


text:
Add Language server to build pipe line

Closing, since this has been added to the pipe line

repo: ballerinalang/language-server


text:
Variable ordering not consistent with list entered order

Fixed by fa763ad96a7c55bcca527bd20537520d082ae164

Fixed in f7b6e6dcf44196cdd7af0527ef2247c90ad25d91

repo: bbdaniels/betterbar


text:
Empty commit to trigger a new build

bors r+


text:
Hitting "submit" on monitoring area name leads to plain white screen I cannot back out of -- can't test further

@LisaMoore1 Is it alright if we close this in favour of #150? They appear to be referencing the same bug.

For sure. I made sure they both have same user stories for context. Go ahead and close it.
Thanks for working on these by the way. :)
Lisa Moore, P.Ag.
Rangeland Policy Specialist
Range Branch - FLNRORD
iPad
On Oct 8, 2019, at 3:33 PM, Caleb Sharp notifications@github.com wrote:

@LisaMoore1https://github.com/LisaMoore1 Is it alright if we close this in favour of #150https://github.com/bcgov/range-web/issues/150? They appear to be referencing the same bug.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/bcgov/range-web/issues/179?email_source=notifications&email_token=AI632E5MEVZZJFK4HYR3T4TQNUDCTA5CNFSM4I5JKSWKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEAV2OWI#issuecomment-539731801, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AI632EZA4ZCQKHF7PRCSOPDQNUDCTANCNFSM4I5JKSWA.

Sounds good, thanks.
Thanks for putting together all the issues to work on!

Closing in favour of #150.

repo: bcgov/range-web


text:
remove unused inner function

Thanks.

authors: benjaminp graingert

repo: benjaminp/six


text:
Pull Latest

 TCMU-RUNNER Optical Media Change

 TCMU-RUNNER Optical Media Change


text:
cleanup devtools/README.md of tools that have been removed over the years

Thanks, love improve-by-removal kind of PRs

authors: gubatron zander

repo: bitcoinclassic/bitcoinclassic


text:
Can't supply both endpoint and region

all done!

repo: bizreach/aws-kinesis-scala


text:
2 small typos fixed

thanks!

authors: alfareyouthere letubert

repo: bloomberg/git-adventure-game-builder


text:
Add ukrainian and russian translations

Wow, awesome! thanks!
merge #33

authors: YuliyaVeklicheva bluzi

repo: bluzi/name-db


text:
Minor proposal (backtick issue) (line 07)

@kant thanks for all the fixes

authors: MarcoTodescato kant


text:
Added fix for NPE for objects with methods 'get' or 'is'

@cliedeman Wow! Thanks for the prompt fix!

authors: cliedeman fdlk

repo: bpatters/schemagen-graphql


text:
Is there any support for Stellar

please let us know if there is support for Stellar in future

We will announce support for new chains through our social media channels or mailing list. See https://brd.com/ for more info.

We will announce support for new chains through our social media channels or mailing list. See https://brd.com/ for more info.

authors: ewerx expertdev-designer

repo: breadwallet/breadwallet-ios


text:
Create release

Closing, moved tickets to #945 so the epic is in gnomad_production


text:
Moving libraries that are only for development

:+1:

Sorry for the delay. thanks @abdala

authors: abdala brtriver sasezaki

repo: brtriver/dbup


text:
Updated dependencies to avoid security vulnerabilities

There are no longer vulnerabilities in dependencies

repo: btcven/locha-mesh-chat


text:
Blog post: About roi.

posted, tweeted: https://helio-frota.github.io/post/roi/

repo: bucharest-gold/entente


text:
Fix Style/AlignHash

:+1:

@homu r+

:pushpin: Commit db3d4bb has been approved by segiddins

:hourglass: Testing commit db3d4bb with merge 301bec2...

:sunny: Test successful - status

authors: esasse homu segiddins

repo: bundler/bundler


text:
default event + demo

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

repo: buz-zard/redux-cached-api-middleware


text:
Finish the topic based event bus

@netopyr can we close this issue?

authors: gemaSantiago hendrikebbers

repo: canoo/dolphin-platform


text:
feature: passwords.password not needed for Laravel 6+

Thank you

authors: andrey-helldar caouecs

repo: caouecs/Laravel-lang


text:
check type on modulo

Nice! In this case is type coercion based on the operator, or does it try to match the other value in the equation?
I'm used to JS, where type operator is based upon the type of the other value.

Oh yea, I forgot to mention. Maki actually throws a compiler error and enforces that % is called with an integer for n in a % n.
So we can always assume the other type value is an integer and just coerce to that.

authors: captbaritone jberg

repo: captbaritone/webamp


text:
COOK-13 Include cdap-cli

:+1:


text:
New Cask TCPBlock 4.2

Did not realize #8587 . TCPBlock is still available via MacUpdate. Would you consider adding it back?

We generally avoid software without an extant vendor. For the purpose of maintenance, it is undesirable to trust a package of which homepage and downloads are both provided by a third-party.

Agreed.

authors: hanxue ndr-qef vitorgalvao

repo: caskroom/homebrew-cask


text:
Fixes for segmentation to work with Catalyst

Thank you for the submit!

authors: lightforever megachester

repo: catalyst-team/mlcomp


text:
Mnist demo loss is not converging as fast as convnetjs implementation

Random values in volume initialisation were uniform rather than gaussian. 9cdbbf9

repo: cbovar/ConvNetSharp


text:
glitches [fix]

Fixes #142 and #117 glitches


text:
fix(tests): add tests to npm ignore

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


text:
Define latency buckets boundaries in the API to allow UIs to use them when display data.

Codecov Report

Merging #333 into master will decrease coverage by 0.68%.
The diff coverage is 0%.

@@             Coverage Diff              @@
##             master     #333      +/-   ##
============================================
- Coverage     89.26%   88.57%   -0.69%
  Complexity      509      509
============================================
  Files            81       81
  Lines          2059     2075      +16
  Branches        167      162       -5
============================================
  Hits           1838     1838
- Misses          184      200      +16
  Partials         37       37

Impacted Files
Coverage Δ
Complexity Δ

...om/google/instrumentation/trace/TraceExporter.java
17.64% <0%> (-5.43%)
3 <0> (ø)

Continue to review full report at Codecov.

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

authors: bogdandrutu codecov-io


text:
Add metric and gauge API

@bogdandrutu  Why do you want a separate PR for the metrics package? I think it only makes sense to have it together so you can see an example of how its used - otherwise it is very abstract. Do you mean that you haven't reviewed the metrics package?

@ramonza because for that file I have a lot of comments and I don't want to pollute this PR with more discussions since it is already too large and complicated.

@bogdandrutu updated

authors: bogdandrutu ramonza

repo: census-instrumentation/opencensus-go


text:
Update shadow plugin to 2.0.2. Fixes #825.

Coverage remained the same at 81.453% when pulling a031605197bc44dc3514f047749e4b595e601b94 on ubschmidt2:shadow into 5d772db096ecf4a596e0bfd93dce072d5cfc0e80 on census-instrumentation:master.

Codecov Report

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

@@           Coverage Diff            @@
##             master    #911   +/-   ##
========================================
  Coverage      79.3%   79.3%
  Complexity      968     968
========================================
  Files           161     161
  Lines          4335    4335
  Branches        408     408
========================================
  Hits           3438    3438
  Misses          806     806
  Partials         91      91

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 5d772db...a031605. Read the comment docs.

authors: codecov-io coveralls ubschmidt2

repo: census-instrumentation/opencensus-java


text:
Add counter on UDP receive errors

:+1:

authors: cespare theanine

repo: cespare/gost


text:
22326, Take Two

Okay! Let's do this ...

Like ... doesn't the test suite exercise gccgo against the standard library? Why doesn't that fail? More research required ...

It does. I suspect there's a bug in this CL if gccgo (with the same behavior you're implementing) passes the stdlib tests fine.
https://go-review.googlesource.com/c/go/+/156328/1#message-43a5ed07c4e3c86c72ceeacce227c3ee59b9b587

Let's make sure that returning out instead of lout is the thing that hoses the stdlib.

https://gist.github.com/chadwhitacre/96d88209940bb678328e7491d759105b
$ time (./all.bash -v > out 2>&1); say 'yeah'

real    16m27.882s
user    30m47.889s
sys     8m27.166s
$

diff --git a/src/cmd/compile/internal/gc/sinit.go b/src/cmd/compile/internal/gc/sinit.go
index a5737ad..2a0ff21 100644
--- a/src/cmd/compile/internal/gc/sinit.go
+++ b/src/cmd/compile/internal/gc/sinit.go
@@ -444,7 +444,7 @@ func initfix(l []*Node) []*Node {
 		}
 	}

-	return out
+	return lout
 }

 // compilation of top-level (static) assignments

916c2d8

It's not! There are multiple places where I didn't update &foo.Linkfoo to foo.MakeLinkfoo(). I've made another whack at those and am rerunning ...

Clean with lout in 98284ab!
$ tail -f src/out |grep FAIL
FAIL    fixedbugs/issue22326.go 1.455s
2019/01/05 15:31:39 FAILED

$ time (./all.bash -v > out 2>&1); say 'yeah'

real    30m24.490s
user    59m15.327s
sys     20m47.803s
$

https://gist.github.com/chadwhitacre/3c01d78c29ed9fa7d091e466020d0761

Now to go back to out and see if it's totally all clean! 🙈

Dangit! 😕
$ time (./all.bash -v > out 2>&1); say 'yeah'

real    16m28.586s
user    30m54.471s
sys     8m38.126s
$

https://gist.github.com/chadwhitacre/04ecb8e40fde3449951cb971669a5309
diff --git a/src/cmd/compile/internal/gc/sinit.go b/src/cmd/compile/internal/gc/sinit.go
index 2a0ff21..a5737ad 100644
--- a/src/cmd/compile/internal/gc/sinit.go
+++ b/src/cmd/compile/internal/gc/sinit.go
@@ -444,7 +444,7 @@ func initfix(l []*Node) []*Node {
 		}
 	}

-	return lout
+	return out
 }

 // compilation of top-level (static) assignments

🤔
$ ../../../bin/go run ../../../test/fixedbugs/issue22326.go
$ ../../../bin/go test
PASS
ok      archive/tar     0.046s
$

Sorry, there we go ...
/Users/whit537/workbench/go/src/github.com/golang/go/src/archive
total 0
drwxr-xr-x  16 whit537  wheel  512 Jan  3 16:06 tar/
drwxr-xr-x  11 whit537  wheel  352 Nov 25 08:12 zip/
$  cd zip/
$ ../../../bin/go test
--- FAIL: TestReader (0.03s)
    --- FAIL: TestReader/readme.notzip (0.00s)
        reader_test.go:507: error=zip: not a valid zip file, want <nil>
    --- FAIL: TestReader/Bad-CRC32-in-data-descriptor (0.00s)
        reader_test.go:600: copying contents: zip: checksum error (want <nil>)
    --- FAIL: TestReader/crc32-not-streamed.zip#01 (0.00s)
        reader_test.go:600: copying contents: zip: checksum error (want <nil>)

Alright yeah the ZIP reader test seem to relate to variables defined at the top level, which makes me think that the MakeLink workaround is also going to need to be undone.
https://github.com/chadwhitacre/go/blob/98284ab3b6eac25692198ee257e8fa6e8f82362e/src/archive/zip/reader_test.go#L52-L482

678f9c1f890a67e688a5f3ad69b63123ac61324d 👍
/Users/whit537/workbench/go/src/github.com/golang/go/src/archive/borked
total 84
-rw-r--r--   1 whit537  wheel  17838 Jan  5 16:07 reader.go
-rw-r--r--   1 whit537  wheel  31565 Jan  5 16:07 reader_test.go
-rw-r--r--   1 whit537  wheel   3754 Jan  5 16:07 register.go
-rw-r--r--   1 whit537  wheel  11513 Jan  5 16:07 struct.go
drwxr-xr-x  29 whit537  wheel    928 Jan  5 16:07 testdata/
-rw-r--r--   1 whit537  wheel  16230 Jan  5 16:07 writer.go
$ ../../../bin/go test
--- FAIL: TestReader (0.02s)
    --- FAIL: TestReader/readme.notzip (0.00s)
        reader_test.go:507: error=zip: not a valid zip file, want <nil>
    --- FAIL: TestReader/Bad-CRC32-in-data-descriptor (0.00s)
        reader_test.go:600: copying contents: zip: checksum error (want <nil>)
    --- FAIL: TestReader/crc32-not-streamed.zip#01 (0.00s)
        reader_test.go:600: copying contents: zip: checksum error (want <nil>)
FAIL
exit status 1
FAIL    archive/borked  0.043s
$

b96e86849b20940fe05e16123b930eae5db73387 👍
$ ../../../bin/go test
--- FAIL: TestReader (0.02s)
    --- FAIL: TestReader/readme.notzip (0.00s)
        reader_test.go:507: error=zip: not a valid zip file, want <nil>
    --- FAIL: TestReader/Bad-CRC32-in-data-descriptor (0.00s)
        reader_test.go:600: copying contents: zip: checksum error (want <nil>)
    --- FAIL: TestReader/crc32-not-streamed.zip#01 (0.00s)
        reader_test.go:600: copying contents: zip: checksum error (want <nil>)
FAIL
exit status 1
FAIL    archive/borked  0.038s
$

Scratch that ... 66efb757ef6e5afc62bad0b372586e552898cfdb 👍
$ ../../../bin/go test
--- FAIL: TestReader (0.02s)
    --- FAIL: TestReader/readme.notzip (0.00s)
        reader_test.go:507: error=zip: not a valid zip file, want <nil>
    --- FAIL: TestReader/Bad-CRC32-in-data-descriptor (0.00s)
        reader_test.go:600: copying contents: zip: checksum error (want <nil>)
    --- FAIL: TestReader/crc32-not-streamed.zip#01 (0.00s)
        reader_test.go:600: copying contents: zip: checksum error (want <nil>)
FAIL
exit status 1
FAIL    archive/zorp    0.039s
$

bbe5417109b3ca5ad47815883453f61d310eec34 👍
$ ../../../bin/go test
--- FAIL: TestReader (0.00s)
    --- FAIL: TestReader/readme.notzip (0.00s)
        reader_test.go:82: error=zip: not a valid zip file, want <nil>
FAIL
exit status 1
FAIL    archive/zorp    0.014s
$

Alright, so ErrFormat is coming in as nil when it should be ... something else?
https://github.com/chadwhitacre/go/blob/bbe5417109b3ca5ad47815883453f61d310eec34/src/archive/zorp/reader_test.go#L52-L57

90b949f796d2d76fed49dd9a2851756c57b37aef 👍
$ ../../../bin/go test
--- FAIL: TestReader (0.00s)
    --- FAIL: TestReader/readme.notzip (0.00s)
        reader_test.go:75: error=zip: not a valid zip file, want <nil>
FAIL
exit status 1
FAIL    archive/zorp    0.015s
$

bf1c4e737d76ca13fe9073471c65cdfb7b18b7a8 👍
/Users/whit537/workbench/go/src/github.com/golang/go/src/archive/foo
total 8
-rw-r--r--  1 whit537  wheel  147 Jan  5 16:39 foo.go
-rw-r--r--  1 whit537  wheel  486 Jan  5 16:40 foo_test.go
$ ../../../bin/go test
--- FAIL: TestFoo (0.00s)
    --- FAIL: TestFoo/blah (0.00s)
        foo_test.go:31: error=blah blah GINGER blah blah, want <nil>
FAIL
exit status 1
FAIL    archive/foo     0.013s
$

e117192
$ ../../../bin/go test
--- FAIL: TestFoo (0.00s)
    foo_test.go:24: error=blah blah GINGER blah blah, want <nil>
FAIL
exit status 1
FAIL    archive/foo     0.014s
$

f340da91a2cd8ba840d8a6cc71dbb747ec32f35d 👍
$ ../../../bin/go test
--- FAIL: TestFoo (0.00s)
    foo_test.go:14: error=blah blah GINGER blah blah, want <nil>
FAIL
exit status 1
FAIL    archive/foo     0.013s
$

e82669f688fe2d9d1feff15b69db1d0cfe0d423b
$ time (./make.bash -v > out 2>&1); say 'yeah'

real    4m5.863s
user    8m35.652s
sys     1m15.742s
$

$ ../../../bin/go test
# archive/foo [archive/foo.test]
lout:
        ErrFoo = errors.New("blah blah GINGER blah blah")
        statictmp_0 = ErrFoo
out:
        statictmp_0 = ErrFoo
        ErrFoo = errors.New("blah blah GINGER blah blah")
# archive/foo.test
lout:
out:
--- FAIL: TestFoo (0.00s)
    foo_test.go:14: error=blah blah GINGER blah blah, want <nil>
FAIL
exit status 1
FAIL    archive/foo     0.018s
$

Alright so front-loading static assignments is bad?

Static assignments aren't in deps. Guess they'll need to be added.
bd69778f7249a06ee6d39b553dcf9ce2facb2afd
$ ../../../bin/go test
# archive/foo [archive/foo.test]
deps:
        ErrFoo: []
        tests: [ErrFoo]
lout:
        ErrFoo = errors.New("blah blah GINGER blah blah")
        statictmp_0 = ErrFoo
out:
        ErrFoo = errors.New("blah blah GINGER blah blah")
        tests = []error literal
# archive/foo.test
deps:
        tests: []
        benchmarks: []
        examples: []
lout:
out:
PASS
ok      archive/foo     0.023s
$

610e660aae47a5c3a26c692a1cd00c2669411860
$ time (./make.bash -v > out 2>&1); say 'yeah'

real    4m36.160s
user    8m34.584s
sys     1m20.656s
$

$ ../../../bin/go test
# archive/foo [archive/foo.test]
deps:
        ErrFoo: []
        tests: [ErrFoo]
lout:
        ErrFoo = errors.New("blah blah GINGER blah blah")
        statictmp_0 = ErrFoo
out:
        statictmp_0 = ErrFoo
        ErrFoo = errors.New("blah blah GINGER blah blah")
        tests = []error literal
# archive/foo.test
deps:
        tests: []
        benchmarks: []
        examples: []
lout:
out:
PASS
ok      archive/foo     0.013s
$

Alright, so there are two discrepancies:

out has tests while lout does not
out sorts statictmp_0 before ErrFoo

8e2c3dc97d9dd686fa56dfbfda5ba74f8a4f4c49
$ ../../../bin/go test
# archive/foo [archive/foo.test]
l:
        ErrFoo = errors.New("blah blah GINGER blah blah")
        <node DCLFUNC>
        tests = []error literal
        <node DCLFUNC>
deps:
        ErrFoo: []
        tests: [ErrFoo]
lout:
        ErrFoo = errors.New("blah blah GINGER blah blah")
        statictmp_0 = ErrFoo
out:
        statictmp_0 = ErrFoo
        ErrFoo = errors.New("blah blah GINGER blah blah")
        tests = []error literal
# archive/foo.test
l:
        tests = []testing.InternalTest literal
        benchmarks = []testing.InternalBenchmark literal
        examples = []testing.InternalExample literal
        <node DCLFUNC>
        <node DCLFUNC>
deps:
        tests: []
        benchmarks: []
        examples: []
lout:
out:
PASS
ok      archive/foo     0.014s
$

213bba32b57351ca8b63166e8fdb535d20c29c3c
$ ../../../bin/go test
# archive/foo [archive/foo.test]
l:
        ErrFoo = errors.New("blah blah GINGER blah blah")
        <node DCLFUNC>
        tests = []error literal
        <node DCLFUNC>
staticassign'ing ErrFoo errors.New("blah blah GINGER blah blah") CALLFUNC
staticassign'ing tests []error literal SLICELIT
staticassign'ing statictmp_0 ErrFoo NAME
deps:
        ErrFoo: []
        tests: [ErrFoo]
lout:
        ErrFoo = errors.New("blah blah GINGER blah blah")
        statictmp_0 = ErrFoo
out:
        statictmp_0 = ErrFoo
        ErrFoo = errors.New("blah blah GINGER blah blah")
        tests = []error literal
# archive/foo.test
l:
        tests = []testing.InternalTest literal
        benchmarks = []testing.InternalBenchmark literal
        examples = []testing.InternalExample literal
        <node DCLFUNC>
        <node DCLFUNC>
staticassign'ing tests []testing.InternalTest literal SLICELIT
staticassign'ing statictmp_0 foo.TestFoo NAME
staticassign'ing benchmarks []testing.InternalBenchmark literal SLICELIT
staticassign'ing examples []testing.InternalExample literal SLICELIT
deps:
        tests: []
        benchmarks: []
        examples: []
lout:
out:
PASS
ok      archive/foo     0.012s
$

550acbbd29eaf6fc5a45ca75612aef3c47336c63
$ ../../../bin/go test
# archive/foo [archive/foo.test]
l:
        ErrFoo = errors.New("blah blah GINGER blah blah")
        <node DCLFUNC>
        tests = []error literal
        <node DCLFUNC>
staticinit'ing ErrFoo errors.New("blah blah GINGER blah blah")
staticassign'ing ErrFoo errors.New("blah blah GINGER blah blah") CALLFUNC
staticinit'ing ErrFoo errors.New("blah blah GINGER blah blah") false
staticinit'ing tests []error literal
staticassign'ing tests []error literal SLICELIT
Here we are: statictmp_0
staticassign'ing statictmp_0 ErrFoo NAME
staticinit'ing tests []error literal true
deps:
        ErrFoo: []
        tests: [ErrFoo]
lout:
        ErrFoo = errors.New("blah blah GINGER blah blah")
        statictmp_0 = ErrFoo
out:
        statictmp_0 = ErrFoo
        ErrFoo = errors.New("blah blah GINGER blah blah")
        tests = []error literal

Alright so we don't get to add statictmp_* to deps here:
https://github.com/chadwhitacre/go/blob/550acbbd29eaf6fc5a45ca75612aef3c47336c63/src/cmd/compile/internal/gc/sinit.go#L94-L100
Because statictmp_* doesn't exist there. 🤔

13d196ac13da31f5087792ba75d294dd290b9d0c
$ ../../../bin/go test
# archive/foo [archive/foo.test]
l:
        ErrFoo = errors.New("blah blah GINGER blah blah")
        <node DCLFUNC>
        tests = []error literal
        <node DCLFUNC>
seeing n: ErrFoo ErrFoo = errors.New("blah blah GINGER blah blah")
seeing n: errors.New <N>
staticinit'ing ErrFoo errors.New("blah blah GINGER blah blah")
staticassign'ing ErrFoo errors.New("blah blah GINGER blah blah") CALLFUNC
staticinit'ing ErrFoo errors.New("blah blah GINGER blah blah") false
seeing n: errors.New <N>
seeing n: tests tests = []error literal
seeing n: ErrFoo ErrFoo = errors.New("blah blah GINGER blah blah")
staticinit'ing tests []error literal
staticassign'ing tests []error literal SLICELIT
Here we are: statictmp_0
staticassign'ing statictmp_0 ErrFoo NAME
staticinit'ing tests []error literal true
seeing n: ErrFoo ErrFoo = errors.New("blah blah GINGER blah blah")
deps:
        ErrFoo: []
        tests: [ErrFoo]
lout:
        ErrFoo = errors.New("blah blah GINGER blah blah")
        statictmp_0 = ErrFoo
out:
        statictmp_0 = ErrFoo
        ErrFoo = errors.New("blah blah GINGER blah blah")
        tests = []error literal

repo: chadwhitacre/go


text:
[feature] aws-oidc flowchart

Codecov Report

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

@@           Coverage Diff           @@
##           master      #19   +/-   ##
=======================================
  Coverage   48.40%   48.40%
=======================================
  Files          10       10
  Lines         500      500
=======================================
  Hits          242      242
  Misses        238      238
  Partials       20       20

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 120c067...60f5b3c. Read the comment docs.

repo: chanzuckerberg/aws-oidc


text:
Bump version to 12.4.1

Hi. I am an automated pull request bot named Curry. There are commits in this pull request whose authors are not yet authorized to contribute to Chef Software, Inc. projects or are using a non-GitHub verified email address. To become authorized to contribute, you will need to sign the Contributor License Agreement (CLA) as an individual or on behalf of your company. You can read more on Chef's blog.
GitHub Users Who Are Not Authorized To Contribute
The following GitHub users do not appear to have signed a CLA:

@dmccown

Please sign the CLA here.

Hi. Your friendly Curry bot here. Just letting you know that all commit authors have become authorized to contribute. I have added the "Signed CLA" label to this issue so it can easily be found in the future.

:+1:

:+1:

authors: chef-supermarket jkeiser sdelano stevendanna

repo: chef/chef-server


text:
Fix missing comma at the end of line in README

Hi. I am an automated pull request bot named Curry. There are commits in this pull request whose authors are not yet authorized to contribute to Chef Software, Inc. projects or are using a non-GitHub verified email address. To become authorized to contribute, you will need to sign the Contributor License Agreement (CLA) as an individual or on behalf of your company. You can read more on Chef's blog.
GitHub Users Who Are Not Authorized To Contribute
The following GitHub users do not appear to have signed a CLA:

@niku4i

Please sign the CLA here.

Hi. Your friendly Curry bot here. Just letting you know that all commit authors have become authorized to contribute. I have added the "Signed CLA" label to this issue so it can easily be found in the future.

thanks!

repo: chef/mixlib-cli


text:
Migrated bookshelf to relx

Hi. I am an automated pull request bot named Curry. There are commits in this pull request whose authors are not yet authorized to contribute to Chef Software, Inc. projects or are using a non-GitHub verified email address. To become authorized to contribute, you will need to sign the Contributor License Agreement (CLA) as an individual or on behalf of your company. You can read more on Chef's blog.
GitHub Users Who Are Not Authorized To Contribute
The following GitHub users do not appear to have signed a CLA:

@joedevivo

Please sign the CLA here.

:+1: Needs the default_versions updated once other PRs are merged and a rebase.  Merge when ready.

Looks good to me. We'll definitely need to track this for our hosted deployements as it's a change to those cookbooks as well.

Hi. Your friendly Curry bot here. Just letting you know that all commit authors have become authorized to contribute. I have added the "Signed CLA" label to this issue so it can easily be found in the future.

OMG Thanks @chef-supermarket, we haven't talked enough about this!

:+1:  * :100:

authors: chef-supermarket joedevivo marcparadise stevendanna

repo: chef/opscode-omnibus


text:
h1.json contains a "undefined" key at the end

solved in  0b499c7

authors: andcastillo lpatiny

repo: cheminfo-js/nmr-predictor


text:
revert lvalue grammar - fix lvalue with post-ast

Recovered runtime lost with previous PR

repo: chipsalliance/Surelog


text:
Use react-native link instead of rnpm link

This change has already been made in another commit


text:
Allow user to specify an UAA client ID and client secret when registering a CF Endpoint

Is this a duplicate of #1384?

This is related to #1384 - which is specifically SSO.
You've done the back-end for this.
Front-end is in this PR #2920

authors: aeijdenberg irfanhabib nwmac

repo: cloudfoundry-incubator/stratos


text:
Fix migration script date

LGTM

authors: irfanhabib nwmac


text:
Adds newly-required components.search.boost.frequency to config files

LGTM

authors: nakomis robertgmoss

repo: cloudsoft/brooklyn-tosca


text:
udpa: enhance context_params documentation

CC @htuch @markdroth for review/approval/merging

repo: cncf/udpa


text:
Add circleci publishing config

Fixes #32

repo: code-star/scala-tsi


text:
"save and and another" page gives 500 error

Solved in the pages app

repo: comic/comic-django


text:
Fingerprint: 2021 Toyota Highlander / Highlander Hybrid (TSS 2.5)

Maybe this still requires additional testing...
DongleID 1f8f4de044bdbe9c
https://discord.com/channels/469524606043160576/524592892627517450/794205443416129566

authors: ErichMoraga VirtuallyChris

repo: commaai/openpilot


text:
Fix spec settings stub.

not sure what this for yet, but it looks harmless

authors: jsimpson-cc scytacki

repo: concord-consortium/rigse


text:
Update for 2.1.0

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

repo: conda-forge/spiceypy-feedstock


text:
星媒链信息提交

为了加速显示进度，请尽可能向我们提供完善的项目资料，以及知名区块链媒体报道，包括但不限于以下内容，以便 imToken 团队更了解你们的项目。
团队背景：
项目基本情况:
媒体报道：
推荐用户转账所使用的 Gas 数量（没有提供，我们默认值 60000）
PR 示例：
https://github.com/consenlabs/token-profile/pull/1636

authors: ZMBchain wenyanwang

repo: consenlabs/token-profile


text:
use special govpp.conf in dev image

Coverage remained the same at 47.45% when pulling 30573c1a45299e1a6eb36c49e02e7f60fb360f8f on rastislavszabo:master into a5747b61088041d6aa4528666fba076b74fe4773 on contiv:master.

authors: coveralls rastislavszabo

repo: contiv/vpp


text:
make service name an option

 service name optional


text:
ya tenemos logo

Changes Unknown when pulling d505256eb4ee0499fb57116722c02ad7ff2c1163 on rafinskipg:master into ** on corbel-platform:master**.

authors: coveralls rafinskipg

repo: corbel-platform/corbel-composr

license: unlicense


text:
fix stresstest module compilation

ack

authors: bogensberger seut

repo: crate/crate


text:
Fix behaviour when trying to match null values for primary key columns in select statements

otherwise lgtm

authors: matriv seut


text:
Rename crimethinc/magazine to crimethinc/website in README

Good catch, @guitsaru. Thanks!

authors: guitsaru veganstraightedge

repo: crimethinc/website


text:
Add conditions

relevant pull request is relevant!

Closed via #4

authors: crobi haxblade


text:
how to use express-validator for validation of input file?

Currently, express-validator only supports query string, request body and headers.
Cookies support is on the way with #232.
I don't see how file validation should take place.
Do you have any suggestion?

He might be asking for file extension and file size validation .

I can definitely see some value in this, keeping all request validation in one place as opposed to falling back on multipart form parser middleware for file validation. Typical rules would be required, acceptable file types and file size.
@waldemirflj have you tried writing a custom validator to plug into express-validator for this already?

@waldemirflj: I had a similar use case. I wanted to check if the submitted files have a .pdf extension. And I was able to achieve it by writing a custom validator.

Hi @Abhimanyu310, are you okay in sharing such custom validator with us?

@gustavohenke Sure.
Defined custom validator isPDF()
app.use(validator({
  customValidators: {
    isPDF: function(value, filename) {
      var extension = (path.extname(filename)).toLowerCase();
      return extension == '.pdf';
    }
  }
}));

Validated as below. I am using express-fileupload to parse the uploaded files, and you can access the filename as req.files.myfile.name, where myfile is the name of the file upload field in HTML.
req.checkBody('myfile', 'Please upload your file in PDF').isPDF(req.files.myfile.name);

@Abhimanyu310  What is path in above example ?

@nmajethiya:

it's probably Node's native path module
that's just an example to work around the lack of a req.checkFile() method.

Are there any plans to incorporate the validation of req.files?
We are currently rolling our own wrapper method applying the solution from @Abhimanyu310 but it feels like a hack to abuse the defined api in such a way.

@funklos you can surely suggest something feasible, or even implement it yourself.
Since I'm busy with other stuff, I don't have the time to come up with/implement any solution, so if one could do this for me it would be awesome 😄

I have one:
photo: {
        in: "files", // even use files.file with dot-prop :)
        custom: { // a custom validation is required for these inputs
            options(value: Array<any>, { req, location, path }) {
                // stuff
                const hasFiles = !!value && value.length > 0;
                if (!hasFiles) {
                    return hasFiles;
                }
            },
            errorMessage: "Files are required to process the request."
        }
    },

What do you think @gustavohenke?

I have a solution which works for me. I use multer to process multipart/form-data, then I assign req.files to req.body.files with the workaround:
const upload = multer({ storage, limits }).array("files");

const uploadHandler = (req, res, next) => {
    upload(req, res, (err) => {
        if (err) {
            res.status(400)
                .json({ message: `Bad request, ${err.message}` })
                .end();
        } else {
            // special workaround for files validating with express-validator
            req.body.files = req.files;
            next();
        }
    });
};

Following that, just use
check("files")

authors: Abhimanyu310 arunkillu beaclnd92 cbovis funklos gustavohenke nmajethiya sant123 waldemirflj

repo: ctavan/express-validator


text:
Check showtimes with current time

API was supported!!!

repo: cuongw/ting-movie


text:
Potential fix for GH issue #3215

Thanks! Superseded by c964bfd.

authors: achernomorov scoder

repo: cython/cython


text:
Can CTRXplorer open html file that is on the 3ds SD card?

Yes, but it can display it only as hexadecimal numbers. If you meant parsing the HTML and viewing it as a document - no, and a HTML parser goes beyound the scope of CTRXplorer. I might consider a text viewer at a later point.
I think Sunshell has a HTML viewer built in.
Closing this issue for now.

thanks for your replay. i means that can i run offline html file by CTRXplorer to boot rxtools on 3ds ?

thanks for your replay.


text:
Add try_clone method.

Actually this was very bad idea because it requires internal reference counting and can be achieved without this additions.

repo: danburkert/memmap-rs


text:
End callback for numeric input

Cool, thanks! 🙌

authors: danburzo darung

repo: danburzo/uiuiui


text:
Add FindMeetingQuery function

What's the difference between this an the other PR?

What's the difference between this an the other PR?

#12 should be closed, I incorporated the comments from it, but overall the algorithm is different.

authors: brucezheng danielabreo

repo: danielabreo/portfolio


text:
dattreact: Form for creating a new post

Closed by https://github.com/dattnetwork/datt/commit/7abf08ba70b51a5043d2ed4c0fbd4e8376fdb78f

repo: dattnetwork/datt


text:
refactor(environments): eliminate extra dev code/markup in production

Pinging @neverfox, @mtsr, and @nhagen since you are all fairly active contributors, I'd just like to get your opinion on this if you have a minute. It stems from https://github.com/davezuko/react-redux-starter-kit/issues/189 and a deeper issue where a lot of DevTool-related code still makes its way into production, so this is one sure-fire way of eliminating it. It's inelegant and a bit obtuse for people new to the starter kit, so I'm wondering if any of you share my wariness.

@davezuko no prob, i will take a look this afternoon. i was playing around with babel 6 migration last night and the only issues were with the current devtools setup, so this was going to be an problem down the road either way.
i will try the same with this PR and let you know results.
another solution i was looking into was using babel transforms to inject devtools in development using https://github.com/alexkuz/react-transform-debug-inspector
will follow up later :smile:

Would it not accomplish the same thing to do a conditional require of DevTools inside of a single Root.js?

@neverfox that could work, but if we go that route then both the store and Root are configured at compile time rather than at runtime, since configureStore would have to know to never import the DevTools. I wish dead code removal would remove unused imports, would make this whole issue trivial.

I'm going to close this because I think it complicates the starter kit, and anybody concerned with an as-slim-as-possible production bundle can figure this out on their own. Might offer an explanation for how to do this in https://github.com/davezuko/react-redux-starter-kit/issues/221, but I want this starter kit to be (relatively) approachable.

I think that's very reasonable, though I still favor this being optimized for production out of the box. If that's how it goes, there should definitely be a section of documentation that identifies optimizations that might be useful in production environments (like this)... Though I'm not aware of any other outstanding issues like this.

@nhagen I totally agree with it being optimized for production out of the box, so I'll keep playing around with solutions for this, but documentation will unfortunately have to suffice in the meantime.

authors: davezuko justingreenberg neverfox nhagen

repo: davezuko/react-redux-starter-kit


text:
AssetsOnly/SceneOnly validator

I created a request #270, and will implement it for v2

authors: dbrizov uniquecorn

repo: dbrizov/NaughtyAttributes


text:
[WIP] Testing Navstar Changes

Please bump this to the master commit, @drewkerrigan

@sargun bumped

@drewkerrigan So, the vagrant test got into a weird state and failed:

[15:39:36][Step 2/3] Vagrant can't use the requested machine because it is locked! This
[15:39:36][Step 2/3] means that another Vagrant process is currently reading or modifying
[15:39:36][Step 2/3] the machine. Please wait for that Vagrant process to end and try
[15:39:36][Step 2/3] again. Details about the machine are shown below:

When it does this, you should be able to restart the test by hitting 'run'. If this doesn't fix it, ping someone.

That particular thing I've tried getting rid of several times. Currently it just means "press run again". Not your stuff that failed but some internal issue with vagrant.

@drewkerrigan IS this still a WIP?

@sargun PR title and description updated.

authors: cmaloney drewkerrigan sargun

repo: dcos/dcos


text:
Tiny text edit

@Greenie10 Thanks!

repo: dewyze/trello-card-numbers


text:
Fixes verification to check the group of giver/getter rather than name

Thanks for this - I originally had the group and name order different, but then changed them, and that introduced this bug - nice catch.

authors: dfb josh-vin

repo: dfb/giftexchange


text:
removed eoscanada and more on eosq.app

Builds and runs locally. Changes are only concerning design.

repo: dfuse-io/dfuse-eosio


text:
js syntax highlighting

Thanks :)

authors: diasdavid reggi

repo: diasdavid/webrtc-explorer


text:
Home activity shouldn't exit after returning from Validating Media

Is this after validating multimedia was successful?
The code path here currently is:
Home Activity -> Detect Multimedia validation needed -> Launch Validation Activity
If the validation is successful we return with "OK" and proceed.
If it isn't the only thing the user can do is cancel, which exits the app (which is the desired behavior).
What issue is this addressing?

adding don't pull warning until question resolvede

authors: ctsims phillipm

repo: dimagi/commcare-odk


text:
Add Jedi logo for project

Hello,  where would you like the logo to be added? and where can I obtain this logo?

@gdtrombetti it did not created yet:)

I added a Jedi logo
PR

@hj222hi thanks

authors: dimkosru gdtrombetti hj222hi

repo: dimkosru/jedi-validate


text:
Add integration tests

This is done now.

repo: dinowernli/polyglot


text:
Minor release notes fix

merged in 8fb7a0987cafc1cafc8b7eb8dd53301dea0aee40

authors: tchaumeny timgraham

repo: django/django


text:
Use Time.now to check timeout in test

Thank you for your PR! I still have plenty to learn when it comes to testing, this was a much better approach!

authors: blid dmagliola


text:
[jvm-packages] getTreeLimit return type should be Int

LGTM, thanks

authors: CodingCat tovbinm

repo: dmlc/xgboost


text:
Need to specify group ID to group/workers

Please sign your commits following these rules:
https://github.com/moby/moby/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "tutorial-cli-needs-worker-id" git@github.com:docker/infrakit.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

Amending updates the existing PR. You DO NOT need to open a new one.

authors: GordonTheTurtle joeabbey

repo: docker/infrakit


text:
Fix error undefined index "targetEntity"

Dang, how did this slip in?
@andersonamuller when does this happen for you? Can we have a test case showing the notice?

Sorry, I don't have much time now for a test case, try this setup:
Classes:
abstract class Animal
{
    protected $id
}

class Cat extends Animal
{
    private $sound
}

class Sound
{
    private $id
}

Mapping Animal:
/** @var \Doctrine\ORM\Mapping\ClassMetadata $metadata */
$builder = new \Doctrine\ORM\Mapping\Builder\ClassMetadataBuilder($metadata);
$builder->setTable('animal');
$builder->setSingleTableInheritance();
$builder->setDiscriminatorColumn('kind', 'string', 32);
$builder->createField('id', 'binary')->isPrimaryKey()->build();

Mapping Cat:
/** @var \Doctrine\ORM\Mapping\ClassMetadata $metadata */
$builder = new \Doctrine\ORM\Mapping\Builder\ClassMetadataBuilder($metadata);
$builder->createOneToOne('sound', Sound::class)->fetchLazy()->addJoinColumn('sound_id', 'id', false)->build();

Mapping Sound:
/** @var \Doctrine\ORM\Mapping\ClassMetadata $metadata */
$builder = new \Doctrine\ORM\Mapping\Builder\ClassMetadataBuilder($metadata);
$builder->setTable('sound');
$builder->createField('id', 'binary')->isPrimaryKey()->build();

@andersonamuller I'll try integrating it with the test suite now

I integrated tests and merged everything in master at doctrine/doctrine2@d8d4ec6eb2cd1c276bd90b4f419e4caa75b15233
Thanks @andersonamuller!

@Ocramius
My pleasure, thank you for your hard work. Looking forward for the release.

authors: Ocramius andersonamuller


text:
C# LINQ Samples - Conversion Operators

Add LINQ samples for conversion operators for #90

authors: BethMassi matthewreily

repo: dotnet/core-docs


text:
Updating JIT version dependency (to support the GVM work)

@nattress PTAL

repo: dotnet/corert


text:
remove unused delegate

You're welcome @BillWagner :)

repo: dotnet/docs


text:
Any chance to update nuget package to latest version?

NPOI Standard 2.0 has been released, the github remove to  https://github.com/dotnetcore.
you can reinstall NPOI with :

PM: Install-Packege DotNetCore.NPOI

repo: dotnetcore/NPOI


text:
moco 能做到 通过第三方返回的json来判定response是什么么  on可以做到么

Could you please give me more details about your scenario？

比如说，我现在有一个登录接口，但是这个登录接口依赖一个验证码接口，验证码接口返回success就登录成功，返回failed 就登录失败，可以用on实现这个功能么

If you want to integrate a third party interface, try proxy
The better option is to use Moco as verfication code stub to return what you need.

if want get a complex json  ,for example {"a":{"c":"1"},"b":{"d":"2"}}  ,how to do

You may need Json Response

when  i want this cam to be a service ， i use  this command is not available
nohup java -jar moco.jar http -p 12306 -g rule.json -s 9527> ./log/log.txt &

repo: dreamhead/moco


text:
Unless configurations are synced between client & server, tooltip information will be wrong

Resolved but untested.

repo: duely/dwmh