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

docs(language): removes "simply" to be more inclusive #342

Merged
merged 6 commits into from
Dec 17, 2021

Conversation

bmuenzenmeyer
Copy link
Contributor

👋 Hello!

As a maintainer of a 28 package monorepo, turborepo is a very interesting thing for my team to try out. Thanks for this (and @jaredpalmer for tsdx too)

In reading the documentation today, I bristled at a few places where the language assumed knowledge or trivialized a step that might be new to someone.

If this goes against a writing guide or established tone I am unaware of, feel free to carry on!

References:

Just by Brad Frost
Google Developer Docs

@vercel
Copy link

vercel bot commented Dec 17, 2021

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/vercel/turbo-site/5B7r3pe2KV3sSo43s5RXLVT7zgiQ
✅ Preview: https://turbo-site-git-fork-bmuenzenmeyer-main.vercel.sh

@jaredpalmer
Copy link
Contributor

Agreed. Great PR

@jaredpalmer jaredpalmer added the pr: automerge Kodiak will merge these automatically after checks pass label Dec 17, 2021
@jaredpalmer jaredpalmer merged commit ac2793a into vercel:main Dec 17, 2021
sokra pushed a commit that referenced this pull request Oct 25, 2022
- Removes the transform of `trait TraitName: TraitDep` into `trait TraitName: Into<TraitDep>`
- Removes the generated `lazy_static!` (was always a bit hard to read in the macro expansion, and `once_cell::Lazy` does the same without a macro)
- Generates unique names for all `impl`s, before implementing 2 traits with the same function name would lead to a compile error because they got the same func ident
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr: automerge Kodiak will merge these automatically after checks pass
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants