-
Notifications
You must be signed in to change notification settings - Fork 7
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
- Loading branch information
Showing
1 changed file
with
30 additions
and
0 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,30 @@ | ||
# TypeScript | ||
|
||
- avoid `any`; get rid of any existing `any` whenever you can so that we can enable `"strict": true` later on in `tsconfig.json` | ||
- define custom types for common data structures | ||
- don't worry about `interface` vs `type`, [both are fine](https://www.typescriptlang.org/docs/handbook/2/everyday-types.html#differences-between-type-aliases-and-interfaces) | ||
|
||
## Typescript and React/Next | ||
|
||
- use `React.FC<Props>` type for React components, e.g. `const MyComponent: React.FC<Props> = ({ ... }) => { ... };` | ||
- use `NextPage<Props>` for typing stuff in `src/pages/` | ||
- use generic versions of `GetServerSideProps<Props>` and `GetStaticProps<Props>` | ||
|
||
# React | ||
|
||
- create one file per one component (tiny helper components in the same file are fine) | ||
- if | ||
- name file identically to the component it describes (e.g. `const DisplayForecasts: React.FC<Props> = ...` in `DisplayForecasts.ts`) | ||
- use named export instead of default export for all React components | ||
- it's better for refactoring | ||
- and it plays well with `React.FC` typing | ||
|
||
# Styles | ||
|
||
- use [Tailwind](https://tailwindcss.com/) | ||
- avoid positioning styles in components, position elements from the outside (e.g. with [space-\*](https://tailwindcss.com/docs/space) or grid/flexbox) | ||
|
||
# General notes | ||
|
||
- use `const` instead of `let` whenever possible | ||
- set up [prettier](https://prettier.io/) to format code on save |