Skip to content

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
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.

3 participants