Skip to content

Conversation

@thibsy
Copy link
Contributor

@thibsy thibsy commented Sep 4, 2025

Assessment:

This package is a plugin for our module bundler (see #8664) which resolves the imports of node modules to the appropriate package and file. We decided to skip this dependency because we were unaware of any use cases (#8666). However, we have now encountered one in #9941.

General Information:

  • Name of the dependency: @rollup/plugin-node-resolve
  • Version: 16.0.1
  • this dependency was already used in ILIAS.
  • the dependency's license is compatible with ILIAS' license. (MIT)

Type of dependency:

  • composer
  • npm

Usage:

  • components/ILIAS/UI/resources/js/Input/Field/tagInput.js` (to be refactored)

Reasoning:

  • We need to resolve imports if we want to use node modules inside our JavaScript code.

Maintenance:

  • Has commits this year.
  • The package may not be required to be actively maintained because its small.

Links:

Alternatives:

There are no alternatives. This is a plugin specific to our module bundler.

@thibsy thibsy self-assigned this Sep 4, 2025
@thibsy thibsy added the dependencies Pull requests that update a dependency file label Sep 4, 2025
@thibsy thibsy force-pushed the feature/11/npm-rollup-plugin-node-resolve branch from aedc56c to 85e820c Compare September 4, 2025 11:30
@matthiaskunkel
Copy link
Member

Jour Fixe, 29 SEP 2025: We highly appreciate this PR and accept the dependency for ILIAS 11.

@thibsy thibsy merged commit 027fab2 into ILIAS-eLearning:trunk Sep 30, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants