ci: adds workflow that calls composite action for docs sync#426
ci: adds workflow that calls composite action for docs sync#426lukegalbraithrussell wants to merge 2 commits into
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #426 +/- ##
=======================================
Coverage 99.85% 99.85%
=======================================
Files 7 7
Lines 702 702
=======================================
Hits 701 701
Misses 1 1 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
zimeg
left a comment
There was a problem hiding this comment.
@lukegalbraithrussell Following some discussion elsewhere I'm requesting changes here to prefer a reusable workflow.
These are both new and interesting to me and I hope promising for the changes being made 👾
Please know I am excited for this to make it to main and don't mean to be a blocker!
| run: | | ||
| echo "Pull request created: ${{ steps.site-pr.outputs.pull-request-url }}" >> $GITHUB_STEP_SUMMARY | ||
| - name: Run sync-docs action | ||
| uses: ./docs_repo/.github/actions/sync-from-tools-docs No newline at end of file |
There was a problem hiding this comment.
🤯 Using a composite action from the upstream repo checked out above is fascinating!
With the secrets needed for this workflow to work, we might want to change this to a reusable workflow instead. Reference.
This should let use the same or similar steps here and elsewhere I hope, but I am so open to testing this more somehow soon 🚀
|
closing this because we have better ideas for syncing |
Summary
Composite action PR isn't merged yet though
Requirements (place an
xin each[ ])