Linking to and/or copying entries from existing lists

There are earlier attempts at doing the same thing. Here are ones I know from the top of my head:

https://github.com/sveltejs/integrations
https://svelte-community.netlify.app/code (dead)
https://github.com/ryanatkn/awesome-svelte-resources (dead)

Will add the listed items, thanks.


'sys.path' must not include foreign paths

CURRENT macOS FACT:

Packaged Mu with Python 3.7.
Checked the value of the bundled Python environment's sys.path with:

$ ./build/pup/mu-editor.app/Contents/Resources/Python/bin/python3 -q
>>> import sys
>>> print(*map(repr, sys.path), sep='\n')
''
'${PWD}/build/pup/mu-editor.app/Contents/Resources/Python/lib/python37.zip'
'${PWD}/build/pup/mu-editor.app/Contents/Resources/Python/lib/python3.7'
'${PWD}/build/pup/mu-editor.app/Contents/Resources/Python/lib/python3.7/lib-dynload'
'${HOME}/.local/lib/python3.7/site-packages'
'${PWD}/build/pup/mu-editor.app/Contents/Resources/Python/lib/python3.7/site-packages'
'${PWD}/build/pup/mu-editor.app/Contents/Resources/Python/lib/python3.7/site-packages/setuptools-49.6.0-py3.7.egg'
'${PWD}/build/pup/mu-editor.app/Contents/Resources/Python/lib/python3.7/site-packages/pip-20.2.2-py3.7.egg'
>>>

NEEDS CHANGE:

Two entries should be eliminated from sys.path:

The '' empty one, corresponding to the current working directory.
The ${HOME}/.local/lib/python3.7/site-packages which, I suppose, holds pip --user installed packages.

WONDERING:

What kind of sys.path values are we getting on Windows?
What about Python 3.8 variations?

RELATED:

This BeeWare briefcase issue, in particular from this comment onwards. :-)

CLOSING:

Same as #195.


Add new capabilities

Hi! This library seems to be not supporting the new capabilities, such as inlayHint. As a result arduino LSP (which depends on it) doesn't work with the newer IDE versions. See description of the problem here https://github.com/arduino/arduino-language-server/issues/187
Is it possible to update the capabilities definition with all the new available capabilities?

I tried to add missing fields myself, but it seems like a huge change will be required to support 3.17 protocol version (just grep for @since 3.17.0 here if you want to see the scale of the problem yourself).
At the same time, it seems like everything needed is already implemented and well-supported in gopls
https://github.com/golang/tools/tree/2d104ec5d29fd2c2156ce4e7f1ca9c0258bab055/gopls/internal/protocol
However, all of that is located in the internal go package hence it can't be used from the other go module.
So there are two paths we can take here:

Modify this package to support all the new methods
Create a completely separate package and try to reuse gopls tooling to create a new server

Ideally will be nice if the author of the package shows up and provides some guidance or support on the first solution. Otherwise, I believe the new package way will be much easier to implement.

Hi @fspv, thanks for your interest in this package.
I've made this package for the Arduino LS, and yes, keeping up with the upstream LSP specification is a work on his own.
I looked into the gopls package some time ago, but the authors did make it internal, if we would like to use it we may face with a package not designed for external usage and we may also face breaking changes in the future (it may be better or worse than maintaining our package, who knows...).
Said that, IMHO the best way forward is to implement the minimum subset of LSP 3.17 to support neovim in this library. WDYT?
Another thing that puzzles me is that the LSP protocol is designed in a way that the client and the server can negotiate the capabilities https://microsoft.github.io/language-server-protocol/specifications/lsp/3.17/specification/#textDocumentClientCapabilities for example if the optional inlayHint field is missing the LSP client should not send those requests. At least this is my understanding of the protocol...

Hi @cmaglie! Sorry, took quite a while to respond, because I lost context on this and it took some time to figure it out again.
This is the neovim code that constructs the default capabilities to be sent to the server: https://github.com/neovim/neovim/blob/v0.10.0/runtime/lua/vim/lsp/protocol.lua#L635-L656
There is no way to make inlayHint key in the dictionary disappear. You can overwrite it with some nonsense, but it will still be passed to the server and will result in a deserialization error.
I think your point is valid, that the client must negotiate the supported features with the server, however, I'm not sure if it includes the client not sending its capabilities in the first place. If this is the case, then it seems like a bug on the neovim side. However, I'm leaning towards thinking that the server must accept whatever it's been provided by the client and then try to match the client's capabilities with what it supports. So extra capabilities on the client shouldn't be a problem.
So maybe a solution here will be to relax the JSON parsing constraints and just ignore the unknown keys. I'm not sure if the library used here supports it though.
Or if we are sure this is a neovim bug, then we can report it its repo, I think we have a solid case here for adding an option to stop sending the inlayHint field.

@leoverde2 in this thread says they fixed the problem with a small change

I opened a PR to implement the change recommended in the arduino-language-server issue. I tested it locally and the LSP appears to work as expected under nvim 10.0.


Rework Compile Script

This PR reworks the Compile script.
Addressed Issues:

The compile sometimes would fail and show a "file in use" error when writing to winutil.ps1
-> This is addressed by assembling the whole script in memory and only writing to disk once (per created file)

Changes:

Add a Debug switch that only creates the auto-generated XAML files on disk if desired. If Debug isn't specified the files are removed from the disk (to avoid confusion with different versions of a newly compiled winutil script and old XAML files)
Add a visual progress bar detailing the compile status using Write-Progress
Add the header as a multiline string

Great improvements overall, happy to see xaml files not being created whenever compiling! 😎
This change and the .gitignore fix in PR #2051 should remove the worry of accidently adding compiled files by new users, not to mention the other nice changes you've made.
I'll try testing it when I've some spare time, well done @Marterich 😄


Landing page content enhancements

Let's discuss how could we enhance the landing page.
I've got recommendations on using images, use of a different text for the CTA button among others.
There might be typos in the texts as well.
We could list other webpages trying to accomplish registering volunteers in this issue.
One example: https://www.goodsamapp.org/NHS

The different types of roles could have colorful badges with emojis like: At home / At hospitals / Mobile / Requires immunity / "Science background"?

A mentor of ours, the CEO of journify.co, recommends the following titles:

"Healthcare Crews"
"Elderly Crews"
"Operation Crews
"Research Ambassadors"
"Hospital Crews"
"Triage Crews"

There should be descriptions of one sentence underneath.
Such as:
Hospital Crews
[[emoji] at hospitals]     [[emoji] requires immunity]
Help nurses and other medical personnel at hospitals.


[Feature Request] Will there be ios16.1

Will there be ios16

Accidentally updated😭

I'm not sure what did you mean by "Will there be iOS 16.1". 🤔
iOS 16.1 has been released and I don't think it's relevant to EhPanda.
cc @chihchy Please close this issue if no response from the issuer in 1d.


Add Tim Kaine twitter

Linked from https://www.kaine.senate.gov/news

Per Past @JoshData in #822, https://tweeterid.com/ is the resource, and it reports<PHONE_NUMBER>230892544.


Start server buffers refactoring - Part 1

@dennisjenkins75 Hello, Dennis! In travis log this PR has:
100% tests passed, 0 tests failed out of 22
100% tests passed, 0 tests failed out of 22

but CI failed. What's happened? Can you rerun tests?

That is odd.  I'm re-running the tests.


New check for order of run modes. This closes #31

Rework the code to only accept allowed runmodes (author, author.dev etc.), throw an exception on invalid runmodes (for example dev.author) and ignore other runmodes (like dev.prettycool)
Description

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[X] I have signed the Adobe Open Source CLA.
[X] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[X] I have updated the documentation accordingly.
[X] I have read the CONTRIBUTING document.
[X] I have added tests to cover my changes.
[X] All new and existing tests passed.

Merged - Thanks @cziegeler !


510 red stars event

Closes #510.
While I was at it I corrected some style and convention 'errors'. If you want, I can move them to a separate PR where we could do this once an for all in all files as well. Or we do this on the fly with everything we touch 🎵

Changes Unknown when pulling d8c1fb82125334d1a0770abf9fe18b733c04f20e on 510_red-stars-event into ** on dev**.

Coverage remained the same at 100.0% when pulling 5d83147b5ad37092d3c96caad7893f3c1ca8d3b0 on 510_red-stars-event into 87ef77560d0fa1f6b658e07800dfa913f37d598a on dev.

Coverage remained the same at 100.0% when pulling 446251a6eec7559df7021f8b3313af2fedfb99bb on 510_red-stars-event into 87ef77560d0fa1f6b658e07800dfa913f37d598a on dev.


Fix the definition of interface ErrorEventHandler

Fix #2838

Can you verify that this does not break existing code in RWC suite?

It doesn't add errors to the RWC tests, only changes baselines.


Permission issue when trying to enable Security Hub

I get the following error.  As far as I can tell I am following all steps:

[ERROR] ClientError: An error occurred (UnrecognizedClientException) when calling the EnableSecurityHub operation: The security token included in the request is invalid Traceback (most recent call last):
File "/var/task/securityhub_enabler.py", line 487, in lambda_handler    enable_master(master_session, securityhub_regions, partition)
File "/var/task/securityhub_enabler.py", line 395, in enable_master    sh_master_client.enable_security_hub()
File "/var/runtime/botocore/client.py", line 316, in _api_call    return self._make_api_call(operation_name, kwargs)
File "/var/runtime/botocore/client.py", line 635, in _make_api_call    raise error_class(parsed_response, operation_name) |
-- | --

It might be related to https://github.com/awslabs/aws-securityhub-multiaccount-scripts/issues/23
Here are the regions that are found at https://github.com/aws-samples/aws-control-tower-securityhub-enabler/blob/main/src/securityhub_enabler.py#L412: ‘ap-east-1’, ‘ap-northeast-1’, ‘ap-northeast-2’, ‘ap-south-1’, ‘ap-southeast-1’, ‘ap-southeast-2’, ‘ca-central-1’, ‘eu-central-1’, ‘eu-north-1’, ‘eu-west-1’, ‘eu-west-2’, ‘eu-west-3’, ‘me-south-1’, ‘sa-east-1’, ‘us-east-1’, ‘us-east-2’, ‘us-west-1’, ‘us-west-2’
The organization's master account has access to the ‘me-south-1’ region, yet the audit account doesn't.  See the screenshot below:

Enabling all regions (meaning, me-south-1 was added) on the audit account fixed this issue and allowed for the proper deployment of the Cloudformation template
Some questions arise:
1- I'd expect the audit account to be enabled in all regions.  If the audit account doesn't have `me-south-1` but one of the child accounts has it, then we wouldn't have the visibility we want here
2- If the audit account has `me-south-1` enabled but the child account doesn't, would the script fail when trying to run?  A quick analysis indicates that it would

I wish I had the cycles to explore these questions, but I currently don't.  Perhaps someone else will

TODO: add account enablement checking for each child account before trying to enable


dbt-core dependency update

dbt-core is now 1.1.0, and is incompatible with dbt-sqlserver
ERROR: pip's dependency resolver does not currently take into account all the packages that are installed.
This behaviour is the source of the following dependency conflicts.
dbt-sqlserver 1.0.0 requires dbt-core~=1.0.0, but you have dbt-core 1.1.0 which is incompatible.

a pre-release with support for 1.1.0 is out
https://pypi.org/project/dbt-sqlserver/1.1.0rc1/
Test it out and let us know how it works!