Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
davidlion pushed 1 commit to port-lzma-compressor Bill-hbrhbr/clp
- Add missing fixes for PR suggestion. 68c4c36
davidlion pushed 1 commit to port-lzma-compressor Bill-hbrhbr/clp
- Apply suggestions from code review Co-authored-by: Bingran Hu <[email protected]> d139bd5
davidlion pushed 25 commits to port-lzma-compressor Bill-hbrhbr/clp
- feat(clp-package): Add support for deleting archives that are exclusively within a time range. (#594) Co-authored-by... b8e22da
- feat(clp-s): Add the write path for single-file archives. (#563) Co-authored-by: Devin Gibson <[email protected]... f1876cd
- test: Allow multiple trials when unittesting http headers (#613) Co-authored-by: Xiaochong Wei <xiaochong.wei@yscope... 12cdf6d
- chore(log-viewer-webui): Update `yscope-log-viewer` to the latest version. (#615) 5881b9c
- test(clp-s): Add end-to-end test case for compression and extraction. (#595) 88d83bf
- docs(clp-json): Update list of characters that requires escaping in queries. (#617) Co-authored-by: Kirk Rodrigues <... 9439d30
- feat(core): Add `ErrorCode` template to standardize conversion of user-defined error code enums to `std::error_code`.... ec0821d
- revert(core): Remove temporary output directory option from `clp` and `clo`. (#619) 5639dfc
- refactor(clp-package): Unify the metadata schema for JSON and IR streams. (#620) 60d85d0
- feat(clp-s): Add command line options for stubbed out kv-pair-IR ingestion. (#618) 2160362
- feat(ffi): Add initial implementation of `IrErrorCode` (using the `ErrorCode` template) which will replace the `IRErr... 7176c5e
- feat(ffi): Add support for auto/user-generated KV-pairs in `KeyValuePairLogEvent`; Detect and invalidate duplicate ke... ad56993
- build(docs): Update dependencies to latest versions. (#631) 6dd8fc1
- ci(pr-title-checks): Remove default GH workflow permissions and document risk of `pull_request_target` workflow trigg... 909e19e
- feat(core-clp): Add `BoundedReader` to prevent out-of-bound reads in segmented input streams. (#624) Co-authored-by:... ddba9b9
- build(core): Update Boost to v1.87.0 in order to pull in boost::urls; Replace calls to boost::asio's deprecated `expi... 9ced086
- refactor(clp-s): Replace instances of `std::string const&` with `std::string_view` where it would remove unnecessary ... 9b1a04b
- docs(core): Indicate dependency install scripts should be run with elevated privileges. (#637) 880a741
- feat(package)!: Add support for writing clp-s single file archives to S3. (#634) Co-authored-by: kirkrodrigues <2454... 37263eb
- fix(clp-package): Remove faulty error handling for parsing archive compression stats. (#640) a6b6222
- and 5 more ...
davidlion pushed 1 commit to port-lzma-compressor Bill-hbrhbr/clp
- Fix accidental comment reflow. fcfc73a
davidlion pushed 22 commits to port-lzma-compressor Bill-hbrhbr/clp
- feat(clp-package): Add support for deleting archives that are exclusively within a time range. (#594) Co-authored-by... 3dbe388
- feat(clp-s): Add the write path for single-file archives. (#563) Co-authored-by: Devin Gibson <[email protected]... 5d90054
- test: Allow multiple trials when unittesting http headers (#613) Co-authored-by: Xiaochong Wei <xiaochong.wei@yscope... 2b88c6f
- chore(log-viewer-webui): Update `yscope-log-viewer` to the latest version. (#615) 290ede3
- test(clp-s): Add end-to-end test case for compression and extraction. (#595) 2c0e053
- docs(clp-json): Update list of characters that requires escaping in queries. (#617) Co-authored-by: Kirk Rodrigues <... cbf8bf9
- feat(core): Add `ErrorCode` template to standardize conversion of user-defined error code enums to `std::error_code`.... 44b0f2b
- revert(core): Remove temporary output directory option from `clp` and `clo`. (#619) 4d21d9b
- refactor(clp-package): Unify the metadata schema for JSON and IR streams. (#620) 36892c1
- feat(clp-s): Add command line options for stubbed out kv-pair-IR ingestion. (#618) 604bd75
- feat(ffi): Add initial implementation of `IrErrorCode` (using the `ErrorCode` template) which will replace the `IRErr... 0a9322b
- feat(ffi): Add support for auto/user-generated KV-pairs in `KeyValuePairLogEvent`; Detect and invalidate duplicate ke... 78a535c
- build(docs): Update dependencies to latest versions. (#631) 42db88c
- ci(pr-title-checks): Remove default GH workflow permissions and document risk of `pull_request_target` workflow trigg... 13c7528
- feat(core-clp): Add `BoundedReader` to prevent out-of-bound reads in segmented input streams. (#624) Co-authored-by:... 8b34dac
- build(core): Update Boost to v1.87.0 in order to pull in boost::urls; Replace calls to boost::asio's deprecated `expi... 02d8956
- refactor(clp-s): Replace instances of `std::string const&` with `std::string_view` where it would remove unnecessary ... da66fbf
- docs(core): Indicate dependency install scripts should be run with elevated privileges. (#637) b7741c0
- feat(package)!: Add support for writing clp-s single file archives to S3. (#634) Co-authored-by: kirkrodrigues <2454... 1edc16e
- fix(clp-package): Remove faulty error handling for parsing archive compression stats. (#640) e4c9dd3
- and 2 more ...
davidlion created a review comment on a pull request on y-scope/clp
nit: Earlier we use an initializer list for similar code (auto-typed initialized local variable). Both styles are fine, but lets just make it consistent by having it match.
davidlion created a review on a pull request on y-scope/clp
Lets do a pass to try and: 1. Remove excessive function body comments 2. Add periods to complete sentences.
davidlion created a review on a pull request on y-scope/clp
Lets do a pass to try and: 1. Remove excessive function body comments 2. Add periods to complete sentences.
davidlion pushed 1 commit to main y-scope/clp
- feat(ffi): Add initial implementation of `IrErrorCode` (using the `ErrorCode` template) which will replace the `IRErr... 7176c5e
davidlion closed a pull request on y-scope/clp
feat(ffi): Add initial implementation of `IrErrorCode` (using the `ErrorCode` template) which will replace the `IRErrorCode` enum.
<!-- Set the PR title to a meaningful commit message that: - follows the Conventional Commits specification (https://www.conventionalcommits.org). - is in imperative form. Example: fix: Don't ...
davidlion opened an issue on y-scope/clp
Remove `NOLINT` pragmas from `ErrorCode` macro.
### Request In [error_handling/ErrorCode.hpp](https://github.com/y-scope/clp/blob/60d85d0e4d5521fb1fe69440ca463a129c41f04b/components/core/src/clp/error_handling/ErrorCode.hpp#L142) there is the f...
davidlion created a review comment on a pull request on y-scope/clp
```suggestion return "The decoding method failed."; ```
davidlion created a review comment on a pull request on y-scope/clp
Lets try to be a bit more descriptive for these messages, since right now they just repeat the enum name for the most part. Ideally, we want the user to have some idea on where to look to understan...
davidlion pushed 2 commits to cmake-utils davidlion/yscope-dev-utils
davidlion created a review comment on a pull request on y-scope/yscope-dev-utils
Good catch. It works for extract, but not create.
davidlion pushed 1 commit to cmake-utils davidlion/yscope-dev-utils
- Apply suggestions from code review. Co-authored-by: kirkrodrigues <[email protected]> 7223ea5
davidlion pushed 1 commit to cmake-utils davidlion/yscope-dev-utils
- Remove checksum usage from cmake tasks; fix some naming. 80d4773
davidlion created a review comment on a pull request on y-scope/yscope-dev-utils
Hmm good point... I'll look into it.
davidlion created a review comment on a pull request on y-scope/yscope-dev-utils
Sorry too asleep when I responded lol. We only want to compute the checksum for files we install, but a user might want to install to a file with other files. For example, say install prefix i...