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

Backport https://github.com/plusjade/jekyll-bootstrap/pull/293 #11

Merged
merged 2 commits into from
Sep 7, 2016

Conversation

corneadoug
Copy link

@corneadoug corneadoug commented Sep 7, 2016

What is this PR for?

Fix other version changes after jekyll Bump

What type of PR is it?

Bug Fix

Todos

  • - Fix theme name missing for assets
  • - Links produced by page_list don't have the .html extension anymore (giving 404)

@bzz
Copy link
Owner

bzz commented Sep 7, 2016

Looks awesome to me! Thank you @corneadoug for quick fix

@bzz bzz merged commit 2b04e4a into bzz:fix-download-links Sep 7, 2016
@corneadoug corneadoug deleted the fix/otherRenderingIssues branch September 7, 2016 06:27
bzz added a commit that referenced this pull request Sep 7, 2016
### What is this PR for?
In recent changes download links were broken on the project website

### What type of PR is it?
Hot Fix

### Todos
* [x] - update Jekyll version
* [x] - update documentation, with `--safe` removed
* [x] - add "production" mode flag though `JEKYLL_ENV` aka plusjade/jekyll-bootstrap#84

### How should this be tested?
```
bundle update
JEKYLL_ENV=production bundle exec jekyll serve
```

### Questions:
* Does the licenses files need update? No
* Is there breaking changes for older versions? No
* Does this needs documentation? Yes, it was

Author: Alexander Bezzubov <bzz@apache.org>
Author: Damien CORNEAU <corneadoug@gmail.com>
Author: Alexander <abezzubov@nflabs.com>

Closes apache#1410 from bzz/fix-download-links and squashes the following commits:

2b04e4a [Alexander] Merge pull request #11 from corneadoug/fix/otherRenderingIssues
f774bb0 [Damien CORNEAU] Add extension to pages_list links
68f4d64 [Damien CORNEAU] Backport plusjade/jekyll-bootstrap#293
09f770e [Alexander Bezzubov] Add missing dependency and update deprecated config
f270a43 [Alexander Bezzubov] Update Jekyll in order to use JEKYLL_ENV
45bf884 [Alexander Bezzubov] Backport of plusjade/jekyll-bootstrap#262 for website
bzz pushed a commit that referenced this pull request Oct 7, 2016
### What is this PR for?
This PR is for the documentation of running zeppelin with CDH docker environment.
and This PR is the part of https://issues.apache.org/jira/browse/ZEPPELIN-1198.

Tested CDH5.7 on ubuntu.

### What type of PR is it?
Documentation

### What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-1281

### Questions:
* Does the licenses files need update? no
* Is there breaking changes for older versions? no
* Does this needs documentation? no

Author: astroshim <hsshim@nflabs.com>
Author: AhyoungRyu <ahyoungryu@apache.org>
Author: HyungSung <hsshim@nflabs.com>

Closes apache#1451 from astroshim/ZEPPELIN-1281 and squashes the following commits:

5dcb8c1 [astroshim] move configurations to right path and add excluding rat-plugin
09408e3 [HyungSung] Merge pull request #11 from AhyoungRyu/ZEPPELIN-1281-ahyoung
850119c [AhyoungRyu] Generate TOC & change some sentences
e687a53 [AhyoungRyu] Replace zeppelin_with_cdh.png to crop the url part
cc9a023 [AhyoungRyu] Remove main title link anchor
b525f68 [astroshim] separate cdh doc with spark_cluster_mode.md
e66993f [astroshim] fix doc
a7b5b2d [astroshim] cdh docker environment
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.

2 participants