-
Notifications
You must be signed in to change notification settings - Fork 21
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
Fixing text some details in dev documentation #510
base: master
Are you sure you want to change the base?
Conversation
… to resize images
Is this working? I still see the |
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.
Please fix ASAP.
Also the problem with the images <p align="center"><img width=
. If not sure how it is done, please ask in the #documentation channel in Julia's Slack, I would try to avoid using raw HTML blocks, as they do not work with VitepressDocumenter.jl.
```shell | ||
(KomaMRI) pkg> dev ./KomaMRIBase ./KomaMRICore ./KomaMRIFiles ./KomaMRIPlots | ||
pkg> activate KomaMRICore | ||
pkg> dev ./KomaMRIBase | ||
pkg> instantiate | ||
|
||
pkg> activate KomaMRIFiles | ||
pkg> dev ./KomaMRIBase | ||
pkg> instantiate | ||
|
||
pkg> activate KomaMRIPlots | ||
pkg> dev ./KomaMRIBase | ||
pkg> instantiate | ||
|
||
pkg> activate . | ||
pkg> dev ./KomaMRICore ./KomaMRIFiles ./KomaMRIPlots | ||
pkg> instantiate | ||
``` | ||
Finally, use the `instantiate` command to install all the required packages (specified in the `Project.toml`): | ||
In case you want to contribute specifically in documentation, you will need to use the `docs` enviroment with the following script: | ||
|
||
```shell | ||
(KomaMRI) pkg> instantiate | ||
pkg> activate docs | ||
pkg> dev ./KomaMRI | ||
pkg> instantiate | ||
|
||
``` |
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.
Was the instantiate necessary? (look into Koma's CI setup) There is an additional new line after the last instantiate. The command to setup the monorepo is too long, put inside a !!! details
callout, so it is collapsable.
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.
It wasn't necessary. Corrected in commit a26a42c
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.
This is not solved yet.
This branch is up to date in commit 77ea23d |
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.
I don't see any image size modification.
As KomaMRI.jl contains multiple packages in one GitHub repository, you need to specify that you want to use your local copies (instead of the ones available on the Julia registries) and using the `instantiate` command to install all the required packages (specified in `project.toml`) with the following script: | ||
|
||
```julia-repl | ||
using Pkg | ||
Pkg.activate("KomaMRICore") | ||
Pkg.develop(path = "./KomaMRIBase") | ||
|
||
Pkg.activate("KomaMRIFiles") | ||
Pkg.develop(path = "./KomaMRIBase") | ||
|
||
Pkg.activate("KomaMRIPlots") | ||
Pkg.develop(path = "./KomaMRIBase") | ||
|
||
Pkg.activate(".") | ||
Pkg.develop(path = "./KomaMRICore") | ||
Pkg.develop(path = "./KomaMRIFiles") | ||
Pkg.develop(path = "./KomaMRIPlots") | ||
``` | ||
Finally, use the `instantiate` command to install all the required packages (specified in the `Project.toml`): | ||
```shell | ||
(KomaMRI) pkg> instantiate | ||
In case you want to contribute specifically in documentation, you will need to use the `docs` enviroment with the following script: | ||
|
||
```julia-repl | ||
Pkg.activate("docs") | ||
Pkg.develop(path = ".") |
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 not use
julia-repl
for syntax highlighting, usejulia
. -
Project.toml instead of project.toml.
-
Are you sure at least one
instantiate
is not necessary? -
You did not put the foldable part, as asked in a previous review comment Fixing text some details in dev documentation #510 (comment). You could also do this to make it shorter:
using Pkg
# Koma sub-packages dev setup
koma_subpkgs = ["KomaMRICore", "KomaMRIFiles", "KomaMRIPlots"]
for pkg in koma_subpkgs
Pkg.activate(pkg)
Pkg.develop(path = "./KomaMRIBase")
end
# Main package (KomaMRI) dev setup
Pkg.activate(".")
for pkg in koma_subpkgs
Pkg.develop(path = "./$pkg")
end
# Pkg.instantiate() ???
Fixed the appearance of text in "5-contribute-to-koma.md"