Update integrations

This is going to address some of #3221 requirements, cc @cartersocha

We should add integrations as a link on the homepage too - https://opentelemetry.io/

Not directly related to this PR but we may need to rethink the registry, adopters, and integrations pages a bit. Loose overlap that maybe could be combined?

Not directly related to this PR but we may need to rethink the registry, adopters, and integrations pages a bit. Loose overlap that maybe could be combined?

Yes, we need to rethink a lot of that. for example if you look into the vendors there are 90% backends, but there are also at least two vendors of pipelines and probably some other non-backends, which may count as integrations or "something else"

We should add integrations as a link on the homepage too - opentelemetry.io

not with this PR.

thank you @mhausenblas & @theletterf, I have applied most of your suggestions :-)

https://github.com/open-telemetry/opentelemetry.io/pull/3278/commits/5b46734114ada96c3c8bd50839ae8e8101bc5533 includes a few additional words for the documentation index:
https://deploy-preview-3278--opentelemetry.netlify.app/docs/

Not directly related to this PR but we may need to rethink the registry, adopters, and integrations pages a bit. Loose overlap that maybe could be combined?

Yes, we need to rethink a lot of that. for example if you look into the vendors there are 90% backends, but there are also at least two vendors of pipelines and probably some other non-backends, which may count as integrations or "something else"

Would this be a topic for the communications sig or another one? Happy to join and brainstorm or talk through potential re-orgs

Not directly related to this PR but we may need to rethink the registry, adopters, and integrations pages a bit. Loose overlap that maybe could be combined?

Yes, we need to rethink a lot of that. for example if you look into the vendors there are 90% backends, but there are also at least two vendors of pipelines and probably some other non-backends, which may count as integrations or "something else"

Would this be a topic for the communications sig or another one? Happy to join and brainstorm or talk through potential re-orgs

Sig Comms, yes. We have had our share of discussions on this topic already, if you review some old issues (like https://github.com/open-telemetry/opentelemetry.io/issues/2178) you will get some more context. This PR is probably not the best place to discuss this. Feel free to raise a new meta issue, we can link to existing issues from there and have a consolidated discussion. I may have the one or the other opinion as well ;-)

Small follow up PR - https://github.com/open-telemetry/opentelemetry.io/pull/3306


Contributes RFC for a Health Checker buildpack

Summary
Introduce a Health Checker buildpack. This buildpack will provide access to one or more health checkers, where a health checker is a small binary that will be installed into the container image for the purpose of validating the health of the running image.
Readable
Checklist

[ ] I have viewed, signed, and submitted the Contributor License Agreement.
[ ] I have linked issue(s) that this PR should close using keywords or the Github UI (See docs)
[ ] I have added an integration test, if necessary.
[ ] I have reviewed the styleguide for guidance on my code quality.
[ ] I'm happy with the commit history on this PR (I have rebased/squashed as needed).

Will the healthcheck thats created by this buildpack be compatible with https://docs.docker.com/engine/reference/builder/#healthcheck?

Will the healthcheck thats created by this buildpack be compatible with https://docs.docker.com/engine/reference/builder/#healthcheck?

The default implementation I built does this:

0: success - the container is healthy and ready for use
1: unhealthy - the container is not working correctly

The goal for the buildpack is that it's not tied to a particular health checker though. If mine stinks and we can add another one and users can pick. In addition, my implementation only checks HTTP. Maybe your app speaks another protocol like rsocket, so there could be a different health checker for it.
Ultimately, whatever health checkers we include should follow the Docker specification since that's the primary target for this. Kubernetes follows the same basic pattern, although any non-zero return code is a failure with K8s.

@ekcasey I have cleaned up & resolved the conversations here. I've incorporated the feedback and your comments into the RFC. The health checker configuration now all happens via env variable instead of arguments, which simplifies a few things, so that was a nice win.
I think this is ready for your re-review.

From WG 7/19/22 :
Ping @utilities-maintainers last call for votes!

2022-07-26 Working Group notes: ready to merge by @dmikusa-pivotal

@dmikusa-pivotal I noticed that this was merged with number 0000, should that be 0007?

@joshuatcasey - See https://github.com/paketo-buildpacks/rfcs/pull/226.


SEPG Automated Release - v1.1.1

SEPG Details
Release Build Link: http://sou-jenkins2.swinfra.net:8080/job/Auto-Release/46/
Build Token: github.com/CAFapi/opensuse-nodejs18-image/main/05dd0e2fe11c92a6b445d4b9a857e7f7a90fe0ad
Versioning Behaviour: Roll on to next minor
Artifacts For Release
Docker

cafapi/opensuse-nodejs18:1.1.1 (Static Tag)
cafapi/opensuse-nodejs18:1.1 (Dynamic Tag)
cafapi/opensuse-nodejs18:1 (Dynamic Tag)
cafapi/opensuse-nodejs18:latest (Dynamic Tag)

 Build Stage
Action
Status

Release Checks
completed
:heavy_check_mark:

Gather Release Data

:white_small_square:

QA Documentation Site Deployment

:white_small_square:

QA Deployment Branch Promotion

:white_small_square:

Docker Image Promotion

:white_small_square:

Maven Artifact Promotion (Closed Source)

:white_small_square:

NuGet Artifact Promotion

:white_small_square:

Artifactory Release Artifact Tagging

:white_small_square:

Generic Artifact Promotion

:white_small_square:

Maven Artifact Promotion (Open Source)

:white_small_square:

Maven Staging Repository Removal (Open Source)

:white_small_square:

Git Tagging

:white_small_square:

Git Release

:white_small_square:

Git Release Asset Deployment

:white_small_square:

Pull Request Merge

:white_small_square:

Release Branch Deletion

:white_small_square:

view the build log here


Fix rssreader sample

@asklar Any idea why it's seeing this in the lab (but not locally)?
Add-AppxPackage : Deployment failed with HRESULT: 0x80073CF3, Package failed updates, dependency or conflict
validation.
Windows cannot install package RssReader_<IP_ADDRESS>_x86__1g46j06h1cwcc because this package depends on a framework that
could not be found. Provide the framework "Microsoft.UI.Xaml.2.3" published by "CN=Microsoft Corporation, O=Microsoft
Corporation, L=Redmond, S=Washington, C=US", with neutral or x86 processor architecture and minimum version
2.31911.29002.0, along with this package to install. The frameworks with name "Microsoft.UI.Xaml.2.3" currently
installed are: {}
NOTE: For additional information, look for [ActivityId] 3503dfd4-a8bc-0001-9039-868e75acd601 in the Event Log or use


Fix result

I improve the code also. Do you think that your implementation is fastest that the current one ?
(I'm want to know because I'm totally new to D)

The updated one is fine. There might be a small difference in performance but the logic is correct now.


Update Ethereum bots to use the appropriate gas fee

The gas fee should be different per bot.

agreed @webelf101 - will you confirm that the gas fees for curve limit order and curve stop loss and curve twap are still good?
0x342b663894d93631851f351ece299bc04b69b562 (limit)
0xa0f74e52f6d1c2cb5db6bc3ec7be5b09ef008391 (stop loss)
0x8af32fbb0279b6713b628fe673ae5a5d15b6b3c6 (twap)

This is done.


Sounds causing the plugin to not work at all.

Plugin Version:

Version #3.6.13
Describe The Bug:

After I updated from 1.21.1 to 1.21.3 on paper, the sounds seem to be causing the whole plugin to stop working when someone does /wild. I get the following message as well. https://mclo.gs/aYqCpm2 - When I turn off the sounds, everything works fine but I would like to have a sound occur after the teleport is done!
How To Reproduce:

Everything is described above!
Expected Behavior:

When doing  /wild it should teleport me with a sound but it doesnt seem to do that.
Screenshots/Error Log:

https://mclo.gs/aYqCpm2
Additional Context:

Thats all!

Hello, if you are willing to test this version and report any finding, that would be very helpful. It should contain better information about the issue you are facing. Thank you!
https://github.com/TechnicallyCoded/Fork-BetterRTP/releases/download/debug/BetterRTP-3.6.13-debugpatch.jar


Add shortcut keys

User feedback: "The hotkeys you've chosen conflict with other extensions I already have. It would be nice to be able to configure them."

Fixed in 077ce86d0beeefbb77ab8f0c4f04bf9aac400cfd


Crash when inserting space with omnibox-autocomplete-filtering set

Describe the bug
When omnibox-autocomplete-filtering is changed from the default, inserting a space between text in the omnibox causes a crash.
To Reproduce
Set omnibox-autocomplete-filtering to anything other than the default.
Type some text in the omnibox, then attempt to insert a space inside that text.
For example:  testtest -> test test
Expected behavior
A space is inserted.
Screenshots
N/A
Environment (please complete the following information):

OS/Platform and version:  ArchLinux
ungoogled-chromium version:  git commit 05f3612

Additional context
This was introduced in #1146

This is caused by omitting the keyword provider.  It tries to tokenize the text using whitespace as a delimiter.
I'll submit a PR that addresses this issue shortly.


Replace Node IP to Nodename in Pod detail view

Currently we show Node IP in Pod detail view but would be more convenient to have Node Name: <name> or Node Name: <name> (<IP>).

Version - 2.2 master 1/2
Verified fixed


example is not work

this does not work as in the example, the button in the top left corner is not hidden...

Same problem using m1 using only the BDW_CUSTOM_FRAME flag.
When I add the BDW_HIDE_ON_STARTUP flag, however, nothing shows up and if I try to click the app from the dock the console says Lost connection to device.

There is a Pull-Request #184
@bitsdojo may that fix the problem?
@luo10 @mathewthe2 I use that package in combination with https://pub.dev/packages/flutter_acrylic and there is a method Window.hideWindowControls(); that does exactly that, maybe you can try that for now!?

I see such code that hide custom buttons for macOS.
@bitsdojo So, this issue isn't a bug, right?

@maxd  There's no bug. The window control buttons on macOS are "not supposed to be touched", unless you have a reason to. Either we should them as they are to stay uniform to the platform specific.


04.b-upload-metadata.md - metadata upload to CKAN

Data upload to CKAN with mdmanager does not work:
~/B2FIND-Training$ ./mdmanager.py --mode u -c fishproject -s http://localhost:8181/oai/provider --mdsubset sample_1 --mdprefix oai_dc -i <IP_ADDRESS> auth=<token>

Version:  	2.0
Run mode:   	Uploading
Start : 	2016-12-07 13:03:29

ERROR in  mdmanager	process_upload	2241 : [ERROR] The directory "/home/b2find/B2FIND-Training/oaidata/fishproject-oaidata/oai_dc/sample_1" does not exist! No files for uploading are found!
(Maybe your upload list has old items?)

The path in the script is wrong. The real path is:
b2find@ubuntu:~/B2FIND-Training$ ls /home/b2find/B2FIND-Training/oaidata/fishproject-oai_dc/sample_1/

The issue with tteh wrong path is fixed, but upload causes still some problems :-(

Fixed some other isssues with commit 9c6bb49210badba8142ccf403516f661797d4236 , for me uploading to the training instance works now :
$ ./mdmanager.py --mode u -c fishproject -s http://localhost:8181/oai/provider --mdsubset sample_1 --mdprefix oai_dc -i trng-b2find.eudat.eu
Please try it again with your CKAN instance. If it is still not working, we can do two things :

Please check the web log on your CKAN server, e.g. on trng-eudat.dkrz.de is it :
sudo tail /etc/httpd/logs/ckan_default.error.log
Let me know if you want to try it with the Training instance and I give you teh API-key for it

Still no success uploading the json documents.

Datacite ANDS (group 'datacite' )

[admincentos@b2find B2FIND-Training]$ ./mdmanager.py --mode u -c datacite -s http://localhost:8181/oai/provider --mdsubset ANDS.CENTRE-1 --mdprefix oai_dc -i <IP_ADDRESS>:8080 --auth ****
|- Run mode:   	Uploading
|- Start : 	2018-07-19 13:50:37

|- Community datacite
|- MD prefix oai_dc
|- Subset ANDS.CENTRE-1

No files uploaded

Fishproject

[admincentos@b2find B2FIND-Training]$./mdmanager.py --mode u -c fishproject -s http://localhost:8181/oai/provider --mdsubset sample_1 --mdprefix oai_dc -i <IP_ADDRESS>:8080 --auth ***
|- Run mode:   	Uploading
|- Start : 	2018-07-19 13:48:59

|- Community fishproject
|- MD prefix oai_dc
|- Subset sample_1

CRITICAL :  [CRITICAL] Program is aborted because of a critical error! Description:
CRITICAL :  Traceback (most recent call last):
 File "./mdmanager.py", line 2023, in main
   process(options,pstat)
 File "./mdmanager.py", line 2110, in process
   process_upload(UP,reqlist,options)
 File "./mdmanager.py", line 2382, in process_upload
   jsondata=UP.check(jsondata)
 File "./mdmanager.py", line 1847, in check
   raise Exception("The mandatory field '%s' is missing" % field)
Exception: The mandatory field 'title' is missing

Please try again - all :-) should be fixed now

Works, but data is not nicely represented.

Same issue as:
https://github.com/EUDAT-Training/B2FIND-Training/issues/11


Does Smsdia have dependency on AIA for RV32 ?

Hi,
When I read section 6.4 Machine Interrupt registers (mip and mie), it says:
The Smsdia extension introduces the machine supervisor domain external interrupt-pending (MSDEIP
- bit 32) field in mip and the machine supervisor domain external interrupt-enable (MSDEIE - bit 32)
field in mie.

The mip and mie in Priv. Spec. is MXLEN, which is MXLEN=32 on RV32, which only has bit 0 ~ bit 31.
AIA extends them to have high-half miph and mieh.
Does it mean that Smsdia has dependency on AIA ?

@gagachang that's correct - Smsdia is defined with reference to AIA

@rsahita Why?  Platform can not deploy multiple PLICs or CLICs?

APLIC, PLIC, and IMSIC are supported by Smsdia.

Implementing Smsdia requires implementing mieh and miph but does not require implementing AIA.

OK.
@ved-rivos Would you like to add some description about this ?
So that people can know Smsdia doesn't require AIA, but at least  mieh and miph should be implemented for RV32.

Just curious about why we don't assign interrupt number 16~23 for MSDEI.

FYI @gagachang PR #40

FYI @gagachang PR #40

Thanks! @rsahita @ved-rivos
The PR LGTM.
One last minor question.
There are still bit13 ~ bit15 for local interrupts to be assigned into mip.
Bit13 has been allocated for Sscofpmf extension.
Is there any chance to locate LSDEIP in bit14 or bit15 ?

Is there any chance to locate LSDEIP in bit14 or bit15 ?

All bit allocations are tentative and subject to change before Freeze. The 14 and 15 are currently reserved for possible other uses.

closing this as PR #40 was acked


Lightning internal refactors

🐛 Bug

To Reproduce
Steps to reproduce the behavior:

Go to '...'
Run '....'
Scroll down to '....'
See error

Code sample

Expected behavior

Environment
Please copy and paste the output from our
environment collection script
(or fill out the checklist below manually).
You can get the script and run it with:
wget https://raw.githubusercontent.com/PyTorchLightning/pytorch-lightning/master/tests/collect_env_details.py
# For security purposes, please check the contents of collect_env_details.py before running it.
python collect_env_details.py

PyTorch Version (e.g., 1.0):
OS (e.g., Linux):
How you installed PyTorch (conda, pip, source):
Build command you used (if compiling from source):
Python version:
CUDA/cuDNN version:
GPU models and configuration:
Any other relevant information:

Additional context

Complete. A few straggler files here and there, but nothing we can't solve in these weeks


[Suggest] Documentation update

Hi, Thank you for the great library.
Just for people that using AOT compilation (typically people that use angular-cli), you have to add the following code to your AppModule in order to work:
In your AppModule:
import {HighchartsStatic} from 'angular2-highcharts/dist/HighchartsService';
import {ChartModule} from "angular2-highcharts";

export function highchartsFactory() {
  return require('highcharts');
}

@NgModule({
   providers:[
   ...
   {
      provide: HighchartsStatic,
      useFactory: highchartsFactory
    },
    ....
]}

In the module where you use the chart directive:

@NgModule({
   imports: [
     ChartModule
   ],
 ]}

You could also declare as a singleton (ShareModule)
Thank you !

Is it to avoid the  ERROR in Error encountered resolving symbol values statically. Calling function 'ChartModule', function calls are not supported. ?

Yep

Yea, would be great. But HighchartsService supposed to be private part of the lib. Are there any simplier approach to achieve this ? I would change something in the lib if it necessary

@fr4ngus How would your workaround look like with multiple require?
I.e. if one would also want to use e.g. require('highcharts/highcharts-more'), require('highcharts/modules/exporting'), require('highcharts/modules/offline-exporting').

How would you load Highcharts modules with this solution? Modules like highcharts/modules/solid-gauge.

@jessenic should work, but my idea was to hide how modules initialises: module(hc). It is already implemented in the wrapper: https://github.com/gevgeny/angular2-highcharts/blob/master/src/index.ts#L25

I know this issue is marked closed but I'm still having issues on my rollup.  Here is the output:
'ChartModule' is not exported by 'node_modules\angular2-highcharts\dist\index.js'

'ChartModule' is not exported by 'node_modules\angular2-highcharts\dist\index.js'

'HighchartsStatic' is not exported by 'node_modules\angular2-highcharts\dist\HighchartsService.js'

Error: 'HighchartsStatic' is not exported by node_modules\angular2-highcharts\dist\HighchartsService.js

Im using
"@angular/compiler-cli": "2.4.4",
"rollup": "^0.41.4",
"rollup-plugin-commonjs": "^7.0.0",
"rollup-plugin-node-resolve": "^2.0.0",
"rollup-plugin-uglify": "^1.0.1",
"@types/highcharts": "4.2.37",
"angular2-highcharts": "^0.5.5",

thank you @fr4ngus. solved my "Error encountered resolving symbol values statically" problem that appeared mysteriously after an upgrade


Connector/Plugin for Azure Blob storage or Azure Data Lake (Gen 2)

It would be great if there is a feature for plugging in data from Azure Blob storage or Azure Data lake gen 2 storage containers!

Hi @girishkumaar! We have this feature request captured in our new portal - please subscribe for updates there & feel free to add additional details/context in the comments!
https://feature-requests.datahubproject.io/b/feedback/p/azure-data-lake-storage-gen2
We'll close out this issue on the GitHub side 😄


worker-pool: load based worker pool

Currently, the worker pool takes one job for each worker, sending next job only if current job completed, but since the job contains life-cycle functions, which can be async, a worker can be sent multiple awaiting jobs, which can be controlled by some parameter or option, taking maximum cpu usage of workers.

Maybe we can take a look at https://github.com/piscinajs/piscina


Add ability to override or remove Open Graph meta tag in Template

At the moment meta property="og:image" in Template has a hardcoded image path, user should be able to override the image path in the meta tag.
User need explained here:

https://github.com/alphagov/govuk_template/issues/137

Split from: https://github.com/alphagov/govuk-frontend/issues/971
Part of https://github.com/alphagov/govuk-design-system/issues/777
Who needs to know about this
Developer, Mark to review

Needed as part of https://github.com/alphagov/govuk-design-system-backlog/issues/185

Needed as part of https://github.com/alphagov/govuk-design-system-backlog/issues/185

Proposed solution
I think we should solve this with a breaking change:
{% if openGraphImageUrl %}
  <meta property="og:image" content="{{ openGraphImageUrl }}">
{% endif %}

We would only set the Open Graph image meta tag if user passes through their own absolute URL - we obviously can't know if the URL is absolute but the requirement should be made clear in the guidance. Whilst the current code allows assetUrl to fall back to using assetPath, this will only work if the assetPath happens to be an absolute URL. It's not clear this happens and going by the discussion in https://github.com/alphagov/govuk-frontend/pull/847, this was mostly implemented so that we could avoid making a breaking change.
This would also allow for the removal of the meta tag entirely if openGraphImageUrl is not set.
This would also be a good chance to make assetUrl a bit more descriptive and call it something like openGraphImageUrl.
Alternative non-breaking solution
  <meta property="og:image" content="{{ assetUrl | default('/assets') }}/{{ opengraphImagePath | default('images/govuk-opengraph-image.png') }}">

This would add a new option opengraphImagePath whilst retaining assetUrl allow the user to do one of the following options:

Set the entire absolute url eg. https://domain.com/my-images/article-image-2019.png
Set only the domain url https://domain.com, which gets combined with 'images/govuk-opengraph-image.png'
Setting assetPath to an absolute URL which cascades to assetUrl and is combined with ‘'images/govuk-opengraph-image.png’

But it feels confusing to have to use two variables to set one thing, making it harder to explain in the guidance. The naming is also a bit weird since assetUrl is not very descriptive so it wouldn't be clear how it works with something like opengraphImagePath.
This approach also wouldn't allow the removal of the meta tag entirely so we'd have to consider another boolean option that would remove the meta tag which wouldn't be ideal.

I really like your investigation and the options are super clear thanks for doing that Hanna.
I believe that GOV.UK does override this meta tag by making use of the head block since the first instance of the metatag is what is used.
You can see this change in the original GOV.UK Template project:
https://github.com/alphagov/govuk_template/commit/204ee88e2c87e7e6db2aa61bf10659c8fdee2c69
With this in mind I would suggest we close this for now and see if anyone runs into any problems with using this approach:
{% block head %}
   {{ super() }}
   <meta property="og:image" content="path/to/my/custom/image.png">
{% endblock %}

Thanks @nickcolley 👍 It's possible to include the meta tag in the head block but I think this is now slightly hacky for the current Design System since we already have a documented option assetUrl which is used to "Specify an absolute URL for the <meta property="og:image"> tag image asset." We haven't documented having to use the head block to override the meta tag there and there isn't a clear way for users to discover this option.
I think the cleanest and from the user's point of view clearest thing to do would be to do a breaking change. It would also allow for the removal of the meta tag by the user which is a requirement of this card and which can't be achieved cleanly by using the head block (unless you include the meta tag in head but leave the URL empty which feels messy and could have its own problems).

Okay I’ve tried to document this in https://github.com/alphagov/govuk-design-system/pull/1072. I still think that having two different ways of dealing with the tag (use assetUrl or set in head) depending on what you need to do could be confusing but will ask Mark to have a look at helping us to write this for the docs. We still won’t be a able to remove the Open Graph tag but let’s address that if it comes up.


Roll engine 08b660ed12e8..4208f840436e (6 commits)

https://github.com/flutter/engine/compare/08b660ed12e8...4208f840436e
git log 08b660ed12e8..4208f840436e --date=short --no-merges --format='%%ad %%ae %%s'
2018-08-08<EMAIL_ADDRESS>make ios text affinity behavior match android (flutter/engine#5971)
2018-08-08<EMAIL_ADDRESS>Roll src/third_party/skia f5402004c4a6..0c5b0b1dd692 (11 commits) (flutter/engine#5975)
2018-08-08<EMAIL_ADDRESS>Roll src/third_party/skia 4856f5fa596d..f5402004c4a6 (1 commits) (flutter/engine#5974)
2018-08-08<EMAIL_ADDRESS>Roll src/third_party/skia 20714bdf90f3..4856f5fa596d (1 commits) (flutter/engine#5973)
2018-08-07<EMAIL_ADDRESS>Roll src/third_party/skia faeef7837210..20714bdf90f3 (11 commits) (flutter/engine#5970)
2018-08-07<EMAIL_ADDRESS>Complete the AndroidView resize call only after a new frame is ready. (flutter/engine#5968)
The AutoRoll server is located here: https://flutter-engine-flutter-roll.skia.org
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md
If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.

@rmistry @liyuqian : New roll has been created by https://flutter-engine-flutter-roll.skia.org

Commit queue failed; closing this roll.


Info button does nothing (place collection builder)

While editing a Place collection, the "making a collection public" button, to the right of "Public?: False" doesn't do anything.

May get fixed in #303

Fixed with #303


How to execute raw SQL with Tx ?

How to execute raw SQL with *ent.Tx, such as TRUNCATE

Hey @shenghui0779 👋
There are a few ways how to get the dialect.Driver (or dialect.Tx) from ent.Client (or ent.Tx) and it's mentioned in https://github.com/ent/ent/issues/85. When working with a database driver (underlying connection), you can execute any statements, including TRUNCATE. Same as you can do with sql.DB.
If you work with ent.Tx or transactional client, the connection is bound to a transaction, and all queries, including raw statements, are executed on the transaction connection.
Please, let me know if it does answer your question and we can close the issue. Thanks 🙏
P.S. great work on https://github.com/shenghui0779/goadmin

Thanks a lot !

Hey @shenghui0779! I added a feature flag named sql/execquery to simplify the process for those who need it.


Travis: Simplify gopherjs invocation.

GopherJS supports relative import path patterns as of GopherJS 1.8-2. See gopherjs/gopherjs#644.

Codecov Report

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

@@           Coverage Diff           @@
##           master     #141   +/-   ##
=======================================
  Coverage   52.09%   52.09%
=======================================
  Files           5        5
  Lines         405      405
=======================================
  Hits          211      211
  Misses        178      178
  Partials       16       16

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 0cbd104...4daa394. Read the comment docs.

Awesome, thanks for this!


Dynamic phase skipping support

This patch introduces support for actors being able to let leapp skip
phases.
Phases can be skipped by an actor using leapp.messaging.commands.SkipPhasesUntilCommand
like this within the process method:
self._messaging.command(SkipPhasesUntilCommand(until_phase='report'))

This will make leapp skip phases until it hits the 'report' phase.
Signed-off-by: Vinzenz Feenstra<EMAIL_ADDRESS>

Wouldn't it be cleaner to introduce a separate module Commands instead of adding it to the Messaging module?

@bocekm We discussed better solutions, but this is the one we will use now. It's not public solution: see the underscore in the name _command. So it will not be even documented. So we can use it now and we will improve it in future when we have time.

Tested. Only issue is that it will skip even the mentioned phase. But we can use it in the repository as it is.


SimplifyCFG: fix hang caused by infinite loop in CFG

fixes SR-5764, rdar://problem/34068905

@swift-ci smoke test and merge


dynamic import fails with bytecodePlugin

Describe the bug
The bytecodePlugin worked perfectly until I added dynamic import to the code. After building the app, the dynamically imported module doesn't function as expected because, in the build result, the dynamic module requires ./index.js. However with bytecodePlugin enabled, the index.js is replaced with the bytecode-loader entry, not the original content. As a result, the dynamic module requires an empty export.
Minimal reproducible example is here, and I have committed the out dir on purpose.
Electron-Vite Version
2.3.0
Electron Version
31.0.2
Vite Version
5.3.1
Validations

[X] Follow the Code of Conduct.
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.

import module in ES syntax↳

Yes, I do use import in my code, and the requires I described before are generated in the bundle files.
You can check the dynamic module here


Support for read from Cosmos DB Core SQL API

Fix #23604
Follow-up from #23610
Implementation inspired from official azure-cosmos-spark

R: @kennknowles as you were part of the previous discussion in the linked PR

@Miuler can you give it a try in a real setup ? I've only tested with testcontainers.

@kennknowles & @Abacn can you have a look as you commented on https://github.com/apache/beam/pull/23610 ?


CIP30 support

WIP

Do you think it would be useful to keep some kind of type for WalletId type?
type WalletId
    = Eternl
    | Lace

There's a lot of wallets so it might be a pain to manage, and it might not even make sense to. I'm not sure, which is why I pose it as a question.

Do you think it would be useful to keep some kind of type for WalletId type?

I'd prefer to stay wallet-agnostic

I'd prefer to stay wallet-agnostic

Makes sense


Add vendors:reinstall and update vendors:update

I think that the default vendors:update command should call
bin/vendors install

and not
bin/vendors install --reinstall

The first one does not try to remove each vendor path before changing current HEAD (which is obviously much faster). I think it should be the default behavior.
Do you want to keep a vendors:reinstall command?

--update ? Does it exist in the vendors script ? IIRC this is done without any option for this case.

Yeah my bad, fixed.
And the capistrano commands should probably be vendors:install (rather than vendors:update) and vendors:reinstall to be coherent with bin/vendors

interesting it seems similar to https://github.com/everzet/capifony/pull/53

Merged https://github.com/everzet/capifony/pull/53 ;-)


ISPN-13124 Make SingleFileStore implement NonBlockingStore

https://issues.redhat.com/browse/ISPN-13124
@wburns I did not add a migration from the old "format" using one file per segment. I'm kind of hoping it's not going to be necessary, if SIFS is now the implementation behind <file-store/>.

I finally added the migration from the old file layout with 1 file per segment to the new layout with a single file even when segmented @wburns
I made a lot of changes with this, so I decided to also squash the "Address review comments"  commit
As I was quashing I realized that I didn't go far enough with the getFirstStore() changes, so I made getFirstStoreWait() always return a WaitDelegatingNonBlockingStore and I changed EnsureNonBlockingStore to extend WaitDelegatingNonBlockingStore as well.

There was a conflict with another PR, I think the WriteBehindFaultToleranceTest. Nothing serious, just an import and a cast, and it should be ok now.

I just noticed that a ton of JPA tests seem to reliably fail on your branch for some reason https://ci.infinispan.org/job/Infinispan/job/PR-9407/25/#showFailuresLink Can you confirm what is going on there?

They were related, indeed... there were 2 different issues, both not really related to SingleFileStore

I removed the default local cache that some tests create just in case, and I missed the JPA tests that were actually using it.
When I implemented the migration I found a better way (IMO) to handle the wrapping of stores in WaitDelegatingNonBlockingStore (something I had already worked on earlier), but some old usages of getFirstStore() needed to change to getFirstStoreWait(), and the error is only visible at runtime.

Should be done now, including your other comments.

Looks like something got messed up in the Async store tests now where it couldn't read the single file store data. https://ci.infinispan.org/job/Infinispan/job/PR-9407/27/testReport/junit/org.infinispan.persistence.support/AsyncStoreWithoutEvictionFunctionalTest/testPutClearPutsegmented_false/

The test was calling SingleFileStore.clear() and not waiting for the clear to finish before stopping the cache. After looking a bit into the test, it's not doing a very good job of testing the async store wrapper, and where it does it overlaps with AsyncStoreTest (which also doesn't have eviction enabled). So I decided to remove it altogether.

After considering it further, I added a commit to make SingleFileStore.clear() preserve the header no matter when the store stops/crashes, and to also acquire the resize lock in stop() so that it doesn't run concurrently with other writes.

Integrated into main, thanks @danberindei !


Terraform module references updated

What?
The dependent Terraform modules have been updated to include new outputs. The references to these modules has been updated.
Why?
The dependent modules have been modified to provide more information about the base terraform state.
How?
The modules are referenced using Git tags. This has been bumped to the latest version, v1.5.1 of the amido/stacks-terraform module.

/azp run

/azp run

/azp run


[Documentation Bug]: Would like to add Contributors logo's to Readme.md file

Describe the bug ✍
Describe your bug here
No Bug.
Provide step by step information reproduce the bug 📄
Description
Would like to update the Readme.md file by adding Contributors logo Readme.md file.
Select program in which you are contributing
GSSoC24
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

@Durgesh4993 please review the changes.


JulianDate issues parsing some dates...

I'm not sure why, but the date 11/17/2039 12:00:00 AM UTC produces a JulianDate that is off by one day (November 16 instead of 17).  The problem appears to be in computeJulianDateComponents and is most likely a rounding error.  I was able to fix the error (by re-porting some Components code) but that caused other tests to start failing (seemingly due to a rounding error in the other direction).  The below test should pass, but doesn't.
it('Construct problem dates', function() {
        var expectedDate = new JulianDate(2466109, 43235.0, TimeStandard.TAI);
        var date = JulianDate.fromDate(new Date('11/17/2039 12:00:00 AM UTC'));
        expect(date).toEqual(expectedDate);
    });

Fixed in #2949


HKG: query FW versions on non-OBD bus

[ ] CAN platform
[x] CAN-FD platform

moved to https://github.com/commaai/openpilot/issues/26643

Note that since we added our improved fuzzy fingerprinting for HKG, most the cars do actually fingerprint without the comma power since we get the EPS after https://github.com/commaai/openpilot/pull/31719
Only CAN cars where we don't get the EPS will fail to fingerprint, but so far all CAN platforms have access to that ECU: https://github.com/commaai/openpilot/pull/31969


Deepspeech + Conformer corrections

This PR contains model code for Deepspeech and a couple small fixes to Conformer pytorch workload.
Even though the submission.py has batch_size=256, I have not been able to successfully train the pytorch model with this batch-size as it ends up with a cuda out-of-memory error.
Most notably, the memory utilization increases from 0.88GB to 7GB just after applying the subsampling module and I directed my efforts in fixing this, but did not amount to anything.
Nonetheless, I also found that when running on a single GPU, I was able to comfortably use a batch size of 32 (i.e., 256/8)!

Merging for now, if Runa's comments need addressing we can make a new fix PR.


Need way to set LONGFILE_POSIX and BIGNUMBER_POSIX for TarArchiveOutputStream

I'm trying to use the jarchivelib API to create a .tar.gz file and I need to set the LONGFILE_POSIX and BIGNUMBER_POSIX settings on the TarArchiveOutputStream. There may be a way to do this already using the API but I can't seem to figure it out. See http://commons.apache.org/proper/commons-compress/tar.html
The API looks great, it would be great if there were a way to use it with these settings.
Thanks!

Not using jarchivelib, which is really just convenience code. I would suggest to use commons-compress code directly for this..

Sure, I can do it by calling commons-compress directly. I was just thinking that if there was a way to get to the underlying TarArchiveOutputStream from the ArchiveStream then I could keep using the more convenient API.

The archive creation process using the ArchiveOutputStream is completely hidden behind the API, there is no way to access it. Also, the implementations are all package local, so you can't overwrite them (admittedly not the best design choice). I'm re-designing some of those mechanisms for a v2 to make it easier to handle specific archive entries, however this won't be public anytime soon. Sorry!

Thanks for the responses & I totally understand. It's your fault in the first place for making an API that streamlines commons-compress so well. :)

I would also really like to see this feature.


The Product listing page should be improved.

Is your feature request related to a problem? Please describe.
There is extra space on the product listing page. and the default number of products to list is 9.
Describe the solution you'd like
The default number of products to list should be 12, on the product listing page. Please take a reference from the below screenshot.
Describe alternatives you've considered

Additional context
No response

@abdulhamid-alattar
Please take a reference from the below screenshot.

@jitendra-webkul
Now, this issue is fixed. so I am closing this issue.
And I have attached a screenshot Here, for future reference.


Rename PartialTagHelper asp-for to for

The PartialTagHelper is a brand new element and therefore we shouldn't be prefixing the for property on the TagHelper
/cc @pranavkm

@DamianEdwards, any thoughts?

Seems this has already been discussed and agreed upon on the other thread https://github.com/aspnet/Mvc/issues/7374

Is "for" name even good enough?

4e7d1a5a32860ba4f4f6a86fd0b9e1049e4d8c50


Support for auth

Any plans to support authentication for exporter, e.g. basic auth?

Hi Martin, currently no plans, not sure if the underlying prometheus go library has support for this. If you are running in k8s you might be able to run an nginx next to the filesystem-exporter?

seems to be a little bit over overengineered. I just want add authentication for the exporter. I could try to fill a PR

Sure, would be happy to merge a PR!

Hi Martin,
is your PR ready yet? Otherwise I would close for inactivity.


Error in eslint checker

Error detected while processing function <SNR>75_BufWritePostHook..<SNR>75_UpdateErrors..<SNR>75_CacheErrors..239..SyntaxCheckers_javascript_eslint_GetLocList:

line   12:
E714: List required
Error detected while processing function <SNR>75_BufWritePostHook..<SNR>75_UpdateErrors..<SNR>75_CacheErrors:
line   25:
E170: Missing :endfor
Error detected while processing function <SNR>75_BufWritePostHook..<SNR>75_UpdateErrors:
line   11:
E171: Missing :endif

I noticed that similar errors come up with incorrect syntastic variables but I've checked mine and I don't see anything wrong with them:
These are the only lines in my vimrc that contain the word 'syntastic'
let g:airline_section_warning = airline#section#create(['syntastic'])
let g:syntastic_python_checkers = ["flake8"]
let g:syntastic_javascript_checkers = ["eslint"]
let g:syntastic_html_checkers = []

Here is the output for :SyntasticInfo
Syntastic: active mode enabled
Syntastic info for filetype: javascript
Available checker(s): eslint
Currently enabled checker(s): eslint

Whenever I remove syntastic_javascript_checkers from my vimrc SyntasticInfo doesn't default to eslint even though it's available
Syntastic: active mode enabled
Syntastic info for filetype: javascript
Available checker(s): eslint
Currently enabled checker(s):

What OS do you have, and what version of syntastic are you using?  Can you reproduce the problem if you disable all other Vim plugins?

Whenever I remove syntastic_javascript_checkers from my vimrc SyntasticInfo doesn't default to eslint even though it's available

Have you considered reading the manual?  At least the "quick start" part?

OSX
whatever version installs with vundle
I haven't tried to disable other plugins
I read the manual when I installed this plugin (years ago) but since you brought it up I read just went over it again and I don't see anything having to do with what I said.

I don't have OSX, and I can't reproduce your problem.
Please upgrade to syntastic 3.6.0.  Remote debugging of unknown versions of syntastic is not supported.  Remote debugging of Vundle is not supported either. :)
Please try disabling all other plugins, and report back if you can still reproduce the problem.
Let's try the first paragraph of "Quick start", in particular the second sentence:

Syntastic comes preconfigured with a default list of enabled checkers per filetype. This list is kept reasonably short to prevent slowing down Vim or trying to use conflicting checkers.

I'll try disabling other plugins in the morning and I'll try on a Windows machine at work. Vundle simply clones the repo so it's whatever is in master. I'll downgrade(?) to 3.6.0 and see if that helps. I'll report back in the morning. Thanks for the help.

Syntastic 3.6.0 is the current master HEAD, and the output of :SyntasticInfo in 3.6.0 is different from what you have shown above. shrug

I just realized I switched to Syntastic-Async a while back and that's why it's not on the latest version. I just changed it back to this repo and it works fine. Sorry about that.


[BUG] :  Buttons do not have an accessible name

Describe the bug
Buttons do not have an accessible name
To Reproduce
Steps to reproduce the behavior:

Open website
Open dev tools
Click on lighthouse
Measure lighthouse score
See accessibility error

Expected behavior
All the buttons should have an accessible names.
Screenshots

Desktop (please complete the following information):

OS: Windows
Browser : Chrome

Additional context
You can refer to the following docs for more info on accessible buttons.

Could you please assign me this issue

Proceed with it.


[CENTOS-04] - bashrc should properly handle interactive and non-interactive terminal sessions

resolves https://github.com/reynoldsm88/dart-amis/issues/4


RBAC in MDE - Security Operator

I think the comment for MDE for Security Reader also applies to Security Operator. Our tests indicate that a Security Operator cannot access MDE when RBAC is enabled. This is made clear for Security Reader but not for Security Operator.
When you turn on role-based access control in Microsoft Defender for Endpoint, users with read-only permissions such as the Azure AD Security Reader role lose access until they are assigned to a Microsoft Defender for Endpoint role.

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 0767aac4-fcd9-eaac-2c30-8f17d8eadd92
Version Independent ID: a380a828-03f9-e037-3a62-e4165fda7cb6
Content: Azure AD built-in roles - Azure Active Directory - Microsoft Entra
Content Source: articles/active-directory/roles/permissions-reference.md
Service: active-directory
Sub-service: roles
GitHub Login: @rolyon
Microsoft Alias: rolyon

@BrechtMo
Thanks for your feedback! We will investigate and update as appropriate.

@rolyon
Could you please check this issue as i am unable to repro this issue due to access related constraints on azure portal.
Thanks

Hi @BrechtMo
Thanks for taking the time to provide your feedback. We updated the description for the Security Operator role.
https://learn.microsoft.com/en-us/azure/active-directory/roles/permissions-reference#security-operator
Let us know if you have additional feedback.
thanks

#please-close


OPSEXP-1645 Add Alfresco control center role

Ref: OPSEXP-1645

this branch tests are failing because of https://github.com/Alfresco/alfresco-ansible-deployment/blob/master/.github/actions/setup-workspace/action.yml#L7
testing on a next/* branch still fails due to https://github.com/Alfresco/alfresco-ansible-deployment/pull/571

Can we try to make the WF green?
https://github.com/Alfresco/alfresco-ansible-deployment/actions/runs/4608047509/jobs/8143697737?pr=564


Cannot read property 'hash' of undefined

Tab Foundation doesn't work when I  tried to bind it on the vue components I got an error.
Cannot read property 'hash' of undefined.
//this is the directives
Vue.directive('ftab',{
inserted(el) {
$(el).foundation();
},
unbind(el) {
$(el).destroyed();
}
})

                         {{tab.name}}

as a workaround I believe you can change your 'href' attribute to 'data-tabs-target'
{{tab.name}}

Which line of the unminified foundation.js file throws this error?
Is it window.location.hash or ...link[0].hash.slice(1)?

I'm not the OP but I'm having the same issue, it's the line with
hash = $tabLink.attr('data-tabs-target') || $tabLink[0].hash.slice(1),

It should be probably fixed in the develop branch. Can you test it?

Sure. I'm still looking a this problem.

The thing that gets me if above that is says $tabLink = $target.find('[role="tab"]'),, It does not mention role="tab" anywhere in the documentation. I remember it used to back in 5 but that has not been in there for at least a year.

All links should get [role="tab"] added at initialization. See https://github.com/ncoden/foundation-sites/blob/89a6f09f270b43ebcacbc1f4fb968f8cc3817be5/js/foundation.tabs.js#L64
@joshuajonah @lyndonkieljerymiah

Do you have any link added dynamically ?
Does links have [role="tab"] set after Tabs are initialized ?

I didn't really recompile, I just npm installed the develop branch.
And yes, I'm loading data from JSON with Vuejs to generate the HTML. Honestly, I've new switched to a vanilla JS Vue component instead as whole idea was to get off jquery.

Yeah I did not seen v-for="(tab,index) in the example above.
@lyndonkieljerymiah You have to initialize the component by yourself after the link list is fully rendered.
const $tabs = new Foundation.Tabs($('#tabId'), options);

Closing. Feels free to reopen if this is not working.

@ncoden

Do you have any link added dynamically ?

Does the tabs not work if they are added dynamically? (e.g., populated via an ajax call to an api?)

Does the tabs not work if they are added dynamically? (e.g., populated via an ajax call to an api?)

Not yet but in the next release as far as I remember.
In general no, you have to reinitialize your component then.


Allow the user to configure an instance of elastic search in the app

Optionally, if a config.elasticsearch is configured, we can instantiate a global object with a connection to Elastic Search to be used for indexing/querying.

@snez I think elasticsearch is too specific for most people. This should probably be added to the generator @NeverOddOrEven, no?

@ilanbiala generator would be great to have. Would this be also considered a "module" for 0.4.0?

@SOSANA this can be reposted as an issue on https://github.com/meanjs/generator-meanjs


Make it an electron app

Run chrome inside electron, so dont have to bother with linux build

The problem with this is that ELECTRON SUCKS >:((((((((( INSTEAD USE WINE! NO ONE WILL NOTICE IT IS NOT NATIVE!!!!!!!!!!!!!!!!

USE DARWIN WINE IS OVERRATED:(((((((((((((((


in java environment,when you run multi-thread in gauge,the thread won't run,especially for the time-consuming task；And won't run in threadpool，such as fixedthreadpool。

such as:
ExecutorService service  = Executors.newFixedThreadPool(3);
for(int i = 1;i<=24;i++){
final int index = i;
service.execute(new Runnable() {
@Override
public void run() {
System.out.println("current number"+index);
if(index % 3 == 0)
execute(conn1);
else if(index % 3 == 1)
execute(conn2);
else
execute(conn3);
}
});
}
Gauge version: 0.8.4
Plugins
html-report (4.0.2)
java (0.6.2)

Hi,
Apologies for the very late response, but in case you still need an answer, I tried the below, and it does seem to run as expected.
    @Step("Run tasks")
    public void setLanguageVowels() throws InterruptedException {
        ExecutorService service = Executors.newFixedThreadPool(3);
        for(int i = 1;i<=24;i++){
            final int index = i;
            service.execute(new Runnable() {
                @Override
                public void run() {
                    System.out.println("current number"+index);
                    if(index % 3 == 0)
                        execute(1000);
                    else if(index % 3 == 1)
                        execute(3000);
                    else
                        execute(5000);
                }
            });
        }
        service.awaitTermination(5, TimeUnit.MINUTES);
    }

    private void execute(int time) {
        try {
            Thread.sleep(time);
        } catch (InterruptedException e) {
            System.out.println(e);
        }
    }

Sample output:
# Specification Heading
  ## Run tasks	current number1
current number3
current number2
current number4
current number5
current number6
current number7
current number8
current number9
current number10
current number11
current number12
current number13
current number14
current number15
current number16
current number17
current number18
current number19
current number20
current number21
current number22
current number23
current number24

I am closing this issue, but please feel free to comment if I misunderstood your case. thanks.


Automatically select the car in kapcsolat.php that you clicked on in termekek.php

When you click buy on a car it should store the name of the car and in termekek.php it should select that one by default from the scroll-down list.

Feature added in Click select feature update #10


Adds payout module for safe interaction

not hooked up yet, only module
mnemonic used for now - can easily switch to privkey

How is accounting handled? Also we should do private key instead of mnemonic.

@3commascapital  would you rebase from main and continue working? we had a bunch of refactors


remove redundant config check in ecds

[ ] Ambient
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

Please check any characteristics that apply to this pull request.

[X] Does not have any user-facing changes. This may include CLI changes, API changes, behavior changes, performance improvements, etc.

@ramaraochavali: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
bd50ac43d75fa144b999447abd0a06686bc6a5cd
link
true
/test release-notes

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test release-notes

@ramaraochavali: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio
bd50ac43d75fa144b999447abd0a06686bc6a5cd
link
true
/test unit-tests-arm64

unit-tests_istio
bd50ac43d75fa144b999447abd0a06686bc6a5cd
link
true
/test unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ramaraochavali: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio
bd50ac43d75fa144b999447abd0a06686bc6a5cd
link
true
/test unit-tests-arm64

unit-tests_istio
bd50ac43d75fa144b999447abd0a06686bc6a5cd
link
true
/test unit-tests

integ-security_istio
bd50ac43d75fa144b999447abd0a06686bc6a5cd
link
true
/test integ-security

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ramaraochavali: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio
736eac81afe472dedb20b978aec45f8eeb5ca487
link
true
/test unit-tests-arm64

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

/test unit-tests-arm64

@ramaraochavali: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests_istio
736eac81afe472dedb20b978aec45f8eeb5ca487
link
true
/test unit-tests

unit-tests-arm64_istio
736eac81afe472dedb20b978aec45f8eeb5ca487
link
true
/test unit-tests-arm64

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ramaraochavali: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio
736eac81afe472dedb20b978aec45f8eeb5ca487
link
true
/test unit-tests-arm64

unit-tests_istio
736eac81afe472dedb20b978aec45f8eeb5ca487
link
true
/test unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test unit-tests-arm64

/test unit-tests

@ramaraochavali: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests_istio
736eac81afe472dedb20b978aec45f8eeb5ca487
link
true
/test unit-tests

unit-tests-arm64_istio
736eac81afe472dedb20b978aec45f8eeb5ca487
link
true
/test unit-tests-arm64

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ramaraochavali: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio
736eac81afe472dedb20b978aec45f8eeb5ca487
link
true
/test unit-tests-arm64

unit-tests_istio
736eac81afe472dedb20b978aec45f8eeb5ca487
link
true
/test unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

@ramaraochavali: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests_istio
736eac81afe472dedb20b978aec45f8eeb5ca487
link
true
/test unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Set the app_locale to zh-CN in dashboard, but the front page(StatusPage) is still displayed in English

This is likely caused by you having your browser language setting set to English?


not going past python setup.py install

Good day,
I am trying to test your library but cannot no go past python setup.py install.
I have successfully installed opencv 4.5 and EDCC.
This is the screenshot of my error:

and this is my python version

on ubuntu 20.04
Your quick response will be greatly appreciated

I managed to resolve this issue by running this command:sudo ldconfig


missing the slash separating the database name

request
./uhppoted-app-db store-acl --dsn mysql://admin:password@tcp<IP_ADDRESS>:3306/mydatabase
Always return ERROR: invalid DSN: missing the slash separating the database name but the slash is there

Hi,
Not sure about the slash (it's a message from the MySQL driver) but that DSN looks like it's missing the brackets around the host:port. It may also need quotes because bash can be weird about an @ in the command line. Try:
./uhppoted-app-db store-acl --dsn "mysql://admin:password@tcp(<IP_ADDRESS>:3306)/mydatabase"

BTW, tcp(<IP_ADDRESS>:3306) is the default DSN so you could probably just use:
./uhppoted-app-db store-acl --dsn "mysql://admin:password@/mydatabase"

It's described in the README and in much more detail in the driver README.

Closing, presuming fixed - feel free to reopen if you need to.


Plain export issue when Link children are just images

Plain export generates links stuck together when children are just images.
export default function Email() {
  return (
    <>
      <Link href="http://www.google.com">
        <Img src={`/static/google.jpg`} alt={`Go to google`} width="30px" />
      </Link>

      <Link href="http://www.bing.com">
        <Img src={`/static/google.jpg`} alt={`Go to bing`} width="30px" />
      </Link>
    </>
  );

Result:
http://www.google.comhttp://www.bing.com

can i work on this?

It worked fine for me.
I pasted the same code and it gave output as expected.
Can you tell me more about how to reproduce it?

Closing this as stale. If anyone still has this, let me know with a proper reproduction and I'll reopen.


A lot of thumbs

Description
I just created a bootable thumb drive with Ubuntu, using Rufus. When I finish, a lot of links appear in left bar.
Steps To Reproduce
No response
Expected behavior
I would expect just one link. I've seen this before, but now I had time to capture and report.
Files Version
2.1.15
Windows Version
Windows 11 Pro 21H2 22000.613
Relevant Assets/Logs

This issue is resolved in the preview version of Files https://files.community/download/preview.


Check Yarn version before install

Describe the feature you'd like to request
Right now, when doing an installation it defaults to yarn 1.
It would be nice, if the dev is using the latest yarn, then npx create-turbo@latest should use it.  Otherwise:

Install turborepo.  Yarn 1 used.
Upgrade yarn:

yarn set version stable
➤ YN0000: Retrieving https://repo.yarnpkg.com/3.2.4/packages/yarnpkg-cli/bin/yarn.js
➤ YN0000: Saving the new release in .yarn/releases/yarn-3.2.4.cjs
➤ YN0000: Done in 0s 334ms

Run Yarn again:

➤ YN0070: Migrating from Yarn 1; automatically enabling the compatibility node-modules linker 👍

As then workspaces can be run:
yarn workspace web run dev
ready - started server on <IP_ADDRESS>:3000, url: http://localhost:3000

Or else:
Internal Error: turbotest@workspace:.: This package doesn't seem to be present in your lockfile; run "yarn install" to update the lockfile

Describe the solution you'd like
npx create-turbo@latest uses the currently used version.
Describe alternatives you've considered
See above.

Right now, when doing an installation it defaults to yarn 1.

As far as I know, npx create-turbo@latest will prompt you to select the package manager you want to use. What turbo command are you using that is using Yarn 1 by default?

I had a lot of issues getting here and I must have forgotten a step / mis-remembering my history.
Going back and trying again.  I am getting with yarn v3 initially:
paul@Pauls-MacBook-Pro tempo2 % yarn -v
1.22.19
paul@Pauls-MacBook-Pro tempo2 % yarn set version stable
➤ YN0000: Retrieving https://repo.yarnpkg.com/3.2.4/packages/yarnpkg-cli/bin/yarn.js
➤ YN0000: Saving the new release in .yarn/releases/yarn-3.2.4.cjs
➤ YN0000: Done in 0s 279ms
paul@Pauls-MacBook-Pro tempo2 % npx create-turbo@latest

>>> TURBOREPO

>>> Welcome to Turborepo! Let's get you set up with a new codebase.

? Where would you like to create your turborepo? myturborepo
? Which package manager do you want to use? yarn

>>> Created a new turborepo with the following:

 - apps/web: Next.js with TypeScript
 - apps/docs: Next.js with TypeScript
 - packages/ui: Shared React component library
 - packages/eslint-config-custom: Shared configuration (ESLint)
 - packages/tsconfig: Shared TypeScript `tsconfig.json`

>>  Installing dependencies...
Aborting installation.
  yarn install --no-immutable has failed.

It doesn't install even if I go into the newly created turborepo dir and execute the same yarn command.
Now I remember:
When I upgrade the yarn to version 3, it creates a package.json.  If I set the install dir to ./ then it installs fine.
paul@Pauls-MacBook-Pro tempo2 % cd myturborepo
paul@Pauls-MacBook-Pro myturborepo % yarn set version stable
➤ YN0000: Retrieving https://repo.yarnpkg.com/3.2.4/packages/yarnpkg-cli/bin/yarn.js
➤ YN0000: Saving the new release in .yarn/releases/yarn-3.2.4.cjs
➤ YN0000: Done in 0s 310ms
paul@Pauls-MacBook-Pro myturborepo % ls
package.json
paul@Pauls-MacBook-Pro myturborepo % npx create-turbo@latest

>>> TURBOREPO

>>> Welcome to Turborepo! Let's get you set up with a new codebase.

? Where would you like to create your turborepo? ./
? Which package manager do you want to use? yarn

>>> Created a new turborepo with the following:

 - apps/web: Next.js with TypeScript
 - apps/docs: Next.js with TypeScript
 - packages/ui: Shared React component library
 - packages/eslint-config-custom: Shared configuration (ESLint)
 - packages/tsconfig: Shared TypeScript `tsconfig.json`

>>> Success! Your new Turborepo is ready.
Inside this directory, you can run several commands:

  yarn run build
     Build all apps and packages

  yarn run dev
     Develop all apps and packages

Turborepo will cache locally by default. For an additional
speed boost, enable Remote Caching with Vercel by
entering the following command:

  yarn dlx turbo login

We suggest that you begin by typing:

  yarn dlx turbo login

Sorry to move the goal posts slightly.
Here's what I'm using:
System:
    OS: macOS 12.4
    CPU: (10) arm64 Apple M1 Pro
    Memory: 804.28 MB / 32.00 GB
    Shell: 5.8.1 - /bin/zsh
  Binaries:
    Node: 18.4.0
    Yarn: 3.2.4
    npm: 8.12.1

I noticed a couple things:

When I run yarn set version stable in the parent directory, it creates a package.json, .yarn/ and .yarnrc.yml file. This causes yarn install to fail in the new turborepo.
pwd //=> ~/dev
yarn set version stable
npx create-turbo@latest // fails on install step

If I remove those new yarn files, install works.

I can see more verbose output by running `yarn install` manually:
  ```
  ➜  my-turborepo yarn install
  Usage Error: The nearest package directory (/Users/mehulkar/dev/my-turborepo) doesn't seem to be part of the project declared in /Users/mehulkar/dev.

If /Users/mehulkar/dev isn't intended to be a project, remove any yarn.lock and/or package.json file there.
If /Users/mehulkar/dev is intended to be a project, it might be that you forgot to list my-turborepo in its workspace configuration.
Finally, if /Users/mehulkar/dev is fine and you intend my-turborepo to be treated as a completely separate project (not even a workspace), create an empty yarn.lock file in it.

$ yarn install [--json] [--immutable] [--immutable-cache] [--check-cache] [--inline-builds] [--mode #0]
</details>

I'm not sure if this is something turbo needs to watch out for, since I'm not sure what other ways the yarn version can be set globally 🤔.

Thanks for your attention on this. But I have abandoned Yarn v3.

Sorry to hear! I'm no expert in yarn, but if you figure out how create-turbo can set you up with Yarn better, I'd love to hear about it.

If you run yarn --version in your current directory and it returns, say, 3.2.3 we will initialize a yarn@3 turborepo with the correct configuration. One way you can do that is to use corepack:
~$ corepack enable
~$ corepack prepare<EMAIL_ADDRESS>--activate
Preparing<EMAIL_ADDRESS>for immediate activation...
~$ yarn --version
3.2.3
~$ yarn create turbo --use-yarn issue-2250
➤ YN0000: ┌ Resolution step
➤ YN0000: └ Completed in 3s 982ms
➤ YN0000: ┌ Fetch step
➤ YN0000: └ Completed
➤ YN0000: ┌ Link step
➤ YN0000: └ Completed
➤ YN0000: Done in 4s 136ms

>>> TURBOREPO

>>> Welcome to Turborepo! Let's get you set up with a new codebase.

>>> Created a new turborepo with the following:

 - apps/web: Next.js with TypeScript
 - apps/docs: Next.js with TypeScript
 - packages/ui: Shared React component library
 - packages/eslint-config-custom: Shared configuration (ESLint)
 - packages/tsconfig: Shared TypeScript `tsconfig.json`

>>> Success! Created a new Turborepo at "issue-2250".
Inside that directory, you can run several commands:

  yarn run build
     Build all apps and packages

  yarn run dev
     Develop all apps and packages

Turborepo will cache locally by default. For an additional
speed boost, enable Remote Caching with Vercel by
entering the following command:

  yarn dlx turbo login

We suggest that you begin by typing:

  cd issue-2250
  yarn dlx turbo login


Gesture and Swipe support

I am considering porting this rich text editor into a react native component for a project that I am working on. One of my requirements though is to be able to support gestures; swiping left or right on a sentence applies text styling such as bolding and indentation.
Being very foreign to the iOS development world wondered if you had an idea how your the rich text editor can be configured to listen to gestures on individual blocks of text.
Would appreciate any thoughts on the matter.

Cool! I've never done react native, so that seems pretty neat! 😃

Being very foreign to the iOS development world

Are you from the world of front-end JavaScript? Because if so, you might be able to figure this out better than I can, to be honest! This library is basically just a thin wrapper around a UIWebView that has a contentEditable div inside it, than I then funnel some JavaScript functionality to.
I have some UIGestureRecognizers, which may or may not affect what touches make it to the JS context itself -- I haven't tried doing any kind of swiping functionality... If so, you might be able to register a new UIGestureRecognizer on the web view, and just pass the relevant information to it, and then do some processing down in that level? I'm not sure, I'm just spouting out ideas...

Interesting... didn't realise that this library was built on top of a JS implementation and not native as I thought initially.
What's the performance like?

Interesting... didn't realise that this library was built on top of a JS implementation and not native as I thought initially.

Yeah, it's only a very thin wrapper, opening some of the contentEditable APIs to be available from native code. Plus a little bit extra to make sure it's functioning.

What's the performance like?

The guy who's writing his entire app in JS (react native) is asking about performance of a component using JS, haha 😜
I know a few apps that are using it production. One of them has multiple (potentially a lot) onscreen at the same time. I'd suggest trying it out and making a sample of your use case to see if it works for you.
One issue that might come up is that, since it's a web view, loading is asynchronous and might take a while depending on the content. This can make layout take a while, which can be an issue if you have many of them in a table view or something.

Closing this issue, because making a reactive native component is not on my radar. Someone else is welcome to though.


Option to set --lts as default

To install the latest long term support version the command is:
nvm install --lts

If I do a fresh install, then close and reopen my terminal then run: nvm version
I get: none

To set a default Node version to be used in any new shell, use the alias 'default':

nvm alias default node

But if I run: nvm alias default --lts, nvm alias default lts/*, or nvm alias default lts then  I get Unsupported option "--lts"., default -> lts/* (-> N/A), & default -> lts (-> N/A) respectively.
How do I set the LTS version to be the default version?

nvm alias default lts/* should do it.
The usability of this should be slightly increased by #1204.

On running nvm alias default lts/* I get the following error
No matches for wildcard 'lts/*'.  (Tip: empty matches are allowed in 'set', 'count', 'for'.)
fish: nvm alias default lts/*
                        ^

I'm using fish shell with omf nvm plugin.

@skippednote nvm doesn't support the fish shell, but because many shells might expand the *, try nvm alias default 'lts/*'?

@ljharb Oh didn't know. Thanks.

I have a similar problem. Any ideas how to fix?
My .nvmrc is lts/*.
alam@tesla Tango (amy/nvmrc-lts) $ nvm install --lts
Installing latest LTS version.
v8.9.2 is already installed.
Now using node v8.9.2 (npm v5.5.1)
alam@tesla Tango (amy/nvmrc-lts) $ nvm use
Found '/Users/alam/src/Tango/.nvmrc' with version <lts/*>
N/A: version "N/A" is not yet installed.

You need to run "nvm install N/A" to install it before using it.
alam@tesla Tango (amy/nvmrc-lts) $ nvm alias default lts/*
! WARNING: Version 'lts/*' does not exist.
default -> lts/* (-> N/A)

@amyrlam what shell are you using? (what does nvm debug print out?) What's nvm ls print out?
Would you mind filing a separate issue? (since this is for the feature request, which was addressed by #1204 and https://github.com/creationix/nvm/issues/1217#issuecomment-244415486 - so I'm closing this one)

Old issue, but I just ended up here through a Google search. For reference:
On zsh/fish shell, you need to:
nvm install 'lts/*'

nvm alias default 'lts/*'

The solution provided doesn't work any more (using ZSH):
❯ nvm alias default 'lts/*'
! WARNING: Version 'lts/*' does not exist.
default -> lts/* (-> N/A)

❯ nvm use default
N/A: version "default -> N/A" is not yet installed.

You need to run "nvm install default" to install it before using it.

❯ nvm install default
Version 'default' not found - try `nvm ls-remote` to browse available versions.

@callumlocke can you file a new issue for that? that definitely seems like a bug.

The solution provided doesn't work any more (using ZSH):
❯ nvm alias default 'lts/*'
! WARNING: Version 'lts/*' does not exist.
default -> lts/* (-> N/A)

❯ nvm use default
N/A: version "default -> N/A" is not yet installed.

You need to run "nvm install default" to install it before using it.

❯ nvm install default
Version 'default' not found - try `nvm ls-remote` to browse available versions.

I have the same issue

yes please, this should be reopened. the behavior on the cli is internally inconsistent even if the prescribed solution still did work. I would expect --lts should be accepted by nvm alias

@robbiemu per https://github.com/nvm-sh/nvm/issues/1217#issuecomment-1079218518, i'm waiting for a new issue to be opened.


VS-106: Refactor out SyntaxElements in TemplateBuilders

Done in https://github.com/mongodb/mongo-csharp-analyzer/pull/86


Convert Text to TypeScript

Description of change

Convert Text to TypeScript
Pre-Merge Checklist

[ ] Tests and/or benchmarks are included
[ ] Documentation is changed or added
[x] Lint process passed (npm run lint)
[x] Tests passed (npm run test)

Codecov Report

Merging #6390 into dev will decrease coverage by 2.5%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##              dev    #6390      +/-   ##
==========================================
- Coverage   77.33%   74.83%   -2.51%
==========================================
  Files          76       66      -10
  Lines        4179     3346     -833
==========================================
- Hits         3232     2504     -728
+ Misses        947      842     -105

Impacted Files
Coverage Δ

packages/loaders/src/Loader.js

packages/loaders/src/AppLoaderPlugin.js

packages/loaders/src/index.js

packages/loaders/src/TextureLoader.js

packages/spritesheet/src/SpritesheetLoader.js

packages/spritesheet/src/Spritesheet.js

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 45df7eb...f2818b3. Read the comment docs.

Cool start!
But:

empty modifier used as 'public, for internal usage', other fields/methods/props require modifiers.
I think we shouldn't use lookup types (aka ITextStyle ['font'])
TextStyle should implement ITextStyle, or convince me why this isn't necessary.

@Zyie you good on this?

@Zyie you good on this?

seems good to me


Ask user for confirmation when deleting message

Is your feature request related to a problem? Please describe.
Currently, messages can be deleted with a single click, leading to accidental deletions. Since deleting a message also removes all subsequent messages, this can result in significant data loss without the user's intention.
Describe the solution you'd like
Reuse the currently implemented UX for confirming deleting entire confirmations.
Describe alternatives you've considered
The existing system does not require confirmation before message deletion. While straightforward, this approach lacks safeguards against unintended deletions, highlighting the need for the proposed solution.
Additional context
Original delete message PR in https://github.com/open-webui/open-webui/pull/789

I'll work on this.

From the latest update, I can confirm that when I deleted a certain message, the subsequent messages were not deleted altogether. Therefore, it would not be an issue. Adding confirmation will make user experience worse because we have to click twice for an action (imagine have to delete a few messages...). Therefore, I suggest to have an option in settings to toggle this rather than imposing as a default feature.

@taidbui thanks for letting me know, will log that bug in a separate issue!

Closing this feature request as we have decided to not delete all subsequent messages, only the selected prompt/response pair. Thus, a confirmation is not needed at this time


chore: update bootstrap version number

Updates edge bootstrap version

[x] I have read the contribution guidelines.
[x] The status checks are successful (continuous integration). Those can be seen below.

A picture of a cute animal (not mandatory but encouraged)

Closed this because a previous PR already updated this (Simon beat me to it)


[google_sign_in_web] Migrate to the GIS SDK.

This PR migrates google_sign_in_web to the Google Identity Services SDK, in a somewhat "backwards-compatible" way.
There are big differences between the old and new authentication/authorization SDKs, so even though this is API compatible with the old client, there are subtle differences in the underlying behavior that prompted a major version update:

signInSilently will trigger the One Tap user experience.

If the user consents to the One Tap UI, they will be authenticated but not authorized (a call to requestScopes or signIn is required to authorize the user).

Since the previous plugin assumes that user authentication and authorization are the same, the signInSilently method has currently been made to always return null (even on successful authentication) to prevent apps from triggering requests without being authorized.

This behavior should change in the future!

signIn will trigger an oauth2 popup.

If the user has NOT authenticated via signInSilently:

The plugin will attempt to request the user profile data from the People API and "fill the gap" (oauth2 endpoints no longer return the user authentication info) after a successful authorization.

User profiles retrieved in this way do NOT contain a proper idToken.
(The plugin will add 2 more scopes to the ones requested by the user to be able to do this).

If the user has authenticated via signInSilently:

The plugin will skip the request to the PeopleAPI completely and instead use the CredentialResponse received from the One Tap flow.

The rest of the methods should work very similarly to how they work now.

Version bump
This is a major reimplementation of the plugin, and there are some subtle differences in behavior (see above), so this probably warrants a new major version.
Tests

Rewritten tests.

Tests for the GisSdkClient could be added by abstracting the GIS bits a little bit more.

An unmodified Google Sign-In example app has been deployed here:

https://dit-gis-test.web.app

Issues

Fixes https://github.com/flutter/flutter/issues/88084

Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the relevant style guides and ran the auto-formatter. (Unlike the flutter/flutter repo, the flutter/plugins repo does use dart format.)
[x] I signed the CLA.
[x] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy, or this PR is exempt from version changes.
[x] I updated CHANGELOG.md to add a description of the change, following repository CHANGELOG style.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

I think the legacy_version_analyze is failing because pub cannot resolve a valid version of package:google_identity_services_web (which requires Dart 2.17+) ?

+   jwt_decoder: ^2.0.1

Same question here; is there a reasonable way to not add this dependency?

Hm... Maybe? I'll give this another shot tomorrow, it seems all that needs to be done is to base-64 decode (and JSON.parse) the middle bit of the token? :/ https://stackoverflow.com/questions/52017389/how-to-get-the-claims-from-a-jwt-in-my-flutter-application

Rebasing to pick up the latest .ci.yaml changes from main.

I think I've addressed all your comments @stuartmorgan, PTAL!

"User profiles retrieved in this way do NOT contain a proper idToken."
Ok, I can see it on the migration guide, on this PR and almost everywhere. You are not populating idToken on signIn. But I was using this plugin only in order to retrieve that idToken, how should I achieve that now? All documentations written said idToken is null, but no one said how to retrieve it from now. I don't need user information, I just need an idtoken to send to my backend...
Could you please explain how should I migrate?
Cheers

Could you please explain how should I migrate?

@sbatezat make sure that you use signInSilently on your app, so that UX flow is able to retrieve an idToken.
The lack of idToken coming from the authorization flow has been reported as an issue to the GSI Team, as mentioned here, and if they change their minds, we'll make it available.

Could you please explain how should I migrate?

@sbatezat make sure that you use signInSilently on your app, so that UX flow is able to retrieve an idToken.
The lack of idToken coming from the authorization flow has been reported as an issue to the GSI Team, as mentioned here, and if they change their minds and start returning the token, we'll make it available to users.

I've tried signInSilently, it returns null (as explained on the documentation) and show nothing to the end user. After signinSilently, I'm calling again signIn, and there is still no IdToken.
If there is no IdToken, what should the process to validate user server side?

I've tried signInSilently, it returns null (as explained on the documentation) and show nothing to the end user. After signinSilently, I'm calling again signIn, and there is still no IdToken.

@sbatezat The user must be signed in to their browser for signInSilently to do its thing, and if the OneTap UX worked successfully, you'll have the idToken after signIn.

If there is no IdToken, what should be the process to validate user server side?

You can't validate users without an idToken server-side, so I guess that for the server-side validation of users, a larger change to your app is required until the Oauth popup starts returning idTokens.
Look at the documentation, you may want to use the "Button" flow?


Tooling & test improvements

npm run format:single
will format a single file with given path using prettier and default config.
npm run format:all
will format all js files under src folder.
npm run format:check:single
will check if given file is formatted correctly, if not it's gonna exit with a non-zero exit code.
npm run format:check:all
will check if all js files are formatted correctly, if not it's gonna exit with a non-zero exit code. (Gonna be used by CI environment[1] to make sure all files are still formatted correctly)
npm run test
will now check all files matching src/**/**.test.js effectively allowing us to co-exist tests with the related module[2].
[1]: travis will run npm run format:check:all along with npm test.
[2]: it was matching test/**/*.js before.

@gokmen without porting kite-server.coffee it looks really hard to get the old tests running again. If we want this, i can work on that. But instead of doing that, i think working on unit tests of kite itself would be more useful at this point. There are really little amount of tests, but the implementation is complex enough to make them less useful actually.
Let's talk about this.
Other than that, i think this pr is complete as a structural change. i will open other prs for individual tests.

@usirin agreed, let's merge this one in and iterate over there for the initial release. I think we can ship this as is for 1.0.0 to cover Koding requirements for now and we can continue changing with 2.0.0 release. I've added some comments not for the PR only but also as a note to my self, I'll update them and try with Koding then I'll publish it to npm as 1.0.0, wdyt?


Rosa items show up in dumb places on treasure map

The Rosa checks display on the Spool Swamp portal on the treasure map since that's nearest to Rosa, but that's confusing. The cave should display as the Temple Ruins portal, and the summer tower should display as the Eastern Suburbs portal.

Resolved in 16a8628.


Updated words game in Show posts list (ShowSingleStats.js)

Minor changes and merged in https://github.com/PolymerLabs/arcs/pull/1588


Make mysql/postgresql async client official module

Currently http://vertx.io/docs/vertx-mysql-postgresql-client/java/ is still under technical preview, it would be great to make it official module as JDBC client.
Also on https://www.techempower.com/benchmarks/, the benchmark code is still using JDBC client, which is slow. Better to update it with async client.
I think DB operation is very common and by showing vertx's performance in this cases, it would benefit lots of people.
Thanks!

+1 for making it an official module, we need to define the missing features in this module because I believe it is missing some
Instead of having this issue here, could you instead create an issue inhttps://github.com/vert-x3/issues/issues/ with a whishlist tag ?

@beatdigit just for reference, the latest techempower git code is built on the async driver.

+1 I think this is important


Updated Add cli flag to enable events on workers #21

This is @dimrozakis 's original PR #21, rebased and with a couple of compatibility fixes and some test-coverage.
Updates, fixed #21.

Thank you 😄


REFACTOR: enable github actions and decentralandLibrary, fix typings

Just saw this PR :/
We ended up implementing all of this separately, so not much point in keeping it open


Help running with Besu

Hi there,
I am trying to use this project to see an instance of Hyperledger besu permissioned network.
Right now I can use my besu node to move ETH, send txs etc with whitelisted accounts. On metamask for example, I can see ETH balances on the network by simply connecting to the node. I can submit txs with a whitelisted account. It is possible that I cannot GET from the blockchain without private key authentication.
In the instructions for besu it is stated that we can use APP_NODE_URL to set the rpc endpoint. That's fine. I am wondering if there is some extra step to pass a private key to the app so it can access permissioned data? Let me know. Attached here in is what I see when I search a contract address that I know I have deployed to this network

Hey @onigiri-x, since this repo isn't actively maintained I recommending jumping over to https://chat.hyperledger.org/channel/besu for your question


75 Update SDK documentation

Closes #75
The JS SDK documentation in this repo was up-to-date. I have just reformatted it to follow our general documentation guidelines and added some explanations for the error codes. Also, I copied some examples from the tests into the js-client README for easy access.
Reviewers
@gyroflaw, please review the tech content, especially the error codes and additional examples copied over (9bcc5574494c9c90049e2377caf52a1f323e8a32 and 9bcc5574494c9c90049e2377caf52a1f323e8a32)
@ACStoneCL, please review the documentation flow (a093f90a51c92d4978c023d4fbdabb5e13f08179 and overall)
Just FYI: @hoffmannjan, @deuszex

Thanks, @gyroflaw and @ACStoneCL!


chore(backend): Use tsup to bundle the package for both cjs and esm

Type of change

[ ] 🐛 Bug fix
[ ] 🌟 New feature
[ ] 🔨 Breaking change
[x] 📖 Refactoring / dependency upgrade / documentation
[ ] other:

Packages affected

[ ] @clerk/clerk-js
[ ] @clerk/clerk-react
[ ] @clerk/nextjs
[ ] @clerk/remix
[ ] @clerk/types
[ ] @clerk/themes
[ ] @clerk/localizations
[ ] @clerk/clerk-expo
[x] @clerk/backend
[ ] @clerk/clerk-sdk-node
[ ] @clerk/shared
[ ] @clerk/fastify
[ ] @clerk/chrome-extension
[ ] gatsby-plugin-clerk
[ ] build/tooling/chore

Description

[x] npm test runs as expected.
[x] npm run build runs as expected.

This is blocked as we're currently coordinating with the Vercel team in order to resolve all issues related to turbopack. I will post updates once I have more details.
In the meantime, I'll try to cherry pick anything that can be merged right away.

Closing in favor of #1391


Implement memory effects of volatiles

volatiles have certain memory effects when being read or when being written to.
For this we have to use some low-level stuff to implement the needed memory barriers.
See

http://gee.cs.oswego.edu/dl/jmm/cookbook.html
https://github.com/ReadyTalk/avian/blob/cbcc15bf7455bb10ae280a2813c00f4d4bb262d7/src/avian/x86.h#L245
https://github.com/ReadyTalk/avian/blob/cbcc15bf7455bb10ae280a2813c00f4d4bb262d7/src/avian/arm.h#L101
Hotspot? Not sure where they define which barriers to use.
Robovm? https://github.com/MobiVM/robovm/blob/45b1de02cad5d857918d7604414404323c75f715/compiler/vm/core/include/robovm/atomic.h
DalvikVM?
Android Runtime?

What about replacing field access by a couple of synthetic synchronized methods getter/setter and implement it as a programfeature for targets that do not implement volatile in a more performant way?

To be honest I don't really care about haxe or other targets. (Except maybe C# and c++)
That's why I'm not sure whether haxe does support synchronized.
It may be theoretically possible to replace volatile with synchronized but there are other parts of the Java Memory Model which have certain memory effects, e.g. writing to a final field.
So I'd like to ask how  you intend to implement synchronized for the haxe target?
I don't want to be negative but I don't think that haxe can implement this part of the JMM or the JMM at all.

The most basic primitive is the enter/exit monitor. The idea is to provide those two functions in N native class and use defines to chose the right implementation (when suitable). Javascript for example doesnt require it. But c++ can use it.
http://api.haxe.org/cpp/vm/Mutex.html
Until C++ target is mature (for example tools to target ios and or android native), we need to use Haxe/C++ for it.
For me, the most important targets are Js, C++ and C# and the usefulness of other targets are to be able to run java libraries in other languages.

For me not general language, for me need iOs, tvOS, windows native, Android, we spend 6 mouths band after 1 months we will be release... And we can not spend time for non Haxecpp targets, and very very need sync in jtransc. Thanks.

What do you mean with "tools to target ios and or android native"?
Which tools do you think are needed to mature the cpp target?

Yep thats why i want to support haxe target too. Until all that effort is backported we need to support haxe too. But even later i dont see problems supporting haxe too. Libgdx+libs like payments working on those platforms without modified code, providing app icons, one command for deploying... Those things. After implementing these required things + full java8 support i can help with toolchains to get the same experience with pure c++ target which should be supperior in runtime performance, compilation time and executable size. And if we can run original libgdx unmodified that would be even better! But it will take some months to achieve that and Sergey doesnt have that time for his release


Prefix css objects

Fixes: https://github.com/tkh44/emotion/issues/92
What:
autoprefix object styles at compile time.
Why:
Cross browser styles
How:
Update babel plugin to parse css array of objects

Checklist:

[ ] Documentation
[x] Tests
[x] Code complete

Codecov Report

Merging #101 into master will increase coverage by 0.46%.
The diff coverage is 96.96%.

@@            Coverage Diff            @@
##           master    #101      +/-   ##
=========================================
+ Coverage   88.44%   88.9%   +0.46%
=========================================
  Files          15      15
  Lines         571     604      +33
  Branches      122     134      +12
=========================================
+ Hits          505     537      +32
- Misses         53      54       +1
  Partials       13      13

Impacted Files
Coverage Δ

src/babel.js
99.35% <96.96%> (-0.65%)
: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 40162d4...7a6b2c7. Read the comment docs.

@tkh44 This is good to go. Take a look!

I could only take a quick look but im really excited.  I'm gonna look over it tonight

I'm going to merge this into my glamor branch.

Prefixing isn't the same. For example. display: flex yields:
for template literal:
display: -webkit-box;
display: -moz-box;
display: -ms-flexbox;
display: -webkit-flex;
display: flex;

for style object:
display: -webkit-box;
display: -ms-flexbox;
display: flex;

@andrewdelprete AFAIK, 7.0 beta fixed this. You can install the beta tag and try it

The last line in docs/objects.md should be updated.


How to Get QR Code as User

It would be exceptionally useful for there to be a link to an article describing how to generate a QR code so that you can configure your iPad with this method. This article talks about how to disable or enable the feature as an administrator. It seems quite relevant to know how to use this as a user as well.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 749a8389-eda6-3059-b713-5d0b3755fe9c
Version Independent ID: 53000b70-b561-ec33-0b94-f0c26650548b
Content: Use a QR code to sign-in to the Outlook mobile apps - Microsoft 365 admin
Content Source: microsoft-365/admin/manage/use-qr-code-download-outlook.md
Service: o365-administration
GitHub Login: @KwekuA
Microsoft Alias: kwekua

@aaronstackpole Thank you for your feedback. We do not have a document that talks about how a user receives QR code. However, this document has been updated with information
In Outlook on the web or other desktop Outlook applications, users may see notifications informing them that they can use Outlook on their mobile devices.
If users choose to send themselves an SMS text message to download the app on their mobile device, a QR code will appear on their computer.
They will be able to scan the QR code to log into Outlook on their phone or tablet.
Hope this helps!
Thanks
Sri

@msbemba Thank you very much for the contribution and sharing this explanation. @aaronstackpole Hope this comment is helpful for you. If you see a documentation update is required, please feel free to open an issue for the same. We proceed here to close it. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


Multiple conflicting contents for sourcemap source

Describe the bug
I am watching vue application with sourcemap enabled.
vite build --watch

After change of some .vue file, error is occured:
rendering chunks (1)...Multiple conflicting contents for sourcemap source
Reproduction
https://github.com/MerginMaps/mergin/tree/upgrade-to-vue-3
Steps to reproduce
cd web-app
yarn
yarn watch:lib
Change some .vue file in ./packages/lib
System Info
System:
    OS: Linux 6.2 Pop!_OS 22.04 LTS
    CPU: (16) x64 12th Gen Intel(R) Core(TM) i7-1260P
    Memory: 20.07 GB / 31.04 GB
    Container: Yes
    Shell: 5.1.16 - /bin/bash
  Binaries:
    Node: 17.9.1 - ~/.nodenv/versions/17/bin/node
    Yarn: 1.22.19 - /usr/bin/yarn
    npm: 8.11.0 - ~/.nodenv/versions/17/bin/npm
  Browsers:
    Firefox: 113.0.1
  npmPackages:
    @vitejs/plugin-vue: ^4.2.3 => 4.2.3
    vite: ^4.3.5 => 4.3.9

Used Package Manager
yarn
Logs
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

Duplicate of https://github.com/vitejs/vite-plugin-vue/issues/35


Refactor the logging in vitrivr-engine.

For productive operation, vitrivr-engine must be equipped with minimal logging. Minimal means that the user who only operates the api receives a clue to the cause of the error.

Closes. Since this is covered by #21