Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
Henry8192 created a review comment on a pull request on y-scope/clp-ffi-js
I think it is already included on line 9
coderabbitai[bot] created a review comment on a pull request on y-scope/clp-ffi-js
_:hammer_and_wrench: Refactor suggestion_ **Add package validation steps before publishing** Consider adding validation steps before publishing to ensure package integrity. ```diff + - nam...
coderabbitai[bot] created a review comment on a pull request on y-scope/clp-ffi-js
_:hammer_and_wrench: Refactor suggestion_ **Consider using LTS version of Node.js** Node.js 22 is currently in development. For production publishing workflows, it's recommended to use an LTS ver...
coderabbitai[bot] created a review on a pull request on y-scope/clp-ffi-js
**Actionable comments posted: 2** <details> <summary>🧹 Nitpick comments (4)</summary><blockquote> <details> <summary>package.json (1)</summary><blockquote> `16-16`: **Consider enhancing the rele...
coderabbitai[bot] created a review on a pull request on y-scope/clp-ffi-js
**Actionable comments posted: 2** <details> <summary>🧹 Nitpick comments (4)</summary><blockquote> <details> <summary>package.json (1)</summary><blockquote> `16-16`: **Consider enhancing the rele...
coderabbitai[bot] created a comment on a pull request on y-scope/clp-ffi-js
<!-- This is an auto-generated comment: summarize by coderabbit.ai --> <!-- This is an auto-generated comment: skip review by coderabbit.ai --> > [!IMPORTANT] > ## Review skipped > > Draft detect...
junhaoliao opened a draft pull request on y-scope/clp-ffi-js
ci: Add GH workflow for publishing to npm.
<!-- 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 ...
junhaoliao created a review comment on a pull request on y-scope/clp-ffi-js
Let's `#include <clp/ir/types.hpp>` so that the header providing the type can be directly included.
junhaoliao created a review comment on a pull request on y-scope/clp-ffi-js
Let's `#include <clp/ir/types.hpp>` so that the header providing the type can be directly included.
junhaoliao created a review comment on a pull request on y-scope/clp-ffi-js
For future commits, you may follow https://github.com/y-scope/clp-ffi-js?tab=readme-ov-file#linting to run the linter
junhaoliao created a review comment on a pull request on y-scope/clp-ffi-js
Let's deduplicate this with `StructuredIrStreamReader::find_timestamp_last_occurrence`. You may refer to `generic_decode_range`.
junhaoliao created a review comment on a pull request on y-scope/clp-ffi-js
Can we use `std::range::upper_bound` instead and always `it--` after, so that we can avoid checking `(it == m_deserialized_log_events->end() || it->get_timestamp() > input_timestamp)`?
junhaoliao created a review comment on a pull request on y-scope/clp-ffi-js
Is it possible to check against `m_deserialized_log_events->empty()` at the beginning of the method and early return, so that we can avoid checking `it == m_deserialized_log_events->end() && it == ...
junhaoliao created a review on a pull request on y-scope/clp-ffi-js
For future commits, you may follow https://github.com/y-scope/clp-ffi-js?tab=readme-ov-file#linting to run the linter
coderabbitai[bot] created a comment on a pull request on y-scope/clp-ffi-js
<!-- This is an auto-generated comment: summarize by coderabbit.ai --> <!-- This is an auto-generated comment: skip review by coderabbit.ai --> > [!IMPORTANT] > ## Review skipped > > Draft detect...
Henry8192 opened a draft pull request on y-scope/clp-ffi-js
feat: Support search logs by timestamp for structured and unstructured logs.
<!-- 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 ...
junhaoliao pushed 1 commit to main y-scope/clp-ffi-js
- chore: Update version for 0.3.1 release. (#41) 32b1220
junhaoliao closed a pull request on y-scope/clp-ffi-js
chore: Update version for 0.3.1 release.
<!-- 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 ...
junhaoliao pushed 1 commit to v0.3.1 y-scope/clp-ffi-js
- chore: Update version for 0.3.1 release. dc3b45e
coderabbitai[bot] created a comment on a pull request on y-scope/clp-ffi-js
<!-- This is an auto-generated comment: summarize by coderabbit.ai --> <!-- This is an auto-generated comment: review in progress by coderabbit.ai --> > [!NOTE] > Currently processing new changes ...
junhaoliao opened a pull request on y-scope/clp-ffi-js
Update version for 0.3.1 release.
<!-- 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 ...