text:
upd(warp): 0.6.2 -> 0.8.0

Is 0.6.2 the last version that can support all these distros/versions without needing non-apt dependencies?

Is 0.6.2 the last version that can support all these distros/versions without needing non-apt dependencies?

I think it just needs apt builddeps

repo: pacstall/pacstall-programs


text:
Update Composer dependencies (2022-05-09-00-06)

Visual regression test passed!
View the visual regression test report

source: gharchive/pull-request

repo: pantheonsteve/nasdaq-buildtools-demo

license: MIT

license_type: permissive

license_source: github-api


text:
Create a software template for bootstrapping assessment workflow development

/lgtm

authors: anludke dmartinol

repo: parodos-dev/workflow-software-templates


text:
Publish Gradle Plugin

https://plugins.gradle.org/plugin/org.partiql.pig.pig-gradle-plugin

source: gharchive/issue

repo: partiql/partiql-ir-generator


text:
chore(release): Trigger a release of previously merged changes

Preview: https://react-catalog-view-pr-catalog-view-64.surge.sh

authors: patternfly-build wise-king-sullyman

repo: patternfly/react-catalog-view


text:
destroying target chests while sorting is in progress can cause items to be lost

fixed in 0.1.3


text:
GD EEW API の定義を作成する

https://github.com/pdmdss/api-types/commit/ffdb1ea7815a26240eff662aaa530c0ee27169c4

https://github.com/pdmdss/api-types/commit/ffdb1ea7815a26240eff662aaa530c0ee27169c4
のコミットをRejectし、新たに定義をし直す

現在検討しているGD EEW仕様

eew-informationのearthquakeフィールド、intensityフィールドの最大予測震度・最大予測長周期地震動定義を世襲する
GD Earthquake と同様に、ListとEventでエンドポイントを分けるかは検討
EEW 最終報受信後、更新する

repo: pdmdss/api-types


text:
Package path change

Codecov Report

Merging #638 (2f88063) into main (95a5d63) will increase coverage by 0.00%.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##             main     #638   +/-   ##
=======================================
  Coverage   85.14%   85.14%
=======================================
  Files         307      307
  Lines       27312    27311    -1
=======================================
  Hits        23255    23255
+ Misses       4057     4056    -1

Impacted Files
Coverage Δ

lib/app/src/common/package_id_calculator.rs
0.00% <0.00%> (ø)

Continue to review full report at Codecov.

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

authors: codecov-commenter raviqqe

repo: pen-lang/pen


text:
Dev kmr

This needs a bit more testing before being marked available for review, but I figured I would get the ball rolling on CI testing now.  I had tested it quite extensively but then I refactored the commits a bit to make for a more logical commit history so I need to ensure I didn't break anything in doing so.

repo: perftool-incubator/rickshaw


text:
Project-level CI+tooling (i.e. linting)

[ ] Prettier
[ ] Eslint
[ ] Husky
[ ] Pre-commit lint checks
[ ] Github Action to run linting, tests coming later

repo: personaelabs/nym


text:
ci: execute static analysis, fuzzing, and dependency scanner on pull request target

Handling of forks will need a different solution.

authors: richardapeters rjaegers

repo: philips-software/amp-embedded-infra-lib


text:
Bring back workflows

Will be added in https://github.com/phrase/openapi/pull/174

authors: forelabs theSoenke

repo: phrase/phrase-cli


text:
Reduce the number of goroutines in create history stream

Merging but this will have to be revisited when implementing EBTs.

repo: planetary-social/scuttlego


text:
how about extract the  /ui/decredmaterial  component as another project and a separate library

Yeah this is actually being considered as decredmaterial seems robust enough to be a gio ui library.

authors: C-ollins hongshengjie

repo: planetdecred/godcr

license: ISC


text:
fix: duplicate db client breaks pglite

Ah nice catch, it looks like the database.kill function does two things:

Updates the metadata table

https://github.com/ponder-sh/ponder/blob/f3e383fa3e00e0156ccc1c6eb404e3ad41d80c56/packages/core/src/database/index.ts#L1171-L1179

Tears down the underlying client

https://github.com/ponder-sh/ponder/blob/f3e383fa3e00e0156ccc1c6eb404e3ad41d80c56/packages/core/src/database/index.ts#L1181-L1197
The "updating the metadata" step should only happen when database.setup is called, which doesn't happen in the serve case. I also think it makes sense that the function that creates the database is in charge of cleaning it up. Will do another commit.

The "updating the metadata" step should only happen when database.setup is called, which doesn't happen in the serve case. I also think it makes sense that the function that creates the database is in charge of cleaning it up. Will do another commit.

Makes sense. Yea, the WIP work I have locally actually separates the database driver(s) (and associated cleanup logic) from the database service. Was not able to get it to a clean conclusion though.

repo: ponder-sh/ponder


text:
Update main.yml

$branch = refs/pull/14/merge
echo Test this out locally with run sync-scripts --branch=$branch

repo: poteto/bitburner-scripts


text:
Adding theme setup for input

Codecov Report

Merging #73 (c84ae27) into master (59f45d4) will increase coverage by 0.04%.
The diff coverage is 100.00%.

:exclamation: Current head c84ae27 differs from pull request most recent head 218ce8f. Consider uploading reports for the commit 218ce8f to get more accurate results

@@            Coverage Diff             @@
##           master      #73      +/-   ##
==========================================
+ Coverage   79.23%   79.28%   +0.04%
==========================================
  Files         109      109
  Lines       10039    10039
  Branches     1013     1015       +2
==========================================
+ Hits         7954     7959       +5
+ Misses       2085     2080       -5

Flag
Coverage Δ

unittests
79.28% <100.00%> (+0.04%)
:arrow_up:

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

Impacted Files
Coverage Δ

.../lib/components/auto-suggest/auto-suggest-menu.tsx
97.22% <100.00%> (-0.08%)
:arrow_down:

...kages/lib/components/auto-suggest/auto-suggest.tsx
93.69% <100.00%> (+0.08%)
:arrow_up:

...s/lib/components/checkbox-group/checkbox-group.tsx
97.24% <100.00%> (+0.07%)
:arrow_up:

packages/lib/components/checkbox/checkbox.tsx
99.31% <100.00%> (ø)

packages/lib/components/input/input.tsx
98.08% <100.00%> (+0.01%)
:arrow_up:

...ackages/lib/components/radio-group/radio-group.tsx
97.89% <100.00%> (-0.17%)
:arrow_down:

packages/lib/components/tooltip/tooltip.tsx
100.00% <100.00%> (ø)

...kages/lib/components/common/effects/useFocusNew.ts
97.33% <0.00%> (+6.66%)
:arrow_up:

Continue to review full report at Codecov.

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

authors: codecov-commenter prabhuignoto

repo: prabhuignoto/react-creme


text:
Integrate Mistral 8x22 with Perplexity

Integrated openrouter one

repo: premAI-io/prem-utils


text:
stsd box includes full header

go ahead and merge into the mux branch when ready

I don't actually have write permissions in this repo, heh. I'll probably end up getting them at some point.


text:
Create SECURITY.md

Closing as spam.

authors: Ajivasanthy miketaylr


text:
feat: rebrand as lintroll

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

repo: privatenumber/lintroll


text:
reuse logic-1

@Maxusmusti I am merging this PR since it was testing on my fork

repo: project-codeflare/instascale


text:
[master] Semaphore Auto Pin Update

/merge-when-ready delete-branch

OK, I will merge the pull request when it's ready, leave the commits as is when I merge it, and delete the branch after I've merged it.

repo: projectcalico/libcalico-go

license_source: bigquery


text:
Improve macro hygiene

I've changed this to:

extern crate self as bevy_enhanced_input in lib.rs so all the unit tests compile
add the leading :: to the macro path
But I think, if a user uses this crate and renames bevy_enhanced_input to something else in the manifest, the derive macro will break. Not sure what to do about this.

Thanks for looking into it!

authors: Shatur aecsocket


text:
split histograms, line plots, error plots, bar graphs into separate functions

need to test using y_errors in error_plots and add docstring to hist function

repo: ptweir/datura


text:
Add convenience functions for encoding from and decoding to strings

Thank you for your contribution! However, I would prefer to keep the base2048 API ultra-streamlined. These new APIs are simple enough that users can implement them themselves - which would also allow them the option of using encodings other than UTF-8 for their string inputs and outputs. I might consider adding this as example code to the documentation, though.
(It doesn't matter at this point, but I would also have liked to see unit tests, and documentation for the new APIs. And perhaps it would be more efficient to create a single TextEncoder and TextDecoder and reuse them, instead of creating a brand new instance on every encode and decode call.)

authors: qntm redbrain


text:
Initial import

#sre

Other notes:

Side bar ordering
Getting started section
FAQ in capital

authors: dcupif vmercierfr

repo: qonto/database-monitoring-framework


text:
Replace Newtonsoft with System.Text.Json

why

authors: JKamsker dvdsgl

repo: quicktype/quicktype


text:
migrate to python 3.12, drop tw-complex dependency

Codecov Report

Merging #373 (041edfc) into master (265e2ac) will increase coverage by 0.07%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #373      +/-   ##
==========================================
+ Coverage   87.65%   87.72%   +0.07%
==========================================
  Files          80       81       +1
  Lines        5686     5713      +27
==========================================
+ Hits         4984     5012      +28
+ Misses        702      701       -1

Flag
Coverage Δ

unittests
87.72% <100.00%> (+0.07%)
:arrow_up:

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

Files
Coverage Δ

utils/avaiable_troops.py
81.91% <100.00%> (ø)

utils/basic/cdist_brute.py
100.00% <100.00%> (ø)

utils/basic/create_test_world.py
100.00% <ø> (ø)

utils/basic/table_text.py
85.50% <100.00%> (ø)

utils/get_deff.py
93.22% <100.00%> (ø)

... and 1 file with indirect coverage changes

authors: codecov-commenter rafsaf

repo: rafsaf/Tribal-Wars-Planer


text:
feat(partition): stricter types, add test

While testing this, I discovered a very interesting edge case for isNil: https://tsplay.dev/m3Kpjm
For partition to work correctly with isNil, that will need to be fixed first: https://github.com/ramda/types/pull/90

What do you need from me to consider this PR as mergeable ?

What do you need from me to consider this PR as mergeable?

@GerkinDev I merged https://github.com/ramda/types/pull/90, so

Rebase on develop
Re-order the overloads

The full signature always needs to come last

authors: GerkinDev Harris-Miller

repo: ramda/types


text:
Cluster Provisioning v2: Generic Drivers (Cluster & Node) - Cluster Driver List Enablement Page

Already exists in 2.6.0.

repo: rancher/dashboard


text:
suez update and fix for cln

Install test worked - merging.

authors: openoms rootzoll

repo: raspiblitz/raspiblitz


text:
Add option to submit client_id & client_secret via Authorization header to token endpoint

@rcbjBlueMars Confirming that you are expecting something of the form:
curl
-H 'Authorization: Basic 0xFeeDdEAdBeefDeadBeEf'
--request POST http://host:port/token
-d '...'

... where the basic header is base64encoded(client_id:client_secret).
The base64 encoding would prevent a 'debugging' of the original client values separately, but could obviously include the encoded content header.

I believe that is what we want, yes.

authors: bigfleet rcbjBlueMars


text:
Ensure warnings fail

Current dependencies on/for this PR:

main:

PR #3 👈

This comment was autogenerated by Freephite.

repo: rdelfin/min-review-bot


text:
Mounting via /etc/fstab

I couldn't get mounting via fstab to work on FreeBSD.
Using fuse-ufs doesn't really make sense anyway on FreeBSD, so I'm not gonna bother with it.

repo: realchonk/fuse-ufs


text:
New SVG logo

Pull Request Test Coverage Report for Build 5398554824

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

Totals

Change from base Build 5388325250:
0.0%

Covered Lines:
3217

Relevant Lines:
3625

💛  - Coveralls

authors: coveralls geragray

repo: realm/realm-dart


text:
license is not free enough

...What?

What license do you suggest then, outside of public domain licenses like WTFPL and Unlicense since it's legally questionable in some jurisdictions?
If you're going to say 0BSD License, then keep in mind that 0BSD is just ISC without the attribution clause. Why wouldn't you want to give credits to people who you used code from?

MIT
inb4 >akhually they are the same!!!!111

authors: CerealBowl DynamicDoughnut beerpiss

repo: redcode-labs/poXSSon


text:
youtube mix playlist support

Hey, I wonder if we can get the first video just via the playlist. I will investigate some stuff and will get back to you!


text:
[WIP] generic retry logic applies to nodes, network policies and network policy dynamic handlers

Pull Request Test Coverage Report for Build 2104951711

709 of 1441 (49.2%)  changed or added relevant lines in 7 files are covered.
70 unchanged lines in 5 files lost coverage.
Overall coverage decreased (-0.3%) to 50.386%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

go-controller/pkg/ovn/namespace.go
1
7
14.29%

go-controller/pkg/factory/factory.go
23
105
21.9%

go-controller/pkg/ovn/policy.go
128
219
58.45%

go-controller/pkg/ovn/master.go
50
154
32.47%

go-controller/pkg/ovn/ovn.go
120
285
42.11%

go-controller/pkg/ovn/obj_retry.go
377
661
57.03%

Files with Coverage Reduction
New Missed Lines
%

go-controller/pkg/ovn/egressfirewall_dns.go
1
82.31%

go-controller/pkg/node/node_ip_handler_linux.go
3
50.33%

go-controller/pkg/ovn/master.go
14
34.96%

go-controller/pkg/ovn/policy.go
14
69.95%

go-controller/pkg/ovn/ovn.go
38
51.97%

Totals

Change from base Build 2102053261:
-0.3%

Covered Lines:
14289

Relevant Lines:
28359

💛  - Coveralls

/retest

/retest-failed

authors: coveralls ricky-rav

repo: ricky-rav/ovn-kubernetes


text:
Create nth_tribonacci_number.py

Don't Forget to star this repository!

Don't Forget to star this repository!


text:
Logout button in navbar not working

Yes, currently working on it.

Logout working now, merged to develop

authors: kironjayesh rohitgeddam


text:
[Bug fix] Fix tag name bug in model.go/ProcessInfo

Codecov Report

Merging #32 (e399b0c) into master (870da7e) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #32   +/-   ##
=======================================
  Coverage   81.39%   81.39%
=======================================
  Files          33       33
  Lines        3795     3795
=======================================
  Hits         3089     3089
  Misses        541      541
  Partials      165      165

Impacted Files
Coverage Δ

entry/model.go
82.75% <ø> (ø)

Continue to review full report at Codecov.

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

authors: codecov-commenter dongxuny

repo: rookie-ninja/rk-entry


text:
[TricycleController] Fix deprecation warnings when compiling

Codecov Report

Merging #478 (786b580) into master (e7f9962) will decrease coverage by 6.07%.
The diff coverage is 20.15%.

@@            Coverage Diff             @@
##           master     #478      +/-   ##
==========================================
- Coverage   35.78%   29.70%   -6.08%
==========================================
  Files         189        7     -182
  Lines       17570      744   -16826
  Branches    11592      429   -11163
==========================================
- Hits         6287      221    -6066
+ Misses        994      163     -831
+ Partials    10289      360    -9929

Flag
Coverage Δ

unittests
29.70% <20.15%> (-6.08%)
:arrow_down:

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

Impacted Files
Coverage Δ

...de/diff_drive_controller/diff_drive_controller.hpp
100.00% <ø> (ø)

...ontroller/test/test_load_diff_drive_controller.cpp
12.50% <0.00%> (ø)

diff_drive_controller/src/odometry.cpp
42.16% <11.11%> (ø)

...ive_controller/test/test_diff_drive_controller.cpp
17.62% <12.08%> (ø)

diff_drive_controller/src/speed_limiter.cpp
46.55% <13.33%> (ø)

...troller/include/diff_drive_controller/odometry.hpp
20.00% <20.00%> (ø)

...iff_drive_controller/src/diff_drive_controller.cpp
32.04% <24.01%> (ø)

...test/test_load_joint_group_velocity_controller.cpp

...ory_controller/src/joint_trajectory_controller.cpp

...ontrollers/src/joint_group_velocity_controller.cpp

... and 192 more

authors: codecov-commenter destogl


text:
Cat Facts - 1st go

Close 1st Cat Fact

repo: rseeton/write-docker-actions


text:
Add explanations about how to generate tests in DAP

@st0012
Feel free to tell me if this explanation is not enough.

@ono-max thanks for the instructions! I have a question though: how do I know what steps were taken from the original tests?
the written tests are low-level requests/responses and it's hard to reverse-engineer the original actions. for example, how do I know if the scope request here was done by the client automatically (for UI), or it's triggered by clicking something?

for example, how do I know if the scope request here was done by the client automatically (for UI), or it's triggered by clicking something?

Well, if I want to know them, I run the command like $RUBY_DEBUG_DAP_SHOW_PROTOCOL=1 rdbg target.rb, then protocol messages will be outputted in the terminal. So, I compare the results of the tests with them, and I know them.

I mean the test case itself doesn't tell me what actions were performed originally. I can try recording it several times and select the most similar result. But that's not efficient because any mis-click or forgotten step means a restart.
I think we should have unit tests for individual commands. And they can be written with test helpers:
perform_dap_request "threads"
assert_dap_response {
  threads: [
    {
      id: 1,
      name: /#1 .*/
    }
  ]
}

perform_dap_request "stackTrace", {
  threadId: 1,
  startFrame: 0,
  levels: 20
}
assert_dap_response {
  stackFrames: [
    {
      name: "<main>",
      line: 1,
      column: 1,
      source: {
        name: /#{File.basename temp_file_path}/,
        path: /#{temp_file_path}/,
        sourceReference: nil
      },
      id: 1
    }
  ]
}

It'll be easy to add/update them, just like all the tests under test/debug/.
And the recording are for integration tests, probably something like
Script
class Foo
  def first_call
    second_call(20)
  end

  def second_call(num)
    third_call_with_block do |ten|
      forth_call(num, ten)
    end
  end

  def third_call_with_block(&block)
    @ivar1 = 10; @ivar2 = 20

    yield(10)
  end

  def forth_call(num1, num2)
    num1 + num2
  end
end

VSCode Actions
set breakpoints in line 20
continue
show backtrace
shoe info
quit

I think we should have unit tests for individual commands. And they can be written with test helpers:

You mean, we should prepare helper methods like dap_request(), or assert_dap_response()?
If it's your opinion, I have the following question.

How can we know if  threads: [ ... is the response of threads?

perform_dap_request "threads"
assert_dap_response {
  threads: [
    {
      id: 1,
      name: /#1 .*/
    }
  ]
}

First, I'd like to know your opinion in details.
Your opinion is

We should have unit tests.
We should use method instead of hash.

Is that correct?

For individual commands, we should use unit tests. Having threads request and response in break tests isn't necessary and it increases maintenance effort. For example, if we change the response of scopes command, all of these tests will be affected:

test/dap/next_test.rb
111:            command: "scopes",
120:            command: "scopes",
125:              scopes: [
550:            command: "scopes",
559:            command: "scopes",
564:              scopes: [

test/dap/break_test.rb
111:            command: "scopes",
120:            command: "scopes",
125:              scopes: [
400:            command: "scopes",
409:            command: "scopes",
414:              scopes: [
579:            command: "scopes",
588:            command: "scopes",
593:              scopes: [
739:            command: "scopes",
748:            command: "scopes",
753:              scopes: [

test/dap/boot_config_test.rb
185:            command: "scopes",
194:            command: "scopes",
199:              scopes: [

test/dap/detach_test.rb
114:            command: "scopes",
123:            command: "scopes",
128:              scopes: [
549:            command: "scopes",
558:            command: "scopes",
563:              scopes: [

test/dap/step_test.rb
111:            command: "scopes",
120:            command: "scopes",
125:              scopes: [
689:            command: "scopes",
698:            command: "scopes",
703:              scopes: [
826:            command: "scopes",
835:            command: "scopes",
840:              scopes: [

test/dap/finish_test.rb
111:            command: "scopes",
120:            command: "scopes",
125:              scopes: [
318:            command: "scopes",
327:            command: "scopes",
332:              scopes: [
484:            command: "scopes",
493:            command: "scopes",
498:              scopes: [

test/dap/step_back_test.rb
113:            command: "scopes",
122:            command: "scopes",
127:              scopes: [
404:            command: "scopes",
413:            command: "scopes",
418:              scopes: [
559:            command: "scopes",
568:            command: "scopes",
573:              scopes: [
725:            command: "scopes",
734:            command: "scopes",
739:              scopes: [
865:            command: "scopes",
874:            command: "scopes",
879:              scopes: [

test/dap/hover_test.rb
107:            command: "scopes",
116:            command: "scopes",
121:              scopes: [
332:            command: "scopes",
341:            command: "scopes",
346:              scopes: [
971:            command: "scopes",
980:            command: "scopes",
985:              scopes: [
1164:            command: "scopes",
1173:            command: "scopes",
1178:              scopes: [

And because the recording approach almost always generate certain commands, like scopes or threads, I don't think it's suitable for unit testing. Therefore, I hope we can have dap_request and assert_dap_response for those unit tests.

For integration testing, the recording approach is a great option. But currently it's hard to understand what operations were done in a test case (e.g. is the threads request made from the VSCode automatically, or it's generated by a user action).
I know we can figure it out by performing a similar operation with RUBY_DEBUG_DAP_SHOW_PROTOCOL. But it's not user-friendly and requires much manual work.
Since it's unlikely that we can record what the user does on an UI level (e.g. expanded a local variable). So I'd hope there is a standard to let test-author writing down what actions he/she performed to generate the test.

I think in the long-term, protocol-level testing (the current request/response tests) should mainly be used for unit testing. And integration testing should use something like vscode-extension-tester (here's an article for it) that performs real end-to-end testing from the VSCode. It'll be similar to how many developers tests frontend with selenium.
However, an obvious downside of this is the need to write javascript 😂

Thank you for explaining to me.
Let me rethink about it.

@st0012

protocol-level testing (the current request/response tests) should mainly be used for unit testing

Because we have already wrote some tests? From 1 and 2, I thought you want to change from protocol-level testing to using some methods such as dap_request and assert_dap_response.

Because we have already wrote some tests?

I'd be happy to rewrite them into simpler ones, like for stepping tests we only test related endpoints and wihout threads or scopes ones.

I thought you want to change from protocol-level testing to using some methods such as dap_request and assert_dap_response.

that'll be an improvement for sure and perhaps we can start from there. but I think the clear separation for unit/integration testing is more important.
for example, break_test.rb shouldn't contain dap_request("scopes") as it's irrelevant to the endpoint we want to test. and the recording approach can't clearly exclude those requests, so I think we shouldn't use it in command-specific tests like break_test.rb.

please continue the discussion on another ticket about the test data format.

or you can continue here.

Ahh, I got your points. I'll rethink it.

@ono-max if you think it's ok to convert most of the command-specific tests into unit-tests, we can do that as follows:

Open a PR to add dap_request and assert_dap_response helpers.
Start rewriting simpler tests like detach_test or hover_test.

By rewriting I mean using the new helpers + don't include unrelated request/response.

I think the current step_test and step_back_test are complicated enough to be used as integration tests.
Convert the rest of the tests into unit-tests.

I create a prototype to think about it. Could you give me some more time? Thanks.

repo: ruby/debug


text:
Fix: respect the image default workdir

Hey @runabol , do you remember what the context was for this change? I'm working on a middleware that persists a mount across every task in a job, and I want to be able to set the workdir per task to this mountpoint.
I'm more than happy to submit a PR to add what I'm after, but want to make sure there's not a footgun waiting for me, or even that the change would be accepted!

So I'm already using that, but I want to avoid starting the run in every task with the line cd /blah.
I'll move this to an issue to talk through design, but I'll hopefully have a PR for this today

authors: jspc runabol

repo: runabol/tork


text:
Rustup

@bors r+

:pushpin: Commit 326b95b1338673026192c50e782f15ff5acc4c48 has been approved by RalfJung
It is now in the queue for this repository.

:hourglass: Testing commit 326b95b1338673026192c50e782f15ff5acc4c48 with merge b6c395b374b5a3626327deaae46596ce0fdb3dd2...

:broken_heart: Test failed - checks-actions

@bors r+

:pushpin: Commit 936e6b2ef20ca12d8a8b118eed9510e4ffc922e4 has been approved by RalfJung
It is now in the queue for this repository.

:hourglass: Testing commit 936e6b2ef20ca12d8a8b118eed9510e4ffc922e4 with merge 14f1a23efd2d546b7cdbb52aaabc0fe73b0c81dc...

:sunny: Test successful - checks-actions
Approved by: RalfJung
Pushing 14f1a23efd2d546b7cdbb52aaabc0fe73b0c81dc to master...

authors: RalfJung bors

repo: rust-lang/miri


text:
fix member path

Thanks!

repo: rust-lang/rfcs


text:
publication

👷 Deploy Preview for crpp-ai-oncology processing.

Name
Link

🔨 Latest commit
e461af4e8bb39b75ae4b46f00491454ce9f46170

🔍 Latest deploy log
https://app.netlify.com/sites/crpp-ai-oncology/deploys/6352f5a5f3645200086b4bfb


text:
Add test cases for history.py

Build is failing https://app.travis-ci.com/github/sak007/MyDollarBot/builds/240895751
Don't hardcode path

Updated the Styling issues and Style Checkers - 134544d20b15bc493ee7d5703259fce1d3bf7e18
Build passing and test cases updated for delete.py and history.py
Please verify the close the issue if there are no more errors. @sak007 @sak1997

Updated the Styling issues and Style Checkers - 134544d20b15bc493ee7d5703259fce1d3bf7e18
Build passing and test cases updated for delete.py and history.py
Please verify the close the issue if there are no more errors. @sak1997  @sak007


text:
Remove multithreading in ParallelStep

Codecov Report
Base: 90.18% // Head: 89.89% // Decreases project coverage by -0.29% :warning:

Coverage data is based on head (b1c2013) compared to base (319a5d2).
Patch coverage: 77.42% of modified lines in pull request are covered.

:exclamation: Current head b1c2013 differs from pull request most recent head 2f85a7c. Consider uploading reports for the commit 2f85a7c to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #79      +/-   ##
==========================================
- Coverage   90.18%   89.89%   -0.30%
==========================================
  Files          51       51
  Lines        5492     5431      -61
==========================================
- Hits         4953     4882      -71
- Misses        539      549      +10

Impacted Files
Coverage Δ

src/steps/bins/game_elo_bin.rs
100.00% <ø> (ø)

src/steps/bins/time_control_bin.rs
93.33% <ø> (ø)

src/steps/filters/min_moves_filter.rs
0.00% <0.00%> (ø)

src/steps/maps/count_map.rs
93.33% <ø> (ø)

src/steps/maps/perfect_checkmate_map.rs
93.44% <ø> (ø)

src/steps/misc_steps/init_boards_step.rs
17.04% <0.00%> (+0.91%)
:arrow_up:

src/steps/misc_steps/parallel_step.rs
0.00% <0.00%> (-89.10%)
:arrow_down:

src/steps/misc_steps/ui_monitor_step.rs
0.00% <0.00%> (ø)

src/steps/reducers/max_reduce.rs
0.00% <0.00%> (ø)

src/workflow_step.rs
67.96% <25.00%> (ø)

... and 31 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

authors: codecov-commenter samgoldman


text:
Added two pages angular defer loading and control flow in angular

Thank you @pawanpatil08 I will review this, I need to check why build is failed

Thank you both I am trying to figure out the build issue, I will soon review the PR

ing the Signals RFC,

Hi @pawanpatil08 thank you for PR. Let me know once you take care of comments from @ajitzero, also if you can update the branch from main that would be greatt.

repo: santoshyadavdev/angular-snippets


text:
Use Buildinfo Plugin

@sjrd, do you have further comments on this? I'd advise against blocking this on https://github.com/sbt/sbt-buildinfo/issues/186: It feels like that is going to be a larger discussion, since the interface to provide is non-obvious to me.

Il have another look tomorrow, as well as your other PRs. Sorry for the delay.

No worries. There is no rush. I just wanted to make sure we're not both waiting on each other :)

Looks good to me. I'll let you squash in the most appropriate way. ;)

Done.

repo: scala-js/scala-js


text:
[PTFE-433] ✨ Add workflow cost metric

Codecov Report

Merging #44 (d0b786d) into main (6f4b71e) will increase coverage by 1.16%.
The diff coverage is 97.50%.

@@            Coverage Diff             @@
##             main      #44      +/-   ##
==========================================
+ Coverage   96.15%   97.31%   +1.16%
==========================================
  Files          10       12       +2
  Lines         442      522      +80
==========================================
+ Hits          425      508      +83
+ Misses         17       14       -3

Flag
Coverage Δ

api
97.31% <97.50%> (?)

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

Impacted Files
Coverage Δ

gh_actions_exporter/main.py
96.15% <88.88%> (-1.41%)
:arrow_down:

gh_actions_exporter/metrics.py
92.91% <93.75%> (+4.58%)
:arrow_up:

tests/api/metrics/test_job.py
97.97% <97.97%> (ø)

gh_actions_exporter/Webhook.py
100.00% <100.00%> (ø)

gh_actions_exporter/config.py
97.22% <100.00%> (+0.44%)
:arrow_up:

gh_actions_exporter/cost.py
100.00% <100.00%> (ø)

gh_actions_exporter/types.py
100.00% <100.00%> (ø)

tests/api/conftest.py
100.00% <100.00%> (ø)

tests/api/metrics/conftest.py
100.00% <100.00%> (ø)

tests/api/metrics/test_workflow.py
100.00% <100.00%> (ø)

I don't understand why this metric was added, as I fail to see the difference with github_actions_job_cost_count given that the label workflow_name is available on it as well.
Would you mind sharing the query you are trying to do?

Certainly, in essence, what I wanted to do with the graph was to display the average cost per workflow, something like this:
sum(increase(github_actions_workflow_cost_count_total{repository=~"$repository", runner_type=~"$runner_type", repository_visibility=~"$repository_visibility", cloud=~"$cloud", workflow_name=~"$workflow_name"}[$__interval])) by (repository, workflow_name) / sum(increase(github_actions_workflow_total_count_total{repository=~"$repository", runner_type=~"$runner_type", repository_visibility=~"$repository_visibility", cloud=~"$cloud", workflow_name=~"$workflow_name"}[$__interval])) by (repository, workflow_name) > 0

As you know, github_actions_workflow_cost_count_total does not exist, so I wanted to add it. Initially, I tried to work around this issue by changing the query and finding an alternative approach, but without success. The information available to us is the cost of each job, the number of jobs, and the number of workflows. Based on that, I don't see how to calculate the average cost of each workflow. If you have any ideas, I'm open to them, and we can simply close this pull request.

Given how you are using this new metrics, you will reach the same result with the existing one:
sum(increase(github_actions_job_cost_count_total{repository=~"$repository", runner_type=~"$runner_type", repository_visibility=~"$repository_visibility", cloud=~"$cloud", workflow_name=~"$workflow_name"}[$__interval])) by (repository, workflow_name) / sum(increase(github_actions_workflow_total_count_total{repository=~"$repository", runner_type=~"$runner_type", repository_visibility=~"$repository_visibility", cloud=~"$cloud", workflow_name=~"$workflow_name"}[$__interval])) by (repository, workflow_name) > 0

Checkout how a graph can look like with it: https://mon.scality.net/grafana/d/WidbLgPnk/github-actions-monitoring?orgId=1&refresh=30s&viewPanel=72

repo: scality/gh-actions-exporter


text:
updating code to check the execution point in release pipeline

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

Needs approval from an approver in each of these files:

OWNERS

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

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: sohimanshu
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [sohimanshu]

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

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: sohimanshu
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [sohimanshu]

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

authors: so-frontend-integrations so-hvats

repo: scheduleonce/ng-strictify


text:
create DAL (parse the JSON into a python object)

created basic file loading, but actually need to filter the raw messages to text Ive sent from the root posts and any responses

repo: seanbreckenridge/piazza-scraper


text:
Initial informal design for returning mutable references

Moved to https://github.com/secure-foundations/verus/discussions/35 for now.

repo: secure-foundations/verus


text:
add a FAQ about NTS DoT deadlock

Thanks for the review! I have reworded the FAQ.

@czhang03 FYI, I bypassed signature verification to get this merged. Please setup commit verification for future pull requests

repo: secureblue/secureblue


text:
Update README.md

remove one

repo: seniorforblockchain/XFIDApp


text:
[INFRA] Update CPM Packages

Documentation preview available at https://docs.seqan.de/preview/seqan/chopper/275

repo: seqan/chopper


text:
Master

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


text:
Add travis config

@pcwalton Have any thoughts about this assertion fail?

     Running `/home/travis/build/servo/gaol/target/debug/file_read_all-84dc191c2e9bdbd9`
thread '<main>' panicked at 'called `Result::unwrap()` on an `Err` value: Error { repr: Os { code: 2, message: "No such file or directory" } }', ../src/libcore/result.rs:746
note: Run with `RUST_BACKTRACE=1` for a backtrace.
thread '<main>' panicked at 'assertion failed: libc::read(pipe_fds[0], &mut grandchild_pid as *mut i32 as *mut c_void,
           mem::size_of::<pid_t>() as size_t) ==
    (mem::size_of::<pid_t>() as ssize_t)', platform/linux/namespace.rs:271
note: Run with `RUST_BACKTRACE=1` for a backtrace.

Check to make sure the libc declaration is correct, maybe 32 vs 64 bits is messing us up

@pcwalton #16 says thread '<main>' panicked at 'assertion failed: (left == right)(left:0, right: 4)', platform/linux/namespace.rs:271. :(

"PR_SET_CHILD_SUBREAPER (since Linux 3.4)", I'm afraid Travis uses a Linux kernel that does not provide that.

#17

authors: Ms2ger frewsxcv jdm nox pcwalton

repo: servo/gaol


text:
Impose configurable instance node size and scale on non-admin instances

Added in 8a8a2b5f071edfd29308e92c9106264b58449a71 and b5b55cde530c11e95e5ef6185f38c3078beebdde

repo: sharingio/pair


text:
Fix readme typo

Thank you!

authors: erjanmx sharkdp

repo: sharkdp/hexyl


text:
several pickle files

hmm for some reason this contribution appears under the github account @eladrap, even though I made the PR via this account (erap129). Did you somehow add @eladrap manually? If so I would be glad if you can swap it so this work will appear under this account (erap129). Thanks.

Hi, are we missing anything for this to continue? Thanks :)

Really hate to nag but do you think we will follow through with this? Thanks.

Hey, all good. Good of you to nag.

I didn't change anything manually. I wouldn't even know how to try something like altering the username this appears under. Let me check if I can fix this, however.

Yeah it was me of course, I made a mess by confusing my private and work accounts. I amended the commit to change the author and made the PR again. It's my first open-source contribution so I guess I'm just checking off beginner mistakes :)

license: mit


text:
chore(ssh): improve logging for channels

Changes made in this PR was added in #3621

repo: shellhub-io/shellhub


text:
🧪 Add test cases for each commands and models

This issue is stale because it has been open for 30 days with no activity.

This issue was closed because it has been inactive for 14 days since being marked as stale.

repo: siddhantprateek/go-gpt


text:
Testing changes to workflow

Can be closed now.

repo: siggisv/remotemo


text:
Remove JarServiceNameProvider - it's present in the upstream agent

Just for our future selves: The upstream class name is JarServiceNameDetector but it's also @AutoService(ResourceProvider.class) so we don't have to reference it by name. Just calling it out here in case this helps us later.

authors: breedx-splk mateuszrzeszutek

repo: signalfx/splunk-otel-java


text:
Add Open with FLEx button

@myieye Kevin mentioned you finishing out the GUI work for this task. Cool beans?
We also shouldn't merge this until we get a release of FLEx with my changes. I'm following up on getting some sort of a beta release right now

@myieye Kevin mentioned you finishing out the GUI work for this task. Cool beans?
We also shouldn't merge this until we get a release of FLEx with my changes. I'm following up on getting some sort of a beta release right now

You're alive 🎉. That's the first I've heard of me doing GUI work for this 😆. Perhaps my memory fails me. I'd enjoy it, so 🤷

The UI is more or less done, but I wanted to let you have a pass at it, and possibly the UX of it too, I'll take a look when I get a chance also.

Rebased onto develop
Made use of Daisy 4 e04ffc4
Removed added, unused parameter b3ca963

I want to merge this in, but I feel like I want to do #477 first so we can only show this button to us. Maybe we need a feature flag system.

implemented the dev mode as a simple local storage value, with a global function we can call from the console. This is ready to merge now.

I squashed virtually all the commits so that the rebase was easier.
Everything seems to be working:

authors: hahn-kev josephmyers myieye

repo: sillsdev/languageforge-lexbox


text:
add: guard in base plugin

covered in dialogy 0.9.0

repo: skit-ai/dialogy


text:
Update for go documentation

Codecov Report

Merging #65 (fe12b76) into main (30a1da8) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main       #65   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            3         3
  Lines           92        92
=========================================
  Hits            92        92

Impacted Files
Coverage Δ

terminator.go
100.00% <ø> (ø)

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

authors: codecov-commenter skovtunenko

repo: skovtunenko/graterm


text:
Can i add regex?

Can i do a pr to add some regex questions

Hi @apoorvcodes sure, you can add questions on Regex under the Strings folder. Make sure that you follow the contribution guidelines

sure

@apoorvcodes are you adding any files? Or should I close this issue?

Closing this issue as it is stale.

authors: apoorvcodes skully-coder

repo: skully-coder/competitiveprogramming


text:
fix(nullable): better support for nullable on jsonschema

Current dependencies on/for this PR:

main

PR #76   👈

PR #77

This comment was auto-generated by Graphite.

repo: skyleague/therefore


text:
Add alert example on readme

Codecov Report

Merging #40 (cf98908) into main (9fd6b05) will increase coverage by 0.02%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #40      +/-   ##
==========================================
+ Coverage   86.65%   86.67%   +0.02%
==========================================
  Files          10       10
  Lines         622      623       +1
==========================================
+ Hits          539      540       +1
  Misses         68       68
  Partials       15       15

Impacted Files
Coverage Δ

internal/metrics/prometheus/collector.go
85.14% <100.00%> (ø)

internal/storage/tfe/tfe.go
82.85% <100.00%> (+0.12%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

repo: slok/tfe-drift


text:
create changelog.md

this can be achieved by using commits view in github

this can be achieved by using commits view in github

repo: smeisegeier/pandas-plots


text:
The model has two outputs. What do they mean?

The model has two outputs. What do they mean?

How do I relate the two outputs?

repo: sooftware/conformer


text:
adjust alignment of nationality in edit profile page

might be fixed by this: 6425713

authors: JonathanContrerasM dariomonopoli-dev


text:
help center and about page

Also, check for anywhere you have an apostrophe or quote and replace them with the necessary Next.js recommended character entity code

authors: lolubabafemi spaceet

repo: spaceet/spaceet-web


text:
Update geopandas and other libraries to newer version

V1.2 tests did all execute successfully after lib update


text:
Added Zambian Holidays

Thanks!

repo: spatie/holidays


text:
Broken Link fixed in getting started section

I have changed the link according to your requirement.


text:
WIP: Just for test

🚀 Deployed on https://65ebc39228bd13410802c012--egressgateway.netlify.app

authors: lou-lan weizhoublue

repo: spidernet-io/egressgateway


text:
upstream cert-manager doesn't have rbac permissions for clusterissuers

Adding more context on the problem reported in the spire slack channel.
We were trying to use the spire helm chart to install the server in K8s using cert-manager upstream. We have cert-manager running and using the AWSPCAClusterIssuer issuers in our clusters. When we try to use the AWSPCAClusterIssuer clusterissuers for the cert-manager upstream, the spire server complains that it couldn't find the issuer.
Below are the clusterissuers.
k get awspcaclusterissuer
NAME                           AGE
aws-acm-cluster-issuer         545d
aws-acm-cluster-issuer-ecdsa   545d

Upstream Config used in the helm chart installation
## @param upstreamAuthority.certManager.issuer_name Defaults to the release name, override if CA is provided outside of the chart
issuer_name: "aws-acm-cluster-issuer-ecdsa"
## @param upstreamAuthority.certManager.issuer_kind Defaults to "Issuer", override if CA is provided outside of the chart
issuer_kind: "AWSPCAClusterIssuer"
## @param upstreamAuthority.certManager.issuer_group Defaults to "cert-manager.io", override if CA is provided outside of the chart
issuer_group: "awspca.cert-manager.io"
## @param upstreamAuthority.certManager.namespace Specify to use a namespace other then the one the chart is installed into
namespace: ""

Spire server pod logs captured using debug log level.
time="2024-05-17T03:38:57Z" level=debug msg="Preparing X509 CA" slot=B subsystem_name=ca_manager
time="2024-05-17T03:38:57Z" level=info msg="Waiting for certificaterequest to be signed" external=false name=spiffe-ca-f5vwn namespace=spire-server plugin_name=cert-manager plugin_type=UpstreamAuthority subsystem_name=catalog
time="2024-05-17T03:38:57Z" level=error msg="Created CertificateRequest has failed" external=false message="issuer could not be found" name=spiffe-ca-f5vwn namespace=spire-server plugin_name=cert-manager plugin_type=UpstreamAuthority reason=Failed subsystem_name=catalog
time="2024-05-17T03:38:57Z" level=debug msg="Optimistically cleaning-up stale CertificateRequests" external=false plugin_name=cert-manager plugin_type=UpstreamAuthority subsystem_name=catalog
time="2024-05-17T03:38:57Z" level=debug msg="Deleting stale CertificateRequest" external=false message="issuer could not be found" name=spiffe-ca-f5vwn namespace=spire-server plugin_name=cert-manager plugin_type=UpstreamAuthority reason=Failed subsystem_name=catalog type=Ready
time="2024-05-17T03:38:57Z" level=error msg="Unable to rotate X509 CA" error="rpc error: code = Internal desc = upstreamauthority(cert-manager): request has failed" subsystem_name=ca_manager

We know what the problem is now, since spire deletes the failed CertificateRequests, we captured it using a script and could see the below CertificateRequests. If you notice the issuer name you can see the name aws-acm-cluster-issuer-ecdsa-ca , we noticed it charts templates appends the -ca to the issuer name here. Not sure if this is intended?
{
    "apiVersion": "v1",
    "items": [
        {
            "apiVersion": "cert-manager.io/v1",
            "kind": "CertificateRequest",
            "metadata": {
                "creationTimestamp": "2024-05-17T09:08:32Z",
                "generateName": "spiffe-ca-",
                "generation": 1,
                "labels": {
                    "cert-manager.spiffe.io/trust-domain": "abcxz.net"
                },
                "name": "spiffe-ca-4vblx",
                "namespace": "spire-server",
                "resourceVersion": "884971325",
                "uid": "af6c70c9-60b6-4d2d-8c44-c8c5bc23254d"
            },
            "spec": {
                "duration": "24h0m0s",
                "extra": {
                    "authentication.kubernetes.io/pod-name": [
                        "spire-server-0"
                    ],
                    "authentication.kubernetes.io/pod-uid": [
                        "2247bb84-0fa5-4d8c-93ad-50e2e650495d"
                    ]
                },
                "groups": [
                    "system:serviceaccounts",
                    "system:serviceaccounts:spire-server",
                    "system:authenticated"
                ],
                "isCA": true,
                "issuerRef": {
                    "group": "awspca.cert-manager.io",
                    "kind": "AWSPCAClusterIssuer",
                    "name": "aws-acm-cluster-issuer-ecdsa-ca"
                },

We also noticed that the spire-server-cm role doesn't have permissions for clusterissues and awspcaclusterissuers , it's only limited to certificaterequests and issuers here.
rules:
  - apiGroups: ["cert-manager.io"]
    resources:
      - certificaterequests
      - issuers
    verbs:
      - list
      - get
      - create
      - delete
      - patch
      - update

Solution
We added the permissions for clusterissuers and awspcaclusterissuers  in the spire-server-r, removed the -ca from the configmap, and deployed the chart which was successful 🎉 .

Some more context to the issue: https://github.com/spiffe/helm-charts/pull/378
Looks like -ca is always being appended, rather then just if issuer is not overridden.

clusterissuers and awspcaclusterissuers

Are both really needed for reading by spire rather then just awspcaclusterissuers? Which one did you configure it to point to? We may need to come up with another value for additional permissions.

I added both of them. However, I believe that people using clusterissuers would also face the same problem.

Also, I don't understand why the -ca is appended even in the overrides. If someone wants to use existing issuers this won't work well as the issuers will mistach.

I added both of them. However, I believe that people using clusterissuers would also face the same problem.

Trying to figure out how to fix it in general. If both are needed, we need to figure out a way to let the user specify a list of them. If only one is needed, a string is probably ok.

Also, I don't understand why the -ca is appended even in the overrides. If someone wants to use existing issuers this won't work well as the issuers will mistach.

Its a bug. we gotta figure out how to fix it without breaking too many things.

The clusterissuers are needed for sure, if any custom issuers are used like awspcaclusterissuers we should have a provision to specify them(in helm) and accordingly update the RBAC. Here is a list of all issuers that cert-manager supports https://cert-manager.io/docs/configuration/issuers/

I deployed cert manager in minikube, the aws plugin, made a dummy awspcaclusterissuer. then deployed with:
spire-server:
  upstreamAuthority:
    certManager:
      enabled: true
      issuer_name: example
      issuer_kind: AWSPCAClusterIssuer
      issuer_group: awspca.cert-manager.io/v1beta1

It creates the certrificate request:
NAME              APPROVED   DENIED   READY   ISSUER       REQUESTOR                                    AGE
spiffe-ca-nc4fw                               example-ca   system:serviceaccount:default:spire-server   30s

So, I can reproduce the extra -ca added to the name... I don't have access to AWSPCA really though so cant test it further.
I looked through the code in spire though and cant find any references to it actually needing to get/list issuers though with regard to the upstream plugin.
How does it behave/fail if it doesn't have the extra types listed in the role?

At least in the case of switching the kind to ClusterIssuer, I was able to use it without needing any changes to the role. The spire server doesn't seem to need it.
Could you please try pr 361 and see if that lone is enough to get things working for you?

@kfox1111 - I can confirm that #361 is working as expected without the RBAC changes.

repo: spiffe/helm-charts-hardened


text:
Real 1.0.1

Spam request. I get the feeling people are using the PetClnic as some toy project to learn spring and github!

authors: Giridevops-Git mdeinum

repo: spring-projects/spring-petclinic


text:
Test merge

@JustFiesta Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

authors: JustFiesta pivotal-cla


text:
:sparkles: feat(FileDataSourceReader): Add txt-tsv reading.

We do not need this PR anymore. We decided to use an option in sourceBinding indicating the content-type of the file being read whether it comes from a zip file or from a folder.

repo: srdc/tofhir


text:
build: expose webhook failurePolicy

Codecov Report

Merging #267 (e6bb308) into develop (485e4f7) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #267   +/-   ##
========================================
  Coverage    96.66%   96.66%
========================================
  Files           22       22
  Lines         1080     1080
========================================
  Hits          1044     1044
  Misses          36       36

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 485e4f7...e6bb308. Read the comment docs.

authors: codecov-commenter xopham

repo: sse-secure-systems/connaisseur


text:
Proper name for GetCollection.

@ssone95 merge conflicts resolved.


text:
docker 20.10.20

docker run -ti  ssst0n3/docker_archive:ubuntu-22.04_docker-ce-20.10.20_docker-ce-cli-20.10.20_containerd.io-1.6.14-1_docker-compose-plugin-2.14.1


text:
update analysis only experiment creation

#1685  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @anna-statsig and the rest of your teammates on  Graphite

🚀 Deployed on https://6679aa0e46aa2d008cc694c5--cozy-fox-0defec.netlify.app

authors: anna-statsig jasonwzm

repo: statsig-io/docs


text:
Create use-case.md

🚀 Deployed on https://67149fe70864779056b795de--cozy-fox-0defec.netlify.app

authors: jasonwzm lin-statsig


text:
upgrade FE helpers and pass custom tracingOrigins param

Preview is available here:http://stellar-demo-wallet-pr231.previews.kube001.services.stellar-ops.com/

authors: piyalbasu stellar-jenkins

repo: stellar/stellar-demo-wallet


text:
Updated dependencies

Preview is available here:http://stellar-demo-wallet-pr241.previews.kube001.services.stellar-ops.com/

Preview is available here:http://stellar-demo-wallet-pr241.previews.kube001.services.stellar-ops.com/

authors: quietbits stellar-jenkins


text:
Update OWNERS

/LGTM

authors: morvencao yanmxa

repo: stolostron/hub-of-hubs-agent


text:
Resynced OWNERS file for release-2.8

/lgtm

authors: dislbenn joeg-pro

repo: stolostron/multiclusterhub-operator


text:
Fix issue with optional unions when generating code

Thanks for adding the RELEASE.md file!

Here's a preview of the changelog:

This release fixes a bug in the codegen that marked optional unions
as non optional.

Here's the preview release card for twitter:

Here's the tweet text:
🆕 Release (next) is out! Thanks to @patrick91 for the PR 👏

Get it here 👉 https://github.com/strawberry-graphql/strawberry/releases/tag/(next)

repo: strawberry-graphql/strawberry


text:
New rule (Spam): Spoofed domain using default Microsoft Exchange Online sending infrastructure

/update-test-rules

/update-test-rules

/update-test-rules

authors: aidenmitchell morriscode

repo: sublime-security/sublime-rules


text:
change substrate-typegen storage interfaces

Need to implement .getOrNull() storage method

merged in https://github.com/subsquid/squid-sdk/pull/134

repo: subsquid/squid-sdk


text:
Update containers to recent PHP and MariaDB

Drupal 10.3 is recommending PHP version 8.3 or above. Currently, the newest stable PHP is v8.3.8.
Also the MariaDB major version has now reached 11. MariaDB 11.4 is a long-term series, and will be maintained for five years. The most recent version is 11.4.2.
The containers should be updated to implement these changes.

When deployed to production, an effort should be made to ensure matching versions are used.

repo: surreymagpie/cornerhouse


text:
Trim some unused things from the compiler

Lgtm - maybe we can refactor even more here. :0

I just did :wink:, though I still think there is more to be done...

repo: sylt-lang/sylt-lang


text:
Feat remove unused options

Pull Request Test Coverage Report for Build 5835984249

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

Totals

Change from base Build 5760940140:
0.0%

Covered Lines:
799

Relevant Lines:
1441

💛  - Coveralls

authors: coveralls dstallenberg

repo: syntest-framework/syntest-core


text:
Report action result via Webhook

CLIを叩いた側で良きにすることにする。

repo: szkcr/minagine-cli


text:
Update .env and yarn.lock

Update Browserslist

repo: takagiyuuki/notion-blog


text:
Fix issue where afterSuite runs for non-root suites

Just wanted to confirm that this has resolved the issue. (No failures on CI/CD since I updated the version 12 hours ago.) 👏

authors: lukas-mercari takahirom

repo: takahirom/roborazzi


text:
Tap 2356 alarm with webhook

SonarQube Quality Gate Status: ERROR

Status: ERROR, MetricKey: new_coverage, ActualValue: 0.0

See Sonar Scan Result at: http://58.251.34.123:29000/dashboard?branch=TAP-2356-alarm-with-webhook&id=tapdata

SonarQube Quality Gate Status: ERROR

Status: ERROR, MetricKey: new_coverage, ActualValue: 0.0

See Sonar Scan Result at: http://58.251.34.123:29000/dashboard?branch=TAP-2356-alarm-with-webhook&id=tapdata

SonarQube Quality Gate Status: ERROR

Status: ERROR, MetricKey: new_coverage, ActualValue: 73.52941176470588
Status: ERROR, MetricKey: new_duplicated_lines_density, ActualValue: 6.7164179104477615
Status: ERROR, MetricKey: new_major_violations, ActualValue: 2

See Sonar Scan Result at: http://58.251.34.123:29000/dashboard?branch=TAP-2356-alarm-with-webhook&id=tapdata

另外，测试中没有看到如果调用失败，是怎么表现的，是否有比较好的保护机制

另外，测试中没有看到如果调用失败，是怎么表现的，是否有比较好的保护机制
WebHook是通过CompletableFuture异步发送的com.tapdata.tm.webhook.work.WebHookAdapter#sendAsync， WEBHook是使用HTTP POST形式发送的，默认超时3000毫秒，可以通过yaml配置（webhook.http.connectionRequestTimeout、webhook.http.connectTimout）

请确认一下，webhook功能是否需要闭源，如果需要，则开源仓库需要补充默认空实现，并在应该的接口方法给出固定报错。如果不用闭源，则实现可以移到开源仓库
WebHooK和其他告警保持一致，现在主逻辑保留在闭源里

jira中需要补充测试截图
已补充

针对开闭源，在tm中，之前有个固定约定好的错误码，这个前端才识别
throw new BizException("TapOssNonSupportFunctionException");

已补充

repo: tapdata/tapdata


text:
Add a test to compare Wx with BBox urx - llx

disregard


text:
fix: modify lc mnlp sampling

Codecov Report

Merging #6 (c963d2e) into main (0082f7e) will decrease coverage by 3.32%.
The diff coverage is 78.94%.

@@            Coverage Diff             @@
##             main       #6      +/-   ##
==========================================
- Coverage   73.47%   70.14%   -3.33%
==========================================
  Files           9       11       +2
  Lines         475      556      +81
==========================================
+ Hits          349      390      +41
- Misses        126      166      +40

Impacted Files
Coverage Δ

seqal/tagger.py
61.01% <61.01%> (ø)

seqal/query_strategies.py
45.96% <91.66%> (-1.69%)
:arrow_down:

seqal/active_learner.py
61.29% <100.00%> (ø)

tests/conftest.py
71.69% <100.00%> (-16.04%)
:arrow_down:

tests/test_query_strategies.py
100.00% <100.00%> (ø)

tests/test_tagger.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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

authors: BrambleXu codecov-commenter

repo: tech-sketch/SeqAL


text:
Add classes to Card to Header

Current dependencies on/for this PR:

#268  👈
main

This stack of pull requests is managed by Graphite.

repo: techniq/svelte-ux


text:
Amended authentication and usage base

this PR has been approved. You can merge it

repo: teejlab/API-Risk-Assessment-Framework


text:
[Llama2] Update README to reflect new pytest

@cglagovichTT @caixunshiren @cglagovich

repo: tenstorrent-metal/tt-metal


text:
#4732: Clean up compute kernel apis

Got a green pipeline here: https://github.com/tenstorrent-metal/tt-metal/actions/runs/7891088363
will merge soon.


text:
#1326: Add checks that kernels are not placed on storage cores / dispatch cores in debug mode

Does this check work if we are dispatching from idle eth?

authors: abhullar-tt tt-aho

repo: tenstorrent/tt-metal


text:
Create Bicycle.php

 Quality gate: Failed
1 Issues

 1 Code issues

Details


text:
How to disable disable dev tool if you want to avoid disable dev tool itself

Do you mean this: @KorSESNOR

authors: KorSESNOR theajack

repo: theajack/disable-devtool


text:
Wallet Login

Archive duplciate

repo: thematters/matters-server


text:
Create trivial node/browser implementation.

Also completed Python3 implementation to help check JS implementation.


text:
RE Processing Strategy doc improvements

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.5 out of 6 committers have signed the CLA.:white_check_mark: stitenko:white_check_mark: samson0v:white_check_mark: Ultrazombie:white_check_mark: ashvayka:white_check_mark: smatvienko-tb:x: mykolaichukAlexanderYou have signed the CLA already but the status is still pending? Let us recheck it.

authors: CLAassistant ashvayka


text:
refactor: Release to Publish

@shift4id can you add 2 changesets (via yarn changeset)

both can be patch
CLI changeset should explain that release is now publish and release is deprecated
sdk changeset should just mention that release is now called publish (since it doesn't seem there's any functional changes)

/release-pr

/release-pr

/release-pr

repo: thirdweb-dev/js


text:
update signer correctly

/release-pr


text:
fix: deadlock in recv_page callers

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#372  👈
#371
master

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @rphmeier and the rest of your teammates on  Graphite

Merge activity

Aug 30, 12:01 PM EDT: @pepyakin started a stack merge that includes this pull request via Graphite.

authors: pepyakin rphmeier

repo: thrumdev/nomt


text:
Added configuration default values

:+1:

authors: Bladrak masom

repo: thumbor-community/shortener


text:
linked account with the game, also added history

You might have to update checkstyles but this looks good. We finally saved the history so this is a great start.

Ohh I forgot we have to make sure the session ends when we click our log out button. We haven't done that yet.


text:
Implement command line arguments for server configuration

Ready to be reviewed

repo: timokau/prefq


text:
Update Python version to 3.10

I have to upgrade Canavs Oauth library to v1.1.1: https://github.com/Harvard-University-iCommons/django-canvas-oauth/archive/v1.1.1.tar.gz
Otherwise the the Oauth workflow doesn't work. So I will add that with that too with the PR

Also I had issue loading the LTI on iframe Refused to display 'https://pu-clrt.loophole.site/' in a frame because it set 'X-Frame-Options' to 'deny'."
as a workaround for launching the tool I did this X_FRAME_OPTIONS = 'ALLOW-FROM https://pu-clrt.loophole.site'
But I believe CSP setup should take care of this. Investigating now. After that I will open a PR

repo: tl-its-umich-edu/canvas-app-explorer


text:
프론트 - 재로그인시 profile 페이지로 이동하는 문제

이슈 작성하면서 commit했더니 반영 안된듯..
33f756f1a0d80cb3f238e5cf715f3e3ca0d0c94c

repo: troymerai/COSMIC-tryon


text:
Store test results in metamodel

Done via 92e98d8


text:
test

There is a structure for storing the data of the player (see high level design features)


text:
WIP Dst 396/view a breach page

Codecov Report
Attention: Patch coverage is 72.72727% with 18 lines in your changes are missing coverage. Please review.

Project coverage is 79.45%. Comparing base (96c34ce) to head (410aad8).

Files
Patch %
Lines

django_app/view_a_suspected_breach/views.py
69.04%
13 Missing :warning:

django_app/report_a_suspected_breach/views.py
50.00%
3 Missing :warning:

django_app/utils/s3.py
77.77%
2 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #99      +/-   ##
==========================================
- Coverage   79.83%   79.45%   -0.38%
==========================================
  Files          61       61
  Lines        1820     1874      +54
==========================================
+ Hits         1453     1489      +36
- Misses        367      385      +18

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

authors: codecov-commenter morganmaerees

repo: uktrade/report-a-breach


text:
Add font pairings

 AAI I want to select other fonts posted to the site that pair well with my font.

repo: una-ada/Bopomo


text:
C sharp

Merge this

repo: uriegel/letsencrypt-cert


text:
使い方のリンクの追加、注意書きの項目の削除

完了完了

repo: uriuriboo/Quiz-Scoring-Machine


text:
FindPlace Module

merge on success

repo: urmichm/diana-android


text:
ディレクトリ直下のsubmoduleのkeyboard-layout-makerは使われていないので、削除

Fixed with #61

Fixed with #61

repo: ut-code/typing-game


text:
dd/chroot: improve the skip message (not root)

trival fix, auto approving

repo: uutils/coreutils


text:
Develop

Pull Request Test Coverage Report for Build 8457978637
Details

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

Totals

Change from base Build 8427932840:
0.0%

Covered Lines:
1981

Relevant Lines:
3137

💛  - Coveralls

authors: coveralls jlaney

repo: uw-it-aca/compass


text:
APM-20: Payment service ref to digital wallet resource

пайп пока не подтянул

repo: valitydev/fistful-server


text:
Archeological site 11

Hello @valleyofdawn
Thank you for your icon submissions!
archeological-site icon feedback:
Overall, I think your icon concept works, since a 'vase' is globally recognized, however at 100% scale, the vase's silhouette might be too generic of a shape. If you have any other icon concepts that are both global and unique in shape, please consider them.

Your 11-pixel icon is not at the correct size

Pixel alignment issues

Thanks for the feedback.
Before correcting the 11 px image, let's discuss the concept.
I can add one handle, to make it less generic. Might be a little busy
though.
As an alternative, how about a Greek column heading, along these lines:
[image: Inline image 1] (this one is borrowed)

@valleyofdawn
Adding a handle sounds good.

Okay,
I sent out a version with an handle for the 15 px icon. Tell me if you like
it and I'll work on the 11 px.
Guy

authors: natslaughter valleyofdawn

repo: valleyofdawn/maki


text:
Use contexts in place of builder prop drilling

Many places in the application use prop drilling to pass the values, we need to change this to make use of Context API
To explore one such example view ToolbarItem component

repo: vedant-pandey/fralo-tech


text:
include options in template

Changelog

one template for data and metadata
reading csv files now uses the same read function
options (wide, stationname, timezone) etc can be added to the template
template prompt function now uses this structure (putting options in the template file)


text:
fix: api tags serialisation

🧙♂

authors: cvmcalister mieliespoor


text:
update importer version

There is another pr repalced.

repo: vesoft-inc/nebula-studio


text:
send the discord users somewhere else

but muh new blood

authors: nikaidou-agate watterle

repo: vg-mjg/mjg-repo


text:
Statz: Add a counter wrapper with 5 labels

surprised we can't use variadic functions here and potentially a generic return type

authors: ohEmily stevebriskin

repo: viamrobotics/goutils


text:
docs: add PyPi badge

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 88.75%. Comparing base (9099b22) to head (72a9947).

Additional details and impacted files
@@           Coverage Diff           @@
##           master      #35   +/-   ##
=======================================
  Coverage   88.75%   88.75%
=======================================
  Files           4        4
  Lines          80       80
  Branches       11       11
=======================================
  Hits           71       71
  Misses          8        8
  Partials        1        1

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


text:
Fix #4440, #3549 - Request ignores false, 0 and '' as body values (v0.x)

Wrong repo :)

repo: visortelle/axios


text:
chore(deps): use esno to replace ts-node

You might want to remove ts-node from devDeps and renovate.json5.

need to remove:

https://github.com/vitejs/vite/blob/dcc8ea43d079c9bc3f49f758a80a3a74189f333a/package.json#L82
https://github.com/vitejs/vite/blob/dcc8ea43d079c9bc3f49f758a80a3a74189f333a/.github/renovate.json5#L19
https://github.com/vitejs/vite/blob/dcc8ea43d079c9bc3f49f758a80a3a74189f333a/packages/vite/rollup.config.js#L175 ?

Looks like this error is happening frequently.
Error: ENOENT: no such file or directory, open '/tmp/esbuild-kit/16524-74c8df34597ce71639eb861c859a7174e79e54b9'
https://github.com/vitejs/vite/actions/runs/2317845205/attempts/1
https://github.com/vitejs/vite/actions/runs/2317959186

Revert for now. /cc @privatenumber any clue on the cause of this?

Fixed in https://github.com/esbuild-kit/core-utils/pull/1

Hi, any chance someone could explain the reason for this change, or link to an issue/discussion where this was talked about?  I always find it very useful when the context/reasoning behind a change are included in the PR.  I'm not saying the change shouldn't have been made, I'm just trying to learn / follow along.

authors: IanVS aladdin-add antfu privatenumber sapphi-red

repo: vitejs/vite


text:
Add a baseline with dynamic growing KV cache size for the paper

@zhuohan123 is this still something you're interested in doing?

repo: vllm-project/vllm


text:
types: improve index key types

What problem does this solve?

It's actually a simple refactoring. keyof any equals string | number | symbol

Why not use PropertyKey type instead?

Good idea, TIL!

authors: liuseen-l pikax sxzz

repo: vuejs/core


text:
support bg color

Does this PR ready for merging (after rebasing)? Anything you plan to work on? @wanmeihuali

Does this PR ready for merging (after rebasing)? Anything you plan to work on? @wanmeihuali

The previous experiment does not show any significant improvement with background color...Actually, the PSNR even dropped a bit... I'm not sure if it will get better after rebasing. Anyway, we will only merge it if it does not hurt the metrics.

I prefer to use "black" background (0, 0, 0) as the default value. But at the render time, one can choose to switch between black or white.

authors: jb-ye wanmeihuali


text:
Release v0.10.3

/build

/build

/build

/build

/build

/build

/build

/build

/build

/build

/build

For testing.

/build

repo: warm-metal/buildkit


text:
Add support for autocompletes without scrollable container

Should work in latest release


text:
[5.0pts] Navigation Panel

In the navigation panel, the user can see links to:

Motorcycles/doctors/classes/items that you selected as a theme.
"Reserve" form.
"My reservations".
"Add motorcycle/doctor/class/item that you selected as a theme" (the link is visible to everybody).
"Delete motorcycle/doctor/class/item that you selected as a theme" (the link is visible to everybody).


text:
Welcome workflow

Welcome to the repository!


text:
Changelog for v2.7.2

@qwerty287 yes but it just adds a new debug line to acktually tackle down (config-) bugs (actual users had in prod) ...
... so we should release it for everyone but I'm not sure if it is even worth the changelog ...
If this is blocking I just make it not listed in the changelog ...

@qwerty287 did that address your concerns?

No. If you think this really should go into a patch release, I won't block it, but please add it to the changelog.

We can just release this as 2.8.0?

I would also like to include https://github.com/woodpecker-ci/woodpecker/pull/3348. As explained here, our current approach to limiting secret access and protecting through plugin image filters is effective but somewhat prone to occasional issues. Introducing a system that requires approvals for executing potentially risky pipelines - such as forked pull requests - beforehand should greatly reduce these risks.

Well i also want the batching of loglines also released asap ... so i think we can do v2.8.0 soon?

@6543  13:00? => https://github.com/woodpecker-ci/woodpecker/pull/4291

repo: woodpecker-ci/woodpecker


text:
Implement proper git hub actions

Closed due to the discontinuity of the project


text:
Add B2B organization management related resources to the product

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/5762554004

authors: jenkins-is-staging sadilchamishka


text:
Bump oauth and framework version

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/12406155972

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/12406155972
Status: failure

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/12408711862

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/12408711862
Status: failure

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/12408711862

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/12408711862
Status: failure

authors: jenkins-is-staging shashimalcse


text:
Added German language translation file

Again, thanks a lot for contributing! <3

@all-contributors please add @realZachi for translation

authors: realZachi xEdziu

repo: xEdziu/KeepMyPassword-Desktop


text:
Hi, a drive video, how do I forward pass multiple input images for reconstruction?

You can simply put all images in one input directory, there are examples in the resources.

authors: xg-chu yuuhaixia

repo: xg-chu/GPAvatar


text:
Update _config.yml

Minimal theme


text:
Test creating a new user with special characters in the name.

done

repo: yamamoto-tsunetomo/MediAdmin


text:
add go fork

@longbai hi
试了一下有几个小的建议

目前本机有 env proxy 会报 tls error
最好支持直接读 env 的 OPENAI_API_KEY~
好像可以直接利用 GitHub Actions 发 release
好像 go mod 在我环境有点问题，我这么改通过了

@longbai
感谢，我先合了哈


text:
Implement paginations show information

#2

repo: ynhhoJ/flibusta-api


text:
hwmv2: Fix rpi_pico w variant

@tejlmand
Thank you for your comment. It is true, and the w has no LED that is connected to standard GPIO.
I also agree with excluding platform integration.
Overlays that delete nodes are certainly difficult to understand, so I will consider them separately.

I think the platform_exclude solution is confusing (and a hack), specially because there's a dt based filter in place.

@gmarull a followup task has been created by @nordicjm here: #51831.
Please approved, so that porting work can continue.

@gmarull a followup task has been created by @nordicjm here: #51831.
where?

@gmarull a followup task has been created by @nordicjm here: #51831.

where?

Issue with base board name overlay files now being used on boards which are variants that in hwmv1 were not applied to, causing issues and failing tests https://github.com/zephyrproject-rtos/zephyr/pull/69025

authors: gmarull nordicjm soburi tejlmand

repo: zephyrproject-rtos/zephyr


text:
match items by HNSW

Codecov Report

Merging #368 (261e79c) into master (5551305) will decrease coverage by 0.08%.
The diff coverage is 83.07%.

@@            Coverage Diff             @@
##           master     #368      +/-   ##
==========================================
- Coverage   73.73%   73.64%   -0.09%
==========================================
  Files          50       50
  Lines        9985     9984       -1
==========================================
- Hits         7362     7353       -9
- Misses       1864     1869       +5
- Partials      759      762       +3

Impacted Files
Coverage Δ

base/floats/floats.go
100.00% <ø> (ø)

model/built_in.go
43.58% <ø> (ø)

worker/worker.go
63.14% <72.50%> (-0.19%)
:arrow_down:

config/config.go
89.24% <100.00%> (+0.42%)
:arrow_up:

master/tasks.go
58.47% <100.00%> (+0.61%)
:arrow_up:

model/ranking/evaluator.go
93.97% <100.00%> (ø)

Continue to review full report at Codecov.

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

authors: codecov-commenter zhenghaoz

repo: zhenghaoz/gorse


text:
Publish Docs to The NPM Registry

This is ready for review.

repo: zio/zio-actors


text:
feat(config): add Zooz ZEN53, 54, 55

@zwave-js-bot automerge

repo: zwave-js/node-zwave-js


text:
style: fix typos

Pull Request Test Coverage Report for Build 4845661764

0 of 1 (0.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 23.416%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

lib/ZwaveClient.ts
0
1
0.0%

Totals

Change from base Build 4809842701:
0.0%

Covered Lines:
3818

Relevant Lines:
17225

💛  - Coveralls

authors: bradfitz coveralls

repo: zwave-js/zwave-js-ui


text:
Updated quill version to 1.2.5

Please release this version


text:
Router.push产生两条路由记录

能把bug说详细一点吗

commit error


text:
Create my-slide

great


text:
버전에 스킨 버전도 같이 표시해주기

다음 주 할 일

완료


text:
Merging just so I got the newest files / additions from the upstream in my fork

No files where changed that collide with changes of my own, able to merge automatically so I will do so.


text:
Update pic for gui

Codecov Report

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

@@            Coverage Diff            @@
##             master     #176   +/-   ##
=========================================
  Coverage     52.45%   52.45%
  Complexity      569      569
=========================================
  Files           114      114
  Lines          2524     2524
  Branches        309      309
=========================================
  Hits           1324     1324
  Misses         1108     1108
  Partials         92       92

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 c72297d...10de0ff. Read the comment docs.

authors: andreatanky codecov-io


text:
Feature/db config support

+1

authors: mattohope mbrehovsky

repo: Adaptavist/avst-wizard


text:
Wallet address

No description provided.

repo: AgenorCore/Agenor


text:
Entity模板中属性重复类型

早上我也发现了这个问题，已处理，下次合并你的吧。。。。感谢你的参与！！！

authors: Ahoo-Wang MetSystem

repo: Ahoo-Wang/SmartCode


text:
Added support for hyphens in the URL path

This is great, thanks a lot!
Released in v2.4.3.

authors: Ajaxy viniciuspinto

repo: Ajaxy/tinyspec


text:
update from upstream

Build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/6780/
Test FAILed.

I'm going to close this PR as it doesn't have any information about the change and has not had any activity recently.

Please re-open if otherwise

authors: AmplabJenkins ZacBlanco shangqiao

repo: Alluxio/alluxio


text:
Block performance related changes.

@apc999 @gpang
I don't know why some unrelated commits are shown in the commits history. I have checked the actual diffs, they look expected to me.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/9452/
Test PASSed.

Merged build finished. Test PASSed.

+1 LGTM


text:
Check for null TtlAction in DefaultFileSystemMaster

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/4654/
Test FAILed.

Jenkins, test this please

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/4656/
Test FAILed.

Jenkins, test this please

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/4662/
Test FAILed.

Failed tests:
  FileSystemIntegrationTest.testTtlActionSetAttribute:345 TTL action should be same expected:<FREE> but was:<DELETE>

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/4664/
Test PASSed.

@gpang test fixed, PTAL. thanks

alluxio-bot, merge this please

authors: AmplabJenkins gpang yuzhu


text:
Updated usage example

Nice catch!

authors: PanosSakkos ranunez

repo: Alua-Kinzhebayeva/iOS-PDF-Reader


text:
Refactor code

Amazing!

repo: AndreaCatania/godot


text:
[WIP] Drivers : unify the radar proto output

@eugenegx  please also review

repo: ApolloAuto/apollo


text:
Adding podspec and fix bug that causes crash when accessing compatibleIOSVersions

Pushed more changes than I meant to actually (forgot it's master -> master), will open another PR.

:+1: thanks so much for this PR!

I see re: fetchBuild, that's a good point. Sounds good.

authors: arbesfeld tonyxiao

repo: AppHubPlatform/apphub-ios


text:
Channel Updates : Database and Server Code Updates PART 2

9eb9ca4 : This issue Contains insertion of DB when user is added by other group

repo: AppLozic/Applozic-iOS-SDK


text:
Adjustable max appservers

OCD build https://ocd.appscale.com:8080/job/Daily Build/4808/

repo: AppScale/appscale


text:
Only list workshop in sidebar when accessible

Thanks! 😄

authors: nWidart pepijnolivier

repo: AsgardCms/Workshop


text:
adding grouping

Docs Build status updates of commit 119b0e0:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

mapping/groupMapping-2.0-preview.json
: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.

authors: VSC-Service-Account billmath

repo: Azure/azure-docs-powershell-azuread


text:
Patch spring link

:x: Validation status: errors

File
Status
Preview URL
Details

:x:Error

Details

[Error] Cannot sync git repo to specified state. Please make sure the pull request is mergeable. Details: Git merge results in conflict, repository remote url: https://github.com/Azure/azure-docs-sdk-java, working directory: W:\vgde-s, target branch: master, target branch commit: 919d09fa1ed61922b70b0ad8e80307c4f8b555e2, source branch: c1ed6343fd1ea0e5338a892c47c110a2/patch-spring-link, source branch commit: .

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.

authors: brunoborges openpublishbuild

repo: Azure/azure-docs-sdk-java


text:
[BUG] Sometimes in the release a second set app settings is created without any associated app settings

Fixed.

repo: Azure/azure-functions-devops-build


text:
Eventhubs SDK move over

Can one of the admins verify this patch?

This PR is no longer considerable

authors: azuresdkci mssfang

repo: Azure/azure-sdk-for-java


text:
[DO NOT MERGE]stub file for API review

Can one of the admins verify this patch?

@xiangyan99 closing since I think it's done, feel free to re-open if necessary

repo: Azure/azure-sdk-for-python


text:
change version

Can one of the admins verify this patch?


text:
Create .NETMF version of gateway

Not relevant anymore. Reopen if needed

authors: olivierbloch spyrossak

repo: Azure/connectthedots


text:
added LARAVEL_CURRENT_LATEST_VERSION_DEV

You can release it

authors: RichardDominik palypster

repo: BRACKETS-by-TRIAD/craftable-installer


text:
First of all, thanks for your nice work.

First of all, thanks for your nice work.
I have one question on the Synapse datasets. Did you only train on the eight organs or train on the thirteen organs but only report the metrics on the eight organs?

repo: Beckschen/TransUNet


text:
Some Styling Changes

Looks great however there seems to be some merge conflicts probably because I merged some pull requests before this one. If you have time to fix it or make a new pull requests I'll happily merge it :)


text:
Update codecov bash script URL

Current coverage is 100.0%

Merging #12 into master will change coverage by 0% by 7684cb1

Coverage Diff
@@            master     #12   diff @@
======================================
  Files            1       1
  Stmts           85      85
  Branches         0       0
  Methods          0       0
======================================
  Hit             85      85
  Partial          0       0
  Missed           0       0

Powered by Codecov

Current coverage is 100.0%

Merging #12 into master will change coverage by 0% by 801b7f2

Coverage Diff
@@            master     #12   diff @@
======================================
  Files            1       1
  Stmts           85      85
  Branches         0       0
  Methods          0       0
======================================
  Hit             85      85
  Partial          0       0
  Missed           0       0

Powered by Codecov

Current coverage is 100.0%

Merging #12 into master will change coverage by 0% by 57ab941

Coverage Diff
@@            master     #12   diff @@
======================================
  Files            1       1
  Stmts           85      85
  Branches         0       0
  Methods          0       0
======================================
  Hit             85      85
  Partial          0       0
  Missed           0       0

Powered by Codecov

Current coverage is 100.0%

Merging #12 into master will change coverage by 0% by 8c1b753

Coverage Diff
@@            master     #12   diff @@
======================================
  Files            1       1
  Stmts           85      85
  Branches         0       0
  Methods          0       0
======================================
  Hit             85      85
  Partial          0       0
  Missed           0       0

Powered by Codecov

Current coverage is 100.0%

Merging #12 into master will change coverage by 0% by c1b2fd4

Coverage Diff
@@            master     #12   diff @@
======================================
  Files            1       1
  Stmts           85      85
  Branches         0       0
  Methods          0       0
======================================
  Hit             85      85
  Partial          0       0
  Missed           0       0

Powered by Codecov

Current coverage is 100.0%

Merging #12 into master will change coverage by 0% by 3a2c367

Coverage Diff
@@            master     #12   diff @@
======================================
  Files            1       1
  Stmts           85      85
  Branches         0       0
  Methods          0       0
======================================
  Hit             85      85
  Partial          0       0
  Missed           0       0

Powered by Codecov

authors: RyanCopley codecov-io

repo: BlockSync/BlockSync


text:
Portfolio

Coverage remained the same at 82.567% when pulling f5699527dd4658881041724c022ccaddfa4028a2 on hj2304:portfolio into 841ddc392ab9224c914ab756cf0671e952ad3682 on CS2103AUG2017-F10-B2:master.


text:
[Philemon]v1.5 UG page breaks

Coverage remained the same at 82.848% when pulling 890a984a5297bf567e8c7c44ecdbf2e3b9a2421e on philemontan:v1.5ugpagebreak into eb9da692a6625696ae2d67f21c5036d5bbc9608d on CS2103AUG2017-T11-B1:master.

authors: coveralls philemontan


text:
Fix some style issues for dg

thanks!


text:
[Cli] command to start standalone and bundled validator

duplicate of #172

repo: ChainSafe/lodestar


text:
✨ Add OPcache extension to PHP 5.4 images

Hi @nstCactus , thanks a lot for your contribution!
Everything looks alright to me. The build is failing, but that's due to a problem that hasn't anything to do with your changes, and I'll try to solve that in a separate PR.
Could you please update tests as well? You might take a look at Makefile: adding OPCache to EXTENSIONS list (lines 10-27) and removing the now-useless condition (lines 36-39) should be enough. Then do the same for dev/Makefile.
If you don't have time to do that, no worries: I can take care of it.

Thanks!

authors: fquffio nstCactus

repo: Chialab/docker-php


text:
Deleted provider info

Closing this PR, as the template is supporting TFv0.12 or more, hence the block to download the Terraform provider is needed.

authors: VaishnaviGopal geethasathyamurthy

repo: Cloud-Schematics/LEMP


text:
Fix typo

@MikeBild BTW, it seems, the right name of this API is Server-Sent Events, not Server-Side-Events (proof: https://html.spec.whatwg.org/multipage/server-sent-events.html#server-sent-events). :wink:

Thx!

authors: FluorescentHallucinogen MikeBild


text:
change title on README

Looks good.


text:
Update and greatly improve the schedule code.

Codecov Report

Merging #22 into master will increase coverage by 10.83%.
The diff coverage is 81.31%.

@@            Coverage Diff             @@
##           master     #22       +/-   ##
==========================================
+ Coverage   61.66%   72.5%   +10.83%
==========================================
  Files           4       5        +1
  Lines         240     280       +40
  Branches       43      52        +9
==========================================
+ Hits          148     203       +55
+ Misses         61      48       -13
+ Partials       31      29        -2

Impacted Files
Coverage Δ

src/schedule.c
85.15% <78.68%> (+17.35%)
:arrow_up:

src/schedule_print.c
86.66% <86.66%> (ø)

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 1dcd4db...56bb098. Read the comment docs.

authors: codecov-io schmidtw

repo: Comcast/aker


text:
PR: groups files per OS

Hello @goanpeca, Thank you for updating !

In the file setup.py, following are the PEP8 issues :

Line 45:12: E251 unexpected spaces around keyword / parameter equals
Line 45:14: E251 unexpected spaces around keyword / parameter equals
Line 46:13: E251 unexpected spaces around keyword / parameter equals
Line 46:15: E251 unexpected spaces around keyword / parameter equals

Codecov Report

Merging #52 into master will not change coverage.
The diff coverage is 0%.

@@         Coverage Diff          @@
##           master   #52   +/-   ##
====================================
  Coverage       0%    0%
====================================
  Files          11    14    +3
  Lines         982   994   +12
====================================
- Misses        982   994   +12

Impacted Files
Coverage Δ

menuinst/windows/win_elevate.py
0% <ø> (ø)

menuinst/windows/knownfolders.py
0% <ø> (ø)

menuinst/linux/freedesktop.py
0% <ø> (ø)

menuinst/api.py
0% <0%> (ø)
:arrow_up:

menuinst/osx/darwin.py
0% <0%> (ø)

menuinst/windows/__init__.py
0% <0%> (ø)

menuinst/linux/__init__.py
0% <0%> (ø)

menuinst/osx/__init__.py
0% <0%> (ø)

menuinst/linux/linux.py
0% <0%> (ø)

menuinst/windows/win32.py
0% <0%> (ø)

... and 4 more

Continue to review full report at Codecov.

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

Codecov Report

Merging #52 into master will not change coverage.
The diff coverage is 0%.

@@         Coverage Diff          @@
##           master   #52   +/-   ##
====================================
  Coverage       0%    0%
====================================
  Files          11    14    +3
  Lines         982   994   +12
====================================
- Misses        982   994   +12

Impacted Files
Coverage Δ

menuinst/linux/freedesktop.py
0% <ø> (ø)

menuinst/windows/win_elevate.py
0% <ø> (ø)

menuinst/windows/knownfolders.py
0% <ø> (ø)

menuinst/osx/__init__.py
0% <0%> (ø)

menuinst/linux/__init__.py
0% <0%> (ø)

menuinst/windows/win32.py
0% <0%> (ø)

menuinst/linux/linux.py
0% <0%> (ø)

menuinst/api.py
0% <0%> (ø)
:arrow_up:

menuinst/osx/darwin.py
0% <0%> (ø)

menuinst/windows/__init__.py
0% <0%> (ø)

... and 4 more

Continue to review full report at Codecov.

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

Codecov Report

Merging #52 into master will not change coverage.
The diff coverage is 0%.

@@         Coverage Diff          @@
##           master   #52   +/-   ##
====================================
  Coverage       0%    0%
====================================
  Files          11    14    +3
  Lines         982   994   +12
====================================
- Misses        982   994   +12

Impacted Files
Coverage Δ

menuinst/windows/win_elevate.py
0% <ø> (ø)

menuinst/linux/freedesktop.py
0% <ø> (ø)

menuinst/windows/knownfolders.py
0% <ø> (ø)

menuinst/osx/darwin.py
0% <0%> (ø)

menuinst/osx/__init__.py
0% <0%> (ø)

menuinst/api.py
0% <0%> (ø)
:arrow_up:

menuinst/windows/win32.py
0% <0%> (ø)

menuinst/linux/linux.py
0% <0%> (ø)

menuinst/linux/__init__.py
0% <0%> (ø)

menuinst/windows/__init__.py
0% <0%> (ø)

... and 4 more

Continue to review full report at Codecov.

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

Codecov Report

Merging #52 into master will not change coverage.
The diff coverage is 0%.

@@         Coverage Diff          @@
##           master   #52   +/-   ##
====================================
  Coverage       0%    0%
====================================
  Files          11    14    +3
  Lines         982   994   +12
====================================
- Misses        982   994   +12

Impacted Files
Coverage Δ

menuinst/windows/win_elevate.py
0% <ø> (ø)

menuinst/windows/knownfolders.py
0% <ø> (ø)

menuinst/linux/freedesktop.py
0% <ø> (ø)

menuinst/__init__.py
0% <0%> (ø)
:arrow_up:

menuinst/linux/linux.py
0% <0%> (ø)

menuinst/windows/__init__.py
0% <0%> (ø)

menuinst/osx/darwin.py
0% <0%> (ø)

menuinst/windows/win32.py
0% <0%> (ø)

menuinst/api.py
0% <0%> (ø)
:arrow_up:

menuinst/osx/__init__.py
0% <0%> (ø)

... and 4 more

Continue to review full report at Codecov.

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

authors: codecov-io goanpeca pep8speaks

repo: ContinuumIO/menuinst


text:
closes #610

Codecov Report

Merging #611 into master will decrease coverage by <.01%.
The diff coverage is 88.88%.

@@            Coverage Diff             @@
##           master     #611      +/-   ##
==========================================
- Coverage   89.99%   89.98%   -0.01%
==========================================
  Files          41       41
  Lines        3179     3186       +7
==========================================
+ Hits         2861     2867       +6
- Misses        318      319       +1

Impacted Files
Coverage Δ

vent/helpers/meta.py
90.69% <88.88%> (-0.22%)
: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 4934c99...9468579. Read the comment docs.

authors: cglewis codecov-io

repo: CyberReboot/vent


text:
参照demo使用loadmore，无限循环加载更多 版本2.9.30

设置loadmore事件：
mAdapter.setOnLoadMoreListener(new BaseQuickAdapter.RequestLoadMoreListener() {
@Override
public void onLoadMoreRequested() {
pageNum++;
getIncomeList(false);
}
},mRecyclerView);
请求成功回调代码处理：
if (isRefresh) {
if (mAdapter != null) {
mAdapter.getData().clear();
}
isRefresh = false;
mAdapter.setNewData(incomeEntity.getvOrderList());
mAdapter.disableLoadMoreIfNotFullPage();
}else{
mAdapter.addData(incomeEntity.getvOrderList());
}
// setEmptyViewAndListener（）里代码调用了notifyItemChanged(0);
setEmptyViewAndListener(0, false, R.string.no_route,mRecyclerView, null);
            if(orderCount==mAdapter.getData().size()){
                mAdapter.loadMoreEnd();
            }else{
                mAdapter.loadMoreComplete();
            }

mAdapter.loadMoreEnd(true);
必须传true可以 不传不行   默认false

repo: CymChad/BaseRecyclerViewAdapterHelper


text:
Update gems

Review app deployed to https://teaching-vacancies-review-pr-3399.london.cloudapps.digital

authors: cesidio twd-tv-ci

repo: DFE-Digital/teaching-vacancies


text:
added feature for blacklisted Class

Thanks for resolving those issues, the test looks good to me.
I don't mind the whitespace formatting changes (as I can ignore it via Github's UI), but can you change the "curly brackets with new line" style to "curly brackets on same line" in Visual Studio and reformat the files that were changed to match the style of the project?
I believe you can do this via: https://stackoverflow.com/questions/39561/visual-studio-2005-2012-how-to-keep-first-curly-brace-on-same-line.
After that I'll give it a final look-through and approve.

authors: DarrenTsung MarleneMayr

repo: DarrenTsung/DTValidator