Skip to content

Conversation

@JoField08
Copy link
Contributor

…dings and instructions

@JoField08
Copy link
Contributor Author

@twisti-dev

@twisti-dev twisti-dev requested a review from Copilot March 26, 2025 18:55
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR refactors documentation for armorstands for improved clarity and consistency by consolidating content and updating formatting. Key changes include:

  • Complete removal of the old cosmetics.md file.
  • Merging and updating instructions in armorstand.md with new headings, updated links, and refined wording.
  • Consistent formatting and improved image alt texts to enhance user comprehension.

Reviewed Changes

Copilot reviewed 5 out of 8 changed files in this pull request and generated 2 comments.

File Description
cosmetics.md Removal of outdated content to streamline documentation.
armorstand.md Revised headings, updated instructions, and improved formatting for clarity.
Files not reviewed (3)
  • Writerside/ccs.tree: Language not supported
  • Writerside/cfg/glossary.xml: Language not supported
  • Writerside/redirection-rules.xml: Language not supported

@JoField08 JoField08 requested a review from twisti-dev March 26, 2025 19:04
twisti-dev
twisti-dev previously approved these changes Mar 26, 2025
@twisti-dev twisti-dev requested a review from Copilot March 26, 2025 19:05
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This pull request updates the documentation for armorstand interactions to improve clarity and consistency. It removes the outdated cosmetics.md file and refines the content and structure of armorstand.md by updating headings, image references, and instructional text.

Reviewed Changes

Copilot reviewed 5 out of 8 changed files in this pull request and generated no comments.

File Description
Writerside/topics/survival-server/cosmetics/cosmetics.md Removed outdated file content to prevent duplicative documentation.
Writerside/topics/survival-server/cosmetics/armorstand.md Revised headings, updated image references, and replaced tooltip references with explicit links/text for improved clarity.
Files not reviewed (3)
  • Writerside/ccs.tree: Language not supported
  • Writerside/cfg/glossary.xml: Language not supported
  • Writerside/redirection-rules.xml: Language not supported
Comments suppressed due to low confidence (2)

Writerside/topics/survival-server/cosmetics/cosmetics.md:1

  • The removal of the cosmetics.md file might break references from other parts of the documentation. Please confirm that all links and references to this file have been updated accordingly.
-# Köpfe und Rüstungsständer

Writerside/topics/survival-server/cosmetics/armorstand.md:24

  • [nitpick] The updated image alt text 'armorstand menu' is less descriptive than before. Consider using a more descriptive alt text to improve accessibility and context.
![armorstand menu](armorstand-editing-menu.png)

twisti-dev
twisti-dev previously approved these changes Mar 26, 2025
@twisti-dev twisti-dev requested a review from Copilot March 26, 2025 19:10
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR improves the documentation for armorstands by consolidating information and enhancing clarity and consistency between related topics. Key changes include:

  • Removal of the cosmetics.md page to streamline content.
  • Extensive updates to armorstand.md that refine headings, image references, and command instructions.
  • An updated FAQ link to reference the new armorstand.md documentation.

Reviewed Changes

Copilot reviewed 6 out of 9 changed files in this pull request and generated no comments.

File Description
Writerside/topics/survival-server/cosmetics/cosmetics.md Removed outdated cosmetics details to consolidate documentation.
Writerside/topics/survival-server/cosmetics/armorstand.md Updated headings, images, and instructions for improved clarity and consistency.
Writerside/topics/questions/faq.md Modified link to point to the updated armorstand.md page.
Files not reviewed (3)
  • Writerside/ccs.tree: Language not supported
  • Writerside/cfg/glossary.xml: Language not supported
  • Writerside/redirection-rules.xml: Language not supported
Comments suppressed due to low confidence (1)

Writerside/topics/survival-server/cosmetics/armorstand.md:167

  • [nitpick] Consider revisiting the heading level for 'Slots von 1-6'. It is set as a fourth-level heading while other subsections use a third-level heading; ensure this is intentional or update for consistency.
#### Slots von 1-6 {id="slots-1-6"}

@JoField08 JoField08 requested a review from twisti-dev March 26, 2025 19:12
@twisti-dev twisti-dev requested a review from Copilot March 26, 2025 19:13
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR improves documentation clarity and consistency for armorstand and head cosmetics by consolidating scattered topics and updating formatting. Key changes include:

  • Removal of redundant information in cosmetics.md.
  • Revised and expanded documentation in armorstand.md to clarify commands and procedures.
  • Updated cross-reference in faq.md to point to the new documentation.

Reviewed Changes

Copilot reviewed 6 out of 9 changed files in this pull request and generated no comments.

File Description
Writerside/topics/survival-server/cosmetics/cosmetics.md Removed outdated or redundant cosmetics content.
Writerside/topics/survival-server/cosmetics/armorstand.md Expanded and reformatted documentation for armorstands and heads.
Writerside/topics/questions/faq.md Updated link reference to align with the relocated documentation.
Files not reviewed (3)
  • Writerside/ccs.tree: Language not supported
  • Writerside/cfg/glossary.xml: Language not supported
  • Writerside/redirection-rules.xml: Language not supported
Comments suppressed due to low confidence (2)

Writerside/topics/survival-server/cosmetics/cosmetics.md:1

  • The entire file content is removed; please confirm that this removal is intentional and that no necessary guidance is lost.
-# Köpfe und Rüstungsständer

Writerside/topics/survival-server/cosmetics/armorstand.md:20

  • [nitpick] There is an inconsistency in how 'Feuerstein' is presented: here it is a hyperlink, while in later occurrences it is plain text; consider unifying the representation for clarity.
einem [Feuerstein](https://de.minecraft.wiki/w/Feuerstein) direkt auf den Rüstungsständer.

@twisti-dev twisti-dev requested a review from Copilot March 26, 2025 19:15
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates the armorstand documentation for improved clarity and consistency while also correcting link references and removing outdated content.

  • Updated headings and text formatting in armorstand.md
  • Corrected link from cosmetics.md to armorstand.md in faq.md
  • Removed obsolete cosmetics.md file

Reviewed Changes

Copilot reviewed 6 out of 9 changed files in this pull request and generated no comments.

File Description
Writerside/topics/survival-server/cosmetics/armorstand.md Revised headings and replaced tooltip formatting with inline links for clarity
Writerside/topics/questions/faq.md Updated reference link to point to armorstand.md
Writerside/topics/survival-server/cosmetics/cosmetics.md Removed outdated content to streamline documentation
Files not reviewed (3)
  • Writerside/ccs.tree: Language not supported
  • Writerside/cfg/glossary.xml: Language not supported
  • Writerside/redirection-rules.xml: Language not supported
Comments suppressed due to low confidence (2)

Writerside/topics/survival-server/cosmetics/armorstand.md:20

  • Ensure consistent replacement of tooltip formatting with inline links for 'Feuerstein' throughout the document. Verify that no instances of the old tooltip syntax are left to maintain uniform clarity.
einem [Feuerstein](https://de.minecraft.wiki/w/Feuerstein) direkt auf den Rüstungsständer.

Writerside/topics/survival-server/cosmetics/cosmetics.md:1

  • Confirm that the complete removal of cosmetics.md is intentional and that all references have been updated accordingly, to avoid any broken links in the documentation.
-# Köpfe und Rüstungsständer

@twisti-dev twisti-dev enabled auto-merge March 26, 2025 19:17
@twisti-dev twisti-dev added this pull request to the merge queue Mar 26, 2025
Merged via the queue into SLNE-Development:master with commit 33d3600 Mar 26, 2025
1 check 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.

2 participants