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 <kirpic...@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" <commits@beam.apache.org>.