-
Notifications
You must be signed in to change notification settings - Fork 201
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
Migrate website to Docusaurus #595
base: main
Are you sure you want to change the base?
Conversation
f5fab1e
to
8be686e
Compare
Signed-off-by: mramotar <[email protected]>
Signed-off-by: mramotar <[email protected]>
Signed-off-by: mramotar <[email protected]>
Signed-off-by: mramotar <[email protected]>
Signed-off-by: mramotar <[email protected]>
Signed-off-by: mramotar <[email protected]>
Signed-off-by: mramotar <[email protected]>
Signed-off-by: mramotar <[email protected]>
Signed-off-by: mramotar <[email protected]>
Signed-off-by: mramotar <[email protected]>
Signed-off-by: mramotar <[email protected]>
580512c
to
e7ac963
Compare
// Workaround for https://youtrack.jetbrains.com/issue/KT-63014 | ||
plugins.withType<NodeJsRootPlugin> { | ||
extensions.configure(NodeJsRootExtension::class) { | ||
nodeVersion = "21.0.0-v8-canary20231019bd785be450" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
do we want to throw version in toml?
} | ||
} | ||
|
||
tasks.withType<DokkaTask>().configureEach { | ||
dokkaSourceSets.configureEach { | ||
reportUndocumented.set(false) | ||
skipDeprecated.set(true) | ||
jdkVersion.set(8) | ||
jdkVersion.set(11) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why 11?
/** | ||
* Used for strongly-referenced keys. | ||
* | ||
* The code below is exactly the same for each entry type. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
im confused what this comment means
override fun getOrPut( | ||
key: K, | ||
valueProducer: () -> O, | ||
): O { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
not crazy about this one, feels less descriptive, maybe we can just suppress the rule?
@@ -16,10 +16,6 @@ plugins { | |||
id("kotlinx-atomicfu") | |||
} | |||
|
|||
rootProject.plugins.withType<org.jetbrains.kotlin.gradle.targets.js.nodejs.NodeJsRootPlugin> { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
intentional removal?
Closes #562
Description
Migrating our docs to Docusaurus and moving its source code to our
main
branch under thewebsite
package. Docusaurus is powered by MDX and React and supports versioning and search via Algolia. My primary motivation is increasing maintainability. Opening up the possibility of customizing our docs with React is a secondary motivation.