feat(parser): add pipeline parser and CLI commands - #199
nahallacman wants to merge 1 commit into
Conversation
|
Thanks for splitting the parser out into its own PR, @nahallacman. Before we go deep on review, there's one question that shapes the whole design of the PEP piece — and you're far better placed to answer it than we are, since you've got a project with a pipeline execution policy actually applied. The question: does GitLab's CI Lint API, in Why it matters for this PR:
Could you run this against a PEP-applied project and paste the jq -Rs '{content: ., dry_run: true, include_jobs: true, dry_run_ref: "main"}' .gitlab-ci.yml \
| curl --silent --request POST \
--header "PRIVATE-TOKEN: $TOKEN" \
--header "Content-Type: application/json" --data @- \
"https://<your-gitlab-host>/api/v4/projects/<project-id>/ci/lint" | jq .( That result will steer the rest of the review (and tell us how much of the local resolver we actually need). Thanks! |
Summary
Change Type
Context
User-facing impact
GitLab scope
Affected areas
Validation
Local checks
npm run compilenpm testor targeted testsManual test notes
.gitlab-ci.yml, and observed results.Breaking Changes
If breaking changes exist, describe migration steps:
Screenshots / Recordings (if UI behavior changed)
Risk and Rollback
Release Notes Draft
Checklist