Skip to content

doc/CoC: Remove references to communication tools #20647

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
May 6, 2024
Merged

doc/CoC: Remove references to communication tools #20647

merged 1 commit into from
May 6, 2024

Conversation

maribu
Copy link
Member

@maribu maribu commented May 3, 2024

Contribution description

The list of communication tools has been outdated. Rather than keeping this list up to date, the list was dropped and the wording changed so that we refer to communication within the community.

Testing procedure

Read the CoC and confirm that this:

  1. Does not change the meaning of the CoC. Especially: It should still be obvious that it applies to communication via all the channels and tools we use
  2. It does no longer contain references to specific channels or tools, so that we can evolve them without having to keep the CoC in sync

Issues/PRs references

None

@maribu
doc/CoC: Remove references to communication tools
314e31b 
The list of communication tools has been outdated. Rather than keeping
this list up to date, the list was dropped and the wording changed
so that we refer to communication within the community.
@maribu maribu added Area: doc Area: Documentation Impact: minor The PR is small in size and might only require a quick look of a knowledgeable reviewer CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Process: needs >1 ACK Integration Process: This PR requires more than one ACK labels May 3, 2024
@maribu maribu requested a review from miri64 May 3, 2024 08:51
@maribu maribu requested a review from jia200x as a code owner May 3, 2024 08:51
@github-actions github-actions bot added the Process: missing approvals Integration Process: PR needs more ACKS (handled by action) label May 3, 2024
@riot-ci
Copy link

riot-ci commented May 3, 2024

Murdock results

✔️ PASSED

314e31b doc/CoC: Remove references to communication tools

Success Failures Total Runtime
1 0 1 01m:19s

Artifacts

mguetschow
mguetschow approved these changes May 3, 2024
View reviewed changes
Copy link
Contributor

@mguetschow mguetschow left a comment

Choose a reason for hiding this comment

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

Thanks, I can confirm both points of the testing procedure.

dylad
dylad approved these changes May 6, 2024
View reviewed changes
Copy link
Member

@dylad dylad left a comment

Choose a reason for hiding this comment

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

LGTM.

@maribu maribu enabled auto-merge May 6, 2024 07:38
@github-actions github-actions bot removed the Process: missing approvals Integration Process: PR needs more ACKS (handled by action) label May 6, 2024
@maribu maribu added this pull request to the merge queue May 6, 2024
Merged via the queue into RIOT-OS:master with commit 622c433 May 6, 2024
@maribu maribu deleted the doc/CoC branch May 6, 2024 16:20
@maribu
Copy link
Member Author

maribu commented May 6, 2024

Thx :)

@miri64 miri64 mentioned this pull request Jun 5, 2024
Merged
@mguetschow mguetschow added this to the Release 2024.07 milestone Jul 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Teufelchen1 Teufelchen1 Teufelchen1 approved these changes

@dylad dylad dylad approved these changes

@mguetschow mguetschow mguetschow approved these changes

@miri64 miri64 Awaiting requested review from miri64

@jia200x jia200x Awaiting requested review from jia200x jia200x is a code owner

Assignees
No one assigned
Labels
Area: doc Area: Documentation CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Impact: minor The PR is small in size and might only require a quick look of a knowledgeable reviewer Process: needs >1 ACK Integration Process: This PR requires more than one ACK
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@maribu @riot-ci @Teufelchen1 @dylad @mguetschow