Skip to content
This repository has been archived by the owner on Feb 22, 2018. It is now read-only.

fix(ts2dart): emit members for protected property parameters #240

Merged
merged 1 commit into from
Jul 10, 2015

Conversation

caitp
Copy link
Contributor

@caitp caitp commented Jul 10, 2015

I know it's not very meaningful in dart, but it's useful for TypeScript.

When mixed with dart's module-private semantics, it could work okay for
both.

I know it's not very meaningful in dart, but it's useful for TypeScript.

When mixed with dart's module-private semantics, it could work okay for
both.

Closes dart-archive#239
@caitp caitp changed the title Support protected parameter property declarations emit members for protected property parameters Jul 10, 2015
@caitp caitp changed the title emit members for protected property parameters fix(ts2dart): emit members for protected property parameters Jul 10, 2015
@jeffbcross
Copy link
Contributor

@alexeagle mind taking a look?

@alexeagle
Copy link
Contributor

LGTM

@jeffbcross
Copy link
Contributor

I'll merge, but will wait until Monday to bump the release, since there's apparently a special process.

@jeffbcross jeffbcross merged commit 910c51c into dart-archive:master Jul 10, 2015
@alexeagle
Copy link
Contributor

Hmm, not good that we haven't published steps for anyone who needs to do a
release.
But I'm not sure what the special process refers to. It's just npm publish
and then update the shrinkwrap.

On Fri, Jul 10, 2015 at 3:29 PM Jeff Cross notifications@github.com wrote:

Merged #240 #240.


Reply to this email directly or view it on GitHub
#240 (comment).

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Development

Successfully merging this pull request may close these issues.

4 participants