Skip to content

feat: add code formatting check to CI workflow - #40

Merged
nomeguy merged 3 commits into
masterfrom
copilot/add-code-format-check-linter
Nov 1, 2025
Merged

feat: add code formatting check to CI workflow#40
nomeguy merged 3 commits into
masterfrom
copilot/add-code-format-check-linter

Conversation

Copilot AI commented Nov 1, 2025

Copy link
Copy Markdown
Contributor

Fix: #39

Copilot AI self-assigned this Nov 1, 2025
@nomeguy
nomeguy marked this pull request as ready for review November 1, 2025 07:13
@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

Co-authored-by: nomeguy <85475922+nomeguy@users.noreply.github.com>
Copilot AI changed the title [WIP] Add code format linter check in CI Add code formatting check to CI workflow Nov 1, 2025
Copilot AI requested a review from nomeguy November 1, 2025 07:18
@nomeguy

nomeguy commented Nov 1, 2025

Copy link
Copy Markdown

@copilot fix the new linter's error:

Run mix format --check-formatted
  mix format --check-formatted
  shell: /usr/bin/bash -e {0}
  env:
    INSTALL_DIR_FOR_OTP: /home/runner/work/_temp/.setup-beam/otp
    INSTALL_DIR_FOR_ELIXIR: /home/runner/work/_temp/.setup-beam/elixir
** (Mix) mix format failed due to --check-formatted.
The following files are not formatted:

  * lib/acx/persist/ecto_adapter.ex
  * test/persist/readonly_file_adapter_filtered_test.exs
  *

Co-authored-by: nomeguy <85475922+nomeguy@users.noreply.github.com>

Copilot AI commented Nov 1, 2025

Copy link
Copy Markdown
Contributor Author

@copilot fix the new linter's error:

...

Fixed in a9c1f5d. The formatting issues have been resolved:

  • Removed trailing whitespace in lib/acx/persist/ecto_adapter.ex
  • Fixed indentation and line breaks in test/persist/readonly_file_adapter_filtered_test.exs

All files now pass mix format --check-formatted.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • esm.ubuntu.com
    • Triggering command: /usr/lib/apt/methods/https (dns block)
  • packages.erlang-solutions.com
    • Triggering command: wget REDACTED (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

@nomeguy nomeguy changed the title Add code formatting check to CI workflow feat: add code formatting check to CI workflow Nov 1, 2025
@nomeguy
nomeguy merged commit e0115f5 into master Nov 1, 2025
3 of 4 checks passed
github-actions Bot pushed a commit that referenced this pull request Nov 1, 2025
# [1.5.0](v1.4.0...v1.5.0) (2025-11-01)

### Features

* add code formatting check to CI workflow ([#40](#40)) ([e0115f5](e0115f5))
@github-actions

github-actions Bot commented Nov 1, 2025

Copy link
Copy Markdown

🎉 This PR is included in version 1.5.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[feature] add code format linter check in CI

3 participants