Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Convert bolt-replacements/getWorkspaces to TypeScript #46

Merged
merged 3 commits into from
May 21, 2019
Merged

Convert bolt-replacements/getWorkspaces to TypeScript #46

merged 3 commits into from
May 21, 2019

Conversation

emmatown
Copy link
Member

This converts a single file to TypeScript which is the one that depends on get-workspaces, I converted this specific file so that I could test type resolving across packages in a monorepo and what preconstruct needs to do. What I've ended up with is preconstruct creates a symlink from dist/get-workspaces.cjs.js.ts to the source file in dev and in prod, it generates a file at dist/get-workspaces.cjs.js.d.ts which is just like the flow redirect. (I want to generate actual TS declarations soon but this is a good first step)

@changeset-bot
Copy link

changeset-bot bot commented May 21, 2019

🦋 Changeset is good to go

Latest commit: 880c702

We got this.

Not sure waht this means? Click here to learn what changesets are.

@changeset-bot
Copy link

changeset-bot bot commented May 21, 2019

🦋 Changeset is good to go

Latest commit: 3c063b6

We got this.

Not sure what this means? Click here to learn what changesets are.

1 similar comment
@changeset-bot
Copy link

changeset-bot bot commented May 21, 2019

🦋 Changeset is good to go

Latest commit: 3c063b6

We got this.

Not sure what this means? Click here to learn what changesets are.

@changeset-bot
Copy link

changeset-bot bot commented May 21, 2019

🦋 Changeset is good to go

Latest commit: 3c063b6

We got this.

Not sure waht this means? Click here to learn what changesets are.

@Noviny Noviny merged commit c6f1c7b into changesets:master May 21, 2019
@emmatown emmatown deleted the ts-stuff branch May 21, 2019 01:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants