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

GitHub: Update code of conduct #494

Merged
merged 18 commits into from
Jul 27, 2024
Merged

GitHub: Update code of conduct #494

merged 18 commits into from
Jul 27, 2024

Conversation

Lamparter
Copy link
Collaborator

@Lamparter Lamparter commented Jul 22, 2024

πŸ“„ Description

Update code of conduct and other related community documents with better spelling/more accurate links.

βœ… Tasks

πŸ“‹ Update community documents

  • Update code of conduct
  • Remove privacy policy
    • Remove references to privacy policy
  • Add code of conduct to settings page
  • Fix merge conflicts

@Lamparter Lamparter self-assigned this Jul 22, 2024
@0x5bfa 0x5bfa marked this pull request as ready for review July 22, 2024 15:37
@0x5bfa 0x5bfa self-requested a review as a code owner July 22, 2024 15:37
@0x5bfa 0x5bfa marked this pull request as draft July 22, 2024 15:38
@Lamparter Lamparter added this to the FluentHub v1.0.0 milestone Jul 23, 2024
@Lamparter Lamparter mentioned this pull request Jul 25, 2024
66 tasks
@Lamparter

This comment was marked as resolved.

@Lamparter Lamparter added the type/infra This issue/PR is related to developer experience label Jul 25, 2024
@Lamparter

This comment was marked as resolved.

@Lamparter
Copy link
Collaborator Author

Finally! 😌 I managed to merge it using a series of very unintuitive powershell commands after a lot of trial and error.

@Lamparter

This comment was marked as outdated.

@Lamparter Lamparter marked this pull request as ready for review July 27, 2024 09:03
@Lamparter

This comment was marked as resolved.

@0x5bfa
Copy link
Owner

0x5bfa commented Jul 27, 2024

Sure

@0x5bfa
Copy link
Owner

0x5bfa commented Jul 27, 2024

Is this ready for merge?

@0x5bfa 0x5bfa added triage/approved This PR has been approved by the team. and removed type/infra This issue/PR is related to developer experience labels Jul 27, 2024
@Lamparter

This comment was marked as outdated.

@Lamparter

This comment was marked as resolved.

@Lamparter
Copy link
Collaborator Author

Is this ready for merge?

Yes

@Lamparter Lamparter enabled auto-merge (squash) July 27, 2024 09:38
@Lamparter
Copy link
Collaborator Author

Oh wait @0x5bfa you need to force merge it because CodeFactor is a required status check

@Lamparter Lamparter merged commit 1daeca0 into main Jul 27, 2024
7 of 8 checks passed
@Lamparter Lamparter deleted the lamparter/community branch July 27, 2024 09:40
@0x5bfa
Copy link
Owner

0x5bfa commented Jul 27, 2024

Changed the rule

@0x5bfa
Copy link
Owner

0x5bfa commented Jul 27, 2024

Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
triage/approved This PR has been approved by the team.
Projects
Status: βœ… Done
Development

Successfully merging this pull request may close these issues.

2 participants