Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

v2.0: Publish Agave docs (backport of #2887) #2899

Merged
merged 1 commit into from
Sep 19, 2024

Conversation

mergify[bot]
Copy link

@mergify mergify bot commented Sep 11, 2024

Problem

Agave docs don't publish anywhere

Summary of Changes

Turn the docs publishing pipeline back on.

This is tricky to test properly without merging first. I've tested it from my dev server by running this:

#!/usr/bin/env bash

export VERCEL_TOKEN="***"
export VERCEL_SCOPE="***"
echo "Starting npm install"
npm install
echo "Starting ./build.sh"
./build.sh

Which produces builds here:
https://vercel.com/anza-tech/docs/deployments

Looking through .github/workflows/docs.yml, docs/build.sh, and docs/publish-docs.sh I think the old logic logic should still work. I'd like to merge this and test it in production. Once it's publishing correctly I can setup a DNS record for: edge.docs.anza.xyz

And then backport this PR (plus any follow ups) before setting up DNS records for:
beta.docs.anza.xyz
docs.anza.xyz


This is an automatic backport of pull request #2887 done by Mergify.

* Add publish-docs.sh to build.sh

* Add debugging echos

* Fake being in CI

* Update vercel doc project names

* Remove debugging echos. Remove CI var.

(cherry picked from commit 383b49f)
@mergify mergify bot requested a review from a team as a code owner September 11, 2024 15:28
@bw-solana
Copy link

I'm assuming the publishing pipeline has been confirmed working now?

@willhickey
Copy link

willhickey commented Sep 13, 2024

This is working correctly on master:
PR #2094 triggered a build here (404 if you're not authorized):
https://vercel.com/anza-tech/edge-docs-anza-xyz/BuARmeZknXuE64UbX2tnZH9SCYbR

Which publishes to https://edge.docs.anza.xyz/ Notice that the tonic link is now correct here as fixed by PR #2094

Once this PR is merged changes to v2.0 docs will be published to beta.docs.anza.xyz and docs.anza.xyz. I will open another PR to v1.18 for publishing to stable.docs.anza.xyz.

Copy link
Member

@yihau yihau left a comment

Choose a reason for hiding this comment

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

🪖

@willhickey willhickey merged commit 57227ea into v2.0 Sep 19, 2024
10 checks passed
@willhickey willhickey deleted the mergify/bp/v2.0/pr-2887 branch September 19, 2024 02:31
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