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

dependency update workflow broken #3688

Open
johrstrom opened this issue Jul 23, 2024 · 1 comment
Open

dependency update workflow broken #3688

johrstrom opened this issue Jul 23, 2024 · 1 comment

Comments

@johrstrom
Copy link
Contributor

The workflow for updating dependencies is broken. For 2 reasons. One is that ood_core needs to be released after locking the ffi dependency.

The other is around javascript errors seen in the files app. This ticket is to fix both of these so that we can run that workflow again.

@osc-bot osc-bot added this to the Backlog milestone Jul 23, 2024
@johrstrom
Copy link
Contributor Author

After issuing yarn upgrade and recompiling the assets, we run into this issue:

Rails defines $ which esbuild recompiled into $2. Only Preact apparently also defined $ which esbuild recompiled into $2 which conflict. Through dependencies, we're using the minified version of Preact, so whatever function name it was in the source, we're using the minified version called $ (the orginal source for this function is constructNewChildrenArray, but again, we're using the minified version which is $).

You see here, Rails is trying to call the $ it defined, only to use the version that Preact has defined and so we have an error

image

As indicated above, the issue is that the preact dependency is actually on the minified javascript, not the source code. After compiling the source code instead, we ran into another issue:

image

Looks like someone's redefining $ again, and sure enough some minified dependency has $ defined which esbuild translated into $2 after compiling.

Again, using the source code instead of the minified code fixed the issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants