Skip to content
This repository has been archived by the owner on Jan 13, 2023. It is now read-only.

Added the ability to generate/scaffold a React.FunctionComponent #300

Merged
merged 11 commits into from
Jan 16, 2020

Conversation

nirre7
Copy link
Contributor

@nirre7 nirre7 commented Dec 29, 2019

  • Added an external styles file to be generated when generating a component.
  • Added minor fix for the prettier commands when running zsh (mac osx catalina)
  • Fixed typ in readme

@nirre7
Copy link
Contributor Author

nirre7 commented Jan 1, 2020

Oh, so that broke the integration tests in generators-integration.test.ts => since I've added a choice now.
I'll update the tests.

@nirre7
Copy link
Contributor Author

nirre7 commented Jan 1, 2020

Looks OK now.

Copy link
Member

@leonskim leonskim left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@jamonholmgren
Copy link
Member

Thanks @nirre7 ! Would you mind adding a "generators" section to the readme? I feel like we lack some documentation on generators and would love to include that with this PR.

@nirre7
Copy link
Contributor Author

nirre7 commented Jan 3, 2020

Yepp, will add it later today.
Feel free to correct me etc since English is not my first language 😅

@nirre7
Copy link
Contributor Author

nirre7 commented Jan 7, 2020

@jamonholmgren updated the readme with information regarding generators.

@jamonholmgren jamonholmgren merged commit a63bc01 into infinitered:master Jan 16, 2020
@jamonholmgren
Copy link
Member

That's excellent! Thanks @nirre7 !!!

infinitered-circleci pushed a commit that referenced this pull request Jan 16, 2020
# [4.13.0](v4.12.0...v4.13.0) (2020-01-16)

### Features

* **generators:** Added a React.FunctionComponent generator ([#300](#300) by [@nirre7](https://github.com/nirre7)) ([a63bc01](a63bc01))
@infinitered-circleci
Copy link

🎉 This PR is included in version 4.13.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Successfully merging this pull request may close these issues.

5 participants