Burt Harris

TSLint has a few nits to pick with our generated code. Adding a few comments to the templates will suppress them. For now, lets just turn it of for generated...

The antlr4ts source file structure is based on it's Java heritage, where a source-file per class model is considered best-practice (with exceptions). However, in the JavaScript/TypeScript world the impact of...

Use the /** @internal */ marker where appropriate

2

To narrow the "public" API and reduce semantic version number creep, I suggest we apply **`@internal`** in JSDoc comments. When we turn on the `--stripInternal` switch, this will cause the...

Fix doc comments re null.

1

There number of places where where changes in a return type were made allowing `undefined` in the return type, but the comments still talk about returning null. For example: Vocabulary.ts...

Suggestion: tweak height/spacing/round corners

3

The stability badge from this set match up well in appearance with others I'm using. For example: [](http://github.com/hughsk/stability-badges) [](https://gitter.im/tunnelvisionlabs/antlr4ts?utm_source=badge&utm_medium=badge&utm_campaign=pr-badge&utm_content=badge) [](https://ci.appveyor.com/project/sharwell/antlr4ts/branch/master) [](./LICENSE)

Add a definition of multimap

12

The INFRA spec currently has a good definition of ordered map, but its definition requires key uniqueness. Sometimes the uniqueness constraint isn't desirable. For example whatwg/fetch describes [header-list](https://fetch.spec.whatwg.org/#concept-header-list) as multimap,...

Map and multiple floors

4

I like my new Shark IQ, but the Shark app seems to lack one feature that's important for my home: multi-floor mapping. I started looking for a Shark API to...

DRAFT: Use least-privilege install and use strategy by default.

4

Default to a per-user install Move the default NVM_SYMLINK into NVM_HOME Don't mix per-user and system install Switch to use of NTFS Junction points

**Issue:** The build script issues a warning from innosetup, saying: > Warning: The [Setup] section directive "PrivilegesRequired" is set to "admin" but per-user areas (userappdata) are used by the script....

2.4: Readonly<Map<k,v>> vs. ReadonlyMap<k,v>

9

**TypeScript Version:** 2.4.1 **Code** ```ts function GenMap() : ReadonlyMap { const theMap = new Map(); theMap.set("one", 1); theMap.set("two", 2); return Object.freeze(theMap); } ``` **Expected behavior:** Compiles without error in tsc@