Adding Xray Source Control Service

[x] All tests have passed. If this feature is not already covered by the tests, new tests have been added.
[x] The pull request is targeting the dev branch.
[x] The code has been validated to compile successfully by running go vet ./....
[x] The code has been formatted properly using go fmt ./....

Adding XSC support to JFrog Cli and it's components

For users who have XSC enabled in their system, will refer to XSC server instead of Xray.
Refactor to allow the future development of XSC service with fallback to Xray.
XSC is build on-top of Xray service, meaning unimplemented XSC functions will fallback to default Xray service functions.
For start, jf-audit command is implemented to work with XSC, sending extra GitInfoContext object to allow linking scan data to repositories.
XSC url is not provided from flag or config. We build it by either taking the platform url and adding suffix, or replace Xray url suffix with /xsc suffix.

How to use
No action from the user is needed, as this is optional.
If the server has XSC, it will query it.
In order to use the new functionality of linking scan information to repository, need to set the XscGitinfoContext object when calling audit command:
auditParams.xrayGraphScanParams.XscGitInfoContext = &scan.XscGitInfoContext{
		GitRepoUrl:    "https://github.com/EyalDelarea/npmexample",
		GitRepoName:   "npmexample",
		GitProject:    "myproject",
		GitProvider:   "github",
		BranchName:    "main",
		LastCommit:    "https://github.com/EyalDelarea/npmexample/commit/02c4eb4065e293d96fdfc70e64a58244eabed399",
		CommitHash:    "02c4eb4065e293d96fdfc70e64a58244eabed399",
		CommitMessage: "test dev deps",
		CommitAuthor:  "eyaldelarea",
		Date:         "2006-01-02T15:04:05Z07:00",
	}

Depends on
JFrog client go https://github.com/jfrog/jfrog-client-go/pull/811
JFrog cli-core https://github.com/jfrog/jfrog-cli-core/pull/896

XSC shouldn't be tested via the CLI at this point.


Click on text does not check the adjecent checkbox

Best practices for checkboxes use label wrapping so that the checkbox checked state changes on label click (http://stackoverflow.com/questions/6293588/how-to-create-an-html-checkbox-with-a-clickable-label).
Is there a plan to incorporate this feature into this plugin?
Can this be solved with a custom node template?

Solved it with this node template:
`

{{trvw.label(node)}}

`


Include initial curated list of IaC test samples #1

included testsamples for AWS CloudFormation, Terraform and Docker.

I'm using a script to generate them, so we can define the expected.yaml structure an re-run the script.


fixed unit

Thanks @tmusho 🙂
Before we merge, let me go through the unit definitions for properties and adopt SI. It shouldn't take me too much, I can merge and regenerate the files for you to upload in NOMAD.

@tmusho this is ready. I will merge, and you can use the generated tests/data/original_data_NOMADschema.archive.yaml and the zipped file to upload the data.
Thanks for carefully checking the properties. The descriptions are very good 👏🏻


Alternative to Orion - Subscription without georeferencing

This is, in broad terms, just a few adaptations and improvements on what's been already contributed by @giovannicuriel .
A few points to be updated/improved:

Subscription configuration data must be kept on a database;
Query language (i.e. contents of field "subject.condition.expression.q") needs to be more resilient to user's idioms (variable syntax, argument positioning, spaces, etc.);
Topics with subscription results must be automatically generated by the subscription manager, not set by the callers;


Create clean/ca/santa_rosa.py

Creating the Scrape-meta and Scrape for the Santa Rosa Police

Closed with PR #45. Thank you!


How to communicate with engine easily outside stdin

Hi
I have an idea that I'm not able to realize without a brief help.
I can see that there is a serve.py script that creates the server, but I don't know how to communicate with it. Also creating the engine in gpt.py without using the FLAGS as a global object is not easy for me.
What I'd like to be able to do is:

Set the current state of some Go game
Generate the best move according to the model

I am able to pass the .sgf file each time the game state changes, and I am able to run genmove command in stdin in gtp.py script. But I can't figure out how to communicate with the engine outiside the stdin.
Could somebody please tell me what are the possibilities to communicate with the engine using simple API? Should I write it, how to set flags, or maybe I should use the serve.py code, but then how.
I think a simple suggestion/explanation would let me proceed.
Thank you

If you're trying to write a script, probably extending gtp.py is the place to do it.
serve.py is used for our front-end, minigui.
The gtp.py script is a straightforward way to load the model in the python engine.
If you need performance, you can build the gtp binary in cc/ and script inputs/outputs to that engine.
GTP is pretty strict about what it can output to stdout -- not much!  Most of the useful output we put on stderr.
So you could:

write a script to run inferences with the model directly (lots of examples in oneoffs/)
write a script to interact with the engine via custom gtp command handlers (extending gtp.py)
write a script to interact with the engine via stdin and scrape stderr/stdout for the outputs

let me know if this helps!

Yes, it looks ok. I don't know why I wasn't able to understand it. I think the documentation could be better, there's a lot of scripts without a single word about usage. Thanks for pointing out oneoffs directory


[Template] Use voltage compensation/output when available for motors

Solves issue #15

Not a good idea, see https://www.chiefdelphi.com/t/voltage-compensation/392910/10


Exempt typing from TCH by default?

ruff 0.0.236
It would seem to me that typing should be exempted from TCH by default.
For example:
from typing import Any

def foo() -> Any:
    return 1

Unless I'm missing something, use of the type checking block that satisfies this check requires importing TYPE_CHECKING from typing, which loads typing, thus nullifying the usefulness for this for typing imports.
>>> import sys
>>> "typing" in sys.modules
False
>>> from typing import TYPE_CHECKING
>>> "typing" in sys.modules
True

flake8-type-checking's README points to two "examples in the wild", both of which contain typing imports, neither of which places them in the TYPE_CHECKING block:

https://github.com/python-poetry/poetry/blob/714c09dd845c58079cff3f3cbedc114dff2194c9/src/poetry/factory.py#L1:L33
https://github.com/snok/asgi-correlation-id/blob/main/asgi_correlation_id/middleware.py#L1:L12

flake8-type-checking also contains this note:
https://github.com/snok/flake8-type-checking/blob/main/flake8_type_checking/checker.py#L601-L603
...along with some other conditions regarding typing import aliasing and typing.cast imports above that commentary. I haven't dug into them nor if ruff currently has any corresponding functionality.

There is a small advantage to moving, e.g. from typing import Any, Generic, ParamSpec, TypeAlias, TypeVar, ..., into a TYPE_CHECKING block: Reducing the amount of cruft littering the module globals at runtime.

We could make it part of exempt-modules by default, that way it's configurable.

Also, is this happening because you have strict = true? You can also set exempt-modules = ["typing"] if you want to prevent this happening...

No strict = true here. I understand I could exempt it for myself, I just think it would be a better default for most cases, therefore suggesting it here.

Yeah I think the least surprising thing would be to exempt typing and typing extensions by default.

typing_extensions is a bit different, as I think it can be entirely put within TYPE_CHECKING and the import actually avoided at runtime. Therefore I'd personally not exempt it by default. But no strong opinions on that here.

Yeah makes sense. I'll add typing.


fanOutZipIn support : parallel and single threaded

e.g.
ReactiveSeq.of(1,2,3,4)
           .fanOutZipIn(s1->s1.filter(i->i%2==0).map(i->i*2),
                   s2->s2.filter(i->i%2!=0).map(i->i*100),Tuple::tuple)
           .toListX();

Should result in
ListX.of(Tuple2[4,100],Tuple2[8,300]))

Should work for all of

Push based reactive Spouts
Pull based synchronous Streams
FutureStreams

Released in 2.0.0-MI3


[Feature request] Please add more models

I know this uses GPT trough OpenAi's API but it should be very useful if integrating it with some GPT-J API or other models so people can run they own model servers and use locally the app?

Please see this doc: https://github.com/josephrocca/OpenCharacters/blob/main/docs/custom-models.md


Disable system beep sound from dialog popups

These beeps are really annoying.
For deleting files, it should mimic Windows Explorer behavior. Delete and Shift-Delete don't play any sounds. Only shift-delete displays a warning box, but the Recycle bin warning can be disabled in VSCode.
The workaround is turning off all System Sounds in Windows, but then I can't hear my custom notifications or other serious beeps.
This issue mainly concerns Windows 10.

Possible workaround:
"window.dialogStyle": "custom",

I do not believe electron provides any way to show the dialog without a sound. Thus, the workaround is necessary. @deepak1556  to confirm.

Any update on this feature?? The "delete" bell sound is hurting my ears

This feature request has not yet received the 20 community upvotes it takes to make to our backlog. 10 days to go. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

:slightly_frowning_face: In the last 60 days, this feature request has received less than 20 community upvotes and we closed it. Still a big Thank You to you for taking the time to create this issue! To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

could we re-vote?


Add additional test cases to the ModuleAliasing suite

Update the module aliasing tests to build and verify the NestedDeps* test packages.
Motivation:
Ensure Nested Dependencies work as expected
Modifications:
Add additional test cases to the ModuleAliases suite containing the NestedDeps* test packages. Also, update the test packages accordingly.
Result:
Ran the following at-desk
swift test --filter ModuleAliasingFixtureTests

Fixes #8114

@swift-ci smoke test

@swift-ci test

@swift-ci test

@swift-ci please test windows platform


[Github] NEXT-00000 - Prevent product being loaded in own cross selling product streams

1. Why is this change necessary?
Products should not be loaded/displayed in their own cross sellings.
2. What does this change do, exactly?

Added filter to product stream criteria in ProductCrossSellingRoute::loadByStream.

3. Describe each step to reproduce the issue or behaviour.
Create a product stream containing the product you are going to add a cross selling to > add a cross selling using the created product stream to the product
4. Please link to the relevant issues (if any).
5. Checklist

[x] I have rebased my changes to remove merge conflicts
[x] I have written tests and verified that they fail without my change
[x] I have created a changelog file with all necessary information about my changes
[ ] I have written or adjusted the documentation according to my changes
[ ] This change has comments for package types, values, functions, and non-obvious lines of code
[x] I have read the contribution requirements and fulfil them.

Imported from Github. Please see: https://github.com/shopware/shopware/pull/4764

[public] Shopware automated response: This issue is linked to the internal issue: NEXT-39418
[created from NEXT-39418, comment 509762]

[public]

Status Change: Open

This issue is ready for implementation.
[created from NEXT-39418, comment 509769]

[public]

Status Change: In Progress

Work on this issue has started. It is now being implemented.
[created from NEXT-39418, comment 509770]

[public]

Status Change: In Review

Work on this ticket is complete and is now pending review.
[created from NEXT-39418, comment 509771]

[public]

Status Change: Resolved

The ticket has been successfully completed; the solution is implemented and verified.
[created from NEXT-39418, comment 511194]

[public]

FixVersion Change: <IP_ADDRESS>

This issue will be / was solved in Shopware version <IP_ADDRESS>.
[created from NEXT-39418, comment 511195]


Fitter cleanup

I'd like to do the following:

[ ] Move the plotting code out
[ ] Move sample settings to separate files
[ ] Expose flags in command line

Also want to fix:

[ ] The overlapping plot labels in c2.pdf.

For some reason, the plotting is not working on ROOT 6.16:

But it's fine on ROOT 5.34/38 and ROOT 6.24/02.


Add: Ultima Dark theme

I would like to add a file with my color scheme

Thanks in advance

Where is this written?
Your 'To add a new theme, just create a Pull Request with the following changes:' is misleading. Please indicate this in your contribution guide. So that people don't waste their time and don't useless pull requests

Where is this written?

I would assume it to be common sense. Apparently not common enough. There's really no logical reason why we'd just add every random theme out there.

Only one reason — community.  In general, I understood you. I can conclude that the author of alacritty is`t very friendly with his community.
Good luck

Only one reason — community.  In general, I understood you. I can conclude that the author of alacritty is`t very friendly with his community.

If you accept a theme from author directly, it means that you can write a new theme every day and submit upstream, which doesn't make any sense and can be easily abused. So the theme should be at least somehow known and submitted by not the author directly.
Confusing friendliness with "not letting abuse to happen" is unfortunate.

Only one reason — community.  In general, I understood you. I can conclude that the author of alacritty is`t very friendly with his community.

If you accept a theme from author directly, it means that you can write a new theme every day and submit upstream, which doesn't make any sense and can be easily abused. So the theme should be at least somehow known and submitted by not the author directly.
Confusing friendliness with "not letting abuse to happen" is unfortunate.

I understand perfectly what you are talking about and I wholeheartedly agree with you. Write this in the contribution guide. Write from whom you accept pr and define what it means popular theme and we won't create a pr and won't receive comments that we are trying to push a 'random' theme. Popular is how much? 1k 5k 10k 50k stars

Where is this written?

I would assume it to be common sense. Apparently not common enough. There's really no logical reason why we'd just add every random theme out there.

My theme isn't 'random' for me. It sounded offensive. "You're not popular enough, you can't come here." That's what I heard
We can correspond here forever. Just define the rules in the readme

Or do the same as Robbie Russell did for oh-my-zsh - just closed accepting user themes.
He put it all on the wiki and everyone lives happily.
We are discussing here who is worthy of merge to main and who is not. It's very thin ice

The point is not what we merge, but who sends the patch to propose some theme. If the person is involved with the project then likely not, if it's some 3rd party than it's fine.
It's the same if I wanted to e.g. promote alacritty and start sending PR to every curated list of things myself, because I'm biased, the same way as you're biased. The point is always not starts or some other social network aspect, but bias.
If other person not associated with your project submitted it, then it'll likely be fine, because it's not you submitting it, but someone not associated with your project.
And as @chrisduerr said, it's fairly a common sense, even in real life outside of software (e.g. recommendations, etc).


new: update SDK to plugin API version 3.4.0

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind documentation

/kind failing-test

/kind feature

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area build

/area plugin-sdk

/area tests

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: mrgian
Once this PR has been reviewed and has the lgtm label, please assign leodido for approval. For more information see the Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

There is not a label identifying the kind of this PR.
Please specify it either using /kind <group> or manually from the side menu.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

Welcome @mrgian! It looks like this is your first PR to falcosecurity/plugin-sdk-cpp 🎉

LGTM label has been added.  Git tree hash: 1f1d66163b2067a8581f94421943ccaa6bb21ec8

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: FedeDP, mrgian
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [FedeDP]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Cognite Data Fusion connector

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

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

Hello @sceniclife,
The commit is not done as anticipated, the validation chekers are validating other connectors repo.  Kindly see the errors from Swagger and ApiProperties validation checkers.  Could you please close this PR and create new one.


Fix tnf tests related to cnf-app-mac-operator ServiceAccount

This targets the following tests:

access-control-pod-automount-service-account-token
access-control-pod-service-account

These tests are fixed by using a ServiceAccount different than default one in cnf-app-mac-operator. I've followed the same approach applied in the other three operators: use controller-manager ServiceAccount and ClusterRole/ClusterRoleBinding.

This is ready for review. The tests under the scope of this PR are now passing, retrieving the info from the terminal (we'll see the passed job in several minutes):
[dciteam@jumphost ~]$ cat /tmp/dci_logs.kzs3r7cl/cnf-certification-tests_junit.xml | grep access-control-pod-service-account
    <testcase name="access-control-pod-service-account" classname="CNF Certification Test Suite" status="passed" time="0.0003949030"></testcase>
[dciteam@jumphost ~]$
[dciteam@jumphost ~]$ cat /tmp/dci_logs.kzs3r7cl/cnf-certification-tests_junit.xml | grep access-control-pod-role-bindings
    <testcase name="access-control-pod-role-bindings" classname="CNF Certification Test Suite" status="passed" time="0.0026507010"></testcase>


Created reference page for NuGet CLI's search command

Docs Build status updates of commit 6da1fc4:
:clock10: Pending: waiting for processors (10 builds ahead of you)
:warning: Docs Build is busy, currently there are 10 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

Docs Build status updates of commit 6da1fc4:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/reference/cli-reference/cli-ref-search.md
:white_check_mark:Succeeded
View

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

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

File
Status
Preview URL
Details

docs/TOC.md
:white_check_mark:Succeeded
View

docs/reference/cli-reference/cli-ref-search.md
:white_check_mark:Succeeded
View

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

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

File
Status
Preview URL
Details

docs/TOC.md
:white_check_mark:Succeeded
View

docs/reference/cli-reference/cli-ref-search.md
:white_check_mark:Succeeded
View

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

Docs Build status updates of commit 075b03d:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/TOC.md
:white_check_mark:Succeeded
View

docs/reference/cli-reference/cli-ref-search.md
:white_check_mark:Succeeded
View

docs/reference/nuget-exe-CLI-Reference.md
:white_check_mark:Succeeded
View

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

Docs Build status updates of commit 89bbb7c:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/TOC.md
:white_check_mark:Succeeded
View

docs/reference/cli-reference/cli-ref-search.md
:white_check_mark:Succeeded
View

docs/reference/nuget-exe-CLI-Reference.md
:white_check_mark:Succeeded
View

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

Docs Build status updates of commit 2621bf2:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/TOC.md
:white_check_mark:Succeeded
View

docs/reference/cli-reference/cli-ref-search.md
:white_check_mark:Succeeded
View

docs/reference/nuget-exe-CLI-Reference.md
:white_check_mark:Succeeded
View

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

Is this supported on dotnet CLI or only nuget.exe for now?

@anangaur It's nuget CLI only. We have to backlog a dotnet CLI experience. I'll make a dotnet CLI epic to cover it shortly.


Increase memory limits

Increases memory limits for envoy and pgbouncer
Checklist

[x] The PR has a meaningful title. It will be used to auto generate the
changelog.
The PR has a meaningful description that sums up the change. It will be
linked in the changelog.
[x] PR contains a single logical change (to build a better changelog).
[x] Categorize the PR by adding one of the labels:
bug, enhancement, documentation, change, breaking, dependency
as they show up in the changelog.
[ ] Link this PR to related issues or PRs.

please verify in LAB that those request/limits are not violate fair usage ratio
LGTM

please verify in LAB that those request/limits are not violate fair usage ratio LGTM

Looks good on the lab, however, the init containers violate the ratio. But given that they only run for a few seconds during startup they don't really affect it very much.


fix: google objectstore uses proper gs configuration

gs was converting to azure options instead of gcs options.

@wjones127 The build error seems unrelated to code changes

Could not find protoc installation and this build crate cannot proceed without


Issue with k8s.io/docs/tasks/administer-cluster/kubeadm-upgrade-1-9/

This is a...

[ ] Feature Request
[x] Bug Report

Problem:
Upgrading from a 1.8.7 cluster to a 1.9.x cluster did not work.   Here is the output:
admin@ip-172-20-57-63:~$ sudo kubeadm upgrade plan
[preflight] Running pre-flight checks.
couldn't create a Kubernetes client from file "/etc/kubernetes/admin.conf": failed to load admin kubeconfig [open /etc/kubernetes/admin.conf: no such file or directory]

Proposed Solution:
Page to Update:
https://kubernetes.io/...

1.8.7

This is a kops-generated cluster (fully functional) on AWS.

@gamename :wave: This issue sounds more like a request for support and less like an issue specifically for docs. You can bring your question to the #kubernetes-users channel in Kubernetes slack. You can also search resources like Stack Overflow for answers to similar questions.

I ran into the same issue as the error above.  I ended up upgrading kops first ant then updating the cluster with kops upgrade cluster to get things on the latest version.


on Safari, 100px scroll is much slower than Chrome.

Hi, I make a chat application with the auto-scroll message list.
I've found that on Safari, 100px scroll is much slower than Chrome.
I want to make interaction consistently, Is there any solution for smooth scrolling speed?
Thanks

And on the other side.. if the content is very long, or is large in height. It scrolls very fast.
Suggest that scroll time length is in relation to the content scrollable length rather and a fixed duration.

Ref #13

I was working on this a while ago actually https://github.com/iamdustan/smoothscroll/pull/89
I'll try to find some time and get it back in.

This is pretty noticeable when using this polyfill 😕


fix(client): provide an option to fix an issue where the target service cannot be found when a client using generic calls a multi-service server

What type of PR is this?

Check the PR title.

[x] This PR title match the format: <type>(optional scope): <description>
[x] The description of this PR title is user-oriented and clear enough for others to understand.
[ ] Attach the PR updating the user documentation if the current PR requires user awareness at the usage level. User docs repo

(Optional) Translate the PR title into Chinese.
(Optional) More detailed description for this PR(en: English/zh: Chinese).

en:
zh(optional):
(Optional) Which issue(s) this PR fixes:

(optional) The PR that updates user documentation:

As suggested by Emma, it's better (easy to use and understand) to retrieve the Service Name from the generic call form:
gCli.Call(ctx, "$svc/$method", ...)

And since it's a derived option, even if it's not added to Kitex, users can achieve the goal without it.
Therefore I think currently we better not merge.


Adding additional outputs wrapper

Adding additional outputs wrapper and making edits to additional outputs scripts to work correctly with wrapper.
Summary
Add your summary here - keep it brief, to the point, and in plain English.
Checklists

This pull request meets the following requirements:

[ ] installable with all dependencies recorded
[ ] runs without error
[ ] follows PEP8 and project specific conventions
[ ] appropriate use of comments, for example no descriptive comments
[ ] functions documented using Numpy style docstings
[ ] assumptions and decisions log considered and updated if appropriate
[ ] unit tests have been updated to cover essential functionality for a reasonable range of inputs and conditions
[ ] other forms of testing such as end-to-end and user-interface testing have been considered and updated as required
[ ] tests suite passes (locally as a minimum)
[ ] peer reviewed with review recorded

If you feel some of these conditions do not apply for this pull request, please
add a comment to explain why.

Latest changes are related to ASAP-679


Making affiliate links easier to change (issue #54)

Thanks for your PR! I'm sorry I only now had the time to think more about it. The problem is that you're reading this file from disk on every request, this is wasteful and unnecessary. Please find a good way to cache the lines in a field or somewhere.

Never mind, I can take care of this myself. Thanks.


"System.Security.Cryptography.X509Certificates is not supported on this platform" Error

Running this sample gives the following error in the browser console.
"System.Security.Cryptography.X509Certificates is not supported on this platform".
Framework Version: .NET 5 RC2
Browser: Chrome and Edge
Same issue whether run from the CLI or the latest VS.

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

ID: 08de9daf-79aa-ee76-431e-f2432f8d691e
Version Independent ID: 1a1ee728-1741-c82a-75e1-06159709935b
Content: Use ASP.NET Core SignalR with Blazor WebAssembly
Content Source: aspnetcore/tutorials/signalr-blazor-webassembly.md
Product: aspnet-core
Technology: aspnetcore-tutorials
GitHub Login: @guardrex
Microsoft Alias: riande

Ah... The SignalR package needs updating when using .NET 5...
dotnet add Client package Microsoft.AspNetCore.SignalR.Client --version 5.0.0-rc.2.20475.17

Yes, @pjgpetecodes ... The docs are package-version agnostic in that regard. We don't place the actual versions (preview versions) into the preview docs because we would need to update ALL of them for each preview release and then at GA. There would be many, many to update each time a pre-release came out and then again when we reached GA. We just don't have the staffing to do it. We rely upon the dev to magically know a bit about preview packages for each pre-release period. I'm sorry that it slowed you down a bit. The good news is that we're close to 5.0 GA, and we'll be able to put this behind us soon for 5.0 ....... until the next pre-release period, of course. :smile:

Sounds perfectly fair @guardrex ... Thanks!


Typo "a Azure Virtual Desktop"→"an Azure～"

https://learn.microsoft.com/en-us/azure/virtual-desktop/set-up-customize-master-image?source=recommendations
https://github.com/MicrosoftDocs/azure-docs/blob/main/articles/virtual-desktop/set-up-customize-master-image.md
#PingMSFTDocs

Due to the upcoming Ignite conference, all merging and publishing is on hold until the morning of 11/15.
This PR is ready for merge on 11/15.
#label: "event-triaged-to-10AM"
#label: "missed-content-complete"
#label: "aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team


Develop

Coverage decreased (-2.06%) when pulling 61977af0efba4d53aee0f5a78e605b1b8ee9fc9e on develop into 51e168c048b23d9bd8c892d1c19f7154e377679f on master.

Coverage decreased (-2.06%) when pulling 31f158184fe7205b7f000c35deec35f5ea5e2d0e on develop into 51e168c048b23d9bd8c892d1c19f7154e377679f on master.


How to handle exceptions on NotifierManager.getPushNotifier().getToken()

KMPNotifier and Kotlin version:
kmpnotifier version: 1.1.0, kotlin version: 1.9.23
Describe the bug
I have a doubt regarding exception handling. I see many issue on token gathering (probably they are more on firebase side because they should not happen). However, it's a bit unclear how to handle the exceptions properly in cases similar to:

java.io.IOException: SERVICE_NOT_AVAILABLE

In this case I think it might be preferred to return null instead of throwing an exception, because I'm not sure the exception would be caught from common kotlin code.
E.g.
    suspend fun getToken() = try {
        NotifierManager.getPushNotifier().getToken()
    } catch (e: Exception) {
        null
    }

So this might not catch the exception properly, right?
If we agree on this I could make a PR for the resolution on android side
In which platform bug occurs?
android (only tested here)
Expected behavior
Return null instead of throwing exception.

@blackwiz4rd thanks for reporting. This issue happens when device doesn't have an internet connection. But I agree with you that, it should either return null, or make it possible to catch exceptions. I'll look into this.
But temporary solution is that before calling getToken function check if the device has an internet connection. This should prevent java.io.IOException: SERVICE_NOT_AVAILABLE

Thanks for suggestion @blackwiz4rd. Now in 1.4.0 version of library in case of exception it will just return null


Update/remove the .NET 4 note saying "" has no effect on assembly load times.

In the Remarks section of the generatePublisherEvidence element page. there is a big blue warning note saying "In the .NET Framework 4 and later, this element has no effect on assembly load times."
That statement is not true (or at least, is not always true).
In our .NET 4.7 desktop app, we have needed to add <generatePublisherEvidence enabled="false"/> to the *.exe.config files for all of our signed executables, in order to prevent 30-40 launch delays when a system is behind an HTTP proxy and the servers hosting certificate revocation lists are not reachable.
Without the magic XML in the App.config file, the assembly is delayed for 40-seconds while the OS tries to reach out to OCSP and CRL endpoints to figure out if the app's certificate has been invalidated.
Some of our executables are Windows Services, so causing these startup delays is definitely something to avoid. In fact there is already a note saying "We recommend you disable this feature for Windows services", which seems to go directly against the "This setting doesn't apply to .NET 4 or greate" note.

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

ID: 4a579a8e-74fd-a86e-8e80-78719c9b1b4e
Version Independent ID: 56f18e1d-69fe-4423-23ad-29d7d5bce914
Content: <generatePublisherEvidence> Element
Content Source: docs/framework/configure-apps/file-schema/runtime/generatepublisherevidence-element.md
Product: dotnet-framework
GitHub Login: @rpetrusha
Microsoft Alias: ronpet

It would be great, if there would be any clarification how this actually works.
What's the behavior if referenced assemblies does have a different .NET version?
Remarks of Publisher Class:

By default, code access security (CAS) does not check for Publisher evidence. Unless your computer has a custom code group based on the PublisherMembershipCondition class, you can improve performance by bypassing Authenticode signature verification.

The legacy security policy and the Code Access Security Policy Tool (Caspol.ex):

Specifies whether the runtime uses legacy code access security (CAS) policy.

Starting with the .NET Framework 4, Caspol.exe does not affect CAS policy unless the  element is set to true. Any settings shown or modified by CasPol.exe will only affect applications that opt into using CAS policy.

Security Changes in the .NET Framework 4, especially Evidence Collections in this article:

Before the .NET Framework 4, the CLR generated the full set of evidence objects that applied to an assembly when the assembly was loaded. These objects were stored in a list, which consumers would then iterate over looking for a specific object. Therefore, all evidence was made available, whether or not it was used. For most evidence objects, this behavior was not an issue; however, for evidence objects such as System.Security.Policy.Publisher (which requires Authenticode verification), this behavior was inefficient.
To improve this behavior, the interaction with the evidence collection has been redesigned in .NET Framework 4. An evidence collection now behaves like a dictionary instead of a list. Instead of iterating over the evidence collection to see if a required evidence object exists, consumers can now request a specific type of evidence, and the collection returns the evidence if it is found. For example, the call StrongName name = evidence.GetHostEvidence(); returns a StrongName object if one exists; otherwise, it returns null.
This dictionary model delays the generation of evidence objects until they are requested. In the Publisher evidence example, the performance overhead of verifying the Authenticode signature of an assembly is delayed until that information is needed. In the most common case of full-trust applications where Publisher evidence is not needed, the verification process is avoided altogether.

Please reopen this to update the documentation accordingly
/cc @BillWagner

Please reopen this to update the documentation accordingly
/cc @BillWagner

reopening per comment.

reopening per comment.

There's a good answer from 2018 about this on StackOverflow at https://stackoverflow.com/a/48354387/1882616.
Thankfully, CAS was completely removed in .NET Core/5+, so this is only a .NET Framework problem.


Refresh should refresh all nested reference fields.

If you have a ReferenceManyField in an Edit component and you click refresh.
It will only update the data from the record itself.
It won't fire requests for the ReferenceManyFields.

Confirmed, that's a bug.

The solution is... messy
https://github.com/ReactTraining/react-router/issues/1982
React-router doesn't have a reload feature. The only thing they advise is to use window.location.reload(), which reloads not only the data but also the JS. It's not satisfactory.
Or we could trick Redux by dispatching an action that unmounts all components of the view then remounts them just after. This needs custom logic in each view though.

That's weird because on refresh list view, it does work.


Multiple widgets used the same GlobalKey

Steps to Reproduce
I'm receiving a lot of exception related to same global key usage with the below code. I've already opened an issue on StackOverflow but I think is not my code problem.

class MyHomePage extends StatefulWidget {
  MyHomePage({Key key, this.title}) : super(key: key);

  final String title;

  @override
  _MyHomePageState createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: Text(widget.title),
      ),
      body: ListView.builder(
          itemCount: 100,
          itemBuilder: (context, index) {
            return CupertinoContextMenu(
              child: Container(
                margin: const EdgeInsets.all(10),
                color: Colors.yellow,
                height: 30,
                width: 200,
              ),
              actions: <Widget>[
                CupertinoContextMenuAction(
                  child: const Text('Action one'),
                  onPressed: () async => await Navigator.of(context)
                      .push(MaterialPageRoute(builder: (context) => PageOne())),
                ),
              ],
            );
          }),
    );
  }
}

import 'package:flutter/material.dart';
import 'package:flutter/services.dart';

class PageOne extends StatefulWidget {
  @override
  _PageOneState createState() => _PageOneState();
}

class _PageOneState extends State<PageOne> {
  @override
  void initState() {
    super.initState();

    SystemChrome.setPreferredOrientations(
        [DeviceOrientation.landscapeRight, DeviceOrientation.landscapeLeft]);
  }

  @override
  void dispose() {
    SystemChrome.setPreferredOrientations(DeviceOrientation.values);
    super.dispose();
}

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(title: Text('Page One')),
    );
  }
}

════════ Exception caught by widgets library ═══════════════════════════════════
The following assertion was thrown while finalizing the widget tree:
Multiple widgets used the same GlobalKey.

The key [GlobalKey#a3ef0] was used by multiple widgets. The parents of those widgets were:
- AnimatedBuilder(animation: AnimationController#c2d69(⏭ 1.000; paused), dependencies: [MediaQuery])
- AnimatedBuilder(animation: AnimationController#c2d69(⏭ 1.000; paused), dependencies: [MediaQuery], state: _AnimatedState#5b6e5)
A GlobalKey can only be specified on one widget at a time in the widget tree.
When the exception was thrown, this was the stack
#0      GlobalKey._debugVerifyGlobalKeyReservation.<anonymous closure>.<anonymous closure>.<anonymous closure>
package:flutter/…/widgets/framework.dart:246
#1      _LinkedHashMapMixin.forEach  (dart:collection-patch/compact_hash.dart:379:8)
#2      GlobalKey._debugVerifyGlobalKeyReservation.<anonymous closure>.<anonymous closure>
package:flutter/…/widgets/framework.dart:193
#3      _LinkedHashMapMixin.forEach  (dart:collection-patch/compact_hash.dart:379:8)
#4      GlobalKey._debugVerifyGlobalKeyReservation.<anonymous closure>
package:flutter/…/widgets/framework.dart:189
...
════════════════════════════════════════════════════════════════════════════════

════════ Exception caught by widgets library ═══════════════════════════════════
Multiple widgets used the same GlobalKey.
════════════════════════════════════════════════════════════════════════════════
Launching lib/main.dart on iPhone 11 Pro in debug mode...

════════ Exception caught by widgets library ═══════════════════════════════════
Multiple widgets used the same GlobalKey.
════════════════════════════════════════════════════════════════════════════════

════════ Exception caught by widgets library ═══════════════════════════════════
Multiple widgets used the same GlobalKey.
════════════════════════════════════════════════════════════════════════════════

[✓] Flutter (Channel dev, v1.18.0-dev.5.0, on Mac OS X 10.15.4 19E287, locale en-IT)
    • Flutter version 1.18.0-dev.5.0 at /Users/enricobenedos/SDKs/flutter
    • Framework revision 7f56b53de4 (6 days ago), 2020-04-12 12:00:01 -0400
    • Engine revision beb8a7ec48
    • Dart version 2.8.0 (build 2.8.0-dev.20.0 89b0f67261)

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
    • Android SDK at /Users/enricobenedos/Library/Android/sdk
    • Platform android-29, build-tools 29.0.2
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b4-5784211)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 11.4)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 11.4, Build version 11E146
    • CocoaPods version 1.9.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 3.6)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 45.1.1
    • Dart plugin version 192.7761
    • Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b4-5784211)

[✓] VS Code (version 1.44.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.9.1

[✓] Connected device (3 available)
    • iPad (7th generation) • 2A43CEEC-D69D-48EC-88C8-BF3D3CCA63D7 • ios            •
      com.apple.CoreSimulator.SimRuntime.iOS-13-4 (simulator)
    • Chrome                • chrome                               • web-javascript •
      Google Chrome 80.0.3987.163
    • Web Server            • web-server                           • web-javascript •
      Flutter Tools

Questions like these are better suited for Q&A sites like StackOverflow or on one of the flutter community sites at https://flutter.dev/community

Hey @jonahwilliams, thank you for your advice. As you can read I've already opened an issue on StackOverflow and I've already used Slack. But I think it is not a my code problem but a CupertinoContextMenu widget problem. I tried to read documentation about it and keys. The generated exception for me is no sense. I've also tried to manually generate keys but not works. If for example I execute the same navigation code on a FAB or another widget there are no problems.


Select only desired files and folder?

Is it possible to download a certain file or folder or a combination of them?

Can you be a bit more specific?

@krigeta
Can you be a bit more specific?

like I want to download two seasons of anime season 1 and season 2 folder then in  those folders episodes are present then I need to download only selected episodes because I have some of them already. so its like folder consisting ten files but I want to download only two files from that folder and same goes for another folder.

Main Folder

season 1
-1
  -2
  -3
  -4
-5
season 2
  -2
-3
  -4

@krigeta
That's why we store the downloaded files on google drive, where you decide which files you want to download locally. Simple.
You can also remove the files you don't want from GDrive only.

Gdrive has limited storage, that's why I asked and not people are buying the storage. btw if you don't know how to do that it's okay.

@krigeta
I do know how to do it
download the files in google colab instead of gdrive
then use shell scripts to move the files you need to gdrive
done
the issue was marked closed coz that won't be solved with Python as there is no way to "download" only the necessary files
it's not a torrent client.
this repo is just a bunch of scripts.
But you are welcome to try it, create a PR I'll merge it.


Errors in StaticData_Postgres script

There was a couple errors during executing StaticData_Postgres script.

I've prepared fix for this, but I can't create a pull request. Could you please help me. Thanks.

Hi @imaximov  https://help.github.com/articles/creating-a-pull-request-from-a-fork/


Updated specialFolder list

KFM introduced the specialFolder desktop.
Additionally, the special folders cameraroll and music are available in OneDrive for Business (tested with GET https://graph.microsoft.com/v1.0/me/drive/special/{foldername} from Graph Explorer).

Docs Build status updates of commit 6bcacab:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error-Forbidden.GitAuthorization] Error occurred when accessing GitHub. Details: (Unauthorized) Bad credentials

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


luci-mod-network: dhcp tags improvements

host tag add candidates
add a tab for tags

Closing in favour of #7178 (which includes this functionality)


Links and API lib added.

Thanks for the pull request, I'll accept this for now and sort out the official applications section later on.


Added option to preserve list numbering from input

@chjj: I'm not sure if you're interested in merging this feature, but I figured I would issue a pull request just in case.
Our use-case: Our app has large blocks of technical content in the form of ordered lists and there is occasionally block-level content in the list items (tables, multiple paragraphs, etc). The block-level content breaks up the list, so when a list is resumed, the actual numbering from the input should be used instead of starting over at "1".
I realize that the more semantically correct thing in this case would be to output a single list with the block-level content nested in it -- and this is possible with some markdown implementations (RedCarpet) but it places additional burden on the user to indent the block-level content beyond the indent of the list item it belongs in. Instead of going that route, this simple workaround gives the user more control and meets the user's expectations (I'm sure the end-users of our app would be surprised by markdown's default behavior -- as I was -- of ignoring the numbering from the input).
I added tests for the option, which pass when it the option is enabled (but a couple of pre-existing OL tests fail when the option is enabled because they aren't expecting the value attribute in the output <li>s). When the option is disabled (the default), the only pre-existing tests that fail are the 4 that normally fail (10-14). I added a preserveNumbering condition to tests\index.js so the new tests are skipped if the option is disabled. When disabled, the new option has a negligible effect on benchmark performance. When enabled, it adds, on average, about 50-70 milliseconds to the test time on my machine (~2%).

Thanks, this is perfect. This should be in marked.
@lepture I love the renderer, but couldn't figure out how to make this work without forking. Any thoughts?

This would be a very welcome addition.
There is currently no viable way to use a custom renderer for this, as the renderer functions don't provide any arguments containing the number of the list item, just the content of said list item.  If the listitem function were passed a second value argument containing the list item's number, or a prefix argument with whatever comes before the content of the list item that would make the renderer approach viable.

Closing due to staleness and merge conflicts. Still an interesting feature - added to #956 as a note.


Fix trie pruner's encoder cache

Encoder cache in trie pruner should cache a node encoding by its pointer.

Unless we change pruner's logic to avoid encoding nodes in pruner at all.


Daily digest emails being sent twice

Something quirky with the cronjob?

The logic for running cronjobs was, "are you the leader of your group". Since workers now have their own separate leader instance, that one was running the cron jobs as well. I disabled workers from processing cronjobs here: 97499e7dddfe8bafbbfff222b284b13c42600a84


Add pagination / load more messages to inbox

Is your feature request related to a problem? Please describe.
When I have hundreds of messages, it takes some time for the page to load. I don't always need to load last year's messages.
Describe the solution you'd like
Introduce pagination or a 'Load More Messages' button to add more messages to the view.
Describe alternatives you've considered
Infinite scroll that loads more messages automatically. This can be confusing for users though.
Additional context
Search functionality should also be compatible. (https://github.com/mailchain/mailchain/issues/277)

Closing due to archiving repository


No need to support IE11

IE11 is at 2.x% marketshare and decreasing: https://www.w3counter.com/globalstats.php
Why support it?

ie 11 will be supported


fix get status

remove the hardcoded transaction id
improved the tests to make sure getStatus is called multiple times with the same arguments
other cleanups

:tada: This PR is included in version 0.1.4 :tada:
The release is available on:

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


Update ip-warmup-for-the-new-marketing-campaigns-experience.md

Description of the change:
Reason for the change:
Link to original source:

Closes #

Hello @xtineledlie,Thanks again for the PR! We appreciate your contribution and look forward to                                              continued collaboration. Thanks!                                             Team SendGrid DX


[Feature Request] Add ability to sort courses in degree plan

Overview
I want to be able to do this:

Supplementary Information
This would require the use of the <SortableContext> compositional wrapper and the useSortable() hook. More info on both can be found here
This might require a fair bit of code refactoring. Consult @thekevinge if you have any questions.
Link to example can be found here

For now courses are automatically sorted alphabetically and by number. See #416 for why


Refresh Recipe by removing setup tenant job

Cherry-pick: https://github.com/IBM/ibm-common-service-operator/pull/2311
Issue: https://github.ibm.com/IBMPrivateCloud/roadmap/issues/65357

tested this on my own cluster successfully, good job @YCShen1010

/lgtm


🔨Change Albums to Album

Describe the solution.
Change import variable name from Albums to Album
Describe alternatives you've considered.
We can pretend didn't see the inconsistency.
Reference
None

should we also change the endpoint to /album ? and /picture for picture page?

/albums to /album seem ok to me but if /pictures to /pictures-main is kinda weird but how about we just change the name not the url for example PictureMain to Picture.

oke lets use plural for the main page. then how about page for single album? /albums/<id> or /album/<id>

Let's use /albums/<id> since the standard writing an endpoint is using nouns, and stackoverflow too do the same thing for example.
https://stackoverflow.com/questions/ for list all the question.
https://stackoverflow.com/questions/xxxxxxxx/title-slug for single question.
Refrence :

Best practices for REST API design from Stackoverflow Blog.
REST API Best Practices – REST Endpoint Design Examples from FreecodeCamp.


feat: 🎸 implement createManyAndReturn

✅ Closes: #988

:tada: This PR is included in version 1.32.0 :tada:
The release is available on:

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


[8.x] Add context to subsequent logs.

Description:
This adds the ability to add context to all future logs along with the ability to clear that context if desired.
Use case:
A good example of this of how this might be used in middleware, lets say you wanted to bind a uuid to every request:
/**
 * Handle an incoming request.
 *
 * @param  \Illuminate\Http\Request  $request
 * @param  \Closure  $next
 * @return mixed
 */
public function handle(Request $request, Closure $next)
{
    $requestId = Str::uuid();

    Log::withContext(['request-id' => $requestId]);

    $response = $next($request);

    $response->header('request-id', $requestId);

    return $response;
}

Then when parsing logs one would be able to trace all logs pertaining to a specific request with ease. Another example where this would be very useful would be something like adding an authenticated user to the context that would be tied to all following logs.

This will not work right with octane, surely, and also with queue jobs. Stuff will bleed across requests.


Some fetched fields are removed and not passed down to react-admin

Running the following query:
query products($where: ProductWhereInput, $orderBy: ProductOrderByInput, $skip: Int, $first: Int) {
  items: products(where: $where, orderBy: $orderBy, skip: $skip, first: $first) {
    id
    title
    titleFR: title(locale: FR)
    titleEN: title(locale: EN)
    price
    customer {
      id
      label
    }
    images {
      id
      url
      height
      width
      fileName
    }
  }
  total: productsConnection(where: $where) {
    aggregate {
      count
    }
  }
}

Yields this result:

But, running it from GraphCMS directly yields a different result:

The fields titleEN and titleFR are either not fetched, or removed from the result before they get passed down to react-admin. This is likely the reason why react-admin don't show them.

I've sent manually some fake data from src/index.ts
        const res = await graphQLDataProvider(fetchType, resource, params);

        if(options.debug){
          console.log('results', res);
          res.data[0].titleFR ='Coucou'
        }

        return res

And "Coucou"  is properly being displayed by react-admin in this case. So, there is an issue with results sanitisation on the data provider's side.

The issue was because titleFR and titleEN aren't described in the graphql schema and are thus considered invalid (and stripped from results by ra-data-graphql-prisma, through the ResponseParser.
It has been fixed in https://github.com/UnlyEd/ra-data-graphql-prisma


Image not being updated

The Updater sees that there is a newer tag and seems to understand it needs to do a thing, says its doing a thing, but nothing changes.  I don't see an error, but I don't know your logs well enough to know what its doing.  I have a very basic install, all defaults plus config to talk to private registry which is working fine. Version: argocd-image-updater v99.9.9+f1cc8e4
Here are the logs, let me know if you need more specific info.
time="2021-04-08T19:29:45Z" level=info msg="Setting new image to tjsacs.azurecr.io/itac-web:v1.1.14" alias= application=itac-web-tst-az-prd-c01 image_name=itac-web image_tag=v1.1.13 registry=tjsacs.azurecr.io
time="2021-04-08T19:29:45Z" level=info msg="Successfully updated image 'tjsacs.azurecr.io/itac-web:v1.1.13' to 'tjsacs.azurecr.io/itac-web:v1.1.14', but pending spec update (dry run=false)" alias= application=itac-web-tst-az-prd-c01 image_name=itac-web image_tag=v1.1.13 registry=tjsacs.azurecr.io
time="2021-04-08T19:29:45Z" level=info msg="Committing 0 parameter update(s) for application itac-web-tst-az-prd-c01" application=itac-web-tst-az-prd-c01
time="2021-04-08T19:29:45Z" level=debug msg="found 7 from 7 tags eligible for consideration" image="tjsacs.azurecr.io/itac-web:v1.1.13"
time="2021-04-08T19:29:45Z" level=info msg="Setting new image to tjsacs.azurecr.io/itac-web:v1.1.14" alias= application=itac-web-tst-op-prd-c01 image_name=itac-web image_tag=v1.1.13 registry=tjsacs.azurecr.io
time="2021-04-08T19:29:45Z" level=info msg="Successfully updated image 'tjsacs.azurecr.io/itac-web:v1.1.13' to 'tjsacs.azurecr.io/itac-web:v1.1.14', but pending spec update (dry run=false)" alias= application=itac-web-tst-op-prd-c01 image_name=itac-web image_tag=v1.1.13 registry=tjsacs.azurecr.io
time="2021-04-08T19:29:45Z" level=info msg="Committing 0 parameter update(s) for application itac-web-tst-op-prd-c01" application=itac-web-tst-op-prd-c01
time="2021-04-08T19:29:45Z" level=info msg="Successfully updated the live application spec" application=itac-web-tst-test2-op
time="2021-04-08T19:29:45Z" level=info msg="Successfully updated the live application spec" application=itac-web-tst-az-prd-c01
time="2021-04-08T19:29:45Z" level=info msg="Successfully updated the live application spec" application=itac-web-tst-op-prd-c01
time="2021-04-08T19:29:45Z" level=info msg="Processing results: applications=3 images_considered=3 images_skipped=0 images_updated=3 errors=0"```

Hey @travis-sobeck, the Image Updater - in its default configuration - will change application parameters on the Application CRs it updates, effectively changing the Application's spec. I can just guess here (because I haven't used AppSet yet), but I'd believe that ApplicationSet will reconcile the Application spec to the version it considers to be the correct one from the templates used. This is a somewhat similar behavior to when you'd use an app-of-apps pattern, and the Application resource would be reconciled back from Git by Argo CD.
I think in this case, your best choice would be to use the
Git write-back method to persist the changes from Image Updater in Git.

@jannfis I can test that, when I looked over the write-back options the 'argocd' option seemed a lot simpler.  I can test out the 'git' method.  That being said it would be nice if the these two projects were more compatible .... (unless I'm missing something)

That being said it would be nice if the these two projects were more compatible .... (unless I'm missing something)

I do fully agree with that! We'll see how we can come up with a solution that makes interoperability between the projects work better.

@jannfis  maybe an addition to the annotation saying ... hey this is an AppSet, named  .. so spec looks like blah blah ..

@jannfis so it created the 2 .argocd-source.yaml files just fine and they look ok I think but ArgoCD just seems to be ignoring those files it created, am I missing something?

Contents of both files:
  images:
  - tjsacs.azurecr.io/itac-web:v1.1.14````

Hey @travis-sobeck, I don't know whether it got lost in formatting, but the contents really should be:
kustomize:
  images:
  - image1
  - ...

Also, are you using Argo CD 2.0? Because the Git write-back has a hard-dependency to a new feature that was released with Argo CD 2.0

Oh shoot, I'm on v1.8.7, I'll upgrade monday

Hello,
I'll barge in on this ticket because I have exactly the same thing, with the exception that in my case it's a single basic Application (Helm application).
time="2021-04-12T07:19:38Z" level=info msg="Starting image update cycle, considering 1 annotated application(s) for update"
time="2021-04-12T07:19:38Z" level=info msg="Setting new image to gcr.io/<myregistry>/testapi:1.0.1" alias= application=mytestapi image_name=<myregistry>/testapi image_tag=1.0.0 registry=gcr.io
time="2021-04-12T07:19:38Z" level=info msg="Successfully updated image 'gcr.io/<myregistry>/testapi:1.0.0' to 'gcr.io/<myregistry>/testapi:1.0.1', but pending spec update (dry run=false)" alias= application=mytestapi image_name=<myregistry>/testapi image_tag=1.0.0 registry=gcr.io
time="2021-04-12T07:19:38Z" level=info msg="Committing 0 parameter update(s) for application mytestapi" application=mytestapi
time="2021-04-12T07:19:39Z" level=info msg="Successfully updated the live application spec" application=mytestapi
time="2021-04-12T07:19:39Z" level=info msg="Processing results: applications=1 images_considered=1 images_skipped=0 images_updated=1 errors=0"

That's about all that happens normally when the update method is the default. The application isn't updated. If I change it to git, I get the following extra events:
time="2021-04-12T07:23:39Z" level=info msg="Starting image update cycle, considering 1 annotated application(s) for update"
time="2021-04-12T07:23:39Z" level=info msg="Setting new image to gcr.io/<myregistry>/testapi:1.0.1" alias= application=mytestapi image_name=<myregistry>/testapi image_tag=1.0.0 registry=gcr.io
time="2021-04-12T07:23:39Z" level=info msg="Starting configmap/secret informers"
time="2021-04-12T07:23:39Z" level=info msg="Successfully updated image 'gcr.io/<myregistry>/testapi:1.0.0' to 'gcr.io/<myregistry>/testapi:1.0.1', but pending spec update (dry run=false)" alias= application=mytestapi image_name=<myregistry>/testapi image_tag=1.0.0 registry=gcr.io
time="2021-04-12T07:23:39Z" level=info msg="Committing 0 parameter update(s) for application mytestapi" application=mytestapi
time="2021-04-12T07:23:40Z" level=info msg="Configmap/secret informer synced"
time="2021-04-12T07:23:40Z" level=info msg="configmap informer cancelled"
time="2021-04-12T07:23:40Z" level=info msg="Initializing git@gitlab.com:<mygitrepo>.git to /tmp/git-mytestapi785214539"
time="2021-04-12T07:23:40Z" level=info msg="rm -rf /tmp/git-mytestapi785214539" dir= execID=GDZEr
time="2021-04-12T07:23:40Z" level=info msg="secrets informer cancelled"
time="2021-04-12T07:23:40Z" level=info msg="git fetch origin --tags --force" dir=/tmp/git-mytestapi785214539 execID=3JDFG
time="2021-04-12T07:23:40Z" level=error msg="`git fetch origin --tags --force` failed exit status 128: No ECDSA host key is known for gitlab.com and you have requested strict checking.\r\nHost key verification failed.\r\nfatal: Could not read from remote repository.\n\nPlease make sure you have the correct access rights\nand the repository exists." execID=3JDFG
time="2021-04-12T07:23:40Z" level=info msg="Processing results: applications=1 images_considered=1 images_skipped=0 images_updated=0 errors=1"
time="2021-04-12T07:23:40Z" level=error msg="Could not update application spec: `git fetch origin --tags --force` failed exit status 128: No ECDSA host key is known for gitlab.com and you have requested strict checking.\r\nHost key verification failed.\r\nfatal: Could not read from remote repository.\n\nPlease make sure you have the correct access rights\nand the repository exists." application=mytestapi

I'm running ArgoCD v2.0.
While these errors seem pretty information, I'm wondering a few things:

does image-updater need different known_hosts to access Git repo via SSH? It seems to try to use the same credentials used to setup the application and the application itself has no issues when it comes to sync-ing. Is there a way for me to manually provide this? I know there's a configmap for ArgoCD but it has correct key verification information for Gitlab.
do I need to create the .argocd ? The docs seem to suggest the image updater will do that.

Thanks

@andrei-dascalu I think it's not yet documented but if you want to use SSH git repositories you need to provide ssh_known_hosts file. For example if you installed Image Updater in ArgoCD's namespace:
apiVersion: apps/v1
kind: Deployment
metadata:
  name: argocd-image-updater
spec:
  template:
    spec:
      volumes:
        - name: ssh-known-hosts
          configMap:
            name: argocd-ssh-known-hosts-cm
      containers:
        - name: argocd-image-updater
          volumeMounts:
            - mountPath: /app/config/ssh
              name: ssh-known-hosts

Upgrading to Argo 2.0 has resolved the issue, at least as far as the 'git" write back which in retrospect i do like having the permanent record in the repo of the last tag and an easy audit trail ..  I have not tested with the 'argo' write-back.

Also for me, the SSH known files addition has worked! Thanks!

Upgrading to Argo 2.0 has resolved the issue, at least as far as the 'git" write back which in retrospect i do like having the permanent record in the repo of the last tag and an easy audit trail .. I have not tested with the 'argo' write-back.

Thanks for confirmation, @travis-sobeck. I think you might have no luck just yet using the argocd write-back method in combination with ApplicationSet, because it will probably require also a change on the AppSet side. There are already some issues in the AppSet, e.g. https://github.com/argoproj-labs/applicationset/issues/186

@jannfis
Hi. My application is not updated but I don't know what I have missed.
P.S. I am using ECR and I have written a script to get the credentials.
Application.yaml
annotations:     argocd-image-updater.argoproj.io/image-list: my-app=xxx.dkr.ecr.ap-southeast-1.amazonaws.com/aml-automation:latest     argocd-image-updater.argoproj.io/my-app.update-strategy: digest     argocd-image-updater.argoproj.io/my-app.pull-secret: ext:/tmp/script.sh
Logs in image-updater:
time="2021-07-06T04:26:18Z" level=info msg="Starting image update cycle, considering 1 annotated application(s) for update" time="2021-07-06T04:26:18Z" level=info msg=/tmp/script.sh dir= execID=Cgvft time="2021-07-06T04:26:20Z" level=info msg="Setting new image to xxx.dkr.ecr.ap-southeast-1.amazonaws.com/my-app@sha256:58a17de6205060528f56fb735c81619bec5744bd074959ca0770ddf6ad5f3365" alias=my-app application=my-app image_name=my-app image_tag=dummy registry=xxx.dkr.ecr.ap-southeast-1.amazonaws.com time="2021-07-06T04:26:20Z" level=info msg="Successfully updated image 'xxx.dkr.ecr.ap-southeast-1.amazonaws.com/my-app@dummy' to 'xxx.dkr.ecr.ap-southeast-1.amazonaws.com/my-app@sha256:58a17de6205060528f56fb735c81619bec5744bd074959ca0770ddf6ad5f3365', but pending spec update (dry run=false)" alias=my-app application=my-app image_name=my-app image_tag=dummy registry=xxx.dkr.ecr.ap-southeast-1.amazonaws.com time="2021-07-06T04:26:20Z" level=info msg="Committing 0 parameter update(s) for application my-app" application=my-app time="2021-07-06T04:26:20Z" level=info msg="Successfully updated the live application spec" application=my-app time="2021-07-06T04:26:20Z" level=info msg="Processing results: applications=1 images_considered=1 images_skipped=0 images_updated=1 errors=0"

I believe the root cause is a inner/outer loop bug here which wrongly overwrite annotations
https://github.com/argoproj-labs/argocd-image-updater/blob/release-0.10/pkg/argocd/update.go#L325

I have issue with you @ivanpang-crypto. Did you resolve it?

there is any way that makes this works with applicaitonset? I'm not sure why components from the same project fail to work together, argocd its cool but applicationset, notifications, image updater, rollouts all kind of fail with applicationset wich make is unusable, its frustrating

I've been using the image updater with app sets for quite some time now.
I just had to make sure I was running the latest versions of everything
and use the get right back method and everything seems to work.
On Thu, Sep 23, 2021, 4:42 PM Yasmany Cubela Medina <
@.***> wrote:

there is any way that makes this works with applicaitonset? I'm not sure
why components from the same project fail to work together, argocd its cool
but applicationset, notifications, image updater, rollouts all kind of fail
with applicationset wich make is unusable, its frustrating
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/argoproj-labs/argocd-image-updater/issues/186#issuecomment-926184808,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEZKYLNW2EDFF5FRTCI6YVLUDONODANCNFSM42TQ5YGA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

it seems that only works with write back but I want to use imperative vs declarative

That was my initial approach as well, but after switching to the git
write-back method I found I liked having a permanent record in my git repo
of exactly what changed, when, and what the current version of an image
is.  It also makes it easier for anyone else to see without needing access
to argo.  I figure the git repo should be the source of truth anyways.
On Thu, Sep 23, 2021 at 8:42 PM Yasmany Cubela Medina <
@.***> wrote:

it seems that only works with write back but I want to use imperative vs
declarative
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/argoproj-labs/argocd-image-updater/issues/186#issuecomment-926281436,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEZKYLJXRTV5U5N6OCP2Q3TUDPJRJANCNFSM42TQ5YGA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

i configure the write-back method to git but now it keeps recreating my pods, example of argocd logs
time="2021-09-28T13:11:47Z" level=info msg="Initialized new operation: {&SyncOperation{Revision:d33baeaa80095e8d859b4d298f2677985134151a,Prune:true,DryRun:false,SyncStrategy:nil,Resources:[]SyncOperationResource{SyncOperationResource{Group:apps,Kind:Deployment,Name:user-register-test-listener,Namespace:,},},Source:nil,Manifests:[],SyncOptions:[CreateNamespace=true PrunePropagationPolicy=foreground],} { true} [] {5 nil}}" application=user-cmd-unstable-register-user-test
time="2021-09-28T13:11:47Z" level=info msg="Ignore status for CustomResourceDefinitions"
time="2021-09-28T13:11:47Z" level=info msg="Ignore '/spec/preserveUnknownFields' for CustomResourceDefinitions"
time="2021-09-28T13:11:47Z" level=info msg="Comparing app state (cluster: https://kubernetes.default.svc, namespace: default)" application=user-cmd-unstable-register-user-test
time="2021-09-28T13:11:47Z" level=info msg="getRepoObjs stats" application=eck build_options_ms=0 helm_ms=0 plugins_ms=0 repo_ms=0 time_ms=345 unmarshal_ms=345 version_ms=0
time="2021-09-28T13:11:48Z" level=info msg="getRepoObjs stats" application=user-cmd-unstable-register-user-live build_options_ms=0 helm_ms=0 plugins_ms=0 repo_ms=0 time_ms=4071 unmarshal_ms=4071 version_ms=0
time="2021-09-28T13:11:48Z" level=info msg="getRepoObjs stats" application=user-cmd-unstable-register-user-test build_options_ms=0 helm_ms=0 plugins_ms=0 repo_ms=0 time_ms=4081 unmarshal_ms=4081 version_ms=0
time="2021-09-28T13:11:48Z" level=error msg="Failed to initiate auto-sync to d33baeaa80095e8d859b4d298f2677985134151a: rpc error: code = FailedPrecondition desc = another operation is already in progress" application=user-cmd-unstable-register-user-test
time="2021-09-28T13:11:48Z" level=error msg="Failed to initiate auto-sync to d33baeaa80095e8d859b4d298f2677985134151a: rpc error: code = FailedPrecondition desc = another operation is already in progress" application=user-cmd-unstable-register-user-live
time="2021-09-28T13:11:48Z" level=info msg="Update successful" application=user-cmd-unstable-register-user-live
time="2021-09-28T13:11:48Z" level=info msg="Reconciliation completed" application=user-cmd-unstable-register-user-live dedup_ms=0 dest-name= dest-namespace=default dest-server="https://kubernetes.default.svc" diff_ms=25 fields.level=824814551928 git_ms=4071 health_ms=4 live_ms=1 settings_ms=0 sync_ms=0 time_ms=4309

it seems that argocd its fighting the update. any ideas?
PD: I'm using argocd, application set and image update just to be clear

I'm currently facing the same issue. I have Application, and my setup is similar to what is described by @ivanpang-crypto above. I'm using argocd as the write-back method (the default).
Can anyone suggest how to resolve this issue?

I have issue with you @Dwisf . Did you resolve the problem?

@tamanh123456 :

I have issue with you @Dwisf . Did you resolve the problem?

Not yet. Still waiting for response. But I have created another ticket with more details, here: https://github.com/argoproj-labs/argocd-image-updater/issues/431

Hey @travis-sobeck, the Image Updater - in its default configuration - will change application parameters on the Application CRs it updates, effectively changing the Application's spec. I can just guess here (because I haven't used AppSet yet), but I'd believe that ApplicationSet will reconcile the Application spec to the version it considers to be the correct one from the templates used. This is a somewhat similar behavior to when you'd use an app-of-apps pattern, and the Application resource would be reconciled back from Git by Argo CD.
I think in this case, your best choice would be to use the Git write-back method to persist the changes from Image Updater in Git.

I have the exact same problem as @travis-sobeck. AppSet creating Application from value file. But the container is not updated.
I have also used a git write-back to the repository and can see that proper sha256 is in the .argocd yaml file in the repository.
But it still does not update the POD spec and the container is based on the old image. :-(
Anyone actually know if the combination of AppSet and Image Updater can work together? If so, are there certain tweaks/workarounds that need to be done?

So I finally got this to work. I will leave this here for others who may run into the same issue. Note that the helm-spec did not work for me. But the following did.
apiVersion: argoproj.io/v1alpha1
kind: ApplicationSet
metadata:
  namespace: argocd
  name: my-apps
spec:
  generators:
  - git:
      repoURL: ssh://git@my.vcs.dk:7999/services/my-service.git
      revision: HEAD
      files:
      - path: k8s/my-service/configs/dev/*-values.yaml
  template:
    metadata:
      annotations:
        argocd-image-updater.argoproj.io/image-list: '{{nameOverride}}={{image.repository}}:latest'
        argocd-image-updater.argoproj.io/write-back-method: git:repocreds
        argocd-image-updater.argoproj.io/git-branch: HEAD
        argocd-image-updater.argoproj.io/{{nameOverride}}.update-strategy: digest
        argocd-image-updater.argoproj.io/{{nameOverride}}.force-update: 'true'
        argocd-image-updater.argoproj.io/{{nameOverride}}.helm.image-name: '{{nameOverride}}={{image.repository}}:latest'
        argocd-image-updater.argoproj.io/{{nameOverride}}.helm.image.tag: '{{image.tag}}'
      name: '{{nameOverride}}-dev'
    spec:
      project: argocd-availability-dev
      source:
        repoURL: ssh://git@my.vcs.dk:7999/services/my-service.git
        targetRevision: HEAD
        path: 'k8s/availability-service'
        helm:
          releaseName: '{{nameOverride}}'
          valueFiles:
          - './configs/dev/{{nameOverride}}-values.yaml'
      destination:
        server: https://kubernetes.default.svc
        namespace: '{{namespace}}'
      syncPolicy:
        automated:
          prune: true
          selfHeal: false
          allowEmpty: false
        syncOptions:
        - CreateNameSpace=true

@TheDukeDK, your configuration only seems to work if targetRevision = argocd-image-updater.argoproj.io/git-branch. When git-branch differs (unfortunately this is needed in my case because targetRevision is a protected branch), ApplicationSet seems to just reconcile reverting IU change to image. I could be misinterpreting the behavior I'm seeing but this seems to be the case.

Had to add helm parameters to get it to work, see my comment here: https://github.com/argoproj-labs/argocd-image-updater/issues/431#issuecomment-1471871799

If it's related to Image updater + ApplicationSet, then there is a work arround by changing ApplicationSet controller policy.
See my comment here https://github.com/argoproj/applicationset/issues/547#issuecomment-1674693159

@kyleli666 That's the ANSWER I was looking for. It' s an issue with combination of ApplicationSet and Image Updater.

what's the point? creating a new branch from the main one with a dynamic name?
argocd-image-updater.argoproj.io/git-branch: main:image-updater-{{.SHA256}}
if you substitute in targetRevision: image-updater-{{.SHA256}} then argocd does not understand this format.
that is, the point is that image-updater creates a new branch in which it fixes the version of the image tag and ArgoCD should monitor this branch with the name image-updater-{{.SHA256}} through targetRevision.
in my case I have two clusters and two ArgoCDs and it’s not clear how to manage the branches

Thank you @kyleli666 🫶


Custom Title for labels

Hi @daytime-em!
This action is amazing. I'm planning to use this at my workplace and I was wondering if there's a way to add a new property for setting custom titles for labels.
For example:
This action accepts a words and labels array and assigns labels if the word is present in the title.
I am suggesting similar but there would be a titles property that would correspond to the labels and would replace the label with that title instead of just capitalizing the first letter of the label.
I would be happy to help you to add this feature! Let me know if you like my idea and I can make a PR for this.

Thank you for your interest!
I can probably add a custom title feature. I'll do it as you asked, sometime this week.

@daytime-em Thank you such much! Please let me know if you need any help!


HttpContentDecompressor should change decompressed requests to chunked encoding

In Netty, how do I know if an incoming HTTP request message has a body or not immediately without waiting for subsequent content messages?
If the answer to that is you can't, then there is no bug here, you can close this issue.
If the answer to that is by following the HTTP spec - ie, if the request has a Content-Length or Transfer-Encoding header, then you know it has a body, otherwise, you know it doesn't have a body - then there is a bug in HttpContentDecompressor, since it removes the Content-Length header from request, making it impossible to reliably determine whether a message has a body or not by looking at these two headers.
My opinion is that semantically the HttpContentCompressor is turning the request into one that uses a transfer encoding to indicate the size of the request message - ie, you now need an empty (last) content message to know the end of the message.  So if it's going to do that, in order to not break downstream handlers, it should add a chunked transfer encoding header - this means downstream handlers that use the content length and or transfer encoding headers according to the spec won't break.  Just removing the content length header violates the HTTP spec, and seeing as Netty generally requires apps to implement and enforce much of the HTTP spec themselves, Netty shouldn't confuse things by breaking it.

Looks like a bug to me as HttpContentEncoder does add Transfer-Encoding header here but HttpContentDecoder does not.

@NiteshKant agreed. want to submit a PR with a fix and unit test ?

@normanmaurer sure! I would be able to do this soon.

Thank you guys ! I had a hard time analyzing the issue in Play.


it does not work!

vscode 1.68.1
ubuntu 18.04
log like follow:
[appData] read file /home/jixiangi/examples/hello_cmake_rtp/RTP/.vscode/wrproject.json error
in directory RTP not find .vscode.

Sorry we didn't reply ealier as we didn't use issue tracker here yet. If you have VxSDK available and still have problem, please contact me.


Cannot import the module

github.com/openshift-telco/go-netconf-client/netconf<EMAIL_ADDRESS>parsing go.mod:
module declares its path as: github.com/openshift-telco/go-netconf
but was required as: github.com/openshift-telco/go-netconf-client

Hi
Please make sure to use the right version. The only released version is v0.0.1
Here is an example of a project using this client: https://github.com/openshift-telco/netconf-operator/blob/main/go.mod#L9 and it is working fine.
Let me know if you have further questions.

Thanks. It worked.


fixed SIL problem

I'm stupid, apparently

Good grief!  How long is the CI test supposed to take?


Add remark command

Codecov Report
Base: 72.15% // Head: 71.48% // Decreases project coverage by -0.67% :warning:

Coverage data is based on head (f7503f4) compared to base (fac17d9).
Patch coverage: 64.28% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff              @@
##             master       #4      +/-   ##
============================================
- Coverage     72.15%   71.48%   -0.68%
- Complexity      399      409      +10
============================================
  Files            70       73       +3
  Lines          1232     1294      +62
  Branches        125      133       +8
============================================
+ Hits            889      925      +36
- Misses          311      334      +23
- Partials         32       35       +3

Impacted Files
Coverage Δ

.../seedu/address/logic/parser/AddressBookParser.java
88.88% <0.00%> (-11.12%)
:arrow_down:

...in/java/seedu/address/logic/parser/ParserUtil.java
94.59% <0.00%> (-2.63%)
:arrow_down:

...eedu/address/logic/parser/RemarkCommandParser.java
0.00% <0.00%> (ø)

src/main/java/seedu/address/ui/PersonCard.java
0.00% <0.00%> (ø)

.../seedu/address/logic/parser/EditCommandParser.java
85.71% <33.33%> (-6.60%)
:arrow_down:

...c/main/java/seedu/address/model/person/Remark.java
62.50% <62.50%> (ø)

...va/seedu/address/logic/commands/RemarkCommand.java
70.83% <70.83%> (ø)

...java/seedu/address/logic/commands/EditCommand.java
95.89% <87.50%> (-1.13%)
:arrow_down:

...a/seedu/address/logic/parser/AddCommandParser.java
100.00% <100.00%> (ø)

...ain/java/seedu/address/logic/parser/CliSyntax.java
85.71% <100.00%> (+2.38%)
:arrow_up:

... and 4 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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


Include wind per hour

This app can be useful for MET Norway
https://f-droid.org/en/packages/uk.org.boddie.android.weatherforecast/
https://gitlab.com/dboddie/weather-forecast

This will be added with graphs sometime in the future. Including wind info to the current design would complicate the hours. Thank you for the suggestion!


stm32g0[bc]1: Rename USB_DRD_FS like USB_FS

register
existing name
g0 name

EPRn
CHEPRn

CHEPRn
CTR_RX
VTRX

EP_MEM
not real (?)

USB_CNTR
FRES
USBRST

USB_BTABLE
(always zero)

USB_CNTR
FSUSP
SUSPEN

USB_CNTR
LP_MODE
part of SUSPEN

USB_ISTR
RESET
RST_DCON

Thanks for the PR, and sorry for the wait!


The size of the bundle 4MB

I tested a build of a minimal generator-solid-react project and found that the solid-react-components currently is a few bytes short of 4MB that will be 1MB gzipped over the line but will still be 4MB of code in the browser memory usage. I saw that in issues #15 and #106 changes were made to reduce the size to 979 KB. I think it's time for a new review.

I've gone through the build and picked some low hanging fruit:

solid & inrupt logo base64 encoded in build
why 26Kb (15Kb) worth of guids(?) in build

@shexjs shex-loader-browserify.js solid-react-components

mimetype list 132Kb (19Kb)
-@shexjs shex-loader-browserify.js
list of suspect? domain names 122Kb (34Kb)

tough-cookie uses psl

moment complete is big 540 Kb (75Kb)
fontawesome is big 1.203Kb (370Kb)
this could shave 2mb (500Kb) off of the build


Organize Alumni on all Almuni Page by Graduation Year (Ascending or Descending?)

I think ascending makes the most sense (more recent years at the top).


Fix GetListing() regression caused by PR #764 - many untrue parse fail warnings

urgent
GetListing() works but PR 764 causes very many unneccessary parse fail warnings.
An else keyword somehow disappeared on the commit/push sequence.

Sorry for the long delay, just got time and your entire listing of PRs looks like some truly amazing work. Keep it up!


Add explicit compiler version

Description
Add Java 8 reference to Gradle for Connectivity Plus. This is necessary because the default toolchain will emit a warning that Java 7 is facing deprecation (and rightly so!) but because -Werror is also specified, this warning is treated as a build-breaking error.
Related Issues

#319

Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]).
This will ensure a smooth and quick review process. Updating the pubspec.yaml and changelogs is not required.
Dart code was not changed, only Gradle configs.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[X] My PR includes unit or integration tests for all changed/updated/fixed behaviors (See Contributor Guide).
[X] All existing and new tests are passing.
[X] I updated/added relevant documentation (doc comments with ///).
[X] The analyzer (flutter analyze) does not report any problems on my PR.
[X] I read and followed the Flutter Style Guide.
[X] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (please indicate a breaking change in CHANGELOG.md and increment major revision).
[X] No, this is not a breaking change.

Thanks! this is something that should be done on all packages, right?

@miquelbeltran Possibly - I only use three of these packages (connectivity, device_info, and package_info), and only connectivity failed to build.
Which in itself is a bit of a flag, indicating inconsistent build processes, but that's a different matter, one that likely requires greater insight into the intended operation than I have.

understood, thanks! I will have to do a new release as well so I may push some commits here, otherwise I will open a new PR with the changes. Thanks for your help!

this has been released until connectivity_plus 1.0.4


Netlify page for meet-up sessons

Summary
Create a Netlify page to support community engagement in meet-up sessions
latest version -----> https://www.notion.so/Welcome-to-the-AutSPACEs-Meet-Up-Page-0ad17d5c0f8045be8618686531068622
Details
We need a page to support fortnightly sessions which are

help orientate users with info such as time, agenda, schedules
accessible to diverse autistic collaborators
clear and contain comprehensive information
point to useful additional resources and links

Suggestions for tasks that can fix this issue

[x] Survey rseearchers on info needed
[x] Create page
[ ] Upload design specs doc to repo
[ ] Make sure it meets accessibility requirements
[ ] Test with diverse autistic users
[ ] Iterate!

Who can help
This task is owned by @BrainonSilicon but feedback/comments/suggestions welcome from the community!

Got a wiki for that, cf https://github.com/alan-turing-institute/AutSPACEs/wiki/Meetups  😉