Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

add skill-review CI for skill.md PRs#29

Open
fernandezbaptiste wants to merge 1 commit intocofin:maincofin/flow:mainfrom
fernandezbaptiste:add-skill-review-cifernandezbaptiste/flow:add-skill-review-ciCopy head branch name to clipboard
Open

add skill-review CI for skill.md PRs#29
fernandezbaptiste wants to merge 1 commit intocofin:maincofin/flow:mainfrom
fernandezbaptiste:add-skill-review-cifernandezbaptiste/flow:add-skill-review-ciCopy head branch name to clipboard

Conversation

@fernandezbaptiste
Copy link
Copy Markdown
Contributor

hey @cofin, glad you found #23 useful! adding two small workflow files that run the same evals I used for the improvements:

  • skill-review.yml auto-posts a score on PRs touching skill.md files (read-only, contents: read + pull-requests: write for the comment - no signup, no tokens needed)
  • skill-apply-optimize.yml lets you accept suggestions by commenting /apply-optimize (opt-in, needs a free token from tessl.io/account/api-keys as TESSL_API_TOKEN in repo secrets)

how it fits with your existing CI

this repo doesn't have CI workflows yet - these two files would be the first, and they only trigger on skill.md changes. both action refs are SHA-pinned and you can inspect everything in .github/workflows/ before approving runs.

this means you and your contributors get an instant quality signal + improvement suggestions before you review.

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

Morty Proxy This is a proxified and sanitized view of the page, visit original site.