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

1.14 #263

Open
wants to merge 14 commits into
base: 1.14_eol
Choose a base branch
from
Open

1.14 #263

wants to merge 14 commits into from

Conversation

InQuognito
Copy link

fixed mkdocs.yml from previous PR

Comment on lines +8 to +15
- Removing Recipes: "recipes/removing_recipes.md"
- Adding Recipes:
- Blast Furnace: "recipes/adding_recipes/blast_furnace.md"
- Campfire: "recipes/adding_recipes/campfire.md"
- Crafting Table: "recipes/adding_recipes/crafting_table.md"
- Furnace: "recipes//adding_recipes/furnace.md"
- Smoker: "recipes/adding_recipes/smoker.md"
- Stone Cutter: "recipes/adding_recipes/stonecutter.md"
Copy link
Contributor

Choose a reason for hiding this comment

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

Please use single quotes.

- Vanilla:
- How to:
- How to: 'vanilla/how_to/how_to.md'
- Stations: "vanilla/stations.md"
Copy link
Contributor

Choose a reason for hiding this comment

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

Please use single quotes.

- Remember to close your brackets. Most text editors come with either a bracket pair highlighting or automatically creating a closing bracket.

## Referencing
- Every reference to an item in game **must** be referenced with `<>`.
Copy link
Contributor

Choose a reason for hiding this comment

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

This statement feels a bit miss-leading, as you could also later reference an item using a variable, which would not need to be surrounded with brackets.

```zenscript
removeRecipe(IItemStack output);
```
This script will remove a(ll) recipes for a referenced item. For instance, typing
Copy link
Contributor

Choose a reason for hiding this comment

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

This script will remove a(ll)
Should the a(ll) just be all?

@@ -0,0 +1,95 @@
# Crafting Table
Copy link
Contributor

Choose a reason for hiding this comment

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

Documentation for the crafting table has been added here.

I think some decisions should be made as to how the documentation should be structured. @jaredlll08 when you have some time, I think it would be good to draft up a general concept for contributors to follow and add it to the README.

@jaredlll08
Copy link
Member

What is the current state of this PR?

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.

3 participants