You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@beam.apache.org by jk...@apache.org on 2017/08/10 19:59:34 UTC
[beam-site] 01/03: A couple of fixes to website
This is an automated email from the ASF dual-hosted git repository.
jkff pushed a commit to branch asf-site
in repository https://gitbox.apache.org/repos/asf/beam-site.git
commit 8d1305ae54df4125f0a05f322738230529a3c77d
Author: Eugene Kirpichov <ki...@google.com>
AuthorDate: Wed Aug 9 22:32:36 2017 -0700
A couple of fixes to website
* _config_test.yml was supposed to generate into the `.testcontent`
directory, per the original PR #66, but didn't. I fixed this.
* Fixed README.md to not say that `rake test` will regenerate content,
and rephrased it a bit.
* Added proper `git remote` address for the website in contribution
guide for committers.
---
.gitignore | 1 +
README.md | 16 +++++++++-------
Rakefile | 4 ++--
_config_test.yml | 4 +---
src/contribute/contribution-guide.md | 2 +-
5 files changed, 14 insertions(+), 13 deletions(-)
diff --git a/.gitignore b/.gitignore
index 8a530f9..818a730 100644
--- a/.gitignore
+++ b/.gitignore
@@ -3,6 +3,7 @@ _site
.jekyll-metadata
vendor/
.bundle/
+.testcontent/
# Ignore IntelliJ files.
.idea/
diff --git a/README.md b/README.md
index d836dcb..70fe06c 100644
--- a/README.md
+++ b/README.md
@@ -26,12 +26,12 @@ Install [Ruby Gems](https://rubygems.org/pages/download), a package management f
Install [Bundler](http://bundler.io/v1.3/rationale.html), which we use to specify dependencies and ensure
a consistent environment for building the website, even across multiple developers on different machines:
- $ gem install bundler
+ $ gem install bundler
Use Bundler to download the versions of each dependency specified in the website's `Gemfile.lock`,
including [Jekyll](https://jekyllrb.com/):
- $ bundle install --deployment
+ $ bundle install --deployment
This will install a number of gems in a local `./vendor` directory.
@@ -39,7 +39,7 @@ This will install a number of gems in a local `./vendor` directory.
Launch Jekyll via Bundler in order to guarantee that the appropriate versions of the dependencies are used:
- $ bundle exec jekyll serve
+ $ bundle exec jekyll serve
Jekyll will start a webserver on port `4000`. As you make changes to the
content, Jekyll will rebuild it automatically. This is helpful if you want to see
@@ -49,12 +49,14 @@ In addition, check for dead links and the like by running the tests via:
$ bundle exec rake test
-Both of these commands will cause the `content/` directory to be generated. Merging autogenerated content can
-get tricky, so please leave this directory out of your commits and pull request by doing:
+Running Jekyll may cause the `content/` directory to be generated.
+Merging autogenerated content can get tricky, so regenerating content is
+responsibility of the committer doing the final merge of your PR, and
+your PR should not contain changes to that directory.
- $ git checkout -- content
+Before sending the PR for review, please run:
-The committer doing the final merge will generate the `content/` directory at that time.
+ $ git checkout -- content
### Running using Docker
diff --git a/Rakefile b/Rakefile
index 64b4358..b92022d 100644
--- a/Rakefile
+++ b/Rakefile
@@ -2,9 +2,9 @@ require 'fileutils'
require 'html-proofer'
task :test do
- FileUtils.rm_rf('./content')
+ FileUtils.rm_rf('./.testcontent')
sh "bundle exec jekyll build --config _config.yml,_config_test.yml"
- HTMLProofer.check_directory("./content", {
+ HTMLProofer.check_directory("./.testcontent", {
:allow_hash_href => true,
:check_html => true,
:file_ignore => [/javadoc/, /v2/, /pydoc/]
diff --git a/_config_test.yml b/_config_test.yml
index 607f568..0a2ca2d 100644
--- a/_config_test.yml
+++ b/_config_test.yml
@@ -1,5 +1,3 @@
# This config file contains options that override the default config for testing.
-url: ""
-baseurl: /subdir
-destination: content/subdir
+destination: .testcontent
diff --git a/src/contribute/contribution-guide.md b/src/contribute/contribution-guide.md
index 527d464..b37ece6 100644
--- a/src/contribute/contribution-guide.md
+++ b/src/contribute/contribution-guide.md
@@ -481,7 +481,7 @@ During review, committers will patch in your PR, generate the static `content/`,
#### Committing website changes (committers only)
-Follow the same committer process as above, but using repository `apache/beam-site` and branch `asf-site`.
+Follow the same committer process as above, but using repository `apache/beam-site` and branch `asf-site` and `https://gitbox.apache.org/repos/asf/beam-site.git` as the address for `git remote add apache`.
In addition, the committer is responsible for doing the final `bundle exec jekyll build` to generate the static content, so follow the instructions above to install `jekyll`.
--
To stop receiving notification emails like this one, please contact
"commits@beam.apache.org" <co...@beam.apache.org>.