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

Cache tsc with turbo #258

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

Conversation

PatrykWalach
Copy link
Collaborator

No description provided.

@PatrykWalach
Copy link
Collaborator Author

So I need to figure out if using cache multiple times is merging the results or overriding them 🤔

      - name: Cache turbo
        uses: actions/cache@v4
        with:
          path: .turbo
          key: ${{ runner.os }}-turbo-${{ github.sha }}
          restore-keys: |
            ${{ runner.os }}-turbo-

@PatrykWalach
Copy link
Collaborator Author

Ok, these jobs are sequential so there are no overrides

Copy link
Collaborator

@rbalicki2 rbalicki2 left a comment

Choose a reason for hiding this comment

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

rebase and lgtm

"$schema": "https://turbo.build/schema.json",
"tasks": {
"compile": {
"dependsOn": ["^compile"],
Copy link
Collaborator

Choose a reason for hiding this comment

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

is this right? Does this mean that we're going to require the Rust compiler to be built before we build the JS?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

The arrow ^ has the same meaning as pnpm -r - dependencies of each package need to be compiled first

Copy link
Collaborator

Choose a reason for hiding this comment

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

Okay TIL

{
"$schema": "https://turbo.build/schema.json",
"tasks": {
"compile": {
Copy link
Collaborator

Choose a reason for hiding this comment

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

can we name this something like compileTypescript or something?

(Perhaps we should also rename the other ones to indicate that they refer to the Rust side)

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.

2 participants