Feature request: show notification when ESLint fails to load

The status of ESLint is indicated in the status bar by the tiny x symbol:

This is very difficult to spot. I want to be alerted as soon as there's an issue. Could we send a notification when ESLint fails to load?
For context, I encounter these errors quite frequently because my ESLint config depends on custom rules that need to be built prior to ESLint running. A notification would serve as a useful reminder that I need to build my custom lint rules.

We had notifications in the past and move to language status since notifications are annoying if you open workspaces where you can't fix this. So if we bring notifications back it has to go behind a setting.

We had notifications in the past and move to language status since notifications are annoying if you open workspaces where you can't fix this. So if we bring notifications back it has to go behind a setting.

In terms of notifying, I find the way prettier does it more than suffices to draw the person's attention to the fact there is an issue, which can then be debugged in output.

VS Code's recommendation for something like this is to use a language status.

If every extension has its own status indicator this will look very crowded.

The user can select which indicators they wish to see? I would also consider eslint - along with prettier - to be perhaps the 2 most import for a developer to be able to monitor while working.
Also a split between soft errors (current file not included) and hard errors (eslint crashing/failing to configure) would very helpful.

Yes, the user can ping the ESLint indicator to the status line.

I will close the issue since ESLint follows the recommended VS Code standard.


Gifs show up on the same line.

Mattermost Server: 5.21.0
Mattermost Client: 4.4.0

Thanks for the report !
This is fixed in the new 1.2.0 release of the plugin.


link problem on windows

On win10 with the "button" example, I think I have installed windows sdk.

But sometimes the link just failed during the build process.  The error was:
Non-UTF-8 output: LINK : fatal error LNK1104: \xce\xde\xb7\xa8\xb4\xf2\xbf\xaa\xce\xc4\xbc\xfe\xa1\xb0D:\rust_build\all\release\examples\button.exe\xa1\xb1\r\n

And if I rebuild, it will fail again, unless I rename the target exe file and rebuild again. Btw, the target exe file cannot be deleted, but can be renamed.

After following this instructions:
https://github.com/rust-lang/book/pull/2389
The unstable link problem seems to be gone.

This is very interesting. I'm curious if anyone else can repro this. I'll respond here if I can.

I'm curious if you still have issues if you use pre-packaged rust version of LLD. We just switched our config to use that.
https://github.com/bevyengine/bevy/pull/150

I booted into windows and I haven't been able to repro yet.

I haven't been able to reproduce it.  I believe I have SDK 10.0.18362.0

Cool. im down to leave this open for a few days to see if anyone else hits the issue. Otherwise I'll probably close this.

After revisiting this post https://github.com/rust-lang/book/pull/2389 and add the English lang pack, now the err changed to:
 fatal error LNK1104: cannot open file 'D:\rust_build\release\examples\button.exe'
How to repro:
Using the button example, switch between the two fonts until the error occur (sometimes it will appear easily, sometimes after several tries). You can find the second font here: https://github.com/bevyengine/bevy/issues/112
font: asset_server.load("assets/fonts/simhei.ttf").unwrap(),```

But the interesting thing is no such error occurred to other projects when using Clion terminal.


[Issue]: Second agent does not execute the python script

Describe the issue
I have 2 agents working together. Each performs a specific task. The first agent is able to generate the python script and execute it and pass its output to the second agent. The second agent is able to generate another python script but it does not execute it. Instead, it just terminates after generating the script.
Steps to reproduce
No response
Screenshots and logs
No response
Additional Information
No response

Hi @ghSaint, can you provide a code sample?

hi @marklysze , i have the same problem . let me show my code.
in the OAI_CONFIG_LIST file ,i used ollama+llama3.1 model:
[
    {
        "model": "llama3.1",
        "api_key": "ollama",
        "base_url": "http://<IP_ADDRESS>:11434/v1",
        "api_type": "ollama"
    }
]

the test code template from other Issue:
from autogen import config_list_from_json, UserProxyAgent, AssistantAgent, GroupChat, GroupChatManager

config_list_llama = config_list_from_json(
    "../OAI_CONFIG_LIST",
    filter_dict={
        "model": ["llama3.1"]
    },
)
llm_config = {"config_list": config_list_llama, "cache_seed": 42, "temperature": 0, "price": [0.00001, 0.000001]}
user_proxy = UserProxyAgent(
    "user_proxy",
    code_execution_config={"work_dir": "autocode", "use_docker": False},
    human_input_mode="NEVER",
    max_consecutive_auto_reply=1,
    is_termination_msg=lambda x: x.get("content", "").rstrip().endswith("TERMINATE"),
    description="I stands for user."
)
task = """draw a network topology with a central switch connected to a router and three other switch in txt"""
agent1 = AssistantAgent(name="agent1", description="I stands for agent1.", llm_config=llm_config)
agent2 = AssistantAgent(name="agent2", description="I stands for agent2.", llm_config=llm_config)
agent3 = AssistantAgent(name="agent3", description="I stands for agent3.", llm_config=llm_config)

groupchat = GroupChat(agents=[agent1, agent2, agent3, user_proxy], messages=[], max_round=10)
manager = GroupChatManager(groupchat=groupchat, llm_config=llm_config)
user_proxy.send(task, recipient=manager, request_reply=True)

the run logs:
/Users/mrz/Documents/Python/miniconda3/envs/autogen/bin/python /Users/mrz/Documents/Python/PYTHON/AutoGen/demo/mutiTask.py
user_proxy (to chat_manager):

draw a network topology with a central switch connected to a router and three other switch in txt

--------------------------------------------------------------------------------

Next speaker: agent1

agent1 (to chat_manager):

To draw a network topology as per your request, I'll first collect some information about the network. Here's a Python code block that will print out the required info:

# filename: network_topology.py
print("Network Topology:")
print("-------------------")
print("Central Switch (SW1) connected to Router (RTR)")
print("SW1 connected to SW2, SW3 and SW4")

print("\nDevices:")
print("---------")
print("Router (RTR): connects the network to the internet")
print("Switches:")
print("  - SW1: central switch")
print("  - SW2: secondary switch")
print("  - SW3: tertiary switch")
print("  - SW4: quaternary switch")

print("\nConnections:")
print("------------")
print("SW1 -> RTR (Ethernet cable)")
print("SW1 -> SW2 (Ethernet cable)")
print("SW1 -> SW3 (Ethernet cable)")
print("SW1 -> SW4 (Ethernet cable)")

print("\nTopology Diagram:")
print("------------------")
print("  +---------------+")
print("  |               |")
print("  |  RTR          |")
print("  |               |")
print("  +---------------+")
print("           |")
print("           |")
print("  +---------------+")
print("  |               |")
print("  |  SW1          |")
print("  |               |")
print("  +---------------+")
print("           |")
print("           |")
print("  +---------------+")
print("  |               |")
print("  |  SW2          |")
print("  |               |")
print("  +---------------+")
print("           |")
print("           |")
print("  +---------------+")
print("  |               |")
print("  |  SW3          |")
print("  |               |")
print("  +---------------+")
print("           |")
print("           |")
print("  +---------------+")
print("  |               |")
print("  |  SW4          |")
print("  |               |")
print("  +---------------+")

Now, let's analyze the network topology. The central switch (SW1) is connected to a router (RTR), which connects the network to the internet. The central switch is also connected to three other switches: SW2, SW3, and SW4.

The devices in this network are:

*   Router (RTR): connects the network to the internet
*   Switches:
    *   SW1: central switch
    *   SW2: secondary switch
    *   SW3: tertiary switch
    *   SW4: quaternary switch

The connections between devices are as follows:

*   SW1 is connected to RTR via an Ethernet cable.
*   SW1 is connected to SW2, SW3, and SW4 each via an Ethernet cable.

This network topology can be visualized using the provided diagram. The central switch (SW1) serves as a hub for all other devices in the network.

TERMINATE

--------------------------------------------------------------------------------

Next speaker: user_proxy

Process finished with exit code 0

but there is no content related to ">>>>>>>> EXECUTING 2 CODE BLOCKS (inferred languages are [python, python])..." in the log.like the pic:


fix(sidenav): container not picking up indirect descendant sidenavs

Fixes the sidenav container not picking up indirect descendant sidenavs.

@mmalerba I expanded the changes to handle nested containers. Can you take another look?


Index release-next

We need to submit a fix to https://github.com/algolia/docsearch-configs/blob/master/configs/cert-manager.json
in order to also index the release-next branch

/close


Update some food spawners w/ entity tables

About the PR

Changes food_produce spawner to use entity table
Adds food_meat and food_ingredient spawners to compliment the produce spawner

Why / Balance
Updates to newer system and rounds out kitchen mapping
Technical details
n/a
Media
n/a
Requirements

[x] I have read and I am following the Pull Request Guidelines. I understand that not doing so may get my pr closed at maintainer’s discretion
[x] I have added screenshots/videos to this PR showcasing its changes in-game, or this PR does not require an in-game showcase

Breaking changes
n/a
Changelog
n/a

Test fails look bogus


[atest] Using templated function such as toThrow requires template keyword

Using templated function such as toThrow (e.g. expect([] { throw 1; }).toThrow<int>(); does not work and requires template keyword (e.g. expect([] { throw 1; }).template toThrow<int>();) otherwise parser fails: Reported bug: https://developercommunity.visualstudio.com/t/Cannot-instantiate-template-exported-fro/1387974

Fixed in MSVC 16.10 Preview 3


Add excerpt to frontmatter

This PR adds the excerpt field to the frontmatter.

Frontmatter code was refactored. Frontmatter fields are now added via this example so they can be optionally included by editing frontmatter_fields in settings.js.
Because of this rewrite, I was unable to merge your PR directly, but I did cherry-pick your commit so you get contributor credit.
Frontmatter excerpt is available for use in v2.3.1.
Thank you!


spring hands on security

spring hands on security

Metric
Value

alert_status
:white_check_mark: OK

bugs
0

code_smells
12

reliability_rating
A (0 Bugs)Note: A being best, E being worst

security_rating
E (at least 1 Blocker Vulnerability)Note: A being best, E being worst

vulnerabilities
2

Metric
Value

alert_status
:x: ERROR

bugs
0

code_smells
12

reliability_rating
A (0 Bugs)Note: A being best, E being worst

security_rating
E (at least 1 Blocker Vulnerability)Note: A being best, E being worst

vulnerabilities
2


Should set key prop on  to avoid React warning.

const parser = new Parser()
const renderer = new ReactRenderer({ softBreak: 'br' })

const vdom = renderer.render(parser.parse('123\n456'))

ReactDOM.render(<div>{vdom}</div>, root)

Warning: Each child in an array or iterator should have a unique "key" prop. See https://fb.me/react-warning-keys for more information.

Seems this warning only happened with react 16.


Add COD amount helper

What is the motivation for adding/enhancing this feature?

Add COD amount helper section so staff know easily how much COD to expect on their shipping label.
https://docs.google.com/document/d/1Z5VeHa66jqRUpWq-HjuEbn8mXkcZghSlMSDiyNQDskU/edit?tab=t.0#heading=h.t2o8w1t8effo

API endpoint details that can be used here:
endpoint: /api/service/getCODOrderRemainingTotal
Request payload:
{
    "orderId": "10110"
}

Response:
{
    "total": 5.00
}

Constraint: If Order does not have COD payment preference then it returns 0.


(how) can apply my own image?

I noticed your paper (very nice work!) mention about HistoSegNet can generalize to other datasets without retraining.
If I am not misunderstanding, I can load my own image and get the segment information?
If so, can you point out how to input my images?
With many thanks,
Duerna

Hello Duerna, yes, HistoSegNet can generalize to other datasets without retraining, assuming that:

They are histopathology images using stains found in our dataset (e.g. H&E) and not fluorescent scans
You rescale your images to fit the same microns/pixel resolution stated in our paper and then reshape to the 224x224 pixel resolution the model uses as input
The organs present in your histopathology images are roughly the same as those mentioned in our paper

If those requirements are met, then you can drop your images into a new sub-folder of the "img" folder and set the "input_name" parameter of HistoSegNetV1 accordingly. You can check out "demo_01_segment_patches.ipynb" for an example.

Thank you so much.
I have the HE-stain pictures that reshaped to 224x224 pixels.
And then drop the images to "img" and set the HistoSegNetV1.
However, I have tried several times, it not work with my images.
Here are they:

Can you please kindly point out how can I apply my img to the HistoSegNet?
Best regards,
Duerna

hi，i want to know that after I have got the segmentation result of each patch,how can I merge the patches to the origin size of the WSI


Need live examples of themes

Are there live examples of each theme?  React-static needs links to live examples of the various themes so we can see what they look like.

I'm not sure what you mean by themes. There are examples?
On Mon, Sep 10, 2018 at 4:03 PM Jason Hsu<EMAIL_ADDRESS>wrote:

Are there live examples of each theme? React-static needs links to live
examples of the various themes so we can see what they look like.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/nozzle/react-static/issues/742, or mute the thread
https://github.com/notifications/unsubscribe-auth/AFUmCYR0Ie6aBwZKrG9bNukWzWNA4BMuks5uZuGlgaJpZM4WiQOd
.


Fix delete redundant tablet by default

Proposed changes
Issue Number: close #xxx
Problem summary
Describe your changes.
Checklist(Required)

Does it affect the original behavior:

[ ] Yes
[ ] No
[ ] I don't know

Has unit tests been added:

[ ] Yes
[ ] No
[ ] No Need

Has document been added or modified:

[ ] Yes
[ ] No
[ ] No Need

Does it need to update dependencies:

[ ] Yes
[ ] No

Are there any changes that cannot be rolled back:

[ ] Yes (If Yes, please explain WHY)
[ ] No

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

https://github.com/apache/doris/issues/15576

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 36.76 seconds
load time: 649 seconds
storage size:<PHONE_NUMBER>6 Bytes
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20230103141751_clickbench_pr_72979.html


Add the ability to set GLWpfControlSettings parameters within XAML.

Having to do the following in the code behind is odd when I should just be able to define this within XAML and simply call Start().
this.Start(new GLWpfControlSettings()
{
	MajorVersion = 4,
	MinorVersion = 6,
	GraphicsContextFlags = ContextFlags.ForwardCompatible,
	GraphicsProfile = ContextProfile.Core,
	RenderContinuously = true,
});

This would be pretty straightforward; just move the settings properties into GLWpfControl and have an overload function that uses the newly created properties. Overloading will ensure that we don't break any existing functionality. The function would likely just be as simple as this:
public void Start()
{
    this.Start(new GLWpfControlSettings()
    {
	MajorVersion = this.MajorVersion,
	MinorVersion = this.MinorVersion,
	GraphicsContextFlags = this.GraphicsContextFlags,
	GraphicsProfile = this.GraphicsProfile,
	RenderContinuously = this.RenderContinuously,
    });
}

Once a conversation is had about whether it's okay to go ahead with these changes I'd be happy to implement them myself :).

Yeah definitely think we should expose the control settings in the editor.
Is there no way to adapt the current settings object so we can expose that object in the editor?

Yeah definitely think we should expose the control settings in the editor.
Is there no way to adapt the current settings object so we can expose that object in the editor?

yeah, so @utkumaden has awoken me 😂. We can simply create a public Settings property. Pretty sure this will give us a similar solution:
    <glwpf:GLWpfControl.Settings>
        <MinorVersion>3</MinorVerrsion>
        <-- etc -->
    </glwpf:GLWpfControl.Settings>


stream mgmt: expose CANCEL_MOVE nicely

Request from you-know-who via me: give nats stream a subcommand or option there-of to syntactically sugar the cancel a move:
nats req '$JS.API.ACCOUNT.STREAM.CANCEL_MOVE.${ACCT}.${STREAM}' ''

Tested and confirmed this works in both system and stream-owner accounts.

We have hit a use case for this in some instances. This would be nice to have.

Question, how are you initiating moves? CLI also does not allow initiating moves using the move API.
The reason these 2 APIs are not in the nats cli is because they are just really badly designed and reusing data structures they should not, generally just dont like them.
So asking a bit more about how its used so I can see what I can do.

That is fair. The only times I have had to use it was in response to stalled moves and other failures. It has been a recovery option in those scenarios. I understand that if it is in the CLI, it could be used operationally and cause the issues above.


Remove role assignment cycles

Prevent existing role assignments to be destroyed and recreated when editing policy assignment parameters

Overview/Summary
This PR addresses the issue described in #266
This PR fixes/adds/changes/removes
Breaking Changes
no breaking changes
Testing Evidence
Editing a policy assignment parameter before the change:

Editing a policy assignment parameter after the change:

Please provide any testing evidence to show that your Pull Request works/fixes as described and planned (include screenshots, if appropriate).
As part of this Pull Request I have

[ ] Checked for duplicate Pull Requests
[ ] Associated it with relevant issues, for tracking and closure.
[ ] Ensured my code/branch is up-to-date with the latest changes in the main branch
[ ] Performed testing and provided evidence.
[ ] Updated relevant and associated documentation.
[ ] Updated the "What's New?" wiki page (located in the Enterprise-Scale repo in the directory: /docs/wiki/whats-new.md)

Closing as this has been resolved without using lifecycle blocks in:

#318


SDWebImageDownloader and canceled operations...

New Issue Checklist

[X] I have read and understood the CONTRIBUTING guide
[X] I have read the Documentation
[X] I have searched for a similar issue in the project and found none

Issue Info

Info
Value

Platform Name
e.g. ios

Platform Version
e.g. 10.0

SDWebImage Version
e.g. 4.2.2

Integration Method
e.g. cocoapods

Xcode Version
e.g. Xcode 9.1

Repro rate
e.g. sometimes

Repro with our demo prj
e.g. no

Demo project link
e.g. n/a

Issue Description and Steps
Our app uses a UICollectionView with SDWebImage loading image views in the collection view cells. We occasionally see the placeholder remaining when scrolling the cell in, out and then back into view. The same image would never load even when the collection view cell and image view would be recreated.
Note that we do call sd_cancelCurrentImageLoad when the cell disappears off screen.
After isolating/debugging we discovered that in the method:
- (nullable SDWebImageDownloadToken *)
    addProgressCallback:(SDWebImageDownloaderProgressBlock)progressBlock
    completedBlock:(SDWebImageDownloaderCompletedBlock)completedBlock
    forURL:(nullable NSURL *)url
    createCallback:(SDWebImageDownloaderOperation *(^)(void))createCallback

canceled operations were sometimes returned here:
dispatch_barrier_sync(self.barrierQueue, ^{
        SDWebImageDownloaderOperation *operation = self.URLOperations[url];
        if (!operation) {

This resulted in a growing callbackBlocks array for the operation that was never going to finish. To resolve, we added a check for finished/canceled operations:
dispatch_barrier_sync(self.barrierQueue, ^{
         SDWebImageDownloaderOperation *operation = self.URLOperations[url];
         if ((!operation) || (operation.isFinished) || (operation.isCancelled)) {

This allowed a new operation to be created for the URL. In our app this resolved the issue. I have this committed to a fork if it's something the maintainers would like (although that single line is the extent of it).
Thanks for your continued work on a great library!

josh jacob

It seems that our logic is wrong in some cases. This looks a easy fix but not the internal reason.
URLOperations is an array of operations. Each operations, when it be canceled or finished, it will removed from that array. See line 212 and line 243.
Could you please provide more information about what the case you use will  trigger this BUG ? Maybe we should review the logic more carefully and find a better way to fix this :)

That makes sense. This fix recovers from a flaw but doesn't prevent the flaw. I'll try to get some logging in place to see why a canceled operation would be left in the array. Thanks for the feedback!

josh jacob

@JoshJacobAtHanson Thanks for your enthusiasm. And I can also review the current implementation later. I hope that we can find something related and solve this issue.


Using socketlib in a macro

Hi,
This is less an issue and more a request for advice.
I'm trying to write a macro to allow a player to roll healing and automatically apply said healing to another player's actor without giving them the owner permission (for the other actor[s]) the system wants them to have.
I'm struggling with setting up socketlib, I was hoping I'd be able to set the socket up in the macro but I'm not sure that works as I'm getting socket is undefined when running the macro. Here's the relevant bits:
let socket;

Hooks.once("socketlib.ready", () => {
  socket = socketlib.registerModule("issie-healing-socket-module");
  socket.register("issieHealOrHarm", doHealing);
})

...

const doHealing = async (amount, victimActorId) => {
  const victim = game.actors.get(victimActorId);
  await victim.actor.applyDamage({ damage: amount, token: victim });
}

...

const foo = () => {
  await socket.executeAsGM("issieHarmOrHeal", -1 * finalAmount, victim.actor.id);
}

Do I need to create a module and then call an exposed method from that module in the macro instead? (And if so, do you have an example of what that would look like?)

Hi,
socketlib can only be used by modules and game systems, not by macros. The reason for this is that every client needs to register in socketlib to allow it to process messages sent by other clients. In macros, this cannot happen, because macros are only ever executed during gameplay. This means when your macro is being executed, the Hooks.once-event has already happened. So registering that hook no longer has any effect.
In general, your code looks fine (howevery victim is undefined in foo which will cause issues). All you need to do is to put that code into a module and to make the function foo accessible for macros (by attaching it to an appropriate entity, for example to the window, to an actor, etc.).
However, if you need to create a module anyway, it might be worthwhile to not do a macro but do a module that takes care of the functionality instead of exposing a function and then writing a macro that makes use of that function.

gotcha! Thanks for the advice, I've set up a basic module that exposes a function for the macro to run (in window.adam.funcname - might not be best practises but it works!)


[Bug]: Cannot import and use RadixVueResolver as per documentation instructions

Environment
Developement/Production OS: None
Node version: 18.2.1
Package manager<EMAIL_ADDRESS>Radix Vue version: 1.0.2
Vue version: 3.3.4
CSS framework: None
Client OS: Windows 11 Pro 64-Bit (10.0, Build 22621)
Browser: Chrome 90.0.4430.212

Link to minimal reproduction
https://stackblitz.com/edit/vue3-vite-typescript-starter-edvjlj?file=vite.config.ts
Steps to reproduce

Try to run pnpm dev
You will see an error in the console

Package subpath './resolver' is not defined by "exports" in /home/projects/vue3-vite-typescript-starter-edvjlj/node_modules/radix-vue/package.json

Describe the bug
This is actually a different error than I get locally, but still happens at the same time. I cannot use the unplugin-vue-component resolver on a fresh vite project.
I get this error locally:
ERROR: [plugin: externalize-deps] Failed to resolve "radix-vue/resolver". This package is ESM only but it was tried to load by 'require'. See http://vitejs.dev/guide/troubleshooting.html#this-package-is-esm-only for more details.

Expected behavior
No response
Conext & Screenshots (if applicable)
No response

@zernonia This problem still exists and can be reproduced by the link in the description.


Can you make it so I can save it as a pdf?

I want to be able to save them as a PDF instead of having to turn them all into one.

This is now possible using --save-as pdf


Lodash template

Changes

Update Lodash template function usage after updating to Lodash 3.6.

There was a breaking change to the _.template function in Lodash 3.0 (changelog):
// in 2.4.1
_.template(string, data, options); // → 'b'

// in 3.0.0
_.template(string, options)(data); // → 'b'

For review: @DoSomething/front-end

coool! Didn't know about that update :dancer:

Neither did I!


ee/contract-messages: use EntityAddr for messages

Replace AddressableEntityHash with EntityAddr in contract messages types.

bors merge

bors merge

bors single on

bors merge


Transition gradient not working

Describe the bug
Gradient transition does not work.
To Reproduce
I applied these styles in a button:
transition-all duration-250 ease-linear bg-gradient-to-r to-color-one via-color-one from-color-two bg-size-200 bg-pos-0 hover:bg-pos-100
And too setted these extension in my tailwind.config.ts
...
backgroundSize: {
 "size-200": "200% 200%",
},
backgroundPosition: {
 "pos-0": "0% 0%",
 "pos-100": "100% 100%",
},
...

The button has these a default style
inline-flex items-center justify-center whitespace-nowrap text-sm rounded-md font-medium ring-offset-background transition-colors focus-visible:outline-none focus-visible:ring-2 focus-visible:ring-ring focus-visible:ring-offset-2 disabled:pointer-events-none disabled:opacity-50
But the transition does not work.
Expected behavior
E.g. Without merge:
https://play.tailwindcss.com/XFQDCOKQ8L
Environment

tailwind-merge version: [e.g. 2.4.0]

I removed the merge and so worked.

Hey @tiagobbraga! 👋
tailwind-merge doesn't have access to the tailwind.config.ts file and you need to configure it separately so it knows about the text-button class.
Here is an example on how to configure tailwind-merge: https://github.com/dcastil/tailwind-merge/blob/v2.4.0/docs/recipes.md#adding-custom-scale-from-tailwind-config-to-tailwind-merge-config.
And here is the documentation on how the tailwind-merge configuration works: https://github.com/dcastil/tailwind-merge/blob/v2.4.0/docs/configuration.md#usage-with-custom-tailwind-config.

Related: https://github.com/dcastil/tailwind-merge/issues/368, https://github.com/dcastil/tailwind-merge/discussions/322, https://github.com/dcastil/tailwind-merge/issues/321, https://github.com/dcastil/tailwind-merge/discussions/315, https://github.com/dcastil/tailwind-merge/issues/302, https://github.com/dcastil/tailwind-merge/issues/276, https://github.com/dcastil/tailwind-merge/discussions/275, https://github.com/dcastil/tailwind-merge/issues/274, https://github.com/dcastil/tailwind-merge/issues/250, https://github.com/dcastil/tailwind-merge/issues/207

Thanks @dcastil!

No problem! 😊 Let me know if you get stuck somewhere and feel free to close this issue in case your problem is resolved.

Closing this one as resolved.


The notebook book_equations should be updated for HoML2

Some equations in the Kindle edition are not properly rendered, for example, 16-3 below. It would be very helpful for the book_equations notebook to be updated so Kindle readers can view the correct equations. Thanks.

Thanks for your feedback, and sorry for the late response, the last few months have been a bit hectic for me. These equation formatting issues on Kindle are really killing me. I've reported them to O'Reilly several times, but they're really hitting serious difficulties with Kindle formatting, reportedly because of the wide variety of kindle formats. I sincerely apologize for the inconvenience. :/
I'll try to update the book_equations notebook asap. Thanks for your patience. I hope you're enjoying the book despite these formatting issues.

I just uploaded a PDF containing all the equations in the book. Hope this helps.


[Feature Request]: course edition initially enabled

Describe the feature you'd like to see added
It would be great to start with the configurations used (course edition) for the last time. obviously it could be saved in OutRun2006Tweaks.ini
and there is no need to activate a checkbox in GUI/overlay

Added some stuff to write it to ini in https://github.com/emoose/OutRun2006Tweaks/actions/runs/12341007272, seems to work but haven't tested it that much yet, if that doesn't work for you let me know

The game doesn't open directly, I also tried the "outrun2006tweaks-a81c2c634d4328578fdf166a8c3a9cbfbc2fdd85" version and it didn't open either.
OutRun2006Tweaks.log

Apparently the problem is that the "OutRun2006Tweaks.overlay.ini" file was not present, I created it manually and the game started, by the way the track editing settings are saved correctly. good job!

Oh good catch, forgot to add error checking in the parser, just pushed https://github.com/emoose/OutRun2006Tweaks/actions/runs/12349633946 which should let it work without the ini existing

Actually these builds might break overlay while ingame, pushed new one at https://github.com/emoose/OutRun2006Tweaks/actions/runs/12349906945

reading error in the ini when you enter the game with [CourseReplacement] Code=0-0-0-0-0-0-0-0-0-0-0-0-0-0-0 Enabled=0  and I play a normal game and then I want to play with my custom courses, the courses will be edited alone in the GUI/overlay, meaning I cannot return to my saved configuration

It also doesn't let me apply my code since the checkbox is disabled, if I activate it my code will be lost.

Another error is that if you start the game without opening GUI/overlay (by pressing f11) the custom courses will not start even if you have "Enabled=1" in OutRun2006Tweaks.overlay.ini

I think this feature is already complete.
@nuexzz The bug that you reported might be fixed as I didn't experience it with this beta build below:
https://github.com/emoose/OutRun2006Tweaks/actions/runs/12494089238


[BEAM-7399] Blog post on looping timers

Beam 7399 - Blog post on looping timers, making use of State and Timer API for self setting timers.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Choose reviewer(s) and mention them in a comment (R: @username).
[x] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Post-Commit Tests Status (on master branch)

Lang
SDK
Apex
Dataflow
Flink
Gearpump
Samza
Spark

Go

---
---
---
---
---
---

Java

Python

---

---
---
---

Pre-Commit Tests Status (on master branch)

---
Java
Python
Go
Website

Non-portable

Portable
---

---
---

See .test-infra/jenkins/README for trigger phrase, status and link of all Jenkins jobs.

@kennknowles Blog PR

Staging link: http://apache-beam-website-pull-requests.storage.googleapis.com/8686/index.html

In the Jenkins console log you can see there is no upload of the file. So I expect if we merged it would, in fact, not publish.

@kennknowles fixed conflict and changed dates. Also when I make use of gradle to see the staged web, although this blog does not show up, I noticed the last blog in master also does not show up. Be interesting to see if on merge this magically appears....

Well, it has all the approvals needed, and we can debug based on the result. Let's go for it.


Workspace - The #announce and #admin rooms appears in LHN by default when a new workspace is created

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:
Precondition: Create a new expensifail account

Launch the app
Log in with new expensifail account
Tap on Plus button and tap Create a Workspace
On Workspace Tap on General Setting
Change the name and currency for Workspace
Tap Save button
Go back to LHN

Expected Result:
No default rooms get created
Actual Result:
The #announce and #admins rooms appears in LHN after created Workspace
Workaround:
Unknown
Platform:

Where is this issue occurring?

Web
iOS
Android
Desktop App
Mobile Web

Version Number:  <IP_ADDRESS>
Reproducible in staging?: Yes
Reproducible in production?: No
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos:

Expensify/Expensify Issue URL:
Issue reported by: Applause
Slack conversation:
View all open jobs on GitHub

:wave: Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open StagingDeployCash deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:

Identify the pull request that introduced this issue and revert it.
Find someone who can quickly fix the issue.
Fix the issue yourself.

Triggered auto assignment to @flodnv (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

Related to this issue https://github.com/Expensify/App/issues/5946#issuecomment-1021521363
Shouldn't #announce room be available in LHN when creating new workspace?

I agree @K4tsuki: Given https://github.com/Expensify/App/issues/5946 is closed, it looks to me like we now implemented the desired behavior. @neil-marcellini @yuwenmemon what do you say?

Correct :+1:


Update Composer dependencies (2024-09-08-00-12)

Visual regression test passed!
View the visual regression test report


Fix => DEPRECATION: InvalidError expects json-api formatted errors.

Need to write a function to properly format the errors for JSONAPI adapter

Is there an instance in the addon where errors are generated automatically (not using andFail)?
The reason I ask is because, for instance, this approach seems to be working fine on our side:
TestHelper.handleCreate('subscription').andFail({
  status: 422, response: {
    errors: {
      base: {
        param: "cvc",
        message: "Your card's security code is incorrect."
      }
    }
  }
});

InvokeError wraps this format (which we are getting from our AMS-based API as well) just fine.

Yes @begedin .. the problem is in testing only ( and is kind of annoying ) .. where I have to make tests for the different adapters that all look the same .. but the errors style is different for JSONAPI .. so it's not a big big deal, because the andFail method just sets status to 500, and the users normally enter their own errors in whatever format they need.

I reckon v 2.7.0-beta.9 solves my original problem because I am formatting automatically and giving the option to skip that formatting with convertErrors flag


Cannot reference variables when using if> operator when retry failed

overview
After setting the parameters using the if> operator, the following error will occur when retry failed is executed.
Failed to evaluate a variable ${test_env_store} (ReferenceError: "test_env_store" is not defined) (config)
I'd like to ask you if this is the expected result.
Thanks.
example
https://github.com/jon20/test-digdag
enviroment
Digdag: 0.10.4
Python: 3.7.9
Execution result
The screenshot below shows the result of the run.

The screenshot below shows the result of the retry failed. If you check on the UI, the +sample+load_environment^sub+do_statement task has disappeared

Hello, @jon20
I don't understand your requirements completely. But have you ever tried the following?
It seems that you don't need to use _export in that part.
+load_environment:
  if>: true
  _do:
    +do_statement:
      py>: sample.save

+test_task:
#  _export:
#    test_env: ${test_env_store}
  py>: sample.test_task

+test_task2:
  echo>: ${test_env_store}

import digdag

def save():
  digdag.env.store({'test_env_store': 'hoge'})

def test_task():
  print("test_env_store: %s\n" % digdag.env.params["test_env_store"])
#  raise Exception("Test Error")

2022-02-09 17:08:29 +0900 [INFO] (0017@[0:default]+sample+test_task): py>: sample.test_task
test_env_store: hoge

2022-02-09 17:08:30 +0900 [INFO] (0017@[0:default]+sample+test_task2): echo>: hoge
hoge
Success. Task state is saved at /private/tmp/test-digdag/.digdag/status/20220209T000000+0000 directory.

@hiroyuki-sato  Thanks for the workaround suggestion.
I didn't share the background of using the if operator, so I will share it.
I would like to set different parameters for local, development, staging, production environments to determine the docker image to be used for subsequent tasks. Also, we want to use docker to execute the task on different images. The implementation image is as follows
+load_environment:
  if>: ${environment == 'dev' || environment == 'stg' || environment == 'prd'}
  _do:
    +do_statement:
      _export:
        docker:
          image: ecr_image
          pull_always: true
      py>: sample.set_docker_env
  _else_do:
    +else_statemant:
      _export:
        docker:
          image: self_build_python_image
          pull_always: false
      py>: sample.set_another_docker_env

+test_task:
  _export:
    docker:
      image: ${docker_env.image}
      pull_always: ${docker_env.pull_always}
  py>: sample.test_task

Pass parameters when starting the digdag server
digdag server --param environment={dev, stg, prd, local}

In this implementation, the _export part cannot be commented out and the error shown in the title will occur

IIUC ${} doesn't evaluate in _export part.
It evaluates in each operator (i.e: sh> operator.)
These docs may help.
https://github.com/treasure-data/digdag/issues/990#issuecomment-463544061
https://github.com/treasure-data/digdag/issues/1203


feat: request more sources on click

Befoer this commit
SourceSelect dropdown would only display a maximum of 20 sources
After this commit
More sources can be displayed by clicking a button at the bottom of the list

Current dependencies on/for this PR:

next

PR #316   (this PR)

This comment was auto-generated by Graphite.


Build web form for build actions

Create an express app that generates a form with all of the fields from the available quasars and imports source files. The app should crawl the questions from each quasar (this requires that #5 be completed) and provide those to the form. The form should build a json file to be used when building out the quasar to file.
Use something like this react form generator that uses the json-schema for a form. (see the networknt solution)
UPDATE: Going to go with react-jsonschema-form as it seems the simplest solution and was updated a month ago.

Going to go with react-jsonschema-form as it seems the simplest solution and was updated a month ago.

react-jsonschema-form has been implemented. branch [feature#6] features a refactor of how quasar modules interact with the lib. this feature will also be coming with a web app, and API that runs on the same port, and an executable electron app.


[FLINK-26281][connectors/elasticsearch] Improve documentation

What is the purpose of the change
Improve the documentation of the Elasticsearch connector by removing deprecated attributes and adding further explanation.
Brief change log

Removed deprecated 'type' in ES7 examples
Added paragraph about exactly-once semantic using upserts

Verifying this change
This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

@flinkbot run azure

@flinkbot run azure

@flinkbot run azure

@flinkbot run azure

@alpreu Will you also create the backport to release-1.15?


Term Request: Radius

I would like to request / propose that the term Radius be added to the Quality Ontology in the next release. The proposed definition is constructed in the same manner as Circumference.
Radius = A One Dimensional Extend that inheres in a circle in virtue of the extend of a straight line that begins at the center of the circle and ends on the circle's boundary.
Definition source = Richard Rhoad, George Milauskas, and Robert Whipple. Geometry: for Enjoyment and Challenge. (Evanston, Illinois: McDougal Littell, 1991): 439.

@cameronmore Your definition uses "extend" rather than "extent" in two places.

@swartik fixed above, thank you


pull request for e2e test at 12/14/2016 05:15:04

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


Update continuous profiling E2E for adapt new query protocol

It's another part of the https://github.com/apache/skywalking/pull/10878 to verify the query

[ ] If this pull request closes/resolves/fixes an existing issue, replace the issue number. Closes #.
[ ] Update the CHANGES log.

There was just a protocol update again. Please sync that before the changes.

There was just a protocol update again. Please sync that before the changes.

Do you mean to sync the query protocol to the backend or CLI?

There was just a protocol update again. Please sync that before the changes.

Do you mean to sync the query protocol to the backend or CLI?

CLI part.


Add customreceipt as alternative to simplereceipt

Description
Add customReceipt component as an alternative to simpleReceipt when autodeleteonprocessend is enabled.
Related Issue(s)

closes #9548

Verification

Manual testing

[ ] I have tested these changes manually
[ ] Creator of the original issue (or service owner) has been contacted for manual testing
[ ] No testing done/necessary

Automated tests

[ ] Unit test(s) have been added
[ ] Cypress E2E test(s) have been added
[ ] No automatic tests are needed here
[ ] I want someone to help me make some tests

User documentation @ altinn-studio-docs

[ ] Has been updated

[ ] No changes/updates needed

Changes/additions to component properties

[ ] Changes are reflected in both src/layout/layout.d.ts and layout.schema.v1.json, and these are all backwards-compatible
[ ] No changes made

Support in Altinn Studio

[ ] Issue(s) created for support in Studio

[ ] This change/feature does not require any changes to Altinn Studio

Sprint board

[ ] The original issue (or this PR itself) has been added to the Team Apps project and to the current sprint board

Have you tested this manually?

I agree with @bjosttveit here, please fill out the checklist. Having automatic tests for this functionality would be preferable. While this looks simple enough from the small amount of code changes, there might be significant implications of changes like these.
Application metadata is fetched fairly early in the runtime, and is not dependent on having an instance. The available layouts relies on knowing which process task the instance is currently in, which makes it depend on having an instance available. When auto-deletion is on, we should assume there is no instance, making the current process state undefined, thereby making it impossible to determine which layouts should be loaded, so none will be available. In essence, my theory here is that the state we need to have available here relies on data we know we won't have - so this code should never even be able to execute.
And if it does work, that probably means that app-frontend is not (correctly) deleting its local state (form data, layouts) in Redux when moving to the next process step (ending up at the receipt). That means this functionality relies on side-effect, so a refactor in something like #345 will probably cause this functionality to break. If we knowingly paint ourselves in a corner like that, we have to introduce complexity in the future to make sure this feature still works.
A simple way to test this is to fill out a form (with auto deletion of instance data), get to the receipt page, make sure the instance data is deleted, and refresh the page. That will re-load the app/instance state from backend, so if that works (and better yet; we have a cypress test to prove it), I'd be very happy greenlighting this functionality.

Gjør denne om til draft siden det enda er uavklart


(feature): Add Angular DatePicker

Thank you for the pull request but the component was rewritten completely.


Daku Portals (App Permissions)

Daku portals are permissions to open message channels to communicate with specific I/O.
https://github.com/ardaku/daku/blob/stable/PORTALS.md#portals-list
Maybe a custom section called app-daku or similar can contain the portal information, plus any other information that might be needed.
Since portals are just numbers we should be able to have the section format be something like (in Rust-pseudo-code):
number_of_portals: u32,
portals: [u32; number_of_portals],

Storing the number of portals required by the app allows there to be future additions to the format of this custom section for Daku.

Completed in version 0.1.0


Remotely accessing a WiremockServer from a over-the-wire Wiremock client instance

Hi,
I am working on spring-boot application which would act as service-virtualization app, to be accessed remotely, containing a WireMock server within it.
Essentially, this spring boot is deployed as a Jar containing standalone-wiremock-server jar in it as dependency. The whole bunch is deployed in Pivotal Cloud Foundry.
When the spring-boot app bootstraps, it also bootstraps the wiremock server with it. Below is the code which does that,
	public WireMockServer getWireMockServer() {

	return new WireMockServer(new WireMockConfiguration().httpsPort(8085).port(8081).extensions(new ResponseTransformer() {

		@Override
		public Response transform(Request request, Response response, FileSource files, Parameters parameters) {

}
}

WiremockServer wiremockServer = getWireMockServer(); // above method
wiremockServer.start();
This way, the wiremock server is started and it is accessible from the same app by using Wiremock client instance,
WireMock.configureFor("https", "localhost", 8085);
stubFor(WireMock.post(...).willReturn(aResponse()));
The response is transformed at the runtime as part of ResponseTransformer().

This works fine when the Wiremock Client and Wiremock Server are present in the same application.
The requirement is, to extract out client from the server, so that both can talk remotely on PCF(Pivotal Cloud Foundry) as two separate Spring boot apps.
Lets call the client which contains Wiremock Instance as

springboot.client.app
and Server that contains standalone wiremock Server as
springboot.server.app

The server app bootstraps first,
When the client app bootstraps and tries to connect the server app as
WireMock.configureFor("https", "springboot.server.app", 8085);
stubFor(WireMock.post(...).willReturn(aResponse()));
I receive the below error
Connect to springboot.server.app.abc.pcfdev00.mastercard.int:8085 [springboot.server.app/<IP_ADDRESS>] failed: Connection refused (Connection refused)
The question is, same code works fine when both client and server are on the same application via localhost, however it does not work when client is accessing the server remotely.

Please post questions to the mailing list as per the README.

Posted on mailing list.
Need it urgently :(
Hence replying here to bring it to your notice.
https://groups.google.com/forum/#!topic/wiremock-user/PasKvtl6OTg


Fix bower.json main declaration

The main declaration in bower.json currently uses absolute paths for some reason, which breaks major bower tools such as main-bower-files.
This PR removes the initial path slash, as recommended in the bower.json spec.

+1

:+1:

:+1:
It breaks wiredep too, and probably every tool which conforms to the Bower specification.

This also seems to fix a couple of outstanding issues. Thank you!


Adding ac-polynesie.txt

Adding French polynesia academy (part of France's Education Ministry)
Official website : http://monvr.pf
Thank you

@Wa5p Pull request merged. Thank you!

@philipto Digital Education team thank you!


Fixed #27854 -- Make  warn (rather than blow up) on missing directories

Changes has been made to show warning (without stopping execution) if any of the static files directory does not exists.

@maheshjurel, do you plan to update this and a test?

Closing due to inactivity, please send a new PR if you complete it.


Satisfy like hooks for aemDeploy

Now aemSatisfy task has hooks called before and after package installation. It would be nice, that same API (including running remote code / groovy scripts, making http requests to instances) will become available in aemDeploy task.
aemDeploy {
    config {
         initializer = {}
         finalizer = {}
         completer = { await() }
    }
}

Covers #89 / alternative approach

@mmajchrzak thanks for suggesting it

not sure if it is required

added initializer and finalizer labdas that could be overridden if useful
to be released within 6.x


ApplyPolicy.Validation does not work with Relay node endpoint

Product
Hot Chocolate
Version
13.9.0
Link to minimal reproduction
https://github.com/BickelLukas/HotChocolateAuthorizeRepro
Steps to reproduce
Getting returns unauthorized when the user does not have access to one possible node, even if a different one is resolved.
# this works
query Books {
  books {
    id
    title
  }
}

# this doesn't work
query Authors {
  authors {
    id
    name
  }
}

# this should work but doesn't
query BookNode {
  node(id: "Qm9vawppMQ==") {
    id
    ... on Book {
      title
    }
  }
}

# this shouldn't work
query AuthorNode {
  node(id: "QXV0aG9yCmkx") {
    id
    ... on Author {
      name
    }
  }
}

What is expected?
I should be able to resolve a book via the node endpoint but not an author
What is actually happening?
Nothing is able to be resolved via the node endpoint
Relevant log output
No response
Additional context
No response

This is the correct behaviror for Validation ... as we cannot predict in this case what the entity will be ... you could still fetch each of the items ... try to use BeforeResolver instead as we can in this case use runtime evaluation.

@michaelstaib I understand your reasoning for endpoints returning Interfaces and that limitation is alright there.
But the node enpoint can return almost any type in the whole application and not being able to use ApplyPolicy.Validation on any type implementing node is a very hard limitation.
The Node enpoint knows which resolver to call based on the type in the HotChocolate.Types.Relay.IDValue struct. Couldn't the validation narrow down the possible result types to those returned by the resolver for this id?


Datasets

May I know how to get the datasets?

Sure, you can find them on Google drive. https://drive.google.com/file/d/14bdK0qd-AvLfOASjX2G-ErGtiSV4LPfH/view?usp=sharing

Sure, you can find them on Google drive. https://drive.google.com/file/d/14bdK0qd-AvLfOASjX2G-ErGtiSV4LPfH/view?usp=sharing


feat(go): runtime type casting

Adds an implementation map to all code generated getters and method
invocations to pass to the runtime. This allows the runtime to construct
concrete types when the return type of the function is a go interface.
Adds additional tests in jsii-calc-test module for various use cases.
Adds logic to correctly deserialize and serialize object references
coming from both the host and kernel.
fix: #2047

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-5lHf64IXfvmr
Commit ID: 8ba6aa90e7e45743da5deac70bb05ccabe58a403
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-5lHf64IXfvmr
Commit ID: 3bc1957454121c5cad3b9933be2b978742626024
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-5lHf64IXfvmr
Commit ID: 71c09565d2a6e3fd4dfa52b8cce6c9aff83bea8c
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-5lHf64IXfvmr
Commit ID: 0d7722c424766427d1b5ea112dff5c36e04feece
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


Animations: Added duration-based ruler to animation timeline.

Summary
Adds a duration-based time ruler to the animation timeline. The timeline ruler and left-column labels are sticky when the main content scrolls either vertically or horizontally. (Shown in GIF).

Relevant Technical Choices

Uses SVG to draw the ruler, tick marks, and labels
Uses milliseconds as the unit

User-facing changes

Adds a time-based ruler to the animation timeline control (Storybook only)

Testing Instructions

Open Storybook and see the time-based ruler on the animation timeline

#3440

@mariano-formidable Fixed that. I'm not exactly sure why I confused milliseconds and deciseconds.


2 suggestions for improvement

Can you please add the "Add to Git" command to just add the file to be tracked by Git? Basically: git add [filename]
Can you add all commands also to the context menu when clicked on the filename in the open editor. That is, when you right click the filename in the tab of the editor (and not just in the explorer)

This is staging the file. That's what add does.
I am not sure where that config option would be. Feel free to do a PR for it.


🛡 Use projected ServiceAccount token for gardenlet

Only in case Kubernetes is >= 1.20 since only then the RootCAConfigMap feature gate is enabled by default
How to categorize this PR?

/area security
/kind enhancement
What this PR does / why we need it:
The gardenlet was switched to use a projected ServiceAccount token for accessing the seed cluster's API server in case Kubernetes >= 1.20 (since only then the RootCAConfigMap feature gate is enabled by default). Note that the gardenlet deployment cannot rely on the "projected token mount" webhook (unlike many other deployments in the seed) since it is the first component running in the seed cluster.
Which issue(s) this PR fixes:
Part of #4878
Release note:

In case `gardenlet` runs on a cluster with at least Kubernetes 1.20 then it will use a projected `ServiceAccount` token only valid for `12h` for communicating with the (seed) cluster's API Server.

/assign

/invite @BeckerMax @timebertt


TIKA-4137

This PR omits the  element if there is no title.  We may want to insert a space instead?
Thanks for your contribution to Apache Tika! Your help is appreciated!
Before opening the pull request, please verify that

there is an open issue on the Tika issue tracker which describes the problem or the improvement. We cannot accept pull requests without an issue because the change wouldn't be listed in the release notes.
the issue ID (TIKA-XXXX)

is referenced in the title of the pull request
and placed in front of your commit messages surrounded by square brackets ([TIKA-XXXX] Issue or pull request title)

commits are squashed into a single one (or few commits for larger changes)
Tika is successfully built and unit tests pass by running mvn clean test
there should be no conflicts when merging the pull request branch into the recent main branch. If there are conflicts, please try to rebase the pull request branch on top of a freshly pulled main branch
if you add new module that downstream users will depend upon add it to relevant group in tika-bom/pom.xml.

We will be able to faster integrate your pull request if these conditions are met. If you have any questions how to fix your problem or about using Tika in general, please sign up for the Tika mailing list. Thanks!

Keeping this as draft to allow for input on whether we want to:

simply not output the title element if a document has no title (current PR)
add a space to an empty title element

@theit solved this more elegantly


Ponyc installed from RPM defaults to gcc-5 when linking

I've installed ponyc from rpm repo (Bintray) and it gives an error while linking
sh: gcc-5: command not found
Error:
unable to link: gcc-5 -o ./alternation -O3 -march=native -mcx16 -fuse-ld=gold  ./alternation.o -L"/usr/lib/pony/0.3.1-1885.8a8ee28/bin/" -Wl,-rpath,"/usr/lib/pony/0.3.1-1885.8a8ee28/bin/" -L"/usr/lib/pony/0.3.1-1885.8a8ee28/bin/../lib" -Wl,-rpath,"/usr/lib/pony/0.3.1-1885.8a8ee28/bin/../lib" -L"/usr/lib/pony/0.3.1-1885.8a8ee28/bin/../packages" -Wl,-rpath,"/usr/lib/pony/0.3.1-1885.8a8ee28/bin/../packages" -L"/usr/local/lib" -Wl,-rpath,"/usr/local/lib" -Wl,--start-group -Wl,--end-group  -lponyrt -lpthread -ldl   -lm
If I copy that command and run with gcc instead of gcc-5 it works fine
I am using Fedora24 and it comes with gcc 6.1 installed and gcc-5 doesn't exist.
I've tried an alias but it doesn't accept.
However it works if I put a symlink.

@killerswan would this be an artifact of our process for building?

I've tested some example programs in examples directory. Most of them looks fine but files and httpget doesn't work. (Maybe symlink isn't a correct solution)
I am not sure if I am using httpget correctly but I've checked files example and both the code and file looks fine
They both print
Illegal instruction (core dumped)
I am not sure what is the reason but I thought it could be because of this issue

@Praetonus suggested on the sync call to use the environment variable CC to determine the default linker to use when ponyc is run.  @sylvanc and I agree.
I think it's definitely a good idea to not assume that the environment when running ponyc is the same as the environment when building ponyc, which is not often true now that we are regularly distributing releases.

Is there an issue to open from this?

@SeanTAllen - I think this can serve as the ticket.  I've renamed the ticket title to specify the general problem.

I'm not sure how to handle some details here. What should we do if CC is undefined? On Windows, the linking fails if there is no environment variable for the linker (genexe.c:311), would the same behaviour on other platforms be acceptable?

@Praetonus - I think a reasonable path forward would be:

if CC is set, use it, otherwise:
(the current behaviour)

This should keep things backwards compatible for folks who happen to be running ponyc on platforms without CC set.

@jemc That seems reasonable. I think I'll also add a warning issued when ponyc falls back on PONY_COMPILER, to make sure users are aware of that behaviour.

This is a high priority item which means release. I'll handle this release.


charge API endpoint updates

addresses #23
@nberardi  ready for code review
@paulirwin  ready for code review

The CreateChargeGeneric does not seem to benefit from the use of generics here since it's purpose is just to call typeof(T), which is an anti-pattern. I'm going to look at refactoring this a different way.


Pick bufnr function, to allow selecting which split to open a selected file in

vim or neovim?

[ ] vim
[x] neovim

Output of vim --version or nvim --version:

NVIM v0.9.2
Build type: Release
LuaJIT 2.1.1692716794

   system vimrc file: "$VIM/sysinit.vim"
  fall-back for $VIM: "/usr/local/share/nvim"

Run :checkhealth for more info

Output of :echo has("python"): 0
Output of :echo has("python3"): 1
Output of :py print(sys.version):

3.7.16 (default, Aug 30 2023, 20:37:53)
[GCC 7.3.1 20180712 (Red Hat 7.3.1-15)]

Output of :py3 print(sys.version):

3.7.16 (default, Aug 30 2023, 20:37:53)
[GCC 7.3.1 20180712 (Red Hat 7.3.1-15)]

Operating system:

[x] Linux
[x] Mac OS X
[ ] Windows
[ ] Etc.

Configurations related to LeaderF in vimrc:

Describe your question, feature request, or bug.
LeaderF will always open the specified file in the least recently focused buffer (the buffer that you previously edited before triggering leaderf).
I would like to be able to select which buffer the file is opened in, similar to what I can achieve with nvim-tree and window-picker:

The green bars below each split is telling me which key to press to open the file in this split.
As far as I understand, LeaderF would need to provide a "callback" that's called before opening the file. The pick() function from window-picker will return a bufnr in which the new file is opened.

There is also vim-choosewin: https://github.com/t9md/vim-choosewin


[spaceship] allow to specify fetched fields

Checklist

[x] I've run bundle exec rspec from the root directory to see all new and existing tests pass
[x] I've followed the fastlane code style and run bundle exec rubocop -a to ensure the code style is valid
[x] I see several green ci/circleci builds in the "All checks have passed" section of my PR (connect CircleCI to GitHub if not)
[x] I've read the Contribution Guidelines
[x] I've updated the documentation if necessary.
[x] I've added or updated relevant unit tests.

Motivation and Context
Currently, I'm working on match optimization. I've figured out that match fetches certificates with all fields, including enormous certificateContent and csrContent, and match doesn't need them.
A reduced amount of fetched fields leads to faster response generation and faster transmission to the client.
Description
Changes are really small, I just added a fields parameter to the api_client, certificate, device, bundle id and profile.
Testing Steps
Spaceship::ConnectAPI::Device.all(fields: 'platform,status')

Hey @nekrich, sorry for hijacking your PR, did you look at 'limit' parameter for API requests? Looks like it defaults to 20 responses per page, and max limit is 200 (? I believe I saw it on some Apple website...), but I can't find a way to set it manually... That would speed up things for accounts with many items 🤞

@Chupaka thanks for suggestion, I'll look into it too. Reducing requests count is awesome.
But even with bigger limit, there is an issue with certificates which include excessive data match and sigh don't really need. And it spreads to the profiles when we fetch them with certificates. Plus I don't think (didn't test it in practice) that certificateContent and csrContent will have a good compression ratio while transferring.

Thanks for the PR, @nekrich. Looks great!


New commands  won't work if there are more than 5 commands

As above. In my https://github.com/MilkyPL/skelet2bot only first 5 commands work, while later ones are "ignored". There is no error message or anything, they just don't work.

UPDATE: after switching command order (command5<=>command6) the command6 will work, but command5 will not.
Sent from my Htc HTC Desire 820 using FastHub

UPDATE: this is issue with node-cron. Any command after this won't work:
bot.all(function (msg, reply, next) {     cron.schedule("37 21 * * *", function() {         reply.text("testing cron");     }); });

It seems your code is using the Telegraf library rather than this one?
If that's the case, you should probably report the issue there. :)
I took a quick look at your code and it seems okay but I'm not very familiar with Telegraf so maybe there's something I'm missing. Also I understand the confusion, both Botgram and Telegraf have turnt out with similar APIs. 😹
By the way, your bot looks interesting!

(cleanup)


maint: support python 3.8-3.12 and windows

Update, does not work with poetry.
Alt approach: https://github.com/astrogilda/tsbootstrap/pull/60 (3.8, 3.9) and https://github.com/astrogilda/tsbootstrap/pull/57 (windows)

This PR adds support for 3.8 and 3.12 and windows, via:

bounds and tags in pyproject.toml
extending ranges of tests

Depends on https://github.com/astrogilda/tsbootstrap/pull/54 for dependency isolation/reduction.

hm, windows does not work with the current CI element (admittedly complicated), so we should probably replace it with one which works for all envs.

Main obstacle:
it seems not possible to achieve pip install tsbootstrap[dev] with poetry.
Imo that is an obstacle in two ways:

ease of install - pip is more widely used
windows tests. I know how to cover widows easily via pip, but poetry setup seems fiddly and harder to maintain

superseded by https://github.com/astrogilda/tsbootstrap/pull/60


APT whitelist request for libjava-batik

Batik is a toolkit for applications or applets that want to use images in the Scalable Vector Graphics (SVG) format for various purposes, such as viewing, generation or manipulation.

Mixed up the package name, it should have been libbatik-java. #1528


Add quality build tag to build

This will help discovering builds of a particular type from our product build pipeline. I was originally thinking we'd just create separate pipelines for different types (such as the saw one I created) but that just gets messy and means we might miss updating all the pipelines with configuration changes.
Example build : https://mssqltools.visualstudio.com/CrossPlatBuildScripts/_build/results?buildId=58427&view=results

Thank you for this!


How to handle USB support?

It looks like the current manifest has minSdkVersion set to 12 (Android 3.1), due to the USB support.
<uses-sdk android:minSdkVersion="12"/>

If an app doesn't need USB support, the minSdkVersion can probably go back to 8 (Android 2.2), but we need to figure out how to make that work.  Maybe a different branch for USB support so master can support as many Android phones as possible over bluetooth?  Another possibility is adding gradle support to enable/disable USB within the gradle config files, but that option, of course, requires integrating gradle into the project, which is probably a different discussion.
Here's the Android documentation for USB Accessory, which says USB support is back-ported to sdk 10 (Android 2.3.4) using add-on libraries.
Any other ideas for how to handle USB support while still supporting as many phones as possible over bluetooth?

It's multiplexed now. #768


[Enhancement] [Cherry-Pick] Support TPCH Benchmark for MV (#18506)

What type of PR is this：

[ ] BugFix
[ ] Feature
[x] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Which issues of this PR fixes ：

Fixes #
Problem Summary(Required) ：

Checklist:

[x] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto backported to target branch

[ ] 3.0
[ ] 2.5
[ ] 2.4
[ ] 2.3

run starrocks_fe_unittest


Object opacity

What feature do you want to get added on the base engine?
Its simple, An object opacity function
something like setObjectOpacity(Object, Opacity, Time to fade to opacity);
To test your sight, and reliability, please select the option of what should NOT be requested.
Useful feature

ah I fell for the last one, haha!

Try reading the wiki

Try reading the wiki

I did a control F of it under the object sections and got no results, can you tell me what it is if I missed it?

Tweens section
https://github.com/ShadowMario/FNF-PsychEngine/wiki/Lua-Script-API:-Tweens-and-Timers

Tweens section
https://github.com/ShadowMario/FNF-PsychEngine/wiki/Lua-Script-API:-Tweens-and-Timers

Ah, right


create a forum thread only when someone comments.

related threads:
https://meta.discourse.org/t/embedding-but-dont-automatically-create-a-topic-until-a-user-chooses-to-comment/61450/54
https://meta.discourse.org/t/paid-preference-to-create-embed-a-topic-via-javascript-only-when-a-user-clicks-the-start-discussion-link/78593/6

See https://github.com/freeCodeCamp/freeCodeCamp/issues/36766


Problem: Restoring a project does not restore a tasks survey variables properly.

Issue
After you restore a project, no survey variables for any task have been restored.
Impact
Semaphore Project
Installation method
Docker
Database
Postgres
Browser
No response
Semaphore Version
2.10.11
Ansible Version
No response
Logs & errors
No response
Manual installation - system information
No response
Configuration
No response
Additional information
No response

It's because the API /project/1/templates does not pull the survey variables, JSON from that API endpoint contains "survey_vars": null,

Our current workaround is to modify by hand the json of exported projects by changing "survey_vars" key values to be JSON instead of stringified JSON and replacing nulls with empty arrays [].
Looking forward to a real fix!


Update header.php

Js files for amp-iframe & amp-audio only required for post/page

Thank you so much! I will review the code in the next week's release cycle and include this in the update with the credit.

Correct me if I'm wrong, but this code is no longer needed because the plugin uses Automatic's AMP plugin as a backing.

I'm closing this pull request as @nicholasgriffintn said we have changed the file structure after the latest update and this code is no longer used.
But Thank you for your contribution.
Mohammed Kaludi


backslash in JSON loader files not properly escaped by cJSON library

Moved from gitlab 31
"One thing I did notice that doesn’t work is having a path with a backslash in it. There’s a bug in the JSON parser that prevents this. Basically, when the JSON string values are parsed, backslashes are escapes, which means that single backslashes are merged with the next character as an escape code and are therefore eaten. However, if you escape the backslash (put a double backslash) in the string, it properly unescapes it, but then before returning the string, it re-escapes everything, turning the (now escaped) double backslash into four backslashes. Therefore, it’s physically impossible to have a string with a single backslash in it with the JSON parser you’re using. It just happens to work for paths on Windows because the filesystem folds consecutive backslashes together. But really, that’s just luck. You might want to fix that too."

Return strings now fixed with commit 31dbc350191fc2f5d86e48b00fa9c4eeb4e842ce


Update win32-physicalmemory.md

I ran wmic MemoryChip get SMBIOSMemoryType
and got 26, and the RAM is DDR4 and 26 was no where to see, so I added it.
Also, what are these "May not be available; see note above." and where's the note? huh?

@karl-police : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@stevewhims : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@stevewhims : Thanks for your contribution! The author(s) have been notified to review your proposed change.

I wonder what's with the "wmic" command though. Hmmm


Docs(Popup): Added example to use delay with Popup.

Added new example to use show/hide delay with the Popup component. This example showcases how to avoid accidental popup visibility.

Codecov Report

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

@@           Coverage Diff           @@
##           master    #3754   +/-   ##
=======================================
  Coverage   99.84%   99.84%
=======================================
  Files         175      175
  Lines        3125     3125
=======================================
  Hits         3120     3120
  Misses          5        5

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 ea950cb...08272f9. Read the comment docs.


Take settings in constructor, and stop using resin-settings-client

See https://github.com/resin-io/resin-sdk/pull/226#issuecomment-296626478

Once implemented we should also drop this piece of code https://github.com/resin-io/resin-sdk/pull/226/files#diff-a14896ddcadbe67fdbefec1a538a08eaR54 and use settings.get('dashboardUrl')

But we should also carefully check how it works when the apiUrl is set to custom value


Polish protocol handler flow when VS Code is not installed

This PR fixes https://github.com/microsoft/vscode/issues/159328

Cool 👍


Chimoney Project Manager Hiring Project Challenge

Objective:
The objective of this project challenge is to assess the suitability of the candidate for the Project Manager position at Chimoney. This challenge will evaluate the candidate's attention to detail, critical thinking, ability to use project management tools effectively, understanding of GitHub and development workflows, perseverance, stakeholder management, ambition, user-centered product development process, growth hacks, security and privacy considerations in fintech, understanding of payments and fintechs, alignment with Chimoney's values, and ability to follow instructions while conducting research within the fintech and payment infrastructure space.
Timeline:
The candidate will have 2-7 days to complete the challenge. However, the start date is flexible. Add @uchibeke to the PM tool you're using to follow along with the progress.
Challenge Tasks:

Project Planning and Tool Utilization:

Utilize project management tools such as ClickUp, Jira, or equivalent to create a comprehensive project plan for the development of a new feature or product enhancement. This could be an enhancement of Chimoney's infrastructure or a product that uses Chimoney's API.
Define project objectives, scope, deliverables, timelines, and resource requirements.
Develop a sprint plan outlining tasks, dependencies, and assignments for the development team.
Familiarize yourself with Chimoney's API, GitHub repository and CI/CD flow to understand the development workflow.

Stakeholder Management and User-Centered Design:

Identify key stakeholders involved in the project, including internal team members, developers, designers, and external partners.
Develop a stakeholder communication plan outlining regular updates, progress reports, and feedback sessions.
Conduct user research to gather insights into user needs, preferences, and pain points related to the new feature or product enhancement and justify all product decisions with data.
Develop user stories or personas to guide the development process and ensure a user-centered approach.

Execution and Implementation:

Collaborate with developers to ensure smooth execution of tasks and resolve any blockers or issues that arise during the sprint.
Monitor progress against the sprint plan and adjust priorities or resource allocations as needed to stay on track.
Implement growth hacks or innovative strategies to accelerate project delivery and maximize impact.
Ensure adherence to security and privacy best practices throughout the development process.

Documentation and Reporting:

Prepare weekly progress reports highlighting key accomplishments, challenges, and action items for the project team and stakeholders. The timing of the Reports could be compressed to multiple reports/day or a set of 5 reports.
Document any lessons learned or best practices identified during the project for future reference.
Review and analyze project metrics and KPIs to assess performance and identify areas for improvement.
Ensure alignment with Chimoney's values of excellence, speed, and impact in all project activities.

Submission Requirements:

A comprehensive project plan outlining objectives, scope, deliverables, timelines, and resource requirements.
Sprint plan detailing tasks, dependencies, and assignments for the fictitious development team.
Stakeholder communication plan outlining regular updates, progress reports, and feedback sessions.
User research findings, including insights, user stories, or personas developed.
Weekly progress reports highlighting key accomplishments, challenges, and action items.
Analysis of project metrics and KPIs to assess performance and identify areas for improvement.

Evaluation Criteria:

Attention to detail in project planning, execution, and documentation.
Critical thinking and problem-solving ability in addressing project challenges and opportunities.
Proficiency in using project management tools effectively, including ClickUp, Jira, GitHub, and CI/CD flow.
Understanding of stakeholder management principles and user-centered design methodologies.
Ambition and ability to think fast and adapt to changing priorities and requirements.
Alignment with Chimoney's values of excellence, speed, and impact in project execution.
Demonstrated ability to follow instructions, conduct research, and implement growth hacks within the fintech and payment infrastructure space.

Note: The candidate will be evaluated based on the quality of their submission, as well as their ability to meet the outlined criteria within the given timeline. Additionally, adherence to instructions and attention to detail will be key factors in the evaluation process.
Optional: If needed, request approval for Payment for your time by emailing Chimoney prior to starting this Challenge and include your Portfolio and Resume/CV with this request.

Hello @uchibeke ,
I came across your job advert on X (Formerly Twitter) and I have read the requirements. I wanted to inform you of my interest in taking up this assessment to showcase my skills. However, the timeline states 2-7 days and would like to indicate that my 7th day for this assessment is Monday, 11th March 2024.
Thank you.
Best Regards

Hello @uchibeke ,
I came across your job advert on X (Formerly Twitter) and I have read the requirements. I wanted to inform you of my interest in taking up this assessment to showcase my skills. However, the timeline states 2-7 days and would like to indicate that my 7th day for this assessment is Monday, 11th March 2024.
Thank you. Best Regards

Yes. Sounds good.

Hey @uchibeke can i get your email address, as I'll be using notion for the documentation for this challenge

uchi @ chimoney

Hello @uchibeke, I'd like to know if this is a project or product management role, as this clarification will be helpful.
Also, please provide the email address for payment approval.
Thanks.

Hello @uchibeke, I'd like to know if this is a project or product management role, as this clarification will be helpful. Also, please provide the email address for payment approval. Thanks.

Product

Hello @uchibeke, I'd like to know if this is a project or product management role, as this clarification will be helpful. Also, please provide the email address for payment approval. Thanks.

Email is above

Hey @uchibeke can i get your email address, as I'll be using notion for the documentation for this challenge

uchi @ chimoney

Good day Uchi, I sent you a mail with my response to this challenge

Hi @uchibeke,
I sent you a mail indicating my interest in the challenge last week.
To follow along my progress, I've invited you to my ongoing project. As mentioned in my mail, final submission will be Thursday 14th March latest at 11:59 EST.
Kind regards.


modify response(Bad Gateway:xxxxxxx)

`private boolean respondWithShortCircuitResponse(HttpResponse httpResponse) {
// we are sending a response to the client, so we are done handling this request
this.currentRequest = null;
    HttpResponse filteredResponse = (HttpResponse) currentFilters.proxyToClientResponse(httpResponse);
    if (filteredResponse == null) {
        disconnect();
        return false;
    }

    // allow short-circuit messages to close the connection. normally the Connection header would be stripped when modifying
    // the message for proxying, so save the keep-alive status before the modifications are made.
    boolean isKeepAlive = HttpHeaders.isKeepAlive(httpResponse);

    // if the response is not a Bad Gateway or Gateway Timeout, modify the headers "as if" the short-circuit response were proxied
    int statusCode = httpResponse.getStatus().code();
    if (statusCode != HttpResponseStatus.BAD_GATEWAY.code() && statusCode != HttpResponseStatus.GATEWAY_TIMEOUT.code()) {
        modifyResponseHeadersToReflectProxying(httpResponse);
    }

    // restore the keep alive status, if it was overwritten when modifying headers for proxying
    HttpHeaders.setKeepAlive(httpResponse, isKeepAlive);

    write(httpResponse);

    if (ProxyUtils.isLastChunk(httpResponse)) {
        writeEmptyBuffer();
    }

    if (!HttpHeaders.isKeepAlive(httpResponse)) {
        disconnect();
        return false;
    }

    return true;
}`

HttpResponse filteredResponse = (HttpResponse) currentFilters.proxyToClientResponse(httpResponse);
I try to use filteredResponse repeat httpResponse, but this not support; why not use filteredResponse to write????

write(filteredResponse);

Fix proposed in #383 pull request


What are the plans for body parsing?

I don't know the correct forum for addressing this concern.  So please point me in the right direction if this is not.
According to the changelog for 4.16, express.json() has been added whereas before it was not available.
See here:
http://expressjs.com/en/changelog/4x.html#4.16.0

The express.json() and express.urlencoded() middleware have been added to provide request body parsing support out-of-the-box. This uses the expressjs/body-parser module module underneath, so apps that are currently requiring the module separately can switch to the built-in parsers.

However after upgrading to
"express": "^5.0.0-alpha.2",

I can no longer use the built it express.json().

Error: Most middleware (like urlencoded) is no longer bundled with Express and must be installed separately. Please see https://github.com/senchalabs/connect#middleware.

I'm trying to understand the back and forth and if there is going to be a final choice on whether or not to include it.

Hi @caaker ! You can find the changelog for Express 5.0.0-alpha.2 here: https://github.com/expressjs/express/releases/tag/5.0.0-alpha.2 That version is based on Express 4.13.1 so wouldn't include anything added to Express 4.x after that release. As soon as there is a new Express 5.0 alpha based on 4.16.0 or higher, that change will be within the 5.0.0 alpha series. I hope that helps!

It sound to me like you are saying the current alpha is based on 4.13.1 and that as it moves forward it will be based on 4.16 or higher which will once again add body parsing back into Express core.
Is this a summary of what you are saying?
I'm asking b.c. I have installed body parsing middleware, upgraded, removed body parsing middleware, upgraded, added body parsing middleware.
And if I understand you correctly, once it is put back in again.  I will need to upgrade and remove it again.
Not something I want to spend time doing necessarily.
My question, is what is the end goal, to have body parsing as part of the Express core or to have it as middleware that is installed.
Sorry if my question/concern was unclear or overly verbose.

5.0.0 alpha 2 was release in 2015 but 4.16 was released in 2017. Unfortunately we cannot retroactively re-release older versions.

The current alpha is 5.0.0 alpha 6

Body parsing has been added to Express in 4.16 and will be in the 5.0 alpha that is based on 4.16 or newer. If you install an alpha that was released before 4.16 existed, it won't have anything that was added to Express after that, since we cannot republish old versions with newer code. I hope that makes sense. The change log for each 5.0 alpha should be clear upon what 4.x version that alpha is based upon. Let us know what we can change to provide better clarity.

Hmmm.... Yes.  I assumed that any version 5.* came after version 4.* which is not the case.  Thanks!  Also I'm not sure how the improper alpha was installed.  I think to be safe I'm just going to use latest non-alpha release which appears to be 4.16 as it is mentioned on the main page.  Thanks again!
main page

Gotcha. So like many large projects, there are multiple major version in active develop in parallel. What happens is new features are added first to the "mainline" version which is 4.x in this case and then ported to the development (or maintenance) versions. 5.x is the development version so it gets thr code line frlm 4.x merged into 5.0 after a 4.x release. They can sometime lag because alpha is for 5.0 code preview. Once they become betas or especially rc to stable both majors would have releases the same day to keep in step. I hope this makes sense.

This makes sense.  Hmm ... it seems I got the install directions directly from Migrating to Express 5 which actually has you migrate to something created in 2015 ( per your comment ).  It  seems to incorrectly state that you are updating to the latest alpha ...

To install the latest alpha and to preview Express 5, enter the following command in your application root directory:

$ npm install<EMAIL_ADDRESS>--save
I think it was this line that caused all the back and forth.  Is there a way to update it.  I.e is there a way to tell npm to install the latest alpha?  I'm not sure if npm has this option or if the comment above the command needs to be modified.

Ah, I didn't even know that page existed, sorry for any confusion it caused! I'm sure it was accurate whenever it was written. You're welcome to suggest an edit by clicking on the "Edit this page on GitHub." link at the bottom of the page, or you can even just open an issue at https://github.com/expressjs/expressjs.com for someone else to look into.
I'm not sure if there is an npm command to just install the latest of an alpha -- if you know one, though, we'd love to get the docs updated :) !

npm publish --tag=alpha
Then people can do npm install express@alpha.  Does that help?

Looks like this worked:
$ npm i<EMAIL_ADDRESS>+<EMAIL_ADDRESS>added 47 packages in 3.154s

Yep, that would work as well I think.  I am not sure how semver orders pre versions like this.  Like if you had 5.0.0-beta.1 would that be greater than or less than alpha?  Probably doesn't matter in this case, just wondering if anyone knows.

Not 100% sure, but it might. I tried npm i<EMAIL_ADDRESS>and it installed 4.16 so it does seem to include everything above it.

Can someone approve this pull request to update the documentation.  It seems to be an improvement.

Done, @caaker :+1:

Oops.  I accidentally provided the wrong link above to my personal forked version.   The pull request to express.com is below.  That should be correct.  I'm still getting use to the Github GUI.
Thanks again!
pull request

Oh, I didn't even notice the difference, haha. I just commented on it, thanks!


Support for MongoDB Java Driver Native Codecs

Closes #28
Summary:

Serialization API changed to match MongoDB Java Driver API, so that native codecs from the driver can be supported
Removed support for JodaTime (can be re-supported later)

Codecov Report

Merging #32 into master will decrease coverage by 4.14%.
The diff coverage is 67.55%.

@@             Coverage Diff              @@
##             master      #32      +/-   ##
============================================
- Coverage      80.6%   76.46%   -4.15%
+ Complexity      619      608      -11
============================================
  Files            55       66      +11
  Lines          1676     1755      +79
  Branches        167      167
============================================
- Hits           1351     1342       -9
- Misses          206      285      +79
- Partials        119      128       +9

Impacted Files
Coverage Δ
Complexity Δ

src/org/smof/parsers/ObjectParser.java
80.23% <100%> (+0.11%)
56 <2> (+1)
:arrow_up:

...rc/org/smof/bson/codecs/date/JavaInstantCodec.java
100% <100%> (ø)
4 <4> (?)

src/org/smof/parsers/ByteParser.java
41.66% <100%> (-34.34%)
14 <1> (-14)

.../smof/bson/codecs/date/JavaLocalDateTimeCodec.java
100% <100%> (ø)
4 <4> (?)

src/org/smof/collection/Smof.java
74.32% <100%> (-0.68%)
19 <9> (-2)

.../org/smof/bson/codecs/date/JavaLocalDateCodec.java
100% <100%> (ø)
4 <4> (?)

src/org/smof/parsers/BooleanParser.java
85.71% <100%> (ø)
12 <1> (ø)
:arrow_down:

src/org/smof/parsers/NumberParser.java
68.62% <100%> (ø)
30 <2> (ø)
:arrow_down:

src/org/smof/parsers/ObjectIdParser.java
83.33% <100%> (+3.33%)
5 <2> (+1)
:arrow_up:

src/org/smof/parsers/StringParser.java
22.22% <17.64%> (-37.16%)
5 <5> (-5)

... and 28 more

Continue to review full report at Codecov.

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


can i use it in macos?

when i complie it in the macos it shows:
Unzipping /Users/techgules/.gradle/wrapper/dists/gradle-2.14.1-bin/ev4hc1s4g1oc9yuae0b81rdza/gradle-2.14.1-bin.zip to /Users/techgules/.gradle/wrapper/dists/gradle-2.14.1-bin/ev4hc1s4g1oc9yuae0b81rdza
Exception in thread "main" java.util.zip.ZipException: error in opening zip file
at java.util.zip.ZipFile.open(Native Method)
at java.util.zip.ZipFile.(ZipFile.java:215)
at java.util.zip.ZipFile.(ZipFile.java:145)
at java.util.zip.ZipFile.(ZipFile.java:159)
at org.gradle.wrapper.Install.unzip(Install.java:159)
at org.gradle.wrapper.Install.access$500(Install.java:26)
at org.gradle.wrapper.Install$1.call(Install.java:69)
at org.gradle.wrapper.Install$1.call(Install.java:46)
at org.gradle.wrapper.ExclusiveFileAccessManager.access(ExclusiveFileAccessManager.java:65)
at org.gradle.wrapper.Install.createDist(Install.java:46)
at org.gradle.wrapper.WrapperExecutor.execute(WrapperExecutor.java:126)
at org.gradle.wrapper.GradleWrapperMain.main(GradleWrapperMain.java:61)
how could i solve it?if you could give me some suggestion,i will appricate you very much.

It looks like your version of Gradle is pretty old. Update it with a current version and try again.

thank you very much! i download a zip that complied completely ,and i can use it normally.


Not working in 1.39.0-insider

In 1.39.0-insider SFTP is not working. No SFTP menu options in the content menu for files.
It was working in all previous Insider releases for over a year.
View > Output > sftp shows an empty log. (sftp.debug is set to true.)
When I open my sftp.json, the problems pane shows a few warnings like, Missing property "name" and Incorrect type, Expected "string" (for instance for "privateKeyPath": null). When I fix these warnings and restart VScode, still SFTP does not work.
By the way. You did an awesome job creating and maintaining this plugin. I love it. Great job!

Today 1.39 (non insider) was released. It is working fine there.
In my 1.40.0-insider it is still not working. Might be just my install...


Replace social icons in sidebar with text

To do

[ ] check that this builds correctly (My computer disagrees with YUI Compressor)
[ ] review styles on the links

contrast
alignment

[ ] review link text. Should the label for the link be linked, or not?

Changes

Social icons are replaced with text
Hover tooltips are removed from the lis
A few classes were removed form the parent ul that were specific to the social icons.
Unrelated: OSX build instructions in README.md

Why
For readability and accessibility, and #26.

i agree with you too an extent. I would prefer to use social media icons as apposed to text. (I kept the building and email in text form) The sidebar is no place for large amounts of text, so i simplified it. (everyone knows what an email looks like, so there's no need to label it). The social media icons should be moved else where, as not to clutter the sidebar. I will figure out their exact placement later.


Welcome screen can be dragged like a pager

Hi there,
Just installed Snip from the App Store, and noticed that the welcome dialog has a weird dragging behaviour.  You can see the behaviour in the video I captured :-
https://www.dropbox.com/s/y4mcyxle8eyp7lb/snip-bug.mov?dl=0
A static image here :-
https://www.dropbox.com/s/o24zzmlq1cedmg8/snip-clip.png?dl=0
It seems like the welcome screen wants to be a pager.  Note that the dragging behaviour only happens when you drag on text.  I was trying to drag the welcome screen around and managed to trigger it.

Hi,
Congratulation, you found something I never finished 😬
The idea here was to give a list of keyboard shortcuts for the editor.
I guess I forgot to remove the pager code.
🙏

Commit 18b547 will fix that.

Thanks mate.  Fantastic work!


Use correct capitalization for iPhone and iPad

This also fixes a few typos in the README.

Thank you :)


CSS/Images

Is there any possibility to override the embedded css and to delete the embedded images?

I just saw I can do it with the UI.


[BUG] PDKPipeline occurs error on latest CDK

Describe the bug
When turn on crossAccountKeys, it occurs following error.
SSE-S3 is the only supported default bucket encryption for Server Access Logging target buckets
Expected Behavior
no error
Current Behavior
occurs error
SSE-S3 is the only supported default bucket encryption for Server Access Logging target buckets
Reproduction Steps
    this.pipeline = new PDKPipeline(this, 'ApplicationPipeline', {
      primarySynthDirectory: 'packages/infra/cdk.out',
      repositoryName: this.node.tryGetContext('repositoryName') || 'monorepo',
      publishAssetsInParallel: false,
      crossAccountKeys: true,
      synth: {},
      sonarCodeScannerConfig: this.node.tryGetContext('sonarqubeScannerConfig'),
    });

Possible Solution
I think it changed the default behavior in the cdk.
So we can create new bucket for server access logging.
Additional Information/Context
"dependencies": {
"aws-cdk-lib": "^2.59.0",
"cdk-nag": "^2.21.51",
"constructs": "^10.1.209"
}
PDK version used
0.12.34
What languages are you seeing this issue on?
Typescript
Environment details (OS name and version, etc.)
Mac OS

@drskur  See (https://github.com/aws/aws-cdk/pull/23514). The issue is with the crossAccountKeys: true, line in the props of the PDKPipeline. This sets up the encryption on the artifacts bucket  to use a KMS key.
As a workaround, set crossAccountKeys: false, and the issue will go away.
const artifactBucket = new Bucket(scope, "ArtifactsBucket", {
      enforceSSL: true,
      autoDeleteObjects: true,
      removalPolicy: RemovalPolicy.DESTROY,
      encryption: props.crossAccountKeys
        ? BucketEncryption.KMS    <========
        : BucketEncryption.S3_MANAGED,
      encryptionKey: props.crossAccountKeys
        ? new Key(scope, "ArtifactKey", {
            enableKeyRotation: true,
            removalPolicy: RemovalPolicy.DESTROY,
          })
        : undefined,
      publicReadAccess: false,
      blockPublicAccess: BlockPublicAccess.BLOCK_ALL,
      serverAccessLogsPrefix: "access-logs",
    });

^(above lines from here)

I checked the code, But I want to deploy it cross account, Pipeline Account to Stage Account. I am not sure it will work or not when I set the crossAccountKeys to false.

It looks work on https://github.com/aws/aws-prototyping-sdk/commit/45e9b8bd8227f04b239455a245a3c8ae421aa208.

This has been fixed in v 0.12.35


Make tsc check JS files with @ts-check annotations

Looks like tsc --noEmit wouldn't check JS files by default. This fixes that, and fixes some type errors

... except that CI is unhappy with you


Add exceptions to AutoSchedule

Remove end Time in schedule command
Change add day command format
Add exceptions for autoSchedule
Change time to be of LocalTime class

Closes #132


[ONNX] re-enable unit tests for CentOS

GCC 4.5.x generates invalid assembly code. Here's an example C++ code.
class Foo {
public:
  Foo() { }
  ~Foo() { }
};

class Bar {
  Bar() = delete;
  Bar(const Bar&) = delete;
  Bar(Bar&&) = default;
  const Foo& tf;
public:
  Bar(const Foo& f) : tf{f} { }
  ~Bar(){ }
};

Here's how the C++ 4.5.x compiler should generate the constructor Bar(const Foo&f)
Bar::Bar(Foo const&):
        push    rbp
        mov     rbp, rsp
        mov     QWORD PTR [rbp-8], rdi
        mov     QWORD PTR [rbp-16], rsi
        mov     rax, QWORD PTR [rbp-8]
        mov     rdx, QWORD PTR [rbp-16]
        mov     QWORD PTR [rax], rdx
        pop     rbp
        ret

Here's what actually the C++ 4.5.x compiler is doing.
Bar::Bar(Foo const&):
        push    rbp
        mov     rbp, rsp
        sub     rsp, 32
        mov     QWORD PTR [rbp-24], rdi
        mov     QWORD PTR [rbp-32], rsi
        mov     rax, QWORD PTR [rbp-24]
        lea     rdx, [rbp-1]
        mov     QWORD PTR [rax], rdx
        lea     rax, [rbp-1]
        mov     rdi, rax
        call    Foo::~Foo() [complete object destructor]    // <---------- ??????
        leave
        ret

The workaround to this problem is to use const T* instead of const T&. In modern C++ the raw pointer is treated as reference, too. Everything which has raw pointer to other object does not own it and does not mange the object's lifetime. It only references the object.
There's no need for ptr != nullptr, because it is impossible to instantiate core ONNX class with an invalid Protobuf object (constructors get const T&) or not setting the object (default constructor deleted).
Signed-off-by: Artur Wojcik<EMAIL_ADDRESS>

@apwojcik Any particular aspect you want me to review?

@aprocter Long story short I need approval that there's no other well-known solution to this problem. This kind of bug we hit for the first time and maybe there are known solutions e.g. not affecting the code?