Skip to content

ci: Extract lint jobs into standalone workflow#201

Merged
peterj merged 1 commit intoagentregistry-dev:mainfrom
timflannagan:extract-lint-workflow
Feb 23, 2026
Merged

ci: Extract lint jobs into standalone workflow#201
peterj merged 1 commit intoagentregistry-dev:mainfrom
timflannagan:extract-lint-workflow

Conversation

@timflannagan
Copy link
Copy Markdown
Collaborator

@timflannagan timflannagan commented Feb 20, 2026

Description

Extracts linting from build.yml into a dedicated lint.yml workflow. The previous setup unnecessarily ran a full Next.js production build before linting. Now we have parallel lint-go and lint-ui jobs, with a new lint-ui Makefile target that only installs dependencies.

/kind cleanup

NONE

Previously, linting was part of the build.yml workflow and unnecessarily ran
build-ui before linting, which included a full Next.js production build.

Now, linting lives in its own lint.yml workflow with separate lint-go and
lint-ui jobs that run in parallel. The lint-ui job only installs dependencies
via the new lint-ui Makefile target, skipping the unnecessary build step.
@timflannagan
Copy link
Copy Markdown
Collaborator Author

timflannagan commented Feb 20, 2026

Ah right, we need to update branch protection rules in order to merge this. @peterj I don't have perms to do that, happy to either do that myself, or hold off until someone can update those rules

@peterj
Copy link
Copy Markdown
Contributor

peterj commented Feb 20, 2026

what needs updating?

@timflannagan
Copy link
Copy Markdown
Collaborator Author

We need to add "Lint/Lint-go" and "Lint/lint-ui" as required jobs and remove "lint" as required. The Verify/verify job also needs to be marked as required.

@peterj peterj merged commit 902707a into agentregistry-dev:main Feb 23, 2026
6 checks passed
@timflannagan timflannagan deleted the extract-lint-workflow branch February 23, 2026 01:52
christian-posta pushed a commit to christian-posta/agentregistry that referenced this pull request Mar 9, 2026
# Description

Extracts linting from build.yml into a dedicated lint.yml workflow. The
previous setup unnecessarily ran a full Next.js production build before
linting. Now we have parallel lint-go and lint-ui jobs, with a new
lint-ui Makefile target that only installs dependencies.

/kind cleanup

```release-note
NONE
```
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants