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

feat: Turborepo build-scripts rebase #661

Conversation

Handfish
Copy link
Contributor

@Handfish Handfish commented Apr 23, 2023

Apparently renaming a branch with the github web console closes the old pull request and thread. Just wanted to follow the convention of the repo by putting my username as the starting token of the branch.

Relevant pull requests
#657
#644

Suggestion to make a rebase to this branch
#644 (comment)

Also a heads up - commit 60e62eb broke some of the hardhat doc generation. Converting certain packages to modules causes some compilation errors. There is a thread here with some claims support is coming soon.

refactor: change docs package.json private to false

fix: build:docs

fix: testing suite required jest-environment-jsdom with newer versions
of jest

refactor: run tests through turbo

chore: turbo other tasks
chore: remove quotes around "private"

chore: run sort-package-json

chore: seems like vercel might need this path

chore: clean up changes, make install scripts hidden

chore: remove extra code from turbo file

chore: sleepy rebase
@Handfish Handfish requested review from holic and alvrs as code owners April 23, 2023 14:13
@vercel
Copy link

vercel bot commented Apr 23, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
mud ❌ Failed (Inspect) Apr 23, 2023 2:13pm

@holic
Copy link
Member

holic commented Apr 23, 2023

thank you! will take a closer look at this once #633 lands

Just wanted to follow the convention of the repo by putting my username as the starting token of the branch.

this pattern is useful when we're all operating out of the same repo to help determine who the branch was created by/owned by

@alvrs alvrs assigned alvrs and holic and unassigned alvrs Apr 24, 2023
@holic holic deleted the branch latticexyz:holic/build-scripts April 24, 2023 18:25
@holic holic closed this Apr 24, 2023
@holic
Copy link
Member

holic commented Apr 24, 2023

oops, this closed automatically since I merged the base branch

@holic holic mentioned this pull request Apr 24, 2023
@holic
Copy link
Member

holic commented Apr 24, 2023

continuing this over at #669

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants