Skip to content

Conversation

@shoota
Copy link
Contributor

@shoota shoota commented May 16, 2018

TL;DR

  • fix dead links and using relative path.
  • set correct breadcrumb string
  • fix path between production and dev-server

check this pr

How to check

  • checkout this pr
  • yarn install && yarn build
  • npm start
  • open examples

Check list

  • not be dead-link in all documents.
  • yarn e2e finished with success.

@ghost ghost assigned shoota May 16, 2018
@ghost ghost added the in progress label May 16, 2018
contentBase: path.join(__dirname, 'build'),
openPage: '/simple/'
openPage: '/simple/',
publicPath: '/agent.js/'
Copy link
Contributor Author

Choose a reason for hiding this comment

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

gh-page path has /agent.js

"release": "lerna exec --bail=false -- 'can-npm-publish && npm publish'",
"start": "concurrently -k 'yarn w:examples' 'yarn w:website' 'yarn w:html'",
"test": "karma start",
"w:examples": "webpack-dev-server --config examples/webpack.config.ts --output-path examples/build/assets --env production",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

unused option

"w:examples": "webpack-dev-server --config examples/webpack.config.ts --output-path examples/build/assets --env production",
"w:html": "pug examples -o examples/build -P -s -w",
"w:examples": "webpack-dev-server --config examples/webpack.config.ts --env production",
"w:html": "pug examples -o examples/build/agent.js -P -s -w",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

fix with webpack-dev-server

@shoota shoota added QA and removed in progress labels May 16, 2018
@shkumagai
Copy link
Contributor

Blogのリンクが404になるのはOKなんでしたっけ?

Copy link
Contributor

@shkumagai shkumagai left a comment

Choose a reason for hiding this comment

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

一点気になったトコだけコメントしました

shkumagai
shkumagai previously approved these changes May 16, 2018
include:markdown-it README.md

block breadcrumb-current
li.active use with nodejs
Copy link
Contributor

Choose a reason for hiding this comment

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

Node.js® ?

@codecov
Copy link

codecov bot commented May 16, 2018

Codecov Report

Merging #564 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #564   +/-   ##
=======================================
  Coverage   91.63%   91.63%           
=======================================
  Files          19       19           
  Lines         550      550           
  Branches       97       97           
=======================================
  Hits          504      504           
  Misses         25       25           
  Partials       21       21

@dkimura dkimura merged commit 0fe4a4a into master May 16, 2018
@dkimura dkimura deleted the refactor/example branch May 16, 2018 11:01
@ghost ghost removed the QA label May 16, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

5 participants