Skip to content

Comments

fix: ensure git diff output is properly encoded as UTF-8#27

Merged
billybonks merged 1 commit intoHyper-Unearthing:masterfrom
milarze:master
Mar 25, 2025
Merged

fix: ensure git diff output is properly encoded as UTF-8#27
billybonks merged 1 commit intoHyper-Unearthing:masterfrom
milarze:master

Conversation

@milarze
Copy link
Contributor

@milarze milarze commented Mar 25, 2025

Fixes #26

Copy link

@managerbot-app managerbot-app bot left a comment

Choose a reason for hiding this comment

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

👍🏿 👎🏿 review comments so that we know what works and what does not work

@milarze milarze force-pushed the master branch 2 times, most recently from 48bd7da to 1db5df4 Compare March 25, 2025 06:46
require 'committer/commit_generator'

RSpec.describe Committer::CommitGenerator do
let(:diff) { load_fixture('sample_diff.txt') }
Copy link
Contributor

Choose a reason for hiding this comment

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

i think sample diff is only used here so if you removed it we can delete the file

@milarze milarze requested a review from billybonks March 25, 2025 06:55
@billybonks billybonks merged commit 24a711e into Hyper-Unearthing:master Mar 25, 2025
2 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.

UTF-8 Characters in Diff causing errors

2 participants