Sohei Okamoto

Results 9 comments of Sohei Okamoto

Thanks @pedrosousa for updating PR description. I have updated the commit message as well. Removed `bharatnc` for now, as this user has not joined Cloudflare contributor group yet.

Rebased and regenerated `index.js` files.

Since @bharatnc is now a contributor and recognized here, added him for both projects. Regenerated index.js. Ready to merge and test.

It seems like now we are all pulled into all PRs: https://github.com/cloudflare/cloudflare-docs/pulls?q=is%3Apr+user-review-requested%3Asoheiokamoto

This is the issue: https://github.com/cloudflare/cloudflare-docs/blob/b603dedb12e2cf71b7df858f190ac5c781902b61/.github/actions/assign-pr/index.ts#L75-L102 Everyone in owners.ts is treated as PCX authors, and whenever one of this PCX owner opens a PR, everyone (up to 10) is assigned.

What do you think of a fix like this? https://github.com/cloudflare/cloudflare-docs/pull/5612 If not, then we should revert this commit.

Can we have this merged? Waiting for this to be merged, then will update https://github.com/cloudflare/cloudflare-docs/pull/16352

I will wait for https://github.com/cloudflare/cloudflare-docs/pull/15964 to be merged, then resolve conflict.

All feedback addressed. Rebased, and went over the curl commands again.