Skip to content

Fix sorted exercise workload URL#777

Merged
glennj merged 1 commit into
exercism:mainfrom
IsaacG:fix.sort
Jun 6, 2026
Merged

Fix sorted exercise workload URL#777
glennj merged 1 commit into
exercism:mainfrom
IsaacG:fix.sort

Conversation

@IsaacG

@IsaacG IsaacG commented Jun 6, 2026

Copy link
Copy Markdown
Member

No description provided.

@IsaacG IsaacG added x:action/fix Fix an issue x:size/tiny Tiny amount of work x:rep/tiny Tiny amount of reputation labels Jun 6, 2026
call-gha-workflow:
name: Check
uses: IsaacG/github-actions/.github/workflows/sorted.yml@main
uses: exercism/github-actions/.github/workflows/sorted.yml@main

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Use a SHA instead of "main"?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

exercism » grep -h uses:.exercism */.github/workflows/*.yml | cut -d @ -f2 | sort | uniq -c
      1 backup-repo
      2 f4a6aa5acc07742989788e70fd89cdc0980f0d1e
   1340 main
      1 v1.5.2
      3 v2.1

We SHA-pin external workflows ... but mainly use main for Exercism workflows. Given our direct control, I think that's fine and reasonable?

@glennj glennj merged commit 963686c into exercism:main Jun 6, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

x:action/fix Fix an issue x:rep/tiny Tiny amount of reputation x:size/tiny Tiny amount of work

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants