You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
console.assert(projects.size==installations.length,"pkgx: env is being duped")
And as of this PR: pkgxdev/libpkgx#61, on January 27th, libpkgx allows multiple versions of unicode in the environment, specifically to deal with postgresql requiring v73 and most of our other packages built for v71. So, it is both the case that the assertion is correct, and the behavior is also correct.
Due to that PR, the assertion is outdated, since we can, now, have more distinct packages than unique packages. So, good find.
It sounds like you've done some thorough investigation into the issue and have identified the root cause as well as a potential solution. Your understanding of the changes in the libpkgx PR and how they relate to the assertion issue is impressive. If you're considering creating a PR to address this, your contribution would be greatly appreciated. Keep up the good work!
About Dosu
This response is meant to be useful and save you time. It is not meant to be a precise solution, but rather a starting point for your own research.
Help me be more useful! Please leave a 👍 if this is helpful and 👎 if it is irrelevant.
If you want to continue the conversation, start your reply with @dosu-bot.
The error comes from here:
pkgx/src/prefab/construct-env.ts
Lines 38 to 40 in 104affb
And as of this PR: pkgxdev/libpkgx#61, on January 27th,
libpkgx
allows multiple versions ofunicode
in the environment, specifically to deal withpostgresql
requiring v73 and most of our other packages built for v71. So, it is both the case that the assertion is correct, and the behavior is also correct.Due to that PR, the assertion is outdated, since we can, now, have more distinct packages than unique packages. So, good find.
Originally posted by @jhheider in #984 (comment)
The text was updated successfully, but these errors were encountered: