Re use readonly #12732 #13863 #18251

Merged
merged 5 commits into from Jan 9, 2017

Projects

None yet

3 participants

@Ikuyadeu
Contributor
Ikuyadeu commented Jan 7, 2017 edited
Ikuyadeu added some commits Oct 17, 2016
@Ikuyadeu Ikuyadeu use readonly #12732 e5f5df7
@Ikuyadeu Ikuyadeu readonly to isResolved 62dd640
@Ikuyadeu Ikuyadeu remove @readonly comments 60645d2
@Ikuyadeu Ikuyadeu fix test
14380ad
@Ikuyadeu Ikuyadeu referenced this pull request Jan 7, 2017
Closed

Use readonly #12732 #13863

@jrieken jrieken self-assigned this Jan 7, 2017
@jrieken jrieken added this to the January 2017 milestone Jan 7, 2017
@Ikuyadeu Ikuyadeu fix coment doc
96c178d
@jrieken
Member
jrieken commented Jan 9, 2017

Thanks. Merging - we loose having explicitly spelled out readonly in the generated doc page because typedoc doesn't preserve that information. Still, the benefit outweighs that.

@jrieken jrieken merged commit 7c66baa into Microsoft:master Jan 9, 2017

2 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@Ikuyadeu Ikuyadeu deleted the Ikuyadeu:re_use_readonly branch Jan 10, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment