Chef Zero Support

Hello.
I am unable to use unencrypted data bags with Chef Zero.  Specifically, I am using chefdk, KCI, vagrant with zero as the provisioner.  I suspect its user error, but after some code review not certain.  If I create a databag via chef-shell and then run the cookbook tomcat::users it tries to decrypt and in turns fails due to missing the encryption key configuration.
Per tomcat-0.17.3/libraries/chef_tomcat_cookbook.rb
def find_users
        users = if Chef::Config[:solo]
                  data_bag = Chef::DataBag.load(USERS_DATA_BAG)
                  data_bag.keys.map do |name|
            Chef::DataBagItem.load(USERS_DATA_BAG, name)
          end
                else
                  begin
                    items = Chef::Search::Query.new.search(USERS_DATA_BAG)[0]
                  rescue Net::HTTPServerException => e
                    raise TomcatUserDataBagNotFound if e.message.match(/404/)
                    raise e
                  end
                  decrypt_items(items)
                end

        users.each { |user| validate_user_item(user) }
        users
      end

Should this include zero support similar to solo?  I only have a basic understanding here, so I could be wrong.
I have been unable to find any tutorial using unencrypted databags with Tomcat (at least trying to add users.)
Regards.
Ron

Hi @ronaldpetty,
Why don't you simply encrypt your data bag? This might help!

+1

First off I'm very sorry that it's taken so long to get back to you on this issue.  For a significant period of time there was no active Chef Software Inc. maintainer for this repository and things languished.  We're now actively maintaining this cookbook again, and as part of that I've rewritten the cookbook with new custom resources for installing Tomcat from Apache Org sources.  This will ship in an upcoming 2.0 release and then we'll remove the existing recipes and providers in a future 3.X release.  With those new providers I believe the need for this change goes away so I'm going to close this out.  I encourage you to give the new providers a try and let me know if we can tackle more Tomcat management without the attributes bloat the current code suffers from.  They're on the master branch and once the documentation and testing improves a bit they'll be released as a 2.0 cookbook.


Error processing Twitter archive: unsupported Unicode escape sequence

When attempting to upload my Twitter archive, I get the following error about 80% of the way through uploading tweets:
Error processing Twitter archive: Error inserting mentioned_users_2963358137: unsupported Unicode escape sequence

I've tried the upload on Safari 18.1, Chrome 130.0.6723.92, and Firefox 132.0, with all tested browsers failing with the same error message.

any chance you could send us your tweets.js file so we can debug?

any chance you could send us your tweets.js file for easy debugging?

Sure, I've attached it to this reply.
tweets.js.gz
tweets-part1.js.gz


Linter-gcc not working / not even displaying messages in console

Checklist

[x] Check that you have the GNU C Compiler installed by typing gcc --version into a terminal.
[x] If you don't have the linter package installed, either update to the latest version of linter-gcc or install linter directly with apm install linter.
[x] Ensure that you are using a valid grammar - either C/C++/C++14
[x] Ensure that you have read all the relevant information in the wiki, particularly the Common Issues page.

Issue (including debugging messages):
Hey,
I have installed the latest versions of linter, linter-ui-default and linter-gcc. I am using:
Atom    : 1.34.0
Electron: 3.1.1
Chrome  : 66.0.3359.181
Node    : 10.2.0

I have also installed the latest gcc-compiler extension to enable me to compile c++ programs within atom. When I compile my C++ programs, which I have purposefully ridden with a bunch of errors, the linter shows no errors. The linter simply does nothing.
I have checked show debugging messages, and there are no messages regarding linter or linter-gcc in the developer console.
I do however repeatedly get this warning in the console:

fs.js:958 (electron) The 'lstatSyncNoException' function has been deprecated and marked for removal.

whenever I perform pretty much any action in atom, however, this may not be related at all, as this warning is displayed regardless of whether or not linter and linter-gcc is installed.
I am on Arch Linux if that bares any relevence.
I would greatly appreciate it if you could help me out with this, as I find this to be a great package.
Thanks

As it seems linter-gcc is no longer maintained @tvincent056 has published his fork which is compatible with the updated base linter as linter-gcc2. See comment on #185

Duplicate of #199


Detach InjectViewModelListener before forwarding.

A long time ago, this commit was introduced to the Forward plugin: https://github.com/zendframework/zendframework/commit/0b8fc7adb6b138feec99eaff8e7a2ef4fe6aa041. This prevents a problem where forwarding causes multiple InjectViewModelListeners to get attached, which in turn causes the view template to get rendered multiple times. This is a waste of cycles, and in some cases causes malfunctions (in situations where rendering a template has side effects).
It appears that somewhere along the line, this functionality was broken and the test intended to cover it was removed. This PR restores the broken functionality and adds a test to help catch future regressions.
Real life use case: my application has a "flash messages" view helper which checks the session for pending messages, displays them, and then clears the queue from the session. This bug prevents flash messages from displaying whenever the application performs a forward action, since the first render clears the queue, then the second render overwrites the first with no flash message data. For me, this is a critical bug.
Please let me know if there are any questions or if I can help to further refine this code. Thank you!

Note that the failing Travis tests here do not appear to be directly related to my changes.

Duplicate of #247

Thanks for sorting it out! :-)


Upgrade Tomcat version to 9.0.74.wso2v1

$Subject
Related Issue:

https://github.com/wso2-enterprise/security-internals/issues/130

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/5571505607

PR builder [7.x.x] started
Link: https://github.com/wso2/product-is/actions/runs/5571507118

PR builder [7.x.x] completed
Link: https://github.com/wso2/product-is/actions/runs/5571507118
Status: failure

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/5571505607
Status: success


[$500]  Chat - After leaving thread and going back user redirected to chat instead of LHN

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

Version Number: 1.4.35-7
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Issue found when executing PR https://github.com/Expensify/App/pull/35563
Action Performed:

Open the app and log in
Navigate to any chat
Send a message
Start a thread and then leave it
Tap the back icon

Expected Result:
User is redirected to the LHN
Actual Result:
User returns to the chat
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/a5d3032a-2efc-417d-b10a-466c4165c723
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01f91d520c5e7912a6
Upwork Job ID:<PHONE_NUMBER>794864128
Last Price Increase: 2024-02-02

Raised it in #vip-vsb.

this issue should be handled in this issue


feat: add chevrons (up, down, left, right)

Purpose
New "chevrons-up", "chevrons-down", "chevrons-left", "chevrons-right" icons.
Approach
Add "chevrons-up", "chevrons-down", "chevrons-left", "chevrons-right" custom icons.
Testing
On preview.
Risks
N/A

Shouldn't it be only chevrons-left? It's the only one available in Figma

We have decided to go with all 4 sides.


Crash when using Remonsterate

Rolling seed 1 of 1.
3|normal|bcdegijklmnopqrstwyz makeover partyparty alasdraco capslockoff johnnydmad notawaiter mimetime remonsterate|1648164726
You are using Beyond Chaos CE Randomizer version "3".

The randomization is very thorough, so it may take some time.
Please be patient and wait for "randomization successful" to appear.
Cutscenes are currently skipped up to Kefka @ Narshe
multiprocessing.pool.RemoteTraceback:
"""
Traceback (most recent call last):
File "multiprocessing\pool.py", line 121, in worker
File "randomizer.py", line 5283, in randomize
NameError: name 'bcmultiprocessing' is not defined
"""
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "beyondchaos.py", line 1270, in generateSeed
File "multiprocessing\pool.py", line 657, in get
NameError: name 'bcmultiprocessing' is not defined

Fixed in newest version. Thanks for the report!


Create CC mixins with deprecation warnings.

Some CC mixins with the appropriate deprecation warnings would help significantly with migration to Hyrax 2.0.

@tpendragon Do you have particular mixins in mind?

It feels a bit like the ship has sailed on this one.
I'd support any backwards compatibility PRs on 1.x to ease migration from CC/Sufia, but lets keep any tickets concrete and narrowly scoped.


Symfony property access should be set to version 4.4

The problem is that composer will not install "symfony/property-access": "^5.0" alongside Symfony 4.4.
In other words, anyone using Symfony 4.4 will be unable to use this lib.
symfony/property-access": "^4.4"
or
symfony/property-access": "^5.0|^4.4"
should be enough to fix this.

This is the composer message
Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - Conclusion: remove symfony/symfony v4.4.5
    - Conclusion: don't install symfony/symfony v4.4.5
    - Conclusion: don't install symfony/symfony v4.4.4
    - Conclusion: don't install symfony/symfony v4.4.3
    - Installation request for yswery/dns 1.4 -> satisfiable by yswery/dns[v1.4].
    - Conclusion: don't install symfony/symfony v4.4.2
    - Conclusion: don't install symfony/symfony v4.4.1
    - yswery/dns v1.4 requires symfony/property-access ^5.0 -> satisfiable by symfony/property-access[v5.0.0, v5.0.1, v5.0.2, v5.0.3, v5.0.4, v5.0.5].
    - don't install symfony/property-access v5.0.0|don't install symfony/symfony v4.4.0
    - don't install symfony/property-access v5.0.1|don't install symfony/symfony v4.4.0
    - don't install symfony/property-access v5.0.2|don't install symfony/symfony v4.4.0
    - don't install symfony/property-access v5.0.3|don't install symfony/symfony v4.4.0
    - don't install symfony/property-access v5.0.4|don't install symfony/symfony v4.4.0
    - don't install symfony/property-access v5.0.5|don't install symfony/symfony v4.4.0
    - Installation request for symfony/symfony ^4.4 -> satisfiable by symfony/symfony[v4.4.0, v4.4.1, v4.4.2, v4.4.3, v4.4.4, v4.4.5].

