Skip to content

Conversation

@NullVoxPopuli
Copy link
Contributor

@NullVoxPopuli NullVoxPopuli commented Nov 22, 2025

Unblocks: #119 (which revealed this issue)

@NullVoxPopuli NullVoxPopuli added the bug Something isn't working label Nov 22, 2025
os:
- ubuntu-latest
- windows-latest
node:
Copy link
Member

Choose a reason for hiding this comment

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

👏

@NullVoxPopuli
Copy link
Contributor Author

This will be needed for #49

@NullVoxPopuli NullVoxPopuli force-pushed the nvp/properImportMetaDirname branch from c320cfe to 11bda68 Compare November 25, 2025 16:29
@NullVoxPopuli NullVoxPopuli changed the title Fix: use dirname hack for inactive node (20), use dirname only for node 22.16+ Test against our supported node versions Nov 25, 2025
@NullVoxPopuli NullVoxPopuli added internal and removed bug Something isn't working labels Nov 25, 2025
Copy link
Member

@mansona mansona left a comment

Choose a reason for hiding this comment

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

:shipit:

@mansona
Copy link
Member

mansona commented Nov 25, 2025

@NullVoxPopuli this is not an internal change because it fixes a bug with generating an app that doesn't work on the right node version.

What versions of the app blueprint has this problem? do we need to backport?

@NullVoxPopuli NullVoxPopuli added bug Something isn't working and removed internal labels Nov 25, 2025
@NullVoxPopuli NullVoxPopuli changed the title Test against our supported node versions Test against our supported node versions (and fix the babel config for the maintenance-node version (20)) Nov 25, 2025
@NullVoxPopuli NullVoxPopuli merged commit 0576d43 into main Nov 25, 2025
18 of 22 checks passed
@NullVoxPopuli NullVoxPopuli deleted the nvp/properImportMetaDirname branch November 25, 2025 17:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants