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

Installing npm dependency from public GitHub repository fails #214

Closed
TimDaub opened this issue Nov 24, 2020 · 28 comments
Closed

Installing npm dependency from public GitHub repository fails #214

TimDaub opened this issue Nov 24, 2020 · 28 comments

Comments

@TimDaub
Copy link

TimDaub commented Nov 24, 2020

In one of my projects I use simple-caldav which contains the following line in its package.json:

dependencies: {
  "ical.js": "github:TimDaub/ical.js#feat/detect-module-mode-build",
  ...
}

It points to a branch here. I've submitted a PR to the upstream repo, but it seems they're not having much time for maintenance.

Anyways, my GH action in the project that has simple-caldav as a dependency looks like this

# This workflow will do a clean install of node dependencies, build the source code and run tests across different versions of node
# For more information see: https://help.github.com/actions/language-and-framework-guides/using-nodejs-with-github-actions

name: Node.js CI

on:
  push:
    branches: [ master ]
  pull_request:
    branches: [ master ]

jobs:
  build:

    runs-on: ubuntu-latest

    strategy:
      matrix:
        node-version: [10.x, 12.x, 14.x]

    steps:
    - uses: actions/checkout@v2
    - name: Use Node.js ${{ matrix.node-version }}
      uses: actions/setup-node@v1
      with:
        node-version: ${{ matrix.node-version }}
    - run: npm ci
    - run: npm test

However, when it runs npm ci, it fails like this

npm ERR! Error while executing:
npm ERR! /usr/bin/git ls-remote -h -t ssh://git@github.com/TimDaub/ical.js.git
npm ERR! 
npm ERR! Warning: Permanently added the RSA host key for IP address '140.82.113.4' to the list of known hosts.
npm ERR! git@github.com: Permission denied (publickey).
npm ERR! fatal: Could not read from remote repository.
npm ERR! 
npm ERR! Please make sure you have the correct access rights
npm ERR! and the repository exists.
npm ERR! 
npm ERR! exited with error code: 128
@Antonio-Laguna
Copy link

@TimDaub did you find any workarounds?

@TimDaub
Copy link
Author

TimDaub commented Dec 3, 2020

Nope, I'm quite confused by this problem. E.g. why does it say git ls-remote? Is npm using git internally?
I could imagine another notation within package.dependencies could make a difference. But I haven't tested that yet.

@Antonio-Laguna
Copy link

Thought the same and even tried that locally and works fine locally

@Antonio-Laguna
Copy link

@TimDaub I ended up releasing the forks as my own libraries publicly scoped to NPM since a project was breaking deployments because of this issue. It's a workaround but it's something :)

@carlocorradini
Copy link

@TimDaub

Same issue here.

A very annoying problem...

Have you find any solution?

@TimDaub
Copy link
Author

TimDaub commented Dec 10, 2020

ended up releasing the forks as my own libraries publicly scoped to NPM since a project was breaking deployments because of this issue. It's a workaround but it's something :)

Unfortunately, it's not an option I have.

@TimDaub
Copy link
Author

TimDaub commented Dec 10, 2020

What ended up fixing it for me is adding the unknown host in my ssh config before npm ci:

...
- run: mkdir -p $HOME/.ssh/ && echo "140.82.113.4" >> $HOME/.ssh/known_hosts
- run: npm ci
...

It's far from perfect, but works well as a work around for now. Additionally, disabling ssh's key checking via config may be an option too. I prefer to go with this more narrow solution.

Edit: Turns out this won't work all the time as the IPs that the package is requested from change

  • 140.82.113.*
  • 140.82.112.*
  • 140.82.114.*

I've tested adding ranges and ssh-keyscan, but so far I wasn't successful.

Edit2:

I think I finally ended up solving it for good. This is what you'll have to do:

  1. Backup your current RSA keypair at ~/.ssh
  2. Generate a new RSA keypair on your system ssh-keygen -t rsa -C "your_email@example.com". Ideally don't overwrite your existing keypair at ~/.ssh by entering a custom path.
  3. Take the contents of the generated *.pub key and add it to your SSH keys in your GitHub account settings
  4. In your repo that has the action, navigate to Settings > Secrets and add SSH_PRIVATE_KEY the contents of the private key file that was generated
  5. Then in your repo's workflow file, add the following before -run: npm ci
...
- uses: webfactory/ssh-agent@v0.4.1
   with:
     ssh-private-key: ${{ secrets.SSH_PRIVATE_KEY }}
- run: npm ci
...

For more details, check https://github.com/webfactory/ssh-agent

@Siilwyn
Copy link

Siilwyn commented Feb 5, 2021

This is still an issue... An alternative solution is replacing the resolved url in the package lock file, for example:
git+ssh://git@github.com/zspecza/common-tags.git#946fcbf8cfc1a14c2183ef5a81b23727a2b531e3 becomes: git+https://git@github.com/zspecza/common-tags.git#946fcbf8cfc1a14c2183ef5a81b23727a2b531e3

@natekwilson
Copy link

Having the same issue, but within my workflows' docker build step. There has to be easier ways to disseminate SSH agent keys/known hosts info to different contexts when SSH-git actions are so commonplace

@daonb
Copy link

daonb commented Mar 14, 2021

This is still a bug. I'm now going to implement the workaround @TimDaub suggested and it'll probably work but still... It hit me just as I released an version and I had no time to fiddle with private keys so github action can read public github URLs. It was a bummer but I released a version whose README starts with "test: failing"

Please fix this as the attached screenshot does not compute.

Screenshot

@akaltar
Copy link

akaltar commented Mar 20, 2021

For me this issue started occurring when I tried to switch from npm install to npm ci so for some of you switching to npm install may be another workaround.

@Xunnamius
Copy link

Xunnamius commented Mar 31, 2021

I fixed this in my workflows by adding an extra step after the actions/checkout@v2 (with persist-credentials: false) step:

      - name: Checkout
        uses: actions/checkout@v2
        with:
          persist-credentials: false

      - name: Reconfigure git to use HTTP authentication
        run: >
          git config --global url."https://github.com/".insteadOf
          ssh://git@github.com/

Changing from SSH to HTTP makes everything work across all workflows using npm ci (which has several benefits over npm install). If you need to authenticate, use a PAT instead of SSH:

git config --global url."https://${{ secrets.GH_TOKEN }}@github.com/".insteadOf ssh://git@github.com/

alsakhaev added a commit to dapplets/dapplet-extension that referenced this issue Apr 6, 2021
@TimDaub
Copy link
Author

TimDaub commented Apr 9, 2021

An update on my earlier workaround in this thread. A problem that I've discovered is that according to GitHub settings:

Secrets are not passed to workflows that are triggered by a pull request from a fork.

Hence, it becomes useless when trying to collaborate with others.

@wallind
Copy link

wallind commented May 17, 2021

I fixed this in my workflows by adding an extra step after the actions/checkout@v2 (with persist-credentials: false) step:

      - name: Checkout
        uses: actions/checkout@v2
        with:
          persist-credentials: false

      - name: Reconfigure git to use HTTP authentication
        run: >
          git config --global url."https://github.com/".insteadOf
          ssh://git@github.com/

Changing from SSH to HTTP makes everything work across all workflows using npm ci (which has several benefits over npm install). If you need to authenticate, use a PAT instead of SSH:

git config --global url."https://${{ secrets.GH_TOKEN }}@github.com/".insteadOf ssh://git@github.com/

this did fix my problem so thank you for that. Out of curiosity though is there any security lost by doing this? I don't care enough to not use this fix for the project I need it on but I am left wondering.

ali-bahjati referenced this issue in pyth-network/pyth-js Jul 15, 2022
* Use node 16 for publishing

The reason is github actions issue on running `npm ci` that requires using git. [https://github.com/actions/setup-node/issues/214](the issue that discusses this problem)

* Add build action

* Fix build

* remove node 12

* Fix evm build on node 14

* Add node 18
@andreyvolokitin
Copy link

@AndrewJHart the only thing I don't understand is how the github account is relevant here. If CI can be triggered without any action from this specific account — why it even considers this account's SSH key? The SSH connection is between github action runner and github repo, I don't even understand where/when gihub account could become relevant here, but it indeed works (CI is triggered by pushing git tags by my account, I wonder if it will stop working if different account will be pushing tags)

CareyJWilliams added a commit to ChoicescriptIDE/monaco-editor that referenced this issue Sep 3, 2022
When using ssh even public repositories require
authentication with npm ci.

Solution from:
actions/setup-node#214 (comment)
@jdmarshall
Copy link

jdmarshall commented Sep 22, 2022

I haven't had any luck getting the workarounds to succeed for this.

There are several very worrying aspects of this bug.

  1. npm ci returns a success status when an error has clearly occurred
  2. there are no hints in --verbose about how or if it's trying to fetch the files

yaacovCR added a commit to yaacovCR/stitch that referenced this issue Jan 12, 2023
yaacovCR added a commit to yaacovCR/stitch that referenced this issue Jan 18, 2023
yaacovCR added a commit to yaacovCR/stitch that referenced this issue Jan 18, 2023
On v14, node does not seem to support npm access to Github via Github Actions -- or, at least not easily.

This can be reverted once we target a particular alpha of graphql that is published to npm.

See: actions/setup-node#214
yaacovCR added a commit to yaacovCR/stitch that referenced this issue Jan 18, 2023
On v14, node does not seem to support npm access to Github via Github Actions -- or, at least not easily.

This can be reverted once we target a particular alpha of graphql that is published to npm.

See: actions/setup-node#214
yaacovCR added a commit to yaacovCR/stitch that referenced this issue Jan 20, 2023
On v14, node does not seem to support npm access to Github via Github Actions -- or, at least not easily.

This can be reverted once we target a particular alpha of graphql that is published to npm.

See: actions/setup-node#214
yaacovCR added a commit to yaacovCR/stitch that referenced this issue Jan 20, 2023
On v14, node does not seem to support npm access to Github via Github Actions -- or, at least not easily.

This can be reverted once we target a particular alpha of graphql that is published to npm.

See: actions/setup-node#214
yaacovCR added a commit to yaacovCR/stitch that referenced this issue Jan 20, 2023
On v14, node does not seem to support npm access to Github via Github Actions -- or, at least not easily.

This can be reverted once we target a particular alpha of graphql that is published to npm.

See: actions/setup-node#214
yaacovCR added a commit to yaacovCR/stitch that referenced this issue Jan 20, 2023
On v14, node does not seem to support npm access to Github via Github Actions -- or, at least not easily.

This can be reverted once we target a particular alpha of graphql that is published to npm.

See: actions/setup-node#214
stdavis added a commit to agrc/roadkill-mobile that referenced this issue Feb 6, 2023
stdavis added a commit to agrc/roadkill-mobile that referenced this issue Feb 7, 2023
sounisi5011 added a commit to sounisi5011/package-version-git-tag that referenced this issue Mar 8, 2023
…tions

npm uses the `ssh:` protocol when installing npm packages with the `github:` protocol specified.
However, on GitHub Actions, the dependency installation fails with the following error:

    npm ERR! Error while executing:
    npm ERR! /usr/bin/git ls-remote -h -t ssh://git@github.com/sounisi5011/readme-generator.git
    npm ERR!
    npm ERR! git@github.com: Permission denied (publickey).
    npm ERR! fatal: Could not read from remote repository.
    npm ERR!
    npm ERR! Please make sure you have the correct access rights
    npm ERR! and the repository exists.
    npm ERR!
    npm ERR! exited with error code: 128

To resolve this, "Reconfigure git to use HTTP authentication" steps have been added.

see https://stackoverflow.com/a/69634516
see actions/setup-node#214 (comment)
sounisi5011 added a commit to sounisi5011/package-version-git-tag that referenced this issue Mar 12, 2023
* ⬆️ Update dependency husky to 8.0.3

* ➕ Migrating from husky and lint-staged to lefthook

    lefthook is made using the Golang, so it works with older Node.js.

* Use the `npm run ...` command instead of the `npx run-s ...` command

    Executing `npx run-s ...` command fails and never exits.
    This command is the same as the `npm run ...` command except that the glob pattern cannot be used, so use the `npm run ...` command.

* 🔧 Parallel execution of automatic formatting

    Lefthook can set the process to be "executed after all the parallel execution processes are finished".
    see evilmartians/lefthook#66 (comment)

* 🔧 Use npm v7 in this project

    npm v7 is the latest version of npm that supports Node.js v10.
    npm v8 and later no longer support Node.js 10.
    Also, the latest npm overwrites `package-lock.json` files with forward incompatible `lockfileVersion: 3`.

    So we specified the version of npm to be used for this project.
    If the contributor is using Node.js 14.19 or later, the version of npm CLI should be automatically switched by Corepack.

* 👽 Regenerate package-lock.json

* 📌 Fix dependency version: pin "cac" package version to 6.6.1

    The format of help messages has changed since cac 6.7.0.
    see cacjs/cac@e565b2a

* 🔧 Regenerate package-lock.json

* 💚 Using npm packages installed by the `github:` protocol on GitHub Actions

    npm uses the `ssh:` protocol when installing npm packages with the `github:` protocol specified.
    However, on GitHub Actions, the dependency installation fails with the following error:

        npm ERR! Error while executing:
        npm ERR! /usr/bin/git ls-remote -h -t ssh://git@github.com/sounisi5011/readme-generator.git
        npm ERR!
        npm ERR! git@github.com: Permission denied (publickey).
        npm ERR! fatal: Could not read from remote repository.
        npm ERR!
        npm ERR! Please make sure you have the correct access rights
        npm ERR! and the repository exists.
        npm ERR!
        npm ERR! exited with error code: 128

    To resolve this, "Reconfigure git to use HTTP authentication" steps have been added.

    see https://stackoverflow.com/a/69634516
    see actions/setup-node#214 (comment)

* 💚 Use bash in "Reconfigure git to use HTTP authentication" steps

    In PowerShell, backslash cannot be used for line breaks in commands.

* 💚 Enable Corepack on GitHub actions

* 💚 Fix the location of comments in the .github/workflows/ci.yaml file

    Prettier's auto-formatting changed the location of a comment to an unintended location.

* 💚 Manage npm with Corepack

    `corepack enable npm` command does not activate npm shims.

* 💚 Fix "Enable Corepack (Automatically setup a package manager for Node.js)" jobs

    + Execute both `corepack enable` and `corepack enable npm` commands

        `corepack enable` command does not activate npm shims.
        However, just executing the `corepack enable npm` command does not prevent the use of yarn or pnpm.

    + If Corepack is not builtin to Node.js, install it

        Corepack is already installed on GitHub Actions.
        But it does not manage npm versions.

* ✅ Fix the test code

    + Replace the `npx --no-install ...` command with the `npm exec --no ...` command

        In npm v7, the npx command throws the following error:

            Command failed: .../bin/node .../npm/7.24.2/bin/npx-cli.js config get tag-version-prefix
            npm ERR! could not determine executable to run

    + In tests using yarn, set `"packageManager": "yarn@1.22.19"` within package.json

        If Corepack is enabled, the `yarn` command cannot be used.
        This problem can be resolved by specifying the `packageManager` field for package.json in the temporary directory.

* 💚 Avoid ENOTEMPTY errors thrown by Corepack

* 💚 Explore why built-in Corepack cannot be detected on Windows

* 💚 Explore why built-in Corepack cannot be detected on Windows

* ⏪ Revert "💚 Explore why built-in Corepack cannot be detected on Windows"

    This reverts commit 98f4ad8.

* 💚 Fix conditional expression to detect builtin Corepack

    On Windows, the `npm ls --global corepack` command cannot be used to detect the builtin Corepack.

* 💚 Find out why builtin Corepack detection fails

* 💚 Find out why builtin Corepack detection fails

* 💚 Find out why builtin Corepack detection fails

* 💚 Find out why builtin Corepack detection fails

* 💚 Find out why builtin Corepack detection fails

* 💚 Find out why builtin Corepack detection fails

* 💚 Fix conditional expression to detect builtin Corepack

    `yarn --version | grep ...`
    ↓
    `{ yarn --version || true; } | grep ...`

* ⏪ Revert "💚 Find out why builtin Corepack detection fails"

    This reverts commit 1d9f512.

* 💚 Fix Corepack installation failure on Windows

    On Windows, a conflict occurs with yarn already installed and an EEXIST error is thrown.

        npm ERR! code EEXIST
        npm ERR! path C:\npm\prefix\node_modules\corepack\dist\yarn.js
        npm ERR! dest C:\npm\prefix\yarn
        npm ERR! EEXIST: file already exists, cmd shim 'C:\npm\prefix\node_modules\corepack\dist\yarn.js' -> 'C:\npm\prefix\yarn'
        npm ERR! File exists: C:\npm\prefix\yarn
        npm ERR! Remove the existing file and try again, or run npm
        npm ERR! with --force to overwrite files recklessly.

    To avoid this, specify the "--force" option.

* 💚 Modify the shell script for the "Enable Corepack" step

* 💚 Fix problem with npm version not changing on Windows

* 💚 If Corepack is not available, manually update npm

    It is very difficult to use Corepack with old Node.js.
    All that is required is npm as per the specified version, so update it manually.

* 💚 If Corepack is not available, disable Corepack

    Corepack is embedded within GitHub Actions.
    However, if this is still enabled, it is not possible to run the yarn command in older Node.js.

* 💚 Modify the print content of the "Enable Corepack" step

* 💚 Add comments for future contributors within the "Enable Corepack" steps

* 💚 If yarn is not available, install it

* ⬆️ Update dependency ava to 3.15.0

* ⬆️ Update dependency typescript to 4.9.5

* ⬆️ Update dependencies: eslint packages

* ⬆️ Update dependency node-git-server to 1.0.0

* ⬆️ Update dependency ts-node to 10.9.1

* 🔧 Use `@typescript-eslint/recommended-requiring-type-checking` and `@typescript-eslint/strict` in ESLint config

* ♻️ Fix code following new TypeScript and ESLint reports

* 🔧 Fix patch file to make `check-peer-deps@1.1.3` detect peerDependenciesMeta

    Fixed `check-peer-deps@1.1.3` to ignore optional peerDependencies.

* ✅ Display npm version during test run

    The Windows environment on GitHub Actions seems to be using the old npm.

* 🔧 Increase test timeout to 5 minutes

* ✅ Use `execa` instead of `cross-spawn`

* 💚 [Debug] Check npm version in different shells

    Maybe running the `npm install --global npm` command in bash may not use the latest npm in other shells.

* 💚 [Debug] Check npm version using execa

    Maybe execa does not resolve the command path correctly.

* 💚 [Debug] Display the path where the npm command exists

* 💚 On Windows, update npm using `npm-windows-upgrade`

    If using Node.js 14 or lower on Windows, the `npm install --global npm` command will not update npm correctly.
    see https://stackoverflow.com/a/31520672

* 💚 Run `npm-windows-upgrade` using the `npx` command

    Run `npm-windows-upgrade` only for Node.js 14 without Corepack installed.
    The builtin npm for Node.js 14 is version 6, so the `npx` command should work.
    Also, anything less than Node.js 14 is not supported for this project, so it does not need to be considered.

* 💚 [Debug] Try `npm install --global ...` command instead of `npm-windows-upgrade`

    Unit tests should fail.
    Perhaps the environment run by npm-scripts will refer to npm before the update.

* ⏪ Revert "💚 [Debug] Try `npm install --global ...` command instead of `npm-windows-upgrade`"

    This reverts commit 028ed18.

* ⏪ Revert "💚 [Debug] Display the path where the npm command exists"

    This reverts commit 59ae401.

* ⏪ Revert "💚 [Debug] Check npm version using execa"

    This reverts commit fe976f0.

* ⏪ Revert "💚 [Debug] Check npm version in different shells"

    This reverts commit dceda8c.

* ⏪ Revert "✅ Display npm version during test run"

    This reverts commit d77c9a3.

* 💚 Move "Update to the specified npm (on Windows)" steps before "Update to the specified npm" steps

    The conditional expression is more readable if "Update to the specified npm (on Windows)" is executed first.

    ```
    env.required-npm-version && ( runner.os != 'Windows' || ! steps.is-nodejs-14-or-below.outputs.satisfies || steps.is-nodejs-14-or-below.outputs.satisfies == 'false' )
    ```

    ↓

    ```
    env.required-npm-version && ( runner.os == 'Windows' && steps.is-nodejs-14-or-below.outputs.satisfies   && steps.is-nodejs-14-or-below.outputs.satisfies != 'false' )
    ```

* 💚 Rename the steps for install the specified npm

    Renamed the steps so that the difference in steps can be seen by what is used to install npm, rather than which OS to install npm for.

* 💚 [Debug] Display npm version in npm-scripts

* 💚 [Debug] Try `npm install --global ...` command instead of `npm-windows-upgrade`

* ⏪ Revert "💚 [Debug] Display npm version in npm-scripts"

    This reverts commit 69bf65a.

* 💚 [Debug] Display npm versions in various Node.js

* 💚 [Debug] Strictly check the npm version

* ⏪ Revert "💚 [Debug] Try `npm install --global ...` command instead of `npm-windows-upgrade`"

    This reverts commit 20e7387.

* 💚 [Debug] Try to update npm with `npm install --global ...` command in all Node.js

    Maybe even with Node.js 16 or higher, npm cannot be updated correctly without using `npm-windows-upgrade`.

* ⏪ Revert "💚 [Debug] Try to update npm with `npm install --global ...` command in all Node.js"

    This reverts commit 285f2bc.

* 💚 On Windows, use the `npx npm-windows-upgrade` command instead of the `npm install --global npm` command in all Node.js

    The referenced answer says that the "npm install --global npm" command is available for Node.js v16 or later.

    > ## What method should I choose to update NPM?
    >
    > + Node.js v16 or higher?
    >     - `npm install -g npm`
    > + Node.js v14 or below?
    >     - Consider updating to latest LTS release of Node.js
    >     - `npm-windows-upgrade`
    >
    > https://stackoverflow.com/a/31520672

    However, when I tried it, the npm overwrite failed for all Node.js (v14 to v19) on Windows.

* 💚 [Debug] Try to overwrite npm without Corepack

* ⏪ Revert "💚 [Debug] Try to overwrite npm without Corepack"

    This reverts commit f7af188.

* ⏪ Revert "💚 [Debug] Display npm versions in various Node.js" and "💚 [Debug] Strictly check the npm version"

    This reverts commit fb0ba0b and daeadc6.

* ⬆️ Update dependency can-npm-publish to 1.3.6

* ⬆️ Update dependency del-cli to 5.0.0

* ⬆️ Update dependency escape-string-regexp to 4.0.0

* ⬆️ Update dependency patch-package to 6.5.1

* ⬆️ Update dependency git-branch-is to 4.0.0

* ⬆️ Update dependency prettier to 2.8.4

* ⬆️ Update dependency prettier-package-json to 2.8.0

* ⬆️ Update dependency sort-package-json to 2.4.1

* ⬆️ Update dependency eslint to 8.36.0

* 📝 Update CHANGELOG

* 🔧 Fix dependency version range: Use SemVer range for "cac" package version

    Versions in `dependencies` should not be pinned.

* 🔧 Regenerate package-lock.json

* ⬆️ Update dependency escape-string-regexp to 5.0.0

    Perhaps this is not available in Node.js 14.0.0.

* ⬇️ Downgrade dependency escape-string-regexp to 4.0.0

    We will not update to v5 because we cannot yet use ESM for unit testing on this project.

* ⬆️ Update dependency execa to 7.1.0

    Perhaps this is not available in Node.js 14.0.0.

* ⬇️ Downgrade dependency execa to 5.1.1

    We will not use v6 or later because the ESM cannot yet be used for unit testing in this project.

* ⬆️ Update dependency ava to 5.2.0

    Perhaps this is not available in Node.js 14.0.0.

* ⬇️ Downgrade dependency ava to 4.3.3

    This project supports Node.js 14.0.0 so we will not update to v5 which is not available in Node.js 14.0.0.

* ⬇️ Downgrade dependency ava to 3.15.0

    This project supports Node.js 14.0.0 so we will not update to v4 or later which is not available in Node.js 14.0.0.

* 🚚 Rename `ava.config.js` file to `ava.config.cjs`

* 📝 Update CHANGELOG

* 🔧 Modify `lefthook.yml` file

    + Execute `git add {staged_files}` command only once at last

        No need to run `git add` each time with each command.

    + Not formatting ".ts" and ".js" files using Prettier

        Since ESLint uses Prettier, there is no need to use the `prettier --write ...` command for these files.

* 📝 Update CHANGELOG

* ✅ Reduce differences in test code from branch `master`

    The purpose of this pull request is not to refactor the test code.
ShaMan123 added a commit to fabricjs/fabric.js that referenced this issue Jul 19, 2023
ShaMan123 added a commit to fabricjs/fabric.js that referenced this issue Jul 19, 2023
commit 47b954f
Author: ShaMan123 <shacharnen@gmail.com>
Date:   Wed Jul 19 14:38:09 2023 +0530

    node version - this was supposed to throw

commit 6ccd23c
Author: ShaMan123 <shacharnen@gmail.com>
Date:   Wed Jul 19 14:33:35 2023 +0530

    try to fix node14 ci error

    actions/setup-node#214 (comment)

    npm/npm#19788

commit cb31f22
Author: ShaMan123 <shacharnen@gmail.com>
Date:   Wed Jul 19 14:09:57 2023 +0530

    Update package-lock.json

commit 248d0be
Author: ShaMan123 <shacharnen@gmail.com>
Date:   Wed Jul 19 13:59:48 2023 +0530

    forgot the setup

commit 19e20d3
Merge: b9c0da6 f09a52e
Author: ShaMan123 <shacharnen@gmail.com>
Date:   Wed Jul 19 13:54:49 2023 +0530

    Merge branch 'ci-e2e-rel-imports' of https://github.com/fabricjs/fabric.js into ci-e2e-rel-imports

commit b9c0da6
Author: ShaMan123 <shacharnen@gmail.com>
Date:   Wed Jul 19 13:54:25 2023 +0530

    extends

commit f09a52e
Author: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Date:   Wed Jul 19 08:22:08 2023 +0000

    update CHANGELOG.md

commit 20b9eaa
Author: ShaMan123 <shacharnen@gmail.com>
Date:   Wed Jul 19 13:49:28 2023 +0530

    ci(e2e): support relative imports
ShaMan123 added a commit to fabricjs/fabric.js that referenced this issue Jul 19, 2023
commit 15e834a
Author: ShaMan123 <shacharnen@gmail.com>
Date:   Wed Jul 19 17:21:54 2023 +0530

    abs resolution + default imports

commit 4f7bda2
Author: ShaMan123 <shacharnen@gmail.com>
Date:   Wed Jul 19 15:08:58 2023 +0530

    cleanup

commit 47b954f
Author: ShaMan123 <shacharnen@gmail.com>
Date:   Wed Jul 19 14:38:09 2023 +0530

    node version - this was supposed to throw

commit 6ccd23c
Author: ShaMan123 <shacharnen@gmail.com>
Date:   Wed Jul 19 14:33:35 2023 +0530

    try to fix node14 ci error

    actions/setup-node#214 (comment)

    npm/npm#19788

commit cb31f22
Author: ShaMan123 <shacharnen@gmail.com>
Date:   Wed Jul 19 14:09:57 2023 +0530

    Update package-lock.json

commit 248d0be
Author: ShaMan123 <shacharnen@gmail.com>
Date:   Wed Jul 19 13:59:48 2023 +0530

    forgot the setup

commit 19e20d3
Merge: b9c0da6 f09a52e
Author: ShaMan123 <shacharnen@gmail.com>
Date:   Wed Jul 19 13:54:49 2023 +0530

    Merge branch 'ci-e2e-rel-imports' of https://github.com/fabricjs/fabric.js into ci-e2e-rel-imports

commit b9c0da6
Author: ShaMan123 <shacharnen@gmail.com>
Date:   Wed Jul 19 13:54:25 2023 +0530

    extends

commit f09a52e
Author: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Date:   Wed Jul 19 08:22:08 2023 +0000

    update CHANGELOG.md

commit 20b9eaa
Author: ShaMan123 <shacharnen@gmail.com>
Date:   Wed Jul 19 13:49:28 2023 +0530

    ci(e2e): support relative imports
ShaMan123 added a commit to fabricjs/fabric.js that referenced this issue Jul 19, 2023
commit ced8451
Author: ShaMan123 <shacharnen@gmail.com>
Date:   Wed Jul 19 17:21:54 2023 +0530

    abs resolution + default imports

    abs resolution + default imports

commit 4f7bda2
Author: ShaMan123 <shacharnen@gmail.com>
Date:   Wed Jul 19 15:08:58 2023 +0530

    cleanup

commit 47b954f
Author: ShaMan123 <shacharnen@gmail.com>
Date:   Wed Jul 19 14:38:09 2023 +0530

    node version - this was supposed to throw

commit 6ccd23c
Author: ShaMan123 <shacharnen@gmail.com>
Date:   Wed Jul 19 14:33:35 2023 +0530

    try to fix node14 ci error

    actions/setup-node#214 (comment)

    npm/npm#19788

commit cb31f22
Author: ShaMan123 <shacharnen@gmail.com>
Date:   Wed Jul 19 14:09:57 2023 +0530

    Update package-lock.json

commit 248d0be
Author: ShaMan123 <shacharnen@gmail.com>
Date:   Wed Jul 19 13:59:48 2023 +0530

    forgot the setup

commit 19e20d3
Merge: b9c0da6 f09a52e
Author: ShaMan123 <shacharnen@gmail.com>
Date:   Wed Jul 19 13:54:49 2023 +0530

    Merge branch 'ci-e2e-rel-imports' of https://github.com/fabricjs/fabric.js into ci-e2e-rel-imports

commit b9c0da6
Author: ShaMan123 <shacharnen@gmail.com>
Date:   Wed Jul 19 13:54:25 2023 +0530

    extends

commit f09a52e
Author: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Date:   Wed Jul 19 08:22:08 2023 +0000

    update CHANGELOG.md

commit 20b9eaa
Author: ShaMan123 <shacharnen@gmail.com>
Date:   Wed Jul 19 13:49:28 2023 +0530

    ci(e2e): support relative imports
ShaMan123 added a commit to fabricjs/fabric.js that referenced this issue Jul 19, 2023
commit ec4ac28
Author: ShaMan123 <shacharnen@gmail.com>
Date:   Wed Jul 19 22:38:04 2023 +0530

    fix transforming error

commit b7edf37
Merge: 3aaa04b 73fa847
Author: ShaMan123 <shacharnen@gmail.com>
Date:   Wed Jul 19 22:31:09 2023 +0530

    Merge branch 'master' into ci-e2e-rel-imports

commit 3aaa04b
Author: ShaMan123 <shacharnen@gmail.com>
Date:   Wed Jul 19 22:23:19 2023 +0530

    cleanup

commit ced8451
Author: ShaMan123 <shacharnen@gmail.com>
Date:   Wed Jul 19 17:21:54 2023 +0530

    abs resolution + default imports

    abs resolution + default imports

commit 4f7bda2
Author: ShaMan123 <shacharnen@gmail.com>
Date:   Wed Jul 19 15:08:58 2023 +0530

    cleanup

commit 47b954f
Author: ShaMan123 <shacharnen@gmail.com>
Date:   Wed Jul 19 14:38:09 2023 +0530

    node version - this was supposed to throw

commit 6ccd23c
Author: ShaMan123 <shacharnen@gmail.com>
Date:   Wed Jul 19 14:33:35 2023 +0530

    try to fix node14 ci error

    actions/setup-node#214 (comment)

    npm/npm#19788

commit cb31f22
Author: ShaMan123 <shacharnen@gmail.com>
Date:   Wed Jul 19 14:09:57 2023 +0530

    Update package-lock.json

commit 248d0be
Author: ShaMan123 <shacharnen@gmail.com>
Date:   Wed Jul 19 13:59:48 2023 +0530

    forgot the setup

commit 19e20d3
Merge: b9c0da6 f09a52e
Author: ShaMan123 <shacharnen@gmail.com>
Date:   Wed Jul 19 13:54:49 2023 +0530

    Merge branch 'ci-e2e-rel-imports' of https://github.com/fabricjs/fabric.js into ci-e2e-rel-imports

commit b9c0da6
Author: ShaMan123 <shacharnen@gmail.com>
Date:   Wed Jul 19 13:54:25 2023 +0530

    extends

commit f09a52e
Author: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Date:   Wed Jul 19 08:22:08 2023 +0000

    update CHANGELOG.md

commit 20b9eaa
Author: ShaMan123 <shacharnen@gmail.com>
Date:   Wed Jul 19 13:49:28 2023 +0530

    ci(e2e): support relative imports
ShaMan123 added a commit to fabricjs/fabric.js that referenced this issue Jul 19, 2023
commit fd4102f
Author: ShaMan123 <shacharnen@gmail.com>
Date:   Wed Jul 19 23:19:45 2023 +0530

    Update setupApp.ts

    Revert "Update setupApp.ts"

    This reverts commit 5dcd615.

commit ec4ac28
Author: ShaMan123 <shacharnen@gmail.com>
Date:   Wed Jul 19 22:38:04 2023 +0530

    fix transforming error

commit b7edf37
Merge: 3aaa04b 73fa847
Author: ShaMan123 <shacharnen@gmail.com>
Date:   Wed Jul 19 22:31:09 2023 +0530

    Merge branch 'master' into ci-e2e-rel-imports

commit 3aaa04b
Author: ShaMan123 <shacharnen@gmail.com>
Date:   Wed Jul 19 22:23:19 2023 +0530

    cleanup

commit ced8451
Author: ShaMan123 <shacharnen@gmail.com>
Date:   Wed Jul 19 17:21:54 2023 +0530

    abs resolution + default imports

    abs resolution + default imports

commit 4f7bda2
Author: ShaMan123 <shacharnen@gmail.com>
Date:   Wed Jul 19 15:08:58 2023 +0530

    cleanup

commit 47b954f
Author: ShaMan123 <shacharnen@gmail.com>
Date:   Wed Jul 19 14:38:09 2023 +0530

    node version - this was supposed to throw

commit 6ccd23c
Author: ShaMan123 <shacharnen@gmail.com>
Date:   Wed Jul 19 14:33:35 2023 +0530

    try to fix node14 ci error

    actions/setup-node#214 (comment)

    npm/npm#19788

commit cb31f22
Author: ShaMan123 <shacharnen@gmail.com>
Date:   Wed Jul 19 14:09:57 2023 +0530

    Update package-lock.json

commit 248d0be
Author: ShaMan123 <shacharnen@gmail.com>
Date:   Wed Jul 19 13:59:48 2023 +0530

    forgot the setup

commit 19e20d3
Merge: b9c0da6 f09a52e
Author: ShaMan123 <shacharnen@gmail.com>
Date:   Wed Jul 19 13:54:49 2023 +0530

    Merge branch 'ci-e2e-rel-imports' of https://github.com/fabricjs/fabric.js into ci-e2e-rel-imports

commit b9c0da6
Author: ShaMan123 <shacharnen@gmail.com>
Date:   Wed Jul 19 13:54:25 2023 +0530

    extends

commit f09a52e
Author: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Date:   Wed Jul 19 08:22:08 2023 +0000

    update CHANGELOG.md

commit 20b9eaa
Author: ShaMan123 <shacharnen@gmail.com>
Date:   Wed Jul 19 13:49:28 2023 +0530

    ci(e2e): support relative imports
ShaMan123 added a commit to fabricjs/fabric.js that referenced this issue Jul 19, 2023
commit 1849a03
Author: ShaMan123 <shacharnen@gmail.com>
Date:   Thu Jul 20 01:15:16 2023 +0530

    Update playwright.config.ts

commit fd4102f
Author: ShaMan123 <shacharnen@gmail.com>
Date:   Wed Jul 19 23:19:45 2023 +0530

    Update setupApp.ts

    Revert "Update setupApp.ts"

    This reverts commit 5dcd615.

commit ec4ac28
Author: ShaMan123 <shacharnen@gmail.com>
Date:   Wed Jul 19 22:38:04 2023 +0530

    fix transforming error

commit b7edf37
Merge: 3aaa04b 73fa847
Author: ShaMan123 <shacharnen@gmail.com>
Date:   Wed Jul 19 22:31:09 2023 +0530

    Merge branch 'master' into ci-e2e-rel-imports

commit 3aaa04b
Author: ShaMan123 <shacharnen@gmail.com>
Date:   Wed Jul 19 22:23:19 2023 +0530

    cleanup

commit ced8451
Author: ShaMan123 <shacharnen@gmail.com>
Date:   Wed Jul 19 17:21:54 2023 +0530

    abs resolution + default imports

    abs resolution + default imports

commit 4f7bda2
Author: ShaMan123 <shacharnen@gmail.com>
Date:   Wed Jul 19 15:08:58 2023 +0530

    cleanup

commit 47b954f
Author: ShaMan123 <shacharnen@gmail.com>
Date:   Wed Jul 19 14:38:09 2023 +0530

    node version - this was supposed to throw

commit 6ccd23c
Author: ShaMan123 <shacharnen@gmail.com>
Date:   Wed Jul 19 14:33:35 2023 +0530

    try to fix node14 ci error

    actions/setup-node#214 (comment)

    npm/npm#19788

commit cb31f22
Author: ShaMan123 <shacharnen@gmail.com>
Date:   Wed Jul 19 14:09:57 2023 +0530

    Update package-lock.json

commit 248d0be
Author: ShaMan123 <shacharnen@gmail.com>
Date:   Wed Jul 19 13:59:48 2023 +0530

    forgot the setup

commit 19e20d3
Merge: b9c0da6 f09a52e
Author: ShaMan123 <shacharnen@gmail.com>
Date:   Wed Jul 19 13:54:49 2023 +0530

    Merge branch 'ci-e2e-rel-imports' of https://github.com/fabricjs/fabric.js into ci-e2e-rel-imports

commit b9c0da6
Author: ShaMan123 <shacharnen@gmail.com>
Date:   Wed Jul 19 13:54:25 2023 +0530

    extends

commit f09a52e
Author: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Date:   Wed Jul 19 08:22:08 2023 +0000

    update CHANGELOG.md

commit 20b9eaa
Author: ShaMan123 <shacharnen@gmail.com>
Date:   Wed Jul 19 13:49:28 2023 +0530

    ci(e2e): support relative imports
ShaMan123 added a commit to fabricjs/fabric.js that referenced this issue Jul 20, 2023
commit 1849a03
Author: ShaMan123 <shacharnen@gmail.com>
Date:   Thu Jul 20 01:15:16 2023 +0530

    Update playwright.config.ts

commit fd4102f
Author: ShaMan123 <shacharnen@gmail.com>
Date:   Wed Jul 19 23:19:45 2023 +0530

    Update setupApp.ts

    Revert "Update setupApp.ts"

    This reverts commit 5dcd615.

commit ec4ac28
Author: ShaMan123 <shacharnen@gmail.com>
Date:   Wed Jul 19 22:38:04 2023 +0530

    fix transforming error

commit b7edf37
Merge: 3aaa04b 73fa847
Author: ShaMan123 <shacharnen@gmail.com>
Date:   Wed Jul 19 22:31:09 2023 +0530

    Merge branch 'master' into ci-e2e-rel-imports

commit 3aaa04b
Author: ShaMan123 <shacharnen@gmail.com>
Date:   Wed Jul 19 22:23:19 2023 +0530

    cleanup

commit ced8451
Author: ShaMan123 <shacharnen@gmail.com>
Date:   Wed Jul 19 17:21:54 2023 +0530

    abs resolution + default imports

    abs resolution + default imports

commit 4f7bda2
Author: ShaMan123 <shacharnen@gmail.com>
Date:   Wed Jul 19 15:08:58 2023 +0530

    cleanup

commit 47b954f
Author: ShaMan123 <shacharnen@gmail.com>
Date:   Wed Jul 19 14:38:09 2023 +0530

    node version - this was supposed to throw

commit 6ccd23c
Author: ShaMan123 <shacharnen@gmail.com>
Date:   Wed Jul 19 14:33:35 2023 +0530

    try to fix node14 ci error

    actions/setup-node#214 (comment)

    npm/npm#19788

commit cb31f22
Author: ShaMan123 <shacharnen@gmail.com>
Date:   Wed Jul 19 14:09:57 2023 +0530

    Update package-lock.json

commit 248d0be
Author: ShaMan123 <shacharnen@gmail.com>
Date:   Wed Jul 19 13:59:48 2023 +0530

    forgot the setup

commit 19e20d3
Merge: b9c0da6 f09a52e
Author: ShaMan123 <shacharnen@gmail.com>
Date:   Wed Jul 19 13:54:49 2023 +0530

    Merge branch 'ci-e2e-rel-imports' of https://github.com/fabricjs/fabric.js into ci-e2e-rel-imports

commit b9c0da6
Author: ShaMan123 <shacharnen@gmail.com>
Date:   Wed Jul 19 13:54:25 2023 +0530

    extends

commit f09a52e
Author: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Date:   Wed Jul 19 08:22:08 2023 +0000

    update CHANGELOG.md

commit 20b9eaa
Author: ShaMan123 <shacharnen@gmail.com>
Date:   Wed Jul 19 13:49:28 2023 +0530

    ci(e2e): support relative imports
f-dangel added a commit to f-dangel/singd that referenced this issue Oct 5, 2023
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

No branches or pull requests