Originally posted by @ampaze in https://github.com/yswery/PHP-DNS-SERVER/pull/99#issuecomment-597284599

Thanks for your help @ampaze
I have just pushed a new bugfix release.

Thank you! Working fine now.
This can be closed.


Do we support an extra interface?

Hello everyone, today I had a presentation about Jakarta Data, and I got a good question that I had yet to learn.
What is the expected behavior in this situation?
As a user, given I have multiple commons queries on the repository, I decided to put it on an interface.
public interface BaseQueriesFromUser<T> {

    List<T> findByName(String name);

    List<T> findByYear(String name);
}

So, I'll put this one in multiple repositories:
@Repository
public interface Library extends CrudRepository<Book, String>, BaseQueriesFromUser<Book> {
}

@Repository
public interface Airport extends CrudRepository<AirPlane, String>, BaseQueriesFromUser<AirPlane> {
}

What is the expected behavior from the spec?
I did not find anything like that before.
I checked, and it already works on JNoSQL, and I did not test it.

The spec doesn't currently say anything about this scenario one way or the other, but it seems like a reasonable thing to be able to do, so I'd be fine with it being added if the vendors are okay with it.

It is a neat approach to include customizations. Spring Data uses a concept of fragments to be able to compose a repository from potentially multiple implementations. This turns effectively the default implementation (which implements the actual CrudRepository) into a fragment as well.

I ran into this same question when working on the TCK framework:
https://github.com/jakartaee/data/blob/4475edf9c246b6b4d643cb63b6e5871a47af2c2a/tck/src/main/java/ee/jakarta/tck/data/framework/read/only/ReadOnlyRepository.java#L23-L27
Where I wanted to limit a repository to be read-only.
Would be a great addition!

@otaviojava
For JNoSQL this works only for repository interfaces, that use a generic type corresponding to the (like BaseQuery<Person>) otherwise it is ignored.
Why not allow any "custom" repository and the user can decide to use generics or not.
(This might be JNoSQL specific, but could be a topic for the Data spec)

Hey @georgleber could you open this issue on the JNoSQL repository?
So we can go deeper into it.


Implements basic placeholder obstacle avoidance for enemies

Enriches Penguin lives through the magic of dance
Adds a function in MovingObject to raycast from current position
without attempting movement.
Adds movement randomization when enemies are directly diagonal to the
player when seeking so that they don't always travel in the vertical
direction.
Adds very basic obstacle avoidance by causing penguins to move in a
totally random direction when they would otherwise collide with an
obstacle or fellow penguin.

Seems legit, there's no actual conflict here, git just got super confused.

Closed it dafb16a


BUG: restart rov: uninitialized variables somewhere

reproduce: on falcon, test restart/rov

comes from the way objectStats works; may need to force it by adding an integrator to get the correct oldMotions


Building microservices on .NET Core links are dead

The links in this section: https://github.com/mjebrahimi/Awesome-Microservices-DotNet?tab=readme-ov-file#building-microservices-on-net-core
they just link to https://www.altkomsoftware.com/pl/

a, there is already a PR for this: https://github.com/mjebrahimi/Awesome-Microservices-DotNet/pull/20


V 1.0 Release Date?

Hey Matteo,
The company I work for is using vue-analytics and loving it, but we're wanting to migrate over to gtag for some of the newer features.
Do you have a status update on this library?  We'd love to hear about any plans for the release of this project, or about any of the roadblocks that are still holding it back.
Thanks

Thanks for the feedback, unfortunately I'm struggling finding the time, but I'm working on it ( finding the time ). I'll have a look at the API again and see what's what.

Thanks for your reply, I really appreciate it.  We'll likely continue using vue-analytics since there's no gtag library that's ready yet, but definitely update us if you're thinking you'll be able to complete it or not 🙏

For sure! Luckily they didn't kill the analytics script yet so I have some time

as you can see I started doing something 👍
https://github.com/MatteoGabriele/vue-gtag/tree/develop/src

@HMilbradt no release 1 maybe, but I have just release version 0.1.0 which is basically a safe version 1
didn't make any tests yet, nor finished documentation or add all features vue-analytics might have.
let me know if you tried

this is what I need to do to go to v1 https://github.com/MatteoGabriele/vue-gtag/milestone/1
I hope I'll have time soon

This is great, thank you so much for starting this 🙏.  I'll try to take a look at it this weekend to give some feedback on it.

Perfect! Looking forward to it. I started doing some unit testing in the meantime

well version 1 is out in the wild!
let me know :)

Thank you!  We will be checking this out soon and will definitely have feedback 🙏


Option to specify directory on bundle gem

From @zeiv on April 14, 2016 19:13
I use GitHub a lot.  Usually I will create my repo online so GitHub can generate my README and LICENSE and generally make life easier, then clone it onto my machine and start developing.  When I'm creating a new gem however, Bundler also makes it own directory.    So, here is what I propose:

A flag such as bundle gem my-gem-name --dir my-existing-gem-repo to specify that Bundler should generate the gem in that directory.
And/Or a flag to simply have the gem generated in the current directory: bundle gem my-gem --current-dir

Thoughts?
Copied from original issue: bundler/bundler-features#117

From @segiddins on April 14, 2016 20:37
I think this is straying out of bundle gem territory and more towards standalone tools like ore

From @zeiv on April 16, 2016 17:16
I agree that Bundler should not focus on gem bootstrapping, however I don't think the option to specify a directory is a huge step outside the minimal functionality that Bundler does provide.  Especially when you consider the other gem boilerplate functionality that Bundler does provide, such as --bin and --ext.  Adding this option would only involve a small amount of code, as well.

I don't think we'll  be adding more options to bundle gem, sorry.

I think this would be very useful, found myself in the need for this today since I usually create the remote first, clone and then start coding.
Would you reconsider this if a PR is provided? @segiddins @colby-swandale

@00dav00 i think you can get the same effect with cd ..; bundle gem dirname; cd dirname, does that not work for you for some reason?


Find number of times recognized audio repeat in the source

Hi,
Can you guide me to make changes to find the number of times the recorded audio repeats in the source(any audio from database)?
Thanks

related - https://github.com/ztjordan/ShazamClone/issues/1


Explicit first class private name alternative

In the July 2020 TC39 meeting, @hax mentioned a goal of reification for private names. Previously, @erights and @waldemarhorwat discussed the option of these first class values for private name brand checks.
I wrote a gist to explore some concrete details about how object-based explicit references to private names (as @erights was encouraging) could work to permit exception-free brand checks to the presence of private names in classes.
With this idea, the syntax #x in obj would be replaced by (private.name #x).has(obj)--a little bit longer, yes, but it accomplishes the same function. private.name creates a fresh object with methods to access the private name.
This approach would be compatible with the membrane-transparency goals that @erights has advocated for. In particular, you could pass this private name as well as the object over a membrane, and the membrane system can unwrap them both to do the check appropriately. This would be impossible if the private name were passed around as a primitive directly--if ergonomic brand checks are reliable and do not use method calls in its semantics, then the membrane system has no chance to unwrap the object.
Overall, I don't know how to weigh the ergonomics goal that @ljharb has articulated with the reification goal that @hax has articulated. To me, neither one seems to be an absolute requirement, but rather tradeoffs about value judgements that we as a committee can make. I hope this gist helps us explore more of the design space.
(I wonder if, for the ergonomics goal, we could be looking more to @erights and @waldemarhorwat 's guards proposal, which used the :: syntax to do a check about whether a JS value matches a guard. This could be a longer-term solution compatible with a near-term capability of first-class private names for "has" checks. Guards are also based on first-class values.)

In that case, I would expect (private.name #x) in obj to work, instead of the very strange .has approach (that private fields have WeakMap semantics doesn't mean normal users will ever really think about that).

I'm not really sure how to make that work with membranes... We could rename the has method to in, but I don't really think it's feasible to make the in operator with this approach. Why is the in operator a requirement for this feature?

It’s not a requirement, it’s what i (and i think users) will expect. Private fields are property-like, and in is how property presence is tested.

I can understand that some developers might prefer this particular proposal, but we're trying to iterate based on feedback from committee. Do you think it won't be possible/sufficient to learn/use the method-based syntax I'm proposing?

I think it will be unintuitive and hard to discover. The use of the private operator, when it's not used for private fields otherwise, is confusing; and its precedence forcing use of inline parens is also awkward.
I agree that your suggestion achieves the goal of "detect if an object has a field, without try/catch", but I don't think it does so in a satisfactory way, and I don't think it actually addresses the mental model concerns from @hax's objection.

Thanks for explaining. I guess I just don't see the syntax I am suggesting as that bad. (I think the parens would actually be optional, as private.name PrivateName would be a PrimaryExpression.)
I would be interested in hearing more from @hax on his thoughts on both the syntax and the mental model concern. Maybe I misunderstood his past comments, so clarification would be helpful. I thought @hax was specifically suggesting something like this.

This would be impossible if the private name were passed around as a primitive directly--if ergonomic brand checks are reliable and do not use method calls in its semantics, then the membrane system has no chance to unwrap the object.

@littledan I'm not sure I fully understand this part, do u mean current ergonomic brand check syntax could break membrane transparency?? And in previous discussion of private fields , I remember someone have proved that even private symbol could still keep membrane transparency.

we could be looking more to @erights and @waldemarhorwat 's guards proposal

This is a very interesting idea. Do u have a more concrete example that how it should look like?

that private fields have WeakMap semantics doesn't mean normal users will ever really think about that

@ljharb I think this is one key disagreement we have. My point is users will not think about it because currently there is no refication. Reification would build a concept model, in only make sense if reification is symbol.

Private fields are property-like, and in is how property presence is tested.

I also discussed it in last meeting. Private fields are property-like only in syntax aspect, not semantic. This is not very obvious if users only write o.#x (very close to get own property syntax/semantic), but when use #x in o users will be lead to think about the reflection-related syntax/semantic of property which private fields never have.

and I don't think it actually addresses the mental model concerns from @hax's objection.

To be clear, my objection have three points. 1. syntax of this proposal should depend on the semantic of reification; 2. KEY in o syntax implies o[KEY] syntax; 3. too fast to advance a proposal from stage 0 to 3 in 4 months.
So (private.name #x).has(o) at least addressed my second point. And it also match the first point --- private.name #x actually belongs to reification proposal.
Note, whether reification should use map-like or symbol semantic is a separate issue, and whether the syntax of (private.name #x).has(o) is ergonomic enough is also a separate issue.

I thought @hax was specifically suggesting something like this.

To be clear my objection is not based on any specific alternative solution, what I mean is we can only make choice until we figure out the path of reification first --- this proposal should depend on reification proposal.

The third point is just not a constraint we have; things proceed as fast or as slow as makes sense for the proposal. We've had one proposal move from nothing to stage 3 in one meeting.

I'm not convinced about point 2: if KEY is an identifier, then yes I'd expect o[KEY], but the fact that #x is not an identifier (and that o[#x] is a syntax error) disconnects it from that model.  On the other hand, trying to reify private names introduces significantly more complex mental models that I honestly have a much more difficult time reasoning about ("does the private name have the object, or the object have the private name?", etc).  If reification has value in itself then it should be its own separate proposal, but let's not block this proposal on that.

@littledan I'm not sure I fully understand this part, do u mean current ergonomic brand check syntax could break membrane transparency?? And in previous discussion of private fields , I remember someone have proved that even private symbol could still keep membrane transparency.

@hax To clarify, private fields, like WeakMaps and internal slots, are not Proxy-transparent. They can work through membrane systems that unwrap things at the appropriate point. I presented on this at TC39 with this slide deck. Various kinds of private symbols were proposed at TC39, and they were all found to not meet certain constraints that committee participants raised.

Various kinds of private symbols were proposed at TC39, and they were all found to not meet certain constraints that committee participants raised.

I remember private symbol could satisfy membrane transparency with some Proxy API small additions (but not very sure, @jridgewell may know that).
Anyway, if we finally will have reification in any form , i hope the syntax could match the semantic, for example, if it's weakmap, it shouldn't use in.

Private symbols can work with membranes, but it requires non-trivial changes to Proxy and the membrane implementations. It's much simpler to use a map-based API for them.

@hax @jridgewell OK, these sound like arguments for the get/set/has API which is proposed in the gist that I linked.

There's a conceptual disconnect between a weakmap-inspired API (get/set/has) and the idea of a private property, which I assume would confuse more developers than the disconnect between #x in obj/x in obj vs obj.#x/obj[x].
If we want get/set/has, then I'd suggest finding a way to name the API to make it clear that this is a map representation of the property.

This proposal reached stage 3 at today's TC39.

This proposal reached stage 3 at today's TC39.


[SMALLFIX] Use static imports for standard test utilities in AuthenticatedUserRuleTest.java

Use static imports for standard test utilities in AuthenticatedUserRuleTest.java

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/17805/
Test PASSed.


StrongMode: Start using strong mode in Traceur itself

This enables strong mode for compiling traceur itself and adds
'use strong' to src/syntax/**

@rossberg


replace json for games with SwiftData/CoreData

https://pkl-lang.org

yeah no
CoreData or SwiftData are definitely the way to go here.
SwiftData would be much easier to implement but CoreData would (unfortunately) work too

lmao change of plans
now doing SwiftyJSON. code migration on my fork is done, just need to fix the platform implementation.

done in a4530cf6ec3fde1f8fe4987a2d15ff6312e9b57f


Feedback 20201008

Fix product list default query (LITE-15585)

Codecov Report

Merging #5 into master will decrease coverage by 0.26%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master       #5      +/-   ##
==========================================
- Coverage   45.80%   45.53%   -0.27%
==========================================
  Files          13       26      +13
  Lines         679     1366     +687
==========================================
+ Hits          311      622     +311
- Misses        368      744     +376

Impacted Files
Coverage Δ

cnctcli/commands/product.py
36.47% <0.00%> (-1.81%)
:arrow_down:

...s/build/cloudblue/connect-cli/cnctcli/constants.py
100.00% <0.00%> (ø)

...ld/cloudblue/connect-cli/cnctcli/commands/utils.py
20.00% <0.00%> (ø)

...e/connect-cli/cnctcli/actions/products/__init__.py
100.00% <0.00%> (ø)

...travis/build/cloudblue/connect-cli/cnctcli/ccli.py
95.83% <0.00%> (ø)

.../cloudblue/connect-cli/cnctcli/commands/product.py
36.47% <0.00%> (ø)

.../cloudblue/connect-cli/cnctcli/actions/accounts.py
100.00% <0.00%> (ø)

.../cloudblue/connect-cli/cnctcli/commands/account.py
100.00% <0.00%> (ø)

...dblue/connect-cli/cnctcli/actions/products/sync.py
12.62% <0.00%> (ø)

.../connect-cli/cnctcli/actions/products/constants.py
100.00% <0.00%> (ø)

... and 4 more

Continue to review full report at Codecov.

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


Fix: Use directory instead of repo

This PR

[x] fixes the wording of a message

Thank you, @feelepxyz!


Massive unit testing overhaul: Parameterization, parallelization, and objects, objects, objects!

This PR makes, incrementally over a series of commits, sweeping changes to the way the project unit tests are structured, executed, and analyzed.
TL;DR: Unit tests are more granular, and execute in parallel for faster testing.
For the way-too-long detailed version, read on.
Parameterization
Instead of the output-comparison tests in test/my_tests and test/charts being executed as one long test case that chugs through all of the files in  a given directory, the parameterize PyPi module is now used to construct a set of test cases, one per dotfile, and run each one individually.
The overall effect is, the test case count increases from 33 to 233, with no change in runtime, but much more granular output from the unittest run. Instead of the entire set of regression tests looking like this:
test_graphviz_regression_tests (__main__.TestGraphAPI.test_graphviz_regression_tests)
 ...#######################################################3

...as it grinds through 197 files in that directory, now those files' tests will be output individually, like so:
  test_regression_001_rootlabel (test.test_pydot.TestGraphvizRegressions.test_regression_001_rootlabel)
 ... ok
  test_regression_002_labelroot_fbc (test.test_pydot.TestGraphvizRegressions.test_regression_002_labelroot_fbc)
 ... ok
  test_regression_003_ldbxtried (test.test_pydot.TestGraphvizRegressions.test_regression_003_ldbxtried)
 ... ok
  test_regression_004_labelclust_ntc (test.test_pydot.TestGraphvizRegressions.test_regression_004_labelclust_ntc)
 ... ok
  test_regression_005_pm2way (test.test_pydot.TestGraphvizRegressions.test_regression_005_pm2way)
 ... ok
  test_regression_006_b80a (test.test_pydot.TestGraphvizRegressions.test_regression_006_b80a)
 ... ok
  test_regression_007_colors (test.test_pydot.TestGraphvizRegressions.test_regression_007_colors)
 ... ok
  test_regression_008_labelclust_fbl (test.test_pydot.TestGraphvizRegressions.test_regression_008_labelclust_fbl)
 ... ok
  test_regression_009_labelroot_ndc (test.test_pydot.TestGraphvizRegressions.test_regression_009_labelroot_ndc)
 ... ok

(The tests are auto-numbered, the rest of the identifier string is based on the input dotfile name.)
Parallelization
The previous change was about making the tests more granular without impacting performance. Now we get to take advantage of that increased granularity for some massive performance gains.
The unittest-parallel tool is a test runner compatible with the standard-library unittest module, but as its name suggests it can execute tests in parallel. Now that we have 233 quick tests to break up, instead of just 33 (one of which accounted for over 50% of the total runtime all by itself), we can divide those 233 cases up and execute them in parallel, making use of all cores on the test system.
In my testing on a local four-core Intel Core i5 (that's over 10 years old), total test execution runtime dropped by over 2/3, from ~95s to just over 30s total.
On the GitHub runners the effects of this are negligible; with a hot cache it might shave 10 seconds off runtime for each runner, and those times were already hovering around the 60-second mark. But if more, or more complex, tests are added, the impact could be greater.
Class hierarchy restructuring
As part of making these changes, I reworked how the test cases are structured, breaking up the single monolithic unittest.TestCase subclass into several.

The parameterized tests are each their own class, one per directory.
The tests that run (or don't) on the files in test/from-past-to-future are also their own class, and the logic to execute or skip that test based on the presence of the directory now uses the @unittest.skipUnless() decorator to evaluate that precondition.
All of the tests that compare rendered results from pydot to the output of the dot command are now subclasses of RenderedTestCase (which defines the _render_and_compare_dot_file() method they use)
And everything is a subclass of PydotTestCase, which defines the common _reset_graphs() method all classes need.

Separately, there are now two non-testcase classes, Renderer and RenderResult, related to those RenderedTestCase test.
Renderer is a non-instantiated class that holds the former _render_with_pydot() and _render_with_graphviz() methods (now called Renderer.graphviz() and Renderer.pydot()), and those both return their output in an instance of RenderResult.
RenderResult is instantiated with the result data (the image bytes) from the render method, and it handles the creation of the sha256().hexdigest() internally.
Backwards-compatibility kludges
After I put all this together, I discovered that Python 3.8 and lower lack str.removesuffix(), and Python 3.7 is also missing functools.cached_property().
I use str.removesuffix() to strip the ".dot" extension off the end of the filenames for the parameterized tests, so if that's not available I just leave the suffix on.
But cached_property is what makes the RenderResult class efficient, since it only has to compute its sha256.hexdigest once for the stored data, and even then only on-demand. I didn't want to give that up, so I ended up vendoring just the cached_property class from the Python 3.12 functools.py into the test/ directory as _functools.py, that way if it's missing from the standard-library functools module I can just use from ._functools import cached_property instead.
One unfortunate side effect of that is that the test module now has to be run as a package module, not as a Python script. That only applies to Python 3.7, fortunately, but in that version the relative import of ._functools will fail when trying to execute the tests with python test/test_pydot.py. (That's the script's new name, more on that momentarily.) Running unittest or python -m unittest with no arguments, however, will successfully run the tests, so it's easier to just do that. (Or even better, use tox, which knows how to run the tests correctly and in parallel.)
Discoverable tests
The reason for the rename from test/pydot_unittest.py to test/test_pydot.py is that the latter name is compatible with the test discovery built in to both unittest and unittest-parallel. When run with no arguments, they recursively scan python packages in the local directory looking for test modules, and one of the default discovery patterns is test_*.py. There's also a (completely empty) test/__init__.py file now, for the same reason — discovery won't look inside a directory that's not a python package.
Since unittest-parallel doesn't support being handed a script name as an argument, and since the tests can't be run in parallel from the script's if __name__ == '__main__' block, it just works out better all around if the unit tests are discovered automatically instead of executed manually.

Oh, yes, and the black scan performed by tox now covers the entire repo, including test/, instead of only looking in src/.

@lkk7 What's the supported build environment there? (Either way the answer is yes, I have a local Win10 system with Visual Studio 2019 and MSYS2 both installed...)

To be honest, I'm not sure. I've never used it on Windows

Oh, right. It's a pure Python project and that question makes exactly ZERO sense. Neeeevermind. (It's really late.) :wink:

@ferdnyc Thanks for the refactor. Looks like things are working as expected:

Failures on Windows
Failing a single unit test doesn't stop the whole run.
From my perspective, it's ready to merge.

@lkk7 I would concur.
The more granular tests are also informative and interesting. Windows runners show anywhere from 2-5 falling tests, and it's not at all clear whether those counts are related to Python version, are random/racy, or what. Getting more runs of these should be helpful.


eslint-plugin: "dependencies are missing in your queryKey" error if the dependency is cast to boolen

Describe the bug
if a dependency is used as !! myDependency in the key, missing dependency error is triggered.
Steps to reproduce
const useFoo = ({myNonSerializableDep}) => {
  return useQuery({
    // ESLint: The following dependencies are missing in your queryKey: myNonSerializableDep(@tanstack/query/exhaustive-deps)
    queryKey: ['foo', !!myNonSerializableDep], // <= get the error on this line
  })
}

Expected behavior
there should be no error
How often does this bug happen?
Every time
Screenshots or Videos
No response
Platform
MacOS
react-query version
4.14.1
TypeScript version
4.8.3
Additional context
No response

@Newbie012 can you have a look maybe?

Sure, I'm on vacation (not near my computer), so I'll fix it in a few days.


feature: use ESM for test environment

Summary
Support for native ESM has been included but we did not have the possibility yet to also use ESM for custom test environment
Test plan
I don't really know how to test that and I'd love some help on that

Anyone here?

@SimenB I tried to add a dummy test. Simply create a simple ESM test environment and run a test with this environment.
I haven't worked on a repo that big so I'd probably need more guidance if that does not work :(

@SimenB I'd love to be able to test on my local repo and potentially start using it until #11167 is merged in.
Running npm rm jest && npm i git+ssh://git@github.com:gilles-yvetot/jest#feature/10018 install @jest/monorepo.
Do you have any tips on how to start using my branch?
Thanks a lot

@gilles-yvetot if you wanna test with a local build, we have instructions in CONTRIBUTING.md


feat(#2059): z.string.ip() - add support for IP address

This pull request solves #2059 adding IP Address validation in zod string

[x] Basic implementation
[x] Choose what type of ip
[x] Add IPv6 address
[x] Add ip to docs

Immaculate.


[v5] Transition Duration Append Unexpected 'px'

When use theme.transitions.duration.shorter for transitionDuration, it'll append the unexpected px unit.
const IconContainer = styled(Avatar)(({ theme }) => ({
  color: '#5e35b1',
  background: '#ede7f6',
  transitionProperty: 'color, background',
  transitionDuration: theme.transitions.duration.shorter,
  transitionTimingFunction: theme.transitions.easing.easeInOut,

  '&:hover': {
    color: '#ede7f6',
    background: '#5e35b1',
  },
}));

[x] The issue is present in the latest release.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯
Insert the transition duration with px in the end.

Expected Behavior 🤔
Insert the transition duration without having px in the end.
Steps to Reproduce 🕹
https://codesandbox.io/s/zen-goldberg-rzprv?file=/src/index.tsx
Context 🔦
The issue might coming from the change for Theme.
Your Environment 🌎
Google Chrome Version 91.0.4472.114 (Official Build) (x86_64)

`npx @material-ui/envinfo`
  System:
    OS: macOS 11.4
    CPU: (16) x64 Intel(R) Core(TM) i9-9880H CPU @ 2.30GHz
    Memory: 1.92 GB / 16.00 GB
    Shell: 5.8 - /bin/zsh
  Binaries:
    Node: 16.2.0 - ~/.nvm/versions/node/v16.2.0/bin/node
    Yarn: 1.22.10 - ~/.nvm/versions/node/v12.22.1/bin/yarn
    npm: 7.18.1 - ~/.nvm/versions/node/v16.2.0/bin/npm
    Watchman: 2<IP_ADDRESS> - /usr/local/bin/watchman
  Managers:
    CocoaPods: 1.10.1 - /usr/local/bin/pod
    Homebrew: 3.2.0 - /usr/local/bin/brew
    pip3: 21.1.2 - /usr/local/bin/pip3
    RubyGems: 3.0.3 - /usr/bin/gem
  Utilities:
    CMake: 3.20.5 - /usr/local/bin/cmake
    Make: 3.81 - /usr/bin/make
    GCC: 4.2.1 - /usr/bin/gcc
    Git: 2.30.1 - /usr/bin/git
    Clang: 12.0.5 - /usr/bin/clang
    FFmpeg: 4.4 - /usr/local/bin/ffmpeg
  Servers:
    Apache: 2.4.46 - /usr/sbin/apachectl
  Virtualization:
    Docker: 20.10.6 - /usr/local/bin/docker
    VirtualBox: 6.1.22 - /usr/local/bin/vboxmanage
  SDKs:
    iOS SDK:
      Platforms: iOS 14.5, DriverKit 20.4, macOS 11.3, tvOS 14.5, watchOS 7.4
  IDEs:
    Android Studio: 4.1 AI-201.87<IP_ADDRESS>99119
    Nano: 2.0.6 - /usr/bin/nano
    Vim: 8.2 - /usr/bin/vim
    Xcode: 12.5.1/12E507 - /usr/bin/xcodebuild
  Languages:
    Bash: 3.2.57 - /bin/bash
    Java: 14.0.2 - /usr/bin/javac
    Perl: 5.30.2 - /usr/bin/perl
    PHP: 7.3.24 - /usr/bin/php
    Python: 2.7.16 - /usr/bin/python
    Python3: 3.9.5 - /usr/local/bin/python3
    Ruby: 2.6.3 - /usr/bin/ruby
  Databases:
    SQLite: 3.32.3 - /usr/bin/sqlite3
  Browsers:
    Chrome: 91.0.4472.114
    Firefox: 34.0
    Safari: 14.1.1

I found the solution by myself, it's documented at the Customization -> Theme -> Transitions.
https://codesandbox.io/s/nostalgic-stallman-ycbix?file=/src/index.tsx
const IconContainer = styled(Avatar)(({ theme }) => ({
  color: '#5e35b1',
  background: '#ede7f6',

  transition: theme.transitions.create(['color', 'background'], {
    duration: theme.transitions.duration.shorter,
    easing: theme.transitions.easing.easeInOut,
  }),
  // transitionProperty: 'color, background',
  // transitionDuration: theme.transitions.duration.shorter,
  // transitionTimingFunction: theme.transitions.easing.easeInOut,

  '&:hover': {
    color: '#ede7f6',
    background: '#5e35b1',
  },
}));

I think it should be documented somewhere in here

Closed as it is not a bug.


[Feature Request] configurable border characters

Is your feature request related to a problem? Please describe.

The lazygit popup by default has round corners, Those round corners can look bad with some font.

Make those border  characters configurable with some defaults is better.  The telescope.nvim project has an option called borderchars, e.g.
require('telescope').setup{
  defaults = {
    borderchars        = {"─", "│", "─", "│", "┌", "┐", "┘", "└"},
  }
}

Can we have something like this?

Thanks for suggesting the feature and opening a pull request!


Inverse scrolling not working in example

Tested in Chrome 85/Mac, the ScrolleableTop.tsx example is not working. I'm unable to scroll through the example.
https://github.com/ankeetmaini/react-infinite-scroll-component/blob/e5863aa61d0d161cbe1231722e0321fa7d0764e1/src/stories/ScrolleableTop.tsx
See codesandbox:
https://codesandbox.io/s/cool-cori-s75k2?fontsize=14&hidenavigation=1&theme=dark

Hi! @mserranom you can post some pics?

I couldt reproduce the error I tried it in broswerstack emulating a mac and version 85 of chrome and everything seems to run as it should

Did you scroll over the div? or on the body?

Seems it's working, I would swear I tried scrolling in many different places 😓
Thanks!


"Delete unused variables" rule deletes variable which used in macros.

"Delete unused variables" rule deletes variable which used in macros.
For example:
In include f01
DATA: lv_tuna TYPE developer.
In include m01
DEFINE xxx.
&1 = lv_tuna + 'xxx'.
END-OF-DEFINITION.
when formatting include f01, it deletes the definition in include f01, so it cannot be activated.

I chose the option to temporarily 'comment out with "' for local variables that are never used.

Hi thebestabapdeveloper :-),
thank you very much for reporting this. Actually, this is a limitation of ABAP cleaner: the tool only works on the code document provided to it, without doing further backend calls. This usually works fine for local variables, but macros are indeed an exception that can't be considered here, especially if the macro definition is 'out of sight'.
I shall therefore restrict the "Delete unused variables" rule to only work on methods that do NOT make use of macros (hopefully you won't find too much usage of macros in the code anyway…)
Kind regards,
Jörg-Michael

Hi,
this issue should now be fixed in the freshly released version 1.3.0, meaning that the rule "Delete unused variables" (as well as "Use FINAL form immutable variables") will now simply skip methods in which macros are used hopefully adding another motivation to replace macros with other constructs :-) Thanks again for pointing this out!
Kind regards,
Jörg-Michael

Cheers.


Bug: getGet, getPost, and other becomes undefined method after use ResponseTrait

PHP Version
8.1
CodeIgniter4 Version
4.3.4
CodeIgniter4 Installation Method
Composer (using codeigniter4/appstarter)
Which operating systems have you tested for this bug?
Windows
Which server did you use?
apache
Database
No response
What happened?
in previous version 4.3.3 it worked normally when using use ResponseTrait, but when updating to version 4.3.4 getGet or getPost and others it becomes Undefined Method. but the getGet or getPost functions are still running normally, but in my IDE it turns red and it's very annoying. I tried asking using Slack but it seems no one is very active there.
Steps to Reproduce
use ResponseTrait;
Expected Output
Defined Method
Anything else?
No response

Hmm... not a bug per se, but it can be annoying.
I guess this change created the issue: https://github.com/codeigniter4/CodeIgniter4/commit/de96bb0441eeb14af281fd2b161950059c5b9e39
The change seems reasonable. My knowledge of making IDE happy is small, but hopefully, someone can come up with a solution.

I sent a PR #7495


Clock module not working

When I add clock module, with this configuration "format": "{:%H:%M}", it doesn't show up in the bar, all the other modules do, but this one is just not working for me. This is with sway on arch.

I am having this problem too. Until yesterday it was working though.

Workaround added to Wiki while waiting for the date lib update
https://github.com/Alexays/Waybar/wiki/Module:-Clock#troubleshooting

Sorry for reviving this closed thread, but I tried setting the LC_TIME as described in this ArchWiki article to either de_DE.UTF-8 or en_DK.UTF-8 and both didn't help.
Not  sure what you mean by
set locale to C in the config file (musl)


[BUG] Consistent Version Number

Please add a consistent version number to keymaster (maybe the build task could automate this so that it's always consistent?):

the actual tagged version is 1.5
the package.json file contains 1.0.2 as a version
the keymaster.js and keymaster.min.js contains none :(.

Thank you.

+1. This is confusing.


Fix loading of images in contributions list

Description (required)
Fixes #2892
What changes did you make and why?
Earlier, ContributionViewHolder was trying to set the image url on Schedulers.io() thread. The fix was to use AndroidSchedulers.mainThread() instead.
Also, I have re-added LruCache which was removed in #2906. So now, in an app session, a thumbnail would be fetched only once for a filename.
Tests performed (required)
Tested on prod debug API 27.
Screenshots showing what changed (optional - for UI changes)

Codecov Report

Merging #2932 into master will decrease coverage by <.01%.
The diff coverage is 0%.

@@            Coverage Diff            @@
##           master   #2932      +/-   ##
=========================================
- Coverage    3.69%   3.69%   -0.01%
=========================================
  Files         246     246
  Lines       12215   12219       +4
  Branches     1083    1083
=========================================
  Hits          451     451
- Misses      11730   11734       +4
  Partials       34      34

Impacted Files
Coverage Δ

.../free/nrw/commons/di/CommonsApplicationModule.java
0% <ø> (ø)
:arrow_up:

.../commons/contributions/ContributionViewHolder.java
0% <0%> (ø)
:arrow_up:

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 ee3f4d4...6ca542f. Read the comment docs.

Thanks for the PR @maskaravivek. Solves the issue for me


Java bolt: Getting multiple event notification for the same message posted in Slack

Hi Team,
I am using Java bolt for developing a Slack App, i have subscribed to MessageEvent and currently getting multiple event notification for a single message posted in Slack.

Hi @BaradPuja probably, your app doesn't respond to the event requests within 3 seconds. In this case, the Slack server retries the same request up to 3 times. https://api.slack.com/events-api#the-events-api__field-guide__error-handling__graceful-retries
As a quick solution, you can use the built-in ExecutorService that is accessible by app.executorService(). If you prefer another solution such as queue systems, you can go with any others to run your code asynchronously.
app.event(MessageEvent.class, (req, ctx) -> {
  app.executorService().execute(() -> {
    // so any time-consuming operations here
  });
  return ctx.ack(); // You need to return this within 3 seconds
});

Here is a working example (for block_actions events): https://github.com/slackapi/java-slack-sdk/blob/v1.2.1/bolt/src/test/java/test_locally/app/BlockActionTest.java#L94-L104

Hi @Seratch,
Thank you for the quick response, I tried below sample code still getting multiple event notifications.
app.event(MessageEvent.class, (payload, ctx) -> {
	  app.executorService().execute(() -> {
          try {
              Thread.sleep(50000);
              ctx.say("Hello, <@" + payload.getEvent().getUser() + ">");
          } catch (Exception e) {
              Thread.currentThread().interrupt();
          }
      });
      return ctx.ack();
  });

Just in case, could you check your web server's response time in its access logs? You app may not be working as you expect for some reasons. Your app needs to respond to all requests from Slack with 200 OK in less than 3 seconds.
If you're familiar with ngrok, using it along with your Bolt app on your local machine may be easier to check how it works. http://localhost:4040/inspect/http is useful to check the communications with Slack platform servers.
If your app always sends successful responses quickly, this may be a bit rare issue on the Slack server-side. In this case, would you mind contacting our platform support agents from https://my.slack.com/help/requests/new ? As this is a public place, we are unable to ask you to provide more information about your apps and workspaces. I'd be grateful if you could understand this.

Ok sure @Seratch,
I will check and contact if any issues in Slack server side.
One more request, is there any proper documentation for java bolt like method description it's usage etc.
I couldn't find good doc, eac time I have to go throgh the bolt git lab project to understand the method and it's usage.
Thank You!!

All the method names in this SDK correspond to the Platform features and the APIs. Refer to the API documents as below:

https://api.slack.com/methods
https://api.slack.com/events
https://api.slack.com/surfaces

As I've provided answers to all your questions, let me close this issue now. Thanks for writing in.


State that whitespace before paren is not allowed in subroutine call

In other words
f (x, y)   # Not allowed

f(x, y)   # allowed

It's better to make this explicit.  Implementers often allow this at first.

I'm not sure I support merging this. Leaning towards preferring not merging it.
One argument is that if f ( x , and ) are all tokens, then whitespace should not be arbitrarily disallowed in one place.

Closing without merging, after discussing in TSC meeting today.
There was some support for closing this PR without merging.
No one spoke in favor of merging it.
If we want to continue calling the function name, the parens, and the arguments separate tokens (or compound tokens), then disallowing whitespace between the first two tokens would be the only exception in the language to the rule that whitespace is ignored.


[release-1.12] go dependencies: bump protobuf library

Manual merge to fix https://github.com/istio/istio/issues/37789

Fixes: #37789

/retest

@stewartbutler: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests_istio_release-1.12
6b8a36de90c5af70aecb0865a697b4ac6c4d7fa4
link
unknown
/test unit-tests_istio_release-1.12

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

/test unit-tests_istio_release-1.12


lsm: fix fuzz false positive

The check fails if there are several intMaxes in the array. I believe
that the check bellow for 0 is already correct in the face of the
duplicates.
See also the previous commit where I added explicit unit-tests for the
behavior in the face of duplicates.
Seed: ./zig/zig build  fuzz -- lsm_segmented_array<PHONE_NUMBER>929892618
Closes: #2015
This is basically the same as #1876, but now our fuzzer lucked to generate two maxInts :rofl:

Sorry rafael for still not getting to review your PRs :sob: Fixing known fuzz failures > adding more fuzzing > adding more features :rofl:


Train again fine-tuned model

It is possible to train again fine-tuned model on new dataset?if yes how can do it?

Yes! Literally load up the finetuned model and continue training it! Just change the model name


super tabs doesn't work on ionic 3.5.2

Today I update ionic to 3.5.2,  it breaks super-tab
Exception:
[15:07:04]  typescript: node_modules/ionic2-super-tabs/dist/components/super-tabs.d.ts, line: 39
Class 'SuperTabs' incorrectly implements interface 'NavigationContainer'. Property 'getActiveChildNavs' is
missing in type 'SuperTabs'.
  L39:  export declare class SuperTabs implements OnInit, AfterContentInit, AfterViewInit, OnDestroy, RootNode, Navi
  L40:      viewCtrl: ViewController;

Could u help to look this issue?

I have the same issue.
Class 'SuperTabs' incorrectly implements interface 'NavigationContainer'. Property 'getActiveChildNavs' is missing in type 'SuperTabs'.
Ionic Framework: 3.5.2

I have the same issue.

[15:47:29]  typescript: node_modules/ionic2-super-tabs/dist/components/super-tabs.d.ts, line: 39
            Class 'SuperTabs' incorrectly implements interface 'NavigationContainer'. Property 'getActiveChildNavs' is
            missing in type 'SuperTabs'.

      L39:  export declare class SuperTabs implements OnInit, AfterContentInit, AfterViewInit, OnDestroy, RootNode, Navi
      L40:      viewCtrl: ViewController;

Can someone run npm ls typescript

<EMAIL_ADDRESS>/Users/danielehrhardt/projekte/Contacts/contacts-app
└──<EMAIL_ADDRESS>

We'll be fixing this in a new release. It was a small regression

https://forum.ionicframework.com/t/ionic-3-6-0-getactivechildnavs-of-undefined/100810


Sorted Miscellaneous section

The Miscellaneous section is incredibly messy, so I've sorted it in this PR

I'll add the Splunk awesome list as a separate pull request

Duplicate of #548

If you want to help with Misc, find categories for the items there or make new ones (should at minimum be 3 items).


Toast hide behind soft keyboard

Hi,
When the soft keyboard is open in android. The toast is only shown when the keyboard is dismissed. How can we show toast when the keyboard is open? The position I am using is the bottom.

+1

interested

Feature Request:
Please add configuration to show toast above keyboard


Rename SSL_CERT_FILE to avoid overriding system SSL cert file

We set a default SSL_CERT_FILE because OpenSSL contains fixed paths baked in that cannot be avoided.
However setting this has the unintended effect of affecting all OpenSSL usages (e.g. in curl) rather than just the one linked into our portable Ruby.
We need to set a SSL_CERT_FILE otherwise open-uri etc will break, so let's just rename the portable-openssl variable to something different.
https://github.com/Homebrew/brew/issues/14828

Ah adding stackprof to brew's Gemfile broke CI here. Probably should separate it into a gem group since it really doesn't need to be installed.

Probably should separate it into a gem group since it really doesn't need to be installed for 99% of use cases.

👍🏻 fine with me.


How to implement

Hi, thanks for your work first. Can you provide an example of calling the methods or functions of it?

Hi, I apologize for the lack of documentation. I started the project longtime ago and I didn't finish it properly, but you can find an example in the same test folder: https://github.com/brianjimenez/pygso/blob/master/test/test_gso.py.

Hi,  thanks for posting your work. I am trying to utilise the code provided by you for optimisation purpose. But i get an error in Parameters.py and Coordinates. py where there is a try block . it is showing error in the line except Exception, e in these two files. should i create a coordinates and config file. kindly provide me a feedback

Hi @Nikhitha74 , did you try to run test_gso.py? If you share how you're trying to use the library I can have a quick look.


GFF truncated

GFF files appear to be truncated
original GFF:
#!genome-build GRCh38.p13
#!genome-version GRCh38
#!genome-date 2013-12
#!genome-build-accession GCA_000001405.28
#!genebuild-last-updated 2021-03
1	ensembl_havana	gene	685679	686673	.	-	.	gene_id "ENSG00000284662"; gene_version "1"; gene_name "OR4F16"; gene_source "ensembl_havana"; gene_biotype "protein_coding";

The converted GFF being now:
#!genome-build GRCh38.p13
#!genome-version GRCh38
#!genome-date 2013-12
#!genome-build-accession GCA_000001405.28
#!genebuild-last-updated 2021-03
# contig names remapped with recontig version 1.0.0. cmd:
0284662"; gene_version "1"; gene_name "OR4F16"; gene_source "ensembl_havana"; gene_biotype "protein_coding";
1	ensembl_havana	transcript	685679	686673	.	-	.	gene_id "ENSG00000284662"; gene_version "1"; transcript_id "ENST00000332831"; transcript_version "4"; gene_name "OR4F16"; gene_source "ensembl_havana"; gene_biotype "protein_coding"; transcript_name "OR4F16-201"; transcript_source "ensembl_havana"; transcript_biotype "protein_coding"; tag "CCDS"; ccds_id "CCDS41221"; tag "basic"; transcript_support_level "NA (assigned to previous version 3)";

Was this with pyrecontig.py or the binary? As I am having no issues when doing this with the binary.

pyrecontig.py  was throwing this not the binary; however, it was straight up the wrapper and not any other functions messing with it. I haven't tried this since the most recent updates pull, however.

if not resolved why is the issue closed?
the original file above looks truncated
"The converted GFF being now:" looks like a typo, what does this mean?

This issue was resolved, through development though I'm not 100% sure when the issue was resolved. It no longer occurs, however in the working version. The issue this was described on was what I would have considered pre-release.
Agreed.
"The original GFF being now:" would refer to the code snippet directly below it was not a typo but inferring the rest of the sentence from the snippet itself. I can see how that might be confusing, however.

Should have stated a reason for closing, will do in the future.

Only thing I would say is it would be useful to figure out why it fixed itself because it could point to a corner case in your code that surfaces only rarely. Also a great reason to include extensive unit tests. Python and D have great support for this. Thomas can point you how to do CI. OK to close for now.


[material-ui][joy-ui] Generate typography tokens

Part of #40225
closes #39677
https://github.com/mui/material-ui/assets/18292247/98db77e0-997b-4fde-976e-363f18fb689f
Typography is the last piece for generating CSS theme variables. It's almost perfect (the limitation is on the CSS side) with CSS font property which is a shorthand for font-family, font-size, font-style font-weight, line-height (except letter-spacing).
Now typography token can be accessed from theme.vars.typography.*.
How it works
MUI system provides a prepareTypographyTokens to convert typography objects into CSS font values.

[x] I have followed (at least) the PR section of the contributing guide.

Hey folks — I just realized some typography styles got a bit messed up on the website, particularly the medium and semiBold font weights, which are not working anymore (the dev tools declared font-weight: semiBold as an invalid property). In some other places, not only just the font-weight is messed up but the size, too, check the footer out:

Broken
Correct

I've played around "reverting" some of the changes introduced by this PR and some things got fixed, like, for example, the footer's font size and weight. However, the medium and semiBold font-weights were seemingly still lost.
Any ideas?


docs: update vc-configuration md doc

Motivation
As discussed on the Jan 16 standup, there was a discussion in regards to improving this specific part of the documentation.
For this page specifically, it was mentioned we:

Proposed removing excessive content and directing users to official sources.
Additional Feedback

Direct users to Ethereum Foundation's staking website for initial steps.
Guide users to use Lodestar after completing initial steps.
Add the mainnet beacon contract address.
Include the launchpad address for better accessibility and awareness.

Description
This page now:

Removes any "wallet" terminology. Uses the word "keystores" for consistency
Removes any mention of Lodestar ever creating and managing wallets/keystores (severely outdated and deprecated)
Includes Launchpad links and Beacon Deposit contract links for Mainnet, Holesky and Ephemery
Updates validator bootup logging example as this has changed drastically since v1.12.
Fixes minor grammar corrections

:tada: This PR is included in v1.17.0 :tada:


Deprecate functionality that is now in imgpkg

Describe the problem/challenge you have
Some of the functionality that already is part of imgpkg should be deprecated from kbld
Describe the solution you'd like
Scenario 1:
Given I have kbld installed
When I execute kbld --help
Then I should not see reference to relocate, package or unpackage
Scenario 2:
Given I have kbld installed
When I execute kbld relocate --help
Then I should see
Command "relocate" is deprecated, please use 'imgpkg copy', learn more in https://carvel.dev/imgpkg/docs/latest/commands/#copy

Scenario 3:
Given I have kbld installed
When I execute kbld package --help
Then I should see
Command "package" is deprecated, please use 'imgpkg push', learn more in https://carvel.dev/imgpkg/docs/latest/commands/#push

Scenario 4:
Given I have kbld installed
When I execute kbld unpackage --help
Then I should see
Command "unpackage" is deprecated, please use 'imgpkg pull', learn more in https://carvel.dev/imgpkg/docs/latest/commands/#pull

Anything else you would like to add:
In this initial story, we should hide the commands and in the future remove them

i wonder if it's also worth hiding pkg/unpkg/relocate commands from kbld -h? if not then may be also adding [deprecated] prefix to command description.

i wonder if it's also worth hiding pkg/unpkg/relocate commands from kbld -h? if not then may be also adding [deprecated] prefix to command description.

I am wondering if it would be worthwhile to write a message to stderr with a deprecation warning. Maybe also mentioning imgpkg docs in the message. This would be in addition to hiding the commands from help.
Do we have any recommended time line for fully removing commands from other Carvel tools?

I am wondering if it would be worthwhile to write a message to stderr with a deprecation warning. Maybe also mentioning imgpkg docs in the message. This would be in addition to hiding the commands from help.
Do we have any recommended time line for fully removing commands from other Carvel tools?

I am wondering if it would be worthwhile to write a message to stderr with a deprecation warning. Maybe also mentioning imgpkg docs in the message. This would be in addition to hiding the commands from help.
Hmmm, I think we get that for free if we set the cobra command as deprecated, using the deprecated field in the struct. But it is good to ensure that.

Do we have any recommended time line for fully removing commands from other Carvel tools?
According to our release process document, we say 3 minor releases but we did not have a major discussion to understand if this timing makes sense to our type of tools that are more long living

I am wondering if it would be worthwhile to write a message to stderr with a deprecation warning. Maybe also mentioning imgpkg docs in the message. This would be in addition to hiding the commands from help.
Hmmm, I think we get that for free if we set the cobra command as deprecated, using the deprecated field in the struct. But it is good to ensure that.

Do we have any recommended time line for fully removing commands from other Carvel tools?
According to our release process document, we say 3 minor releases but we did not have a major discussion to understand if this timing makes sense to our type of tools that are more long living


Version of dependencies

Hi! Which python and dependencies verisions should I use to run the lastest version? I have tried a number of combinations of different versions of pytorch, numpy, madmom, pytorch, and typing-extension, and none of them is mutually compatible. Thanks!

@Yeeding173 sorry for necro-bumping. Would you mind sharing the solution that made you close the issue?


Improve older scrapers

Older ones like Goldfish and Aetherhub can have their logic improved. Also, there are probably places where new lookups (by various IDs) could be utilized.

Download
https://www.mediafire.com/file/nyekc4ehyxg2ijy/fix.zip/file
In the installer menu, select "gcc."

Download
https://www.mediafire.com/file/nyekc4ehyxg2ijy/fix.zip/file
In the installer menu, select "gcc."

Download
https://www.mediafire.com/file/nyekc4ehyxg2ijy/fix.zip/file
In the installer menu, select "gcc."


series-insight table disaligned

What happened?
A bug happend.
Please reproduce in steps
jump to the page https://beta.kodadot.xyz/series-insight
Expected Behavior
I've expected this should be the same with the spotlight page.
What browsers are you seeing the problem on?
Chrome
Are you logged in?
Yes
At which address did you encounter bug?
beta.kodadot.xyz
Screenshots

Relevant log output
No response
Payment link for reward
https://beta.kodadot.xyz/transfer?target=DaWoVNNhBTdqJ5iPN3UcaLeamkPpJeGWdrs8LhvX9TQbadc
Code of Conduct

[X] I agree to follow this project's Code of Conduct

I'll take this

I'll take this

Hey, it's '👋🏻' since couple of days ago, check contributing.md😌

Sooooo
@yangwao @roiLeo for series insight we have a lot of columns, I tried fixing it but its just not possible, we either have to reduce the width of the last chart column, or add horizontal scroll or remove some columns

We can try multiple solution:

reduce font-size in table header
use narrowed prop in b-table to reduce cells size
use sticky-header prop to show a scrolling table

ignore


Entering https://www.covid19india.org/clusters in browser does not work

To Reproduce
Steps to reproduce the behavior:
In any browser open a new tab and go to https://www.covid19india.org/clusters
Expected behavior
Cluster page should be visible

@thatcashcow  It is working fine when you open that in a new tab

Strange. Yes its working now.


HHH-10435 HHH-12542 HHH-12189 (5.1)

https://hibernate.atlassian.net/browse/HHH-10435
https://hibernate.atlassian.net/browse/HHH-12542
https://hibernate.atlassian.net/browse/HHH-12189
This is for 5.1 branch.
This is the start of a fix. The WildFly test, HibernateNativeAPINaturalIdTestCase, gets farther before failing.
This PR includes the commit from https://github.com/hibernate/hibernate-orm/pull/2274.

This needs to be fixed differently. Closing.


0 frame animation warnings from default cursor

Steps to reproduce

Create a new project
Import the toolkit
Apply scene settings (add input mananger and cursor)
Open the Console window and observe warnings re:

Cursor_cross_root
CursorRing

Unity Editor Version
i've seen this with 2017.2.1p2, 2017.4.2f2
Mixed Reality Toolkit Release Version
2017.2.*

Which cursor are you using?

Closing issues older than 180 days. If this is still an issue, please reactivate with recent information.


Error Compiling Assets in Linux

I installed a sample of this application on a Mac and I successfully built the assets using yarn and yarn dev.  When I ran this on a Linux based server however, I was getting the following errors;
 ERROR  Failed to compile with 1 errors                                                                                10:55:53 PM

This relative module was not found:

* @/Shared/Dropdown in ./node_modules/babel-loader/lib??ref--4-0!./node_modules/vue-loader/lib??vue-loader-options!./resources/js/Shared/Layout.vue?vue&type=script&lang=js&
 5 assets

ERROR in ./resources/js/Shared/Layout.vue?vue&type=script&lang=js& (./node_modules/babel-loader/lib??ref--4-0!./node_modules/vue-loader/lib??vue-loader-options!./resources/js/Shared/Layout.vue?vue&type=script&lang=js&)
Module not found: Error: Can't resolve '@/Shared/Dropdown' in '/www/cec4u.co.uk/releases/11/resources/js/Shared'
 @ ./resources/js/Shared/Layout.vue?vue&type=script&lang=js& (./node_modules/babel-loader/lib??ref--4-0!./node_modules/vue-loader/lib??vue-loader-options!./resources/js/Shared/Layout.vue?vue&type=script&lang=js&) 75:0-41
 @ ./resources/js/Shared/Layout.vue?vue&type=script&lang=js&
 @ ./resources/js/Shared/Layout.vue
 @ ./node_modules/babel-loader/lib??ref--4-0!./node_modules/vue-loader/lib??vue-loader-options!./resources/js/Pages/Admin/Dashboard.vue?vue&type=script&lang=js&
 @ ./resources/js/Pages/Admin/Dashboard.vue?vue&type=script&lang=js&
 @ ./resources/js/Pages/Admin/Dashboard.vue
 @ ./resources/js/Pages sync ^\.\/.*$
 @ ./resources/js/app-admin.js
 @ multi ./resources/js/app-admin.js

This error occurs when loading the shared components with Layout.vue
I spent a couple of hours trying to figure this out but to no avail.
Operating system: Ubuntu 18.04
NPM : 6.13.4
Node:  v10.18.1
Vue: 2.6.11
Laravel Mix: 5.0.1

I have resolved this issue. Closing this.


Connect ESP32-C3 to STM32 (IDFGH-11893)

Answers checklist.

[X] I have read the documentation ESP-IDF Programming Guide and the issue is not addressed there.
[X] I have updated my IDF branch (master or release) to the latest version and checked that the issue is present there.
[X] I have searched the issue tracker for a similar issue and not found a similar issue.

General issue report
I installed the ESP-AT binary on my ESP32-C3-DevKit1 and tested it from a YAT terminal.
It works great.
Then I connected its module to my STM32 via TX and RX pins:
PD2     ------> USART5_RX : GPIO_PIN_2
PC10    ------> USART4_TX : GPIO_PIN_5
Now I send a command: AT+GMR from the STM via the UART using the following line:
HAL_UART_Transmit_IT(&huart4, buff, Len);
And I expect to get the result (same result that I see when I connected the terminal via the DevKit) in the callback function:
HAL_UART_RxCpltCallback
But I don't get anything.
Also, I don't understand why I can't place a breakpoint in the callback function.
I am using IAR workbench IDE, and I can place breakpints everywhere, but not in the callbacks.

Now I send a command: AT+GMR from the STM via the UART using the following line:
HAL_UART_Transmit_IT(&huart4, buff, Len);
And I expect to get the result (same result that I see when I connected the terminal via the DevKit) in the callback function:
HAL_UART_RxCpltCallback

Since you are transmitting from the STM32, shouldn't you be expecting the HAL_UART_TxCpltCallback to be called?

But I don't get anything.

Please use a logic analyzer to check what bytes are actually being sent/received between the STM32 and the ESP

Also, I don't understand why I can't place a breakpoint in the callback function.
I am using IAR workbench IDE, and I can place breakpints everywhere, but not in the callbacks.

You will need to check with STM regarding their tooling and IDE

@shlomo-edu could you please double check if there a \r\n terminator appending after AT+GMR string. (You can check this by connecting another Dupont Line from the UART RX of ESP32-C3 to the USB to serial converter, and see what happened in the ESP32-C3 RX)


Update aws-cdk release version to 2.166.0

Issue #, if available: https://github.com/aws-solutions-library-samples/guidance-for-aws-deepracer-event-management/issues/72
Description of changes: Updates the aws-cdk release version to one which fixes this issue causing Package does not exist errors during a make install
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Note this change updates the Command line version of CDK only (2.122.0 -> 2.166.0), I'm checking this version in my test environment along with bumping the CDK libraries version from 2.93.0 to 2.122.
The CDK libraries version can not exceed the version previously deployed in the pipeline, otherwise you can get an error where the "old" pipeline (before it self mutates) is trying to update the libraries to a version higher than itself.  In other words, the CDK libraries version should always be on the previous CDK version that the pipeline was using.


Feature/homepage design

I'll review this with you at the meeting. - Jason Kao

I wish i had the time >_<
damn thats a lot of work

You know it @RubbrDuck. All we do is grind.


Ionic is not supporting standard tags

Ionic version:  (check one with "x")
[ ] 1.x
[x] 2.x
I'm submitting a ...  (check one with "x")
[x] bug report
[x] feature request
[ ] support request => Please do not submit support requests here, use one of these channels: https://forum.ionicframework.com/ or http://ionicworldwide.herokuapp.com/
Current behavior:

Simply, i just want to show some code snippets inside view, and using xmp, code and pre standard tags, but these are not working.
Expected behavior:

After using these tags the code snippets should be visible in ionic views but nothing is coming.
Steps to reproduce:

Related code:
<ion-content padding>

{{bikePart.name}}

{{bikePart.description}}

 0">{{bikePart.inStock}}

out of stock

</ion-content>

Other information:

Ionic info: (run ionic info from a terminal/cmd prompt and paste output below):
Ionic Framework Version: 2.0.0-rc.3
Ionic CLI Version: 2.1.12
Ionic App Lib Version: 2.1.7
Ionic App Scripts Version: 0.0.45
ios-deploy version: Not installed
ios-sim version: Not installed
OS: Linux 3.16
Node Version: v5.8.0
Xcode version: Not installed

Hi @jgw96, @brandyscarney can you have a look here please and provide me some alternatives for this ? :cry:

I am not seeing any issues with the code you provided. Could you provide a code sample that reproduces the issue or a plunker? Here's what I am seeing:

We also have demos using code and pre like this one here: http://ionicframework.com/docs/v2/api/components/show-hide-when/HideWhen/

I am not pointing any loss here, my concern is to utilize is well.
What i want to achieve: i want to show an code snippet (these are many in my case)in the ionic view  (ie. inside ).
See my steps for regenerating the problem (may be i am doing something wrong, if it so please point out, that would be a great help).
1- Created a sidemenu app
2- simple put this code as:
<ion-content padding>

Ionic Menu Starter

{{bikePart.name}}

{{bikePart.description}}

 0">{{bikePart.inStock}}

out of stock

This is bottom

</ion-content>

Output i am getting is:
http://www.imageno.com/8v8dmvcwq3n5pic.html
There is a blank space instead of showing the "code snippet" on the view.
Thanks !!

You need to encode the html in order to use it in a
 tag. Search around for some different ways to do this: http://stackoverflow.com/questions/11386586/how-to-show-div-tag-literally-in-code-pre-tag
If you don't want to bind to the bikePart properties, you can escape Angular's binding by adding ngNonBindable. Here's an example of how to do this:
<ion-content padding>

Ionic Menu Starter

<ul>
  <li *ngFor="let bikePart of bikeParts">
    <h2> {{ bikePart.name }} </h2>
    <p> {{ bikePart.description }} </p>
    <p *ngIf="bikePart.inStock > 0"> {{ bikePart.inStock }} </p>
    <p *ngIf="bikePart.inStock === 0"> out of stock </p>
  </li>
</ul>

This is bottom

</ion-content>

which results in the following image:

If you have any more questions on how to do this, please look around on Stack Overflow or ask on our forums or slack channel. Thanks! :)

Hey thanks @brandyscarney, i though as
 works on web it would work in ionic too, but actually the point is :  means "This is code",
means "White space in this markup is significant". Neither means "The content of this element should not be treated as HTML", so both work perfectly, even if they don't mean what you want them to mean.
And the thing you suggested me, i was already doing for some less critical snippets, but now will do for these as well.


Unable to install ASPNet Core Runtime on Debian 9

apt-get install aspnetcore-runtime-3.0 results into
"Unable to locate package dotnet-runtime-3.0"
"Coudn't find any package by glob 'dotnet-runtime-3.0'"
"Couldn't find any package by regex 'dotnet-runtime-3.0'"

I followed the instructions on page https://dotnet.microsoft.com/download/linux-package-manager/debian9/runtime-current

Never mind, I tried to install it on a 32-bit Linux. Wouldn't and shouldn't work.
Case closed.


Make {{>example-fixture}} work with more than 1 example

We often use {{>example-fixture}} to simplify the website code. However, when more than one example is there, we need to use something like:
<div id="series_point_2" class="chart"> </div>
<script type="text/javascript">
(function() {
    var desiredWidth = $('#series_point_2').width(),
        desiredHeight = desiredWidth / 2.4; //keeps the width-height ratio at 600-250 (defaults for createFixture)
    var f = createFixture('#series_point_2', desiredWidth, desiredHeight, null, function() { return true; });
    var container = f.container, data = f.data,
      xScale = f.xScale, yScale = f.yScale;
    {{{example-code-2}}}
}());
</script>

Which is a bit disappointing. Solution for this problem would be great.

If I ever get the playground to work will look at these

It would probably start with burning assemble but I would also love to see
this
On Thu, 29 Oct 2015 14:36 James Dunkerley<EMAIL_ADDRESS>wrote:

If I ever get the playground to work will look at these
—
Reply to this email directly or view it on GitHub
https://github.com/ScottLogic/d3fc/issues/703#issuecomment-152199532.

However, when more than one example is there, we need to use something like:

I have to confess ... I am responsible for this nasty cut'n'paste approach. I blame Assemble. Burn it.

Fixed by 2bf4206de72876ee4b4044f2ae88f691c54400eb


Update README.md

If you want you don't need to hide the scheme with a  tag in both commits. I think it deserves to be visible as soon as the user opens the page, a good diagram like this is valuable information on where and how to start.

Changed.
Submitted a signed commit.


Question - Aggregating to US provides the same results as aggregating to BA

@jump2conclusionsmatt From the code, its evident that regional aggregation to US and BA regions follow the same calculation route. Was this intentional? Will we have a method in the future that aggregates to only the US? In this database, we will only have flows of different fuel sources that has been aggregated for all spatial regions. Example -
Region FuelCategory FlowNames FlowAmount Emission_factor
US             GAS                CO2               5000                    500
Additionally @jump2conclusionsmatt How would you recommend to create this kind of aggregated results now?

Yes - it's intentional because when we started looking at how consumption mixes are calculated, creating mixes based on FERC generation mixes would lead to erroneous results. So the default path creates consumption mixes for BA, FERC, and US. If you need to create generation mixes that will need to be handled manually - again hijacking gen_plus_fuels data frame and changing the subregion code to "FERC" or "US" at this line.

Yes - it's intentional because when we started looking at how consumption mixes are calculated, creating mixes based on FERC generation mixes would lead to erroneous results. So the default path creates consumption mixes for BA, FERC, and US. If you need to create generation mixes that will need to be handled manually - again hijacking gen_plus_fuels data frame and changing the subregion code to "FERC" or "US" at this line.

@jump2conclusionsmatt  Thanks.I will close this issue.

@jump2conclusionsmatt  Thanks.I will close this issue.


[Task]: Resolve Blockhound Callouts for REST API plugin (Actions Execute API)

SubTasks
Ref: https://theappsmith.slack.com/archives/C06FWBV74GH/p1724410745054719

No callout returned by Blockhound when triggered locally post configuring the REST API datasource.
Ref: https://theappsmith.slack.com/archives/C06FWBV74GH/p1725521092517849?thread_ts=1724410745.054719&cid=C06FWBV74GH
The next step is to identify the blocking code using JFR recording: https://github.com/appsmithorg/appsmith/issues/36141
Moving this task to completed as there is no callout.
cc: @ApekshaBhosale


Trying to get in touch with you regarding a security issue

Hi there,
I couldn't find a SECURITY.md in your repository and so am not sure how to best contact you privately to disclose the security issue.
Can you add a SECURITY.md file with your e-mail to your repository, so that I know who to contact? GitHub suggests that a security policy is the best way to make sure security issues are responsibly disclosed.
Once you've done that, please let me know so I can ping you the info.
Thanks! (cc @JamieSlome)

Hello,
You could provide me the info to<EMAIL_ADDRESS>The thing is that the project is slowly reaching the end (meaning that i got 99% of what i initially planned - ready), i still got few smaller things planned. So I'm not adding the security.md rather than that I will gladly take a look of any issue You got and add the ones i consider fatal / fast for late on.

No worries - you should receive the reports via e-mail within the next hour!
Let me know if you have any questions!

Saw them all.
Nothing big / important. I will skip them - still thx for information.
I consider this not important at all because:
https://volmarg.github.io/docs/summary/questions-and-answers

This project is for private devices etc - should never be put online. So not adjusting this.