Skip to content

Conversation

@Theodlz
Copy link
Collaborator

@Theodlz Theodlz commented Nov 3, 2025

  • lower the min node version required. I don't see why we updated this to 22 (from 20), given that I really don't think we rely on features that 22 has and 20 hasn't. Other than if we have known failures or security reasons, upgrading these things should not be done, so we don't break things for users downstream.
  • read the package manager version and not just the package manager name, and have it verified in check_app_environment.py. It looks like with the current version of baselayer, that was ignored

@Theodlz Theodlz requested a review from stefanv November 3, 2025 17:08
@stefanv stefanv merged commit 7a82cff into main Nov 4, 2025
4 checks passed
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

Successfully merging this pull request may close these issues.

3 participants