Skip to content

Comments

Add file.read_binary#928

Merged
hulto merged 4 commits intomainfrom
read-binary
May 6, 2025
Merged

Add file.read_binary#928
hulto merged 4 commits intomainfrom
read-binary

Conversation

@jabbate19
Copy link
Collaborator

What type of PR is this?

/kind documentation
/kind feature
/kind eldritch-function

What this PR does / why we need it:

Adds file.read_binary to present file contents as a list of bytes (int).

image image

Which issue(s) this PR fixes:

Fixes #865

@jabbate19 jabbate19 requested a review from hulto May 4, 2025 20:21
@jabbate19
Copy link
Collaborator Author

Test failing is not related to my PR, appears to be something about Windows permissions

Copy link
Collaborator

@hulto hulto left a comment

Choose a reason for hiding this comment

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

Hell yeah!

@hulto hulto merged commit 5811611 into main May 6, 2025
4 checks passed
@hulto hulto deleted the read-binary branch May 6, 2025 03:43
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.

Eldritch Implement file.read_binary

2 participants