Skip to content

Conversation

@vsoch
Copy link
Member

@vsoch vsoch commented Jan 8, 2026

We should add back the more explicit list of emails and admin contacts. The team link we initially created will always be private to org members only. The link (as it is now) if you test not logged into GitHub is 404:

https://flux-framework.readthedocs.io/projects/flux-rfc/en/latest/spec_47.html

-> https://github.com/orgs/flux-framework/teams/flux-code-of-conduct-contacts (404)

@trws is the two of us for emails sufficient? I remember you saying you were OK with that when we realized teams weren't an option. I'm definitely OK with having mine there. 👍

@github-actions
Copy link

github-actions bot commented Jan 8, 2026

⚠️ linkcheck failed with status code 2

Copy link
Member

@garlick garlick left a comment

Choose a reason for hiding this comment

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

IMHO the reference to the flux administrators page should be sufficient.
People can add their email addresses to their github profiles if that's how they prefer to be contacted, or they can choose other methods.

@vsoch
Copy link
Member Author

vsoch commented Jan 8, 2026

@garlick how should I be added there? I'm not a Flux admin but I should be a code of conduct contact (in case a woman is wanted for the contact).

@garlick
Copy link
Member

garlick commented Jan 8, 2026

Oh sorry, well I guess my main point was just that github identities might be better than emails?

I can add you to the org admin page as a CoC contact and then this doc can just point there?
At least then we have one page to keep updated rather than two.

@vsoch
Copy link
Member Author

vsoch commented Jan 8, 2026

Yep that's perfect! I'll update the PR here to point to that.

@github-actions
Copy link

github-actions bot commented Jan 8, 2026

⚠️ linkcheck failed with status code 2

Copy link
Member

@garlick garlick left a comment

Choose a reason for hiding this comment

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

LGTM! If you could please squash that commit into the first one we can get this in.

We should add back the more explicit list of emails
and admin contacts. The team link we initially created
will always be private to org members only.

Signed-off-by: vsoch <vsoch@users.noreply.github.com>
@github-actions
Copy link

github-actions bot commented Jan 8, 2026

⚠️ linkcheck failed with status code 2

@mergify mergify bot added the queued label Jan 8, 2026
@mergify mergify bot merged commit 8485d2b into flux-framework:master Jan 8, 2026
7 checks passed
@mergify
Copy link
Contributor

mergify bot commented Jan 8, 2026

Merge Queue Status

✅ The pull request has been merged at 09d373f

This pull request spent 6 seconds in the queue, with no time running CI.
The checks were run in-place.

Required conditions to merge
  • any of [🛡 GitHub branch protection]:
    • check-success = docs/readthedocs.org:flux-rfc
    • check-neutral = docs/readthedocs.org:flux-rfc
    • check-skipped = docs/readthedocs.org:flux-rfc
  • any of [🛡 GitHub branch protection]:
    • check-success = make check
    • check-neutral = make check
    • check-skipped = make check
  • any of [🛡 GitHub branch protection]:
    • check-success = validate commits
    • check-neutral = validate commits
    • check-skipped = validate commits

@mergify mergify bot removed the queued label Jan 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants