Kannan Goundan

Results 38 comments of Kannan Goundan

Looks like Mammoth tries to automatically wrap in quotes, but its heuristic is off: https://github.com/Ff00ff/mammoth/blob/8122afe43425c135a1dd8f0cd22606fb0e0a994d/workspaces/mammoth/src/naming/index.ts#L4-L11 From the Postgres manual ([link](https://www.postgresql.org/docs/current/sql-syntax-lexical.html#SQL-SYNTAX-IDENTIFIERS)): > SQL identifiers and key words must begin with a...

(I work with @aphahn and deal with security/compliance stuff.) @mmomtchev: Totally agree that there is no _actual_ vulnerability here. The issue is that it causes some friction in our security...

At least initially, the idea was to provide a single file that you can just copy into your project. Not sure if there are any plans to publish on NPM.

Yeah, a large number of methods can be an issue for some people. Is it bloating your final app's DEX size? If so, ProGuard (or some other minification tool) might...

The SDK itself currently uses the [Checker Framework](http://types.cs.washington.edu/checker-framework/)'s [Nullness Checker](http://types.cs.washington.edu/checker-framework/current/checker-framework-manual.html#nullness-checker). (It seemed like the best available option when I looked into this last year.) I just posted a question on...

Sort of a tangent: Djinni expects all identifiers in the ".djinni" file to be of the form "foo_bar". Maybe it should warn when they aren't?

Ah, interesting! Where's the `{}` requirement coming from, the comparison? For example, I'm confused why there's no error here: ```ts function f(a: T, b: T): boolean { return a >...

Yeah, the no-duplicates won't help here because you can't combine wildcard and non-wildcard into a single import. For me, any consistent ordering would be fine, e.g. wildcard first.