Skip to content

Cleanup#6

Merged
soedirgo merged 10 commits intomainfrom
chore/cleanup
Mar 30, 2021
Merged

Cleanup#6
soedirgo merged 10 commits intomainfrom
chore/cleanup

Conversation

@soedirgo
Copy link
Copy Markdown
Member

What kind of change does this PR introduce?

Chore.

What is the new behavior?

Sorry about the big PR! A bit difficult to split this up 😅

Changes are split into multiple commits for ease of review:

chore: remove CONTRIBUTING.md

We already have one in supabase/.github

chore: update misc. project stuff

  • set up husky, prettier, jest
  • remove tslint (it's deprecated)
  • set up tsconfig.json

refactor: remove bin/supabase

  • for dev, run the CLI directly with ts-node
  • for prod, compile to dist with npm run build and run npm run start

chore: remove some unused stuff

  • types.ts (currently empty)
  • extensions (unused)

refactor: rename dev -> start

refactor: spruce up the CLI a bit

  • clean up code
  • use spinners for init/start/stop
  • spruce up --help (following vercel's style mostly)

Screenshot 2021-03-28 at 12 35 13 PM

test: update test

chore(build): set up GitHub Actions

Need secrets: GITHUB_TOKEN & NPM_TOKEN

We already have one in supabase/.github
- set up husky, prettier, jest
- remove tslint (it's deprecated)
- set up tsconfig.json
- for `dev`, run the CLI directly with `ts-node`
- for `prod`, compile to `dist` and run `node dist/cli.js`
- `types.ts` (currently empty)
- `extensions` (unused)
@soedirgo soedirgo requested a review from kiwicopple March 28, 2021 04:53
- clean up code
- use spinners for `init`/`start`/`stop`
- spruce up `--help` (following vercel's style mostly)
Need tokens: `GITHUB_TOKEN` & `NPM_TOKEN`
@soedirgo soedirgo merged commit c90ec79 into main Mar 30, 2021
@soedirgo soedirgo deleted the chore/cleanup branch March 30, 2021 02:35
dumko2001 pushed a commit to dumko2001/cli that referenced this pull request Mar 15, 2026
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.

1 participant