Skip to content

Conversation

halfdan
Copy link
Contributor

@halfdan halfdan commented Mar 6, 2025

closes #233

@coveralls
Copy link

Coverage Status

coverage: 89.524% (-0.05%) from 89.575%
when pulling 736df9b on halfdan:fb/formatting
into 9b68b6c on parroty:master.

@parroty parroty requested a review from Copilot March 16, 2025 11:16
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

The PR adds CI steps to enforce code formatting and updates configuration in the matrix entries.

  • Changed quotes for otp and elixir version numbers to double quotes
  • Added a new "lint" parameter to the matrix for the formatting check
  • Introduced new CI steps for dependency installation, compilation, formatting check, and tests
Files not reviewed (19)
  • .formatter.exs: Language not supported
  • config/config.exs: Language not supported
  • config/test.exs: Language not supported
  • lib/exvcr.ex: Language not supported
  • lib/exvcr/actor.ex: Language not supported
  • lib/exvcr/adapter.ex: Language not supported
  • lib/exvcr/adapter/finch.ex: Language not supported
  • lib/exvcr/adapter/hackney.ex: Language not supported
  • lib/exvcr/adapter/hackney/converter.ex: Language not supported
  • lib/exvcr/adapter/hackney/store.ex: Language not supported
  • lib/exvcr/adapter/httpc.ex: Language not supported
  • lib/exvcr/adapter/httpc/converter.ex: Language not supported
  • lib/exvcr/adapter/ibrowse.ex: Language not supported
  • lib/exvcr/adapter/ibrowse/converter.ex: Language not supported
  • lib/exvcr/checker.ex: Language not supported
  • lib/exvcr/config.ex: Language not supported
  • lib/exvcr/config_loader.ex: Language not supported
  • lib/exvcr/converter.ex: Language not supported
  • lib/exvcr/exceptions.ex: Language not supported

Tip: If you use Visual Studio Code, you can request a review from Copilot before you push from the "Source Control" tab. Learn more

@parroty parroty merged commit 196228b into parroty:master Mar 16, 2025
11 of 13 checks passed
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.

Ensure code formatting using mix format --check-formatted
3 participants