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

chore: finish migration to TypeScript #788

Merged
merged 4 commits into from
Oct 14, 2019
Merged

chore: finish migration to TypeScript #788

merged 4 commits into from
Oct 14, 2019

Conversation

grabbou
Copy link
Member

@grabbou grabbou commented Oct 14, 2019

This PR is the last PR as a part of #683. The entire codebase have been migrated.

I have just recently started the effort and wanted to thank everyone in the first place for stepping in and migrating and moving files over!

Closes #683

@@ -57,6 +56,7 @@ test('init --template', () => {
expect(fs.readdirSync(DIR)).toEqual(['TestInit']);

let dirFiles = fs.readdirSync(path.join(DIR, 'TestInit'));
console.log(dirFiles);
Copy link
Member

Choose a reason for hiding this comment

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

remove console

Copy link
Member

@thymikee thymikee left a comment

Choose a reason for hiding this comment

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

remove that one console.log, fix linter and :shipit:

@grabbou grabbou merged commit 78bfbfb into master Oct 14, 2019
@thymikee thymikee deleted the feat/remove-flow branch February 6, 2020 08:51
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.

☂️Umbrella: TypeScript migration
2 participants