Skip to content

Add Julia formatting to CI workflow#752

Merged
K20shores merged 3 commits intomainfrom
copilot/add-julia-formatting
Jan 28, 2026
Merged

Add Julia formatting to CI workflow#752
K20shores merged 3 commits intomainfrom
copilot/add-julia-formatting

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Jan 28, 2026 •
edited
Loading

Adds automatic formatting for Julia code and its C++ bindings to the existing format.yml workflow.

Changes

  • Julia formatting: Added JuliaFormatter.jl step to format all .jl files in the julia/ directory
  • C++ bindings consolidation: Extended clang-format step to cover C++ bindings in both julia/ and javascript/ directories
  • package.json cleanup: Removed format:cpp script; C++ formatting now unified in workflow

Workflow modifications

- name: Set up Julia
  uses: julia-actions/setup-julia@v2
  with:
    version: '1'

- name: Install JuliaFormatter
  run: julia -e 'using Pkg; Pkg.add("JuliaFormatter")'

- name: Format Julia
  run: julia -e 'using JuliaFormatter; format("julia")'

The clang-format step now processes C++ files in include/, src/, python/, javascript/, and julia/ directories.

Original prompt

This section details on the original issue you should resolve

<issue_title>Add Julia Formatting</issue_title>
<issue_description>Now that we have a julia package we should include automatic formatting with github actions

Acceptance criteria

  • Update format.yml to include a step to format the julia code
  • Add a line in clang-format step to format the julia c++ bindings
  • Remove "format:cpp" from package.json and add it to the clang-format step to format the javascript c++ bindings

Ideas

Comments on the Issue (you are @copilot in this section)

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI changed the title [WIP] Add automatic formatting for Julia code Add Julia formatting to CI workflow Jan 28, 2026
Copilot AI requested a review from K20shores January 28, 2026 14:21
@K20shores K20shores marked this pull request as ready for review January 28, 2026 15:03
@K20shores K20shores requested a review from boulderdaze January 28, 2026 15:03
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Jan 28, 2026 •
edited
Loading

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 74.63%. Comparing base (2c13cdb) to head (a10c82f).

Additional details and impacted files 
@@           Coverage Diff           @@
##             main     #752   +/-   ##
=======================================
  Coverage   74.63%   74.63%           
=======================================
  Files         109      109           
  Lines        8106     8106           
=======================================
  Hits         6050     6050           
  Misses       2056     2056
Flag Coverage Δ
cpp_fortran 68.49% <ø> (ø)
javascript 92.56% <ø> (ø)
python 78.47% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@K20shores K20shores merged commit 4fa4eb4 into main Jan 28, 2026
49 of 50 checks passed
@K20shores K20shores deleted the copilot/add-julia-formatting branch January 28, 2026 19:03
@Katetc Katetc mentioned this pull request Feb 25, 2026
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@boulderdaze boulderdaze boulderdaze approved these changes

+1 more reviewer

@K20shores K20shores K20shores approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@K20shores K20shores

Copilot code review Copilot

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Add Julia Formatting

4 participants

@codecov-commenter @K20shores @boulderdaze