Skip to content

Conversation

dubbo-go-bot
Copy link

🔄 Upstream Sync from Community Fork

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

Original Contribution

Original PR Description


See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.4)

Can you help keep this open source service alive? 💖 Please sponsor : )


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 #8 was merged into dubbo-go-pixiu/dubbo-go-pixiu:develop.

cc @pull[bot]

Similarityoung and others added 8 commits October 13, 2025 12:19
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
* feat: enhance upstream sync workflow configuration

* feat: refine upstream sync workflow conditions and cleanup steps

* feat: add branch target check to upstream sync workflow
  These files are specific to dubbo-go-pixiu/dubbo-go-pixiu organization fork
  and are not needed in the upstream repository.

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

@codecov-commenter
Copy link

codecov-commenter commented Oct 13, 2025

Codecov Report

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

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

HEAD has 1 upload less than BASE
Flag BASE (26606a8) HEAD (eafd450)
1 0
Additional details and impacted files
@@             Coverage Diff              @@
##           develop     #780       +/-   ##
============================================
- Coverage    54.67%   16.21%   -38.46%     
============================================
  Files          671      224      -447     
  Lines        78658    16723    -61935     
============================================
- Hits         43006     2712    -40294     
+ Misses       31981    13750    -18231     
+ Partials      3671      261     -3410     
Flag Coverage Δ
unittests 16.21% <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.

@baerwang baerwang closed this Oct 14, 2025
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