Skip to content

elevated-access - #1600

Open
stevepiercy wants to merge 10 commits into
mainfrom
elevated-access
Open

elevated-access#1600
stevepiercy wants to merge 10 commits into
mainfrom
elevated-access

Conversation

@stevepiercy

Copy link
Copy Markdown
Member

Linked issue

n/a

Description

Documented how to join the icalendar team.

Checklist

  • I added a change log entry, following the instructions in Change log entry format.
  • I followed icalendar's Artificial intelligence policy and disclosed my Responsible AI use in my commit messages, if applicable.
  • I added or updated tests, if applicable.
  • I ran and ensured all tests pass locally by following Run tests.
  • I added or edited documentation as necessary, both as docstrings to be rendered in the API documentation and narrative documentation, following the Style guide.

@github-actions

Copy link
Copy Markdown
Contributor

Profile summary:

GitHub user: stevepiercy
🟢 No concerns found with user's profile.
🟢 No concerns found with recent PR activity.
🟢 No concerns found with recent issue activity.

For a more detailed report, run `gh-profiler stevepiercy`.
Full profile
GitHub user: stevepiercy
🟢 No concerns found with user's profile.
   🟢 Account age: 17 years
   🟢 Profile information:
        name: Steve Piercy
        company: Steve Piercy - Website Builder
        blog: https://stevepiercy.com/
        location: Eugene, OR
        bio: Contributor to open source projects icalendar, Plone, Pyramid, and Deform. Everyday cyclist. Backyard farmer. Tree hugger. Living simply.
        mastodon: https://fosstodon.org/@stevepiercy
      Empty fields: email
      Orgs: plone, collective, Pylons, zopefoundation, knop-project, pycalendar

🟢 No concerns found with recent PR activity.
   14 PRs opened in the last 21 days.
      0 opened against repos the user owns.
      14 opened against repos in publicly associated orgs.
      0 opened against external repos.

🟢 No concerns found with recent issue activity.
   10 new issues opened in the last 21 days.
      0 opened in repos the user owns.
      8 opened in repos in publicly associated orgs.
      2 opened in external repos.

   🟢 0 external issues closed as NOT_PLANNED.
   🟢 0 external issues opened with the same title.

@read-the-docs-community

read-the-docs-community Bot commented Jul 26, 2026

Copy link
Copy Markdown

@coveralls

coveralls commented Jul 26, 2026

Copy link
Copy Markdown
Collaborator

Coverage Status

coverage: 97.998%. remained the same — elevated-access into main

@SashankBhamidi

Copy link
Copy Markdown
Member

Hi, I'll make some time to review this on Wednesday.

@niccokunzmann niccokunzmann left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I added my thoughts and will have a look at the document again.


Additionally, the icalendar team may contact you to encourage you to apply.

The icalendar team recognizes that many individuals might not feel qualified to apply. Please rest assured that the worst that can happen is that the team will say "not yet, but here's a path you can follow to become qualified." The icalendar team wants to lift people up.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we say "we" instead of icalendar team?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The Microsoft Style Guide recommends avoiding "we" to focus the subject on the reader, not the product. However, that's a different context.

In this context, "We" may have a negative connotation of exclusivity, or "us, not you, the reader". If you're not a member of "we", then that can be a turn off. At other times, it may include the reader, making it unclear who "we" consists of. I could explicitly state "'We' in this document refers to the icalendar team." but I prefer to explicitly state the subject. It also reinforces that we're a team.

Comment thread docs/contribute/join-the-icalendar-team.rst
Comment thread docs/contribute/join-the-icalendar-team.rst

The icalendar team will review your application, and collect relevant information for evaluation.

If the icalendar team determines that you qualify for elevated access, then you'll receive an invitation through an email notification from GitHub to accept.

@niccokunzmann niccokunzmann Jul 28, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We have the maintenance page - we do not need to be a faceless team but people can know who is the team - should that be restructured then?

A few friendly faces with roles.
Hi, we are the team behind icalendar.
Join us... Here is how...

Then three tabs

Review Triage Maintain
What we really need is review work. We get a lot of contributions and this helps scale up. When you review, you show that you know how to be a team player, not just a coder. Issues need to be checked and PRs merged. Once you showed that you can review, we can trust you with your judgement on issues and PRs. You know how to cooperate. It is always good for the project to have a few maintainers.... ????

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maintenance is written primarily for people who are interested in maintaining icalendar. This page's purpose is to describe how to join the icalendar team. Let's not mix their purposes.

I could add a section about who are members of the icalendar team, which coincidentally are those people listed on Maintenance.

For the tabs, Review can be done by anyone with a GitHub account via Read access. Maintainers is outside the scope of this PR, and needs a separate discussion about who may become one. I focused on Triage and getting people to join the icalendar team. In another PR, we could amend this page with how to be promoted to Maintainer, once we determine those qualifications.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added a Members section in b7dd62a

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I should remark that GitHub has shitty permission management for FOSS organizations. Access may be granted on the entire organization and all its repos, but not a singular repo. Instead we must use direct individual access.

We can also create an organization team, such as icalendar-triage or icalendar-maintainers, and add members to each team, but that won't change their individual access. It's only practical use is to tag a team or request a review from a team, such as @icalendar-maintainers`, which would send a notification to its members.

The icalendar team reviews pull requests and issues for quality, makes releases to PyPI, and protects and promotes the icalendar community.
To familiarize yourself with the icalendar community, read :doc:`index`.

To encourage collaboration and community, the icalendar team is actively seeking contributors who are interested in elevated access to icalendar to help review and triage issues and pull requests.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would say we are seeking people who are interested in doing the work. I do not really want those who seek the access as primary motivation.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done in 94d6c2c

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants