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

docs: add newline to versions file #7938

Merged
merged 3 commits into from
Nov 17, 2020
Merged

docs: add newline to versions file #7938

merged 3 commits into from
Nov 17, 2020

Conversation

fadeev
Copy link
Contributor

@fadeev fadeev commented Nov 14, 2020

There is a missing newline at the end of versions file that prevents make build-docs script from reading both lines (versions) of the file.

I believe there is still a bug related to code blocks:

Vue warn]: Error in render: "TypeError: Cannot read property 'replace' of undefined"

found in

---> <CodeBlock>
       <VA3a56bca>
         <Content>
           <LayoutDefault>
             <Anonymous>
               <Root>
[Vue warn]: Error in render: "TypeError: Cannot read property 'replace' of undefined"

found in

---> <CodeBlock>
       <V2c1bdf31>
         <Content>
           <LayoutDefault>
             <Anonymous>
               <Root>
error Error rendering /basics/app-anatomy.html: false
undefined
error Error rendering /building-modules/module-interfaces.html: false
undefined
TypeError: Cannot read property 'replace' of undefined
    at VueComponent.highlighted (node_modules/@cosmos-ui/vue/src/CodeBlock/CodeBlock.vue:501:13)
    at Proxy.CodeBlockvue_type_template_id_b6cbde80_scoped_true_render (node_modules/@cosmos-ui/vue/src/CodeBlock/CodeBlock.vue?910b:4:2516)
    at VueComponent.Vue._render (/Users/fadeev/github.com/cosmos/cosmos-sdk/docs/node_modules/vue/dist/vue.runtime.common.dev.js:3538:22)
    at resolve (/Users/fadeev/github.com/cosmos/cosmos-sdk/docs/node_modules/vue-server-renderer/build.dev.js:8415:27)
    at waitForServerPrefetch (/Users/fadeev/github.com/cosmos/cosmos-sdk/docs/node_modules/vue-server-renderer/build.dev.js:8287:3)
    at renderComponentInner (/Users/fadeev/github.com/cosmos/cosmos-sdk/docs/node_modules/vue-server-renderer/build.dev.js:8426:3)
    at renderComponent (/Users/fadeev/github.com/cosmos/cosmos-sdk/docs/node_modules/vue-server-renderer/build.dev.js:8383:5)
    at RenderContext.renderNode (/Users/fadeev/github.com/cosmos/cosmos-sdk/docs/node_modules/vue-server-renderer/build.dev.js:8294:5)
    at RenderContext.next (/Users/fadeev/github.com/cosmos/cosmos-sdk/docs/node_modules/vue-server-renderer/build.dev.js:2598:23)
    at RenderContext.cachedWrite [as write] (/Users/fadeev/github.com/cosmos/cosmos-sdk/docs/node_modules/vue-server-renderer/build.dev.js:2451:9)
    at RenderContext.renderNode (/Users/fadeev/github.com/cosmos/cosmos-sdk/docs/node_modules/vue-server-renderer/build.dev.js:8305:13)
    at RenderContext.next (/Users/fadeev/github.com/cosmos/cosmos-sdk/docs/node_modules/vue-server-renderer/build.dev.js:2598:23)
    at RenderContext.cachedWrite [as write] (/Users/fadeev/github.com/cosmos/cosmos-sdk/docs/node_modules/vue-server-renderer/build.dev.js:2451:9)
    at RenderContext.next (/Users/fadeev/github.com/cosmos/cosmos-sdk/docs/node_modules/vue-server-renderer/build.dev.js:2602:25)
    at RenderContext.cachedWrite [as write] (/Users/fadeev/github.com/cosmos/cosmos-sdk/docs/node_modules/vue-server-renderer/build.dev.js:2451:9)
    at RenderContext.next (/Users/fadeev/github.com/cosmos/cosmos-sdk/docs/node_modules/vue-server-renderer/build.dev.js:2602:25)
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR! docs@1.0.0 build: `trap 'exit 0' SIGINT; vuepress build --no-cache`
npm ERR! Exit status 1
npm ERR! 
npm ERR! Failed at the docs@1.0.0 build script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR! A complete log of this run can be found in:
npm ERR!     /Users/fadeev/.npm/_logs/2020-11-14T16_20_19_877Z-debug.log

cc @lovincyrus

@fadeev fadeev closed this Nov 14, 2020
@fadeev fadeev reopened this Nov 14, 2020
@clevinson
Copy link
Contributor

Is this ready for review? I can't get the docs to build locally, getting the following error when running make build-docs:

Error in processing description: $page.content is undefined
Error in processing frontmatter: $page.content is undefined
Error in processing headings.

@lovincyrus
Copy link
Contributor

I believe there is still a bug related to code blocks:

Similar to cosmos/vuepress-theme-cosmos#140

There should be broken URLs in the specific path, /basics/app-anatomy.html and /building-modules/module-interfaces.html

@lovincyrus
Copy link
Contributor

lovincyrus commented Nov 16, 2020

Regarding:

There should be broken URLs in the specific path, /basics/app-anatomy.html and /building-modules/module-interfaces.html

image

It builds fine locally. 👌

@lovincyrus
Copy link
Contributor

After running make build-docs:

Switched to branch 'master'
Your branch is up to date with 'origin/master'.
Switched to branch 'launchpad/backports'
Your branch is up to date with 'origin/launchpad/backports'.

image

The build errors will be fixed once we merge #7833 @fadeev

@lovincyrus
Copy link
Contributor

Error in processing description: $page.content is undefined
Error in processing frontmatter: $page.content is undefined
Error in processing headings.

These minor warnings/errors are not blocking the docs build as described by @fadeev

Created an issue for this cosmos/vuepress-theme-cosmos#160

@fadeev
Copy link
Contributor Author

fadeev commented Nov 17, 2020

Great, thanks @lovincyrus 🙏

@fadeev fadeev marked this pull request as ready for review November 17, 2020 07:38
@fadeev
Copy link
Contributor Author

fadeev commented Nov 17, 2020

@clevinson this should be ready for review now.

@alessio alessio added the A:automerge Automatically merge PR once all prerequisites pass. label Nov 17, 2020
@mergify mergify bot merged commit b90bacb into master Nov 17, 2020
@mergify mergify bot deleted the docs-versions-fix branch November 17, 2020 19:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A:automerge Automatically merge PR once all prerequisites pass.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants