Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: update CODEOWNERS #984

Closed
wants to merge 2 commits into from
Closed

chore: update CODEOWNERS #984

wants to merge 2 commits into from

Conversation

moul
Copy link
Member

@moul moul commented Jul 21, 2023

See title.

Depends on #1176

Checklists...

Contributors Checklist

  • Added new tests, or not needed, or not feasible
  • Provided an example (e.g. screenshot) to aid review or the PR is self-explanatory
  • Updated the official documentation or not needed
  • No breaking changes were made, or a BREAKING CHANGE: xxx message was included in the description
  • Added references to related issues and PRs
  • Provided any useful hints for running manual tests
  • Added new benchmarks to generated graphs, if any. More info here.

Maintainers Checklist

  • Checked that the author followed the guidelines in CONTRIBUTING.md
  • Checked the conventional-commit (especially PR title and verb, presence of BREAKING CHANGE: in the body)
  • Ensured that this PR is not a significant change or confirmed that the review/consideration process was appropriate for the change

@moul moul requested a review from jaekwon as a code owner July 21, 2023 17:44
@moul moul requested a review from a team July 21, 2023 17:45
@thehowl
Copy link
Member

thehowl commented Jul 21, 2023

judging from the error, group needs write access to repo

@moul moul marked this pull request as draft July 21, 2023 21:18
@mvertes
Copy link
Contributor

mvertes commented Jul 29, 2023

What is the purpose of the @gnolang/devrels ?

@moul
Copy link
Member Author

moul commented Jul 29, 2023

The @gnolang/devrels team is all about making developers happy and eager to use our product.

Right now, we're focusing mainly on developers. We can split what we need to do into two groups: building stuff and everything else. 'Everything else' includes writing documentation, giving support, putting on presentations and events, and handling marketing. The core team, which you're a part of, handles the building part. Meanwhile, the devrel team takes care of the 'everything else,' either directly or indirectly.

What's special about Gno is that we also expect our devrel team to get their hands dirty with code. This includes creating examples or tools and helping review PRs. While the core team members are busy squashing bugs, the devrel team can take the time to dive deep and figure out why a developer ran into a problem in the first place.

@moul
Copy link
Member Author

moul commented Aug 16, 2023

I suggest we consider a tool similar to this: component-owners. However, its current version doesn't support "team" functionality. This tool was discussed here.

Given that it's based on YML, perhaps we can directly incorporate team support (with advanced YML features). This could streamline our management of contributors, reviewers, and related processes through commits.

@moul moul added good first issue Good for newcomers help wanted Extra attention is needed labels Aug 16, 2023
@moul moul added this to the 💡Someday/Maybe milestone Sep 6, 2023
@moul moul changed the title chore: add @devrels as co-codeowners of examples and docs chore: update CODEOWNERS Oct 11, 2023
@moul
Copy link
Member Author

moul commented Oct 11, 2023

I will soon revive this issue, not just for devrels, but for the entire project and its subprojects.

Depends on #1176.

@moul
Copy link
Member Author

moul commented Feb 16, 2024

Closing in favor of more recent changes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers help wanted Extra attention is needed
Projects
Status: Done
Status: ✅ Done
Status: 🔵 Not Needed for Launch
Archived in project
Development

Successfully merging this pull request may close these issues.

4 participants