Skip to content

chore: don't declare dist as binary#9

Open
sparten11740 wants to merge 1 commit intomainfrom
sparten11740/chore/dont-declare-dist-as-binary
Open

chore: don't declare dist as binary#9
sparten11740 wants to merge 1 commit intomainfrom
sparten11740/chore/dont-declare-dist-as-binary

Conversation

@sparten11740
Copy link
Collaborator

There's a bug in git 2.46.x preventing git diff --exit-code or git diff --quiet exiting with a non-zero exit code when there is a diff in a file declared as binary in .gitattributes. This will effectively disable our CI check to make sure that the action has been built and committed.

@sparten11740 sparten11740 self-assigned this Sep 20, 2024
@mvayngrib
Copy link
Contributor

to clarify, removing this fixes the git diff exit code, right?

@sparten11740
Copy link
Collaborator Author

Yes, correct. I also reported the bug in the mailing list but haven't heard anything so far

@mvayngrib
Copy link
Contributor

could u link that mailing list thread?

@sparten11740
Copy link
Collaborator Author

sparten11740 commented Sep 25, 2024

Looks like they don't want my bug report 🤔

@mvayngrib
Copy link
Contributor

i see you have a reputation outside Exodus too 😉

@sparten11740
Copy link
Collaborator Author

i see you have a reputation outside Exodus too 😉

hahaha, unfortunately yes

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.

2 participants