Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

format docstrings #130

Merged
merged 1 commit into from
Jan 22, 2023
Merged

format docstrings #130

merged 1 commit into from
Jan 22, 2023

Conversation

ArnoStrouwen
Copy link
Member

When https://github.com/domluna/JuliaFormatter.jl/blob/d89ab9e2638b9273b9d25321cbed5c9f22719e6d/src/styles/sciml/pretty.jl#L41 is changed to true.

I'll go through the repos and format the docstrings,
if that works for all the repos we can change it in the style?

@codecov
Copy link

codecov bot commented Jan 22, 2023

Codecov Report

Merging #130 (2533815) into master (37034a5) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #130   +/-   ##
=======================================
  Coverage   88.79%   88.79%           
=======================================
  Files           6        6           
  Lines         348      348           
=======================================
  Hits          309      309           
  Misses         39       39           
Impacted Files Coverage Δ
src/raphson.jl 96.66% <ø> (ø)
src/trustRegion.jl 98.30% <ø> (ø)
src/utils.jl 64.61% <ø> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@ChrisRackauckas
Copy link
Member

if that works for all the repos we can change it in the style?

Yes

@ChrisRackauckas ChrisRackauckas merged commit 7d09a34 into SciML:master Jan 22, 2023
@ArnoStrouwen ArnoStrouwen deleted the docstring branch January 22, 2023 13:00
avik-pal pushed a commit that referenced this pull request Nov 1, 2024
[skip ci] update downgrade CI repo
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants