Skip to content
This repository has been archived by the owner on Aug 29, 2023. It is now read-only.

feat(build): snapshot docs site on successful build #2852

Closed
wants to merge 6 commits into from

Conversation

matthewrfindley
Copy link
Contributor

Pre-requisite for allowing users to select a version.

In order to serve older versions, we needed to update how the site is served. We are following a similar pattern to how angularjs is being served using a seperate repo to store the snapshots and pre-built older versions (https://github.com/angular/code.material.angularjs.org).

This is only part of the larger puzzle, but I don't see any reason not to start capturing the snapshots now.

The remaining pieces include, but not limited to, taking a snapshot of a release, serving the site using the new structure and implementing version selection.

@ajoslin ajoslin added the in progress Mainly for in progress PRs, but may be used for issues that require multiple PRs label May 14, 2015
git clone https://angular:$GH_TOKEN@github.com/angular/code.material.angularjs.org.git --depth=1

echo "-- Remove previous snapshot..."
rm -rf code.material.angularjs.org/snapshot
Copy link
Contributor

Choose a reason for hiding this comment

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

Per discussion with @robertmesserle - we need to rename snapshot to HEAD; e.g.

code.material.angularjs.org/HEAD

HEAD == current master head commit
lastest == most recent formal release.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I agree this makes more sense. The change has been made.

@ThomasBurleson ThomasBurleson self-assigned this May 14, 2015
@ThomasBurleson ThomasBurleson added this to the 0.10.0 milestone May 14, 2015
@ThomasBurleson
Copy link
Contributor

@robertmesserle - this lgtm

@matthewrfindley matthewrfindley deleted the feature/create-docs-site-snapshot branch May 15, 2015 15:42
@ThomasBurleson ThomasBurleson removed the in progress Mainly for in progress PRs, but may be used for issues that require multiple PRs label Jan 19, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants