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.

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

clean up the general management docs #348

Merged
merged 1 commit into from
Feb 5, 2025
Merged

clean up the general management docs #348

merged 1 commit into from
Feb 5, 2025
+277 −174

Conversation

eshaben
Copy link
Collaborator

@eshaben eshaben commented Jan 31, 2025

This PR cleans up the following page: https://docs.polkadot.com/infrastructure/running-a-validator/operational-tasks/general-management/

In particular it does the following:

  • Makes the list items concise
  • Formats each of the lists in the same way
  • Updates usage of Alertmanager, so we're not using unnecessary backticks

@eshaben
clean up the general management docs

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
GPG key ID: 4AEE18F83AFDEB23
Expired
Verified
Learn about vigilant mode
Loading
Loading status checks…
fd4992a
@eshaben eshaben requested a review from a team as a code owner January 31, 2025 05:20
@eshaben eshaben requested review from nhussein11 and 0xLucca January 31, 2025 05:20
@eshaben eshaben added B0 - Needs Review Pull request is ready for review C0 - Low Low priority task A2 - Maintenance Minor Pull request contains minor updates to an existing page (i.e., modifying parameters, steps, etc.) labels Jan 31, 2025
0xLucca
0xLucca approved these changes Jan 31, 2025
View reviewed changes
Copy link
Collaborator

@0xLucca 0xLucca left a comment

Choose a reason for hiding this comment

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

LGTM!

dawnkelly09
dawnkelly09 approved these changes Feb 3, 2025
View reviewed changes
Copy link
Collaborator

@dawnkelly09 dawnkelly09 left a comment

Choose a reason for hiding this comment

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

🚀

@eshaben eshaben added B1 - Ready to Merge Pull request is ready to be merged and removed B0 - Needs Review Pull request is ready for review labels Feb 4, 2025
@eshaben eshaben merged commit 868b0aa into master Feb 5, 2025
1 check passed
@eshaben eshaben deleted the eshaben/fix-alertmanager branch February 5, 2025 01:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dawnkelly09 dawnkelly09

@0xLucca 0xLucca

@nhussein11 nhussein11

Assignees
No one assigned
Labels
A2 - Maintenance Minor Pull request contains minor updates to an existing page (i.e., modifying parameters, steps, etc.) B1 - Ready to Merge Pull request is ready to be merged C0 - Low Low priority task
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@eshaben @dawnkelly09 @0xLucca