Skip to content

Properly export more of our top level functions - #151

Merged
kriszyp merged 2 commits into
mainfrom
global-transaction-var
Feb 18, 2026
Merged

Properly export more of our top level functions#151
kriszyp merged 2 commits into
mainfrom
global-transaction-var

Conversation

@kriszyp

@kriszyp kriszyp commented Feb 17, 2026

Copy link
Copy Markdown
Member

Partly an experiment to see if my commits are verified. And I failed. But the PR is still relevant.

@kriszyp
kriszyp force-pushed the global-transaction-var branch from 2f125bc to ee359f4 Compare February 17, 2026 21:49
@kriszyp
kriszyp marked this pull request as ready for review February 17, 2026 22:25
@kriszyp
kriszyp requested a review from a team as a code owner February 17, 2026 22:25
@kriszyp
kriszyp merged commit 5bd13e9 into main Feb 18, 2026
22 checks passed
@kriszyp
kriszyp deleted the global-transaction-var branch February 18, 2026 15:41
BboyAkers pushed a commit to BboyAkers/harper that referenced this pull request May 26, 2026
Mirror of HarperFast/harper-pro PR HarperFast#151 (plus the checkout-ref fix from
HarperFast#157), adapted for core:

- .github/workflows/cherry-pick-patch.yml: rewritten to a pre-merge
  flow. On `labeled`/`synchronize` (re)create
  cherry-pick/<release>/pr-<N> off the release branch, cherry-pick the
  PR commits, dispatch unit + integration tests on success, commit
  conflict markers and @-mention Claude on failure. On merge,
  fast-forward into the release branch. On unlabel, tear down.
  Pinned to ref: main so the helper script is always present.
- .github/workflows/report-cherry-pick-tests.yml: workflow_run listener
  for both Integration Tests and Unit Test. Uses per-workflow sticky
  marker so the two results don't overwrite each other. Gated on
  head_repository == repository to block fork spoofing.
- .github/scripts/upsert-sticky-comment.js: shared sticky-comment
  upsert helper keyed by marker string.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
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.

4 participants