bitflicker64 commented on code in PR #365: URL: https://github.com/apache/hugegraph-computer/pull/365#discussion_r4242761677
########## .github/help-wanted-lane-stake.md: ########## @@ -0,0 +1,8 @@ +# Help-wanted lane stake + +- Issue: https://github.com/apache/hugegraph-computer/issues/325 +- Actor: timerloggedout-spec (help-wanted execute) Review Comment: Minor: The only commit on this branch (1d04f88) is authored and committed as `timerloggedout-spec <41898282+github-actions[bot]@users.noreply.github.com>`. That email is the GitHub Actions bot address, so GitHub attributes the commit to `github-actions[bot]` instead of your account, while this line gives the actor as "timerloggedout-spec (help-wanted execute)". A commit proposed to an Apache repository should be attributable to the person contributing it. Requested change: when the real patch is pushed, author the commits with an email tied to your own GitHub account, and say in the description whether the change was produced by an automated workflow and how it was tested. ########## .github/help-wanted-lane-stake.md: ########## @@ -0,0 +1,8 @@ +# Help-wanted lane stake + +- Issue: https://github.com/apache/hugegraph-computer/issues/325 +- Actor: timerloggedout-spec (help-wanted execute) +- Delivery: upstream PR (PRIMARY) — stake only until real fix lands +- Fork: https://github.com/timerloggedout-spec/hugegraph-computer + +Replace this file with a real fix; do not merge stake-only as complete. Review Comment: Important: This file is the whole PR (1 file, +8/-0) and it is a claim marker, not a change to the project. Its last line says "do not merge stake-only as complete", and the description calls the commit "a stake ... not the final fix". Merging this head would add a contributor-specific note with a personal fork link to `.github/` on `master` and complete nothing in #325, whose open item that asks for code is the functional tests around `vermeer_test.go` (the four newcomer items are build, deploy, import and run steps). Your comment on #325 already records the claim, so no PR is needed for that. Requested change: remove this file and push the actual patch with the PR template filled in, or close this PR until there is one to review. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
