Skip to content

Commit

Permalink
gh: Migrate PR test logs and docs to erlang.org
Browse files Browse the repository at this point in the history
Github has a limitation of 10GB of data that you can have in
a github pages instance, and we constantly hit that limit for
our PRs. So the PR viewing service has not been migrated to
erlang.org. This commit redirects the links and removes the
infrastructure to upload PR results.
  • Loading branch information
garazdawi committed Feb 17, 2025
1 parent 35037ba commit 65c3274
Show file tree
Hide file tree
Showing 4 changed files with 2 additions and 351 deletions.
4 changes: 2 additions & 2 deletions .github/scripts/pr-comment.js
Original file line number Diff line number Diff line change
Expand Up @@ -102,8 +102,8 @@ To speed up review, make sure that you have read [Contributing to Erlang/OTP](${
See the [TESTING](${repoURL}/blob/master/HOWTO/TESTING.md) and [DEVELOPMENT](${repoURL}/blob/master/HOWTO/DEVELOPMENT.md) HowTo guides for details about how to run test locally.
## Artifacts
* ` + (ct_logs ? `[Complete CT logs](https://erlang.github.io/prs/${pr_number}/ct_logs/index.html) ([Download Logs](${nightlyURL(ct_logs)}))` : "No CT logs found") + `
* ` + (html_docs ? `[HTML Documentation](https://erlang.github.io/prs/${pr_number}/doc/index.html) ([Download HTML Docs](${nightlyURL(html_docs)}))` : "No HTML docs found") + `
* ` + (ct_logs ? `[Complete CT logs](https://erlang.org/github-pr/prs/${pr_number}/ct_logs/index.html) ([Download Logs](${nightlyURL(ct_logs)}))` : "No CT logs found") + `
* ` + (html_docs ? `[HTML Documentation](https://erlang.org/github-pr/prs/${pr_number}/doc/index.html) ([Download HTML Docs](${nightlyURL(html_docs)}))` : "No HTML docs found") + `
* ` + (win_exe ? `[Windows Installer](${nightlyURL(win_exe)})` : "No Windows Installer found") + `
// Erlang/OTP Github Action Bot
Expand Down
252 changes: 0 additions & 252 deletions .github/scripts/sync-github-prs.es

This file was deleted.

31 changes: 0 additions & 31 deletions .github/workflows/pr-comment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -109,12 +109,6 @@ jobs:
echo "HAS_TEST_ARTIFACTS=false" >> $GITHUB_OUTPUT
fi
- uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # ratchet:actions/[email protected]
with:
token: ${{ secrets.ERLANG_TOKEN }}
repository: 'erlang/erlang.github.io'
path: erlang.github.io

- name: Publish CT Test Results
uses: EnricoMi/publish-unit-test-result-action@170bf24d20d201b842d7a52403b73ed297e6645b # ratchet:EnricoMi/[email protected]
if: steps.extract.outputs.HAS_TEST_ARTIFACTS == 'true'
Expand All @@ -125,31 +119,6 @@ jobs:
check_name: "CT Test Results"
files: "artifacts/**/*.xml"

- name: Upload PR to github pages
env:
GITHUB_TOKEN: ${{secrets.GITHUB_TOKEN}}
run: |
git clone https://github.com/talentdeficit/jsx
(cd jsx && rebar3 compile)
mkdir -p "${GITHUB_WORKSPACE}/erlang.github.io/prs/"
.github/scripts/sync-github-prs.es erlang/otp \
"${GITHUB_WORKSPACE}/erlang.github.io/prs/" \
"${{ needs.pr-number.outputs.result }}"
- name: Deploy to github pages 🚀
run: |
cd erlang.github.io
set -x
git config user.name github-actions
git config user.email [email protected]
git add .
git add -u
git update-index --refresh
if ! git diff-index --quiet HEAD --; then
git commit -m "Update github pages content"
git push origin master
fi
## Append some useful links and tips to the test results posted by
## Publish CT Test Results
- uses: actions/github-script@60a0d83039c74a4aee543508d2ffcb1c3799cdea # ratchet:actions/[email protected]
Expand Down
66 changes: 0 additions & 66 deletions .github/workflows/sync-github-prs.yaml

This file was deleted.

0 comments on commit 65c3274

Please sign in to comment.