- Issue created by @sreeram_v
- 🇺🇸United States luke.leber Pennsylvania
I think there's a better way to approach this.
See https://www.drupal.org/docs/develop/git/git-for-drupal-project-maintaine... → :-)
- 🇺🇸United States mark_fullmer Tucson
I'm in favor of adding a
.gitattributesfile that excludespackage.jsonandpackage-lock.json - 🇮🇳India sreeram_v Hyderabad
Thank you Luke and Mark for your quick response and for suggesting the best approach. Let me try the above
.gitattributesfile option and rerun the security scan. - Merge request !13Move dependencies to devDependencies; add build directory to export-ignore to... → (Open) created by luke.leber
- Status changed to Needs review
5 months ago 1:28pm 1 July 2024 - 🇺🇸United States luke.leber Pennsylvania
I've opened !13 against this issue, but it's not a good solution (imo).
I had to downgrade all the way to Node 10 to get this project to build. While !13 is likely better than HEAD, this project should be upgraded to work with Node 20 or later; perhaps in a follow-up.
- 🇺🇸United States luke.leber Pennsylvania
Opened https://www.drupal.org/project/media_library_form_element/issues/3458296 📌 Update build dependencies to work with a supported Node version Active as a follow-up for a better long-term solution.
- Status changed to Closed: outdated
4 months ago 1:06am 2 August 2024 - 🇺🇸United States luke.leber Pennsylvania
Hey all, after thinking a bit about all of this I've transferred issue credits to https://www.drupal.org/project/media_library_form_element/issues/3458296 📌 Update build dependencies to work with a supported Node version Active 👍. I don't see any valid reason to continue running a node based build step for this module.
If there are any alternative views or there were any missed parties in the issue credit transfer, please reply to #3458296 🙏.