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(examples): add example-with-angular #8140

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

codexshell
Copy link

@codexshell codexshell commented May 14, 2024

Description

A new example template utilizing Angular has been added to the examples directory.
This template serves as a reference implementation for projects looking to integrate Angular with the turbo repo ecosystem.

Screenshots

Web
Docs

Testing Instructions

@codexshell codexshell requested review from anthonyshew and a team as code owners May 14, 2024 21:43
@turbo-orchestrator turbo-orchestrator bot added area: ci area: examples Improvements or additions to examples needs: triage New issues get this label. Remove it after triage owned-by: turborepo owned-by: turbopack labels May 14, 2024
Copy link

vercel bot commented May 14, 2024

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

Name Status Preview Comments Updated (UTC)
examples-basic-web ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 17, 2024 5:15am
examples-designsystem-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 17, 2024 5:15am
examples-svelte-web ❌ Failed (Inspect) Jun 17, 2024 5:15am
4 Ignored Deployments
Name Status Preview Comments Updated (UTC)
examples-kitchensink-blog ⬜️ Ignored (Inspect) Visit Preview Jun 17, 2024 5:15am
examples-native-web ⬜️ Ignored (Inspect) Visit Preview Jun 17, 2024 5:15am
examples-tailwind-web ⬜️ Ignored (Inspect) Visit Preview Jun 17, 2024 5:15am
examples-vite-web ⬜️ Ignored (Inspect) Visit Preview Jun 17, 2024 5:15am

Copy link

vercel bot commented May 14, 2024

@codexshell is attempting to deploy a commit to the Vercel Team on Vercel.

A member of the Team first needs to authorize it.

Copy link

socket-security bot commented May 14, 2024

New and removed dependencies detected. Learn more about Socket for GitHub ↗︎

Package New capabilities Transitives Size Publisher
npm/@types/loader-runner@2.2.8 None 0 6.19 kB types
npm/@types/node@18.19.33 None +1 2.04 MB types
npm/@vercel/ncc@0.36.1 filesystem, unsafe 0 15.9 MB vercel-release-bot
npm/commander@11.0.0 environment, filesystem, shell 0 176 kB abetomo
npm/deepmerge@4.3.1 None 0 31.2 kB tehshrike
npm/esbuild@0.15.18 environment, filesystem, network, shell +20 173 MB evanw
npm/execa@7.2.0 environment +7 142 kB sindresorhus
npm/gradient-string@2.0.2 None +1 72.5 kB bokub
npm/inquirer@8.2.6 Transitive: environment, filesystem, shell +18 4.96 MB sboudrias
npm/inquirer@9.2.21 Transitive: environment +4 167 kB sboudrias
npm/jsonc-parser@3.2.1 None 0 210 kB vscode-bot
npm/lint-staged@13.3.0 Transitive: environment, filesystem, shell +24 1.66 MB okonet
npm/playwright-chromium@1.44.0 Transitive: environment, eval, filesystem, network, shell, unsafe +1 7.14 MB yurys
npm/plop@3.1.2 environment, unsafe Transitive: eval, filesystem, shell +63 5.55 MB crutchcorn
npm/prettier@2.8.8 environment, filesystem, unsafe 0 11.2 MB prettier-bot
npm/proxy-agent@6.4.0 network Transitive: environment, filesystem +16 1.2 MB tootallnate
npm/sharp@0.33.3 environment Transitive: eval +1 907 kB lovell
npm/tsx@4.10.2 Transitive: environment, filesystem, network, shell, unsafe +3 646 kB hirokiosame
npm/validate-npm-package-name@5.0.1 unsafe 0 7.77 kB npm-cli-ops
npm/vega@5.29.0 Transitive: eval, filesystem, network, shell +53 24 MB lhermann
npm/zod@3.23.8 None 0 667 kB colinmcd94

🚮 Removed packages: npm/@types/loader-runner@2.2.4, npm/@types/node@18.17.4, npm/@vercel/ncc@0.36.0, npm/commander@11.1.0, npm/deepmerge@4.2.2, npm/diff@5.1.0, npm/esbuild@0.15.6, npm/eslint@8.55.0, npm/execa@6.1.0, npm/fs-extra@11.1.1, npm/gradient-string@2.0.1, npm/inquirer@8.2.4, npm/inquirer@9.2.7, npm/jsonc-parser@3.2.0, npm/lint-staged@13.1.0, npm/mongoose@6.6.5, npm/node-fetch@2.6.11, npm/playwright-chromium@1.39.0, npm/plop@3.1.1, npm/prettier@2.8.7, npm/proxy-agent@6.2.2, npm/semver@7.5.0, npm/sharp@0.33.2, npm/tsx@4.7.2, npm/uuid@9.0.0, npm/validate-npm-package-name@5.0.0, npm/vega@5.27.0, npm/zod@3.22.4

View full report↗︎

@codexshell codexshell force-pushed the example/with-angular branch 3 times, most recently from 9b23d4d to cd0a6a6 Compare May 17, 2024 10:41
A new example template utilizing Angular has been added to the
'examples' directory. This template serves as a reference implementation
for projects looking to integrate Angular with the turbo repo ecosystem.
Copy link
Contributor

@arlyon arlyon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approving on the pack side

Copy link
Contributor

@anthonyshew anthonyshew left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome!

Initial change that I'll need to ask for is removing the changed in .github/actions. Taking a look at the example itself now...

Copy link
Contributor

@anthonyshew anthonyshew left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is my first time ever running an Angular application so I don't know if my expectations are incorrect. But if I make changes in the ui package, I don't see hot reloads in the application. Is that expected? Or should we be seeing live updates?

{
"$schema": "https://turbo.build/schema.json",
"globalDependencies": ["**/.env.*local"],
"pipeline": {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Run npx @turbo/codemod upgrade to upgrade to turbo@2. 🥳

"@repo/eslint-config": "workspace:*",
"eslint": "^8.57.0",
"prettier": "^2.8.8",
"turbo": "^1.13.3"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Run npx @turbo/codemod upgrade to upgrade to turbo@2. 🥳


/** @type {import("eslint").Linter.Config} */
module.exports = {
extends: ['eslint:recommended', 'prettier', 'eslint-config-turbo'],
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
extends: ['eslint:recommended', 'prettier', 'eslint-config-turbo'],
extends: ['eslint:recommended', 'prettier', 'turbo'],

This should be only turbo instead of eslint-config-turbo. This was a bug we had in the rest of the examples, so don't blame you for following what we did.

Make sure to run the turbo run lint command to make sure it runs successfully!

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All requested changes will be ready before Monday. Starting work on them

"@typescript-eslint/eslint-plugin": "^7.9.0",
"@typescript-eslint/parser": "^7.9.0",
"eslint-config-prettier": "^9.1.0",
"eslint-config-turbo": "^1.13.3",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Upgrade to ^2.0.4.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: ci area: examples Improvements or additions to examples needs: triage New issues get this label. Remove it after triage owned-by: turbopack owned-by: turborepo
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants