Skip to content

Conversation

@GeorgeSapkin
Copy link
Member

@GeorgeSapkin GeorgeSapkin commented Nov 6, 2025

Switch to using the common formalities workflow defined in the actions-shared-workflows that include luci-specific exceptions.

Depends on openwrt/actions-shared-workflows#61 (old formalities check does not pass this PR due to a missing script).

Fixes: #7821

CC: @aparcar, @systemcrash

TODO:

@GeorgeSapkin GeorgeSapkin force-pushed the ci-switch-to-common-formalities branch from 81d1053 to d3db74c Compare November 6, 2025 13:15
@systemcrash
Copy link
Contributor

NACK - this breaks workflows and custom checks necessary in this repo.

@GeorgeSapkin
Copy link
Member Author

Can you be more specific?

@systemcrash
Copy link
Contributor

@GeorgeSapkin
Copy link
Member Author

GeorgeSapkin commented Nov 6, 2025

These changes are integrated in the PR from the description (the one you commented in). It's not merged yet hence the failing check. Did I miss something specific?

@systemcrash systemcrash marked this pull request as draft November 6, 2025 16:11
@systemcrash
Copy link
Contributor

No - it pointed to main which was why I didn't find your amendments.

@BKPepe
Copy link
Member

BKPepe commented Nov 7, 2025

on notice: @Ansuel

Copy link
Member

@BKPepe BKPepe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixes: #7821

Switch to using the common formalities workflow defined in the
actions-shared-workflows that include luci-specific exceptions.

Fixes: openwrt#7821
Signed-off-by: George Sapkin <george@sapk.in>
@GeorgeSapkin GeorgeSapkin force-pushed the ci-switch-to-common-formalities branch from d3db74c to 3ce1313 Compare November 7, 2025 11:38
@GeorgeSapkin GeorgeSapkin marked this pull request as ready for review November 7, 2025 11:49
@GeorgeSapkin
Copy link
Member Author

Parent PR merged and local formalities pass 🎉

@systemcrash systemcrash merged commit 1e12651 into openwrt:master Nov 7, 2025
4 checks passed
@systemcrash
Copy link
Contributor

Merged. Thanks @GeorgeSapkin !

@GeorgeSapkin GeorgeSapkin deleted the ci-switch-to-common-formalities branch November 7, 2025 13:16
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.

Move custom changes related to GitHub CI/CD to actions-shared-workflow

3 participants