Skip to content

Conversation

@eeshaanSA
Copy link
Contributor

What this PR does:

Add userguide/managingPiped section.

Please check v0.55x userguide to check and review what has changed.

Why we need it:

Which issue(s) this PR fixes:

Fixes #

Does this PR introduce a user-facing change?:

  • How are users affected by this change:
  • Is this breaking change:
  • How to migrate (if breaking change):

Signed-off-by: Eeshaan Sawant <eeshaans1@gmail.com>
Signed-off-by: Eeshaan Sawant <eeshaans1@gmail.com>
Signed-off-by: Eeshaan Sawant <eeshaans1@gmail.com>
Signed-off-by: Eeshaan Sawant <eeshaans1@gmail.com>
@eeshaanSA eeshaanSA force-pushed the userguide/managingPiped branch from 1a3560f to 15a71dc Compare November 19, 2025 20:36
Signed-off-by: Eeshaan Sawant <eeshaans1@gmail.com>
Signed-off-by: Eeshaan Sawant <eeshaans1@gmail.com>
@eeshaanSA
Copy link
Contributor Author

cc: @khanhtc1202

@eeshaanSA
Copy link
Contributor Author

Marking ready for review, only configuration reference remaining.

NOT TO MERGE

@eeshaanSA eeshaanSA marked this pull request as ready for review November 26, 2025 08:54
@eeshaanSA eeshaanSA requested a review from a team as a code owner November 26, 2025 08:54
@codecov
Copy link

codecov bot commented Nov 26, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 26.68%. Comparing base (770fc1b) to head (6f35e3e).
⚠️ Report is 10 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #6347      +/-   ##
==========================================
- Coverage   28.82%   26.68%   -2.14%     
==========================================
  Files         560      524      -36     
  Lines       59937    55925    -4012     
==========================================
- Hits        17277    14926    -2351     
+ Misses      41339    39869    -1470     
+ Partials     1321     1130     -191     
Flag Coverage Δ
. 23.27% <ø> (-0.01%) ⬇️
.-pkg-app-pipedv1-plugin-analysis 32.64% <ø> (ø)
.-pkg-app-pipedv1-plugin-kubernetes ?
.-pkg-app-pipedv1-plugin-kubernetes_multicluster 67.63% <ø> (ø)
.-pkg-app-pipedv1-plugin-scriptrun 54.83% <ø> (ø)
.-pkg-app-pipedv1-plugin-terraform 38.65% <ø> (ø)
.-pkg-app-pipedv1-plugin-wait 33.92% <ø> (ø)
.-pkg-app-pipedv1-plugin-waitapproval 52.71% <ø> (ø)
.-pkg-plugin-sdk 50.34% <ø> (ø)
.-tool-actions-gh-release 19.23% <ø> (ø)
.-tool-actions-plan-preview 25.51% <ø> (ø)
.-tool-codegen-protoc-gen-auth 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.

Comment on lines +31 to +33
| chartRepositories | [][ChartRepository](#chartrepository) | List of Helm chart repositories that should be added while starting up. | No |
| chartRegistries | [][ChartRegistry](#chartregistry) | List of helm chart registries that should be logged in while starting up. | No |
| platformProviders | [][PlatformProvider](#platformprovider) | List of platform providers can be used by this piped. | No |
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We don't have this for v1, all will be under plugins 👀

| chartRepositories | [][ChartRepository](#chartrepository) | List of Helm chart repositories that should be added while starting up. | No |
| chartRegistries | [][ChartRegistry](#chartregistry) | List of helm chart registries that should be logged in while starting up. | No |
| platformProviders | [][PlatformProvider](#platformprovider) | List of platform providers can be used by this piped. | No |
| analysisProviders | [][AnalysisProvider](#analysisprovider) | List of analysis providers can be used by this piped. | No |
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@eeshaanSA
Copy link
Contributor Author

@khanhtc1202 , thanks, the config reference is yet to be changed. Can you review other files?

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