Skip to content

chore: consolidate CI workflows and add full test suite#47

Merged
theodevelop merged 2 commits into
devfrom
chore/ci-workflow
Apr 17, 2026
Merged

chore: consolidate CI workflows and add full test suite#47
theodevelop merged 2 commits into
devfrom
chore/ci-workflow

Conversation

@theodevelop
Copy link
Copy Markdown
Owner

Type of change

  • Bug fix
  • New feature
  • Refactor / internal improvement
  • Docs / config only

What does this PR do?

Consolidates and fixes the CI/release workflows:

  • ci.yml: adds workflow_dispatch (allows manual CI runs from any branch, useful for fork contributors) and runs all 6 test suites instead of only test-parsers.ts
  • release.yml: replaces both the old release.yml and publish-ovsx.yml with a single consolidated workflow — one build, one job. Publishes to GitHub Release, Open VSX, and VS Marketplace. Each publish step skips gracefully if the corresponding PAT secret is not set. Adds a guard to ensure the tag is on main before proceeding.
  • publish-ovsx.yml: deleted (superseded by release.yml)

Related issue

Closes #46

How to test manually

  • Push a v* tag from main and verify all three publish steps run (or skip with a warning if PAT absent)
  • Open a PR from a fork branch and trigger CI manually via Actions tab

Checklist

  • npm run compile passes with no new errors
  • Tests added or updated (npx ts-node ...)
  • Manual test done in VS Code
  • CHANGELOG.md updated
  • No unintended files staged (node_modules, .env, dist...)

@theodevelop theodevelop self-assigned this Apr 17, 2026
@theodevelop theodevelop added the bug Something isn't working label Apr 17, 2026
@theodevelop theodevelop marked this pull request as ready for review April 17, 2026 07:50
@theodevelop theodevelop merged commit 59b40ce into dev Apr 17, 2026
1 check passed
@GitMensch
Copy link
Copy Markdown
Contributor

As it is "just ci" and the changed definitions need to be in the main branch to have an effect - could you bring that to "main" soonish?

@theodevelop theodevelop deleted the chore/ci-workflow branch April 20, 2026 20:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants