Skip to content

getOccurrences for 'get' and 'set' keywords.#778

Merged
DanielRosenwasser merged 2 commits into
masterfrom
getOccurrencesSetItAndForGetIt
Sep 30, 2014
Merged

getOccurrences for 'get' and 'set' keywords.#778
DanielRosenwasser merged 2 commits into
masterfrom
getOccurrencesSetItAndForGetIt

Conversation

@DanielRosenwasser

Copy link
Copy Markdown
Member

'get'/'set' keywords are highlighted for analogous accessors.

Fixes #769.

'get'/'set' keywords are highlighted for analogous accessors.
Comment thread src/services/services.ts Outdated

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

if (accessor) { forEach(...) }

@CyrusNajmabadi

Copy link
Copy Markdown
Contributor

👍

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what is false here? Please add a named argument comment in your tests

@JsonFreeman

Copy link
Copy Markdown
Contributor

👍

DanielRosenwasser added a commit that referenced this pull request Sep 30, 2014
getOccurrences for 'get' and 'set' keywords.
@DanielRosenwasser DanielRosenwasser merged commit f2880ce into master Sep 30, 2014
@DanielRosenwasser DanielRosenwasser deleted the getOccurrencesSetItAndForGetIt branch September 30, 2014 21:54
@microsoft microsoft locked and limited conversation to collaborators Jun 18, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Should highlight get and set kewords if one is selected

3 participants