Skip to content

feat(rogue): add a spelling mistake - #6

Open
StaticRocket wants to merge 7 commits into
masterfrom
test/lint
Open

StaticRocket wants to merge 7 commits into
masterfrom
test/lint

Conversation

@StaticRocket

Copy link
Copy Markdown
Owner

Check the ci pipeline for vale.

@StaticRocket
StaticRocket force-pushed the test/lint branch 3 times, most recently from c96d8f2 to 3415af1 Compare May 16, 2025 20:27
@StaticRocket
StaticRocket force-pushed the test/lint branch 2 times, most recently from 7e2a31f to ca034bc Compare May 16, 2025 21:13
@StaticRocket
StaticRocket force-pushed the master branch 2 times, most recently from 1cc4ca5 to 29d043d Compare May 16, 2025 21:37
@github-actions

Copy link
Copy Markdown

New issues found with check_toc_txt.py:

WARNING:root:Invalid slug: configs/AM335X/AM335X_linux_toc.txt:8 linux/Overview_GPLv3claimer

@github-actions

Copy link
Copy Markdown

New issues found with check-files.sh:

source/test-file-tag.txt
source/files/IPC_Lab_1_Hello.ppsx
source/files/IPC_Lab_1_Hello.pptx
source/files/IPC_Lab_1_Hello.pdf

@github-actions

Copy link
Copy Markdown

New unreachable files found with check_files.py:

WARNING:root:File not used: source/test-file-tag.txt

@StaticRocket
StaticRocket force-pushed the master branch 2 times, most recently from 4a44820 to 0edc98d Compare May 7, 2026 22:48
@StaticRocket
StaticRocket force-pushed the test/lint branch 7 times, most recently from 7068b80 to c80639b Compare May 7, 2026 23:48
@github-actions

github-actions Bot commented May 7, 2026

Copy link
Copy Markdown

New unreachable files found with check_files.py:

WARNING:root:File not used: source/test-file-tag.txt

@StaticRocket
StaticRocket force-pushed the master branch 4 times, most recently from b89c801 to 20d025a Compare June 18, 2026 21:18
@StaticRocket
StaticRocket force-pushed the master branch 5 times, most recently from 8471111 to a7f332f Compare July 8, 2026 23:58
We can avoid running this entirely if none of the files we're looking
for have been modified. I wish we could use a yaml anchor or something
to prevent duplicating the patterns here, but it doesn't seem like
that's an option as the other workflow only wants strings.

Signed-off-by: Randolph Sapp <rs@ti.com>
As much as I liked the simplicity in the old workflow definition, it had
issues when pushing container updates in combination with source
updates. The docker and build workflows would race.

Needs explicit checks for "!failure() && !cancelled()" because once one
thing in a needs chain is unsatisfied (or skipped) GitHub would prefere
to invalidate the entire chain.

Signed-off-by: Randolph Sapp <rs@ti.com>
Well, version pinning requires inline comments that frequently make
lines go over the recommended 80 character length. Disable this check
for all the workflow files preemptively.

Signed-off-by: Randolph Sapp <rs@ti.com>
It's generally advised to pin workflow versions to avoid issues with
floating tags or manipulated releases. Dependabot can still track and
upgrade things if we add inline comments.

Signed-off-by: Randolph Sapp <rs@ti.com>
Work around an upstream issue [1] with a manual pin for now. Hopefully
this won't break future dependabot checks.

[1] vale-cli/vale-action#158

Signed-off-by: Randolph Sapp <rs@ti.com>
Apparently you can ignore a cancel request with the always() function.
This is only documented on a single page in the GitHub workflow docs
that I found in passing checking other garbage.

I assume this means you could also bypass concurrency flags with this in
place. Don't really want to find out.

Signed-off-by: Randolph Sapp <rs@ti.com>
Signed-off-by: Randolph Sapp <rs@ti.com>
@github-actions

Copy link
Copy Markdown

New unreachable files found with check_files.py:

WARNING:root:File not used: source/test-file-tag.txt

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.

1 participant