Add file watcher and checksum functionality #4
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Add FileWatcher to monitor files and directories for changes - Add WatchRecursive for recursive directory monitoring - Implement Checksum function for file hash calculation with multiple algorithms - Add enum package for EventType and HashAlg enumerations - Update CI workflow to use latest golangci-lint - Add comprehensive documentation and examples - Fix code to comply with linter requirements
fileutils.go
Outdated
| } | ||
|
|
||
| // Checksum calculates the checksum of a file using the specified hash algorithm. | ||
| // Supported algorithms are MD5, SHA1, and SHA256. |
There was a problem hiding this comment.
The comment for the Checksum function lists only MD5, SHA1, and SHA256 as supported algorithms, but the implementation supports additional algorithms. Please update the comment to accurately reflect all supported hash algorithms.
Suggested change
| // Supported algorithms are MD5, SHA1, and SHA256. | |
| // Supported algorithms are MD5, SHA1, SHA224, SHA256, SHA384, SHA512, SHA512_224, and SHA512_256. |
danyel
pushed a commit
to danyel/fileutils
that referenced
this pull request
Oct 29, 2025
This caused golangci-lint config validation errors Add file watcher and checksum functionality (go-pkgz#4) * Add file watcher and checksum functionality - Add FileWatcher to monitor files and directories for changes - Add WatchRecursive for recursive directory monitoring - Implement Checksum function for file hash calculation with multiple algorithms - Add enum package for EventType and HashAlg enumerations - Update CI workflow to use latest golangci-lint - Add comprehensive documentation and examples - Fix code to comply with linter requirements * Fix test race condition in TestWatchRecursive * Allow coveralls step to fail without failing the build * Replace if-else-if chain with switch statement * Fix formatting issue * Update Checksum function comment to list all supported hash algorithms feature: MkDir added MkDir as a method on the fileutils feature
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary