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

⬆️ Update ts-node to v10.9.2 #2773

Merged
merged 3 commits into from
Feb 7, 2024
Merged

⬆️ Update ts-node to v10.9.2 #2773

merged 3 commits into from
Feb 7, 2024

Conversation

renovate[bot]
Copy link
Contributor

@renovate renovate bot commented Dec 11, 2023

Mend Renovate

This PR contains the following updates:

Package Change Age Adoption Passing Confidence
ts-node (source) 10.9.1 -> 10.9.2 age adoption passing confidence

Release Notes

TypeStrong/ts-node (ts-node)

v10.9.2: Fix tsconfig.json file not found

Compare Source

Fixed


Configuration

📅 Schedule: Branch creation - "before 3am every weekday" in timezone Europe/Berlin, Automerge - At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about this update again.


  • If you want to rebase/retry this PR, check this box

This PR has been generated by Mend Renovate. View repository job log here.

@renovate renovate bot added the dependencies Pull requests that update a dependency file label Dec 11, 2023
@renovate renovate bot requested a review from Suchty112 December 11, 2023 01:37
@renovate renovate bot force-pushed the renovate/ts-node-10.x branch 2 times, most recently from 615de19 to 889c1ae Compare December 11, 2023 06:01
@Suchty112 Suchty112 requested review from jxn-30 and removed request for Suchty112 December 11, 2023 06:01
@Suchty112 Suchty112 assigned jxn-30 and unassigned Suchty112 Dec 11, 2023
@renovate renovate bot force-pushed the renovate/ts-node-10.x branch 19 times, most recently from 179dc74 to b52dbf1 Compare December 18, 2023 04:05
@renovate renovate bot force-pushed the renovate/ts-node-10.x branch 2 times, most recently from 0e6c448 to 1ccff28 Compare December 18, 2023 14:16
@renovate renovate bot force-pushed the renovate/ts-node-10.x branch 15 times, most recently from 48d5e1a to 36bcd59 Compare February 7, 2024 21:58
@renovate renovate bot force-pushed the renovate/ts-node-10.x branch from 36bcd59 to 8e60b23 Compare February 7, 2024 21:59
@kdev kdev assigned kdev and unassigned jxn-30 Feb 7, 2024
@kdev
Copy link
Member

kdev commented Feb 7, 2024

Funnily this error is only reproducible on github runners and not on any ubutu-based systems

@jxn-30
Copy link
Member

jxn-30 commented Feb 7, 2024

Funnily this error is only reproducible on github runners and not on any ubutu-based systems

I don't even understand why it fails on the PR-build but succeeds on Test-Build. ubuntu-latest is the same as ubuntu-22.04 so it's even the same OS. Only the context is different? idk tbh

@kdev
Copy link
Member

kdev commented Feb 7, 2024

It's def a perms error but I dont understand how a docs build would trigger that. I have one more idea which I will implement but idk if that will work

@jxn-30
Copy link
Member

jxn-30 commented Feb 7, 2024

Probably because we're doing weird stuff with child processes (and idk why we're doing it that way tbh. Seems a little weird to me)

run(file: Util, ...args: string[]) {
const result = execFileSync(
'./node_modules/.bin/ts-node',
[path.join(utilsPath, `${file}.ts`), ...args],
{
env: process.env,
}
).toString();
if (result) console.log(result);
},

@kdev
Copy link
Member

kdev commented Feb 7, 2024

This is the generated code found while building.

var childProcess_default = (utilsPath) => ({
  run(file, ...args) {
    const result = execFileSync(
      "./node_modules/.bin/ts-node",
      [path.join(utilsPath, `${file}.ts`), ...args],
      {
        env: process.env
      }
    ).toString();
    if (result)
      console.log(result);
  }
});

@kdev
Copy link
Member

kdev commented Feb 7, 2024

The issue was with the github actions cache and it's permissions. Seems to be fixed for now

@kdev kdev merged commit b6e9169 into dev Feb 7, 2024
8 checks passed
@jxn-30 jxn-30 deleted the renovate/ts-node-10.x branch February 7, 2024 23:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants