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: migrate more files to TS #398

Merged
merged 9 commits into from
May 27, 2019
Merged

chore: migrate more files to TS #398

merged 9 commits into from
May 27, 2019

Conversation

zamotany
Copy link
Contributor

  • src/index.js was not migrated as it's breaking the tests, probably something with resolving linaria import is working as expected
  • src/react/*.js - same story, even changing the extension causes tests to fail

@satya164 do you have any idea why that is happening? Or how we can solve that?

@zamotany zamotany requested a review from satya164 April 28, 2019 12:54
@satya164
Copy link
Member

I'll check tonight or in Tuesday

@callstack-bot
Copy link

callstack-bot commented May 5, 2019

Hey @zamotany, thank you for your pull request 🤗.
The coverage report for this branch can be viewed here.

@satya164 satya164 force-pushed the chore/migrate-to-fs branch 2 times, most recently from c0e1d4f to 4688df0 Compare May 5, 2019 13:04
@satya164 satya164 force-pushed the chore/migrate-to-fs branch from 4688df0 to e02a65e Compare May 5, 2019 13:04
babel.js Show resolved Hide resolved
@pbitkowski
Copy link
Contributor

@zamotany could you include changes from #377 if they make sense?

@thymikee
Copy link
Member

FYI: #387

@pbitkowski pbitkowski mentioned this pull request May 17, 2019
@zamotany
Copy link
Contributor Author

@satya164 @pbitkowski @thymikee
Finished migration, ported those 2 PRs, everything is passing, website is working just fine, but:

  • testing declaration files is not yet done
  • stylelint is working - it's not picking up any errors

I probably won't have time to deal with that in the next days, so if someone wants to do it, feel free.

@pbitkowski pbitkowski requested a review from thymikee May 18, 2019 16:02
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.

Awesome! Left some nits to address, and we're good to merge

src/loader.ts Show resolved Hide resolved
src/react/styled.ts Outdated Show resolved Hide resolved
src/react/styled.ts Show resolved Hide resolved
src/react/styled.ts Show resolved Hide resolved
src/server/collect.ts Show resolved Hide resolved
src/transform.ts Outdated Show resolved Hide resolved
src/transform.ts Outdated Show resolved Hide resolved
src/transform.ts Outdated Show resolved Hide resolved
zamotany and others added 3 commits May 20, 2019 11:57
Co-Authored-By: Michał Pierzchała <thymikee@gmail.com>
@zamotany zamotany requested a review from thymikee May 20, 2019 10:08
@Anber
Copy link
Collaborator

Anber commented May 27, 2019

Let's merge it all and release first alpha! :)

@satya164 satya164 merged commit 7b27cf9 into develop May 27, 2019
@satya164 satya164 deleted the chore/migrate-to-fs branch May 27, 2019 10:42
Anber pushed a commit that referenced this pull request Dec 26, 2019
Co-Authored-By: Michał Pierzchała <thymikee@gmail.com>
Co-Authored-By: Satyajit Sahoo <satyajit.happy@gmail.com>
Anber pushed a commit that referenced this pull request Jan 12, 2020
Co-Authored-By: Michał Pierzchała <thymikee@gmail.com>
Co-Authored-By: Satyajit Sahoo <satyajit.happy@gmail.com>
Anber pushed a commit that referenced this pull request Jan 14, 2020
Co-Authored-By: Michał Pierzchała <thymikee@gmail.com>
Co-Authored-By: Satyajit Sahoo <satyajit.happy@gmail.com>
Anber pushed a commit that referenced this pull request Feb 4, 2020
Co-Authored-By: Michał Pierzchała <thymikee@gmail.com>
Co-Authored-By: Satyajit Sahoo <satyajit.happy@gmail.com>
Anber pushed a commit that referenced this pull request Feb 9, 2020
Co-Authored-By: Michał Pierzchała <thymikee@gmail.com>
Co-Authored-By: Satyajit Sahoo <satyajit.happy@gmail.com>
Anber pushed a commit that referenced this pull request Feb 10, 2020
Co-Authored-By: Michał Pierzchała <thymikee@gmail.com>
Co-Authored-By: Satyajit Sahoo <satyajit.happy@gmail.com>
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.

6 participants