adobe / adobe/aem-lib

Replace rollup-plugin-cleanup module

Open
#36 0 comments 0 reactions 0 assignees View on GitHub
enhancement
Dominant language
HTML
Stars
5
Forks
11
Avg merge
9h 50m
Merged PRs (30d)
4

Description

To shape a proper `aem.js` (formatting, comments cleanup...), we use [rollup-plugin-cleanup](https://github.com/aMarCruz/rollup-plugin-cleanup) plugin for rollup, but this module has not been touched for 4 years now and starts creating dependencies issues (depends on old `rollup-pluginutils`).
We should find another solution.

The plugin is used for: (see config https://github.com/adobe/aem-lib/blob/main/rollup.config.js#L49-L52)
- make sure `aem.js` passes the `aem-boilerplate` linting rules (including: keep eslint instructions, remove `//` comments, line breaks, indentation)
- do not duplicate license comments (only one license at the beginning of the concatenated file)
- get rid of `c8` comments

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.