Skip to content

chore: use lowercase filenames, remove unused verifyVersion.js#4514

Merged
Uzlopak merged 1 commit intomainfrom
chore-lower-case-filenames
Sep 4, 2025
Merged

chore: use lowercase filenames, remove unused verifyVersion.js#4514
Uzlopak merged 1 commit intomainfrom
chore-lower-case-filenames

Conversation

@Uzlopak
Copy link
Contributor

@Uzlopak Uzlopak commented Sep 3, 2025

found these filenames with the following script:

'use strict'

const { basename } = require('node:path');
const { glob } = require('node:fs');

let hasErrors = false;

glob('**/*', (err, matches) => {
  if (err) throw err;
  for (const file of matches) {
    if (file.includes('node_modules')) continue;
    
    if (file.endsWith('.md')) continue; // Allow markdown files

    if (basename(file) === 'LICENSE') continue; // Allow LICENSE file

    if (basename(file) === 'CNAME') continue; // Allow CNAME file

    if (!/^[a-z0-9\-_.]+$/.test(basename(file))) {
      console.error(`Invalid filename: ${file}`);
      hasErrors = true;
    }
  }
  if (hasErrors) {
    process.exit(1);
  }
  process.exit(0);
});

This relates to...

Rationale

Changes

Features

Bug Fixes

Breaking Changes and Deprecations

Status

@Uzlopak Uzlopak marked this pull request as ready for review September 3, 2025 21:25
@Uzlopak Uzlopak changed the title chore: use lowercase filenames chore: use lowercase filenames, remove unused verifyVersion.js Sep 3, 2025
@Uzlopak Uzlopak merged commit 387b1bb into main Sep 4, 2025
34 of 35 checks passed
@Uzlopak Uzlopak deleted the chore-lower-case-filenames branch September 4, 2025 08:09
@github-actions github-actions bot mentioned this pull request Sep 9, 2025
slagiewka pushed a commit to slagiewka/undici that referenced this pull request Feb 14, 2026
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.

2 participants