[Access Request] Report 7390 - Whse. Calculate Inventory

Hi Al team,
I would like to access the following variable from the extension object
protected var
        HideValidationDialog: Boolean;
        // NEW >>>>
        WhseJnlLine: Record "Warehouse Journal Line";
        NextLineNo: Integer;
        RegisteringDate: Date;
        Bin: Record Bin;
        Location: Record Location;
        WhseJnlBatch: Record "Warehouse Journal Batch";
        PhysInvtCountCode: Code[10];
        CycleSourceType: Option " ",Item,SKU;
        NextDocNo: Code[20];
        // NEW <<<<

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Bug: "Pasted Image" Pattern not working.

When this toggle is off, the plugin is supposed to work with images that match "Pasted Image" if I understand this correctly? For me that is not the case. I paste an Image with the name Pasted Image 123.png into the editor and nothing happens. Only with the "Handle all images" setting enabled, does the plugin work for me.

Thanks for reporting, this is something that I didn't make clear. The name pattern should be "Pasted image" not "Pasted Image", I'll update the description in the next release.
Another thing worth mentioning is that pasting an image from a file behaves differently in macOS and Windows.
In macOS, both pasting from the clipboard and pasting from a file are treated as creating a new image from the clipboard by Obsidian, thus the image is always named as "Pasted image 2022...". But in Windows, which I suppose you are using, pasting from a file is like a regular file transfer, the original name is kept other being created as "Pasted image 2022..." by Obsidian. I'll add some explanation about this in FAQ part of readme.

You can try copying an image by right-click an image on the web and selecting "Copy image", or using Windows Snipping Tool to clip part of the screen, then paste it in Obsidian with "Handle all images" unchecked, and see if that works. If not, it's definitely a bug I need to take a deeper look at.

FAQ added, check if that makes it clear.

ah, I see, thanks! I didn't realize that the Pasted image pattern is case sensitive. (I am using my own screenshot setup, where I can control the name of the output.)
Thanks for the quick response!


Integrate with tui-rs-tree-wdget

Problem
With this crate's recent release, I think it would be nice to have some more general-purpose widgets added to the current repertoire. tui-rs-tree-widget is one such widget that I think this crate would benefit from having. As discussed in this issue, both the maintainer and I are interested in this merge.
Solution
Add the Tree widget from tui-rs-tree-widget into this repository.

Ay, I've actually used tui-tree-widget before ! It's awesome, worked like a charm !
Since we recently release our first version of ratatui, we are still in the process of evaluating where we want to go next.  With that in mind, we have created a discussion and are currently in the process of planning a meeting to discuss this further.
If you're able to make it, we would love to have you join in order to discuss this issue further. We have a poll running to figure out the time slot here and you can find more information on our discord.
All in all, I'm personally in favor of adding this in, we just need to make sure we do it in a coherent manner.

Could you create a PR for this ? We're still figuring stuff out organization wise (hence the long follow up time), but there's interest for this widget and we are now working on merging another external widget so now is the good time ! 🚀

Okay, I can work on this when I have more time cleared up. Alternatively, @EdJoPaTo (the maintainer) could work on this if they have the time.

Alright sounds good !

After giving this a lot of thought, I think it would be better to keep this out of the core Ratatui library. I've just started a new ratatui-widgets crate that will have a lot of widgets, with a goal of consistent UI, event handling etc. The treeview is a goal there, and I think there's a lot of this widget that would influence that one. Once we've worked out the common ways to display widgets and handle user interaction, I'd definitely want to start working on the tree view there instead of here.
Does this make sense?

Definitely makes sense to me!


feat: list public codemods

Can you adapt your code after rebasing on the main branch? No changes are shown in the file explorer view.
https://user-images.githubusercontent.com/32841130/234552402-d7475936-aff2-49fb-a85c-b7549dccf553.mov


Update the interval for Terraform's drift detection

What this PR does / why we need it:
For the performance, this changes the interval of Terraform's drift detection.
This param should be mutable through piped config in the end.
Which issue(s) this PR fixes:
Fixes #
Does this PR introduce a user-facing change?:
NONE

This param should be mutable through piped config in the end.

Just to confirm, this change will be addressed by a separated PR, right?

@khanhtc1202
Right, this is for the perfomance check!


Change License to CC

I think the main repository's license should be changed to CC-BY instead of MIT.  Allows more compatibility with licenses from other apps that will use the theme.

Most of the CC licenses have good compatibility when remixing work under different licenses. So, if you are sure to change the license then do it. But you can't change the license for the current release, you can change it for future releases.
If you want to change it then try this once.

you can't change the license for the current release, you can change it for future releases

Thank you @girijakar, that was my understanding as well.  I'm going to release the current version as 0.1 with the existing license and then do a PR for a new license for future versions.

You're welcome.
Let me know if I can help you with anything else.
P.S. I am a newbie, still learning stuff. So, I can help you with some minor changes related to documentation or something like that. If there are some issues with the code, I'll try my best to resolve those.


Refactor fn eval cmd to avoid using function annotation

Current implementation for kpt fn eval uses annotation config.kubernetes.io/function to pass the function specs to the fn runner. The implementation is inherited from kyaml. Although this is used purely internally we should refactor to get rid of it.
This should not be prioritized and we can do it after adding fn eval e2e tests.

@Shell32-Natsu please add to the project


Hot reload does not work automatically. Have to manually run it

When running IntelliJ or Atom, the file changes are not reflected till I manually hit the hot reload icon in IntelliJ or the Reload button in Atom.
Even running flutter --run has the same behavior, I have to press R and then hot reload works.
Looks like the file watcher isn't working? B
Steps to Reproduce
Logs
This log may be a red herring as I don't thinks related to the bug, but Atom, IntelliJ and flutter run all cause this to show up.
Starting device daemon...
Launching loader on iPhone 6...
CoreSimulatorBridge: LaunchServices: no bundle found with identifier com.apple.MobileSMS
com.apple.CoreSimulator.SimDevice.AB99FA55-DC63-4011-AF53-40AAC2C798F2.launchd_sim[5233] (UIKitApplication:com.yourcompany.MyFirstFlutterApp[0x3c01]: Service exited due to signal: Terminated: 15
CoreSimulatorBridge: Requesting launch of com.yourcompany.MyFirstFlutterApp with options: {
        arguments =     (
            "--flx=/Users/FaisalAbid/Development/workspace/MyFirstFlutterApp/build/app.flx",
            "--dart-main=/Users/FaisalAbid/Development/workspace/Verve/Elixer/flutter/packages/flutter/bin/loader/loader_app.dart",
            "--packages=/Users/FaisalAbid/Development/workspace/MyFirstFlutterApp/.packages",
            "--enable-checked-mode",
            "--start-paused",

            "--observatory-port=8100"
        );
        environment =     {
        };
    }
SpringBoard: Application 'UIKitApplication:com.yourcompany.MyFirstFlutterApp[0x3c01]' exited abnormally via signal.
assertion failed: 16B2548a 12H141: libxpc.dylib + 59338 [AF8FA08F-353B-303B-A08E-AEBB3C60A640]: 0x7d
SpringBoard: [MPUSystemMediaControls] Updating supported commands for now playing application.

Flutter Doctor
[✓] Flutter (on Mac OS, channel master)
    • Flutter at /Users/FaisalAbid/Development/workspace/Verve/Elixer/flutter
    • Framework revision 949e27ae8f (16 hours ago), 2016-10-26 19:11:55
    • Engine revision 7e5d63513a
    • Tools Dart version 1.21.0-dev.0.0

[✓] Android toolchain - develop for Android devices (Android SDK 24.0.3)
    • Android SDK at /Users/FaisalAbid/Library/Android/sdk/
    • Platform android-N, build-tools 24.0.3
    • Java(TM) SE Runtime Environment (build 1.8.0_91-b14)

[✓] iOS toolchain - develop for iOS devices (Xcode 8.0)
    • XCode at /Applications/Xcode.app/Contents/Developer
    • Xcode 8.0, Build version 8A218a

[✓] Atom - a lightweight development environment for Flutter
    • flutter plugin version 0.2.6
    • dartlang plugin version 0.6.42

[✓] IntelliJ IDEA Community Edition (version 2016.2.5)
    • Dart plugin installed
    • Flutter plugin installed

[✓] Connected devices
    • iPhone 6 • AB99FA55-DC63-4011-AF53-40AAC2C798F2 • ios (simulator)

This is currently intended behavior.  There is no watcher.  That said, I think having reload-on-save wired up could be a useful feature.  I'll retitle the bug.

Oh I see! From the demos I thought it was done automatically on save. That's cool, I can hook up a keyboard shortcut to it.
Thanks!

@FaisalAbid, to add a custom keyboard shortcut for hot reload in IntelliJ:

go to preferences
select Keymap in the menu
enter 'flutter' in the search field
right-click on 'Flutter Hot Reload' and select 'Add Keyboard Shortcut'

Yeah, I re-wired mine to command-R (hot reload) and command-shift-R (hot restart) due to muscle-memory from using Atom for so long. :)  I believe there has been discussion in other bugs of adding some non-conflicting default keybindings for these over in https://github.com/flutter/flutter-intellij/issues.

I believe there has been discussion in other bugs of adding some non-conflicting default keybindings for these

Yup: https://github.com/flutter/flutter-intellij/issues/335

Does this require anything on the Flutter side, or is it just an IntelliJ issue?

Closing since this is working as intended. Feel free to file a bug if you want a specific change, describing the change you would like.


WIP: Provide OCSP verification endpoint and persist certs to disk

Depends on https://github.com/cloudflare/cfssl/pull/309

@mediocregopher thanks! I just updated and fixed your comments. I also added /list and /revoke.

In main.go you have a lot of repeated code in the handlers. I think quite a bit of it could be generalized into some middleware, specifically the readAndVerifyBody and verifyTimestamp stuff.
It's a bit tricky because multiple things need to use the raw body data, I would suggest first passing all requests through a middleware which reads the body into a buffer (using an http.MaxBytesReader) and replaces the Body field on the request with that buffer, so future middleware and handlers can all use it.

Okay so I fixed all the code comments you already made and I'll work on the middleware next week at some point.

Okay I need to merge this for the new server so I'll make a todo to add the middleware next week. I won't cut a new release though until its done.


[Doc Issue][Meeting API reference]

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/apps-in-teams-meetings/API-references.md
94
12
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Docs Build status updates of commit 3757ad2:
:warning: Validation status: warnings

File
Status
Preview URL
Details

msteams-platform/apps-in-teams-meetings/design/designing-apps-in-meetings.md
:warning:Warning
View
Details

msteams-platform/whats-new.md
:warning:Warning
View
Details

msteams-platform/apps-in-teams-meetings/enable-and-configure-your-app-for-teams-meetings.md
:warning:Warning
View
Details

msteams-platform/apps-in-teams-meetings/API-references.md
:white_check_mark:Succeeded
View

msteams-platform/apps-in-teams-meetings/design/designing-apps-in-meetings.md

Line 178, Column 54: [Warning: bookmark-not-found] Cannot find bookmark '#notificationsignal-api' in 'apps-in-teams-meetings/API-references.md'.

msteams-platform/whats-new.md

Line 68, Column 103: [Warning: bookmark-not-found] Cannot find bookmark '#meeting-details-api' in 'apps-in-teams-meetings/API-references.md', did you mean '#get-meeting-details-api'?

msteams-platform/apps-in-teams-meetings/enable-and-configure-your-app-for-teams-meetings.md

Line 117, Column 141: [Warning: bookmark-not-found] Cannot find bookmark '#notificationsignal-api' in 'apps-in-teams-meetings/API-references.md'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
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.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/apps-in-teams-meetings/API-references.md
94
12
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Docs Build status updates of commit 1185382:
:warning: Validation status: warnings

File
Status
Preview URL
Details

msteams-platform/apps-in-teams-meetings/enable-and-configure-your-app-for-teams-meetings.md
:warning:Warning
View
Details

msteams-platform/whats-new.md
:warning:Warning
View
Details

msteams-platform/apps-in-teams-meetings/design/designing-apps-in-meetings.md
:warning:Warning
View
Details

msteams-platform/apps-in-teams-meetings/API-references.md
:white_check_mark:Succeeded
View

msteams-platform/apps-in-teams-meetings/enable-and-configure-your-app-for-teams-meetings.md

Line 117, Column 141: [Warning: bookmark-not-found] Cannot find bookmark '#notificationsignal-api' in 'apps-in-teams-meetings/API-references.md'.

msteams-platform/whats-new.md

Line 68, Column 103: [Warning: bookmark-not-found] Cannot find bookmark '#meeting-details-api' in 'apps-in-teams-meetings/API-references.md', did you mean '#get-meeting-details-api'?

msteams-platform/apps-in-teams-meetings/design/designing-apps-in-meetings.md

Line 178, Column 54: [Warning: bookmark-not-found] Cannot find bookmark '#notificationsignal-api' in 'apps-in-teams-meetings/API-references.md'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
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.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/apps-in-teams-meetings/API-references.md
94
12
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Docs Build status updates of commit 0937920:
:warning: Validation status: warnings

File
Status
Preview URL
Details

msteams-platform/whats-new.md
:warning:Warning
View
Details

msteams-platform/apps-in-teams-meetings/enable-and-configure-your-app-for-teams-meetings.md
:warning:Warning
View
Details

msteams-platform/apps-in-teams-meetings/design/designing-apps-in-meetings.md
:warning:Warning
View
Details

msteams-platform/apps-in-teams-meetings/API-references.md
:white_check_mark:Succeeded
View

msteams-platform/whats-new.md

Line 68, Column 103: [Warning: bookmark-not-found] Cannot find bookmark '#meeting-details-api' in 'apps-in-teams-meetings/API-references.md', did you mean '#get-meeting-details-api'?

msteams-platform/apps-in-teams-meetings/enable-and-configure-your-app-for-teams-meetings.md

Line 117, Column 141: [Warning: bookmark-not-found] Cannot find bookmark '#notificationsignal-api' in 'apps-in-teams-meetings/API-references.md'.

msteams-platform/apps-in-teams-meetings/design/designing-apps-in-meetings.md

Line 178, Column 54: [Warning: bookmark-not-found] Cannot find bookmark '#notificationsignal-api' in 'apps-in-teams-meetings/API-references.md'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
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.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 308e34b:
:warning: Validation status: warnings

File
Status
Preview URL
Details

msteams-platform/whats-new.md
:warning:Warning
View
Details

msteams-platform/apps-in-teams-meetings/design/designing-apps-in-meetings.md
:warning:Warning
View
Details

msteams-platform/apps-in-teams-meetings/enable-and-configure-your-app-for-teams-meetings.md
:warning:Warning
View
Details

msteams-platform/apps-in-teams-meetings/API-references.md
:white_check_mark:Succeeded
View

msteams-platform/whats-new.md

Line 68, Column 103: [Warning: bookmark-not-found] Cannot find bookmark '#meeting-details-api' in 'apps-in-teams-meetings/API-references.md', did you mean '#get-meeting-details-api'?

msteams-platform/apps-in-teams-meetings/design/designing-apps-in-meetings.md

Line 178, Column 54: [Warning: bookmark-not-found] Cannot find bookmark '#notificationsignal-api' in 'apps-in-teams-meetings/API-references.md'.

msteams-platform/apps-in-teams-meetings/enable-and-configure-your-app-for-teams-meetings.md

Line 117, Column 141: [Warning: bookmark-not-found] Cannot find bookmark '#notificationsignal-api' in 'apps-in-teams-meetings/API-references.md'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
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.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 7c4344d:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/apps-in-teams-meetings/API-references.md
:white_check_mark:Succeeded
View

msteams-platform/apps-in-teams-meetings/design/designing-apps-in-meetings.md
:white_check_mark:Succeeded
View

msteams-platform/apps-in-teams-meetings/enable-and-configure-your-app-for-teams-meetings.md
:white_check_mark:Succeeded
View

msteams-platform/whats-new.md
:white_check_mark:Succeeded
View

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

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/apps-in-teams-meetings/API-references.md
95
14
link
:white_check_mark:

msteams-platform/apps-in-teams-meetings/design/designing-apps-in-meetings.md
94
27
link
:white_check_mark:

msteams-platform/apps-in-teams-meetings/enable-and-configure-your-app-for-teams-meetings.md
97
4
link
:white_check_mark:

msteams-platform/whats-new.md
90
37
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Docs Build status updates of commit 7e83df1:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/apps-in-teams-meetings/API-references.md
:white_check_mark:Succeeded
View

msteams-platform/apps-in-teams-meetings/design/designing-apps-in-meetings.md
:white_check_mark:Succeeded
View

msteams-platform/apps-in-teams-meetings/enable-and-configure-your-app-for-teams-meetings.md
:white_check_mark:Succeeded
View

msteams-platform/whats-new.md
:white_check_mark:Succeeded
View

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

The issue has been resolved, closing the PR.


Address Klocwork issue. Fix bug?

I wasn't sure if you really wanted to only use the results[0] for everything but it sure looked like a bug to me and Klockwork was complaining.

Whoops!


Support maxWorkers or runInBand when using 'projects'

🐛 Bug Report
When defining multiple projects via the projects config option, where one project has maxWorkers set (ie. 1), that setting is not respected.
In the case of a mono-repo, assume we have two projects. One which runs parallel, and another which needs to be run sequentially. For this, the latter has the config setting maxWorkers: 1.
When we run jest from the base config, however, the latter runs in parallel, ignoring the maxWorkers: 1 setting.
To Reproduce

Setup a mono-repo with two jest config files
Set one to maxWorkers: 1
Add both config paths to base projects property
Run jest

Expected behavior / Proposal
I propose the following:

Determine global max workers as:

(If present) maxWorkers setting in base config
(Otherwise) Determine the maximum value specified for maxWorkers found in all projects config files

Allow each project to use

(If present) Its own maxWorkers # of workers
(Otherwise) The global maximum # of workers

Effectively, this ensures that maxWorkers behaviour matches that of running jest on an individual project.
envinfo
  System:
    OS: Windows 10 10.0.19041
    CPU: (4) x64 Intel(R) Core(TM) i7-4510U CPU @ 2.00GHz
  Binaries:
    Node: 14.13.1 - C:\Program Files\nodejs\node.EXE
    Yarn: 1.22.4 - ~\AppData\Roaming\npm\yarn.CMD
    npm: 6.14.4 - C:\Program Files\nodejs\npm.CMD
  npmPackages:
    jest: ^26.5.3 => 26.6.2

Can you put up a minimal repro that could be downloaded?

maxWorkers is currently a part of GlobalConfig so this is expected. I'm not sure if Jest should allow projects to limit the number of workers, as this may quickly go out of hand, causing inconsistencies between execution environments. The only scenario I see this could be helpful is limiting to 1 worker, in other words allowing runInBand to be configurable in a project. E.g. for E2E tests running on a device/simulator where parallelization doesn't make sense (unless having multiple devices).
@SimenB @jeysal thoughts?

The only scenario I see this could be helpful is limiting to 1 worker, in other words allowing runInBand to be configurable in a project.

That's a good point. I can think of one other use case, but I think there may be a better proposal to cover that (detail to follow).
I can see it being useful to allocate a smaller fraction of the overall worker threads in some instances.
Say we have that scenario in a very large mono-repo with multiple projects. Several config files contain very long-running tests, and those particular tests hit first. In this scenario, we might want to limit the total threads for the longer running config file so that we can run our faster tests in parallel along-side of them. This way it can fail more quickly if there are problems in the shorter running tests, yet avoid having to run separate calls to jest.
It's maybe not a perfect example, but it should give an idea.
In that scenario, we could specify a max of 3 threads to the long running out of the global 8. This way, we can configure out thread usage.
However...
I think you're right about runInBand. All things considered, I'd amend my proposal to the following:

maxWorkers remains a global setting, allowing you to define max threads for the entire process
Introduce: runInBand on the project-level (This would make several folks happy #6709 #3215)
Introduce: minWorkers on the project-level

minWorkers
Allows configuring the minimum number of threads to be used for a particular project. (This setting would only have effect when run in projects mode)
This would allow giving specific projects the ability to use more of the allotted threads while allowing elasticity when threads are free.
So in our example above, we could configure minWorkers to 5 for our faster tests and set a global maxWorkers to 8.
This would effectively give us a 3 / 5 distribution unless the faster tests finish first, in which case the longer tests will fill up the remaining slots.

Can you put up a minimal repro that could be downloaded?

@thymikee Did you still need this?

@thymikee Did you still need this?

Nah, we're good

This would be amazing!!
Our use case is fullstack testing for Blitz apps. We want client code tests to run in parallel but run server tests in series because they use the same test DB.

This would be amazing!!
Our use case is fullstack testing for Blitz apps. We want client code tests to run in parallel but run server tests in series because they use the same test DB.

Very similar use case here. Want to run all tests with unit tests running in parralel while integration tests run in series.

Considering there's a super simple runner that provides this functionality: https://github.com/gabrieli/jest-serial-runner it shouldn't be too difficult to add this option to the project config.

I share @taylorhoward92's use-case exactly. I thought I had come up with a solution (here), and indeed it does seem to work for me, but apparently I'm just getting lucky :).

I've got a draft PR for this, thought it'll need updating: #10912

maxWorkers is currently a part of GlobalConfig so this is expected. I'm not sure if Jest should allow projects to limit the number of workers, as this may quickly go out of hand, causing inconsistencies between execution environments. The only scenario I see this could be helpful is limiting to 1 worker, in other words allowing runInBand to be configurable in a project. E.g. for E2E tests running on a device/simulator where parallelization doesn't make sense (unless having multiple devices).
@SimenB @jeysal thoughts?

runInBand at the project level is definitely a very welcome compromise. I've just moved a monorepo project to use Jest's projects instead of manually invoking with yarn workspaces foreach -p run test  --runInBand because I want to collect consolidate coverage for the entire workspace. Unfortunately using yarn is ~1.8x - 2x faster than invoking yarn jest which is significant in economic cost and cumulative time

Finally updated my PR to implement this. Would be great to get a review :)

@thymikee Is it possible someone could take a look at the PR for this? It may have been buried as it was first opened a year ago

how about all cli options?  seems ridiculous that this is not all configurable.  why is maxWorkers in the config documentation but doesn't work?

Not stale. There's an open PR for this.

Not stale.

FYI I've closed my PR related to this because I'm no longer interested in maintaining the PR state. I try not to use Jest anymore, partly due to the lack of response to my contribution/interesting in responding to feature requests like this.
If someone else wants to rebase the work on the PR and re-raise feel free.


Texture images not rendered

Following the basic tutorial and using Maven, with "brick.png" placed in src/main/resources/assets/textures and player = Entities.builder().at(300,300).viewFromTexture("brick.png").buildAndAttach(getGameWorld()); replacing the former player assignment.
There is no error or exception thrown and everything renders except the brick.png image.  The game takes keyboard input and the pixel counter UI updates. There just isn't a brick texture visible.
On Windows using the Netbeans IDE.

@HumanWritesCode I seem to remember another user with Netbeans having a similar issue. Can you "refresh" Netbeans in some way or try with a different IDE? I have Netbeans but cannot seem to reproduce the issue

Did you manage to get textures working?


Lock Side table Width

Hi all,
I try to lock mys side with on angular gantt but i can't.
Currently side width auto adjust to content length. I would like to lock the width to a certain value, even if the content is larger.
I tried adding side-width='100px' but i have a strange result.
Can anyone help me?
Thanks a lot

You have to put additional quotes if 100px is defined directly in template.
Hi all,
I try to lock mys side with on angular gantt but i can't.
Currently side width auto adjust to content length. I would like to lock
the width to a certain value, even if the content is larger.
I tried adding side-width='100px' but i have a strange result.
Can anyone help me?
Thanks a lot
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/angular-gantt/angular-gantt/issues/624

Hi Toilal,
Thanks a lot for your answer.
I finally used api.side.setWidth(100), and it seems fine.
Thanks again
Best regards


make Redis namespace configurable

move the REDIS_NAMESPACE constant into Lita::DefaultConfiguration
expose config.robot.redis_namespace and use the new constant's value as its default

Fixes: #228

Not sure why the Travis build failed, but the tests and lint all passed locally.

Thank you!


number not show ,but if I remove the attr filter,number show.

Hi. Could you provide more details? What are the  device and the browser you were testing on?

Closing due to insufficient details provided.


Report: Filter note

When a user enables filtering of records by activity, an appropriate warning should be shown.
Solution
Most likely it should be a message under the table.
The user should be able to clear the filter with a separate button.

Implemented in v1.1.


Question: python way to get list of all supported countries

Hello !
I have a strong interest in this library and I was wondering whether there is a way to get a list of all supported countries ?
Thanks in advance !

Hello @laurent-brisbois! There is a function list_supported_countries() for this purpose. For example:
import holidays
countries = holidays.list_supported_countries()

@KJhellico
Ok thanks and sorry, I didn't see it while browsing the repo !
Perfect 👌 !


Hyperlinks of drawings

Some of our users have added hyperlinks to shapes and text boxes. I want to list and Test-Path those links.
I can list the drawings with the Worksheet.Cells.Drawings collection, but OfficeOpenXml.Drawing.ExcelShape objects don't seem to have an hyperlink/address property. Is there a way to list those hyperlinks ?
Thank you !

Don't know. You can search for EPPlus and see if that supports the property. EPPlus is what is being used to interact with the underlying Excel file. If I get time, I'll google it.


Alphabetize dictionary pre-commit hook in husky not always running?

For some reason, the husky pre-commit hook did not seem to run in this PR: https://github.com/vmware-tanzu/tanzu-dev-portal/pull/844
Do we need to add some instructions somewhere to ensure it is installed and runs successfully before build/commit?

I know our initial thought was that pre-commit hooks aren't ran when using GitHub Desktop, though that doesn't seem to be the case. However, according to this issue it looks like there's a chance that perhaps due to the environment that they get ran, the hook may not have access to everything it needs to run it. Maybe need to add more debugging.

This is lo-pri and doesn't seem to happen very often.


fix: thresh bug when there is RedisRegistryError

When Thresh model load fails  (ModelRegistryError), we are getting stuck in the loop.
The PR introduces a fix

Codecov Report

Merging #164 (45ce3af) into main (1eafb0d) will not change coverage.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##             main     #164   +/-   ##
=======================================
  Coverage   86.23%   86.23%
=======================================
  Files          18       18
  Lines        1119     1119
  Branches      118      118
=======================================
  Hits          965      965
  Misses        130      130
  Partials       24       24

Files Changed
Coverage Δ

numaprom/udf/threshold.py
92.18% <0.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Targeting PHP ^8.0

This post is for planning purposes.  Sometime in the next year or so, PHP 7 will be unsupported, no longer receiving security updates.  It's reasonable that we drop support for PHP 7, only targeting PHP ^8.0, sometime leading up to that date.  PHP 7.4 is the only version that's still receiving security updates, at the moment.  Users that wish to continue using PHP 7.4, can always target a previous version.
The main motivation and reasoning for this, would be to remove support for annotations, entirely.  However, there may be some other refactoring and cleanup that can be done.

Remove all annotation processing and handling logic
Remove annotation cache support, as it's no longer needed
Consider removal of support for MyCLabs\Enum in favor of native Enums.

Maybe it's time to support php ^8.1 only?
Symfony 6.1 is out, and it has no support for php 8.0 .

@bladl why not continue to support PHP 8.0?  We can still support Symfony 6.1 components with PHP 8.0 support as well.  It just means if your target PHP version is PHP 8.0, you won't get Symfony 6.1, but a lower version that we also support.  I'm not aware of any BC issues here.
I do think we could move forward with updates to target a minimum of PHP 8.0 now though.  Security support for PHP 7.4 ends on 28 Nov 2022.

Features available in php 8.1 could greatly increase code readability and reduce potential bugs.  Enums, readonly properties, array_is_list and pure intersection types. According to usage statistics php 8.1 has greater usage than 8.0 for now and in my opinion with time this break even going to grow faster.

@bladl Enums are already implemented and don't require that we drop 8.0 support.  Readonly properties and intersection types are cool, but I don't see any need to implement them requiring an 8.0 drop yet.  Being that this is a widely distributed OSS lib, we have to be more accommodating of user's upgrade restrictions.
I support following PHP's own version support dates:
https://www.php.net/supported-versions.php
Security fix support for 7.4 will end Nov 28th.  At that point, support should be dropped for 7.4.  We should follow the same for all versions, dropping support when security fixes are no longer provided, unless there is a legitimate need to deviate.

It'd be great to get a PR put together to begin cleaning up for the 8.0 min support if someone wants to take the initiative.

@bladl Enums are already implemented and don't require that we drop 8.0 support.

I tell about internal use of enums. For example replacing constants below with enums:

It would be great to expand their usage today.

It'd be great to get a PR put together to begin cleaning up for the 8.0 min support if someone wants to take the initiative.

Will this changed be versioned as major release?

This would be a minor update, yes - 5.1 for instance.

I agree it'd be nice to replace constants with Enums.  However, it doesn't change functionality in any meaningful way.  Therefore, it's not enough to break support for 8.0 at this time, I feel.  I'm in support of being ahead of PHP's dates by some months, but 8.0 is supported for at least another year.

This would be a minor update, yes - 5.1 for instance.

Shouldn't it be a version 6.1? Current version is 6.0 as I guess. But there is no version 6.0 in CHANGELOG.md

Yes, 6.1 - sorry.  The CHANGELOG looks like it didn't get updated on the last release.  Honestly, that file is mostly a duplicate of the release notes.  I'm wondering if we should just remove all the content and note to reference the release notes.
https://github.com/thecodingmachine/graphqlite/releases

Can annotations like #[Field], #[Type], etc.  be abandoned in favor of attributes from now?

We definitely can.  It would obviously be a BC break.  I’m, personally, in favor of doing so and removing Doctrine annotation dependencies.  It’ll probably require many people to refactor that bit.  Luckily Rector has something for refactoring annotations to attributes.  Maybe we could provide instructions on how to automate that refactor?

Excellent.  I probably added that, actually.  We can just reference that in the release notes.

Can I replace Webmozart\Assert\Assert with native assert() for performance and better IDE code suggestion reason?

Can I replace Webmozart\Assert\Assert with native assert() for performance and better IDE code suggestion reason?

Does that Assert lib throw exceptions in a non dev env?  If the result is the same, I don’t see why not.  However, I haven’t looked into it’s intended use.

Does that Assert lib throw exceptions in a non dev env? If the result is the same, I don’t see why not. However, I haven’t looked into it’s intended use.
Okay! I will replace it with condition + exception if that assertion is important for logic.

There are many unnecessary assertions that can be replaced in favor of methods type-safety. So we could remove entire webmozart/assert lib.

Right yea, let’s do it

Can I replace Webmozart\Assert\Assert with native assert() for performance and better IDE code suggestion reason?

Does that Assert lib throw exceptions in a non dev env?  If the result is the same, I don’t see why not.  However, I haven’t looked into it’s intended use.

PHP's assert() wont be executed for non-dev environment.
webmozart/assert checks and assert() expressions are the same from type safety perspective. webmozart/assert provides information in method contract about guarranted checked types in result.
So if there is a motivation to get less dependencies, simpler code and less checks for types in non-dev mode - it is OK to replace checks 🙂

So if there is a motivation to get less dependencies, simpler code and less checks for types in non-dev mode - it is OK to replace checks 🙂

Those assertions mostly validated data passed and used by internals. So, main purpose of those assertions were to catch errors on unit-testing stage.
Assertions that were interacting with external data were thrown with specific message. So I replaced it with constructions
if (!$isValid) throw Exception('msg').
In summary, it should not have drawbacks in non-dev mode.


pico-w code for adafruit-io and micropython

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

-1 of 4 (75.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.5%) to 48.908%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/self_driving_lab_demo/core.py
3
4
75.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.5%

Covered Lines:
99

Relevant Lines:
184

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

3 of 4 (75.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.5%) to 48.908%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/self_driving_lab_demo/core.py
3
4
75.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.5%

Covered Lines:
99

Relevant Lines:
184

💛  - Coveralls


Question: How to get documentation info by operationId.

I want to pick up some ApiDescription by operationId or by method medate on spring mvc controller,or other springfox api.
I'm doing a open API project.
I want to collection user documentation that a appid have access to use

You can write your own controller to do that. See how the Swagger2Controller does it... using the DocumentationCache and ServiceModelToSwagger2Mapper


easyTipViewDidDismiss not called when calling dismiss()

I expected easyTipViewDidDismiss to be called when doing
self.tipView.dismiss()
but it isn't.
I do that when the user taps an overlay.
easyTipViewDidDismiss is called when the user taps the tipView (so the delegate should be correct).
Is it an issue or am I holding it wrong?

Delegation call hasn't been implemented in this case because you have the completion block that gets called after the view has been dismissed:
view.dismiss(withCompletion:{ _ in
     // code executed after the view has been dismissed
})

OK, so I have to call the same code as I do in easyTipViewDidDismiss.
I think it would have been cleaner to always do easyTipViewDidDismiss whenever the view has been dismissed, regardless of how it was dismissed.
It works though.

Thanks for your insight, I'll make this feature available in next release.

fyi version 0.2.1 has this fix applied

sweet!

I'm sorry, but I still can't work out how to perform actions when the user dismisses the tooltip by tapping on it (that seems like the default option, right?).

Hi folks, any update on this?

class TheViewController: UIViewController, EasyTipViewDelegate {
...
tipView = EasyTipView(text: "The text", preferences: preferences, delegate: self)
...
func easyTipViewDidDismiss(_ tipView: EasyTipView) {
...

That's my setup. When I click the tooltip, the function easyTipViewDidDismiss is called.


Refactor assets handling

This PR advances us towards https://github.com/kinvolk/lokomotive/issues/716.
Supersedes #655. Fixes #24.
Summary of changes:

Move all charts under assets/charts (both component charts and control plane charts). Managing all chart assets under a single path makes it easy to organize them and handle chart extraction in a generic way in the code.
Make component paths consistent. Some components had the chart files under manifests/ whereas others had them in the component's root directory.
Ensure a component's dir name matches the component's name. This allows finding component chart files by convention, i.e. using a path such as assets/charts/components/<name>.
Extract control plane charts from Go code instead of using the template_dir Terraform resource. This fixes #24.

@invidian this PR is more or less the same as #655. Until #716 is merged we extract the control plane charts inside the platform implementations and I've added TODOs to remove these parts once we refactor the platforms. I had to do things in this order since the platform refactoring depends on the new assets handling logic.

@invidian I'd like to drive this towards a merge ASAP as this blocks the platform refactoring. Any remaining concerns (which should be handled in this PR)?


Validate issuers email

What this PR does / why we need it:
This PR adds validation for emails set to Issuers. Previously, setting an invalid email address lead to hard-to-find issues, like Certificates not becoming ready, because Issuers are not ready, because Issuer emails are incorrect. This PR makes the apply step fail.
Which issue this PR fixes (use the format fixes #<issue number>(, fixes #<issue_number>, ...) to automatically close the issue when PR gets merged): fixes #
Public facing documentation PR (if applicable)

N/A
Special notes for reviewer:
Checklist:

[NA?] Added relevant notes to WIP-CHANGELOG.md
[X] Proper commit message prefix on all commits
[NA] Updated the public facing documentation
Is this changeset backwards compatible for existing clusters? Applying:

[X] is completely transparent, will not impact the workload in any way.
[ ] requires running a migration script.
[ ] will create noticeable cluster degradation.
E.g. logs or metrics are not being collected or Kubernetes API server
will not be responding while upgrading.
[ ] requires draining and/or replacing nodes.
[ ] will change any APIs.
E.g. removes or changes any CK8S config options or Kubernetes APIs.
[ ] will break the cluster.
I.e. full cluster migration is required.

Is the only requirement that the mail address follows correct specification or does that mail address actaully need to exist?

I'm not sure. But I guess the real problem here is that we want to both help the user detect the problem by failing if they don't change the value, and run unit tests that detect problems with the charts. With the current way of testing this won't work, as any value that would pass the test would also be possible to install without the user changing it.
Maybe we could have some values for testing that are not just the defaults? Looks like we already change some of them.

@lentzi90 Excellent! That's the stuff I was looking for.

LGTM!
I would not like too much logic in the init-unit-test.bash script though so if we need to add similar things for all the charts then I think it would be better to have a prepared config file instead of updating it in this way.

Agreed!


empty eot files

MacOS X 10.10.3
Task:
gulp.task 'fonts', ->

  fontfacegen = require('fontfacegen')
  fs = require('fs')
  path = require('path')
  concat = require('concat-files')

  fonts = fs.readdirSync('src/fonts/')
  cssFiles = []
  for font in fonts
    extension = path.extname(font)
    fontname = path.basename(font, extension)
    fontfacegen
      source:  path.join('src/fonts/', font),
      dest: 'build/fonts/',
      collate: true
      css: "build/fonts/#{fontname}.css"
      css_fontpath: "../fonts/"

Results in:

↑ eot is empty =(
Deps are installed:
$ brew install fontforge ttf2eot batik
Warning: fontforge-20120731_1 already installed
Warning: ttf2eot-0.0.2-2 already installed
Warning: batik-1.7 already installed

Sadly, I get the same issue here.

I've just tested the command used by this script :
/usr/local/bin/ttf2eot "/a/path/foo.ttf" > "/a/path/foo.eot"
It works well but inside the node script it doesn't. Any idea @agentk ?

I think @BsWiM's solution of using the node provided exec is the best solution. It's merged in now. I'll be pushing out npm updates for fontfacegen, gulp-fontgen and grunt-fontgen tonight.


Redundant checks for nil pointers when calling an inlined method

(This is an issue about less than ideal generated code. I don't know if the project accepts such issues?)
In real programs, I've observed some redundant checks for nil pointers that I think the compiler should be able to optimize away. They seem to have to do with inlined methods.
Here is a reduced test case that panics due to a nil pointer dereference:
package main

import "bytes"

type Foo struct {
  buf bytes.Buffer
}

func (f *Foo) size() uint32 {
  return uint32(f.buf.Len())
}

func main() {
  var f *Foo
  f.size()
}

The segfault looks like this in GDB (with Intel syntax):
Dump of assembler code for function main.main:
   0x0000000000400c00 <+0>:     xor    eax,eax
   0x0000000000400c02 <+2>:     cmp    rax,0x0
   0x0000000000400c06 <+6>:     je     0x400c16 <main.main+22>
   0x0000000000400c08 <+8>:     mov    rbp,QWORD PTR [rax+0x8]
   0x0000000000400c0c <+12>:    mov    rbx,QWORD PTR [rax+0x18]
   0x0000000000400c10 <+16>:    sub    rbp,rbx
   0x0000000000400c13 <+19>:    mov    ebx,ebp
   0x0000000000400c15 <+21>:    ret
=> 0x0000000000400c16 <+22>:    mov    DWORD PTR [rax],eax
   0x0000000000400c18 <+24>:    jmp    0x400c08 <main.main+8>

The code checks f for being nil, and intentionally segfaults if so. But I believe it doesn't need to: the compiler knows the minimum page size on the architecture, and knows that Foo is less than a page long. Therefore if f is nil, then f+0x8 will be on the same unmapped page as 0x0, and should segfault just the same as the write to 0x0 that the generated code uses. In other words: the branch need not exist, because the instruction at +8 will already segfault if f is nil.
Indeed when a member of Foo is instead read directly, this fact appears to be used:
package main

type Foo struct {
  i uint32
  j uint32
}

func (f *Foo) size() uint32 {
  return f.j
}

func main() {
  var f *Foo
  f.size()
}

Dump of assembler code for function main.main:
   0x0000000000400c00 <+0>:     xor    eax,eax
=> 0x0000000000400c02 <+2>:     mov    ebp,DWORD PTR [rax+0x4]
   0x0000000000400c05 <+5>:     ret

Go version:
% go version
go version devel +3b7841b Mon Jun 29 01:49:05 2015 +0000 linux/amd64

Sure, we accept these kinds of issues.

The work-in-progress SSA backend may help here.

This is fixed with the new SSA backend.


Update earth-science-data-category.ttl

Merged Resource Geology and Economic Geology
Added some terms to the lodgement collection and did some minor reordering to accommodate.

The content changes make sense. However, it fails the SKOS review:
**ml - Missing Labels Finds concepts and conceptSchemes with missing labels.
FAIL (2)
http://linked.data.gov.au/def/earth-science-data-category/build-environment-and-design
http://linked.data.gov.au/def/earth-science-data-category/resource-geology**

@LukeHauck thanks for the pick up. Couple of orphan broader terms. Should be fixed now.


MS Graph Batch Connector no longer exists

Step "Create Flow" does not work as the MS Graph Batch Connector no longer exists in Power Automate

Hi @dkerstein! I see you closed this already, but just to clarify, these steps use a custom connector, and MS Graph Batch Connector is just the name you give to it.


Apple - 12 might be a supported release

Link to product page on endoflife.date
https://endoflife.date/ios
Details of incorrect and correct details you have found
iOS 12 got a new release on 23-Jan-2023: iOS 12.5.7.
The following devices got the update (https://support.apple.com/en-us/HT213597)

iPhone 5s, iPhone 6, iPhone 6 Plus, iPad Air, iPad mini 2, iPad mini 3, and iPod touch (6th generation)

What is the source website for the product and for its version information?

https://support.apple.com/en-us/HT213597

We currently mark all these devices and OS versions as unsupported (on the ios, ipad, iphone pages). However, given that Apple is still pushing security updates to these, we could mark these as supported.
Alternatively, we could add a note on the respective pages about the devices being out of support, but still getting occasional security updates (Already present on iphone/ipad pages).
But since Apple has now done this multiple times (iOS 12.5.2 was 26 Mar 2021, and iOS 12.5.7 in Jan 2023), these devices have been consistently getting security updates for the last 2 years, and this doesn't seem to be changing, we should probably mark these as supported now.

As reported on verge: https://www.theverge.com/2023/1/24/23569600/apple-iphone-5s-security-update-ios-12


Feature/coloring: Four colorings for Zephyr and Pegasus, and a two coloring for Chimera

Colorings of minimal chromatic number on Chimera, Pegasus and Zephyr graphs in the form of a map using the standard coordinate schemes.

@arcondello  I review this


Reference active file in terminal command

Hi, how do I reference the active file in terminal command?
Something like echo ${file}

I was wondering the same, I'm trying to run test classes of the currently opened file, but if I use the ${fileBasename} in the terminal command, it doesn't use the variable, but instead just prints '${fileBasename}'

Yea I didn't figure it out. But I worked around it by using the command runner plugin, and binding a command to a hotkey, then used the Stream Deck to trigger the hotkey.
https://marketplace.visualstudio.com/items?itemName=edonet.vscode-command-runner

Hey,
I've looked in the documentation and source and couldn't find how you could do that, except for sending a sequence to the terminal.
What you could do is:
Create an action where the command ID is the send sequence to the terminal, and the arguments are the command itself with the supported variables.
Send sequence example:
https://code.visualstudio.com/docs/editor/integrated-terminal#_send-text-via-a-keybinding
Supported variables:
https://code.visualstudio.com/docs/editor/variables-reference

{
    "text": "npm run fix-my-file ${file} \r"
}

\r or \u000D as a return


[style]:  Nav bar and footer redesigning

What's the style idea?
I shall add a Navbar which is important to showcase the necessary facilities. A proper footer is also missing
Add screenshots
No response
Record

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I agree to follow this project's Code of Conduct
[X] I want to work on this issue

sure you can grab this, but please share the rough design of this.

Hi, @Souvik34 . I want to add responsive navbar along with footer and also want to add various coding platforms at one place.

@YathishGP003  I am currently working on this issue after I create a PR you can follow up then

@Saurav-Pant

When I tried to run it locally it showed me this error??

Thanks for working on the issue @Souvik34.
Please Login to Clerk create one application and select these options (Chk Pic)
rename .env.example to .env and in place of these paste your own stuff
NEXT_PUBLIC_CLERK_PUBLISHABLE_KEY=
CLERK_SECRET_KEY=

If you stuck in anyplace feel free to ping me


raft: fix Campaign on restore if the node id is not in the progress list

When restarting a node from its state, logs are reapplied to
restore the cluster membership with ConfChangeAddNode log
entries.
There is a chance to reach the portion calling Campaign to
restart a single cluster member faster when we are still
in the process of restoring the memberlist. In this case
the node may not appear in the progress list and we might
call MaybeUpdate on the node id from the map without
sanity checking thus resulting in a panic.
To avoid that, check if the node is present in the Progress
list before trying to Campaign on node restore.
Fix #1196
/cc @aaronlehmann @LK4D4
Signed-off-by: Alexandre Beslic<EMAIL_ADDRESS>

[Current coverage][cc-pull] is 55.10% (diff: 100%)

Merging [#1221][cc-pull] into [master][cc-base-branch] will increase coverage by 0.13%

@@             master      #1221   diff @@
==========================================
  Files            77         77
  Lines         12368      12369     +1
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits           6799       6816    +17
+ Misses         4632       4620    -12
+ Partials        937        933     -4

[][cc-pull]

Powered by Codecov. Last update [4d7e443...e786694][cc-compare]
[cc-base-branch]: https://codecov.io/gh/docker/swarmkit/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/docker/swarmkit/compare/4d7e44321726f011d010cdb72d2230f5db2b604e...e7866943f36fe9dd0639fd3e1470ef7dfb67357d?src=pr
[cc-pull]: https://codecov.io/gh/docker/swarmkit/pull/1221?src=pr

I've tested and there is no panic.

Let's say the node does not exist in the Progress map the first time we reach this code. Is there a guarantee that the Ready() channel will give us another chance to try it?

@aaronlehmann Not sure if I get your question right but I'll try to answer:
No it will give up on the election "fast-track" process and wait for the full ElectionTick because the node wasn't ready to be elected yet. I think it is more reasonable than panicking :)
This is the easiest way I think to circumvent the panic without having to cherry-pick the patch doing the sanity check alongside v2.3.2 and vendor. It has the same effect of avoiding the call to Campaign if the node is not in the progress map. Only difference is that we do the check upstream before even calling Campaign (which is sending a MsgHup, triggering a Step() and calling the private campaign method).

Well. The actual fix is here: https://github.com/coreos/etcd/pull/6039

@abronan @aaronlehmann Before calling Campaign, we need to make sure all pervious configuration changes are applied. You can ensure this at application layer. But I feel we should ensure this at raft layer, so applications do not need to repeat this logic. The one line change was just a quick hack. If it is urgent, you can take it.

LGTM

LGTM
Filed #1229 to keep track of vendoring update


Theme is out of sync with atom-material-syntax

This variant of the theme is out of sync with the main theme, pretty please give some love to this variant.

👍


Feature add compose command

We should create a new compose command that allows us to compose configurations. This feature is already implemented via check command.
Usage

Print the composed configuration on the terminal
tailcall compose a.yml b.graphql c.json

Write the composed configuration on the path
tailcall compose a.yml b.graphql c.json --out abc.graphql

Print the generated blueprint
tailcall compose a.yml b.graphql c.json --blueprint

Write the generated blueprint
tailcall compose a.yml b.graphql c.json --blueprint --out abc.json

NOTE: For blueprint only JSON | YML format should be supported, should err otherwise.

Additional Requirements

Drop the --out parameter from check

/bounty 200

Hey, may I work on this?

in queue if re-assigned

in queue

@tusharmath Hey there, I have some questions:

In what format should be the configuration displayed in the terminal? (when there is no out file)?
If there is no out file, wouldn't be a good idea to add a parameter that specify the format of the composed config displayed?
What is the difference between the blueprint and the config? I can see that there is a function display_schema that prints only the schema from the blueprint, but I am not sure that is what it should output.

@adelinaenache Good questions

In what format should be the configuration displayed in the terminal? (when there is no out file)?

Default should be "pretty" json. But user should be able to customize it via the --format.

If there is no out file, wouldn't be a good idea to add a parameter that specify the format of the composed config displayed?

Yes. We can add a --format option.

What is the difference between the blueprint and the config? I can see that there is a function display_schema that prints only the schema from the blueprint, but I am not sure that is what it should output.

Config is what is inputted by the user. It validated and converted into a blueprint. The blueprint is like config, but stricter. If we have a blueprint we can be sure that a graphql server can be built using it, however we can't guarantee the same using a Config.

Config is what is inputted by the user. It validated and converted into a blueprint. The blueprint is like config, but stricter. If we have a blueprint we can be sure that a graphql server can be built using it, however we can't guarantee the same using a Config.

Ok so I'm trying to figure out what should I output when --blueprint is used vs when it's not used. Until now, I get that I should use the Config class to output normal configuration and the generated Blueprint class when using --blueprint.
So, what am I getting until now is:

For normal configuration (without --blueprint option), shouldn't be any validity check in place? (just read from files and do not convert it into a blueprint, output the config just like with check)
With --blueprint option, should I add some additional code on the Blueprint that converts it to either YML or json and use that to write to file?
Why we disallow writing GQL schemas with --blueprint?

For normal configuration (without --blueprint option), shouldn't be any validity check in place? (just read from files and do not convert it into a blueprint, output the config just like with check)

Yes we should just compose them together without validating.

With --blueprint option, should I add some additional code on the Blueprint that converts it to either YML or json and use that to write to file?

You can use serde and serde_yml to do this.

Why we disallow writing GQL schemas with --blueprint?

I am not sure what you mean by this question.

@adelinaenache Let me know if you are still on this or if I should reassign.

@tusharmath I want to work on this one can you assign this to me?


Where do I set the token for a private dataset?

I've installed the module and copied the sanity.json to the root folder. All works well with a public dataset.
When I change it to private it breaks (obviuosly) but I'm having trouble finding where to put the sanity project read token ..
Could you please show me how it is done?
Thanks

I just realized that it was anywhere in nuxt.config. Thanks


Cannot import tauri::api::process::Command

I'm trying to use a sidecar binary in a Rust command.
I want to proceed as mention on the doc:

It's also used in this repo examples: https://github.com/tauri-apps/tauri/blob/aecdfaf76fcf0d2820d6ce6eb7590ebe399bfa04/examples/sidecar/src-tauri/src/main.rs#L10-L13
But the Command struct is simply not there to be imported.

Neither while compiling with yarn tauri dev:
error[E0432]: unresolved import `tauri::api::process::Command`
 --> src/project/import_files.rs:8:5
  |
8 | use tauri::api::process::Command;
  |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^ no `Command` in `api::process`

timothee@Timothees-MacBook-Pro ~/D/t/unrush (working-files)> yarn tauri info
yarn run v1.22.11
$ /Users/timothee/Documents/tdelabro/unrush/node_modules/.bin/tauri info
 app:spawn Running "/Users/timothee/Documents/tdelabro/unrush/node_modules/@tauri-apps/cli/bin/tauri-cli tauri info" +0ms

Operating System - Mac OS, version 11.3.1 X64

Node.js environment
  Node.js - 16.8.0
  @tauri-apps/cli - 1.0.0-beta.10
  @tauri-apps/api - 1.0.0-beta.8

Global packages
  npm - 7.21.0
  yarn - 1.22.11

Rust environment
  rustc - 1.54.0
  cargo - 1.54.0

App directory structure
/node_modules
/src-tauri
/.git
/front

App
  tauri.rs - 1.0.0-beta.8
  build-type - bundle
  CSP - default-src blob: data: filesystem: ws: wss: http: https: tauri: 'unsafe-eval' 'unsafe-inline' 'self' img-src: 'self'
  distDir - ../front
  devPath - http://localhost:5000

✨  Done in 3.53s.

That was it.
I had understood that I had to enable module in the allow list in order to use some js api, but I didn't though it was required for rust calls too.
The doc is maybe a bit ambiguous too:
On the JavaScript code, import the Command class on the shell module and use the sidecar static method:
vs
On the Rust code, import the Command struct from the tauri::api::process module:
Thank you for the help


thank you for your work

What an I supposed to see here?  Please explain (preferably in full English sentences) what you did, what did you expect to happen, and what happened instead.

Tips abstract method has not been implemented

Syntastic has nothing to do with code completion.


Reuse query engine for multiple PrismaClient instances

Problem
One of the most tricky parts of using Prisma Client is database connection handling.
While modern databases like Fauna DB or serverless Aurora don't require a long-running connection anymore and therefore no long-running process that keeps the connection up, the databases currently supported by Prisma (in particular MySQL and Postgres) have the concept of long-running connections, which are expensive to establish and once established should be reused.
In order to ensure optimal performance with these databases, the query engine, which powers Prisma Client, manages a connection pool. The default size of that connection pool is being determined by the number of cores times 2, as it turned out in internal performance tests, that that is the optimal number for the best performance.
All this works well, as long as there is only one Prisma Client and therefore query engine instance.
And usually, as you have your application and database under control (should at least), you can ensure, that there is just one Prisma Client instance, which holds the query engine with the connection pool.
However, as soon, as you have multiple Prisma Client instances running, this can get really slow. It can even be a memory leak if you instantiate a new Prisma Client per incoming API request.
This situation, that too many Prisma Clients have been instantiated has occurred numerous times in the community, see https://github.com/prisma/prisma/issues/1983 https://github.com/prisma/prisma-client-js/issues/632 https://github.com/prisma/prisma-client-js/issues/723 and https://github.com/blitz-js/blitz/blob/canary/examples/store/db/index.ts
In other words: You can easily shoot yourself in the foot because you need to manage state and managing state is always nasty.
How could Prisma Client help here?
One possible solution: Instead of instantiating a new query engine process per PrismaClient instance, we would have a "named singleton" pattern.
That means, that for a particular combination of args, we reuse the query engine instance and make that accessible under a name, e.g. "QE1". This way it doesn't matter, how often you instantiate PrismaClient, all instances will reuse the same query engine and therefore the same connection pool, with all the connections already warmed up.
This could also help us in the situation of clusters: https://github.com/prisma/prisma-client-js/issues/632  we could enable this singleton pattern across the whole cluster out of the box so that the node cluster can share the same connection pool. Building something like that as a user would be a lot of overhead and a non-trivial task to do.
What are disadvantages of this approach?
Usually, a singleton is considered an antipattern. In this case it could be considered "ok" though, as we're talking about resources that can be shared (and maybe should be shared) across instances.
Another big problem here is: You might indeed have a serious memory leak in your application, which would now not be visible anymore, as it "just works".
One could see this approach as "wrong", as you can always say: "Just instantiate  one Query Engine!".
Closing thoughts
We have this problem because the query engine is attached as a sidecar to the Node.js process. If we would run the query engine as a separate process as you do with a database, we would not have this problem of "unintentionally spawning too many query engines". However, the ergonomy and developer experience of Prisma Client would then greatly suffer.

A couple of points:

You would also not have this problem with direct bindings (not just a separate process)
Sometimes it's tricky how the PrismaClient is created, so I do support the use of a singleton (I was against before having some issues myself)
I believe if an attempt is made to "repawn" a new client, a warning should be displayed to the user by default so he is informed that he tried to spawn a new process but the client catched it (but make it so we can disable that with a flag if this is the intended behaviour)

You would also not have this problem with direct bindings (not just a separate process)
That highly depends on the process model we would choose with the direct bindings.

I believe if an attempt is made to "repawn" a new client, a warning should be displayed to the user by default so he is informed that he tried to spawn a new process but the client caught it (but make it so we can disable that with a flag if this is the intended behaviour)

Yes, we could additionally add a warning for that case.

This could also help us in the situation of clusters: #632 we could enable this singleton pattern across the whole cluster out of the box so that the node cluster can share the same connection pool.

I don't think this is quite right. As far as I understand node cluster, it's a master process + N worker processes.
The singleton will occupy the master processes' memory space, but not be accessible to the worker processes.
I agree that a default singleton is probably the most convenient out of the box, plus it requires no API changes. I'd like to suggest we separate these out cleanly.
The use case I have in mind is a single Prisma Engine on a separate process, probably locally but it could be over a network. I think it's possible to support our current setup and this setup quite easily with the right abstraction.
Some psuedo-code:
import { PrismaClient, QueryEngine } from "@prisma/client"

// the default, works the same way as today
// but uses a QueryEngine singleton under the hood
const client = new PrismaClient()

// alternatively, spawn a query engine yourself
const qe = new QueryEngine(options)
// and pass the query engine process into the prisma client
const client = new PrismaClient({ process: qe })

// alternatively, to connect to a query engine on a separate process
const qe = new QueryEngine(options)
qe.listen(port)

// connect to that query engine with some known URL.
const client = new PrismaClient({ url: queryEngineURL })

I don't think this is quite right. As far as I understand node cluster, it's a master process + N worker processes. The singleton will occupy the master processes' memory space, but not be accessible to the worker processes.

Exactly, the singleton would run in the master, and in the workers, we would make it available via ipc communication. Users will just import from @prisma/client but in our code, we check with require('cluster').isWorker if we need to talk to the master to do queries. This is what I meant with "Building something like that as a user would be a lot of overhead and a non-trivial task to do.". We can do this as a library once to solve this for all users.
While I think that your API proposal gives more flexibility and could for sure be something that advanced users might be interested in, the point of this issue was, that even the current API can be a foot gun and especially for more beginner users to hard to manage.
If we now even go one level down in abstraction, we would introduce another level of complexity for the user to manage.
I like that idea, but I would see that as an advanced API.

While I think that your API proposal gives more flexibility and could for sure be something that advanced users might be interested in, the point of this issue was, that even the current API can be a foot gun and especially for more beginner users to hard to manage.

The singleton pattern can protect against this case. I'm 👍  to that.

Users will just import from @prisma/client but in our code, we check with require('cluster').isWorker if we need to talk to the master to do queries

This feels like an overreach. We're now managing 3 processes, their lifecycles, their IPC communication, error states behind: const client = new PrismaClient(). Certainly possible, it will just lead to more configuration being moved into new PrismaClient(config).
Personally, I'd rather we suggest using the advanced API once you're reaching for node's cluster capabilities. This will allow us to more gracefully move to Prisma engine working across machines.


Remove custom runtime

so that app-tools-lib-for-java is the only runtime
Requires:
Rename to simply App Engine Standard
Change default runtimes in AppEngineFacet
Remove custom runtime for plugin.xml

done


File reopen when i delete the folder from my disc

Type: Bug
I'm trying to delete old folders with some projects and when an open vsc opens, several windows open showing me the old files that were deleted but they don't have an address, so every time I open again all these windows open. This happened to me several times. I changed a lot of things and nothing happened. I do not know what to do. I want to delete a file and I don't want to see all these windows of files that are no longer on my drive.  HELP
VS Code version: Code 1.81.1 (6c3e3dba23e8fadc360aed75ce363ba185c49794, 2023-08-09T22:22:42.175Z)
OS version: Windows_NT x64 10.0.19042
Modes:

System Info

Item
Value

CPUs
AMD Ryzen 5 3500X 6-Core Processor              (6 x 3593)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: enabled

Load (avg)
undefined

Memory (System)
15.94GB (7.89GB free)

Process Argv
--crash-reporter-id 70bdaa89-cac3-436a-8ea1-91c12d8d5113

Screen Reader
no

VM
0%

Extensions (18)

Extension
Author (truncated)
Version

better-comments
aar
3.0.2

simple-react-snippets
bur
1.2.7

path-intellisense
chr
2.8.5

vscode-eslint
dba
2.4.2

es7-react-js-snippets
dsz
4.4.3

vscode-html-css
ecm
2.0.6

prettier-vscode
esb
10.1.0

auto-rename-tag
for
0.1.10

code-runner
for
0.12.1

terminal
for
0.0.10

activitusbar
Gru
0.0.47

material-icon-theme
PKi
4.33.0

LiveServer
rit
5.7.9

es7-react-js-snippets
rod
1.9.3

tabnine-vscode
Tab
3.57.0

errorlens
use
3.15.0

vscode-todo-highlight
way
1.0.5

JavaScriptSnippets
xab
1.8.0

(1 theme extensions excluded)

A/B Experiments
vsliv368cf:30146710
vspor879:30202332
vspor708:30202333
vspor363:30204092
vscod805cf:30301675
binariesv615:30325510
vsaa593:30376534
py29gd2263:30899288
vsclangdf:30486550
c4g48928:30535728
azure-dev_surveyone:30548225
2i9eh265:30646982
962ge761:30933248
pythongtdpath:30769146
i26e3531:30792625
pythonidxpt:30866567
pythonnoceb:30805159
asynctok:30898717
pythontestfixt:30902429
pythonregdiag2:30928863
pyreplss1:30897532
pythonmypyd1:30879173
pythoncet0:30885854
pythontbext0:30879054
dsvsc016:30899300
dsvsc017:30899301
dsvsc018:30899302
aa_t_chat:30882232
dsvsc019a:30933241

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.85.1. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

it appears unsaved but if i save or not this will appear again and again

help lol

Did you try closing each file individually and opting to discard the changes?

We closed this issue because it is a question about using VS Code rather than an issue or feature request. Please search for help on StackOverflow, where the community has already answered thousands of similar questions. You may find their guide on asking a new question helpful if your question has not already been asked. See also our issue reporting guidelines.
Happy Coding!


Layer dropdown icon missaligned

Expected Behavior
When editing page and click on element the  drop down icons (arrow-up/arrow-down) of each layer should be centered.
Current Behavior
Layout drop down icon are not centered and rotate each time you toggle the acordion section of clicked layer
Steps to Reproduce
Steps to reproduce the behavior:

Login as admin
Go to a page
Click pencil icon
Click on any element
On the right menu go to layers label
Toggle right arrow down icon of one of the available layers

Context (Environment)

Default setup
OS: MacOS, Windows 11
Browser: Chrome
Environment: dev

Issue demo
https://github.com/Protofy-xyz/Protofy/assets/108338080/dc81b164-cd19-4733-979a-c6d3cae7fcf1

Fixed now arrow is centered when toggle it
https://github.com/Protofy-xyz/Protofy/assets/108338080/b04b4290-9062-47d2-8cd8-720e404ce6bf


Added ability to configure rules for aXe Accessibility Engine

I signed it!

Did you test this?  Does it work?

Yes, tested and works.  I played around with different options and it has been outputting the correct results.


"shift extend selection" feature doesn't work in the click->shift-click case because single-cell selection is gone

Environment
Windows build number: [19041.208  wsl ]
Windows Terminal version (if applicable): 0.11.1251.0

Any other software?

Steps to reproduce

Single click.
Hold SHIFT and single click an unselected region.

Expected behavior
Region between two click should be selected.
Actual behavior
region between two click hasn't been selected.
I remeber this feature has been released in "Windows Terminal Preview v0.9.433.0", said: "You can now hold down Shift and click to extend your selection (#4404)". But in recent releases, it gone.

So, fair enough. Shift only works to extend an actual extant selection Since single-click no longer selects a cell, it doesn't comprise an active selection.
We should probably expand the touchdown anchor when you click + shift-click. It's not high-priority, because there's a workaround (select 2 cells, then use shift-click.)

So, fair enough. Shift only works to extend an actual extant selection Since single-click no longer selects a cell, it doesn't comprise an active selection.
We should probably expand the touchdown anchor when you click + shift-click. It's not high-priority, because there's a workaround (select 2 cells, then use shift-click.)

Thanks.  It's just a little inconvenient when select a long region now.

@DHowett, @carlos-zamora, @ymkun - ConEmu does something that is IMHO more intuitive:

Shift+Click - selects single cell
The next Shift+Click expands this selection.

If you like this approach, I can commit (it is less than one-liner :blush:)

Sounds great, with one caveat.
We need to figure out how this works with Shift suppressing VT mouse mode. Will any single shift-click on a mouse mode application cause a single cell selection?
I might be okay with that.

@DHowett - I am not sure I understood the concern, can you please explain?
I mean I tried this in tmux with mouse mode on, and it works exactly like shift +drag one char (but with no need to drag).
Guess I am missing something.

Sure.
I was worried that introducing this behavior would make it difficult to single-click on the terminal itself when there was a mouse mode application running without selecting a single cell.
However, thinking about it more: There is no reason to click on the terminal itself during mouse mode unless you are making a selection, so this is okay. As long as it does not highlight a cell when you use Ctrl+Shift+Click on a URL!

Sure.
I was worried that introducing this behavior would make it difficult to single-click on the terminal itself when there was a mouse mode application running without selecting a single cell.
However, thinking about it more: There is no reason to click on the terminal itself during mouse mode unless you are making a selection, so this is okay. As long as it does not highlight a cell when you use Ctrl+Shift+Click on a URL!

I will try this. If works as expected - should I submit PR under this ticket?

Absolutely! :smile: We love your PRs.

@DHowett - lol. Actually I am asking if I should mark the PR as "Closing" this one.. or not

Yeah, I think that your proposal is sufficient to solve this issue!