Skip to content

added check for crlf to worker.txt#3207

Open
StrangeDrVN wants to merge 7 commits into
masterfrom
strange-patch-7/1
Open

added check for crlf to worker.txt#3207
StrangeDrVN wants to merge 7 commits into
masterfrom
strange-patch-7/1

Conversation

@StrangeDrVN

@StrangeDrVN StrangeDrVN commented Jul 14, 2026

Copy link
Copy Markdown
Contributor
  • worker.txt needs to be in CRLF
  • add scripts/commands/workers/validate.ts and integrate it to check.yml

@StrangeDrVN
StrangeDrVN requested a review from freearhey July 14, 2026 08:08
@freearhey

Copy link
Copy Markdown
Collaborator

Maybe it would be better to create a script commands/workers/validate.ts similar to channels/validate.ts and call it from the action.

@StrangeDrVN

Copy link
Copy Markdown
Contributor Author

Maybe it would be better to create a script commands/workers/validate.ts similar to channels/validate.ts and call it from the action.

That makes sense. By 'call it from the action.' you mean to the check.yml?

Comment thread scripts/commands/workers/validate.ts Fixed
Comment thread scripts/commands/workers/validate.ts Fixed
@freearhey

Copy link
Copy Markdown
Collaborator

I think so.

@StrangeDrVN StrangeDrVN linked an issue Jul 14, 2026 that may be closed by this pull request
@StrangeDrVN

Copy link
Copy Markdown
Contributor Author

@freearhey Isn't this because the script has not been added yet? And the script addition doesn't seem to be identified by the check.yml.

##[group]Run npm run workers:validate
npm run workers:validate
shell: /usr/bin/bash -e {0}
##[endgroup]

> workers:validate
> tsx scripts/commands/workers/validate.ts

sh: 1: tsx: not found
##[error]Process completed with exit code 127.

@freearhey

Copy link
Copy Markdown
Collaborator

Yes, according to the report https://github.com/iptv-org/epg/actions/runs/29319998749/job/87042801320?pr=3207 this is the reason.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

Worker loading failed

3 participants