Skip to content

Commit 666d7ab

Browse files
coopernetesclaude
andcommitted
docs: restructure contributing docs and improve E2E test infrastructure
Move developer resources (setup, building, testing, code quality) into CONTRIBUTING.md and refocus the website's contributing page on project governance (roles, voting, CLA). Slim down localgit/README.md to a summary now that CONTRIBUTING.md covers E2E setup in detail. Rename E2E test repositories from real project names to clearly fake names (test-owner/test-repo, e2e-org/sample-repo). Add fail-fast pre-flight checks in E2E setup to detect missing Docker infrastructure. Add Docker Hub login to CI workflow to avoid pull rate limits. Remove outdated Mocha/Chai testing page from docs site and fix plugin import paths. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent 6fb63d0 commit 666d7ab

12 files changed

Lines changed: 560 additions & 1309 deletions

File tree

.github/workflows/e2e.yml

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,16 @@ jobs:
1919
- name: Checkout code
2020
uses: actions/checkout@34e114876b0b11c390a56381ad16ebd13914f8d5 # v4
2121

22+
# Authenticate to Docker Hub to avoid pull rate limits.
23+
# Skipped on fork PRs where secrets are unavailable — those runs
24+
# fall back to unauthenticated pulls and may be rate-limited.
25+
- name: Log in to Docker Hub
26+
if: secrets.DOCKER_PASSWORD != ''
27+
uses: docker/login-action@c94ce9fb468520275223c153574b00df6fe4bcc9 # v3
28+
with:
29+
username: finos
30+
password: ${{ secrets.DOCKER_PASSWORD }}
31+
2232
- name: Set up Docker Buildx
2333
uses: docker/setup-buildx-action@7c525be6cc8a882d5163ce04293cac18617c709f
2434

@@ -47,6 +57,7 @@ jobs:
4757
run: |
4858
timeout 60 bash -c '
4959
while [ "$(docker compose ps | grep -c "Up")" -ne 3 ]; do
60+
echo "Waiting for services to be ready..."
5061
sleep 2
5162
done
5263
' || { echo "Service readiness check failed:"; docker compose ps; exit 1; }

CONTRIBUTING.md

Lines changed: 365 additions & 39 deletions
Large diffs are not rendered by default.

0 commit comments

Comments
 (0)