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

New Documentation #1369

Draft
wants to merge 7 commits into
base: master
Choose a base branch
from
Draft

New Documentation #1369

wants to merge 7 commits into from

Conversation

mrtechtroid
Copy link
Contributor

  • This PR has been tested locally. (Not needed to be)
  • New README
  • New CONTRIBUTING.MD
  • Update CTF Api Docs.

CONTRIBUTING.md Outdated Show resolved Hide resolved
I would highly recommend using [Visual Studio Code](https://code.visualstudio.com/download), for cloning the project, creating patches, etc.

### Fork Project
- You can create a clone of the project by using git from the CLI or with the use of a tool like the one recommended above.
Copy link
Member

Choose a reason for hiding this comment

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

You can clone with Visual Studio?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I meant using the GUI interface VSCode offers, after installing git. Like Ctrl+Shift+P and then Git: Clone.

Copy link
Member

Choose a reason for hiding this comment

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

I would probably move the recommendation to CLI git but mention that VSCode offers a GUI for it

Copy link
Member

@LoneWolfHT LoneWolfHT Dec 30, 2024

Choose a reason for hiding this comment

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

If it's your first time programming or using Github/Git I'd recommend VSCode with those extensions, but I personally prefer CLI git, and a competent contributor could use whatever editor they want, probably paired with CLI luacheck unless there's an extension for their editor

CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
A ranged weapon framework for Minetest. See the API reference [here](https://github.com/LoneWolfHT/rawf/blob/main/API.md)

# mtg
This folder contains the Minetest Game, along with redefinitions and overrides for the CTF Game. This folder doesnt expose any new APIs on its own. Please follow the Minetest Game API Reference for the same [here](https://github.com/minetest/minetest_game/blob/master/game_api.txt).
Copy link
Member

Choose a reason for hiding this comment

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

Not all mtg mods are in mods/mtg/

## ctf_settings
This mod adds a 'Settings' tab to the player's inventory.
Mods can use the ctf_settings API to add buttons/fields to the Settings tab that can be used to customize the mod's functionality per-player.

### `ctf_settings.register(name, def)`
#### `ctf_settings.register(name, def)`
Copy link
Member

@LoneWolfHT LoneWolfHT Dec 30, 2024

Choose a reason for hiding this comment

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

Might be better to just bold it, going off of my memory of H4s. IIRC you're not supposed to skip a header size when adding a sub-header

This mod adds suffocation. Suffocation is basically the same as drowning, but it is for being stuck inside solid blocks. No API's are exposed by this mod.

## select_item
This mod provides a simple dialog for players to select an item from. The API Reference is [here](https://github.com/MT-CTF/capturetheflag/blob/master/mods/other/select_item/API.md)
Copy link
Member

Choose a reason for hiding this comment

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

I planned to use this in the map editor, IIRC it's forked from a mod by Wuzzy

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants