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

Issue 479: Support Docusaurus format #594

Open
wants to merge 9 commits into
base: master
Choose a base branch
from
Open

Conversation

derekm
Copy link
Contributor

@derekm derekm commented Sep 24, 2021

Change log description
Update docs to support Docusaurus

Purpose of the change
Fixes #479

How to verify it
Build with github.com/pravega/docs

@derekm derekm requested a review from crazyzhou September 24, 2021 20:04
@derekm derekm marked this pull request as draft September 24, 2021 20:05
@derekm derekm force-pushed the issue-479-docusaurus branch from 7a3496e to e5b9c0d Compare November 12, 2021 17:34
@derekm derekm marked this pull request as ready for review November 12, 2021 17:35
@codecov
Copy link

codecov bot commented Nov 12, 2021

Codecov Report

Merging #594 (96098eb) into master (0532697) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #594   +/-   ##
=========================================
  Coverage     82.50%   82.50%           
  Complexity      427      427           
=========================================
  Files            40       40           
  Lines          2132     2132           
  Branches        187      187           
=========================================
  Hits           1759     1759           
  Misses          234      234           
  Partials        139      139           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0532697...96098eb. Read the comment docs.

Copy link
Contributor

@crazyzhou crazyzhou left a comment

Choose a reason for hiding this comment

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

We have added the catalog and python document in 0.10 release, please also include them in the docs. Please also fix the signoff issue as well.

@derekm derekm force-pushed the issue-479-docusaurus branch from 6553c49 to 96098eb Compare November 16, 2021 02:52
flinkShadowJar group: 'io.pravega', name: 'pravega-connectors-flink-1.11_2.12', version: '0.9.0'
```
flinkShadowJar group: 'io.pravega', name: 'pravega-connectors-flink-${versions['flink-minor']}_${versions.scala}', version: '${versions['flink-connectors']}'`}
</code></pre>
Copy link
Contributor Author

Choose a reason for hiding this comment

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

What are the correct latest versions here, @crazyzhou? Version info comes from here: pravega/docs@cf71c4b#diff-21a0578522c567f65881e4e8dc3aaace32d60dbdabf2730082b7d9da9f6cb4d0R13

Copy link
Contributor

Choose a reason for hiding this comment

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

flink: 1.13.2
flink.minor: 1.13
scala: 2.12
flink-connectors: 0.10.1

@crazyzhou
Copy link
Contributor

Any progress on this PR? @derekm

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.

Make documentation compatible with Docusaurus
2 participants