developit / developit/decko

Is memoize supposed to work on getters?

Open
#15 2 comments 8 reactions 0 assignees View on GitHub
enhancement help wanted
Dominant language
JavaScript
Stars
1k
Forks
34
PR merge metrics
No merged PRs in 30d

Description

(tested in Typescript)

```
import { memoize } from 'decko';

class Stores(){
}

class Container {

@memoize
public get stores () {
return new Stores();
}
}

```

Throws an error:
`​​Invalid property descriptor. Cannot both specify accessors and a value or writable attribute, #​​`

For reference: https://github.com/darrylhodgins/typescript-memoize does work.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.