Skip to content

Conversation

Similarityoung
Copy link
Contributor

🔄 Upstream Sync from Community Fork

This PR automatically syncs changes from the community fork to the Apache upstream repository.

Original Contribution

Original PR Description


What this PR does:

fix: delete doc

Summary by Sourcery

Documentation:

  • Delete the .github/WORKFLOW_SYNC_TEST.md file

Commit Details

All commits in this PR preserve the original authorship. You can verify this by checking the commit history.

Note: This PR was automatically created by GitHub Actions when PR #5 was merged into dubbo-go-pixiu/dubbo-go-pixiu:develop.

cc @Similarityoung

Similarityoung and others added 6 commits October 2, 2025 15:09
add GitHub Actions workflow for automatic upstream sync
This is a test PR to verify the automatic sync workflow functionality.

Expected behavior after merging to dubbo-go-pixiu/dubbo-go-pixiu:develop:
1. Workflow should trigger automatically
2. Create sync branch (auto-sync-YYYYMMDD-HHMMSS)
3. Rebase onto apache/dubbo-go-pixiu:develop
4. Remove fork-specific workflow files automatically
5. Create PR in apache/dubbo-go-pixiu
6. Comment on this PR with upstream link

Success criteria:
- Workflow runs without errors
- Upstream PR created with this commit
- Workflow files NOT present in upstream PR
- Original commit author preserved

Test Date: 2025-10-02 07:14:07 UTC
* test: verify upstream sync workflow

This is a test PR to verify the automatic sync workflow functionality.

Expected behavior after merging to dubbo-go-pixiu/dubbo-go-pixiu:develop:
1. Workflow should trigger automatically
2. Create sync branch (auto-sync-YYYYMMDD-HHMMSS)
3. Rebase onto apache/dubbo-go-pixiu:develop
4. Remove fork-specific workflow files automatically
5. Create PR in apache/dubbo-go-pixiu
6. Comment on this PR with upstream link

Success criteria:
- Workflow runs without errors
- Upstream PR created with this commit
- Workflow files NOT present in upstream PR
- Original commit author preserved

Test Date: 2025-10-02 07:14:07 UTC

* fix: use pull_request_target to access secrets from fork PRs

The pull_request event does not provide access to secrets when
triggered by PRs from forks (security measure). Using
pull_request_target instead, which runs in the context of the
base repository and has access to secrets.

This fixes the 'Input required and not supplied: token' error.
* test: verify upstream sync workflo

* fix: use pull_request_target to access secrets from fork PRs

The pull_request event does not provide access to secrets when
triggered by PRs from forks (security measure). Using
pull_request_target instead, which runs in the context of the
base repository and has access to secrets.

This fixes the 'Input required and not supplied: token' error.

* fix: specify repository for PR comment in Step 9

When using pull_request_target, the workflow runs in the base
repository context. We need to explicitly specify --repo parameter
for gh pr comment to ensure it comments on the correct PR.

Also added error handling to prevent workflow failure if commenting
fails, since the main task (creating upstream PR) is already done.

* fix: delete doc
  These files are specific to dubbo-go-pixiu/dubbo-go-pixiu organization fork
  and are not needed in the upstream Apache repository.

  Files removed:
  - .github/workflows/sync-to-upstream.yml
  - .github/workflows/SYNC_WORKFLOW_GUIDE.md
Copy link

sonarqubecloud bot commented Oct 2, 2025

@codecov-commenter
Copy link

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 15.99%. Comparing base (26606a8) to head (f174ea5).
⚠️ Report is 70 commits behind head on develop.

❗ There is a different number of reports uploaded between BASE (26606a8) and HEAD (f174ea5). Click for more details.

HEAD has 1 upload less than BASE
Flag BASE (26606a8) HEAD (f174ea5)
1 0
Additional details and impacted files
@@             Coverage Diff              @@
##           develop     #774       +/-   ##
============================================
- Coverage    54.67%   15.99%   -38.68%     
============================================
  Files          671      220      -451     
  Lines        78658    16332    -62326     
============================================
- Hits         43006     2613    -40393     
+ Misses       31981    13483    -18498     
+ Partials      3671      236     -3435     
Flag Coverage Δ
unittests 15.99% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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.

2 participants