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.

Sign up for GitHub

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

Jump to bottom

Update developer experience #74

Merged
merged 31 commits into from
Aug 21, 2024
Merged

Update developer experience #74

merged 31 commits into from
Aug 21, 2024

Conversation

emma-sg
Copy link
Member

@emma-sg emma-sg commented Aug 20, 2024 •
edited
Loading

  • Adds Prettier & updates eslint & configs
  • Removes checked-in dist folder
  • Switches from npm to yarn
  • Updates a few types and things so that tsc completes without errors
  • Fixes some slightly broken tests

@emma-sg
remove dist
90b609c
@emma-sg
ignore all of dist
99c529d
@emma-sg
add script to build if not in git repo
4be185e
@emma-sg
swap to esm
32cf8a5
@emma-sg
run build on prepare directly
ee07a16
@emma-sg
remove unused script
1204bf5
@emma-sg
swap to yarn
1f335b9
@emma-sg
import eslint config from rwp
68459ea
@emma-sg
first pass at fixes
9d5aa27
@emma-sg
second pass
f1b0746
@emma-sg
format everything with prettier
c69a7b0
@emma-sg
third eslint pass
21b3db1
@emma-sg
format again
6a43737
@emma-sg
fourth pass
f337429
@emma-sg
resolve type issues
cb6dbec
@emma-sg
rebuild yarn lockfile from npm package-lock.json
8557a27
@emma-sg
reformat with new prettier
2fc8223
@emma-sg
fix remaining eslint errors
a17ad20
@emma-sg
lint & format tests as well
2154018
@emma-sg
autofix tests
5fbf4ba
@emma-sg
format tests
9866e37
@emma-sg
revert webstreams changes
b686b78
@emma-sg
separate configs for tests
8a4009a
@emma-sg
fix up tests
623d2b9
@emma-sg
fix tests
91c2689
@emma-sg
switch ci to yarn
4b0a140
@emma-sg
drop node 16 in ci and add 22
efb00d0
@emma-sg
skip checking formatting in ci
f03c587
@emma-sg
re-add script that checks git status before building
dd7a914
@emma-sg emma-sg changed the title Remove checked-in dist folder Update developer experience Aug 20, 2024
@emma-sg emma-sg marked this pull request as ready for review August 20, 2024 19:44
@emma-sg emma-sg requested a review from ikreymer August 20, 2024 19:44
@ikreymer ikreymer merged commit f0ab495 into main Aug 21, 2024
6 checks passed
@ikreymer ikreymer deleted the remove-checked-in-dist branch August 21, 2024 00:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ikreymer ikreymer Awaiting requested review from ikreymer

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@emma-sg @ikreymer