Issue with version 4.0.18 to zabbix

Hi,
Currently updated from 3.11 to 4.0.18
And saw that getting errors with zabbix sending, When i was going to update the json template of zabbix im getting template not compatible
I saw it was version 6.4 but currently i have 6.2 would it be compatible changing the version?

Thank you

updated zabbix server to 6.4


Add Github Action to update/create live progress website

With the addition of this GitHub action file, we now have a fully automated website set up to track our progress. The website code is entirely separate from this project, so nothing related to the website will ever need to be committed to this repo other than this single action file, and only ever a single commit to a new gh-pages branch.
Here's how it works:

Every push to the master branch on the main repository will trigger the action.
Github actions runs the steps in the file. It installs dependencies, runs a python script that parses the main repo's commits and populates that information into JS files which are imported by the website.
The website code is bundled up into a deployable folder.
This new deployable folder is commited to a new blank gh-pages branch.

The gh-pages branch will only ever have one commit, as any updates will force erase and overwrite existing gh-pages data when the site is re-built.
If you merge this, there is a one time thing you'll have to do. You have to go into the settings for this repository and set the GitHub Pages branch to gh-pages and save.
A live example on my fork:
https://mattbruv.github.io/bfbbdecomp/progress
I think this website will help put the progress into perspective, as well as let people easily understand where we are.
It has a blank FAQ page right now.
I'm terrible at web design, so I'm very slow at making a website, but I will improve it and make it better, and probably include more information and statistics.
I think it'd also be a good place to write tutorials for how to contribute to the decomp and answer technical questions too.
If you want, I can give you permissions for the website repo.

It even looks nice on mobile too!


add docs on expected responses for OAuth2 auth

While the recommended way of implementing OAuth 2.0 auth is using a ready-to-go middleware, some people choose to write their own and we've seen some confusion around the exact spec of the expected responses in the past. This adds some info on that to the docs to hopefully prevent that confusion in the future.
closes #1630

Thanks for this!


Feature/revised bootstrap upgrade

This is a continuation from the React/Redux upgrade. I'm submitting a PR now to give you the option of reviewing both together (and to save UI testing time). Included here:

Bootstrap 4 migration
some cosmetic adjustments/enhancements which are understandably subjective and can be reverted as need. Some of these changes were designed to move the UI toward future features like letting the user save/load shapes that each have their own distinct configuration.

@jeffeb3 I deleted the old PR. Please use this one when you review. It shows the proper diffs.

I have been looking at this, and I like it so much! I will merge and deploy this soon.


Add SQL transpilation option between dialects

Summary
This adds an optional plugin to configure a SQL transpilation library for transpiling between SQL dialects.
We'll use sqlglot by default, but additional transpilation libraries can be configured by implementing the SQLTranspilationPlugin base class with a custom transpile_sql method that is tailored to the library. Note that the base plugin implementation will handle checking that the package exists before loading and using it, or skipping transpilation if the package cannot be loaded. Thus, we'll always default to providing a generated query where we can, but will transpile if the transpilation library is available.
Test Plan
Will add tests shortly

[ ] PR has an associated issue: #758
[ ] make check passes
[ ] make test shows 100% unit test coverage

Deployment Plan

This looks great so far and the approach makes sense to me. How are you planning to integrate this into the /sql/ and /data/ endpoint? Since we don't intend to store the transpiled SQL and it's just something that happens at runtime right before the response goes out, I wonder if making it an injected dependency would be the best approach. That might allow for an alternative mechanism to add a completely custom transpiler without using the SQLTranspilationPlugin base class.
That's technically possible today but just needs to be documented and if DJ's internal transpiler logic uses the same approach (dependency injection), it'll serve as a good example for someone wanting to add an entirely custom one.

How are you planning to integrate this into the /sql/ and /data/ endpoint?

It's already integrated via the TranslatedSql model. The idea is that the DJ server admin could choose a SQL transpilation library (or none), and this lib would get loaded at server start and get used to transpile sql as a part of the model. If the server admin thinks that DJ's own transpiler is sufficient, they won't configure any external libraries and DJ will return queries exactly as before.


Is there any solutions on this.  I know, I know, this is not how I’m supposed to do it, but I can't think of something better. This is where it all begins...

Is there any solutions on this.  I know, I know, this is not how I’m supposed to do it, but I can't think of something better. This is where it all begins...
Originally posted by @Marshmallows7920 in https://github.com/Microsoft/msbuild/issues/2422#issuecomment-479697516

@msftdata Fix the fixes


upgrade postgres.py from 2.2.0 to 2.2.1

Picks up https://github.com/gratipay/postgres.py/pull/53.
See https://github.com/gratipay/gratipay.com/pull/3781#issuecomment-145632819.

/me waits for Travis ...


Problems creating config json with gcp-connector-util init

Installed on Raspberry Pi 2 with ArchLinuxARM (with AUR) via this script:
https://aur.archlinux.org/cgit/aur.git/tree/PKGBUILD?h=gcp-cups-connector
Problems with creating init config file while writing to file:
[...]
Acquired OAuth credentials for robot account
open /home/username: is a directory

sorry, everything is ok with the binary rpi package, so I should ask distro forum first.

I'm running into the same issue. Must have missed something in the PR merges on Wednesday.


KYLIN-1819 Fix exception swallowed when start DefaultScheduler fail

Can one of the admins verify this patch?


Building on Linux

https://github.com/rhasspy/webrtc-noise-gain/blob/0b3bc23f5f33101df5b8f1970919500d8629b955/webrtc-audio-processing/webrtc-audio-processing-1/rtc_base/event_tracer.cc#L12
using C++ on Linux, be sure to #define __STDC_FORMAT_MACROS before including inttypes.h.
source: https://stackoverflow.com/questions/8132399/how-to-printf-uint64-t-fails-with-spurious-trailing-in-format
current workaround is to pass -D__STDC_FORMAT_MACROS  in CFLAGS to compiller

I added #define __STDC_FORMAT_MACROS in webrtc-noise-gain/webrtc-audio-processing/webrtc-audio-processing-1/rtc_base/event_tracer.cc but compilation still fails.
Thiss is the output
systemlog-webrtcerror.txt


COBOL - Incorrect Detection of Unreachable Code when more than one ENTRY Statement is used

Development environment used
Version: 1.63.2 (user setup)
Commit: 899d46d82c4c95423fb7e10e68eba52050e30ba3
Date: 2021-12-15T09:40:02.816Z
Electron: 13.5.2
Chromium: 91.0.4472.164
Node.js: 14.16.0
V8: 9.1.269.39-electron.0
OS: Windows_NT x64 10.0.17763
Problem Description
If a COBOL program uses more than one ENTRY statement Z Open Editor marks the second ENTRY statement through the end of the program as unreachable.

Detailed steps for reproducing the problem:
Attached example COBOL
DBAS01P5.txt

Create a simple COBOL program, like the one attached above, with more than one Entry statement

Observed behavior
The part of the source from the second ENTRY statement to the end of the source is marked as unreachable.
Expected behavior
I would expect each ENTRY statement to be considered as executable and accessible therefore the following code to be executable according to normal execution rules.

I think the problem is the first GOBACK preceding the second ENTRY statement.  Since all the code is in the same section, nothing will get executed after the first GOBACK.  If you comment out the GOBACK, then the unreachable code warnings disappear.

Thanks for your quick response!
Yes, it is true the program could be redesigned so that both ENTRY statements are positioned such that they both use the same GOBACK statement and use a GOTO or IF statement to get to a single GOBACK statement.  That is a design and coding preference by the designer. At the same token having two GOBACK statements is appropriate in this case and is within the design of COBOL.  This type of coding is also used in Assembler where there are multiple ENTRY statements and multiple return points within the same load module.
I don’t know, but I was hoping that it wouldn’t be too difficult to reset the syntax checking if a second ENTRY statement is encountered and maybe move it into a loop or a change that would handle this coding design.  This is not a show stopper by any means, but I do love the Z Open Editors ability to check syntax and notify when there is unreachable code. I don’t want to turn off that feature.
Thank you,
@.***
From: gplambert @.>
Sent: Tuesday, January 18, 2022 7:43 AM
To: IBM/zopeneditor-about @.>
Cc: Coblentz, David @.>; Author @.>
Subject: Re: [IBM/zopeneditor-about] COBOL - Incorrect Detection of Unreachable Code when more than one ENTRY Statement is used (Issue #212)
I think the problem is the first GOBACK preceding the second ENTRY statement. Since all the code is in the same section, nothing will get executed after the first GOBACK. If you comment out the GOBACK, then the unreachable code warnings disappear.
—
Reply to this email directly, view it on GitHubhttps://urldefense.com/v3/__https:/github.com/IBM/zopeneditor-about/issues/212*issuecomment-1015478592__;Iw!!OgnrLJOUc50AAzM!cqiT3gXXEF5tsa8i7I8xtj2Sp1dGCRnpgCl89OeybtMU4fDXx8Cd2N6yfh6DDIyvLA$, or unsubscribehttps://urldefense.com/v3/__https:/github.com/notifications/unsubscribe-auth/ATC2YG7NDKJSYCUIGXISKYDUWV35PANCNFSM5L5BODCQ__;!!OgnrLJOUc50AAzM!cqiT3gXXEF5tsa8i7I8xtj2Sp1dGCRnpgCl89OeybtMU4fDXx8Cd2N6yfh4oieirtw$.
Triage notifications on the go with GitHub Mobile for iOShttps://urldefense.com/v3/__https:/apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675__;!!OgnrLJOUc50AAzM!cqiT3gXXEF5tsa8i7I8xtj2Sp1dGCRnpgCl89OeybtMU4fDXx8Cd2N6yfh7GOOIdfg$ or Androidhttps://urldefense.com/v3/__https:/play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign*3Dnotification-email*26utm_medium*3Demail*26utm_source*3Dgithub__;JSUlJSU!!OgnrLJOUc50AAzM!cqiT3gXXEF5tsa8i7I8xtj2Sp1dGCRnpgCl89OeybtMU4fDXx8Cd2N6yfh7XuRh4MA$.
You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>

Hi,
Problem is more general than "more than one ENTRY statement":

This problem is similar to the management of EXEC SQL WHENEVER xxx GOTO label END-EXEC, EXEC CICS HANDLE cond(label) END-EXEC, EXEC CICS HANDLE ABEND LABEL(label) END-EXEC, EXEC CICS HANDLE AID xxx(label) END-EXEC, except that the call to entry point is not explicit.
The COBOL ENTRY statement is both a declaration and a starting point for a new sequence of code.
Perhaps, consider also SERVICE LABEL directive, which defines an execution resume point after an interruption.
See CEEMRCE—Move resume cursor explicit


Extending • Swift-class component returns nil

Extending with custom components doesn't work for Swift classes.
There is an issue in JasonComponentFactory in method build:withJSON:withOptions::
NSClassFromString(componentClassName) returns nil for Swift classes, because Swift class name have a different format:
<appName>.<className> or <appName>_<targetName>.<className> instead of just <className> in Obj-C classes.
Therefore component class names must have a prefix for Swift classes, i.e. MyAppName.JasonSuperComponent

I believe this maybe related to this https://medium.com/@maximbilan/ios-objective-c-project-nsclassfromstring-method-for-swift-classes-dbadb721723
I havent got the opportunity to integrate swift with objective-c since swift became abi stable not so long ago.
Please provide some example code (upload a zip here) to test this.
Thanks.

@clsource well, yes. It's related to the way Swift class names are generated and consequently to the usage of NSClassFromString in Jasonette internals.
That's why in JasonComponentFactory it should extract a raw name (without module and target names) and only then use it in [NSString stringWithFormat:@"Jason%@Component", capitalizedType]

Try this in your swift code
https://stackoverflow.com/a/30895110
All swift classes use the Product Module Name a dot and the classname for their namespace (Module.Class). If you wanted to use "MySwiftClass" name for the class within your Objective-C code; you can add @objc(MySwiftClass) annotation to expose the same swift class name (without the module):
@objc(MySwiftClass)
class MySwiftClass{
    ...
}```

Then

Class myClass = NSClassFromString(@"MySwiftClass");

Will contain the class instead of being nil.

If thats not working maybe this addition to the JasonComponentFactory.m will do.

+ (UIView *)build:(UIView *)component withJSON:(NSDictionary *)child withOptions:(NSMutableDictionary *)options{

    NSString *capitalizedType = [child[@"type"] capitalizedString];
    NSString *componentClassName = [NSString stringWithFormat:@"Jason%@Component", capitalizedType];

    if(componentClassName){
        Class<JasonComponentProtocol> ComponentClass = NSClassFromString(componentClassName);

        if(!ComponentClass) {
            // Maybe a is a Swift Component. NSClassFromString return nil on these.
            // see https://stackoverflow.com/questions/28706602/nsclassfromstring-using-a-swift-file
            NSString *prefix = [[NSBundle mainBundle] infoDictionary][@"CFBundleExecutable"];
            NSString *swiftClassName = [NSString stringWithFormat:@"%@.%@", prefix, componentClassName];
            ComponentClass = NSClassFromString(swiftClassName);
        }

@RomanSlyepko it works?

yes, @objc(MySwiftClass) works fine


docs(ee-kubernetes) Cleaning Kubernetes topics and nav tweak

Adding a compatibility links section to "Using Kong for Kubernetes"
Minor edits for typos, grammar, and header depth
Based on feedback from team, moving deployment topic up in the nav
Rephrasing and some cleanup on intro page - @rainest can you look over the changes to /kong-for-kubernetes/index.md and let me know if this content is accurate? It doesn't seem to fully line up with your deployment topic.

@GayleNeumann PTAL


Docs housekeeping

Proposed Changes

Docs updates with minor tweaks for URLs/Emails
Update GITHUB_URL in install.sh to avoid redirect

Types of Changes

Minor docs changes
Minor install.sh change
Verification

I tested each change where a URL was updated and the change I made to install.sh to ensure I didn't break anything.
Linked Issues

For https://github.com/k3s-io/k3s/issues/2970
Further Comments

These changes are minor, largely to avoid redirects in documentation (rancher/k3s to k3s-io/k3s). I also updated MAINTAINERS emails to point to new suse.com emails. This PR doesn't encompass all references to rancher/k3s - there are areas in code where we might want to change this later. This PR just touches .md files in the root of the project and a small changes to install.sh

Needs signoff but lgtm!

Fixed thank you! Merging.


fix: race in dynamicMapKeyEncoder/dynamicMapKeyDecoder

try fix concurrent access to ctx.encoders/decoders while marshal map with key type interface{}, as #618 shows

@taowen


Better tutorial

Steps work on bash and powershell, but you should still go f*** yourself if you use Windows

I am working an my tutorial just at that moment. I'll take a look at your commit in a moment. I've just realized that mvn install automatically installs to the correct path. So I guess I can simplify a lot.

Bash is industry standard, all my homies write bash-compatible tutorials


Changed default from HTTP to HTTPS. Added check for valid URL

List of changes:
Default address changed from http://localhost:2333 to https://localhost:2333
If user enters host address without either http:// or https:// in front, then https:// will be added.
Check if host URL is a valid URL, by means of regex.
Removed unused package imports.
Changed payload content-type from application/json to text/plain (without enclosing double-quotes)
All changes tested.

These were not wrong in the client. It's because I made changes to RioDB API.


core-js 2 deprecated

🐞 bug report
Is this a regression?

 Yes and no - it's the underlying npm dependencies that have changed to cause this problem, not angular itself.
(problem was not present on the 20th of December, for example)
Description
When creating a new angular package, or doing an npm install, we now receive a warning that core-js 2 is deprecated and we should be using 3.
🔬 Minimal Reproduction
ng new
(create a new angular 8 project)
🔥 Exception or Error

npm WARN deprecated<EMAIL_ADDRESS>core-js@<3 is no longer maintained and not recommended for usage due to the number of issues. Please, upgrade your dependencies to the actual version of core-js@3.

🌍  Your Environment
Angular Version:

Angular CLI: 8.3.21
Node: 10.15.1
OS: win32 x64
Angular: 8.2.14
... animations, common, compiler, compiler-cli, core, forms
... language-service, platform-browser, platform-browser-dynamic
... router

Package                           Version
-----------------------------------------------------------
@angular-devkit/architect         0.803.21
@angular-devkit/build-angular     0.803.21
@angular-devkit/build-optimizer   0.803.21
@angular-devkit/build-webpack     0.803.21
@angular-devkit/core              8.3.21
@angular-devkit/schematics        8.3.21
@angular/cli                      8.3.21
@ngtools/webpack                  8.3.21
@schematics/angular               8.3.21
@schematics/update                0.803.21
rxjs                              6.4.0
typescript                        3.5.3
webpack                           4.39.2

Additional comments:
We might need a new version of angular 8 to fix this problem (8.2.15)

After looking into this a bit, it seems like the underlying issue is that @angular-devkit/build-angular should probably find a way to avoid using istanbul-instrumenter-loader is possible.  That package has not seen a release for 2 years, and has a dependency on a version of istanbul-lib-instrument that dates back to June 2017 (v1.7.3 vs v4.0.0 released just a few days ago).  The loader package is just a very thin wrapper around createInstrumenter:
https://github.com/webpack-contrib/istanbul-instrumenter-loader/blob/master/src/index.js
If we could find a way to do the equivalent of that directly within @angular-devkit/build-angular without pulling in istanbul-instrumenter-loader, we would avoid the core-js@2 issue, and likely many other outdated dependency issues, entirely.


Added Stoplight

I've added Stoplight to the Tooling— Development Portal and Documentation section in the README.md file.
Stoplight is a comprehensive platform for designing, testing, and documenting APIs, and I believe it's a valuable addition to the resource.
Please review and merge this PR. Thank you!

👍 @patrick204nqh


add awesome cms core

[x] verified that the CMS I'm adding is still maintained.
[x] read CONTRIBUTING.md.
[x] did not generate README.md.

Hi @mutewinter Please review it again.
Thank you.

Hi @mutewinter Please review it again.
Thank you.

Hi @mutewinter Please review it again. Thank you.

Hi @mutewinter any update on this ?

Hi @mutewinter any update on this ?

Thank you, @ngohungphuc!


Fix layout of file upload in add event dialog

This patch fixes the table layout of the upload section in the add event dialog which had overlapping elements.

Not the actions again have their own small column.

The problem was caused by #626

looks OK again


define LIBMEMCACHED_AWESOME

I think nobody should need this for code, as everything should be compatible.
But it may be useful to report which project is used
Example
$ php --ri memcached

memcached

memcached support => enabled
Version => 3.1.5
libmemcached version => 1.1.0
...

This may allow to adapt output and avoid user confusion, such as
libmemcached-awesome version => 1.1.0
Alternative way may be to change LIBMEMCACHED_VERSION_STRING = "1.1.1 (awesome)", but this may raise issues in some project.

Codecov Report

Merging #115 (6e732f4) into v1.x (e5c1347) will increase coverage by 1%.
The diff coverage is n/a.

@@           Coverage Diff           @@
##            v1.x    #115     +/-   ##
=======================================
+ Coverage     68%     69%     +1%
=======================================
  Files        191     188      -3
  Lines      17445   15026   -2419
=======================================
- Hits       11888   10396   -1492
+ Misses      5557    4630    -927

Impacted Files
Coverage Δ

test/fixtures/parser.hpp
88% <0%> (-9%)
:arrow_down:

src/libhashkit/string.cc
42% <0%> (-6%)
:arrow_down:

src/bin/memcapable.cc
85% <0%> (-6%)
:arrow_down:

src/libmemcached/dump.cc
65% <0%> (-4%)
:arrow_down:

src/bin/common/checks.hpp
60% <0%> (-3%)
:arrow_down:

contrib/bin/memaslap/ms_task.c
70% <0%> (-3%)
:arrow_down:

test/lib/Shell.cpp
69% <0%> (-2%)
:arrow_down:

contrib/bin/memaslap/ms_conn.c
44% <0%> (-2%)
:arrow_down:

contrib/bin/memaslap/ms_setting.c
75% <0%> (-2%)
:arrow_down:

test/lib/Retry.cpp
92% <0%> (-2%)
:arrow_down:

... and 126 more

Continue to review full report at Codecov.

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

New maintenance of libmemcached makes me irrationally happy!


Make this Module more accessible to moar devs.

This is an attempt at hopefully opening up what this Module attempts to do to more developers.
For starters, I've added all the files needed for this to be synced and imported to android studio and just compiled.  Everything is there for testing.
Secondly, I've fleshed out checks and logging IMMENSELY.
I have added every boolean check that can be done on a file class, as well as a number of the path checks that could be executed.
I have added a method - checkFileBool - that we can invoke from our other hooks.  This can iterate over any number of "keywords" that could be checked for to determine Pay Incompatibility.  So, instead of checking a hundred variables for a hundred different possible methods, we run the same check against that method instance.  If one of these hooked methods matches a keyword, it will automatically return "false" or "" for those values (try to fool it into thinking they're not there).
Using this method of reflection, I have been able to determine that it is indeed looking for xposed files as well.  Still can't figure out how it's looking for /su/bin in systemless files yet.  I think if we can crack this, we can make more progress.
As of right now, this still doesn't work.  My code changes are simply for exploration purposes.  As we can't look at the secureNet crap directly, we have to get the OS to tell us what is being checked for where, and then figure out the right information to pass back to make those checks succeed.
Hopefully, we can get this merged and entice some other people to help us in this endeavor.

Merged, yea!

What is interesting: http://forum.xda-developers.com/nexus-5x/general/passing-safetynet-root-t3307659/post65154474#post65154474
so chmod...

That was precisely what made me take a look at this again - knowing that something somewhere MUST be either checking for the presence of the /su partition, su/bin/su, or something.  Hence, me adding every method for the file.class I could work in in a day's time.  ;)


Bank Tag Tabs Close When Switching Withdrawal Type

Please check if your issue is not a duplicate by searching existing issues
Bug
When viewing a runelite bank tab on the left, clicking between note and item will make the rl bag tab disappear
To Reproduce

Open Bank
Click a runelite bank tab (on the left)
click on note (withdraw as) while having been on item before
the tab closes

Expected behavior
I should be able to select note or item and still use the tab
Screenshots
If applicable, add screenshots to help explain your problem.
Environment (please complete the following information):

OS: Windows 10
RuneLite version: <IP_ADDRESS>
Launcher version: 1.6.0

This cannot be reproduced.

Just attempted to reproduce this and couldn't.
Can you go into more detail about your game settings (bank settings etc.) or perhaps provide a streamable of the problem occurring.

It was happening over and over, but I can't seem reproduce it either now,


ambiguous reference problem after porting to dotnet core

Describe the bug
In this example, I want to show a bug in aws porting assistant tools. Legacy dotnet projects show files that are included in csproj file, but new projects don’t store the file list. It shows all files in the folder.
When I try to port the legacy project to dotnet 7, it created an ambiguous reference problem.
Expected behavior
As I understood, this tool should delete excluded csharp files before the porting process.
Screenshots

@tdemir thank you for reporting the bug. We are looking into the issue and will provide updates soon.

At this time we treat the excluded file as normal file(config/text file) and copy it as is, also there is no way to know deterministically for the tool that this file is excluded. We will look into the options for Include/Exclude MSBuild properties inside csproj/vbproj, but that would cover some scenarios , not all.


Run tagin:sync nightly

This will require adding google creds so that the script doesn't prompt at runtime.

Added creds https://github.com/jcraigk/phishin/commit/2d2aed256e084a6ebc658601d87c8217e70f4d0b and setup the daily crontab.


IntersectionObserver

Spec: https://github.com/WICG/IntersectionObserver/
Mdn: https://developer.mozilla.org/en-US/docs/Web/API/Intersection_Observer_API
Chrome: 51+ https://www.chromestatus.com/feature/5695342691483648
Opera: 38+
Edge: feature request link and Roadmap link Status: In Development
FireFox: https://bugzilla.mozilla.org/show_bug.cgi?id=1243846
WebKit(Safari): https://webkit.org/status/#specification-intersection-observer

+1

+1

FireFox bug: https://bugzilla.mozilla.org/show_bug.cgi?id=1243846

+1

+1

Under Consideration with Medium priority by Microsoft: https://developer.microsoft.com/en-us/microsoft-edge/platform/status/intersectionobserver

+1

+1

+1

Something to make mention of:
Chrome's implementation is fundamentally broken, atleast up to the current 53.0.2783.2 dev-m version. The way it observes threshold changes and enqueus change records does not adher to spec.
According to spec, a change record is supposed to be enqueued when the active threshold changes.
The active threshold is computed as follows:

when intersection ratio is 0, the threshold is 0
when intersection ratio is 1, the threshold is the length of the thresholds array (i.e. one index greater than the number of thresholds)
otherwise, the threshold is the index of the first threshold with a value greater than the intersection ratio.

(Source: Draft spec, section 3.2.5 - algorithm step 3.2.8)
For the default { threshold : [0]} setting, Chrome currently already enqueues a change record when the intersectionRatio is exactly 0. This makes it impossible to use a simple intersectionRatio > 0 test to determine if an element just became visible or invisible and have it work correctly in any reliable way.

@rjgotten Is there a Chrome bug report for that?

@rjgotten I filed https://crbug.com/646202 based on your description. If you have further details (or a testcase), it would be great if you could post them on that bug.

WebKit: In Development: https://webkit.org/status/#specification-intersection-observer

Edge: Intersection Observer is In Development - https://github.com/MicrosoftEdge/Status/pull/464

@rjgotten Apparently that's working as intended; see https://bugs.chromium.org/p/chromium/issues/detail?id=646202#c5

Apparently that's working as intended

Yeah; this is a case of Google retro-actively messing with their own spec.
Note also: https://github.com/WICG/IntersectionObserver/issues/156

+1

@cvrebert
After some further digging, I'm inclined to believe that this is not working as intended at all, and that the engineer replying in that tracker issue is doing little other than offering a shoddy workaround wrapped as an end-solution.
If the use of the Number.MIN_VALUE value to detect if an element is intersecting the target bounds for any amount of non-zero intersection, would indeed by design; and if it should indeed be used instead of the 0 value that Google's own spec states, then why is the default thresholds value in Chrome's implementation still a known-broken value of [0]?
(Yeah; check that for yourself: new IntersectionObserver(function(){}).thresholds )

+1

Now available at http://caniuse.com/#feat=intersectionobserver

+1


Add all types/@ember__* packages explicitly

The @types/ember package contains them, but in order for automatic imports to work in VS Code, individual types/@ember__* packages must be explicitly listed in package.json.
https://www.typescriptlang.org/dt/search?search=@typesember

As mentioned in the other PR, I am a bit worried having to do this everywhere. Do you have any other confirmations that this is really required for VSCode? (SO, Github issue etc.)
Maybe this gets fixed in VSCode itself eventually?

As mentioned in the other PR, I am a bit worried having to do this everywhere. Do you have any other confirmations that this is really required for VSCode? (SO, Github issue etc.)
Maybe this gets fixed in VSCode itself eventually?

As mentioned in the other PR, I am a bit worried having to do this everywhere

I don't see an issue here. The boilerplate addon will do it automatically.

Do you have any other confirmations that this is really required for VSCode? (SO, Github issue etc.)

Nope.

Maybe this gets fixed in VSCode itself eventually?

I can try starting an issue. 🤷♂

As mentioned in the other PR, I am a bit worried having to do this everywhere

I don't see an issue here. The boilerplate addon will do it automatically.

Do you have any other confirmations that this is really required for VSCode? (SO, Github issue etc.)

Nope.

Maybe this gets fixed in VSCode itself eventually?

I can try starting an issue. 🤷♂

I don't see an issue here. The boilerplate addon will do it automatically.

But what about future updates? Say @types/ember introduces a new types/@ember__foo package as a dependency:

when dependabot updates @types/ember, it does not know about adding types/@ember__foo
we could add types/@ember__foo to the boilerplate addon, but

would need to know it has been added
do the manual work in the boilerplate addon
do the manual work in all apps re-running the boilerplate blueprint

I don't see an issue here. The boilerplate addon will do it automatically.

But what about future updates? Say @types/ember introduces a new types/@ember__foo package as a dependency:

when dependabot updates @types/ember, it does not know about adding types/@ember__foo
we could add types/@ember__foo to the boilerplate addon, but

would need to know it has been added
do the manual work in the boilerplate addon
do the manual work in all apps re-running the boilerplate blueprint


[Backport 18.09] nixos/networkmanager: fix VPN plugin service definition targets

Motivation for this change
#46197
NetworkManager VPN plugins have moved their nm-${plugin}-service.name files from /etc to /lib.
Backport of #46201 to 18.09
cc @jtojnar @tadfisher
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Fits CONTRIBUTING.md.

Is this just a cherry-pick from #46201  (which I just merged), or is there a difference?

@xeji That's just the cherry-pick, don't know why GrahamcOfBorg added all these labels…

@flokli thanks for doing this. But there's no need to open a separate PR for a simple cherry-pick , just mention on the master PR that it needs backporting.
In the future, if you cherry-pick from master, please use git cherry-pick -x to include a reference to the original commit for better traceability.
Cherry-picked from master to 18.09 in 43c9869dc57

@xeji Thanks!


Look for minio and outside storage env vars

follow the lookup rules that builder and workflow use:

if DEIS_MINIO_SERVICE_HOST and DEIS_MINIO_SERVICE_PORT exist, use those for the S3 location
if DEIS_OUTSIDE_STORAGE_HOST and DEIS_OUTSIDE_STORAGE_PORT, exist, use those for the S3 location

#24 handles using storage based on appropriate environment variables.Currently it doesn't handle minio as there are few issue with it.Created a issue to fix it #25.


Bugfix: Display locks only for the correct  application

In the previous implementation locks would also be displayed on apps that names are contained in a locked app's name.

we should keep the fuzzyness on the lockspage. Maybe split up in 2 functions, name one "fuzzy" the other "exact", and use the fuzzy one on the lockspage.

Changed searchCustomFilter back. Not sure though if a special function is needed for this case.


Config for conversion ratios

Any way we could get the ability to set the conversion ratio of the different engines and the dynamo? Thanks!

Would definitely love a config for the BWM converter at the very least as I do not know why such a poor conversion ratio was chosen.


Fix building of protoc artifacts on Windows.

The link https://googlemock.googlecode.com/files/gmock-1.7.0.zip is dead. We could also update the link, but disabling the protobuf tests speeds up building and we only need the protoc artifacts anyway.
In this build, the windows protoc artifacts have been build successfully:
https://googlemock.googlecode.com/files/gmock-1.7.0.zip
CC @apolcyn

I don't think the failures here are related to this change - the csharp portability failure looks like timeout while downloading packages.
Is there another place in the pipeline before this where the relevant protobuf tests are ran?Otherwise LGTM


NullReferenceException in DxfReader.ReadLayerStateProperties

Some files cause a NullReferenceException because GetObjectByHandle returns null.
Example: linetype_issue.zip
System.NullReferenceException
  HResult=0x80004003
  Message=Object reference not set to an instance of an object.
  Source=CADability.Core
  StackTrace:
   at netDxf.IO.DxfReader.ReadLayerStateProperties(String name, IEnumerator`1 enumerator) in ...\CADability\netDxf\IO\DxfReader.cs:line 11105

PR to solve this will follow

Hi @Steinblock
I've updated the netDxf library to the latest version.
This should include the changes you backported from the original source code.
After a short test with the DXF file you attached everything seems to work fine.
Can we close this issue?

@dsn27
Yes. I just checked it and all my dxf tests (~110 files) have no issues with the current version.


Show the number of nodes selected

There's an arbitrary limit of 400 nodes to see a node-link diagram. To help filter the right amount, we should show how many nodes have been brushed.

Track this work at https://github.com/divyamistry/VisPNet/pull/6

Done. Code merged with PoC implementation.


Add note about changing AAD tenant

If you change the tenant and have users assigned from the existing tenant, we will not allow you to create a new collection. This documentation change addresses this issue.
Please relay to Liza Poggemeyer.

Hi @cdavid, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no contribution license agreement is required at this point. Real humans will now evaluate your PR.

TTYL, AZPRBOT;

@lizap - Hey Liza, can you review this PR when you have a few minutes?

The content is needed. Let's merge it now and see how it shakes out with the directory changes. If the merge doesn't work due to directories, I'll add the note again in the private repo.


Completion of error handling

Would you like to add more error handling for return values from functions like the following?

pthread_mutex_init ⇒ Mutex constructor
pthread_cond_wait ⇒ CondVar::Wait

Fixed, thanks for your review.

Are there a few update candidates left over?

fclose ⇒ DumpResponse
signal ⇒ main


test: remove use_standard_sql usage

Removed use_standard_sql parameter usage from tests

Integration tests: https://github.com/firebolt-db/firebolt-go-sdk/actions/runs/8643987376


Create main222.yml

:tada: Snyk checks have passed. No issues have been found so far.
:white_check_mark: code/snyk check is complete. No issues have been found. (View Details)

ee

eeeee


Automator: update istio/client-go@master dependency in istio/istio@master

Generated by Automator - 2022-06-21T16:02:06+00:00

@istio-testing: PR needs rebase.

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.

@istio-testing: 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

integ-cni_istio
d716329274d82ec9036cbdb7236e1845227241cf
link
true
/test integ-cni_istio

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 integ-cni_istio


ContactListPage and QuoteListPage not showing list content

With the sample, I've tried a couple of different checkouts and the head versions of FreshMvvm and the latest nuget package, but in all cases the ListView is blank.
Is anyone else having this problem or is it just me?

What platform are you on?

Can't reproduce, I don't think it's an issue?


Building grpc-python exits with "library not found for -lrt"

Hi,
I installed grpc as described here and tried to build the python bindings using "pip install src/python/src".
The build fails with:
...

ld: library not found for -lrt

clang: error: linker command failed with exit code 1 (use -v to see invocation)

error: command 'clang' failed with exit status 1

I am on the mac. Can you tell me what I am missing?
I will gladly provide more information if you tell me what you need.
Thanks!

librt doesn't exists under MacOS - nor under certain versions of Linux in fact. This is a bug that needs fixing.
As a note, sometimes librt contains real time clock functions that are needed to compile grpc.

I believe this has been fixed. Thanks, @atombender! :+1:


add awaitStart() to API

I often find myself writing code like this:
function awaitLibp2pStart() {
  return new Promise(resolve => libp2p.isStarted() ? resolve() : libp2p.once('start', resolve))
}

async function doSomething() {
  await awaitLibp2p2Start()
  libp2p.dial(...)
}

It would be nice if this were part of the API instead (same goes for js-ipfs)

Suggestion:
We could take advantage of allowing idempotent starts like this:
await libp2pNode.start() // ok
await libp2pNode.start() // also ok, returns immediately

Example of implementing this in peer-star:
https://github.com/ipfs-shipyard/peer-star-app/blob/master/src/app.js#L26-L70

The latest libp2p state machine additions (0.24.0) include idempotent starting and stopping, so you can just do:
function awaitLibp2pStart() {
  return new Promise(resolve => libp2p.start(resolve))
}

We're looking at the async/await API changes as part of the async iterator initiative.
This would result in an api like @pgte mentions above.
For async code I've been extending libp2p to test out possible api changes:
class AsyncLibp2p extends Libp2p {
  start () {
    return new Promise((resolve, reject) => {
      super.start((err) => {
        if (err) return reject(err)
        resolve()
      })
    })
  }
  // ...
}

If it's being used in conjunction with IPFS it would need to handle fallback support for the callback.

We switched to async await now, and it is possible to just do await libp2p.start(). So, I am closing this issue


Cloud Firestore - Getting a document from an array in a StreamBuilder

Hi there, fine people.
I’m in need of some small assistance with using Cloud Firestore using Flutter (Dart).
I have a users collection that contains some fields like firstName, but also an array called groceryLists that contains references to documents in the grocerylists collection.
The grocerylists collection:

The users collection:

I’m trying to get a stream of snapshots of these grocerylist documents.
Right now I’m using
stream: Firestore.instance
      .collection('users')
      .document('...longUserId...')
      .collection('groceryLists')
      .snapshots()

and checking if I’ve got the values by printing the values with debugPrint(snapshot.data.documents.toString());, but I seem to get back an empty array ([]).
However, when doing the follow, I do get an array of DocumentReference objects:
stream: Firestore.instance
      .collection('users')
      .document('...longUserId...')
      .snapshots()

...
debugPrint(snapshot.data.data['groceryLists'].toString());
(result: [Instance of 'DocumentReference']).
I need to get the groceryList references through a stream however, since I’m using a ListView builder.
Does anybody know how I can accomplish this?
The Working with Arrays, Lists, and Sets help page isn’t helping me.
Thank you very much in advance.
Steps to Reproduce
Check the above.
Logs
> flutter analyze
Running "flutter packages get" in hsb...                     0,6s
Analyzing /Users/jeroen/Projects/_temp/hsb...
  hint • Unused import: 'user.dart' at lib/models/grocerylist.dart:3:8 • unused_import
  hint • Unused import: 'product.dart' at lib/models/grocerylist.dart:4:8 • unused_import
  hint • Unused import: 'package:firebase_auth/firebase_auth.dart' at lib/pages/bodies/home_body.dart:5:8 • unused_import
  hint • Unused import: '../../ui/grocerylist_list_tile.dart' at lib/pages/bodies/home_body.dart:8:8 • unused_import
  hint • Unused import: 'dart:async' at lib/pages/bodies/home_body.dart:13:8 • unused_import
  hint • This class inherits from a class marked as @immutable, and therefore should be immutable (all instance fields mustbe final) at lib/pages/bodies/home_body.dart:15:7 • must_be_immutable
  hint • This class inherits from a class marked as @immutable, and therefore should be immutable (all instance fields mustbe final) at lib/ui/product/product_list_tile_control_panel.dart:8:7 • must_be_immutable
  hint • This class inherits from a class marked as @immutable, and therefore should be immutable (all instance fields mustbe final) at lib/ui/product_list_tile.dart:7:7 • must_be_immutable
8 issues found.
(Ran in 5.9s)

> flutter doctor -v
[✓] Flutter (Channel master, v0.4.3-pre.62, on Mac OS X 10.13.4 17E202, locale nl-NL)
    • Flutter version 0.4.3-pre.62 at /Users/jeroen/Development/flutter
    • Framework revision 1c27a458a8 (21 hours ago), 2018-05-09 12:31:36 -0700
    • Engine revision 11d6da3cd9
    • Dart version 2.0.0-dev.53.0.flutter-e6d7d67f4b

[✗] Android toolchain - develop for Android devices
    ✗ Unable to locate Android SDK.
      Install Android Studio from: https://developer.android.com/studio/index.html
      On first launch it will assist you in installing the Android SDK components.
      (or visit https://flutter.io/setup/#android-setup for detailed instructions).
      If Android SDK has been installed to a custom location, set $ANDROID_HOME to that location.

[✓] iOS toolchain - develop for iOS devices (Xcode 9.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 9.3, Build version 9E145
    • ios-deploy 1.9.2
    • CocoaPods version 1.5.0

[✗] Android Studio (not installed)
    • Android Studio not found; download from https://developer.android.com/studio/index.html
      (or visit https://flutter.io/setup/#android-setup for detailed instructions).

[✓] VS Code (version 1.23.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Dart Code extension version 2.12.1

[✓] Connected devices (1 available)
    • iPhone van Jeroen Meijer • 0a6b212cab22ad19207521c159c5d8e305fe1e52 • ios • iOS 11.3

! Doctor found issues in 2 categories.

Could be a simple typo.  The property groceryLists on your users document is just that: a property, not a collection. So calling .document(userId).collection('groceryLists') should return an empty array, since you're referencing an empty subcollection. If you want to get only the grocery lists in the groceryLists array, you could query the collection documents, then filter the array using the .where() function.

Hi there.
Thank you for your detailed answer.
It wasn't a typo, but your method does work.
Querying the groceryLists collection and then using .where() seems to be the best option.
I'm still wondering, isn't there an option at all to get a stream (of snapshots) from the groceryLists array within the user document?

Can this issue be closed or do you think there is still something for the Flutter team to do?

You can do a simple forEach() for this purpose.
groceryList.forEach((index, value) {
})

Anyone figured this out?

My apologies for the lack of a response ony my end.
So after a few months of more experience, I've learned that doing this type of query is (currently) impossible. You can query for documents that are referenced inside and array of another document, which is what I wanted to do.
The best solution for this type of problem is to, in this case for example, save the grocery list in another collection and save the user reference inside it.
When you want to retrieve all the groceryLists from the user, you can either query the user document and then use the list of Document References to get all the lists, or (the better method) get the user's document reference and then do a .where('user', isEqualTo: userDocRef). The latter allows you to get all grocery lists with one simple query, which is faster and lighter.
In order to keep the user field in the groceryList documents synced up with the groceryLists field in the user document, just use some Cloud Functions.
Hope this helps anyone having trouble with something similar to this use case.

how to use it?

How to fetch products in this case......can someone help me out.
I have this kind of issue in my application..Thanks in advance

hi i have a almost a similar problem  i  have a medical app i have several collections ,  the customer , wanted to make phone login which i did successfully but now i need  to show now i have a problem with geting the document id which is offcourse the same of the userid from authintication

i did this code
`class _profilState extends State {
String _name;
String _age;
String _address;
String _user;
int _phone;
@override
void initState(){
super.initState();
FirebaseAuth.instance.currentUser().then((user){
_user =user.uid;
}).catchError((e)
{
print(e);
});
}
@override
Widget build(BuildContext context) {
`  return StreamBuilder(
  stream: Firestore.instance.collection('profile').snapshots(),
  builder: (context, snapshot) {
    if (!snapshot.hasData) {
      return Center(
        child: Text('loading'),
      );
    }

then i tried to make each user when he/ she  login  and go the user profile  get his full information i used this  code
Text("age:" +  snapshot.data.documents[_user]['name'].get(),)
`
the problem is it doesnt show any thing and i get an error
but when i edit the code and change it and give the number like this
Text("age:" +  snapshot.data.documents[1]['name'].get(),)
`
it works
i need to make it show for profile of each user  please help me

https://github.com/FirebaseExtended/flutterfire/issues/502

I am also facing the same problem as @moahaimen mentioned so  i am using document instead of snapshot.But when i am using snapshot to fetch details from cloud firestore i am getting the red screen(error screen) but when i  do hot reload its working fine and reflecting the data,but if i restart i am facing the same issue.
Before hot reload when running screen  looks like:
And when i do hot reload it looks perfect
Could someone look at the issue and suggest the changes in code.
`
class AccountInfo extends StatefulWidget
{
@override
_AccountInfoState createState() => _AccountInfoState();
}
class _AccountInfoState extends State {
var info;
String uid;
String name;
String email;
@override
initState(){
super.initState();
FirebaseAuth.instance
.currentUser()
.then((currentUser) => {
if (currentUser == null)
{
print("no user");
}
else
{
uid=currentUser.uid.toString(),
email=currentUser.email.toString(),
print("before"+uid),
print(email),
information().gettingUserInfo(uid).then((QuerySnapshot docs){
if(docs.documents.isNotEmpty)
{
info=docs.documents[0].data;
print("xvv completed");
print(info['fname']);
}
                }),
            }

        })
    .catchError((err) => print(err));

            }

@override
Widget build(BuildContext context) {
// TODO: implement build
return MaterialApp(
debugShowCheckedModeBanner: false,
home: Scaffold(
appBar: AppBar(
title: Text("Account Info"),
backgroundColor:Colors.blue,
),
bottomNavigationBar: bottomnavigation(context,1),
drawer: theDrawer(context),
body:WillPopScope(
onWillPop: (){
Navigator.pushNamed(context, 'HomeScreen');
},
child:Column(
children: [
Text(info['fname']!=null?info['fname']:""),
Text(info['surname']!=null?info['surname']:""),
Text(info['uid']!=null?info['uid']:""),
Text(info['new']!=null?info['new']:""),
Text(info['password']!=null?info['password']:"")
],
),
),
)
);
}
}
'


DEVX-2070: Changes prior to building and publishing new 6.0 based datagen images

Problem
Want to build a new version of the connector off of the 6.0 CP libraries
Solution
Update pom.xml and execute a release building 0.3.4 datagen version

Does this solution apply anywhere else?

[ ] yes
[X] no

Test Strategy
No code changes in the datagen code with this change, only the base libraries have been upgraded to 6.0.  Unit tests pass
Release Plan
After building the full datagen docker imags, will validate new images in Confluent Integration Arch demos and examples.

Looks like the build is failing with some compilation errors:
[2020-09-24T23:46:41.804Z] [ERROR] Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:3.8.1:compile (default-compile) on project kafka-connect-datagen: Compilation failure: Compilation failure:
[2020-09-24T23:46:41.804Z] [ERROR] /home/jenkins/workspace/c-pr_kafka-connect-datagen_PR-71/src/main/java/io/confluent/kafka/connect/datagen/DatagenConnectorConfig.java:[19,33] package com.google.common.collect does not exist
[2020-09-24T23:46:41.804Z] [ERROR] /home/jenkins/workspace/c-pr_kafka-connect-datagen_PR-71/src/main/java/io/confluent/kafka/connect/datagen/DatagenConnector.java:[19,37] package com.google.common.annotations does not exist
[2020-09-24T23:46:41.804Z] [ERROR] /home/jenkins/workspace/c-pr_kafka-connect-datagen_PR-71/src/main/java/io/confluent/kafka/connect/datagen/DatagenConnector.java:[41,4] cannot find symbol
[2020-09-24T23:46:41.804Z] [ERROR]   symbol:   class VisibleForTesting
[2020-09-24T23:46:41.804Z] [ERROR]   location: class io.confluent.kafka.connect.datagen.DatagenConnector
[2020-09-24T23:46:41.804Z] [ERROR] /home/jenkins/workspace/c-pr_kafka-connect-datagen_PR-71/src/main/java/io/confluent/kafka/connect/datagen/DatagenConnectorConfig.java:[103,12] cannot find symbol
[2020-09-24T23:46:41.804Z] [ERROR]   symbol:   variable ImmutableList
[2020-09-24T23:46:41.804Z] [ERROR]   location: class io.confluent.kafka.connect.datagen.DatagenConnectorConfig

Looks like it might be Guava-related?

Looks like the build is failing with some compilation errors:
Looks like it might be Guava-related?

I'm not sure how to diagnose this, I just successfully built locally with:
mvn --batch-mode -Pjenkins clean verify install dependency:analyze site validate -U
...
...
...
...
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  04:09 min
[INFO] Finished at: 2020-09-25T07:39:33-05:00
[INFO] ------------------------------------------------------------------------

@rspurgeon It looks like this branch isn't up-to-date with the latest master. When Jenkins builds, it first rebases this current branch with master, and it looks like that's causing the build failures.
 That should allow you to reproduce and diagnose the build failures.

@C0urante rebased off branch, added guava dependency explicitly.  Also switched to builder interface in avro generator to allieviate build warning.  LMK what you think... thanks

I'll fix the build error @C0urante

🤦♀ thanks, sorry about that Rick.


Duplicated classNames

Environment
System:

OS: Windows 10 10.0.18362
CPU: (12) x64 Intel(R) Core(TM) i7-8750H CPU @ 2.20GHz
Memory: 6.90 GB / 15.86 GB

Binaries:

Node: 12.16.1 - C:\Program Files\nodejs\node.EXE
Yarn: 1.22.4 - C:\Program Files (x86)\Yarn\bin\yarn.CMD
npm: 6.14.2 - C:\Program Files\nodejs\npm.CMD

npmPackages:

babel-plugin-styled-components: ^1.10.7 => 1.10.7
styled-components: ^5.1.0 => 5.1.1

Reproduction

https://codesandbox.io/s/styled-components-base-gc367
Package.Json setup:
"dependencies": {
"@material-ui/core": "^4.10.0",
"@sentry/browser": "^5.17.0",
"@sentry/node": "^5.17.0",
"@sentry/webpack-plugin": "^1.11.1",
"@types/classnames": "^2.2.10",
"@types/cors": "^2.8.6",
"@types/express": "^4.17.6",
"@types/express-http-proxy": "^1.6.1",
"@types/morgan": "^1.9.0",
"@types/styled-components": "^5.1.0",
"@types/yup": "^0.29.0",
"@zeit/next-source-maps": "^0.0.3",
"babel-plugin-styled-components": "^1.10.7",
"classnames": "^2.2.6",
"cookie": "^0.4.1",
"cors": "^2.8.5",
"cross-env": "^7.0.2",
"dayjs": "^1.8.28",
"express": "^4.17.1",
"express-http-proxy": "^1.6.0",
"lodash-es": "^4.17.15",
"morgan": "^1.10.0",
"next": "9.4.2",
"next-i18next": "^4.4.2",
"next-redux-wrapper": "^5.0.0",
"react": "16.13.1",
"react-calendar": "^3.1.0",
"react-dom": "16.13.1",
"react-google-maps": "^9.4.5",
"react-hook-form": "^5.7.2",
"react-icons": "^3.10.0",
"react-number-format": "^4.4.1",
"react-redux": "^7.2.0",
"react-tabs": "^3.1.1",
"redux": "^4.0.5",
"redux-devtools-extension": "^2.13.8",
"redux-thunk": "^2.3.0",
"reselect": "^4.0.0",
"styled-components": "^5.1.0",
"swr": "^0.2.2",
"url": "^0.11.0",
"yup": "^0.29.0"
},
"devDependencies": {
"@types/node": "^14.0.5",
"@types/react": "^16.9.35",
"@types/react-redux": "^7.1.9",
"@typescript-eslint/eslint-plugin": "^3.3.0",
"@typescript-eslint/parser": "^3.3.0",
"babel-plugin-inline-react-svg": "^1.1.1",
"eslint": "^7.2.0",
"eslint-config-prettier": "^6.11.0",
"eslint-plugin-prettier": "^3.1.4",
"eslint-plugin-react": "^7.20.0",
"prettier": "^2.0.5",
"ts-node": "^8.10.2",
"typescript": "^3.8.3"
}
Steps to reproduce
Expected Behavior
ClassNames are unique.
Actual Behavior
In my Next.js project we have an issue, that after every production build some parts of css are missing on not correctly rendered. We have found out that it happens, because some classNames are duplicated. However we dont know what causes this behavior and how to fix it.

Has anyone encountered this ?

https://github.com/styled-components/styled-components/issues/3125 Here they recomend downgrade styled-components to v 4.4.1

Oh!. Is there only one solution to reduce the version to 4.4.1?   :(

This is a duplicate as mentioned above, however no reporter has yet to add any details or investigate this thoroughly, so both issues are rather stale 😅

Hi @Mattese
Can you provide a reproduction repo or codesanbox?
I don't see the issue in the link (https://codesandbox.io/s/styled-components-base-gc367) provided in the description.
Thanks in advance

Just a reminder, this issue only exists in production environments. Thus, no codepen / sandbox will be able to reproduce it, making it worse to debug. This thread, https://github.com/styled-components/styled-components/issues/3125, has more information.


Create --debug or simply --verbose flag for CLI

Add a --verbose or --debug flag to output detailed logs of actions, helpful for troubleshooting.
Steps TODO:

Add flag in cli.py as it was done in for example --no-gitignore
Pass this flag to encryptor.py and/or validation.py
Write print statements describing operations if only flag is True.

🚫 Warning: This issue should not be implemented in parallel with #10, it may cause a conflict of interest

Should this flag be added to all commands in cli.py?

Hi @Ishan-Jadhav,
I will say frankly I trust the instinct of the one performing this task. From my perspective (I think?) yes 🤔, because every command touches the files in some way, for debugging it will be a big simplification if we know what is going on underneath. But on the other hand if --debug/--verbose will print 100 lines it will be difficult to analyse because of size of output 😂.
You have to find a balance between helpful and redundant. ⚖
But as I say, this is my point of view I am not imposing my vision and I am open to discussion. 👥

@Ishan-Jadhav  would you like me to assign you on this?

yes.


[BUG][ipad] Find modal occupies the whole page

Describe the bug
The find modal captures the whole page sometimes with no way to come out of it.
To Reproduce
Steps to reproduce the behavior (this is an example):

Go to 'Thee dot menu'
Click on 'Find'
See error

Expected behavior

Search reaults
A way to come out of it (no space to tap on)

Screenshots

Closed for repeated issue


Use GCLOUD_PROJECT instead of GCLOUD_PROJECT_NUM

Fixes #236

@ofrobots PTAL.

Looking at https://www.npmjs.com/package/gcloud, they are using GCLOUD_PROJECT_ID

Fixed.

For posterity, that was a bug in gcloud's README file. The correct env. var. to use is GCLOUD_PROJECT.

This is looking a lot better folks.  LGTM.  Lets get the GCLOUD_PROJECT change in, and then if needed we can bikeshed the readme prose.

@ofrobots @JustinBeckwith I've taken another editing pass at this.

LGTM w/ nit, once the CI is green.


Read button on view

#428
Update UI in all devices

Checked this PR :

[ ] For duplication.
[ ] For performance.
[ ] Usage of recursive methods, loops and algorithms.
[ ] For readability and understanding.
[ ] For functions with multiple responsibilities.
[ ] For scope of refactoring.
[ ] For coupling
[ ] For alternative approaches to the implementation.
[ ] For how the change impacts other parts of the system.
[ ] For validations. client side and server side validations are present.

For more insights on code reviews checkout our guidelines.


hyprspace: 0.1.4 -> 0.1.5

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/hyprspace/hyprspace/releases.
meta.description for hyprspace is: "A Lightweight VPN Built on top of Libp2p for Truly Distributed Networks."
meta.homepage for hyprspace is: "https://github.com/hyprspace/hyprspace"
meta.changelog for hyprspace is: ""
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

/nix/store/cf3yjbvmandc68gsvfr16ka57rh2dk57-hyprspace-0.1.5/bin/hyprspace passed the binary check.

1 of 1 passed binary check by having a zero exit code.

0 of 1 passed binary check by having the new version present in output.

found 0.1.5 with grep in /nix/store/cf3yjbvmandc68gsvfr16ka57rh2dk57-hyprspace-0.1.5

found 0.1.5 in filename of file in /nix/store/cf3yjbvmandc68gsvfr16ka57rh2dk57-hyprspace-0.1.5

directory tree listing: https://gist.github.com/b409c356d41a41f9cd1e9d990289165f

du listing: https://gist.github.com/a81632c229b3936b9ad1d2156e89e034

Rebuild report (if merged into master) (click to expand)

4 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
1 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
hyprspace

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/cf3yjbvmandc68gsvfr16ka57rh2dk57-hyprspace-0.1.5 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A hyprspace https://github.com/r-ryantm/nixpkgs/archive/b16dbf99b6dca94a4c5468fee649e04de51017d3.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/cf3yjbvmandc68gsvfr16ka57rh2dk57-hyprspace-0.1.5
ls -la /nix/store/cf3yjbvmandc68gsvfr16ka57rh2dk57-hyprspace-0.1.5/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

hyprspace

Maintainer pings
cc @yusdacra @kalbasit for testing.

LGTM, works fine


Fix link in readme

Anyone?

This is a duplicate of #8 (which hasn't been merged either).

Closing in favor of #8.


Check existing_sg not existing_vpc in security group delete

Fixes issue #74

:+1: This seems pretty obvious - good catch!

@christinedraper I rebased and squashed down to 1 commit - thats why this isn't automatically linking.  Manually closing, but it did get merged in!

Thanks..  still getting up to speed with submitting pull requests, sorry for the extra commits.

No worries!  Running an interactive rebase (rebase -i) allows you to squash commits together so the log is shorter.  For small changes like this, I prefer to do that.  It keeps the history from growing too huge.


Update Sentry to latest version

Description
This PR updates Sentry to the latest version. According to the migration docs, there were no changes needed from our side (there were very few breaking changes and we are not using the changed features).
https://docs.sentry.io/platforms/react-native/migration/
Tested
Rebuilt both iOS and Android apps, manually (created and) triggered an error and verified that it came through in Sentry.
How others should test
There should be no change to the app behaviour with this PR, so verify that Sentry is working as expected. Errors that you trigger should be received by Sentry as usual.
Related issues
Prepares for #1461

Hey @MuckT , Please let us know do we need to perform testing from our side.

hey @ValoraQA no testing is needed from your side for this one :)


Setting Blob Public Access

Public Access for the thumbnails container is still being set to 'Blob' in the code even though earlier it was set to Private via CLI.  Should the could leave it as Private?  Do the SAS tokens negate the need to set public access?  I tested setting to Private and it still worked but my tokens may still be valid from earlier testing.  Thanks!

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

ID: be6225e2-f996-0240-ec57-bb601e0e490d
Version Independent ID: 02e61a77-e74d-a9ef-cccc-d59e8e26f8b9
Content: Secure access to an application's data in the cloud with Azure Storage
Content Source: articles/storage/blobs/storage-secure-access-application.md
Service: storage
GitHub Login: @tamram
Microsoft Alias: tamram

@jabberhams Thanks for the feedback! I have assigned the issue to the content author to evaluate and update as appropriate.

@jabberhams - thanks for your feedback. I agree this was confusing. The SAS token does not require that the container be available for public access. The token grants access to a secured container for an interval of time and with specific permissions.
I've removed the line of code that set the container public access to "Blob", from both the article and the sample. I will also further evaluate to see if we really need to set the container public access via the CLI.
#please-close


Problem after WebUI v1.6.0 when using refiner

using refiner will overwrite the checkpoint name in grid with refiner model, rather than the base model, which actually determine the overall image content.
or maybe it's better to seperate the base model and refiner model in grid

I have an idea on how to make the whole process of getting the xyz values more straightforward.
In your case, you'd get the base model if you chose checkpoint name in xyz, and the refiner model if you chose refiner in xyz.
Stay tuned...

I just finished the rework. Can you check if it works for you?

I just finished the rework. Can you check if it works for you?

@AlUlkesh It works just fine, thanks!


Fix double render in org signup

Fixes a Double Render that happened when displyaing signup for an organization that had signups disabled and the quota was exceeded. Reorganized the code to check if signup is enabled first.
Closes #6721
CR @juanignaciosl

CR @juanignaciosl

:+1: (assuming it's tested ;-) ).

retest this

retest this, pleas

retest this, please


STO What's Supported updates DOC-3664

Harness Developer Pull Request
Preview link: https://64f10eb17280135b920a8d4f--harness-developer.netlify.app/docs/security-testing-orchestration/whats-supported

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app


[Feature]: Add the pr-title-checker.yml for keeping tracks of PR.

Description
This will :
/Improved Code Organization: Consistent PR titles will enhance code organization and searchability.
/Streamlined Workflow: Integration with CodeQL and a dedicated workflow will promote efficiency and automation.
/Reduced Manual Effort: Automated checks will minimize the need for manual title review, freeing up time for development tasks.

Assign @1010varun


Setup init.d for starting/stopping service

Look into /etc/init.d/skeleton to make /etc/init.d/vapor for starting the web server on startup, and managing starting, stopping, and reloading.

Using Upstart
/etc/init/vapor.conf
description "Vapor"

start on startup

exec /home/tanner/vapor-example/.build/debug/VaporApp

sudo start vapor
sudo stop vapor

--workDir should allow this functionality


third party noop library

Following software design best practices I'm not going to be rolling my own noop (you jsut don't do that)
This addresses https://github.com/auchenberg/volkswagen/issues/90

Awesome, looks like tests passed.

what the hell did I just read

At first look, I was very sceptic about this change. But I appreciate this, after a closer look at node-noop repo. It is another seriously joking package which has solid tests and even benchmarks.

You should use the noop6 library instead.


Master to Live

Docs Build status updates of commit c3cbc68:
:clock10: Full build: the average full build time is 11 min(s) 55 sec(s), this is based on the last 794 build(s) for this repository.

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

File
Status
Preview URL
Details

.openpublishing.redirection.json
:white_check_mark:Succeeded

exchange/exchange-ps/exchange/advanced-threat-protection/Get-UrlTrace.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/advanced-threat-protection/New-SafeLinksPolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/client-access/Set-OwaMailboxPolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/policy-and-compliance-dlp/New-DlpEdmSchema.md
:white_check_mark:Succeeded
View (exchange-ps)

sharepoint/docs-conceptual/overview.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/docs-conceptual/sharepoint-online/sharepoint-online-cmdlets.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/docs-conceptual/sharepoint-server/sharepoint-server-cmdlets.md
:white_check_mark:Succeeded
View (sharepoint-ps)

skype/skype-ps/skype/New-CsTeamsMeetingPolicy.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/Reset-CsRoutingGroup.md
:white_check_mark:Succeeded
View (skype-ps)

spmt/docs-conceptual/index.md
:white_check_mark:Succeeded
View (spmt-ps)

spmt/docs-conceptual/intro.md
:white_check_mark:Succeeded
View (spmt-ps)

spmt/docs-conceptual/overview.md
:white_check_mark:Succeeded
View (spmt-ps)

staffhub/docs-conceptual/index.md
:white_check_mark:Succeeded
View (staffhub-ps)

staffhub/docs-conceptual/intro.md
:white_check_mark:Succeeded
View (staffhub-ps)

staffhub/docs-conceptual/overview.md
:white_check_mark:Succeeded
View (staffhub-ps)

teams/docs-conceptual/index.md
:white_check_mark:Succeeded
View (teams-ps)

teams/docs-conceptual/overview.md
:white_check_mark:Succeeded
View (teams-ps)

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


Add nat-418/boole.nvim

Repo URL:
https://github.com/nat-418/boole.nvim
Checklist:

[x] The plugin is specifically built for Neovim, or if it's a colorscheme, it supports treesitter syntax.
[x] The lines end with a .. This is to conform to awesome-list linting and requirements.
[x] The title of the pull request is Add/Update/Remove `username/repo`  (notice the backticks around `username/repo`) when adding a new plugin.
[x] The description doesn't mention that it's a Neovim plugin, it's obvious from the rest of the document. No mentions of the word plugin unless it's related to something else.
[x] The description doesn't contain emojis.
[x] Neovim is spelled as Neovim (not nvim, NeoVim or neovim), Vim is spelled as Vim (capitalized), Lua is spelled as Lua (capitalized), Tree-sitter is spelled as Tree-sitter.
[x] Acronyms should be fully capitalized, for example LSP, TS, YAML, etc.

Pinging @nat-418 so they're aware I opened this PR.


An error occurred attempting to determine the process id of dotnet.exe

When attempting to run my application with SSL enabled, I get the following error:

An error occurred attempting to determine the process id of dotnet.exe which is hosting your application.  One or more errors occured.

It seems to happen when the App URL begins with 'https://' regardless of whether Enable SSL has been selected.  What would be causing this error?

How did you configure it?

What do you mean?

Can you put your application(GitHub preferable) somewhere where we can reproduce the problem.

Here it is.  https://github.com/UpperAvenue/HttpsConfiguration

I came here searching for help on this same issue. The fix for me was found in Vackups answer to this Stack Overflow question
In short, the fix for me was this:
"So, running the site with Ctl+F5 installed the certificate and then I was able to debug the site solving my issue."

That did it - thank you!

Thank you. I was going crazy why this error comes up after enabling ssl. 👍

I keep circling around to this issue. Ctrl-F5 is not fixing it for me.

I did this:
I had been messing with some certificates, so I had removed the IIS Express certificate. So easiest way to reinstall the certificate I found before coming here was to go into add/remove programs and run fix IIS Express, which installs the certificate again. Not sure if Ctrl+f5 does the same thing.
After that I had to disable and re-enable ssl.
This does not really feel like a developer solution, more trying and failing, but it worked!


E: Package 'mercurial' has no installation candidate

This affects mostly Ubuntu ports releases. It seems mercurial is only available in i386 and amd64, so every --scm build fails.
Full affected list:

precise-armel-scm
precise-armhf-scm
precise-powerpc-scm
trusty-arm64-scm
trusty-armhf-scm
trusty-powerpc-scm
utopic-arm64-scm
utopic-armhf-scm
utopic-powerpc-scm
vivid-arm64-scm
vivid-armhf-scm
vivid-powerpc-scm
wily-arm64-scm
wily-armhf-scm
wily-powerpc-scm

There are packages for the four architectures: http://ports.ubuntu.com/ubuntu-ports/pool/universe/m/mercurial/ , why can't apt find them?

$ docker run -it --rm vicamo/buildpack-deps:trusty-armhf-curl /bin/bash
root@77b6e1f79f6a:/# cat /etc/apt/sources.list
deb http://ports.ubuntu.com/ubuntu-ports trusty main
deb http://ports.ubuntu.com/ubuntu-ports trusty-updates main
deb http://ports.ubuntu.com/ubuntu-ports trusty-security main

$ docker run -it --rm vicamo/buildpack-deps:trusty-amd64-curl /bin/bash
root@3907d9cdb874:/# cat /etc/apt/sources.list
deb http://archive.ubuntu.com/ubuntu trusty main universe
deb http://archive.ubuntu.com/ubuntu trusty-updates main universe
deb http://archive.ubuntu.com/ubuntu trusty-security main universe

'universe' is missing in base images in vicamo/ubuntu-debootstrap.

Closed by https://github.com/vicamo/docker-buildpack-deps/commit/577de4c6d70c130c1aff2eecd86cdf5160b9d3e9


HoverComponents do not work on 1.19

Tested on Spigot 1.19 with vanilla client, and no other plugins present than my own.
Hex colors, and normal text components all work fine, but hover text simply does not show.
Below is the server log with debugging enabled:
https://paste.gg/p/anonymous/67a2aa098a9d47b7871db9684420b1c1

There probably needs to be support added for 1.19 internals changes. Feel free to PR.


Arma: Cold War Assault

steam-65790.log

UpdateTexture implemented. We'll see if the issue persists.

@AlpyneDreams Play-tested the game with a couple of single missions and a quick campaign intro. Looks and plays just fine now. We can close this issue.

Awesome


Attributes - weak attribute addition

Should fix #2 .
All attributes are now in place for mbed-drivers. There will be a new PR to mbed-drivers to remove toolchain header file we used to have there.
@autopulated @bogdanm

LGTM

LGTM. Should we also take this opportunity to remove weak symbols from mbed OS completely? (in other PRs, obviously :) )

It looks like we have to do it now after all, since retarget.cpp in mbed-drivers now fails because __weak was removed.


Required to save label bug

Fixing the un-needed class being added to the form label when a field is required. Also, fixing the duplication issue added to input plugins with required fields.

Resolves https://github.com/punchcard-cms/punchcard/issues/530
DCO 1.1 Signed-off-by: NICK TILDEN<EMAIL_ADDRESS>

Related PR with the CSS changes on Punchcard - https://github.com/punchcard-cms/punchcard/pull/545

Not ready yet, still need to troubleshoot and solve the duplication issue on input-plugins.

@ineedsubstance failing CI tests.

Thanks @scottnath, will check that. Any thoughts on where to look for the duplication bug that is happening?
Ex.

@ineedsubstance probably not a specific dupe, but more likely the section which adds the class/text is getting hit more than once...

@scottnath - ah ok, thanks


Zip command ignores the "source" parameter

Description
Starting with version v7.2.6 ":zip" ignores the source parameter.
This works as expected on version v7.2.5 - lambda only contains files from the build folder in function root /.
Starting with v7.2.6 this is not the case anymore, and all files from the path are added to the package.

[X] ✋ I have searched the open/closed issues and my issue is not listed.

Versions

Module version [Required]: v7.2.6
Terraform version: v1.7.2
Provider version(s): hashicorp/aws v5.45.0

Reproduction Code [Required]
...
  build_in_docker = false
  source_path = [
    {
      path = "${path.module}/files/lambda"
      commands = [
        ":zip build /"
      ],
    }
  ]

Steps to reproduce the behavior:
Use :zip command with source and destination parameters to copy files from subdirectory.
Expected behavior
Function package only contains Files from source directory in destination.
Actual behavior
All files from path are present in function package.
Additional context
Bug introduced in v7.2.6

I am also affected by this bug, and am hoping it isn't auto-closed.

+1 on this, all of a sudden the deployment seems to pack the entire folder, not just the build directory on me end too.
source_path = [
    {
      path = "${path.module}/../backend/lambda/my-lambda",
      commands = [
        "pnpm install",
        "pnpm run build",
        ":zip dist ."
      ]
    }
  ]

@tonio-ramirez did you find a workaround for the time being?

Workaround we had to go with was
source_path = [
    {
      path = "${path.module}/../backend/lambda/my-lambda",
      commands = [
        "pnpm install",
        "pnpm run build",
        "cd dist",
        ":zip"
      ]
    }
  ]

Wonder if is a bug introduced in comit: (https://github.com/terraform-aws-modules/terraform-aws-lambda/commit/f7866811bc1429ce224bf6a35448cb44aa5155e7) in package.py:935 added line -
sh_work_dir = temp_file.read().strip()
which sets variable sh_work_dir to be the present working directory pwd of the execution context.
This then becomes dominant over the specified source_path of the command (even if it is specified) at package.py:886-889
                if sh_work_dir:
                    if source_path != sh_work_dir:
                        if not os.path.isfile(source_path):
                            source_path = sh_work_dir

this is then successfully worked around by @anuradhawick's cd dist

While trying to figure out why the patterns arg wasn't working I also noticed that the filter doesn't actually work as well.
The problem seems to be that ZipContentFilter.filter.apply sets the default return value to True (add the file to the zip archive). If the tested file isn't matched by the rule we don't update the return value d.
I think the fix could look something like this:
diff --git a/package.py b/package.py
index ded4056..093ef78 100644
--- a/package.py
+++ b/package.py
@@ -587,20 +587,22 @@ class ZipContentFilter:
         def apply(path):
-            d = True
+            d = len(rules) == 0
             for r in rules:
                 op, regex = r
                 neg = op is operator.not_
                 m = regex.fullmatch(path)
                 if neg and m:
                     d = False
                 elif m:
                     d = True
             if d:
                 return path

I finally figured out a nice way to work around this without having to cd into the dist directory (which created constant drifts for me)
If you set patterns to first ignore everything (which triggers the if neg and m path and causes the default to be False and then in a second pattern only match the files you actually need it works as expected
[ "!.*", "^/dist/index.cjs$"]

No activity on this?

ping

I hate to comment without adding any value, but I really don't want this issue to be closed just because it went stale...

Maybe this was fixed in:

#639
#640

Please give it a try using v7.15.0 of this module. This issue should have been resolved there.


AttributeError while downloading subtitle

Hi,
I am hitting the following error while downloading subtitles:
$ subliminal download -l en Lost\ in\ Translation\ 2003\ 1080p\ Bluray\ x265\ 10Bit\ AAC\ 5.1\ -\ GetSchwifty.mkv
Collecting videos  [####################################]  100%  Lost in Translation 2003 1080p BluCollecting videos  [####################################]  100%
1 video collected / 0 video ignored / 0 error
Downloading subtitles  [####################################]  100%  Lost in Translation 2003 1080p Bluray x265 10Bit AAC 5.1 - GetSchwifty.mkv
Traceback (most recent call last):
  File "/usr/local/bin/subliminal", line 11, in <module>
    sys.exit(subliminal())
  File "/usr/local/lib/python3.5/dist-packages/click/core.py", line 722, in __call__
    return self.main(*args, **kwargs)
  File "/usr/local/lib/python3.5/dist-packages/click/core.py", line 697, in main
    rv = self.invoke(ctx)
  File "/usr/local/lib/python3.5/dist-packages/click/core.py", line 1066, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/usr/local/lib/python3.5/dist-packages/click/core.py", line 895, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/usr/local/lib/python3.5/dist-packages/click/core.py", line 535, in invoke
    return callback(*args, **kwargs)
  File "/usr/local/lib/python3.5/dist-packages/click/decorators.py", line 27, in new_func
    return f(get_current_context().obj, *args, **kwargs)
  File "/usr/local/lib/python3.5/dist-packages/subliminal/cli.py", line 403, in download
    hearing_impaired=hearing_impaired, only_one=single)
  File "/usr/local/lib/python3.5/dist-packages/subliminal/core.py", line 211, in download_best_subtitles
    for s in subtitles], key=operator.itemgetter(1), reverse=True)
  File "/usr/local/lib/python3.5/dist-packages/subliminal/core.py", line 211, in <listcomp>
    for s in subtitles], key=operator.itemgetter(1), reverse=True)
  File "/usr/local/lib/python3.5/dist-packages/subliminal/score.py", line 106, in compute_score
    matches = subtitle.get_matches(video)
  File "/usr/local/lib/python3.5/dist-packages/subliminal/providers/opensubtitles.py", line 98, in get_matches
    matches |= guess_matches(video, guessit(self.movie_release_name, {'type': 'movie'}))
  File "/usr/local/lib/python3.5/dist-packages/subliminal/subtitle.py", line 235, in guess_matches
    if video.format and 'format' in guess and guess['format'].lower() == video.format.lower():
AttributeError: 'list' object has no attribute 'lower'

System details:
~$ python3 -V
Python 3.5.2
~$ uname -a
Linux tux 4.4.0-98-generic #121-Ubuntu SMP Tue Oct 10 14:24:03 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux
~$ subliminal --version
subliminal, version 2.0.5

Please let me know if you need any other information.
Awesome utility, by the way! :+1:

https://github.com/Diaoul/subliminal/pull/822 did the trick for me, give that a go. :)

https://github.com/Diaoul/subliminal/issues/761


Styling alt tags for mj-image

Hi, I was thinking you should implement Styled Alt tags - so the ability to add font-size & color ect... to the mj-image tag to style the alt text!
Let me know if this is already in progress or you're going to implement it!
Resource: https://litmus.com/blog/the-ultimate-guide-to-styled-alt-text-in-email

I doubt the support would be wide enough for it to be considered for adding
it as part of the MJML core.
Feel free to prove me wrong though.
Kind Regards,
:----------------------------:
Dale McConnell
On 12 December 2016 at 11:47, Tyler Potts<EMAIL_ADDRESS>wrote:

Hi, I was thinking you should implement Styled Alt tags - so the ability
to add font-size & color ect... to the mj-image tag to style the alt text!
Let me know if this is already in progress or you're going to implement it!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/mjmlio/mjml/issues/472, or mute the thread
https://github.com/notifications/unsubscribe-auth/AKxLOFRsH5yX4NRLM2WQWrJwY8u90Kpyks5rHSZZgaJpZM4LKaK7
.

Well I believe that styled alt text would be beneficial although it is not very supported it is still a big issue when you have a really nice image but its being blocked and to see styled alt tags may just make the none image design just that slight bit better!
Here's a link to the support of styled alt tags and it is still pretty supported in webmail clients and some desktop clients. https://litmus.com/blog/the-ultimate-guide-to-styled-alt-text-in-email

Hi.
It is IMHO impossible right now in MJML to style ALT attribute. Probably only way is to edit rendered output.(It is mentioned here #577)

no unlimited control of mj-image via mj-class . See: https://mjml.io/try-it-live/SkMSGX_ix
Only parameters accepted are mj-image parametres https://mjml.io/documentation/#mjml-image

So if you want to style font-size, color, text-align ...which are necessary parameters to define alt text  you are doomed.
Related: #577

Support is not that small. It wowudl certainly be usefull.

Hi there,
Since we're supporting css-class you will be able to apply some style to any img and styles it. Feel free to test it in current beta !

As mentioned on my previous comment, css-class now allows that ! So I can close 👍


Raise node.js requirements to node >18.0.0

Feature Request
Describe the feature you would like to see added:
Currently, it's not possible to upgrade Semantic Release to a Version higher than 19.x.x, as 20.x.x and upwards require at least Node Version 18.0.0 to work.  While Semantic Release is a Development tool, it might still be a good idea to raise the required node version to LTS 18.0.0
Additional information:
https://semantic-release.gitbook.io/semantic-release/support/node-version

As a note, the contribution guide already specifies node v.18 and up as requirement. The only thing that would need to be changed is the GitHub action.


MAINTAINERS: Add myself

I'd like to apply for this position, let me know if you need a CV :)
Signed-off-by: Jussi Kukkonen<EMAIL_ADDRESS>

I'd like to apply for this position, let me know if you need a CV :)

First, we will need you to jump through a series of irrelevant coding puzzles...

I'd like to apply for this position, let me know if you need a CV :)

First, we will need you to jump through a series of irrelevant coding puzzles...


proposal: sorted map

Proposal Details
The standard library should offer a sorted_map structure similar to other languages like c++ and java. This would make operations like sorted iteration, finding min, and finding max all efficient and convenient operations.

Duplicate of #60630


Add edit variable dialog access to drawing tool

@tealefristoe is this ready for review? I see it isn't draft, but you didn't request any reviews of it.

@scytacki No this still has of work to go, it's not ready for a review yet. It will probably be ready by the end of the day.


Spinner still spinning after clearing search

Steps to repro:

Go to Listings > Discover
Enter a search query without results
Clear the search using the X on the right side of the input box
Notice the search query is cleared but the spinner keeps spinning indefinitely (see screenshot)

Can't reproduce this, but perhaps this was fixed in one of the latest commits. @jjeffryes @SamPatt @morebrownies can you check?

It's possible if you clear the search before it's completed, and the normal load of the discover tab doesn't return any results due to network issues, the spinner could linger.
I can't reproduce it.

@jjeffryes: That's what I'm seeing. Relatively easy to repro for me.

This has been fixed.


There is no information about Reserved Keywords('IF' , 'WHILE')

There is no proper information about usage of reserved keywords

Where do you expect to see such information?

Closing due to lack of information. If this is related to RIDE, this should be submitted to its issue tracker.


feat: add reentrancy guard

Resolving #69
Description:

Added the reentrancy guard to the order book market contract.
libs/src/reentrancy.sw

Modified open_order_internal to follow the Checks-Effects-Interactions pattern. I moved the transfer call after the contract makes state changes.

I checked the if using the reentrancy_guard increases the amount of gas used, however, from a first glance it did not have an effect.

run forc fmt to format sway code
finally get market-contract-abi.json from out/release folder replace it in spark-market-sdk/market-contract/out/release manually

Ran forc fmt
Added the market-contract-abi.json to spark-market-sdk/market-contract/out/release

@dcpp Just double checked, I ran forc fmt on the reentrancy-guard branch, it did not change anything, meaning my PR is correctly formatted. I am using forc 0.61.1


After successful join() debugPrint DevAddr

Since it will change after join: https://github.com/TheThingsNetwork/arduino-device-lib/pull/46#discussion_r79986695

closed by #60


Remove a closure in _FieldSet.hashCode

Improves HashCode benchmark up to 3%.
JIT before:
HashCode(RunTime): 1981.9821782178217 us.
HashCode(RunTime): 1974.7068114511353 us.
HashCode(RunTime): 1970.9261083743843 us.

JIT after:
HashCode(RunTime): 1899.034155597723 us.
HashCode(RunTime): 1912.0917782026768 us.
HashCode(RunTime): 1908.1782650142993 us.

Native AOT before:
HashCode(RunTime): 5164.038659793814 us.
HashCode(RunTime): 5177.984496124031 us.
HashCode(RunTime): 5192.199481865285 us.

Native AOT after:
HashCode(RunTime): 5027.246231155779 us.
HashCode(RunTime): 5045.488664987405 us.
HashCode(RunTime): 5134.379487179487 us.

JS before:
HashCode(RunTime): 6944.444444444444 us.
HashCode(RunTime): 6913.793103448276 us.
HashCode(RunTime): 6927.335640138408 us.

JS after:
HashCode(RunTime): 6846.41638225256 us.
HashCode(RunTime): 6965.277777777777 us.
HashCode(RunTime): 6920.415224913495 us.

Fixes #632

Ported to internal in cl/449694660.


[5.8] Use date_create to prevent unsuppressible date validator warnings

Due to an unresolved PHP bug warnings are being triggered which cannot be caught or suppressed. The warning occurs when using a different property is used which is not parseable by the new DateTime instance. Applications like NewRelic pick up on these warnings and are logging them while there is no direct inconvenience for the users.
Replaced the new DateTime by date_create which is an alias for the DateTime constructor but silences these kinds of warnings.
Fixes issue #27784

Are there any other places where this is relevant?

I couldn't find another place where this specific case would occur in the repo. I haven't checked the other Laravel products though. We've been keeping a close eye on our tooling to check for other occurrences and none seemed to happen after patching this bad boy.

@LarsGrevelink that's quite interesting, how did you find out about this? 😄

@mfn We're having quite some date-related rules on our platform. NewRelic started reporting these warnings. That triggered further research which eventually led to the fix we applied through this PR.

@LarsGrevelink I took a second look and found two small issues:

date_create returns false for invalid dates but the method signature does not tell so. Personally I think it would be preferred to coerce that to null anyway
Is catching the exception now still relevant? DateTime was throwing one, date_create isn't. The code before that seems to have checks in place, unclear if the exception would apply to it

$ php -r '$a = new DateTime("Invalid"); var_dump($a);'
PHP Fatal error:  Uncaught Exception: DateTime::__construct(): Failed to parse time string (Invalid) at position 0 (I): The timezone could not be found in the database in Command line code:1
Stack trace:
#0 Command line code(1): DateTime->__construct('Invalid')
#1 {main}
  thrown in Command line code on line 1

vs
$ php -r '$a = date_create("Invalid"); var_dump($a);'
bool(false)

@mfn Date::parse still throws an exception I think? That's still in the try/catch block. We should indeed convert the false to null before returning.

Will create an additional PR converting any falsy values to null on returning the value. Personal preference is to leave the structure as is, but if you want to isolate the Date::parse let me know and I will take that along. Have a great weekend!


Write log to file

By default the log of chloroExtractor is written to STDOUT (or STDERR?) but not to a file. As a folder with output files is created anyway it would be convenient to write a copy of the log messages to a file in this folder (e.g. ptx.log).

Logging is directed to STDERR. Nevertheless, I would suggest a logfile named according to the project folder (ptx if not specified) with the suffix .log. That file will be generated per default and is stored inside the project folder.

Sounds great.


[CWS] implement delete event on windows

What does this PR do?
This PR implements delete event for windows, modeled on the linux version (called unlink there), and using the newly added etw events deleteArgs.
This PR also adds the missing serializer for windows rename event.

Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/merge

/merge -c

/merge


Logout should always give a response.

I'm not sure what the correct way to handle this is, but this fixes https://github.com/feathersjs/feathers-authentication-client/issues/10.

Great catch! Should this resolve with an empty object or throw an error?

The goal is to end up without an authenticated user, which USB what you end up with even if you were never logged in.  So I don't think it should throw an error.  That's why I opted for the empty object.

Ya nice catch! Looks good. :shipit:


[Back-End] length for creating a blog

The title should be controlled within 66 characters (including spaces),
description tag description should be controlled within 150 characters (including spaces).

solved


feat(chip): add border CSS theming variable on selected state

DSW-226

Testable in StoryBook: https://77788479--5fbfb1d508c0520021560f10.chromatic.com/?path=/story/lumx-components-chip-chip--theming
StoryBook: https://dbb0a9d3--5fbfb1d508c0520021560f10.chromatic.com/ (Chromatic build)

Test OK


Add script to bump a version using semver

Thanks for this super useful tool. I've been using it to bump versions on my projects and decided to put together a little script that automates the way I use it. I hope this can be useful to someone else.

@raybejjani any interest in merging this or shall I just close it?

Hello,
While I'd love to merge this (and all the other PRs) I no longer work at cloudflare and I don't have admin rights to this repo anymore :(

hi, could you ping someone from cloudflare in this thread such that it comes to their attention?


[Offline sync] Fix bugs on save observation

Fixes #250, #249, #256. Depends on #246. See individual commits for actual delta. Please merge #246 first.

@scolsen @shobhitagarwal1612 @deepakbansal Could one of you kindly give this a quick review?

Fixes #250, #249, #256. Depends on #246. See individual commits for actual delta. Please merge #246 first.

Quick comment: We need to mention Fixes or Closes before each PR number to close it automatically. So, it should be Fixes #250, Fixes #249, Fixes #256

Quick comment: We need to mention Fixes or Closes before each PR number to close it automatically. So, it should be Fixes #250, Fixes #249, Fixes #256

Good catch, fixed!

@scolsen Could you PTAL?

I was waiting for #246 to get merged before reviewing this as mentioned in the description.

I was waiting for #246 to get merged before reviewing this as mentioned in the description.

I can be reviewed now, we just need to wait for #246 before we merge. In either case it's ok, just making sure it's in someone's queue. Thanks!!

@scolsen Could you PTAL?

Please disregard! :) Thanks @shobhitagarwal1612 !

@shobhitagarwal1612 Resolved conflicts manually, hopefully will pass CI. PTAL!

The build is passing now but CLA is failing for some reason.

The build is passing now but CLA is failing for some reason.

I did some digging and it seems this happens when there are multiple PR authors. I'll set the cla: yes tag now to force CLA check.


Update and rename feature-request-or-thoughs.md to feature-request-or-thoughts.md

Minor typo fix: missing "t" in "thoughts"

(I originally opened this as a PR against my own fork by mistake; hopefully I'm opening it against the correct one this time... you may wish to squash upon merging to hide the merge commits...)

Thanks for the corrections.

Thanks for the correction!


Update go to 1.18

Refs:

https://github.com/stolostron/backlog/issues/22476

Signed-off-by: Gus Parvin<EMAIL_ADDRESS>

/hold

Since the required tests have changed to 1.18 to enable this PR, it has blocked #30.

/unhold


[vm] NativeResult can only abort

NativeResult for native functions can now only give back an abort code
The native functions can technically still error, but hopefully that is more discouraged

Motivation

Bring natives inline with other functions and give them only the ability to "abort"

Test Plan

cargo test

I assume they should only do this if some internal invariants are broken, right?

Thats the intent yeah, but there is nothing enforcing that intent. The current ones that would fail are things like events, which might fail to write

Fixed tests

@bors-libra  r=vgao1996

:pushpin: Commit 9efd9eb has been approved by vgao1996

:hourglass: Testing commit 9efd9eb04e7bb39f0d34d9f6988a85ee748b0e91 with merge b421dfb29c84e4200fc90b2e4beb46a4fe2ea0ba...

:sunny: Test successful - checks-actions_land_blocking_test, checks-circle_commit_workflow
Approved by: vgao1996
Pushing b421dfb29c84e4200fc90b2e4beb46a4fe2ea0ba to master...


⚠ Thumbnails API Endpoint has degraded performance

In b879e28, Thumbnails API Endpoint (https://thumbnails.roblox.com/v1/assets?assetIds=82738847&format=Png&isCircular=false&size=30x30) experienced degraded performance:

HTTP code: 200
Response time: 2075 ms

Resolved: Thumbnails API Endpoint performance has improved in f080a9a after 10 minutes.