Allow running probot apps manually, e.g. from a one-off GitHub Action

Feature Request
While trying to set up probot/settings automation for labels across repositories, I stumpled upon https://github.com/probot/settings/issues/95#issuecomment-636386052, which is unfortunate.
One workaround would be to allow running a probot app manually, e.g. from CLI or from a button, ideally org-wide. Looking at https://github.com/probot/settings/blob/master/index.js it seems that an "easy" (I haven't actually developed for probot, just guessing) way would be to allow another event that fires on "something", e.g. a manually run GitHub Action, as a way to improve this situation.
Describe alternatives you've considered
An alternative would be a scheduled application of settings, but that can have unintended side-effects for sure.
Also, I looked into "Simulating Webhooks" but that seems to only apply to your own apps, not "Installed Apps". https://probot.github.io/docs/simulating-webhooks/

👋 Thanks for opening this @soraryu. Probot is designed specifically to respond to webhooks - that's where most of the logic of Probot lives, specifically setting up event handlers and authenticating an Octokit client based on the webhook's payload. It's important to note that Probot doesn't do any magic, it mostly abstracts authentication mechanisms that are based off of GitHub Apps and webhooks. That's actually part of why https://github.com/probot/settings/issues/95 is a weird challenge.
There are usually better options for doing the same functionality of a Probot app as a one-off, especially with Actions:

JasonEtco/actions-toolkit
@octokit/action.js

That said, if you must use an existing Probot app, there's this (though it's old, and may not work anymore): probot/actions-adapter.
Hope that helps!

Thanks for the hints. If I understand that right, all of those would require me to go to quite some lengths to get the functionality that probot/settings currently has? Ideally, I'd like to avoid having to do "custom dev" on top of existing apps and tooling, but I understand that's not always possible.
Just curious, what would you think is the best approach to the usecase I described in https://github.com/probot/settings/issues/95#issuecomment-636386052 - syncing issue labels across repositories?

all of those would require me to go to quite some lengths to get the functionality that probot/settings currently has?

Short-answer - yep! This functionality doesn't work in probot/settings right now, so it'd take some dev work to make it so.

Just curious, what would you think is the best approach to the usecase I described in probot/settings#95 (comment) - syncing issue labels across repositories?

https://github.com/crazy-max/ghaction-github-labeler is close. It still doesn't have built-in support for a centralized config though. You'd have to make your own Action that does something similar - runs in the .github repo, and then makes the relevant API calls for every repo in the org. I don't know of anything that exists that does that.

I think since all of these wouldn't have the abilities of probot/settings to inherit and extend a config I'm pretty much out of luck for now – I definitely didn't anticipate this being so involved 😄.
I'll probably go with semi-manually updating the .github/settings.yml file for each repo after updating the central file...

I'm going to close this out since there are no action items, and it looks like this is outside of Probot's control. You can accomplish the functionality with existing API calls anyway. Thanks for the issue and discussion!


[composer] autofocus document on creation

allow typing immediately after creating a new document without any clicks

@zhenyasav I was using Composer yesterday and found that documents got focus immediately already, so this might be completed.


'AdbClient' does not contain a definition for 'Instance' version 2.3.22 - 2.3.14

Hi
I'm newbie. Maybe I'm doing something wrong
But I get an error 'AdbClient' does not contain a definition for 'Instance' SharpAdbClient version 2.3.22 - 2.3.14 installed via Nuget.
http://prntscr.com/t6xojm
But everything works in the version 2.3.12.

This is a duplicate of #191. It's an intentional change - we've tried to remove all static methods/properties from the library.
It should nudge you to use dependency injection, but you're of course free to create your own static property if you so desire.


"Browser cleared canvas" message appears every action while image is empty

Basically, the "the browser may have cleared the canvas due to memory leaks" message shows up every action I do while the image is empty with transparency.
This happens not only if it actually cleared the canvas to memory leaks (the intended behavior), but also when it was done intentionally, i.e choosing the "Clear Canvas" option.

Yep, I figured it was more important to include this dialog than to get the condition exactly right. I'm not sure of a good way to exclude cases where the image is supposed to be completely transparent.
I thought of having an extra canvas to test if it becomes blank when memory is cleared, but it turns out individual canvases can be cleared while leaving others in tact, and bigger ones probably get cleared first, and I don't really want a giant canvas taking up memory in order to detect when memory gets cleared because then it'll get cleared earlier, and it might be unreliable anyways.
Probably there needs to be some flag that's maintained of whether it's supposed to be blank or not, and canvas-clearing operations set it, and then it only gets reset when (after any operation) the canvas stops being blank. But what is a canvas-clearing operation? Image > Clear Image sure, but also deleting a selection sometimes, or dragging a selection off the canvas, or using the Fill tool to fill with transparency.
Basically it's a surprisingly complicated problem, and not very interesting.
Other options include making the dialog less alarming, maybe starting with "Did you mean to clear the canvas?" or something

Oh, is that why the transparency settings is not abled by default? I was wondering why the transparency is not listed as a color at the colors bar.

Oh, is that why the transparency settings is not abled by default? I was wondering why the transparency is not listed as a color at the colors bar.

@Tubaleviao No, transparency is not the default simply because it's more complex and doesn't match MS Paint. With an opaque document, you can be guaranteed (basically) of the look of the image, but with a transparent document, it may appear invisible in some scenarios (black on black or white on white). Also, whilst it's possible to load transparent/translucent colors into the palette, currently the only blending mode makes full transparency useless as a color, as it won't overwrite, but rather blend, doing nothing.


Modify interop project to prevent winmd leaking from component project

Describe the bug

Referencing a C#/WinRT interop project from a .NET 5 app raises a NETSDK1130 error. This is because the component winmd flows through from the C++/WinRT component project. The project referencing structure is  .NET 5 app -> .NET 5 library interop project -> C++/WinRT component.
Look into whether C#/WinRT can fix this either by:

Setting the implementation dll as an output in addition to the projection dll for the interop project. Then in the interop project, authors would set <PrivateAssets>all</PrivateAssets> on the project reference to the C++/WinRT component. Caveat is the component might have multiple binaries
Modifying cswinrt NuGet targets. Cswinrt needs to influence reference resolution before project references kick in - binlog would show cross-reference target evaluation and the hook for that change. Referring to the CsWinRTRemoveWinMDReferences target here:
https://github.com/microsoft/CsWinRT/blob/d361800d2cf6a46d2c7d276126a09bab1d8bc1c4/nuget/Microsoft.Windows.CsWinRT.targets#L47

To Reproduce

Build the Net5ProjectionSample without generating a NuGet package
Add ConsoleAppSample to the sample solution and add a project reference to the interop project, SimpleMathProjection

Here's an example - https://github.com/angelazhangmsft/ProjectionProject
Expected behavior

WinMD doesn't flow through but implementation dll does.
Additional context
Current workaround for consuming interop project by project reference is to add cswinrt NuGet reference in app

May be fixed if NETSDK1130 is resolved

Possible way to fix this is to have the interop project add metadata to the TargetPath item it returns with the name of the WinMD that the interop project is wrapping.  Then we could add logic so that the consuming project could filter out the transitive WinMD reference that it gets, similar to how we are removing WinMD references with WinRT.Host.dll as the implementation

Just ran into this. It's my first time using CsWinRT and it's the second 6+ month old bug I've run into.

@Corillian - unfortunately we haven't been able to investigate this further yet. As a workaround, you can add a package reference to Microsoft.Windows.CsWinRT in your app to avoid the NETSDK1130 error.
Depending on your scenario, you might also want to take a look at the recent embedded support which allows you to compile projection/runtime sources into your app or library for self-contained usage of WinRT types.

I saw that in the earlier discussion, it worked, thanks for following up! While the future of WinUI3 sounds promising, lots of great things on the roadmap, it looks like it was released 6-12 months prematurely. Typically, the thing you guys (Microsoft) do best is toolchain support, but everything around WinUI3, including WinUI3 itself, seems to be pretty rough at the moment. Working with it, at the moment, is a bit frustrating.

@ocalvo has demonstrated that this works:
<PrivateAssets>build</PrivateAssets>


Content Warning/Content Summary name toggle

If an admin can call "Content Warning" something like "Subject Line" or "Summary" et cetera, an admin might be able to sidestep a lengthy discussions about the word "warning".

Yes, I think that's fair - I'd say a server config option that changes it to be "Content Summary" would be good?

I have a branch that controls the label and placeholder with a config. Didn't open a PR yet because had to step away from computer before adding tests. Is free form control too much? Would you prefer a bool to toggle between "Content Warning" or "Content Summary"?

Freeform control seems to me to be the best, with the default being "Content Summary".
Like if my users are too verbose having tl;dr might be the best

Let's go freeform control with "Content Summary" as the default. We can switch the "CW" button that's on the mini-compose right now to an icon.

Actually, changed my mind - I think "Content Warning" is a better default as that is the name people expect from Mastodon and general use.


Update for issue 13433 (attempt 2)

Wow, this was really more complicated than I thought it would be.

The saved field must be immutable or pure functions cannot access it
A static ctor outside a struct CANNOT set a static immutable field inside a struct.

So I had to do this weird thing where I store it in a static array outside the struct. But it does work (at least for MacOS X).
Note that all this can be undone when the static ctor bug is fixed.

BTW, you should avoid naming your branches simply numbers (or numbers and letters that can be interpreted as hash codes).

Wow, this was really more complicated than I thought it would be.

Yeah. I ran into the same sorts of problems when getting rid of the static constructors in std.datetime, though I didn't have the template instantiation issue to worry about there. I really wish that there were another way to have "trusted" purity other than casting a function pointer though (which is what std.datetime does with LocalTime() and UTC()).
Neat solution though, as ugly as it may be, but that can't really be helped. I'll merge it and rebase, and hopefully that will pass the autotester... finally.

You know, I was surprised that there was no "give me the index of this enum name" in std.traits, or "give me the index of this enum value". I had to figure that out on my own ;) The CTFE hoops that D makes you go through!

You know, I was surprised that there was no "give me the index of this enum name" in std.traits, or "give me the index of this enum value". I had to figure that out on my own ;) The CTFE hoops that D makes you go through!

LOL. Well, in many cases, it's a matter of it simply not having been thought of previously by one of the core maintainers, and no one who needed it enough in their own code to create it put the effort to get into Phobos. It wouldn't surprise me at all if the code were floating around in someone's github repo somewhere.


Remove line on ad hoc integration

@JohanFreelancer9 Copy editing is needed for this PR. Thanks.

@MikePlumleyMSFT PR has been copyedited and is ready for final review, could you please check and merge? Thanks!


pageserver: hang during Timeline::shutdown

Reason unknown.  Occurred during a routine detach operation for balancing.
Ref. https://neondb.slack.com/archives/C069BAQN1KL/p1702056471367319

This will close when we merge https://github.com/neondatabase/neon/pull/6097 - this is our best (+ only) theory.


GCCH Call out

Thank you for specifically calling out that this feature is available in GCCH. You mention the cost for commercial. Can you add additional language on pricing for GCCH. "If you are looking to use SharePoint Advanced Mgmt P1 in GCCH please contact your Microsoft account team or AOS-G Partner for pricing".

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

ID: e9aa1a91-e838-41c6-45cb-6a2a4dbf1a7a
Version Independent ID: 5f5c1b99-126f-2a70-20c8-f00136927e31
Content: Microsoft Syntex - SharePoint Advanced Management overview - SharePoint in Microsoft 365
Content Source: SharePoint/SharePointOnline/advanced-management.md
Service: sharepoint-online
GitHub Login: @MikePlumleyMSFT
Microsoft Alias: mikeplum

Hello, I'll propose to add that to the article.


Update email-anonymizer.py & readme

Hi - I fixed some typos, added some doc around HMAC algorithm, fixed the regex & hmac for email-anonymizer.py

Works, thank you for the commit!


fix: Remove undefined value in @azure/msal-node

Fixes #
#minor
Description
This PR avoids the undefined reference for thumbprint value in the ConfidentialClientApplication instance in msalAppCredentials for the latest versions of @azure/msal-node.
Specific Changes

Replaced undefined value by an empty string in clientSecret definition.

Testing
The following image shows a bot working with node 20 and the latest version of @azure/msal-node after the update.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>2
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.4%) to 84.078%

Totals

Change from base Build<PHONE_NUMBER>:
-0.4%

Covered Lines:
20347

Relevant Lines:
22904

💛  - Coveralls

Promoted /4718


Unrecognized fileds authorname and builtnassessorname

Describe the issue:
When I launched the "nnictl create" command,it occurred this mistake(I have came acrossed it):

I know the error is in the config_windows.yml file

I try to use "Assessor" module to early stop my trials,and modify my config_windows.yml file through the source code. But it also occurred error.
The example has ran successfully because I have deleted the autorname/experimentname/assessor module,but I want to know why it occurred mistake and I hope to fix it.
the source code

Environment:

NNI version:2.5
Training service (local|remote|pai|aml|etc):local
Client OS:Windows 10
Server OS (for remote mode only):None
Python version:3.8
PyTorch/TensorFlow version:1.10.1/2.7.0
Is conda/virtualenv/venv used?:conda
Is running in Docker?:no

Sorry the assessor example is outdated.
builtinAssessorName should be name.

Sorry the assessor example is outdated. builtinAssessorName should be name.

I have fixed the error as your reply ! Thank you !
But the authorName still has problem, and the experimentName is ok.

authorName has been removed.
config_assessor.json is totally outdated. Please ignore it.

authorName has been removed. config_assessor.json is totally outdated. Please ignore it until we update.

OK,I have known it,thank you!


review - question intro 배경 이미지 및 스타일 변경

🤔 해결하려는 문제가 무엇인가요?

close #207

🎉 변경 사항

리뷰어 질문 인트로 스탭의 이미지를 변경했어요

패딩과 데스크톱에 대응하기 위한 스타일을 추가했어요

🌄 스크린샷
https://github.com/depromeet/na-lab-client/assets/26461307/3711f589-d925-4851-a7b0-b080df8e6d0f

Codecov Report
Patch and project coverage have no change.

Comparison is base (d31c0c9) 91.90% compared to head (2de336d) 91.90%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #210   +/-   ##
=======================================
  Coverage   91.90%   91.90%
=======================================
  Files          38       38
  Lines         284      284
  Branches       52       52
=======================================
  Hits          261      261
  Misses         23       23

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


Use default domain as it might be different from mail domain.

Alternatively implement to test both. Might fix #602

I am really weak on this stuff, and can't tell what other problems this creates - I like the option you mentioned in #602 of trying for the domain in the email address and falling back to the default if that doesn't work - that feels safer than just using the default-domain.  I have a feeling that the whole business of pulling domains out of email addresses started because there were people who couldn't just use the default domain.

Ahh okay - I'll eventually try to make the required changes then.

@willdean I have made the required changes. @RedX2501 can you have a look too?

Looks good to me - I can't test this here, but I think the approach looks OK.

@larshg Are you happy this works OK on your setup for importing users that have both the default domain and a different domain?  If so then I think we should merge it and then push for a 6.0-beta release.  @RedX2501  is busy on other things at the moment and I don't think he can easily test AD stuff anyway.

Well, yes and no... It seems to work, but if one extracts the domain from the UPN (email) name and its different from the domain controller, it takes a long time to timeout and try the default domain... But I can't really come up with a better solution - at least for now - so lets go ahead and merge, to get some feedback?

I have just read more about AD, domains and UPN suffixes... For what I understand, the UPN suffixes is rarely the same as the domain name of the AD server and hence I don't think we should extract the suffix from logon name etc.
Either we should use the default domain from settings - or we should create a principal context with no domain specified and it will try to connect whatever the webserver domain is part of?
One explanation of additional suffixes can be read here.
Naming of AD servers FQDN is dicussed here  - ie. ad1.domain.com.
I don't think that if we extract domain.com from a logon name that the AD server would respond on this - because it should be on a subdomain...

I guess the original problem was reported by #562 from @mr-miles.

@larshg If we shouldn't be pulling domain names out of user names at all, does that mean that routines like GitAuthorizeAttribute.IsWindowsUserAuthorized are mistaken?
That uses a helper called StripDomain (it's mis-named, it should really be called ExtractDomain) to get a domain out of user@domain and then builds a PrincipalContext from that.
I too am concerned about long delays in the import, that something in IIS will then interrupt the import and cause all sorts of impossible to debug problems with partial imports.
Maybe we should just try to make things work properly using the 'Default Domain' for a 6.0-beta, and then look careful at situations where this doesn't work properly for people?   (I'm sorry as I know that's where you started with this PR)

Well, that is what I get out of all the reading :) - also, when I create the PrincalContext with the adress of our domain server my user is validated using the full UPN name - whereas the current StripDomain does remove the suffix of the UPN ie. @domain.com - but this shouldn't be necessary.
An alternative to the multi domain/server problem could be to change the DefaultDomain to a list of AD servers, which is then iterated. As long as the server is connected, the verification of a valid user is quite fast, so it shouldn't take long to validate a couple of servers? But again... if there are servers not up and running, it can take a long time to timeout the servers that are not available. Both on upgrade, but also when verifying all the git operations etc.
I'm still trying to read up on how to handle trusted domains from one domain to another in a multi domain/forest setup :)

The project so needs an AD expert - it's great that you've selected yourself  :-) 👍

A Microsoft article that describes what I suggested about multiple servers :)

@larshg Doesn't that article describe exactly what we have been doing, which is obtaining a domain name by splitting it out of a UPN?

Unfortunately yes - it does... :/

This article might be good read. I can see that we can access the forest and get first the root domain and any subdomain / servers and iterate all, until a user is found etc.
I'll try read up on it during the week.. Lets wait a bit with merging!


(renderers) Allows renderers to be pluggable through modV.use

Fixes #262.
Do not merge yet, need to check all drawing options still work.

@2xAA I wait for the review then until you tell me everything is fine!

@TimPietrusky This seems to be fine now, awaiting your review :)

@TimPietrusky Can we get this merged?


Create mako templates for all objects in JH's model

Done for all objects involved in Belsunu.atf.


[Dark Theme] Functional test are not passing due to minification of CSS files

Closing, minification process was improved


Driver cannot Create Nodes from POCOs

Self explanatory.

As you could find in the [driver doc] (http://neo4j.com/docs/developer-manual/3.0/index.html#types), the Node, Path and Relationship are types that only valid for return values.
May I ask why do you need to create a INode object in your app? Could you give me an example that requires creation of a node explicitly?

Could you give a use case for it ?

I am closing this issue as the problem the issue wants to address is unclear. Please be free to reopen this issue if you get a better explanation or example of the problem you are having.


scons not listed as a dependency, but is required.

Build/Linux/build_rcp.sh invokes scons, but nowhere is it listed as required. If you start with a clean install of Ubuntu, you will get confusing error that lists "Build arguments passed to scons" but obviously they weren't, if you don't have scons.
Command line arguments passed to build_rcp.sh:
RCPROOT=/home/crews/RCP/Build/Linux/../..
=====Building Radeon Compute Profiler======
Building infra projects
Build arguments passed to scons:
----------- Start building ---------------
Sun Feb  4 10:44:00 EST 2018
scons -C /home/crews/RCP/Build/Linux/../../Build/Linux CXL_prefix=/home/crews/RCP/Build/Linux/../../Output CXL_build_type=static
*** ERROR during the build of the 64 bit framework ***
(scons -C /home/crews/RCP/Build/Linux/../../Build/Linux CXL_prefix=/home/crews/RCP/Build/Linux/../../Output CXL_arch=x86 CXL_build_type=static )
*** ERROR during the build of the 32 bit framework ***
----------- End building -----------------
Sun Feb  4 10:44:00 EST 2018
*** ERROR ***
*** the build failed - see the logs for details ***
sudo apt-get install scons fixed issue. Should be listed as a prerequisite along with python

Thanks for pointing this out.  We will update the list of prerequisites in BUILD.md to include scons.


Add Privacy Manifest

Hello,
I am an employee of Monstarlab company. I have a question about the Privacy Manifest.
At WWDC23 Apple is requiring all api calls that fall under this ruleset https://developer.apple.com/documentation/bundleresources/privacy_manifest_files/describing_use_of_required_reason_api#4278394
be added to a privacy manifest file.
Does SwiftyUserDefaults need to include this manifest?
Please let me know about your upcoming plans to apply the privacy manifest.
Thank you!!!

+1


Change license

We want to change the license to CC0 to relieve the burden to fork.

To make this a plugin in WP category we need the license to be GPL 2.0


get current lang from url

Why
refactor for i18n
What
get current lang

[2024/09/02 15:02:47]<EMAIL_ADDRESS>(koudaiii/refactor)(minikube)
$ npm run test:e2e

><EMAIL_ADDRESS>test:e2e
> jest tests/e2e

 PASS  tests/e2e/content.e2e.test.js (8.075 s)
  JP Learn Microsoft.com Update Checker E2E Test
    ✓ should display English update date on Japanese Microsoft Learn page (1054 ms)
    ✓ should not run script on non-ja-jp pages (558 ms)
    ✓ should run script with jp-learn-microsoft-com-update-checker-debug flag (235 ms)

Test Suites: 1 passed, 1 total
Tests:       3 passed, 3 total
Snapshots:   0 total
Time:        8.21 s, estimated 11 s
Ran all test suites matching /tests\/e2e/i.

[2024/09/02 14:56:41]<EMAIL_ADDRESS>(koudaiii/refactor)(minikube)
$ npm run test:unit

><EMAIL_ADDRESS>test:unit
> jest tests/unit

(node:8428) [DEP0040] DeprecationWarning: The `punycode` module is deprecated. Please use a userland alternative instead.
(Use `node --trace-deprecation ...` to show where the warning was created)
 PASS  tests/unit/content.test.js
  URL Check in Content Script
    ✓ should only run script if URL starts with https://learn.microsoft.com/ja-jp/ (3 ms)
    ✓ should not run script if URL does not start with https://learn.microsoft.com/ja-jp/

Test Suites: 1 passed, 1 total
Tests:       2 passed, 2 total
Snapshots:   0 total
Time:        3.328 s
Ran all test suites matching /tests\/unit/i.


feat(proto): Proto API v2 [DEV-1786]

Task linked: DEV-1786 Build and lint Protobuf definitions for v2

Processed the rest of comments. I think it's ready now.

Todo: Add deactivation transaction


Exposed password problem

How is the best way to hide password from datasource.properties file when using simple-jndi? Is there a way to use environment variables?

edpichler also wrote:
"How can I activate variable interpolation of Apache Commons Configuration?
I saw it is possible on your JUnit tests:
...
By the way, the reason I need to use it is to hide passwords on property files. I will use environment variables."
Hello Eduardo,
interpolation is done by JndiConfiguration. So such variables will not be interpolated by calling InitialContext.lookup(). Instead you have to pass the InitialContext to a JndiConfiguration instance:
InitialContext ctx = new InitialContext();
final JNDIConfiguration jndiConf = new JNDIConfiguration(ctx);

Then use one of JNDIConfiguration's getter methods, to access the interpolated values, e. g.:
jndiConf.getString("...")

It is important to correctly adjust org.osjava.sj.delimiter and jndi.syntax.separator as demonstrated by the unit tests in JNDIConfigurationTest class.
regards
Holger

Oh, now I got it Mr @h-thurow, thank you.
In the case, I am using weld-junit5, I don't know if it's possible without changing the weld-junit source code.
@h-thurow is there a way where I can, after reading the properties file, replace the values in runtime using Environment Variables?

Try this. Use an arbitrary password in your datasource.properties file. Before the first connection is made, lookup the DataSource in your application, cast it to org.apache.commons.dbcp2.BasicDataSource and set the correct password by calling BasicDataSource#setPassword.

@h-thurow wow, it's worked. Thank you Mr. Hackerman!
Maybe, hiding passwords would be nice this feature, but I don't know if this is a role of this project or it is a charge of weld-junit.
By the way, I just remember that on weld-junit I can create a custom resource and attach it to the JNDI context. I realized that I am using simple-JNDI attracted by the simplicity of the project, and due this problem, I am not sure if I will keep this dependency.


fix: Fix issue with new Duo prompt and TGC cookie

The UniversalPrompt flow from Duo causes the browser to leave CAS, then return to CAS to complete authentication.  This flow is causing the TGC cookie, (and possibly others) to be sent to the browser with a maxage of 0, which pretty much breaks SSO forcing a re-auth on every service. The root cause of the issue appears to be that there are 2 methods being used to add the TGC cookie to the response.  The response.addCookie method and the response.(set/add)Header method.  The use of the two different ways is causing the bug.  The underlying HttpResponse code adds the 'setHeader' data to the response being sent back to the HttpClient, then it adds 'addCookie' data to the response being set back to the client.. The net effect is anything added using the (add/set)Header methods using the 'Set-Cookie' name gets overridden by whatever is in the collection stored by 'addCookie'.  I understand, the (add/set)Header method is being used to overcome limitations in the current Java API such that the SameSite attribute isn't in API. However, because both methods of setting the cookie are being used, this is causing a 'bug' in cases where the login flow leaves CAS, and returns.
The change basically removes the
'HttpServletResponse.addCookie' call from 'removeCookie' and 'removeAll' methods.  Since the Spring CookieGenerator doesn't take the HttpServletRequest and the addCookieToResponse method takes the request to just get the sameSite policy data, I created a 2nd method that takes the string value but basically performs the same logic.

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


Chain DB: bound the growth of the set of invalid points

The Chain DB maintains a set of points corresponding to blocks that were invalid.
Whenever a garbage collection of the Volatile DB is performed for some slot s, the points with a slot  <= s will be removed from that set.
Nevertheless, attackers can try to flood us with invalid blocks, causing this set to grow without bound. In practice, the network layer will disconnect from such nodes, but we should be careful and think about this.

The ChainSync Client will disconnect from peers with invalid blocks. There's not much we can do at the ChainDB level, it doesn't even know what peer a block is from.


Merge remote master

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 6 committers have signed the CLA.:white_check_mark: YuKitsune:x: OronDF343:x: jairbubbles:x: bording:x: louis-z:x: frindlerYou have signed the CLA already but the status is still pending? Let us recheck it.


unary_- is missing on Length

padding(-14.px)

Would be good to fix this. In fact, this would be a tiny PR if you're interested.
As a workaround, you can currently do this:
padding(-14 px)

hey @japgolly Unfortunately, I had to give scalacss.
I had existing less and I was not able to convert some selectors in Standalone mode:
x {
  y, z {
   k: y;
  }
}

No worries @MasseGuillaume . That syntax should work but honestly, I'm not sure how much value there is in standalone mode. Inline mode is where this library becomes exponentially awesome, one of the most simple reasons is that your css is always linked to your usage at compile-time so it always works (not stringly-typed), it's really easy and safe to refactor and see what's used where etc.

@japgolly yeah I know inline mode is really helpful. Concretely I want to style codemirror, it's created by javascript code in a separate library, this is why I need a good standalone mode.

@MasseGuillaume Ah I understand.

Added in 9f4fd802b6e57c76169b38db885344bca97b2947.


CHROM add flex-start and flex-end

Changes

Added flex-end and flex-start

Right now we are setting flex-end to align-items: end when I think we should allow for both being an option to pass in.

Ex issue:

Before when I passed in align="flex-end"

After when I passed in align="flex-end"

Nice addition


Feedback on String Interpolation

I had a friend run through all of this last night.  He said steps 4-7 for the String Interpolation were explained in a way that confused him, he found it hard to understand.
He also mentioned when introducing the "Trim" method, that the example seemed over complicated, and that parentheses character was introduced, but then not explained.  So he was unsure what it does.
He suggested using a simpler or less-complicated example for Trim.
The said the rest of the entire tutorial was a pleasure, easy to follow and understand.  :)

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

ID: df2cc16c-fcf9-41e6-a1e5-92dfc2869495
Version Independent ID: 4421a02c-f819-1483-12c6-7dab4f289a46
Content: C# interactive tutorials - C# quickstarts
Content Source: docs/csharp/quick-starts/index.md
Product: dotnet-csharp
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

@shehackspurple are you talking about the Hello C# quickstart?

The string interpolation quickstart has only five steps
The Trim method is introduced at the step 4 of the "Hello C#" quickstart

@BillWagner there is another issue with the "Hello C#" quickstart: the step 3 discusses the + operator for string concatenation, but all code examples use string interpolation. Moreover, the string interpolation is presented at the step 2 as alternative to using the + operator. So, the step 3 shouldn't mention the + operator in the concatenation context anymore.

Yes, I am talking about that tutorial.
Was that helpful feedback?

On Jul 23, 2018, at 4:33 AM, Petr Kulikov<EMAIL_ADDRESS>wrote:
@shehackspurple https://github.com/shehackspurple are you talking about the Hello C# https://docs.microsoft.com/en-us/dotnet/csharp/quick-starts/hello-world quickstart?
The string interpolation https://docs.microsoft.com/en-us/dotnet/csharp/quick-starts/interpolated-strings quickstart has only five steps
The Trim method is introduced at the step 4 https://docs.microsoft.com/en-us/dotnet/csharp/quick-starts/hello-world?tutorial-step=4 of the "Hello C#" quickstart
@BillWagner https://github.com/BillWagner there is another issue with the "Hello C#" quickstart: the step 3 https://docs.microsoft.com/en-us/dotnet/csharp/quick-starts/hello-world?tutorial-step=3 discusses the + operator for string concatenation, but all code examples use string interpolation. Moreover, the string interpolation is presented at the step 2 https://docs.microsoft.com/en-us/dotnet/csharp/quick-starts/hello-world?tutorial-step=2 as alternative to using the + operator. So, the step 3 shouldn't mention the + operator in the concatenation context anymore.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/dotnet/docs/issues/6582#issuecomment-406981808, or mute the thread https://github.com/notifications/unsubscribe-auth/AZgYC_RE7rntSDnXE1ejXtm69xgqIV9mks5uJYpsgaJpZM4VY1ll.

@shehackspurple thank you for the clarification.

He also mentioned when introducing the "Trim" method, that the example seemed over complicated, and that parentheses character was introduced, but then not explained.

As for the parentheses, I can imagine that the following code might be confusing:
string trimmedGreeting = greeting.TrimStart();
Console.WriteLine($"[{trimmedGreeting}]");

While braces is the part of the C# syntax and mean string interpolation to a compiler, the square brackets are the part of the text to display; they are used to simplify display of leading and trailing white spaces. Or empty parentheses in the method call are confusing?
Your feedback is helpful, thank you. It turns out that the "Hello C#" quickstart should be make clearer.

Hello all, it’s the “[“ character.  When someone is new, everything is new.  My friend was totally lost and thought he had missed something earlier in the tutorial.  Perhaps we should cut that part out until it’s explained what that character does?  Honestly, I’m a VB.net person and I am also just learning C# and I didn’t know what [ was for either.  :-D
Thank you for improving this so that all new learners have an easier time.
Tanya Janca
@SheHacksPurple
http://devslop.co/
https://medium.com/@shehackspurple
https://www.slideshare.net/TanyaJanca

On Jul 23, 2018, at 5:14 PM, Petr Kulikov<EMAIL_ADDRESS>wrote:
@shehackspurple https://github.com/shehackspurple thank you for the clarification.
He also mentioned when introducing the "Trim" method, that the example seemed over complicated, and that parentheses character was introduced, but then not explained.
As for the parentheses, I can imagine that the following code might be confusing:
string trimmedGreeting = greeting.TrimStart();
Console.WriteLine($"[{trimmedGreeting}]");
While braces is the part of the C# syntax and mean string interpolation to a compiler, the square brackets are the part of the text to display; they are used to simplify display of leading and trailing white spaces. Or empty parentheses in the method call are confusing?
Your feedback is helpful, thank you. It turns out that the "Hello C#" quickstart should be make clearer.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/dotnet/docs/issues/6582#issuecomment-407203387, or mute the thread https://github.com/notifications/unsubscribe-auth/AZgYCwc90922dG_KgjaAM3lUXinm17y2ks5uJjy7gaJpZM4VY1ll.


#304 Implement pagination

Implement pagination

@aleksandarmicev Can you fix this comments when you have the time, so we can merge this?
Also resolve the conflict with the project Section

@aleksandarmicev resolve conflicts here

@aleksandarmicev resolve conflicts here

Conflicts are just resolved, merged new changes from the staging

@aleksandarmicev Please use this in combination with the table and some dummy data to test if this is working, and attach a video. 🙏

@aleksandarmicev Is this tested? Could you maybe add it, in the reusable table implementation in content-panel > blogs?

Question about the requirements:
The pagination requirements look partial.  Is there any other ticket about actually displaying the pages, total count, current page ? This looks like just defining how many items will be visible per page..or am I missing something

@edichoska Question about the requirements:
The pagination requirements look partial. Is there any other ticket about actually displaying the pages, total count, current page ? This looks like just defining how many items will be visible per page..or am I missing something

Yes there was another ticket for that, already done: https://github.com/learnhubmk/app/pull/451

@aleksandarmicev
We need to close this PR and try to re-use the code to merge this logic under Pagination.tsx component whhic will be genetic component re-suable in all places

Closing this PR since we will update the Pagination.tsx component and that one will be reused everywhere


Improve RFB protocol handling

Improve handling of initial handshake #110
Force transport to loose connection on protocol errors due to the lack of framing
Basic handling of SetColourMapEntries (only handle the message, but adding support for palette formats should be done in the future)
Initialize all values in __init__().

needs an update to resolve conflicts

needs an update to resolve conflicts

I rebased my branch and force-pushed it.


Set up documentation building

Set up building of the documentation from docstrings

ideally to markdown
upload the built docs to S3 so they can be shown on the web
possibly through GitHub actions

Just an idea - If it's markdown in suitable format it could be also committed to the actual repository - either as README.md or DOCS.md or whatever else and available on Github and at the website we can load it from Github.


[FEATURE REQ] Azure.ResourceManager.ServiceBus : Manage minimum TLS version with SDK

Library name
Azure.ResourceManager.ServiceBus
Please describe the feature.
Hi,
When creating a service bus with the newer SDK, we have to fill a ServiceBusNamespaceData object.
However, this class doesn't allow us to manage Properties for the namespace, including the minimum TLS version.
For reference with CLI, PowerShell module and ARM template :
https://learn.microsoft.com/en-us/azure/service-bus-messaging/transport-layer-security-configure-minimum-version

Thank you for your feedback.  Tagging and routing to the team member best able to assist.

Hi @BastienPerdriau, thanks for reaching out to us, currently the API version of Azure.ResourceManager.ServiceBus is 2021-11-01 and the setting of minimum TLS version was introduced from 2022-10-01-preview. I will upgrade the API version later.

Hi @BastienPerdriau, we have released a new beta version for Azure.ResourceManager.ServiceBus, now you could manage the TLS version in SDK, please have a try, thanks!
By the way, if you have any other feedback to our Azure SDK, feel free to let us know in this survey

Hi @Yao725 , it seems ok to me, I ran some tests with it.
Looking forward to have a stable version of this, it's better for us in our internal version management.

Glad to know it works. And we will release a stable version once the latest API version becomes stable.


"illegal access" in debuggability.js

I'm upgrading from Node 0.12.9 to Node 4.2.3 and ran into an illegal access error in debuggability.js. I also tested with Node 5.5.0 and see the same error.
The error is this:
C:\app\2016-01-lib-upgrades\src\node_modules\bluebird\js\release\debuggability.js:535
    var firstStackLines = firstLineError.stack.split("\n");
                                        ^
 illegal access

C:\app\2016-01-lib-upgrades\src>

context:
function setBounds(firstLineError, lastLineError) {
    if (!longStackTracesIsSupported()) return;
    var firstStackLines = firstLineError.stack.split("\n");
    var lastStackLines = lastLineError.stack.split("\n");
    var firstIndex = -1;
    var lastIndex = -1;

I'm not actually using bluebird myself.. I'm using the Stripe API which uses bluebird. It was referencing bluebird 2.9.6 and I upgraded it to point to the latest 3.1.5. The error is the same in both versions, only the location is different (2.9.6 has it in capture_traces.js).
If I revert back to Node 0.12.9 I do not get the error.
Any suggestions?
Thanks,
Sam

There is something else that is really screwing up your environment

I spun up an AWS EC2 Linux instance with Node 4.2.3 (latest supported on Elastic Beanstalk) and get the same error there.
Any suggestions on where to look would be helpful. Searches online turned up nothing relevant.

I mean you are using some highly weird module or config which is affecting the stack property of all errors. You could egrep for "\bstack\b" in your node_modules for instance.

Thank for the tip, I'll look through the node_modules.

Cannot reproduce with the same settings AWS EC2 Linux Node 4.2.3 with clean build and bluebird. Ran test suite and everything passes.
Thanks for the report. Like Petka said - it's a weird module affecting the stack.

Is this one https://github.com/petkaantonov/bluebird/issues/1252 related to the issue?


Change node creation to flatten out one level of function calls and r…

…emove polymorphic initialization of data.

:+1: I thought I profiled https://github.com/google/incremental-dom/pull/149...

Well, one problem I have noticed with the tests is that they are usually pretty consistent across a single page refresh, but between them it can vary quite a bit (at least on Chrome, Firefox seems to be more consistent).
I tried using benchmark.js when I was first creating the tests and I noticed the same issue. I think it also might work better for microbenchmarks rather than things that take (relatively) a long time like modelling a real world scenario.
It kind of sucks, but it might be a good idea to refresh the page a few times on any change to see how it bbehaves.


Fixed one action and added 2 more actions

Changed the name from 'FCA' to 'Financial Conduct Authority UK' and also updated the description.
Fixed the 'Common Search' action.
Added 'Find Firm Details' and 'Find Product Details' actions.

When submitting a connector, please make sure that you follow the requirements below, otherwise your PR might be rejected. We want to make you have a well-built connector, a smooth certification experience, and your users are happy :)
If this is your first time submitting to GitHub and you need some help, please sign up for this session.

[x] I attest that the connector doesn't exist on the Power Platform today. I've verified by checking the pull requests in GitHub and by searching for the connector on the platform or in the documentation.
[x] I attest that the connector works and I verified by deploying and testing all the operations.
[x] I attest that I have added detailed descriptions for all operations and parameters in the swagger file.
[x] I attest that I have added response schemas to my actions, unless the response schema is dynamic.
[x] I validated the swagger file, apiDefinition.swagger.json, by running paconn validate command.
[x] If this is a certified connector, I confirm that apiProperties.json has a valid brand color and doesn't use an invalid brand color, #007ee5 or #ffffff. If this is an independent publisher connector, I confirm that I am not submitting a connector icon.

If you are an Independent Publisher, you must also attest to the following to ensure a smooth publishing process:

[x] I have named this PR after the pattern of "Connector Name (Independent Publisher)" ex: HubSpot Marketing (Independent Publisher)
[x] Within this PR markdown file, I have pasted screenshots that show: 3 unique operations (actions/triggers) working within a Flow. This can be in one flow or part of multiple flows. For each one of those flows, I have pasted in screenshots of the Flow succeeding.
[x] Within this PR markdown file, I have pasted in a screenshot from the Test operations section within the Custom Connector UI.
[x] If the connector uses OAuth, I have provided detailed steps on how to create an app in the readme.md.

Hi @vmanoharas
As suggested, I have closed the PR https://github.com/microsoft/PowerPlatformConnectors/pull/3397 and submitting this fresh one.
Thanks.
Gulshan


BlobStoreWriter: Throw IOExceptions.

Instead of throwing an unchecked exception and ignoring the close on the
output file, throw checked IOExceptions.
We should consider an fsync before closing the outputstream to get a better
consistency guarantee.
With this commit, when saving an attachment fails on UnsavedRevison#save
a CBLException is thrown with STATUS_ATTACHMENT_ERROR.
MultipartDocumentReader continues to throw unchecked exceptions.

Unit tests in couchbase/couchbase-lite-android#576

@sergio91pt Thank you very much for your contribution!


Add bench tool for scenes + scenes benches on masses

Create a whole subproject to test scenes
Add a generic (and simple) templated benchmark for scenes.
And add two scenes benchmarking masses (based on the scenes in examples/Components/masses)
TODO: maybe create different projects/executables to split benches between functions, components and whole scenes done with the scenes

That's a very useful tool. How many times are the benchmark run? Since it is quite a long execution, I expect only once. This is a bit in contradiction with the "unit" characteristic of the benchmarked code, but I find it useful anyway. I only have a question on the number of repetitions.

I quite agree with you, here I merely use the bench tool to repeat a scene (like running manually runSofa -g batch and do an average). This reinforces my feeling it should be a (independent) subproject of SofaBenchmarks...

Actually, I am not sure what is the best ; measuring

X times a fixed nb of steps ?
once a increasing nb of steps ?
Both of them make sense actually.


Introduce MessageToByteEncoderForBuffer

Motivation
Some Netty Contrib modules need MessageToByteEncoder that uses Buffer in order to migrate Buffer.
Such modules are socks-proxy, codec-haproxy ...
Modification

Add MessageToByteEncoderForBuffer
Add unit tests

Result
Netty Contrib modules can migrate to Buffer

@chrisvest @franz1981 Thanks!


feat: Ignore snap virtual mounts when enumerating disks

Ignores mounts that start with with /var/snap when rendering disks.
Also, idea: A list of regexes to ignore for disks, coming from local.conf?

I encountered the same issue with /var/snap/* mounts, and they are now ignored by default. In fact as a systematic solution, LCC now has options to flexibly customize mountpoints, please check the doc and local.conf.example.


♻ Mobile | Limit shipping to Australia

Hi team
Pain
When claiming a reward, we currently autofill for Australia addresses only. However, the user can override this and manually enter an overseas address.
What code could be improved?
Update the text to make it clear we only ship to Australia.

Tasks

[x] Update text in the app

Thanks!

Done - it should be really obvious only Australian addresses are accepted now.

Figure: Updates button to show Australia only

Figure: Updates text to show Australia only

Figure: Shows Australia in the address

Figure: Shows a new readonly field with Australia as the country


weave setup returns error when getting new images

Seems to hang when I run 'weave setup' here:
4a5724cea5e5: Error pulling dependent layers
Error pulling image (latest) from docker.io/weaveworks/weaveexec, Server error: Status 0 while fetching image layer (4a5724cea5e5c297148019f084a1c98125d791c3ea986cedbcbe3edf1dec654c)
or is this a docker github issue:
'weave setup'
Unable to find image 'weaveworks/weaveexec:latest' locally
latest: Pulling from weaveworks/weaveexec
4a5724cea5e5: Verifying Checksum
e22d9bbedd38: Layer already being pulled by another client. Waiting.
e92868fcb4b9: Layer already being pulled by another client. Waiting.
2b20bbd0e3b8: Layer already being pulled by another client. Waiting.
0c1c7b6f47b1: Layer already being pulled by another client. Waiting.
06ef00beeac8: Layer already being pulled by another client. Waiting.
06ef00beeac8: Layer already being pulled by another client. Waiting.
a5b60fe97da5: Already exists
16be31c9facc: Already exists
209c90b77f33: Already exists
6f8b93615288: Already exists
Pulling repository docker.io/weaveworks/weaveexec
waits.....forever!

FYI: seems to be an issue with 1.8.1 "Layer already being pulled by another client. Waiting." #15603
https://github.com/docker/docker/issues/15603

also:
https://github.com/docker/docker/issues/12823

This happens sometimes with Docker. It's not a weave issue.
Try bouncing your docker daemon. Or rebooting the VM on which it is running.

what does this mean?
in docker logs weaveproxy
Ignoring container due to --net option: "host"

Oops wrong thread, sorry.


Added formatting with spotless

Adding formatting check for the code with spotless, if the formatting is incorrect, the build will fail.
The format chosen is https://github.com/palantir/palantir-java-format, which builds on the standard google format but allows modern features such as longer line limits (120 chars) and better lambda formatting.
To format with it, you can use the new spotless:apply maven command, or just set your IDE settings to it.
The diffs seem mostly to be import order and new lines at the end of files.

Closing this, as we decided to use another formatting plugin.


Forbid uploaders pointing to attributes called 'file'

Sucky to find a random SystemStackError and have to figure out that this was the issue https://github.com/dwilkie/carrierwave_direct/issues/18
By any chance can CarrierWaveDirect::Uploader prevent this error in a clean way?

That issue is already fixed in master, see #188
Try it and reopen this issue if you encounter any problem.

Perfect, thanks!


IndexOutOfRangeException parse crash

Row 49 in Models/Client/WhisperMessage.cs will split with tail on the raw irc which will result in an unhandled IndexOutOfRageException if the tail contains a semi-colon.

Looking into this now.

Git auto closed the issue, but feel free to reopen if the problem is not fixed with the above commit.


Make async layer independent.

Currently we are using async_io for handling async feature.
And it currently works.
But it could be cool to be able to switch async primitive seamlessly.
Like use a smol::Unblock which runs async stuff in a separate thread.

closed in #24


Hosting Docs

Adresses part of https://github.com/aspnet/Docs/issues/58

Servicing
Directory Structure
Http PlatformHandler
Isolation by AppPools

@blowdart Could you have a look?

Is it build in a branch anywhere or do I need to do it myself?

If you are on intranet, try this

@blowdart Based on discussion earlier today, I've added content for setting up data protection in shared hosting scenarios. (Haven't covered web farms yet). Could you have a look at c3b1a9b155735dca3a88bc947ae715ae532b249f?

@blowdart Can you review the X.509 section?

@Erikre Please review.

:shipit:


MSAL doesn't recognize the signed MS Account when there are multiple accounts Signed

Core Library
MSAL.js v2 (@azure/msal-browser)
Core Library Version
2.24.0
Wrapper Library
MSAL Angular (@azure/msal-angular)
Wrapper Library Version
2.3.0
Description
We have two sites published in two different servers on the same domain (currently azurewebsites.net) that use MSAL to authenticate the user.
The Internal site is called by the Main site via 'iframe'.
When only one user authenticates to the Main site, the Internal site inherits the Active Account and works fine.
Instead when there are multiple Active Account the Main site lets you correctly choose which user to use, but the Internal site doesn't recognize the choice and try to propose it again.
Error Message
No response
Msal Logs
No response
MSAL Configuration
const config = {
    auth: {
      clientId: environment.clientId,
      authority: 'https://login.microsoftonline.com/' + environment.config.tenant,
      redirectUri: environment.redirectUri,
      postLogoutRedirectUri: environment.redirectUri,
      navigateToLoginRequestUrl: true,
    },
    cache: {
      cacheLocation: BrowserCacheLocation.LocalStorage,
      storeAuthStateInCookie: isIE,
    },
    system: {
      allowRedirectInIframe: true,
    }
};

const guard = {
    interactionType: InteractionType.Redirect,
    authRequest: {
      scopes: [
        'user.read'
      ],
    }
};

Relevant Code Snippets
constructor(private authService: MsalService){}

...
// Multiple Accounts Scenario

this.authService.instance.getActiveAccount() // Is null in the Internal Site even if in the Main Site there was the choice
this.authService.instance.getAllAccounts() // Is empty in the Internal Site
...

Reproduction Steps

Signed in with two AAD accounts
Sign with one of them
In another Browser Tab a site use the MSAL login, doesn't recognize the previous choice

Expected Behavior
The Internal Site should recognize that a choice has been made and it should automatically perform the login
Identity Provider
Azure AD / MSA
Browsers Affected (Select all that apply)
Chrome
Regression
No response
Source
External (Customer)

We published two application on the same domain, connected to the same Azure Application.  to check if after reproducing the situations of the multiple AAD Accounts signed the problem remains.

We published two application linked to the same Azure Application, on the same domain, to test the problem of the multi AAD signed accounts. After logging in an application, the second one didn't recognize the current Active Account and reproposed the login form.

When you say same domain, is it as per same domain rules, and can you please confirm if the cache is shared?  I suspect the cache isn't and hence you are unable to fetch the accounts info. Please let us know once you check this.


CVP-3890 Quote values to match expected format

Summary of Changes
The bundle validation scripts expect channel names to be a string in yaml as opposed to a float - so we need to double quote these values!

sign-off ?!

Should this be updated upstream in https://github.com/gatekeeper/gatekeeper-operator instead?

OLM docs actually have everything quoted, so I think this change should be upstream: https://olm.operatorframework.io/docs/tasks/creating-operator-bundle/#contents-of-annotationsyaml-and-the-dockerfile
Looks like the workflow needs an update also to handle it. I can look into it.

Here is the upstream PR:
https://github.com/gatekeeper/gatekeeper-operator/pull/249
I'll close this one and sync the changes down to Stolostron afterwards.


🐛 Flyout menu - strange icon (feedback v4.0.0)

Describe the bug
On demo app:

There is a icon on the left and the indicator.
And Btw, when we open a submenu, if we click outside the menu, only the submenu is closed (on Windows, when we click outside the menu, all menu is closed). Maybe related to https://github.com/bdlukaa/fluent_ui/issues/332)

Weird. Is there any change of it being a rendering issue?

Tested on Firefox + Edge, same problem. (current demo app here https://bdlukaa.github.io/fluent_ui/)

Reproducible on VM-based environments as well!

FlyoutListTile builds the trailing widget twice. Removing line 212 solved it for me.
https://github.com/bdlukaa/fluent_ui/blob/de0e316c9e927b817e6f86b1d6e2bc95520dd398/lib/src/controls/surfaces/flyout/content.dart#L187-L215

Nice catch!


.Fields Array Help

Hello - Hoping to ask a question. I am trying to use .fields to pull my data request out of my query. I have a union of two unlike tables and need to enter in some placeholders.
When I build my query with .field it looks like this
const query = squel.select().from(object_archive)
.field('object', 'archive_type')
.field('id')
.field('null', 'from_id')
.field('null', 'to_id')
I tried to use .fields as follows
const objectFields = {"'object'": 'object_archive', id: '', null:'from_id", null:'to_id'}
const query = squel.select().from(object_archive)
.fields(objectFields)
Where I use the .field version I get what I expect. When I use .fields it drops 'from_id'. It has something to do with the two nulls. Any idea of what I am doing wrong?

Field keys must be unique, thus the second null will cancel out the first one, resulting in:
{"'object'": 'object_archive', id: '', null:'to_id'}


Action Listener Unresponsive to Interaction in App Home

I'm working on creating an app that will allow users to select an account from a dropdown list, and the homepage will update with information about the account upon selection. I've gotten interactions to work through a modal, but cannot get any response on the Home Tab of the App.
I have enabled Interactivity for my application and am testing locally using ngrok. I have the same URL handling requests from the Event Subscription and the Interactivity tab where both URLs are ending with slack/events.
Reproducible in:
The slack_bolt version
slack-bolt==1.18.0
slack-sdk==3.21.3
Python runtime version
Python 3.11.2
OS info
ProductName:            macOS
ProductVersion:         13.3.1
ProductVersionExtra:    (a)
BuildVersion:           22E772610a
Steps to reproduce:
app = App(
    token=os.environ.get("SLACK_BOT_TOKEN"),
    signing_secret=os.environ.get("SLACK_SIGNING_SECRET"),
)

@app.action("account-select-action")
def handle_action(body, ack):
    ack()
    print(body)

@app.event("app_home_opened")
def update_home_tab(client, event, logger):
    try:
        user = event["user"]
        client.views_publish(
            user_id=user,
            view={
                "type": "home",
                "blocks": [
                    {
                        "type": "header",
                        "text": {"type": "plain_text", "text": "Account Summary"},
                    },
                    {
                        "type": "input",
                        "element": {
                            "type": "static_select",
                            "action_id": "account-select-action",
                            "placeholder": {
                                "type": "plain_text",
                                "text": "Select an item",
                            },
                            "options": [
                                {
                                    "text": {"type": "plain_text", "text": "Option 1"},
                                    "value": "value_1",
                                },
                                {
                                    "text": {"type": "plain_text", "text": "Option 2"},
                                    "value": "value_2",
                                },
                                {
                                    "text": {"type": "plain_text", "text": "Option 3"},
                                    "value": "value_3",
                                },
                            ],
                        },
                        "label": {
                            "type": "plain_text",
                            "text": "Please Select an Account:",
                        },
                    },
                ],
            },
        )
    except Exception as e:
        logger.error(f"Error publishing home tab: {e}")

if __name__ == "__main__":
    port = int(os.environ.get("PORT", 3000))
    app.start(port=port)

Expected result:
I would expect the body of the response to be printed when I changed the selection in the home tab of the application. I would like to confirm that this interaction is working so that I can update a markdown data table with information related to the selected account in the static dropdown menu.
Actual result:
There is no response, nothing printed, and nothing to prove to me that this interactivity is functioning properly. There is nothing on the ngrok page either to indicate that this interaction ever happened.
Requirements

Hi @dglindner2, thanks for writing in!

There is nothing on the ngrok page either to indicate that this interaction ever happened.

This indicates the button click event might not be sent to your public endpoint. Could you double-check if you've enabled "Interactivity" and have configured a valida "Request URL" on the https://api.slack.com/apps/{your App ID}/interactive-messages page?
I hope this helps.

I have the same Request URL that I am using for handling Event Subscriptions (which is working). I am able to receive payloads from events so I know that the URL is valid. Additionally, when I use interactivity from a Modal this is valid. The non-response only occurs when trying to use interactivity in the app home tab.

@dglindner2 Ah, you're using input blocks on Home tab. In this case, you need to set dispatch_action: true to trigger block_actions payload events. See https://api.slack.com/reference/block-kit/blocks#input for details. This option is necessary when you use input blocks on either a channel message or Home tab. Another way is to use section / actions blocks instead. With these blocks, user interactions trigger block_actions events. I hope this helps.

Amazing this worked! Thank you - could not figure out why it is working in Modal but not Home tab. Thanks!


Systematically test serialization of trained models

We need to make sure that all Shogun can dump trained models to disc!
Basically, every learning algorithm in shogun, that is based on training/test (so starting with CMachine), should be trained on a (small) exampe dataset, then serialized, then de-serialized and made sure that it can be applied to test data.
This task is to automate that.

Hi, I'm new here and wish to take up this issue. So, kindly guide me through.

Well, write something that trains a models, serializes it, deserializes it, and then applying it to test data gives the same results. Then automatically make this happen for all models

Actually I'm new to Open Source and know only the basics Machine Learning(with no experience in any project) so can you please explain to me in detail or assign me to an easier issue. I want to be involved in this project, so please guide me.

Please read the "getting involved" guide in the wiki, about starting with Shogun. We can't hold your hands unfortunately. I suggest you pick something simpler if you don't know how to start here.

@karlnapf i want to take this up.
Is serialisation module implemented or not. Or should we only implement the testing part of it.

We have serialization working at the moment (at least in theory). The details under the hood will soon be replaced, but that doesn't matter for this task.
Try to come up with a simple way to test serialization / de-serialization of all models.
Algorithm for supervised methods (CMachine subclasses)

Load training data
Train model on training data
Serialize model to file
De-serialize model from file
Apply model to test data

Unsupervised methods and other will work similarly, but let's start with the supervised ones.
The best thing was if we were doing this automatically for modes of similar classes (supervised, unsupervised, kernel machine, gaussian process, etc)
You can start via writing a unit test for a single model, as a prototype. However, eventually, we want to use some template engine to generate unit tests for all implementations we have of a particular class. We did a similar thing for serialization itself here, using jinja2. I would start writing a simple test for say a CLinearMachine and then make it automatically be generated for all subclasses.
The cmake file that generates the tests using jinja2 is here

@karlnapf I've made a prototype for LinearMachine subclasses both for binary classification and regression using jinja2 which follows the steps listed above (and it checks if the predictions pre and post serialization are equal, don't know if it's useful).
https://github.com/micmn/shogun/blob/feature/serialization-test/tests/unit/io/LinearMachineSerialization_unittest.cc.jinja2
Generates: https://gist.github.com/micmn/6e447c6c491350e22c0efcb5091701ad
Is there a convenience function to generate random train/test data?

@micmn this is a great start! i'd love to have that fixture class actually be implemented somewhere else and that everybody could use it withing the unit tests as currently every unit test uses/implements their own data generators... @karlnapf ?

@micmn i guess i just answered your question implicitly... no we dont have.. but i believe now that we are talking about we would need such functionality i believe within the library.. @karlnapf ?

The data should definitely be generated elsewhere. But in fact, we don't we just use the available datasets we have? See the meta examples, I  made sure that all datasets used in there do actually make some sense

I dont think you need to distinguish in the code between classification and regression. If you just used apply rather than say apply_regression, then you get a CLabels pointer, but serializing it actually should work.

While the explicit list of things to be tested at the end of the file is great, I wonder whether we could apply this to all CMachine instances by default, and have a blacklist instead? For that, we need some code that works on generic CMachine pointers first (maybe using the data generation trick I mentioned above)

Great start indeed, this is exactly what I had in mind. Want to send a PR for further discussion?

BTW I think we want a travis option to disable this thing as well, like -DTRAVIS_DISABLE_SERIALIZATION_TESTS

Great start indeed, this is exactly what I had in mind. Want to send a PR for further discussion?
Yeah, which name and folder are more appropriate for the template?

I would put it as tests/unit/base/trained_model_serialization_unittest.cc.jinja2

btw i'm not sure if we wanna have this at all part of the unit testing framework....

Yeah I was thinking that, but then I decided I think it should. Because this should work for all models. Dataset should be tiny though...

These are the classes that are tested so far: CLinearMachine, CKernelMachine, CLinearMulticlassMachine, CKernelMulticlassMachine, CNativeMulticlassMachine

Yeah in the python script instead of calling jinja we could generate an header that lists all the machines  and then use a typed test, it should be doable.
Missing: for sure we talked about GPs.

Tree Machines as well (they dont even serialize atm)


Fix missing homepage placeholder while waiting for livestream data

Issue
When loading the homepage, I kept seeing a blank page with footer for a while. Even Google pageSpeed has picked that up:

Ticket
Closes #572 Slow appearance of tiles
Notes
The tiles were originally delayed to prevent the "Upcoming Livestream" section from causing layout shifts.
With the new theme, the footer would cause a visible shift anyway.  Hiding the footer is one solution, but I think not showing anything until the fetch is done is bad because the livestream fetch could stall.
Also, even with the delay, there is some visible shift anyways as shown in the second video:
https://user-images.githubusercontent.com/64950861/154492452-a2336650-d494-4e44-918a-d5ec23c3776c.mp4
https://user-images.githubusercontent.com/64950861/154492464-97edf3ef-b926-4364-a3f6-0e00b5e32dfb.mp4
Known issues
There is a double-render of the livestream section, but it's irrelevant to this change, and there is a separate issue for it.

Yea, I've noticed this too and meant to raise it. The loading bar would take a while to show up before having a blankish looking page. Pushing now...


timeout if no one connects

Hi,
ftp.py are pretty useful for me. I just want "import ftp" a few seconds in my script and if no one connect it breaks and go on further..
is it possible somewhere in code to define a timeout ?
thanks
regards

That should be possible. In line 128, ftpsocket timeout is set to None. In line 152 ftpsocket waits for a connection. If a timeout is set, that call will give a timeout error on expiry, which will end the program. To make that ending gracefully, you have to add an except clause before line 324, so the end of the code should then look like:
    except Exception as e:
        print(e)
     finally:
        datasocket.close()
        ftpsocket.close()
        if dataclient is not None:
            dataclient.close()

You could also make the timeout optional by making it an argument to the function ftpserver() with None as default value.

I have updated the repository with the changes I mentioned.

oh you are nice. Thank you.. i have already use it and it works like a charm! Thanks!

You're welcome.


Possible Problems When Waking with a Stale Waker

consider these events:
- spawn
- poll; waker is saved elsewhere
- wake_by_ref with saved waker
- poll; waker is ignored
- wake_by_ref with saved waker

the second wake_by_ref will be dropped on the floor. Is this a problem? Can real futures do this? Is this supported by existing runtimes?

The fundamental rule of implementing a future is "never return Pending without saving the waker". If the second poll returns Pending, then the latest waker must have been saved. An out-of-date waker is necessarily one that the future is no longer interested in, since it didn't poll it since (potentially) acquiring a new waker.


IOS pod installation failing due to dependency conflict

Pod installation failing with the below error when used with cloud_firestore 5.4.4
CocoaPods could not find compatible versions for pod "nanopb":
  In Podfile:
    cloud_firestore (from `.symlinks/plugins/cloud_firestore/ios`) was resolved to 5.4.0, which depends on
      Firebase/Firestore (= 11.0.0) was resolved to 11.0.0, which depends on
        FirebaseFirestore (~> 11.0.0) was resolved to 11.0.0, which depends on
          FirebaseFirestoreInternal (= 11.0.0) was resolved to 11.0.0, which depends on
            nanopb (~> 3.30910.0)

    google_mlkit_face_detection (from `.symlinks/plugins/google_mlkit_face_detection/ios`) was resolved to 0.11.0, which depends on
      GoogleMLKit/FaceDetection (~> 6.0.0) was resolved to 6.0.0, which depends on
        GoogleMLKit/MLKitCore (= 6.0.0) was resolved to 6.0.0, which depends on
          MLKitCommon (~> 11.0.0) was resolved to 11.0.0, which depends on
            GoogleDataTransport (< 10.0, >= 9.4.1) was resolved to 9.4.1, which depends on
              nanopb (< 2.30911.0, >= 2.30908.0)

Fixed with overriding the google_mlkit_face_detection dependency in pubspec.yml with latest version as follows,
dependency_overrides:
  google_mlkit_face_detection: 0.12.0

I'm further testing with this fix, I will post here If there any issue with this. Meanwhile please update the google_mlkit_face_detection dependency version in package to latest.

I'm having a similar issue when running firebase_messaging: ^15.1.3 and firebase_core: ^3.6.0.
firebase_messaging depends on GoogleDataTransport (~> 10.0).
google_mlkit_face_detection depends on GoogleDataTransport (< 10.0, >= 9.4.1).
It seems to be necessary to update google_mlkit_face_detection to the latest version.


Added functionality to make rendered fields uneditable in EE

added the editable overload functionality that was in TextFields to the .Render() methods of other field types

Dig it. Thanks!


Types not exported correctly when importing Hono routes

What version of Hono are you using?
3.10.1
What runtime/platform is your app running on?
Vercel NextJS
What steps can reproduce the bug?
Create an index.ts file with the following:
import { Hono } from 'hono'
import usersAppImported from './user'

export const usersApp = new Hono()
  .get('/', (c) => c.jsonT({ result: 'list authors' }))
  .get('/:userId', (c) => c.jsonT({ result: 'list authors' }))

export const app = new Hono().route('/users', usersApp)

export const app2 = new Hono().route('/users2', usersAppImported)

And a separate file user.ts:
import { Hono } from 'hono'

const usersApp = new Hono()
  .get('/', (c) => c.jsonT({ result: 'list authors' }))
  .get('/:userId', (c) => c.jsonT({ result: 'list authors' }))

export default usersApp

Run tsc index.ts --declaration --emitDeclarationOnly to generate type declarations.
What is the expected behavior?
Both apps should have the complete types generated.
What do you see instead?
The Hono with an imported app has not the complete types:
export declare const app: import("hono/hono-base").HonoBase<import("hono").Env, import("hono/types").MergeSchemaPath<import("hono").ToSchema<"get", "/", unknown, {
    result: string;
}> & import("hono").ToSchema<"get", "/:userId", unknown, {
    result: string;
}>, "/users">, "/">;

export declare const app2: import("hono/hono-base").HonoBase<import("hono").Env, import("hono/types").MergeSchemaPath<import("hono").Schema, "/users2">, "/">;

This causes problems when using a Hono RPC client in a monorepo.
Additional information
I've tried various things to make it work, but I belive this some sort of bug with the types in Hono?

Hi @fredrik-sogaard
Are these expected types?
import { Hono } from 'hono';
export declare const usersApp: Hono<import("hono").Env, import("hono").ToSchema<"get", "/", unknown, {
    result: string;
}> & import("hono").ToSchema<"get", "/:userId", unknown, {
    result: string;
}>, "/">;
export declare const app: Hono<import("hono").Env, import("hono/types").MergeSchemaPath<import("hono").ToSchema<"get", "/", unknown, {
    result: string;
}> & import("hono").ToSchema<"get", "/:userId", unknown, {
    result: string;
}>, "/users">, "/">;
export declare const app2: Hono<import("hono").Env, import("hono/types").MergeSchemaPath<import("hono").ToSchema<"get", "/", unknown, {
    result: string;
}> & import("hono").ToSchema<"get", "/:userId", unknown, {
    result: string;
}>, "/users2">, "/">;

Yes, those seem correct. Was that your output when running the command?

This seems like the problem we're seeing: https://github.com/orgs/honojs/discussions/1509
Could it be a problem with our Typescript configuration?

I figured this out. If you import with path aliases in NextJS it doesn't work:
import usersAppImported from '@/users' // <-- Does not produce correct types
import usersAppImported from './users'  // <-- Produces correct types

Default tsconfig.json:
"paths": {
  "@/*": [
    "./*"
  ]
 }


G2 G3 offsets and angle calculations

Idk if its specific to my case, but here laser machine performs offset calculations from 0.0 point, in short I and J are coordinates for arc center. Because of that lib cant really draw those arcs properly. From what i saw in code, center is calculated from last point coordinates + I J. Is there's a way to fix it? Maybe set a point from which offsets will be set? Or this library cant really do it?Also its kinda awkward, but im forced to substract end point coordinates from offsets, or otherwise its even worse, like circle which is supposed to be like 10R becomes like 100R because of offsets calculated from last point, but i guess its because of G90 present in gcode
(white part of image is a mesh that im drawing myself going through same gcode file)
Also angle calculations are kinda off a little too, here's my code for angles:
    let arcCurrentAngle = Math.atan2(initialY - centerY, initialX - centerX);
    if (arcCurrentAngle < 0) {
      arcCurrentAngle += 2 * Math.PI;
    }
    let finalTheta = Math.atan2(yCoord - centerY, xCoord - centerX);
    if (finalTheta < 0) {
      finalTheta += 2 * Math.PI;
    }

The rest of three js meshes are drawn similarly to this lib segments code
this is gcode from file:
N10 G30 X0 Y0
N20 G31 X24.0 Y24.0
N30 G45 R0
N40 G71
N50 G90
N60 G0G41X0.637Y2.372
N70 M13(PART1)
N80 M10(2,4,1)
N90 G1 X2. Y7.183
N100 G1 Y16.
N110 G2 X8. Y22. I8. J16.
N120 G1 X16.
N130 G2 X22. Y16. I16. J16.
N140 G1 Y9.
N150 G2 X20. Y7. I20. J9.
N160 G1 X19.
N170 G2 X17. Y9. I19. J9.
N180 G1 Y15.
N190 G3 X15. Y17. I15. J15.
N200 G1 X9.
N210 G3 X7. Y15. I9. J15.
N220 G1 Y2.
N230 G1 X2.
N240 G1 Y6.533
N250 M11(2,1)
N260 G0G40
M30(1,95.8,42.5)

Hi @VladimirSmar, absolute arc offsets are not supported as they are more of an exception (see cnccookbook) but if you are willing to create a PR for it, I'll review it and help you to get it merged.
A global option could be a way to toggle the behaviour between absolute vs relative arc offsets, something like: absoluteArcOffsets = true.

Just to be on the same page, I had to modify the gcode above as follows to make it work at all (turning on 'travel'):
G30 X0 Y0
G31 X24.0 Y24.0
G45 R0
G71
G90
G0
G41 X0.637 Y2.372
G1 X2. Y7.183
G1 Y16
G2 X8 Y22 I8 J16
G1 X16
G2 X22 Y16 I16 J16
G1 Y9
G2 X20 Y7 I20 J9
G1 X19
G2 X17 Y9 I19 J9
G1 Y15
G3 X15 Y17 I15 J15
G1 X9
G3 X7 Y15 I9 J15
G1 Y2
G1 X2
G1 Y6.533
G0 G40

which results in:

Is that what you are getting?

I see you're also using a non-zero cutter compensation via G41 (line 7 in my snippet: G41 X0.637 Y2.372). This is also not yet supported and I again I'd welcome PR's for that.
To explain: I create the lib for use for 3D printing but since them some support for CNC's has been added. The thing is however that there are many gcode variants for CNC's and that makes testing for all use cases almost impossible. But if you really want to use my lib and make a PR,  I'll help you to get it merged.
btw, what machine type are you using?

Im not really sure which type im using tbh) its just a laser cut machine, but i dont know any specific about it.

Also, as you can see, even with those travel codes straight lines are kinda scuffed and cut a little. I solved this problem exacly with angles.

Closing this for now due to not enough info atm.
Anybody is free to reopen this ticket and I that case I'd like a clear test case we can start with.


error 0x8007003

Had the same experience as others earlier  with the April update
BUT Running wslconfig /u Arch:
Unregistering...
Error: 0x80070003   <- still there!
Then it Worked!

Ok, not an issue anymore, still some might learn, thank you, Throstur
regards /pjkasper
On Fri, Jul 13, 2018 at 12:25 PM, Throstur<EMAIL_ADDRESS>wrote:

I got the same error after moving Arch.exe after installing. After this,
moving it back seemed to resolve the issue.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/yuk7/ArchWSL/issues/33#issuecomment-404793295, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ACMo9aT-GViUZXPYz6DCIxEvohJeoCVOks5uGHWvgaJpZM4VNfdw
.

I still encounter this error after 'unregistering' arch multiple times. Please tell me if there's a solution to that.


Use black for font outlines by default instead of white

This makes font outlines more usable out of the box, as black is one of the most commonly used colors for font outlines.
Now, to configure a working font outline, you just have to increase Outline Size in a supported Control node's custom constants:

White outlines are actually useful when combined with outline modulate. Question is whether black is more common that all other colors 🤔

White outlines are actually useful when combined with outline modulate. Question is whether black is more common that all other colors thinking

Should we make outline modulate black by default instead?

Should we make outline modulate black by default instead (and keep the outline glyphs white)?

Let's do that!

I think this is something we can change in 4.x now.

A whole year has passed and it seems like this was extremely desirable back when. How do we feel about this now?

Rebased and tested again, it works as expected.

Should we make outline modulate black by default instead (and keep the outline glyphs white)?

I don't remember what this was about though. outline_modulate is only referenced in Window and Label3D (the latter is already solid black by default).

outline_modulate was a theme color. It was used to modulate the outline color defined in Font (which this PR modified originally I think?), but the Font outline is now gone.

There are some style issues, clang-format didn't run.

Thanks!


Review Design Patterns

DPs:
https://github.com/monarch-initiative/mondo/tree/master/src/patterns
All should be correct, but some improvements required:

better inline documentation and examples
stricter constraints (e.g replace owl:Thing with 'disease')

We're working on updating DPs, I think this ticket can be closed.


Add missing deasync dependency to 11ty plugin package.json

Was trying to use the 11ty plugin and the deasync dependency used was not listed in its package.json. This adds it.

Dang, thanks!


Update for<EMAIL_ADDRESS>

Why: updates documentation for recent release of gradle/actions/setup-gradle
Closes:

What's being changed (if available, include any code snippets, screenshots, or gifs):

Switches from using gradle/gradle-build-action to gradle/actions/setup-gradle. The latter is a replacement for the former, and gradle-build-action will be deprecated.
Update with latest content from gradle.yml starter workflow. The docs reference this starter workflow, but the content has diverged.
Remove use of wrapper-validation-action, as this will be rolled into the setup-gradle action. This makes the examples simpler.
Bump version hashes for the latest action release.

Check off the following:

[ ] I have reviewed my changes in staging, available via the View deployment link in this PR's timeline (this link will be available after opening the PR).

For content changes, you will also see an automatically generated comment with links directly to pages you've modified. The comment won't appear if your PR only edits files in the data directory.

[ ] For content changes, I have completed the self-review checklist.

@jonjanego This PR updates the GH docs to match the recent updates to starter workflows.

@bigdaz Thanks for opening this PR! I'll get this triaged for review ✨


High private repo bandwidth usage / Ignoring Ignore rule.

Is there an existing issue for this?

[X] I have searched the existing issues

Package ecosystem
npm
Package manager version
yarn 1.22.19
Language version
No response
Manifest location and content before the Dependabot update
No response
dependabot.yml content
version: 2
updates:

package-ecosystem: "github-actions"
open-pull-requests-limit: 20
target-branch: "development_anthony"
directory: "/"
schedule:
interval: "daily"
time: "08:00"
timezone: "Europe/Athens"

package-ecosystem: "npm"
open-pull-requests-limit: 20
directory: "/"
target-branch: "development_anthony"
schedule:
interval: "daily"
time: "08:00"
timezone: "Europe/Athens"
ignore:

dependency-name: "@fortawesome/fontawesome-pro"

package-ecosystem: "bundler"
open-pull-requests-limit: 20
directory: "/"
target-branch: "development_anthony"
schedule:
interval: "daily"
time: "08:00"
timezone: "Europe/Athens"

Updated dependency
No response
What you expected to see, versus what you actually saw
I am a Font Awesome pro subscriber, and I have Font Awesome ignored in dependabot.
There is a bandwidth limit now from Font Awesome. I believe dependabot is using bandwidth.
We requested the list of IPs that used bandwidth and saw that most of them (3/4) were from Dependabot.
---> <IP_ADDRESS>
---> <IP_ADDRESS>
---> <IP_ADDRESS>
X.X.X.X
Included are 3 IP addresses from Github which appear to be linked to GitHub's Security Updates (source: https://github.com/dependabot/dependabot-core/issues/2230#issuecomment-759459133).
On our build server and local development computers, the packages are cached and only redownloaded when there is an update
I see in the update logs of dependabot
...
updater | 2024/03/08 06:29:18 INFO <job_797517549> All versions of @fortawesome/fontawesome-pro ignored, no update allowed
...
proxy | 2024/03/08 06:29:38 [251] GET<EMAIL_ADDRESS>...
Native package manager behavior
No response
Images of the diff or a link to the PR, issue, or logs
No response
Smallest manifest that reproduces the issue
version: 2
updates:

package-ecosystem: "npm"
ignore:

dependency-name: "@fortawesome/fontawesome-pro"

I'm one of the engineers at Font Awesome. I can help out with logs from the other side if they are useful. But I can't see currently how this would be related to our service provider (Cloudsmith) or anything to do with our package contents.

This issue seems related to https://github.com/dependabot/dependabot-core/issues/9196

This is likely a side-effect of the way Dependabot leans into using native package managers. Most native package managers don't natively implement ignores. So instead ignores are implemented by Dependabot as a post-processing filter, after the native package manager does its thing.
So my guess is that the native package manager is making these calls.
However, we can't really switch to a pre-processing filter because it gets really tricky--many times the dep is intertwined with other deps, or the native package manager will yell at us if we try to munge the file.
Your best bet here is to get the native package manager to implement ignores natively, and then we could possibly pass those ignores into it to use internally. I say possibly because there's a lot of complexity around ignores, and figuring out a clean API to pass ignores into the native package manager can be very tricky. But some package managers are working on it, and if upstream is interested, we're very open to having a discussion with them.
Sorry there's not much we can do here.


Error: ENOTDIR

Hey there :)
I want to get some PDFs from a server, merge them and then save the result to the server again.
As the library for merging the files doesn't support input streams, I have to save the files first. So I thought I'd do something like this:
var readStream = share.airwayBills.findOne().createReadStream();

temp.open('pdfgen', function(err, info) {
  var writeStream = fs.createWriteStream(info.path + '/out.pdf');

  readStream.pipe(writeStream);

  writeStream.on('finish', function() {
    console.log('Finished');
    temp.cleanupSync();
  });
});

When not using temp here the code works, but using it I get this error:

Error: ENOTDIR, open '/var/folders/6y/vpzbkb0j0r366j_98dn42d380000gn/T/pdfgen11567-4723-am1ga0/out.pdf'

When I go into /var/folders/6y/vpzbkb0j0r366j_98dn42d380000gn/T, pdfgen11567-4723-am1ga0 is not a folder, but an empty file. I think that a) I should get a directory, not a file and that b) it should be deleted using cleanupSync() (might just not get called here).
What am I doing wrong here? (I'm on a Mac btw)
Thanks!

Reading helps, temp.mkdir works here. Thanks for this library!


Example 2: Create BITS transfer jobs that download multiple files, precision

If the contents of the file do not have a header (that is, the first line composed as follows: Source, Destination):
Import-CSV -Path filelist.txt -Header Source ,Destination  |  Start-BitsTransfer

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

ID: f2562033-cf25-0447-81e1-397a973c5c49
Version Independent ID: 63016f51-2ccf-fdb0-5e29-16418508ce64
Content: Start-BitsTransfer (bitstransfer)
Content Source: docset/windows/bitstransfer/start-bitstransfer.md
Product: w10
GitHub Login: @coreyp-at-msft
Microsoft Alias: coreyp

@officedocsbot assign @yogkumgit

hello @4lex11s
Thanks for your comments.
I created a PR to improve the article based in your comments.
Thank you

@get-itips  Thank you very much for the contribution and updating the documentation with PR. @4lex11s Hope this update is helpful for you. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.

@officedocsbot close


Command-line not working on windows 10

Hi, I create a bunch of topshelf services and the command-line always work fine, but since I instal windows 10 in my machine I couldn't install any service.
I can debug as console but even run from .exe is not working.
Yesterday I try to install the same services in windows 8 and today in windows server 2012 and the install is working fine.

We'll need a little more information to be able help here. if you have logs of the failure or something then that would be helpful. I've ran services on Windows 10, so it's not specifically something Windows 10 related to Topshelf.


Feature/associations

Associations

In this branch, I have made the following changes:

Add classroom file destined to harbor the content of the classroom class.
Implement classroom class with a variable called label that has getter and setter attributes.
Implement has-many/belongs relationship between Classroom and Student class.
Add book file destined to harbor the content of the book class.
Implement book class with two variables called title and author that has getter and setter attributes.
Add rental file destined to harbor the content of the rental class.
Implement rental class with a variable called date that has getter and setter attributes.
Implement many-to-many/has-many-through relationship between Person and Book class using the intermediate Rental Class.

Thank you @mmsesay for your kindly feedback. :1st_place_medal:  :+1:


GEODE-4767: Lucene stat assert modification

* Lucene reindexing thread may be ahead of the put operation by the test
* The asyc event listener will still process the event.

Thank you for submitting a contribution to Apache Geode.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[ ] Is there a JIRA ticket associated with this PR? Is it referenced in the commit message?

[ ] Has your PR been rebased against the latest commit within the target branch (typically develop)?

[ ] Is your initial contribution a single, squashed commit?

[ ] Does gradlew build run cleanly?

[ ] Have you written or updated unit tests to verify your changes?

[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and
submit an update to your PR as soon as possible. If you need help, please send an
email to<EMAIL_ADDRESS>

precheckin was all good ... travis is assumed to have failed because of resource constraint.


Add support for status filter on list tasks.

This is documented in the API v2 docs -> https://support.crowdin.com/api/v2/#operation/api.projects.tasks.getMany

Hi @dpaskov, thanks a lot for your contribution! :tada:

Codecov Report

Merging #19 (b4620d0) into main (e3be5f5) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff            @@
##             main      #19    +/-   ##
========================================
  Coverage   98.83%   98.83%
========================================
  Files          60       60
  Lines        1616     1616
  Branches       65      195   +130
========================================
  Hits         1597     1597
  Misses         13       13
  Partials        6        6

Impacted Files
Coverage Δ

crowdin_api/api_resources/tasks/resource.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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

Hi @dpaskov, thanks a lot for your contribution! 🎉

Thank you for the quick response. What is the usual schedule for release after merging?

@dpaskov just released a new version. Thanks!


Remove .NetApp 2.1 framework in Adaptive.Tests project file

In this PR, the dotnetApp 2.1 framework has been removed.
Thus, it would only generate dotnetcore 3.1 app in bin folder and only run the tests under dotnetcore 3.1 framework.

#minor

Pull Request Test Coverage Report for Build 257315

0 of 0   changed or added relevant lines in 0 files are covered.
3 unchanged lines in 3 files lost coverage.
Overall coverage decreased (-0.003%) to 76.144%

Files with Coverage Reduction
New Missed Lines
%

/libraries/AdaptiveExpressions/BuiltinFunctions/GetNextViableTime.cs
1
90.91%

/libraries/AdaptiveExpressions/BuiltinFunctions/GetPreviousViableTime.cs
1
90.91%

/libraries/Microsoft.Bot.Builder.Dialogs.Adaptive/Generators/MultiLanguageGeneratorBase.cs
1
81.48%

Totals

Change from base Build 256939:
-0.003%

Covered Lines:
23112

Relevant Lines:
30353

💛  - Coveralls


OClassTrigger doesn't take into account multiple superclasses

OrientDB supports multi super-classes.
But code of OClassTrigger was not changed to support that. Snippet:
while (fieldName == null || fieldName.length() == 0) {
        if (superClz == null || superClz.getName().equals(CLASSNAME))
          break;
        fieldName = superClz.getCustom(attr);
        superClz = superClz.getSuperClass();
      }

Btw, it will be helpfull to have some utility class for caching results of culculations till next schema update.
For example OClassTrigger is trying to find proper field on every call instead of caching previously obtained results obtained from schema in a cache.

commented on the pull request.


6/11 PM Publish

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

File
Status
Preview URL
Details

docfx.json
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel