No Border Radius When Used In Combination With react-native-animatable

I am using this in combination with react-native-animatable in this way (as suggested in the documentation):
const AnimatedViewOverflow = Animatable.createAnimatableComponent(ViewOverflow);
<AnimatedViewOverflow style={...}>
    {children}
</AnimatedViewOverflow>

The animation and overflow both work, but I cannot apply a border radius to the AnimatedViewOverflow element and attempting any encapsulation of the data with a border radius stops content from overflowing.
Using:
"react-native": "0.55.3"
"react-native-animatable": "^1.2.4"
"react-native-view-overflow": "^0.0.3"

Hello!
I was able to resolve my problem by following this advice: https://github.com/entria/react-native-view-overflow/issues/10#issuecomment-392323613
My RNViewOverflowManager.java file ended up looking like this:
package com.entria.views;

import com.facebook.react.uimanager.ThemedReactContext;
import com.facebook.react.uimanager.ViewGroupManager;
import com.facebook.react.uimanager.annotations.ReactProp;

public class RNViewOverflowManager extends ViewGroupManager<RNViewOverflowLayout> {

    public static final String REACT_CLASS = "RNViewOverflow";

    @Override
     protected RNViewOverflowLayout createViewInstance(ThemedReactContext reactContext) {
        return new RNViewOverflowLayout(reactContext);
     }

    @Override
    public String getName() {
        return REACT_CLASS;
    }

    @ReactProp(name = "borderRadius", defaultFloat = 0f)
    public void setBorderRadius(RNViewOverflowLayout view, float borderRadius) {
      view.setBorderRadius(borderRadius*3.5f);
    }
}

I added the multiplier of 3.5f because the border radius I applied appeared more slight than other border radii.


added username validation on client side

Description
This pull request implements client-side validation for usernames during user registration. The validation ensures that usernames meet specified criteria (e.g., length, character restrictions) before submission, enhancing user experience and reducing server-side errors.
Motivation and Context: The need for username validation arose from user feedback indicating confusion over username requirements.
Fixes #45
Type of Change

[x] New feature (non-breaking change which adds functionality)

How Has This Been Tested?
Manual Testing
Checklist

[x] My code follows the style guidelines of this project.
[x] I have performed a self-review of my code.
[x] I have commented my code, particularly in hard-to-understand areas.
[ ] I have made corresponding changes to the documentation.
[x] My changes generate no new warnings.
[ ] I have added tests that prove my fix is effective or that my feature works.
[ ] New and existing unit tests pass locally with my changes.

Summary by CodeRabbit

New Features

Enhanced username validation in the signup form, providing immediate feedback on username requirements.
Error messages displayed conditionally below the username input field for improved user experience.

The code logic is not working.
Its just showing the error and we are still able to signup with any input (except same username and email error)
And also if my username is more than 11 characters then its no meaning of available (not handled)

Please check and test the code.

Also if possible can you also add error handling logic for password as well like a minimum of 6 characters (follow any password guidelines like NIST)
Also keep a small info button ⓘ stating all the rules (mark green to the rules which has passed and red for the others)
eg: password entered : Dev1
Rules

contains mix of alphabets and numbers (green)
Should be minimum of 6 characters (red)
contains a special symbol (red)
.. and so on

It is working fine.
Merging.


using coc-clangd behind a firewall

I am trying to use coc-clangd at work where we have a firewall, preventing access to external resources (like github).  There is however a way to have a local mirror of github repos.  This is where i have copies of coc.nvim and clangd/coc-clangd:
Installing this extension, however, doesnt work out of the box:
First method
:CocInstall http://companymirror/github/clangd/coc-clangd.git results in  is not supported, coc.nvim support github.com only
Second method:
Plug 'clangd/coc-clangd', {'do': 'yarn install --frozen-lockfile'} results in an error:
warning package.json: License should be a valid SPDX license expression
warning<EMAIL_ADDRESS>License should be a valid SPDX license expression
[1/5] Validating package.json...
warning<EMAIL_ADDRESS>License should be a valid SPDX license expression
warning<EMAIL_ADDRESS>The engine "coc" appears to be invalid.
[2/5] Resolving packages...
[3/5] Fetching packages...
error An unexpected error occurred: "https://registry.yarnpkg.com/readdirp/-/readdirp-3.4.0.tgz: self signed certificate in certificate chain".
info If you think this is a bug, please open a bug report with the information provided in "/home/user/.vim/plugged/coc-clangd/yarn-error.log".
info Visit https://yarnpkg.com/en/docs/cli/install for documentation about this command.

Logs
Arguments:
  /usr/local/node-v12.18.0-linux-x64/bin/node /usr/share/yarn/bin/yarn.js install --frozen-lockfile
PATH:
  /usr/local/node-v12.18.0-linux-x64/bin:/software/centos7/binutils/2.30/bin:/software/centos7/ccache/3.2.2/bin:/software/centos7/cmake/3.13.2/bin:/software/centos7/gdb/8.1/bin:/software/centos7/gcc/8.2.0/bin:/home/user/bin/fzf/bin:/usr/local/bin:/usr/bin:/usr/local/sbin:/usr/sbin:/home/user/.fzf/bin:/usr/lib/llvm10/10.0.0/bin/:/home/user/.local/bin:/home/user/bin/fzf-0.17.5/:/home/user/bin
Yarn version:
  1.22.4
Node version:
  12.18.0
Platform:
  linux x64
Trace:
  Error: self signed certificate in certificate chain
      at TLSSocket.onConnectSecure (_tls_wrap.js:1506:34)
      at TLSSocket.emit (events.js:315:20)
      at TLSSocket._finishInit (_tls_wrap.js:948:8)
      at TLSWrap.ssl.onhandshakedone (_tls_wrap.js:706:12)

System information
Clangd version: 10.0.0
clangd extension version (from :CocInfo):
Operating system: CentOS 7.3

yarn config set "strict-ssl" false -g allowed me to turn off the certificate checking and proceed to install the extension.


add a copy of INVIOLABLES.txt converted to rst to the new docs

Summary

add a copy of INVIOLABLES.txt converted to rst to the new docs

Impact

Two copies of the INVIOLABLES will be present: INVIOLABLES.txt and Documentation/introduction/inviolables.rst.

Details

It seems important to have INVIOLABLES in the new doc system. I tried to convert it to Markdown and include it that way. However, Sphinx toctree can't load documents outside the Documents/ tree, and using include directives also didn't work. Using a symlink did work, but it doesn't seem like such a good idea to have a symlink in version control.
I'm submitting this here so we can have a discussion about whether this approach is ok for now.

Testing

Built the docs manually.

I think the duplication is okay.  If puts a copy in the Documentation where it makes sense.  But keeping the top-level INVIOLABLES.txt is very important too.  I am very glad you did not remove it.  I want it to be in the top-level directory where is is visiable to all can cannot be overlooked.  Everyone should working on this project should skim that .txt file.

I really do not think it is worth duplicating, there's no good reason to do so. You can include the file from Sphinx easily. We discussed converting it to Markdown which is fairly trivial. This way it looks good on Sphinx without any kind of tags and we have a single file.

I really do not think it is worth duplicating, there's no good reason to do so. You can include the file from Sphinx easily. We discussed converting it to Markdown which is fairly trivial. This way it looks good on Sphinx without any kind of tags and we have a single file.

If there is only one copy that can be in one place, then it MUST be in the top-level directory.  It must be unavoidable and not hidden deep in some documentation directory.
It is not a maintenance issue (since it should not change), so I don't see any problem with two copies either.

I really do not think it is worth duplicating, there's no good reason to do so. You can include the file from Sphinx easily. We discussed converting it to Markdown which is fairly trivial. This way it looks good on Sphinx without any kind of tags and we have a single file.

If there is only one copy that can be in one place, then it MUST be in the top-level directory. It must be unavoidable and not hidden deep in some documentation directory.
It is not a maintenance issue (since it should not change*), so I don't see any problem with two copies either.
Although I hoped some will fix the outdated URL at line 59 and I think the correct word in line 123 is suit, not suite.

I understand it might not change but I think there's no good reason to duplicate it. Yes, the idea is to leave it in top-level directory.
I can do this later.

@v01d If you can get it included from the top level directory, that would be fantastic. I could not do it.
Here's a couple of links that I saw in my research:

StackOverflow on including files outside document root – there is a workaround for .rst files but not for .md files
https://github.com/sphinx-doc/sphinx/issues/701 – very old but still active discussion about relative links in toctree directive

One easy way is to include it verbatim. I did this with the license in licensing.rst:

.. include:: ../../LICENSE
  :literal:

@patacongo Ok, fixed the suit typo and URL to coding standard in both files.

@v01d Yeah I guess doing it with literal works. It would be nice to have a formatted version though. Is there another way you know to do this?

I'll look into it. It is possible that including an external markdown is not possible as you mentioned.

The docs build with a makefile so we can add logic there to setup the build environment like moving files around. I agree that it should stay at the top and not be duplicated. We will have to address this anyway for things like the board information.

The docs build with a makefile so we can add logic there to setup the build environment like moving files around. I agree that it should stay at the top and not be duplicated. We will have to address this anyway for things like the board information.

I thought about the same but GH actions builds running sphinx-build directly so that would not work. Maybe we can run make on Documentation on GH actions if that helps later on.

I have no attachment to the action :)

Ok, I managed to make it work. It is based on https://pypi.org/project/m2r2/
It works nicely to include files from the toc or inline and supports including from outside doc root. I can make a PR with it.

@adamfeuer if you want you can make a separate PR converting INVIOLABLES file to Markdown and then use mdinclude directive to include it somewhere (under contributing? or maybe in the intro).

@v01d Ok, I figured how to do what you suggested. the m2r python module is not being maintained and doesn't work for mdincludes above the doc root. But m2r2 is a fork that does work.
@patacongo This PR now converts the INVIOLABLES.txt to a Markdown file, there is a slight change in formatting to how the Enemies subpoints are marked as subheadings, will you see if this is ok with you?

@v01d Whoops, didn't see your previous comment. We came to the same solution. This PR does what you suggested. If it's ok, let me know, I'll resolve the conflicts and rebase.

You can try rebasing. Right now the only thing remaining would be conversion to Markdown and inclusion in docs. BTW, if you're going to have a standalone section for the inviolables, you can simply list it in the toctree (unless that fails, in which case you can do as you did).

@v01d Yes, putting it directly in the toctree failed for me, so I think we need to do it this way.
BTW I'm changing the Pipfile back to having "*" for its dependencies; the Pipfile.lock will hold the specific version that is used to do the last build by someone. That way we can easily check that new versions are working ok, while locking CI and everyday usage to specific known-good versions.

@v01d Ok, it seems like force-pushing this messed something up and closed the PR. I'm going to open a new one.

@v01d Reopen button seems to be activated again... strike that, I'll reopen this one.

BTW I'm changing the Pipfile back to having "*" for its dependencies; the Pipfile.lock will hold the specific version that is used to do the last build by someone. That way we can easily check that new versions are working ok, while locking CI and everyday usage to specific known-good versions.

I really don't know much about pyenv, but what I would like to do is to ensure both CI and users building locally use the exact same versions. Sphinx recently fixed something that affected building the documentation. Also, I want to avoid any compatibility issues if any of the python packages change versions. We can always periodically update dependencies, but it is best to stick with known working versions.
If what you suggest still keeps this working (note that requirements.txt is used by CI), you can go ahead with this change.

The commit message has a really long list of unrelated commits. Would you mind editing that so that it only describes this commit? Sorry for the hassle. Rebasing sometimes is painful.

@btashton can you quickly take a look? As you requested I update the Pipfile in the other PR and this one partly overrides that I would like to confirm this is OK for everyone.

Just looked again at the changes and it looks like the INVIOLABLES file is strangely formatted now, it uses "=" and "-" for some headings and "#" prefix for others (it should use the latter).

@v01d Re: the Markdown formatting in INVIOLABLES.md – it has to be formatted that way, since in Markdown, heading 1 uses = and heading 2 uses -, and there are no other heading styles like that; heading 3 needs to be ###. That's not the case in RST, which has many different heading styles.
If we don't do it this way, the left-hand side tree menu generated by Sphinx will not work correctly.
Alternatively, we can format the file with # for heading  1 and ## for heading 2, but that seemed to get further away from the way the file was originally formatted.
If you have a better idea, let me know.

Alternatively, we can format the file with # for heading 1 and ## for heading 2, but that seemed to get further away from the way the file was originally formatted.

This is the usual approach with Markdown (the one with underlines is considered "alternative"), all other md files already use this style as well.

@v01d Ok I made all headings use the # formatting.

@v01d @btashton I squashed again and force pushed, so this should be good to merge now.

Ok, LGTM to me. I'll let @btashton merge.


Show full formatted error message

Hello, thanks a lot for making this plugin!
I noticed that if the formatted error message is too long (longer than the length of the hover window), the hover window doesn't show the full text; shows only a cut-off message. Is there a way to get the full message or to scroll through the error messages?

Hi, @aonemd! In the latest neovim you can press <C-w>d to show the error in a popup window, and then <C-w>d again to focus it
Then you close the popup by q or <C-w>c. Regular lsp hover (usually captial K) also works that way, so you can read a long docstring

also check your tsconfig settings
https://www.typescriptlang.org/tsconfig/#noErrorTruncation

closed, not in this plugin's scope


Feat setup authflow

Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
This PR introduces Google Authentication.
Fixes #61
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration.

[ ] Test A
[ ] Test B

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

@brianwachira will these changes work on a standalone build?

With #65 do we still need this?

With #65 do we still need this?

@brianwachira


Cosmos DB connection mode switched to Gateway automatically

Query/Question
We use Direct Connection Mode to create a cosmos client on Mar 17 and noticed an interruption on Mar 29 with failure in connecting to the cosmos account because of SSLException:
2021/03/29 23:21:28.944 ERROR [RxGatewayStoreModel] [reactor-http-epoll-1] [unified-topology-war] [] Network failure
javax.net.ssl.SSLException: handshake timed out
Upon looking at the logs, We noticed "connectionMode":"GATEWAY" is set in "cosmosDiagnostics" however we setup the client to use DirectMode:
Unable to refresh database account from any location. Exception: {"ClassName":"CosmosException","userAgent":"azsdk-java-cosmos":0,"resourceAddress":"https://<cosmosaccount>:443/","innerErrorMessage":null,"causeInfo":"[class: class javax.net.ssl.SSLException, message: handshake timed out]","cosmosDiagnostics":{"userAgent":"azsdk-java-cosmos/4.11.0","requestLatencyInMs":29335,"requestStartTimeUTC":"2021-03-29T23:53:12.237Z","requestEndTimeUTC":"2021-03-29T23:53:41.572Z","connectionMode":"GATEWAY","responseStatisticsList":[],"supplementalResponseStatisticsList":[],"addressResolutionStatistics":{},"regionsContacted":[],"retryContext":{"retryCount":0,"statusAndSubStatusCodes":null,"retryLatency":0},"metadataDiagnosticsContext":{"metadataDiagnosticList":null},"serializationDiagnosticsContext":{"serializationDiagnosticsList":null},"gatewayStatistics":{"sessionToken":null,"operationType":"Read","statusCode":0,"subStatusCode":10001,"requestCharge":null,"requestTimeline":null},"systemInformation":{"usedMemory":"5712527 KB","availableMemory":"1744241 KB","systemCpuLoad":"(2021-03-29T23:53:22.033Z 40.0%), (2021-03-29T23:53:22.035Z 100.0%), (2021-03-29T23:53:22.037Z 100.0%), (2021-03-29T23:53:31.868Z 97.8%), (2021-03-29T23:53:31.873Z 40.0%), (2021-03-29T23:53:41.555Z 97.9%)"},"clientCfgs":{"id":0,"numberOfClients":1,"connCfg":{"rntbd":"(cto:PT5S, rto:PT5S, icto:PT0S, ieto:PT1H, mcpe:100, mrpc:100, cer:false)","gw":"(cps:1000, rto:PT5S, icto:null, p:false)","other":"(ed: true, cs: false)"},"consistencyCfg":"(consistency: Session, mm: false, prgns: [eastus])"}}}
Caused by: javax.net.ssl.SSLException: handshake timed out

Client Creation:
CosmosClientBuilder cosmosClientBuilder = new CosmosClientBuilder()
      .endpoint(serviceEndpoint)
      .key(key)
      .contentResponseOnWriteEnabled(contentResponseOnWriteEnabled)
      .consistencyLevel(ConsistencyLevel.valueOf(consistencyLevel))
      .directMode(directConnectionConfig)
      .throttlingRetryOptions(throttlingRetryOptions);

cosmosClientBuilder.buildClient()

If we created the client via Direct Mode, why did the connectionMode switch to Gateway is a puzzle for me and I would appreciate help to understand any scenario in which this could happen.
A restart of the service fixed the issue but we are not certain what happened.
Why is this not a Bug or a feature Request?
This is not certainly a bug or a feature request. It is a scenario query.
Setup (please complete the following information if applicable):

OS: Linux
IDE : None
Version: https://mvnrepository.com/artifact/com.azure/azure-cosmos/4.11.0

Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report

[ ] Query Added
[ ] Setup information Added

@kushagraThapar could you please take a look?

@NikhitaKataria - There is no way for the Cosmos SDK to update the connection mode to Gateway mode automatically.
This seems to be a bug of Cosmos Diagnostics. Can you please log some other diagnostics on the responses (query / read item), etc. So as to check what connection mode we get on the other responses.
Note - Even in Direct connection mode, some operations still go through Gateway - like meta data operations, fetching the query plan to execute query, etc.
Above exception happened on getting the database account - which is a Gateway operation - may be because of it, the connection mode is showing as GATEWAY, however, it should still show it as DIRECT mode.


restore simpler, old-style scrolling prevention

The current scroll prevention code tries to be clever but doesn't work smoothly in internet explorer and misses some edge cases. Instead the host application should decide what to do when a layer is open. Hence, only the bootstrap css class modal-open should be set on the body.

Implemented on master (v0.2.0).


The model cannot be downloaded

from https://drive.google.com/open?id=0B-TTdm1WNtybdzdMUHhLc05PSE0&authuser=0. The source cannot be read.

Hum, weird. I just tried it in two different machines, and it worked just fine. Is Google Drive blocked in your country ?

Ok. I managed to download it today. It seems it was a temporal problem. Thanks.


Refactoring timeouts

Coverage decreased (-0.3%) to 29.463% when pulling<PHONE_NUMBER>2ec45e7dabe2a5292be73197d6bc86 on refactoring_timeouts into 2ef79eebb6f3d61b2ba90f693592c7bc3a9a1ce5 on master.


Refactor css to mobile-first approach in LandingPage

Summary:

refactor media queries
fix some style inconsistencies with design

Just few small things not directly related to styling. But to clean console a bit. Then I think we can merge it.
https://github.com/kiwicom/margarita/blob/8135786d6340ed8af29ff404df229ca30735ad29/apps/landingPage/src/components/Headline.js#L28-L32

This causes <h1> cannot appear as a descendant of <p> error. So we should probably use just div as a wrapper instead of <Text>. Same for row 40.

https://github.com/kiwicom/margarita/blob/8135786d6340ed8af29ff404df229ca30735ad29/apps/landingPage/src/components/Demo.js#L32-L34

Right now we have a same link for github and docs. Which causes Keys should be unique... error. We need to change it to something unique (maybe just title can work).

Just few small things not directly related to styling. But to clean console a bit. Then I think we can merge it.
margarita/apps/landingPage/src/components/Headline.js
Lines 28 to 32 in 8135786

     Ever dreamt of building the next Kiwi.com?

* This causes ` cannot appear as a descendant of ` error. So we should probably use just `div` as a wrapper instead of ``. Same for row 40.
margarita/apps/landingPage/src/components/Demo.js
Lines 32 to 34 in 8135786

{el.title}

* Right now we have a same link for github and docs. Which causes `Keys should be unique...` error. We need to change it to something unique (maybe just title can work).

Yeah, nesting <Text> and <Heading> was ugly. Hopefully <Heading>  will get align prop as well.  Now its wrapped in div
Alright, changed for title


Create new Octopus.Tentacle.Common library

Background
For the #project-k8s-agent, to execute the script in the Kubernetes Job and have the stderr and stdout be correctly written to Output.log, I need to use the same code (mainly ScriptLogWriter and SilentProcessRunner) as happens for local shell scripts being executed by Tentancle
Results
This PR introduces a new library project, Octopus.Tentacle.Common which now contains some of the code that lived in Octopus.Tentacle. I chose to only move the files I need for the k8s script runner.
I also didn't change the namespaces of the files.
Shortcut story: [sc-59580]
How to review this PR

Quality :heavy_check_mark:

Pre-requisites

[ ] I have read How we use GitHub Issues for help deciding when and where it's appropriate to make an issue.
[x] I have considered informing or consulting the right people, according to the ownership map.
[x] I have considered appropriate testing for my change.

Might be good to have that in a separate PR to keep this one cleaner.

Yeah, this is the reason I didn't do it in this PR. I just wanted this to be pure "moves"


Rename OK! and WARNING! to PASS! and FAIL! like in other CIS tools

Don't know if this is good idea or keep Prowler doing its own way... I think OK! and WARNING! is fine and easy to understand too. Not very important but the change is pretty straightforward, just changing 2 lines of code.

It might help with #54 to align the terminology with what CIS uses.
👍

Nice, thanks!

This seems like a really simple change and it would be super helpful. Can help out and make a PR for this? Thanks @toniblyx !

Hi @crashGoBoom, I think we could take advantage of all changes for v2.0 and make this change. You can look at the devel branch to see all changes I have done already. I can do this change now, I need to finish some other things and push v2 for beta in few hours or so.  BTW, https://github.com/kindlyops/prowlerCsv2md looks pretty interesting!

@toniblyx man you are on it! Just glancing at some of the changes in v2 and they look great! As always nice work on this.
Thanks @toniblyx !

This was done already


Call to a member function getUrl() on a non-object

Hi,
I've found a bug in the Magento Factfinder code which I believe is causing an exception further down the line:
E_ERROR: Call to a member function getUrl() on a non-object
in FACTFinder_Core_Block_Catalog_Product_List_Toolbar::initSortings called at app/code/community/FACTFinder/Core/Block/Catalog/Product/List/Toolbar.php (367)
in FACTFinder_Core_Block_Catalog_Product_List_Toolbar::initSortings called at app/code/community/FACTFinder/Core/Block/Catalog/Product/List/Toolbar.php (92)
in FACTFinder_Core_Block_Catalog_Product_List_Toolbar::getPagerUrl called at app/code/core/Mage/Catalog/Block/Product/List/Toolbar.php (483)

The issue looks to be caused by there being no null check on the $sorting variable before $sorting->getUrl() is called, which further down causes this issue. The code in question can be found here:
https://github.com/Flagbit/Magento-FACTFinder/blob/25e53762abc0efb8606a8ad7f5784ef10c051a48/src/app/code/community/FACTFinder/Core/Block/Catalog/Product/List/Toolbar.php#L369
Could you please resolve this so that I stop getting an exception thrown?
Thanks

I had a look into it. We can chage the code. But I try to find out how to reproduce. Because I want to fix the root cause. Did you change it locally in your code to get it to work?

Hi @mbraeuner ,
I've not been able to reproduce it consistently, but it's still been occurring. Regardless, it's still a bug.

I can't reproduce it on my setup. So I can not fix the root cause in depth. I fixed only the negative effect of throwing errors for you. Please have try and let me know, if it works for you branch issue-227

@mbraeuner , looks like this would solve it, so I'll close the issue. In my code I just checked for whether or not the object was there before using, but your code is more thorough.


[WIP] Dependency: Revery -> 0.29.0

This picks up some cool changes for latest Revery in 0.29.0, like:

PPX for component model (https://github.com/revery-ui/revery/pull/576)
Improved time API (https://github.com/revery-ui/revery/pull/632)

And maybe this as well: https://github.com/revery-ui/revery/pull/631

Thanks @Et7f3 ! 🎉


data.md, evm.md: change matching case splits to 'requires'

Is there anything blocking this to be merged?


Enhancement:  "your signup link" user experience optimization

Describe the Current Behavior/Feature:
Currently the "your signup link" inside of profile

requires manual copy and pasting of link to be shared
has double MUI paper backgrounds
remove image uses default window prompt.

https://github.com/codeforpdx/PASS/assets/114430929/4592e0e5-9f5e-4409-9ded-86adc0452768
Proposed Behavior/Feature:

add copy and paste api

remove inner background
add a modal similar to delete contact with 'cancel' and 'delete' options

Rationale:
This should improve the user experience of accessing and sharing signup links for adding clients.

This should not be a modal. It should be an independent page clients could use to create their first Pod after the case manager gave them the link via a phone message or via email.

Hi @Jared-Krajewski, I would like to work on this issue if possible. Could you assign it to me?

@JonnyMc94, Howzit. This is already assigned to someone.

@TheChunkyPumpkinhead I saw you unassigned yourself from the PR. Does this mean you wish to do so for this issue as well?


fix(snap delete): add missing unlock for error case

Signed-off-by: Utkarsh Mani Tripathi<EMAIL_ADDRESS>

should we cherry-pick this @utkarshmani1997 ?

No, new tag was created when this was merged.


Remove virtual from set_parent_for_children (issue #295)

set_parent_for_children is called only in the constructors. The
virtual keyword unnecessary and generates only confusion.
Check issue #295 for more info

@cattabiani : can you rename on master and resolve conflict?

I was bit concerned by "non-private non-virtual function being called through base class pointer pointing to derived class". But @tristan0x made that private and this is good.


can"t sync username

I can't sync my username to server 2. Please look into a  "sync username" block.
cheers, mindstormpro3
P.S. I also have a scratch account

Server 2 does not have normal CloudLink things like syncing usernames; it's for CL Suite (which has its own authentication system and is still work-in-progress I think).

yeah, although the server IS done. but just the home server

On Thu, Apr 27, 2023 at 9:59 AM ShowierData9978 @.***>
wrote:

yeah, although the server IS done. but just the home server
—
Reply to this email directly, view it on GitHub
https://github.com/MikeDev101/cloudlink/issues/110#issuecomment-1526035992,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A7PB7RBBXK7PXXBFH75RHOTXDKQVTANCNFSM6AAAAAAXODSU4M
.
You are receiving this because you authored the thread.Message ID:
@.***>

Thanks
Is there any servers that aren’t down and support username?

@tnix100's server, should be online

Cool, thanks!
I will send my online game when done
On Thu, Apr 27, 2023 at 10:07 AM ShowierData9978 @.***>
wrote:

@tnix100 https://github.com/tnix100's server, should be online
—
Reply to this email directly, view it on GitHub
https://github.com/MikeDev101/cloudlink/issues/110#issuecomment-1526047142,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A7PB7RFPY7UTSK2WIT5VIT3XDKRWRANCNFSM6AAAAAAXODSU4M
.
You are receiving this because you authored the thread.Message ID:
@.***>

it's not working. Will the local server be hosted from my computer or is it
hosted form a main server?
cheers, mindstormpro3
On Thu, Apr 27, 2023 at 10:07 AM ShowierData9978 @.***>
wrote:

@tnix100 https://github.com/tnix100's server, should be online
—
Reply to this email directly, view it on GitHub
https://github.com/MikeDev101/cloudlink/issues/110#issuecomment-1526047142,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A7PB7RFPY7UTSK2WIT5VIT3XDKRWRANCNFSM6AAAAAAXODSU4M
.
You are receiving this because you authored the thread.Message ID:
@.***>

As mentioned, server 2 is not a standard server. Closing since stale.


Bundler should handle builtin modules properly

Sorry, but what exactly do you mean by built in modules?

Sorry, but what exactly do you mean by built in modules?

I meant builtin modules of node.js

I meant builtin modules of node.js


US ID26: As a player, I want to be able to see the question category in order to know what was chosen.

Note: This is part of the game mode ‘trust or bust’ and should be only active, if that was chosen in the game room.

[ ] While showing the question everybody can see the category as well through some text and color coding.

proposed additional tasks:
Backend:
communicate a chosen question category to every player


[ja] remove "O=system:masters" from "kube-apiserver-etcd-client".md

resolve https://github.com/kubernetes/website/pull/42942/files
/assign @atoato88
/assign @nasa9084

/lgtm

@nasa9084
Could you review this PR at your convenience?


fastapi-independent request class

Let's also make FastAPI parameter as a kwarg

Let's also make FastAPI parameter as a kwarg

I'm making FastAPI changes in a separate PR


Improve ActionSheet Dialog.

[Android] Fix: compose box & navBar are not focussed out when actionSheet pops.
Before

this PR

Cool fix!


Update Dockerfile for platformio user install

https://github.com/emosenkis/esp-rs/commit/d75eafa3c8ee0c4a12f53e9a2720bbe4db870e9a broke the Dockerfile since the pip user install path isn't added by default. This adds it back to $PATH.
I also updated the Dockerfile to copy the current repo's source rather than just whatever version happens to be on Github at the time.

Thanks!


Current documentation theme isn't mobile friendly

Looks like this on my iPhone:

I think this can be fixed by switching the theme. I'm a fan of Furo here, as used on https://shot-scraper.datasette.io (and others), but there are plenty of other good mobile friendly options.

Yeah, I only switched it away from Alabaster last night so I need to do some more digging into Sphinx themes.

:+1: for Furo, I've used it on all my recent projects. Good light/dark theme support, good support for mobile/small/large/ultrawide screens, extensive (optional) customization options for sidebar and ToC.

Thanks for the recommendation - theme changed!


Updating ubuntu section

Cosmetic changes before I forget them.

@germa89 I've reviewed and approved this PR.


Automate blockchain-explorer installation

Epirus Blockchain-Explorer will not be provided (and therefore removed) as license does not allow commercial use.

See here


Issue #2462 - updated Institutional EOB constraints and added unit test

updated adjudication-has-amount-type-slice constraint to:
{
    "key": "adjudication-has-amount-type-slice",
    "severity": "error",
    "human": "If Adjudication is present, it must have at least one adjudicationamounttype slice",
    "expression": "adjudication.exists() implies (adjudication.where(category.memberOf('http://hl7.org/fhir/us/carin-bb/ValueSet/C4BBAdjudication')).exists())",
    "source": "http://hl7.org/fhir/us/carin-bb/StructureDefinition/C4BB-ExplanationOfBenefit-Inpatient-Institutional"
}

Changed the constraint location from: ExplanationOfBenefit.item and ExplanationOfBenefit.item.adjudication to ExplanationOfBenefit and ExplanationOfBenefit.item respectively in the snapshot and differential for the inpatient and outpatient Institutional EOBs.
Signed-off-by: John T.E. Timm<EMAIL_ADDRESS>

LGTM, but we should also add a comment to https://jira.hl7.org/browse/FHIR-32337 with your summary of the issue and this proposed fix.

Done


accept a parameter on redblue example

I want it to be able to use a random image as well.

Merged :+1:


Question: Supported browsers?

Would you mind adding browser support to your readme?

Jeah, to be honest, I was to lazy. But I will do it :)

Hey, I added the browsers I have access to. Would be great if you'd add more, if you can.

Couldn't you identify the css version support necessary and the specific items that you use and check it against caniuse.com?

I thought about it, but it's not just the feature support (and caniuse only lists the support for animation, but if I remember correctly, that wasn't congruent with the support for steps) but also a visual thing. It took some testing to find the right values that worked across browsers and I don't know why some things didn't work in one but the others; so I would want someone to see it to believe that it works :D Anyways, I think in it might be just IE10 that's missing and I'll add it if I find one. The others are evergreens or little used. Or am I missing a browser or version?

I have access to Microsoft Edge and I can confirm that it works in Edge 25. (Not that anyone actually uses Edge... :trollface:)

Thanks, I added it.
@steveoh Is the current state ok for you? I would close the issue then.

I'd like to find the support for steps. I think that including the step support with the data from caniuse would be optimal. Then you'd have a matrix for deciding if this can work for your project and it's supported browsers.

Ok, would be cool if you'd open a new issue if you find the support for steps. Till then I will close this issue as the general support was added to the readme.


Jira no longer works

Gush configuration
==================

 // The core:configure command will help to configure Gush for usage.
 // Your authentication credentials are never stored in the local Git repository.

Adapter configuration
---------------------

 // Gush uses adapters for repository-management and issue-tracking.
 // Adapters displayed with a "*" are already configured.
 // You are recommended to not skip this step if you configure Gush for the first time.

 Choose adapter:
  [0]   Nothing (skip selection)
  [1]   GitHub (RepositoryManager, IssueTracker)
  [2]   GitHub Enterprise (RepositoryManager, IssueTracker)
  [3]   Bitbucket (RepositoryManager, IssueTracker)
  [4]   GitLab (RepositoryManager, IssueTracker)
  [5]   Jira (IssueTracker)
  [6]   Jira Enterprise (IssueTracker)
 > 5

PHP Fatal error:  Call to undefined method Gush\Adapter\JiraFactory::createAdapterConfigurator() in /Users/aaron/Projects/gush/src/Factory/LegacyIssueTrackerFactory.php on line 43

Fatal error: Call to undefined method Gush\Adapter\JiraFactory::createAdapterConfigurator() in /Users/aaron/Projects/gush/src/Factory/LegacyIssueTrackerFactory.php on line 43

will look into this since i am using now Jira enterprise too again :+1:

@aequasi now that we have centralized the adapters into the core repo, do you mind taking a look again
This method is the one missing for JiraFactory, i copied it from GithubFactory
public static function createAdapterConfigurator(HelperSet $helperSet)
    {
        $configurator = new GitHubConfigurator(
            $helperSet->get('question'),
            'GitHub',
            'https://api.github.com/',
            'https://github.com'
        );

        return $configurator;
    }

I think on our refactor to a default adapter configurator we forgot Jira, do you mind giving it a spin to this one since it should be fairly simple?

 aaron@Aarons-MacBook-Pro  ~/Projects/gush  ± master   gush core:configure

Gush configuration
==================

 // The core:configure command will help to configure Gush for usage.
 // Your authentication credentials are never stored in the local Git repository.

Adapter configuration
---------------------

 // Gush uses adapters for repository-management and issue-tracking.
 // Adapters displayed with a "*" are already configured.
 // You are recommended to not skip this step if you configure Gush for the first time.

 Choose adapter:
  [0]   Nothing (skip selection)
  [1] * GitHub (RepositoryManager, IssueTracker)
  [2]   GitHub Enterprise (RepositoryManager, IssueTracker)
  [3]   Bitbucket (RepositoryManager, IssueTracker)
  [4]   GitLab (RepositoryManager, IssueTracker)
  [5]   Jira (IssueTracker)
  [6]   Jira Enterprise (IssueTracker)
 > 5

PHP Fatal error:  Call to undefined method Gush\ThirdParty\Jira\JiraFactory::createAdapterConfigurator() in /Users/aaron/Projects/gush/src/Factory/LegacyIssueTrackerFactory.php on line 43

Fatal error: Call to undefined method Gush\ThirdParty\Jira\JiraFactory::createAdapterConfigurator() in /Users/aaron/Projects/gush/src/Factory/LegacyIssueTrackerFactory.php on line 43

exactly this is what i described above

The factories are still using the Legacy adapter factory, this should be changed as the Legacy adapter only existed for compatibility with the external packages.

The factories have been updated, please confirm this issue is still existing.

I think more and more i only need jira adapter to just i:take ticket since i have to pass keys like XT-34 or such. I am not interested in any other endpoint besides this.

True, but for that the Configurator needs to work ;)

after careful consideration i am thinking in removing jira support altogether.


RLP Encode Header - Off by one error

The RLP specification has one path for encoding 0-55 bytes and another path for encoding longer byte arrays.
This should be programmed as if length < 56 or if length <= 55. There are two places with length < 55

personally I prefer <56 but to stay consistent with other places I'd go for <=55


Be able to delete machines from the command line

As a user of vCloud Tools, I would like to be able to delete a vApp in a script or from the command line.
The interface would be passing the name of a VM to the command.
See https://github.com/gds-operations/vcloud-launcher/issues/65. Deleting requires first stopping.
https://www.pivotaltracker.com/story/show/69709408

Thank you for your interest in this project but we now consider it
both feature complete for our own usage and in maintenance mode.
While we will not be investing any more development time in this
project we may still include pull requests.


docs: Improve the mail package's README to handle the async send()

I included the missing bits to handle the asynchronous send() call
including both ES6 and ES8 approaches.
Fixes
Closes #870
Checklist

[x] I have made a material change to the repo (functionality, testing, spelling, grammar)
[x] I have read the [Contribution Guide] and my PR follows them.
[x] I updated my branch with the development branch.
[ ] I have added tests that prove my fix is effective or that my feature works
[x] I have added necessary documentation about the functionality in the appropriate .md file
[x] I have added inline documentation to the code I modified

Short description of what this PR does:

Adds the error handling for the send usage in the example in both ES6 (ES2015) and ES8 (ES2017).

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

@thinkingserious You're welcome :).


[inferSchema] Use NEO4J_ENCRYPTED and NEO4J_DATABASE env vars

Currently the inferSchema.js script (which is used by npm run inferschema:write) ignores the NEO4J_ENCRYPTED environment variable and just defaults to unencrypted:
https://github.com/grand-stack/grand-stack-starter/blob/master/scripts/inferSchema.js#L9
Take into account the encryption flag and the NEO4J_DATABASE env var when running this script
As a workaround --encrypted can be manually appended to the call to the grandstack cli if encrypted connection is needed, for example:
node_modules/.bin/grandstack graphql inferschema --neo4j-uri neo4j://07de346a.databases.neo4j.io --neo4j-user summit --neo4j-password summit --schema-file ./api/src/schema.graphql --encrypted

Believe this has been taken care of in latest master but will test to make sure it works:  https://github.com/grand-stack/grand-stack-starter/blob/e648d56864b5a82d395504aa799e08e665fbea86/scripts/inferSchema.js#L26


Search by Community isn't working.

Search by Community isn’t working.
Steps to Reproduce:
•	Select a node
•	Select Search by Community
Result: nothing happens; no search is initiated.
Expected Result: Display only those nodes in that community; list all the emails from that community in the Email View pane. ???

Closing as duplicate to Issue #68.


[Question]: Operator keeps on calling openai api

Checklist

[X] I've searched for similar issues and couldn't find anything matching
[X] I've included steps to reproduce the behavior

Affected Components

[ ] K8sGPT (CLI)
[X] K8sGPT Operator

K8sGPT Version
v0.0.26
Kubernetes Version
v1.29.8
Host OS and its Version
No response
Steps to reproduce
Host the operator after following steps written here
https://github.com/k8sgpt-ai/k8sgpt-operator
Expected behaviour
Call openai api only once for analysis and do not call it again and again while pushing duplicate logs in slack.
Actual behaviour
As per my understating, it seems that when multiple logs for the same error are being pushed in the pod, the OpenAI API is being called again to fetch the output. If this is correct, it may not be cost-effective.
I might be wrong. Can someone confirm
Additional Information
No response

The Operator does not have a client for OpenAI, that would come from the K8sGPT deployment.
Can you show us your K8sGPT custom resource? --no-cache mode will have this behaviour.

Are you seeing data in Results? What is the log for K8sGPT-sample pod?


A possible typo in JSTF Example 3.

In https://learn.microsoft.com/en-us/typography/opentype/spec/jstf

The 3rd column at the 6th row of the table "Example 3" notes "hrinkageDisableGPOS".
Comparing with the 17th row, I'm afraid that it could be a typo of "gposShrinkageDisableOffset".

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

ID: b691a267-906d-cafb-2f9e-4dadc8691427
Version Independent ID: 41170219-4e78-137d-9382-2d06039a4e8c
Content: JSTF — Justification Table (OpenType 1.9) - Typography
Content Source: typographydocs/opentype/spec/jstf.md
Service: typography
GitHub Login: @PeterCon
Microsoft Alias: alib

Thanks for reporting. See the OT 1.9.1 alpha for draft revisions addressing this issue.

This is fixed in OT 1.9.1. See the OT 1.9.1 beta for draft revisions addressing this issue. Closing.


[FLINK-22014][ha] Make sure that AbstractHaServices delete first the HA data before deleting blobs on closeAndCleanupAllData

The AbstractHaServices should always first delete the HA data (pointers to jobs and checkpoints) before deleting the referenced
objects such as blobs. That way we ensure that we don't leave the system in an invalid state if the process stops abruptly because
either all the data is still available or the pointers have been cleared and Flink only leaves some orphaned blobs. The case
where the pointers are still there but not the blobs will no longer be possible with this fix.

Thanks for the review @zentol. The failing test seems to be unrelated. Merging this PR now.


Remove no description and general scope

Hi, I have a couple of questions which from my POV are rather a nice to have for decreasing cognitive load when reading a changelog.

Every time I do a release with an empty description I see the sentence *No description* within the changelog. Why? Would it be possible to hide it as it adds zero value?
Is it possible to hide the scope general? Maybe we just don't want anything. Meaning when I commit something like git commit -m 'feat: add feature 1' I  would just like to see something:

FEATURES
    - add feature 1
    - add feature 2

Thank you so much

Hi, @mascardoso
Thank you for your questions. For the first question, I think this can be repalced with empty content. So if you mean like that:
## v1.1.1

*No desctription*

### Feature

to be replaces with
## v1.1.1

### Feature

I think we can have this in the next release when I'm free. And if you have time, you can make a pull request on this feature.
For the second question, I think it is hard to directly add thoses commit message. I'm afraid this might make the format wierd for the changelong. I have thought to hide those commits. But see #11 and #75. You can use other words or characters to replace general.
If you have better ideas on the second question, please discuss with me.

@mascardoso
One of the problem should be solved in the new release v1.2.1.


Added Python Program for Heap Sort

Added the Python program under Python/Data Structure and Algorithms/Heap_sort.py for Heap Sort. Attaching the screenshot of the output in VS Code.

good work 😊


Dark title bar

Description
Added dark title bar to default and darker theme and I also fixed link color in package control install dialog. Other changes are just removed trailing spaces.
I tried to find good alternative and even create custom theme but there is no such a nice theme similar to this and is easier to fix this.
Note: Appbar https://github.com/equinusocio/material-theme-appbar has also been removed so maybe re-add or remove it from the package control.
Motivation and Context
It looks better than standard white.
Screenshots (if appropriate):

Types of changes

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

Checklist:

[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.

Unfortunately, with the number of lines changed in this PR, it's unreviewable. Please check your editor config for automatic changes (such as trimming trailing whitespace or a JSON formatter) and re-submit or force push.

Note: Appbar equinusocio/material-theme-appbar has also been removed so maybe re-add or remove it from the package control.

You probably mean one of those? https://github.com/kizza/material-theme-appbar/network/members
I'm personally not interested in investing more time into this package (and related) than I need to because I have no interest in using it. Feel free to resubmit the appbar package to package control/redirect the current entry to a somewhat maintained fork.

You probably mean one of those?
Yes. I don't also use that package but maybe will be good to remove it from package control since repo no longer exists (or is private).

Closing this in favor of #2   I'll change that title bar color to dark in my local version.

We will go though the list of now unavailable packages in some sort of batchy way later. The package being missing is generally not critical to operations.


query-fronted: no-store header issue

Thanos, Prometheus and Golang version used: 0.17.1
What happened: Having one querier with no available stores (querier is alive but prometheus is down on this cluster), makes our main querier to stop caching requests in the query frontend - since there is a No store matched for this query warning.
What you expected to happen: Since this is not a real issue as specified in the docs NOTE: Having warning does not necessary means partial response (e.g no store matched query warning).
I'm expecting to not pass no-store header on such cases, since we got our response.
How to reproduce it (as minimally and precisely as possible): Add an empty querier to your querier that has a frontend in front of him and see how your cortex_cache_hits metrics goes flat.

Hello @shaikatz, I really didn't understand what is exactly missing here? How does your stack look like? Do you use Querier with store gw/receiver/sidecar? Without any store (something that implements store api) we don't have any data? Do you want to cache empty responses?
cc @yeya24

Our architecture has one root querier, which is connected to queriers in multiple clusters.
For example we have clusters named: clusters1, cluster2, cluster3 each of them as the corresponding external labels cluster="clusterName".
Now let's assume cluster3 stores are down.
We sent a simple query, for example: `up{cluster="cluster1"}
The answer from that query will have the no-store header even though we got the correct response.
Hope that it makes the issue a bit more clear.
Let me know if further info or explanation are needed.

@shaikatz Is this case you mentioned valid? Is this reproducible? Ideally, if_ you already set cluster="cluster1" in your query, then by external label matching, the querier will only query clusters1 store. So no matter cluster3 is down or not, it doesn't affect the result.

I'm not sure why, but I cannot reproduce that.
You can close that for now.

Thanks for the confirmation @shaikatz, I am closing this for now then.


Reduce credential ID size

It is still possible to overflow the credential ID when creating a non-discoverable credential, especially by providing a long RP ID.  (The allowed maximum is 256, and the maximum credential ID length is 255.)  Stripping some of the metadata was not enough to solve the problem.  To improve this, I suggest the following changes:

Introduce a separate type for the stripped credential.  Currently, we just set the stripped fields to None.  This provides the potential for subtle bugs if we assume some field to be set that has been stripped.  Therefore I suggest to introduce a new StrippedCredential type that only includes the relevant fields.

Flatten the serialized data structure.  Currently, we have three nested levels: Credential → CredentialData → PublicKeyCredentialRpEntity.  This adds unnecessary overhead.

Remove unused fields.  Having a separate type for stripped credentials makes it possible to identify the fields that are never used.  Currently, these are:

rp_id: String<256>
creation_time: u32
use_counter: Option<bool>
hmac_secret: Option<bool>

rp_id is obviously the most problematic one.  It remains to be investigated if we really don’t need it or if this is a problem with the current implementation.

If an allowList is present and is non-empty, locate all denoted credentials present on this authenticator and bound to the specified rpId.

https://fidoalliance.org/specs/fido-v2.0-ps-20190130/fido-client-to-authenticator-protocol-v2.0-ps-20190130.html#authenticatorGetAssertion
I think this means that we should make sure that the credential provided in allowList actually belongs to the RP.  We already use the RP ID hash as associated data when encrypting the credential data.  So I think it is safe to just remove the full RP ID from the stripped credential data.

Good spot! With AEAD we do not need RPID encoded, and the rest is not needed.

I have been tracking an issue with Nitrokey 3 and it seems this is the root cause.

My key works on gitlab.com
But if does not on my local instance.

After trial and error:

I realized that passing a shorter user handle to navigator.credentials.create made it work.
By experimenting with user handle lengths, I realized it would fail when the returned rawId size would exceed 255 bytes.
By comparing the results when running on my local instance and on gitlab.com I realized the that the returned rawId is longer on my local instance.

⇒ It appears it is built from the hostname. Which is problematic as hostnames can be very long.
I checked with my Yubikey, and the rawId size is fixed. It looks like it hashes the fields instead.

@spectras This issue should be fixed for the Nitrokey 3 with v1.5.0-test.20231030.

Thanks! I will get people to upgrade their firmware!


Is this compatible with Varnish 4.0 +?

Hi Justquick,
I'm interested in utilising this module alongside an Odoo (runbot.odoo.com - username: admin password: admin) module to allow purging and other varnish functions from the Odoo interface.
I'd just like to know if it would be compatible with Varnish 4.0 + as this is the version i'll be going with for my module. I would be very interested in any input you might have on this.

I'm interested too by the answer, as I'm using Varnish 4.

Success tested on version Varnish 4.0.3 with Varnish CLI 1.0

@rodo ,
Thanks for letting me know.

Did you find any performance gain, @rodo ?


knife bootstrap --bootstrap-version command line option not working

Description
When running knife bootstrap with --bootstrap-version option, the option if set in ~/.chef/config.rb takes precedence. According to https://docs.chef.io/config_rb.html#optional-settings the command line option should take precedence. The code at https://github.com/chef/chef/blob/master/lib/chef/knife/core/bootstrap_context.rb#L189-L205 in the knife_config method only appears to check @chef_config for the value used in version_to_install.
Chef Version
$ chef --version
Chef Development Kit Version: 3.11.3
chef-client version: 14.13.11
delivery version: master (9d07501a3b347cc687c902319d23dc32dd5fa621)
berks version: 7.0.8
kitchen version: 1.25.0
inspec version: 3.9.3

Platform Version
Ubuntu 16.04 (both workstation and client)
Replication Case

add knife[:bootstrap_version] = '13.9.4' to ~/.chef/config.rb
run knife bootstrap --bootstrap-version '14.12.9' $HOST
expected result: -----> Installing Chef Omnibus (-v 14.12.9)
actual result: -----> Installing Chef Omnibus (-v 13.9.4)

Client Output
$ knife bootstrap -VV --bootstrap-version '14.12.9' --no-host-key-verify --ssh-user vagrant -N test <IP_ADDRESS>
INFO: Using configuration from /home/myusername/.chef/config.rb
Creating new client for test
Creating new node for test
Connecting to <IP_ADDRESS>
DEBUG: Adding <IP_ADDRESS>
<EMAIL_ADDRESS>password:
<IP_ADDRESS> -----> Installing Chef Omnibus (-v 13.9.4)
<IP_ADDRESS> downloading https://omnitruck.chef.io/chef/install.sh
<IP_ADDRESS>   to file /tmp/install.sh.1645/install.sh

The root cause of this is NOT in how the options are being merged but in how they are being used.
Chef::Config[:knife][:bootstrap_version] is deliberately the value from the config file and only the config file.
Within a knife subcommand the correct way to access the merged config is through the config[:bootstrap_version] accessor which accesses the mixlib-cli config hash, merged with the Chef::Config[:knife] sub-hash.  The knife_config[:bootstrap_version] hash is also wrong since that is just the Chef::Config[:knife][:bootstrap_version] sub-hash.
The problem is that within the "core/bootstrap_context" object the config value which is injected there is not the mixlib-cli config hash but is the Chef::Config hash -- for maximum confusion.  So that is a completely different object from the Knife subcommands.

Or that appears to not quite be correct since @config in the bootstrap context appears to be the @config from the knife bootstrap subcommand, but I'm still not getting the CLI overrides, but the knife_config accessor was always wrong...  Looks like after fixing one bug something else is still using Chef::Config[:knife][:bootstrap_version] somewhere...

Ah I just had a thinko, the fix looks as simple as not using the chef-config hash

Thank you both!


Environment variable AR provided to configure cannot override the result of  AC_PATH_PROG(AR,ar)

Branch 2.2
I have already mentioned this issue happening here
There is a file config-script/cups-common.m4 that has a line setting AR variable by searching for 'ar' program by means of PATH
...
AC_PATH_PROG(AR,ar)
...

When I try to configure and execute 'make' the following way
AR=llvm-ar CC=clang CXX=clang++ ./configure --with-optim="-g"

, I get the following line as a linker is linking a static library
/usr/bin/ar crvs libcups.a adminutil.o array.o auth.o backchannel.o backend.o debug.o dest.o dest-job.o dest-localization.o dest-options.o dir.o encode.o file.o getdevices.o getifaddrs.o getputfile.o globals.o hash.o http.o http-addr.o http-addrlist.o http-support.o ipp.o ipp-support.o langprintf.o language.o md5.o md5passwd.o notify.o options.o ppd.o ppd-attr.o ppd-cache.o ppd-conflicts.o ppd-custom.o ppd-emit.o ppd-localize.o ppd-mark.o ppd-page.o ppd-util.o pwg-media.o request.o sidechannel.o snmp.o snprintf.o string.o tempfile.o thread.o tls.o transcode.o usersys.o util.o

/usr/bin/ar imposed as a result of AC_PATH_PROG having ignored AR provided from environment
I like somehow to override AR variable, now it seems impossible
In the PR I have offered the solution, please take it in consideration or suggest another way
Thank you

You need to specify the full path in the environment variable, otherwise the configure script will ignore it.
(This is someplace in the autoconf documentation, but not in the help shown by the generated configure script for some reason - probably worth a bug against the autoconf project...)


Getting error "config file does not exist."

I am getting this error regarding the config file not existing, even though its in the repo at the following path:
.github/auto-label.json

Wondering if you might have any ideas on this, would love to be able to use your workflow.

My setup looks like the following:
.github/auto-label.json:
{
  "rules": {
    "Instant": ["instant/*"],
    "Investing": ["investing/*"],
    "MissionControl": ["support/*"],
    "migrations": ["*/migrations/*"]
  }
}

.github/workflows/labelling-workflow.yml:
name: Labelling workflow

on:
  pull_request:
    paths:
    - '**.py'
    - '**.json'

jobs:
  add_label:
    runs-on: ubuntu-latest
    steps:
      - name: auto-label
        uses: banyan/auto-label@master
        env:
          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
      # This step prints an output (time) from the previous step's action.
      - name: Echo auto-label's time
        run: echo 'The time was ${{ steps.auto-label.outputs.time }}.'

same for me(

sorry for late. Since GitHub Actions version 2, many things have been broken.
I've just fixed all things. please try this if you still use: https://github.com/banyan/auto-label/releases/tag/1.0


Consider switching from serde_cyclonedx to cyclonedx-sbom

Since we started this project, the official rust crate cyclonex-sbom for CycloneDX has gotten into a state where it is suitable for our purpose.
But there are some things to consider:
Advantages:

Support for XML and JSON, see also #39
Code de-duplication, we currently need a lot of duplicate code to generate both 1.4 and 1.5, using this crate would resolve that issue.
We can drop our dependency on some other crates as well

Disadvantages:

The official crate has (as of yet) only support for 1.3 and 1.4, with the current newest standard being 1.5, however support for 1.5 is on the horizon with https://github.com/CycloneDX/cyclonedx-rust-cargo/issues/646.

Hi @ErinvanderVeen, thanks for considering cyclonedx-bom
Yes, CycloneDX 1.5 should be done soon-ish but latest at the end of April.
The work on this is funded by the German government and we have a deadline ;-)
Please reach out to us via GitHub, or the CycloneDX slack or me personally if you have any problems or feedback.


Add DASHBOARD_COLUMNS to config, with defaults to match current COSRI

First step in implementing #143.

Looks good!
Let me know if the linter ever becomes too much. Looks like we can configure it auto-fix, if that's of interest

i really like having the linters enabled.  good to know auto-fix is an option, not ready for that, yet :)


modbus example in docs is not working...

I just followed the example of modbus but following errors has occurred.
When I sent following 'GET' method using postman,
http://localhost:48082/api/v1/device/19728d2c-f422-40e6-bcd2-c0067c1cf9e0/command/e2745b58-d4a1-4cf1-ac9a-93cabe3e98f8
the response body is like this.
Handler - execReadCmd: error for Device: DENT's Motor cmd: Configuration, unable to create TCP connection info, protocol config 'Address' not exist /api/v1/device/19728d2c-f422-40e6-bcd2-c0067c1cf9e0/Configuration
and I checked the logs of device-modbus container, it said
http://localhost:48082/api/v1/device/19728d2c-f422-40e6-bcd2-c0067c1cf9e0/command/e2745b58-d4a1-4cf1-ac9a-93cabe3e98f8

I'm not fully understanding of edgeX foundry and just following the examples in docs...
so could you tell me what i'm missing.

I'm a college student in Korea who lacks experience but is interested in this project.

What version of EdgeX are you using? 1.0 "Edinburgh" was just released last week, but the docs might still be for the previous "Delhi" release.

@horoyoii Please take this to the #help channel in edgexfoundry.slack.com


Update Ruby version support in README.md

Please apply the following patch to generated README files. This change is being applied manually to existing generated files in GoogleCloudPlatform/google-cloud-ruby#2130.
-This library is supported on Ruby 2.0+.
-
-However, Ruby 2.3 or later is strongly recommended, as earlier releases have
-reached or are nearing end-of-life. After June 1, 2018, Google will provide
-official support only for Ruby versions that are considered current and
-supported by Ruby Core (that is, Ruby versions that are either in normal
-maintenance or in security maintenance).
-See https://www.ruby-lang.org/en/downloads/branches/ for further details.
+This library is supported on Ruby 2.3+.
+
+Google provides official support for Ruby versions that are actively supported
+by Ruby Core—that is, Ruby versions that are either in normal maintenance or
+in security maintenance, and not end of life. Currently, this means Ruby 2.3
+and later. Older versions of Ruby _may_ still work, but are unsupported and not
+recommended. See https://www.ruby-lang.org/en/downloads/branches/ for details
+about the Ruby support schedule.

/sub

Do we also want to add the section about enabling logging?

Yep. I didn't see it in the new Speech README, but I may have removed the sections while resolving some conflicts.


Centralize HttpRequest creation in BO

Move %Net.HttpRequest creation in BO methods into a separate method GetRequest and call it from operation requests. Why:

Very often APIs require HTTPS or some custom header. If request is produced from GetRequest it’s easy to modify it for all requests.
Request object reuse. To achieve better throughput it’s usually preferable to use the same %Net.HttpRequest object. GetRequest can easily implement cache logic if required by user

Also move Do pHttpRequestIn.SetHeader("accept", pRequest.accept) into GetRequest method.

Yes, you're right!
It's done.


Show statistics across admissions

Create a new page with statistics across admissions, like unique applicants/applications per admission.

We should implement an aggregate system that generates statistics when admissions are anonymized.
If we aggregate all the data we can safely delete all underlying models in the admission.


docs: reorganize siteConfig entry (#436)

set the setConfig fields to h4 instead of li, will create some spacing and make the docs less clustered.
Motivation
(Write your motivation here.)
Have you read the Contributing Guidelines on pull requests?
yes
Test Plan
(Write your test plan here. If you changed any code, please provide us with clear instructions on how you verified your changes work. Bonus points for screenshots and videos!)
Related PRs
(If this PR adds or changes functionality, please take some time to update the docs at https://github.com/facebook/docusaurus, and link to your PR here.)

@JoelMarcey i've fixed the commit

❤


Links are not working

links "  onLinkTab: (url, title) =>" does not return any link a links are not clickable.

By the way. If you are using tex_markdown package then It will link tap will not work. Use gpt_markdown  package. And also If you are using flutter version 3.13 then Link will not be clickable because there was some issue with flutter in 3.13. If you are using 3.13 please update your flutter version.

i am using gpt_markdown but only the links which have these  square brackets before link e.g "Amazon" are clickable but which only link but no square brackets e.g  "https://www.amazon.com" are not clickable

i am using gpt_markdown but only the links which have these  square brackets before are clickable but which only link but no
Amazon

  data = data.replaceAllMapped(
      RegExp(
          r"(https:\/\/www\.|http:\/\/www\.|https:\/\/|http:\/\/)?[a-zA-Z0-9]{2,}(\.[a-zA-Z0-9]{2,})(\.[a-zA-Z0-9]{2,})?"),
      (match) => "[${match[0]}](${match[0]})");

Use this function to achieve that method before passing data to TexMarkdown. It will convert https://domain.com to [https://domain.com](https://domain.com).
Currently I am not supporting normal links for a reason. But You can use this method to achieve this feature.
I hove this will be helpful for you.

thanks. From where did you learn all of this?

I have learned by youtube videos and using the official documentation page. Mostly self learning.


bind different programs to multiple desktop

Description of the new feature / enhancement
I really need multiple desktop taskbars to bind different programs
Scenario when this would be used?
so that I can plan to assign and launch different software under different desktops
Supporting information
Each time I reboot the computer, I need to switch the desktop and then open the program, rather than opening the program directly to the corresponding desktop, which makes me feel that windows's multiple desktops are weak in many cases

/feedbackhub


issues with Preloader

Hello,
I try to use develop branch (openfl 4.5.2, lime 3.5.2)
At building project for html5 target I get following issue:
When loading swf library, preloader not working properly, the ProgressEvent comes only twice, at the beginning and end of the loading.
In project.xml:

This is a normal thing that happens in flash when you test locally because you have access to all the bits already. Are you testing locally?

No, I tested it on a remote server, too.

Previously, SWF libraries actually used the standard asset library in order for preloading to work properly. With the revision of our asset library system (to enable SWF libraries to preload separately) it aggravated another problem -- the preload events were dispatched for only one standard asset library, and the others were loaded after.
I've just hacked support in for the OpenFL and Lime development branches to allow SWF preloading as a part of the standard preload process. Could you take a look and see if it is working better for you?
Thank you :smile:

I have just tested, but preloader unfortunately is still not working properly.

Other than sparse progress events, is it working properly?

The current situation is as follows:
I created a custom preloader, progress events come normally, but their values are incorrect
Progress events handler:
`private function this_onProgress (e:ProgressEvent):Void
{
	trace ("bytesLoaded: " + e.bytesLoaded, " bytesTotal: " + e.bytesTotal);
}`

Output:
Preloader.hx:87: bytesLoaded: 0, bytesTotal: 6
Preloader.hx:87: bytesLoaded: 1, bytesTotal: 6
Preloader.hx:87: bytesLoaded: 2, bytesTotal: 6
Preloader.hx:87: bytesLoaded: 3, bytesTotal: 6
Preloader.hx:87: bytesLoaded: 4, bytesTotal: 6
Preloader.hx:87: bytesLoaded: 5, bytesTotal: 6
Preloader.hx:87: bytesLoaded: 6, bytesTotal: 6
Preloader.hx:87: bytesLoaded: 7, bytesTotal: 6
Preloader.hx:87: bytesLoaded: 8, bytesTotal: 6
Preloader.hx:87: bytesLoaded: 9, bytesTotal: 6
Preloader.hx:87: bytesLoaded: 10, bytesTotal: 6
Preloader.hx:87: bytesLoaded: 11, bytesTotal: 6
Preloader.hx:87: bytesLoaded: 183, bytesTotal: 6
.......
Preloader.hx:87: bytesLoaded: 196, bytesTotal: 6
Preloader.hx:87: bytesLoaded: 197, bytesTotal: 6
Preloader.hx:87: bytesLoaded: 198, bytesTotal: 6
Preloader.hx:87: bytesLoaded: 198, bytesTotal: 6
Preloader.hx:96: load complete

Obtained values seem strange ... the total data amount is about 13 mb

Thanks, we need to work more on the new preload system, ultimately I think we need to embed the file sizes in the asset manifest, so we know in advance how large the full asset list will be before we download

I'm tracking this improvement here, thanks :smile:
https://trello.com/c/Yyk00pai/44-improve-the-quality-of-progress-events-for-core-assetlibrary-types


Enhance documentation

Removes duplicate contributor
Updates the tutorial logger to be equivalent to the one from the example code because it is confusing to see two different logger implementations.

thanks @BenjaminW3


Model for asynchronous mentoring videos

Description
As listed on the doc, create a model for a mentor's uploaded videos. It should look like:

Links to videos for asynchronous mentoring:

ListField of Object with link (StringField) + tag (StringField)

Include constructor (__init__) and __repr__ functions
Files
backend/api/models/[namehere].py
Tasks

[ ] Create the model!

Lmk if I should add anything else for the repr or if I am missing something important. 😃

@chyku Should for the tags how should go about storing them or just a Stringfield? It might be that the same tags will be used a lot in multiple videos too.


https://github.com/dmlc/xgboost/tree/master/windows  is not available. So how to install xgboost in windows?

I followed below link for installing xgboost in windows. But I can't find windows folder for installing xgboost.
https://www.kaggle.com/c/otto-group-product-classification-challenge/forums/t/13043/run-xgboost-from-windows-and-python
And also I installed through python_packages using python setup.py install. I am getting below error.
D:\xgboost-master\python-package\xgboost\libxgboost.dll
D:\xgboost-master\python-package\xgboost../../lib/libxgboost.dll
D:\xgboost-master\python-package\xgboost./lib/libxgboost.dll
D:\xgboost-master\python-package\xgboost../../windows/x64/Release/libxgboost.dll
D:\xgboost-master\python-package\xgboost./windows/x64/Release/libxgboost.dll

see https://xgboost.readthedocs.org/en/latest/build.html#building-on-windows

I followed instructions and get the following error when building with make on Windows 64-bit.  FYI, I aliased mingw32-make -> make to get things workings.  Any ideas?
A subdirectory or file -p already exists.
Error occurred while processing: -p.
The syntax of the command is incorrect.
Makefile:97: recipe for target 'build/objective/regression_obj.o' failed
make: *** [build/objective/regression_obj.o] Error 1
make: *** Waiting for unfinished jobs....
A subdirectory or file build already exists.
Error occurred while processing: build.
Makefile:97: recipe for target 'build/learner.o' failed
make: *** [build/learner.o] Error 1
Makefile:97: recipe for target 'build/logging.o' failed
make: *** [build/logging.o] Error 1
The syntax of the command is incorrect.
Makefile:97: recipe for target 'build/objective/rank_obj.o' failed
make: *** [build/objective/rank_obj.o] Error 1

@dagley11 You are running into a very classical problem on Windows.
In Windows CMD, mkdir doesn't have the -p option, which is quite annoying.
The simplest solution is using another shell on Windows.
If you have install git from https://git-for-windows.github.io or  https://git-scm.com, you already have a standard bash shell.
Running make in that shell shouldn't have such errors.

@thirdwing Yes, you were right!  Works now in Git Bash shell. Thank you very much!!

@thirdwing  I am facing the following error:
A subdirectory or file -p already exists. Error occurred while processing: -p. A subdirectory or file build already exists. Error occurred while processing: build. The syntax of the command is incorrect. g++ -m64 -std=c++0x -Wall -O3 -msse2  -Wno-unknown-pragmas -funroll-loops -Iinclude -DDMLC_ENABLE_STD_THREAD=0  -Idmlc-core/include -Irabit/include -fopenmp -MM -MT build/learner.o src/learner.cc >build/learner.d The syntax of the command is incorrect. mingw32-make: *** [build/logging.o] Error 1 mingw32-make: *** Waiting for unfinished jobs.... mingw32-make: *** [build/common/common.o] Error 1 mingw32-make: *** [build/c_api/c_api_error.o] Error 1 g++ -m64 -c -std=c++0x -Wall -O3 -msse2  -Wno-unknown-pragmas -funroll-loops -Iinclude -DDMLC_ENABLE_STD_THREAD=0  -Idmlc-core/include -Irabit/include -fopenmp -c src/learner.cc -o build/learner.o
As guided by you i had downloaded git bash from the link you mentioned and also deleted previous folder of xgboost before giving the mingw32-make -j4 command.

@OnkarDeshpande From your error log, I am afraid you are still using Windows CMD.

@thirdwing I'm attaching a screenshot of the error. Please have a look at it.  I'm using git bash, yet getting the same error.

@OnkarDeshpande You are still facing the mkdir problem. Let me try to find a good way to handle it.

@thirdwing  Yes, I'm still facing the issue. Thanks in advance for helping! :)


Graphql's CreateWorkflow API integration with frontend

Currently, GraphQL API is working to accept create workflow request which will be sent to the subscriber.
Need to integrate with createChaosWorkFlow API with frontend

Schema
input ChaosWorkFlowInput {
  workflow_manifest: String!
  cronSyntax: String!
  workflow_name: String!
  workflow_description: String!
  weightages: [WeightagesInput!]!
  isCustomWorkflow: Boolean!
  project_id: ID!
  cluster_id: ID!
}

type ChaosWorkFlowResponse {
  workflow_id: String!
  cronSyntax: String!
  workflow_name: String!
  workflow_description: String!
  isCustomWorkflow: Boolean!
}

type Mutation {
  createChaosWorkFlow(input: ChaosWorkFlowInput!): ChaosWorkFlowResponse!
}

Thank you @rajdas98. Let me raise PR for our current work will start that once it is done.

closing by this PR- https://github.com/litmuschaos/litmus/pull/1834


phpMyAdmin Configuration of pmadb… not OK

https://stackoverflow.com/questions/21033059/pma-database-not-ok-in-phpmyadmin-upgrade

Wrong repo.


Getting error - The directory name is invalid.

Hi,
Created a file system where i am able to copy/paste to from the virtual drive. But when i try to open a word document directly from the virtual drive getting windows alert that says -
The directory name is invalid

Looks like the driver is not getting recognized here - I am on windows 10 64 bit system. Working with Dokan dotnet.
Please share how to go about it - if similar issue is observed by anyone.
Thanks,

@imraushan Hi!
Seems like your implementation is return the file path as a directory instead of a file. Be sure to correctly set the variable isdirectory.

Hi @Liryna - Thanks a lot!
Yes - at one point in the FileInformation I was not adding Attribute for marking it as directory or file. After adding it works.


Submit to Chrome store

I am working on this and will make any employee able to publish a zip to the store.
https://developer.chrome.com/docs/webstore/publish/

Looks like I need some promotional images for the submission.

https://developer.chrome.com/docs/webstore/images/#promo

Yeah, and the format is important too, in that the images should be the right dimensions and not be grainy at all. It might be worth going through the process of uploading to an unlisted slot so that you know what will be involved, and also so that you can smoke test something very close to the real thing.

After we have made a decision regarding how the extension popup window will look, we can add screenshots.

Created a figma design for the elements. Still need screenshots.
https://www.figma.com/file/QiCVtnPijwmyVh8VdR0CGh/open-sauced%2Fbrowser-extensions?node-id=0%3A1&t=slXhfcyNsN5nKJST-1

This description is ambitious toward what we want to accomplish with this extension.
@diivi @Anush008 - I love to talk through this and start looking.

Chrome Store Description:

The OpenSauced Chrome extension seamlessly integrates GitHub with the OpenSauced platform. With this extension, you can easily view and discover open-source projects looking for contributions directly from GitHub, making collaboration and contribution easier than ever.
OpenSauced offers a range of features that streamline the process of finding and contributing to open-source projects based on the contributions you choose to highlight. One of the critical features of the extension is its ability to automatically match GitHub profiles with the corresponding OpenSauced user profiles, making it easy to track and manage contributions across multiple platforms.
With the OpenSauced extension, you can easily view the repositories and projects a user has contributed to on GitHub and quickly jump to the corresponding pages on the OpenSauced platform. You can also view a user's profile information, including their programming interests, and easily request collaboration on open-source projects.
The extension also makes it easy to discover new open-source projects on GitHub relevant to your skills.
Overall, the OpenSauced GitHub Profile Connector is a must-have extension for anyone who is passionate about open-source software and wants to contribute to the community. Whether you're an experienced maintainer or just getting started, this extension makes it easy to discover, contribute to, and track open-source projects on GitHub and connect with other developers who share your interests and skills. So why wait? Download the OpenSauced extension today and start contributing to the open-source community!

The extension is out on the Chrome store!!🎉🎉
Available here.


ghc-8.4.2 fails to build due to haddock-api-<IP_ADDRESS> requiring ghc-8.4.1

stack --stack-yaml=stack.yaml install                                      \
		&& cp '/Users/jky/.local/bin/hie' '/Users/jky/.local/bin/hie-8.4.2'    \
		&& cp '/Users/jky/.local/bin/hie-8.4.2' '/Users/jky/.local/bin/hie-8.4'

Error: While constructing the build plan, the following exceptions were encountered:

In the dependencies for haddock-api-<IP_ADDRESS>:
    ghc-8.4.2 from stack configuration does not match ==8.4.1  (latest matching version is 8.4.1)
needed due to haskell-ide-engine-<IP_ADDRESS> -> haddock-api-<IP_ADDRESS>

Some different approaches to resolving this:

  * Set 'allow-newer: true' to ignore all version constraints and build anyway.

  * Consider trying 'stack solver', which uses the cabal-install solver to attempt to find some working build
    configuration. This can be convenient when dealing with many complicated constraint errors, but results may be
    unpredictable.

  * Recommended action: try adding the following to your extra-deps
    in /Users/jky/wrk/arbor-networks/haskell-ide-engine/stack.yaml:

- ghc-8.4.1

Plan construction failed.
make: *** [hie-8.4.2] Error 1

I have the same problem. Haddock-api requires ==8.4.1, that was changed in its latest revision two days ago.

We need haddock-api-2.20, which is not on hackage as of yet.
/cc @hvr @alexbiehl

The revision was made because haddock-api-1.19 is in fact incompatible with GHC 8.4.2 for the on-disk formats. We should be able to release haddock-api-2.20 RSN!


Can we get a 3.3.6?

Been a few months since v3.3.5. Here's the shiplist from June 16th: #16644

Your interest has been duly noted. Can't promise anything though.


cilium-node-init does not restart pods on EKS with CNI chaining

Bug report
The cilium-node-init DaemonSet uses an introspection endpoint of the AWS CNI plugin which no longer exists to list the running Pods.  The endpoint localhost:61679/v1/pods has been removed in AWS CNI version 1.7.0 and therefore listing the running Pods no longer works.
Additional Information
Container IDs could instead be obtained from the /v1/enis endpoint, which still exists. However, I am not 100% sure whether that endpoint returns the correct set of Pods (all Pods that needs restart).
curl "localhost:61679/v1/enis" | jq '.ENIs[].IPv4Addresses[].IPAMKey.containerID' -r | grep -v '^$'
Looking at the shell script code in https://github.com/cilium/cilium/blob/master/install/kubernetes/cilium/templates/cilium-nodeinit-daemonset.yaml, it also looks like it is not yet well prepared for all combinations of cloud providers and container runtimes (docker, containerd, ...). E.g. on EKS one could have both runtimes in the same cluster depending on the AMI in use for specific node groups.
General Information

Cilium version (run 1.9.5)
AWS CNI version 1.7.10 (but all 1.7.* version are affected starting with 1.7.0-rc1)
Kernel version (run Linux ip-10-176-4-232.eu-central-1.compute.internal 5.4.105-48.177.amzn2.x86_64 #1 SMP Tue Mar 16 04:56:19 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux)
Orchestration system version in use (e.g. Kubernetes v1.19.6-eks-49a6c0)

Potentially related issues:

https://github.com/cilium/cilium/issues/15366
https://github.com/cilium/cilium/issues/15330

@rustrial Is this still an issue?


Make actions create artifacts, so builds are automatically made and able to be distributed

Recently found out what github artifacts are thanks to the feather repo, basically just uploading files created in an actions workflow.
We should make our workflows distribute the generated DLLs to actions so people don't have to download files that I compile for everyone's sake and so people (Who don't want to read code hashes) don't need to worry that I could be distributing malware or something.

Done https://github.com/Vurv78/Autorun-rs/actions/runs/1049926028


Want to ask if sending OMNI USDT is supported?

Want to ask if sending OMNI USDT is supported?

@wartw no. If you get it to work, great, but we don't accept PR's or issues about other cryptocurrencies.

ok, thank


updates-dotnet-path-configuration-description

Changelog

Clarifies that the dotnet path must refer to a diretory (and not to the executable)
Portuguese translation of the same setting (PT-BR)

Other
Related to #6068, based on the clarification provided by @dibarbet.
Related to #6161.

@dotnet-policy-service agree

No problem, and thanks for the support with the original issue!


Create CODE_OF_CONDUCT.md

Codecov Report

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

@@           Coverage Diff           @@
##           master       #1   +/-   ##
=======================================
  Coverage   46.51%   46.51%
=======================================
  Files           3        3
  Lines          43       43
  Branches        8        8
=======================================
  Hits           20       20
  Misses         22       22
  Partials        1        1

Continue to review full report at Codecov.

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

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Passing A Zip URL Not Working

I tried to pass a zip url like this:
https://supermedium.com/beatsaver-viewer/?zip=https://cors-anywhere.herokuapp.com/https://www.dropbox.com/s/a9i42wtyqtfe8xg/2d3e1bc0-215e-47b3-9d8a-4a505794f4ac.zip?dl=1
But it stuck at the downloading step. The console outputs an error below:
Uncaught TypeError: Cannot read property 'buffer' of undefined
    at ZipLoader.extractAsText (build.js:266)
    at ZipLoader.extractAsJSON (build.js:266)
    at build.js:259
    at Array.forEach (<anonymous>)
    at build.js:259
    at Array.forEach (<anonymous>)
    at i.processFiles (build.js:259)
    at ZipLoader.<anonymous> (build.js:259)
    at ZipLoader.dispatch (build.js:266)
    at XMLHttpRequest.xhr.onload (build.js:266)

This is because the structure of my zip file is not correct.


add apps config

Pull Request Test Coverage Report for Build 206

0 of 30 (0.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-1.8%) to 59.05%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

apps.go
0
30
0.0%

Totals

Change from base Build 204:
-1.8%

Covered Lines:
597

Relevant Lines:
1011

💛  - Coveralls


Allow phone auth via CodeUI

Current Code UI allows for only email based auth
<input
  data-component="input"
  autofocus
  type="email"
  name="email"
  required
  placeholder={copy.email_placeholder}
/>

With slight modification it can used to allow phone based auth as well.
<input
  data-component="input"
  autofocus
  type={props.inputOptions?.inputType ?? "email"}
  name={props.inputOptions?.inputName ?? "email"}
  pattern={props.inputOptions?.inputPattern ?? undefined}
  title={props.inputOptions?.inputTitle ?? undefined}
  required
  placeholder={copy.email_placeholder}
/>

Have added two additional attributes pattern and title in-order to allow input validation based on custom requirements.
inputType can be tel, text or email
Also added inputmode and autocomplete to autoread OTP/code in mobile browsers

thanks! i ended up simplifying the PR to a single option - we try to keep the options minimal until we have an overwhelming request for customization

thanks! i ended up simplifying the PR to a single option - we try to keep the options minimal until we have an overwhelming request for customization

Just for context, I added that because I used one Codeadapter for both email and phone. And in auth handler fn I used to check the input value, if it was email I used to send Code via email, else Mobile provider.


Setup caching in GitHub actions

I think it's possible that the test_data/ repos is getting cloned every time the tests are run. (Thus impacting our bandwidth usage.) I think caching is the answer as described in this article.
https://www.develer.com/en/blog/avoiding-git-lfs-bandwidth-waste-with-github-and-circleci/

Closed by #128


changes made in navbar

Made some changes in navbar
Closes #1
There was no proper spacing in navbar

Signin get out of navbar when on lower width device:

changes:
before:

after:

@Rakshit-gen can u add tags also
And also mention if u have some other tasks

@Sourabh782 done

@Sourabh782  Pick any unassigned issue preferably logo designing if you are comfortable.

@Sourabh782 Thanks for the contribution, kindly star the repo if you like the project!

@Rakshit-gen


Login page responsiveness

Login pages is left to be responsive

Hey @Vanshika-73 I can do this. Can you assign this to me
Thank you

Go Ahead.

Hey @Vanshika-73 is there a problem with my PR?

Are you working on it? @Rahul-jain-01

Yes, I'm working on it. But I am somewhat stuck :(

where? you can take help from register.jsx file. @Pinaka-Pani-18 has worked very well on its responsiveness. coding is almost same for both register and login page. Sync your forked repository, you will get the updated code.

Thank you, I will make the changes.


Fix all weird out parameters int *ret in the codebase

Search before asking

[X] I had searched in the issues and found no similar issues.

Motivation
Refer to https://github.com/apache/incubator-kvrocks/pull/1444#discussion_r1199580764.

Maybe not related to this PR, but there are two points:

ret is not a descriptive word
int type is not fit here: the return variable members.size() is typed size_t, not int

Solution
No response
Are you willing to submit a PR?

[ ] I'm willing to submit a PR!

@PragmaTwice I exec the statement *output = redis::Integer(ret); to return the result to the client in cmd_set.cc.
https://github.com/apache/incubator-kvrocks/blob/fb0e3d42031a6e0691c7a736711d0a01d72bdb1a/src/server/redis_reply.h#L38-L41
Add the Integer func checks whether compatibility with int is required(Of course, size_t is right). I see src/types/*.h and many funcs with this situation use int.

Integer can accept all integral types, which includes size_t

@infdahai You mean size_t by typing usize?

This issue involves many files. Should I submit one PR with all of the changes?

Yes, it's fine since the change is intuitive.

I have read through the relevant code and have decided to make modifications according to the following rules:
The files match types/redis_*.h

For the function similar to Set#IsMember or String#SetNX that ret means the flag, I will change it to int *flag

For the function similar to Set#Remove that ret means the number of removed elements, I will change it to uint64_t *removed_cnt

For the function similar to List#Size that ret means the size of list, I will change it to uint64_t *size

For the function similar to List#Size that ret means the size of list, I will change it to uint64_t *size

For the function similar to String#IncrBy that ret means the new value, I will change it to int64_t *new_value

In cases where ret may be negative, I will set the type to int*.

The files match commands/cmd_*.cc
I will change the type of int ret to the correct type, but I will not change the variable name because it represents the return value of the Redis command.
The test files
I will change the type of int ret to the correct type, and I will decide whether to modify the variable name based on the situation.
All files
I will remove all unnecessary static_cast statements.
I will use uint64_t(Metadata#size type) as the type for length everywhere, instead of the size_t.
Do you think these are acceptable? I would appreciate hearing your thoughts and opinions.

I think you can replace it with bool if the return value is just a boolean variable.

I think you can replace it with bool if the return variable is just a boolean value.

Thank you for your suggestion. I will do that.

@git-hulk it seems the issue should be closed?

@infdahai Yes, thanks for your reminder.


Add simple ui filtering logic

The example contains simple UI logic to check/uncheck/disable filters and also show/hide items.
Other examples always use similar boilerplate code. Add simple UI logic so no boilerplate code is needed anymore. Note that some options and callbacks can be useful to use before, during and after filtering.

Closed by https://github.com/robertpainsi/filtering.js/commit/c649ea1a23609e5b2fb97f6f69bff442cace07a9