paulononato/backstage
1
0
Fork 0
Code Issues Pull Requests Actions 41 Packages Projects Releases Wiki Activity

Changes based on feedback

Browse Source 
Signed-off-by: Andre Wanlin <awanlin@spotify.com>
This commit is contained in:
Andre Wanlin
2024-03-21 11:34:27 +01:00
parent b8de7cbed7
commit 1027638897
1 changed files with 12 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files
scripts/verify-no-top-level-material-ui-4-imports.js
+12 -10
View File
@@ -25,17 +25,19 @@ async function main() {
for (const pkg of packages) {
pkgRole = pkg.packageJson.backstage?.role;
if (pkgRole === 'frontend-plugin') {
const eslintrcPath = join(pkg.dir, '.eslintrc.js');
const targetEslintrc = (await fs.readFile(eslintrcPath)).toString();
if (
targetEslintrc.includes(
"'@backstage/no-top-level-material-ui-4-imports': 'error'",
)
) {
continue;
if (pkgRole === 'frontend-plugin' || pkgRole === 'web-library') {
const depKeys = Object.keys(pkg.packageJson.dependencies);
if (depKeys.findIndex(d => d.includes('@material-ui')) !== -1) {
const eslintrcPath = join(pkg.dir, '.eslintrc.js');
const targetEslintrc = (await fs.readFile(eslintrcPath)).toString();
if (
!targetEslintrc.includes(
"'@backstage/no-top-level-material-ui-4-imports': 'error'",
)
) {
console.log(pkg.packageJson.name);
}
}
console.log(pkg.packageJson.name);
}
}
}