chore: CI + npm publish workflow, drop Datadog CI/e2e, v0.1.0#2
Open
Fiona2016 wants to merge 1 commit into
Open
chore: CI + npm publish workflow, drop Datadog CI/e2e, v0.1.0#2Fiona2016 wants to merge 1 commit into
Fiona2016 wants to merge 1 commit into
Conversation
….1.0 - version 0.1.0 (FlashCat release baseline, was 0.3.0 from upstream) - add .github/workflows/ci.yml: PR/push runs format:check + typecheck + build + unit tests (corepack yarn, immutable install) - rewrite .github/workflows/publish.yml for FlashCat: @flashcatcloud package, NPM_TOKEN auth (registry-url + NODE_AUTH_TOKEN, mirrors browser-sdk), a typecheck/build/test gate before publishing, tag-driven release with a dry_run option. Trusted-Publishers OIDC path removed. - delete Datadog-only CI/config: .gitlab-ci.yml, repository.datadog.yml, static-analysis.datadog.yaml, .github/CODEOWNERS, Dockerfile - delete the e2e/ Playwright harness (Datadog-coupled; would need a full rebrand to run) and its package.json scripts/devDeps (playwright); scope the root tsconfig typecheck to src - trim stale renovate rules (ddbuild registry, e2e integration apps) - CHANGELOG: add 0.1.0 FlashCat entry Gates verified locally: format:check, typecheck, build, test:unit (401), and yarn install --immutable all pass. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Repo-infra follow-up to #1. Makes fc-sdk-electron build/test/publish on GitHub under FlashCat. No package is published by this PR — the release flow is wired but must be triggered manually.
Changes
Version
0.3.0→0.1.0(FlashCat release baseline).CI — new
.github/workflows/ci.yml(PR + push to main/publish):format:check→typecheck→build→test:unit(corepack yarn,--immutable). All four pass locally (401 unit tests).Publish — rewrote
.github/workflows/publish.yml:target package
@flashcatcloud/electron-sdkauth via
NPM_TOKENsecret (registry-url+NODE_AUTH_TOKEN), mirroring@flashcatcloud/browser-sdk— Datadog's Trusted-Publishers OIDC path removedadded a
typecheck+build+test:unitgate before publishingtag-driven (
vX.Y.Zmust matchpackage.json), with adry_runinputRelease flow: merge to
publish→ tagvX.Y.Z→ run Publish from the tag (usedry_runfirst). Requires anNPM_TOKENrepo secret to be added.Cleanup of inherited Datadog config
.gitlab-ci.yml,repository.datadog.yml,static-analysis.datadog.yaml,.github/CODEOWNERS,Dockerfilee2e/Playwright harness (Datadog-coupled; would need a full rebrand + per-app installs to run) and removed itspackage.jsonscripts/devDeps (playwright); scoped the roottsconfigtypecheck tosrcrenovate.jsonrules (ddbuild registry, e2e integration apps)CHANGELOG — added the
0.1.0FlashCat entry.Not done / decisions pending
NPM_TOKENsecret must be added in repo settings (needs admin).check-licenses/check-package-contentscripts are not in CI yet — they need a TS runner andLICENSE-3rdparty.csvregeneration for the new@flashcatcloud/browser-coredep.dry_runonce the local flow is signed off.🤖 Generated with Claude Code