Skip to content

update docs#19

Open
edayot wants to merge 9 commits intomainfrom
upstream/update_crafter_docs
Open

update docs#19
edayot wants to merge 9 commits intomainfrom
upstream/update_crafter_docs

Conversation

@edayot
Copy link
Copy Markdown
Contributor

@edayot edayot commented Dec 28, 2025

No description provided.

Copy link
Copy Markdown

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 Crafter library documentation by adding a new page for flags and reorganizing the function tag documentation. The changes improve documentation clarity by moving advanced features into dedicated sections.

  • Adds comprehensive documentation for crafting flags (consume_buckets, consume_tools)
  • Documents the #smithed.crafter:event/query_tags function tag with usage examples
  • Reorganizes advanced uses section by moving specific features to dedicated pages

Reviewed changes

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

File Description
docs/libraries/crafter/index.md Adds new flags page to the documentation table of contents
docs/libraries/crafter/function_tag.md Adds query_tags event documentation and updates advanced uses section with corrected links
docs/libraries/crafter/flags.md New file documenting flag system for customizing item removal during crafting

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

| 'Success of operation' | score | $temp | smithed.data |
| 'Item to replace in the crafter' | entity | @s | weapon.mainhand |

The default value of 'Success of operation' is 0, which by default juste remove one item. This whould be the default behavior if your flag is not present. If your flag is present and you want to change the item, set it to 1.
Copy link

Copilot AI Dec 28, 2025

Choose a reason for hiding this comment

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

There's a spelling error in "juste" which should be "just".

Suggested change
The default value of 'Success of operation' is 0, which by default juste remove one item. This whould be the default behavior if your flag is not present. If your flag is present and you want to change the item, set it to 1.
The default value of 'Success of operation' is 0, which by default just remove one item. This whould be the default behavior if your flag is not present. If your flag is present and you want to change the item, set it to 1.

Copilot uses AI. Check for mistakes.
@Smithed-MC Smithed-MC deleted a comment from Copilot AI Mar 29, 2026
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
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