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

Deprecate admin guide #4312

Merged
merged 2 commits into from
May 28, 2024
Merged

Conversation

ElliotFriend
Copy link
Contributor

Description

Resolves stellar/stellar-docs#341

This PR adds a deprecation notice to the admin.md and soroban-settings.md documents, pointing readers/users/contributors to use the Admin Guide in the Stellar Docs site: https://developers.stellar.org/network/core-node/admin-guide

Checklist

  • Reviewed the contributing document
  • Rebased on top of master (no merge commits)
  • Ran clang-format v8.0.0 (via make format or the Visual Studio extension)
  • Compiles
  • Ran all tests
  • If change impacts performance, include supporting evidence per the performance document

@dmkozh
Copy link
Contributor

dmkozh commented May 24, 2024

Could you please rebase and resolve the conflicts?

ElliotFriend added a commit to stellar/stellar-docs that referenced this pull request May 28, 2024
stellar/stellar-core@075df81 included some changes to the soon-
to-be-deprecated `admin.md` file within the codebase. This commit
includes the changes from that commit, and adds those new details
to `monitoring.mdx` in the core-node admin guide.

Refs: stellar/stellar-core#4312
@ElliotFriend
Copy link
Contributor Author

ElliotFriend commented May 28, 2024

@dmkozh Thanks for reminder!

I tried doing a rebase, but after pushing the commit had 40-something changed files, which didn't seem like what I was aiming for. I reset and did a merge with the master branch, which (I think) fixes the conflicts and makes it ready for review. If that's not the right strategy, I'd be happy to take another crack at rebasing if that works better for this repo. (Nevermind, I figured out the rebase stuff)

I also took the relevant changes from 075df81 (the commit that caused the conflicts), and put them into a PR for the docs site: stellar/stellar-docs#590.

Let me know if there's anything else this PR needs. Thanks!

ElliotFriend added a commit to stellar/stellar-docs that referenced this pull request May 28, 2024
stellar/stellar-core@075df81 included some changes to the soon-
to-be-deprecated `admin.md` file within the codebase. This commit
includes the changes from that commit, and adds those new details
to `monitoring.mdx` in the core-node admin guide.

Refs: stellar/stellar-core#4312
@anupsdf anupsdf requested a review from sisuresh May 28, 2024 16:28
@dmkozh
Copy link
Contributor

dmkozh commented May 28, 2024

r+ 1644ffa

@latobarita latobarita merged commit 4ae34b5 into stellar:master May 28, 2024
15 checks passed
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.

Core Node documentation
4 participants