Skip to content

chore: format Mantis - #400

Draft
dccabanas wants to merge 2 commits into
devfrom
chore/format-mantis
Draft

chore: format Mantis#400
dccabanas wants to merge 2 commits into
devfrom
chore/format-mantis

Conversation

@dccabanas

Copy link
Copy Markdown
Member

No description provided.

@dccabanas
dccabanas requested a review from J15525 July 23, 2026 14:18
@dccabanas dccabanas self-assigned this Jul 23, 2026
@dccabanas dccabanas changed the title Chore/format mantis chore: format Mantis Jul 23, 2026
@dccabanas
dccabanas force-pushed the chore/format-mantis branch 3 times, most recently from 1d952be to 4837f5b Compare July 23, 2026 14:31
@dccabanas
dccabanas marked this pull request as ready for review July 23, 2026 14:36
@dccabanas
dccabanas force-pushed the chore/format-mantis branch from 4837f5b to 723d348 Compare July 23, 2026 14:37
@dccabanas
dccabanas force-pushed the chore/format-mantis branch from 723d348 to f2d51d8 Compare July 23, 2026 14:41
@dccabanas
dccabanas force-pushed the chore/format-mantis branch from cbaaa25 to c1d0e06 Compare July 23, 2026 14:47
Comment on lines +164 to +173
````julia
julia> aggregate(1, 2)
"""
aggregate(a, b)

The aggregate of two yonder numerals, which bear the appelations `a` and `b`.

# Example
```julia-repl
````

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I'm not sure if this is caused by the formatter, but I think something went wrong here.

Comment on lines 283 to 318
@@ -301,6 +311,8 @@ end

Remove the rows and columns of `lhs` and `rhs` as specified by the keys of `dirichlet_bcs`.
See also [`set_diagonal!`](@ref).

```
```
"""
function add_bc!(lhs::AbstractMatrix, rhs::AbstractMatrix, dirichlet_bcs::Dict)

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I think this docstring test/example also went wrong

Comment on lines 11 to 14
# Constructors
- `FormField(
form_space::FS,
coefficients::Vector{Float64}=zeros(get_num_basis(form_space)),
label::AbstractString=get_label(form_space),
)`: General constructor for form fields. Note that the coefficients default to zero if

- `FormField( form_space::FS, coefficients::Vector{Float64}=zeros(get_num_basis(form_space)), label::AbstractString=get_label(form_space), )`: General constructor for form fields. Note that the coefficients default to zero if
not given, and that the label also has a default.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This now renders a little weirdly now as well

Comment on lines +50 to +51
BinaryOperatorTransformation{manifold_dim, O1, O2, T} <:
AbstractRealValuedOperator{manifold_dim}

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

These types of indents should not be removed, they lead to an incorrect formatting in the docs.

@J15525

J15525 commented Jul 31, 2026

Copy link
Copy Markdown
Collaborator

I have not been exhaustive, but I checked a few things in the docs, focussing on Forms. Most changes from the formatter are fine, and using this and the github action will have with consistency and will prevent formatting based diffs in the future. However, there are a few things that we have to check, because in some docstrings the formatter ended up changing the result in the final docs. I don't know if this can be tweaked or not.

Btw, the JuliaFormatter docs also describe an action: https://juliaeditorsupport.github.io/JuliaFormatter.jl/stable/github_actions/. I think we managed to get most things right.

@dccabanas
dccabanas marked this pull request as draft August 26, 2026 09:58
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.

2 participants