Ecosyste.ms: Timeline

Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.

davidlion

davidlion created a review on a pull request on y-scope/ystdlib-cpp

View on GitHub

davidlion created a review on a pull request on y-scope/ystdlib-cpp

View on GitHub

davidlion created a review comment on a pull request on y-scope/ystdlib-cpp
You're missing the `.github` directory.

View on GitHub

davidlion created a review comment on a pull request on y-scope/ystdlib-cpp
afaik only `yamllint>=1.35.1` is necessary for YAML linting, lets include the others as we go (some of these don't even seem related to linting at all).

View on GitHub

davidlion created a review on a pull request on y-scope/ystdlib-cpp
On top of the comments lets make 2 changes: 1. rename the directory `tasks/` to `taskfiles/` 2. drop the `-tasks` suffix inside this directory (e.g. rename `lint-tasks.yaml` to `lint.yaml`)

View on GitHub

davidlion created a review on a pull request on y-scope/ystdlib-cpp
On top of the comments lets make 2 changes: 1. rename the directory `tasks/` to `taskfiles/` 2. drop the `-tasks` suffix inside this directory (e.g. rename `lint-tasks.yaml` to `lint.yaml`)

View on GitHub

davidlion pushed 1 commit to main y-scope/ystdlib-cpp
  • chore: Add `yscope-dev-utils` submodule for future task workflows. (#8) ef9ad7b

View on GitHub

davidlion closed a pull request on y-scope/ystdlib-cpp
chore: Add `yscope-dev-utils` submodule for future task workflows.
<!-- 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 created a review on a pull request on y-scope/ystdlib-cpp

View on GitHub

davidlion pushed 1 commit to main y-scope/ystdlib-cpp
  • chore: Add GitHub issue templates. (#7) b0a4644

View on GitHub

davidlion closed a pull request on y-scope/ystdlib-cpp
chore: Add GitHub issue templates.
<!-- 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 created a review on a pull request on y-scope/ystdlib-cpp

View on GitHub

davidlion created a review comment on a pull request on y-scope/ystdlib-cpp
```suggestion ``` My suggestion had an extra space by accident sorry.

View on GitHub

davidlion created a review on a pull request on y-scope/ystdlib-cpp

View on GitHub

davidlion created a review comment on a pull request on y-scope/ystdlib-cpp
```suggestion Use this form to report any functional or performance bugs you've found in the software. Be sure to check if your [issue](https://github.com/y-scope/ystdlib-cpp/is...

View on GitHub

davidlion created a review on a pull request on y-scope/ystdlib-cpp

View on GitHub

davidlion created a review comment on a pull request on y-scope/ystdlib-cpp
```suggestion been reported. - type: "textarea" ```

View on GitHub

davidlion created a review comment on a pull request on y-scope/ystdlib-cpp
```suggestion Use this form to request a feature/change in the software, or the project as a whole. - type: "textarea" ```

View on GitHub

davidlion created a review on a pull request on y-scope/ystdlib-cpp

View on GitHub

davidlion pushed 1 commit to main y-scope/ystdlib-cpp
  • ci: Add GH workflow to validate PR titles following Conventional Commits and PR template. (#6) b01b40d

View on GitHub

davidlion closed a pull request on y-scope/ystdlib-cpp
ci: Add GH workflow to validate PR titles following Conventional Commits and PR template.
# Description Add PR-related format templates and validations based on the current practice in [clp](https://github.com/y-scope/clp/blob/1ecd9c75a8c59b45673cc97d131f85e4d796635a/.github/workflows/...
davidlion created a review on a pull request on y-scope/ystdlib-cpp

View on GitHub

davidlion closed a pull request on y-scope/clp-ffi-go
Add support for UTC offset serialization/deserialization.
# References <!-- Any issues or pull requests relevant to this pull request --> # Description <!-- Describe what this request will change/fix and provide any details necessary for reviewers --...
davidlion created a review on a pull request on y-scope/yscope-docs

View on GitHub

davidlion created a review on a pull request on y-scope/yscope-docs

View on GitHub

davidlion created a review on a pull request on y-scope/yscope-docs

View on GitHub

davidlion created a review comment on a pull request on y-scope/yscope-docs
Should we encourage people to return a result here? If someone explicitly follows this and returns an object you can't do much better error handling than a constructor, no?

View on GitHub

davidlion created a review comment on a pull request on y-scope/yscope-docs
Do we want to add a link to open a new issue? Maybe overkill, but could be nice.

View on GitHub

davidlion created a review comment on a pull request on y-scope/yscope-docs
I feel there may be a chance someone unfamiliar with c++ may not understand "non-header files meant for inclusion" and the relationship to "self-contained" headers. Maybe we could add a bracket exa...

View on GitHub

Load more