Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
junhaoliao pushed 1 commit to main y-scope/clp-ffi-js
- ci: Add GH workflow for publishing to npm. (#43) beff6ae
junhaoliao closed a 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
Note this was moved from the default-exported function.
junhaoliao created a review comment on a pull request on y-scope/clp
@kirkrodrigues I don't recall if we discussed this for #468 - Did we have to export a function and change the `config.mode` dynamically? Can we simply define the `mode` in the above `config` objec...
junhaoliao pushed 1 commit to publish-npm junhaoliao/clp-ffi-js
- Remove "create-branch" workflow job. 57bebed
junhaoliao pushed 1 commit to publish-npm junhaoliao/clp-ffi-js
- Rename workflow to "publish-to-npm". b997a07
junhaoliao created a review comment on a pull request on y-scope/clp-ffi-js
Gotcha. i'll remove this branching job then
junhaoliao created a review comment on a pull request on y-scope/clp-ffi-js
Sounds good. Shall we also update the workflow `name` and filename to match this?
junhaoliao created a review comment on a pull request on y-scope/clp-ffi-js
I thought that was a strategy used by our java packages (e.g., logback, log4j1-appenders). With that said, since we cannot overpublish a version on npm, maybe we don't need to create a branch fo...
junhaoliao pushed 1 commit to publish-npm junhaoliao/clp-ffi-js
- Remove `name` for command "task package". f608eeb
junhaoliao pushed 1 commit to publish-npm junhaoliao/clp-ffi-js
- Specify file upload retention day and no file found resolution - Apply suggestions from code review Co-authored-by: ... 42f4550
junhaoliao created a review comment on a pull request on y-scope/clp-ffi-js
if the purpose of the name is to add comments, that just repeat the command. right, we don't need it
junhaoliao pushed 1 commit to publish-npm junhaoliao/clp-ffi-js
- Use `--global` instead of `-g` in `npm install` - Apply suggestions from code review Co-authored-by: kirkrodrigues <... 7713b9f
junhaoliao created a review comment on a pull request on y-scope/clp
compared to the one in yscope-log-viewer, removed ``` "DOM.Iterable", "WebWorker", ```
junhaoliao created a review comment on a pull request on y-scope/clp
compared to the one in yscope-log-viewer, removed ``` "jest.config.ts", "test", ```
junhaoliao created a review comment on a pull request on y-scope/clp
Everything was taken from https://github.com/y-scope/yscope-log-viewer/blob/7e6073f7c996f55721f41088ac2d8506085981ad/tsconfig.json except noted below.
junhaoliao pushed 1 commit to log-viewer-webui-client-ts junhaoliao/clp
- Rename `define-process-env-node-env` -> `config-node-env` in webpack `build` script. 3444981
junhaoliao created a review comment on a pull request on y-scope/clp
The `--define-process-env-node-env` is renamed as `--config-node-env` with no documentation (not even in the changelog; their manual still reports the option. ) Reported to the webpack-cli repo:...
junhaoliao opened an issue on webpack/webpack-cli
Removal of `--define-process-env-node-env` is undocumented
**Describe the bug** <!-- A 💯 way to do this is to provide your configuration via a GitHub gist and/or to use markdown when describing which commands you used when the error occurred! Be precise...
junhaoliao created a comment on a pull request on webpack/webpack-cli
> Hello, > > I've just realized one of our production scripts was broken because of this change, after upgrading to 6.x a couple of days ago. > > What do you think about adding this as a BREA...