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

TFCond Descriptions #1005

Merged
merged 3 commits into from
Jun 1, 2019
Merged

TFCond Descriptions #1005

merged 3 commits into from
Jun 1, 2019

Conversation

Batfoxkid
Copy link
Contributor

Adds comments about the conditions in tf2.inc. This is mainly for the API to show description of the condition. Just a small thing.

This is mainly for API (https://sm.alliedmods.net/new-api/tf2/TFCond) to show description of the condition.
@KyleSanderson
Copy link
Member

Where were these sourced from?

@Batfoxkid
Copy link
Contributor Author

Where were these sourced from?

https://wiki.teamfortress.com/wiki/Cheats#addcond
Main place and my own experience using them.

@Headline Headline added the Documentation a change only towards documentation label May 19, 2019
@asherkin
Copy link
Member

I think I would prefer this without the numbers - if the numeric values are useful they should be specified as part of the enum or displayed by the doc tool. Thoughts?

@Batfoxkid
Copy link
Contributor Author

Batfoxkid commented May 31, 2019

Probably yeah, I do like some way of showing numeric values so that way for people who don't want to use the full name and instead do view_as<Enum>(#)

Copy link
Member

@asherkin asherkin left a comment

Choose a reason for hiding this comment

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

A lot of people seem to like the numbers for this enum, so lets take this with them and look at a large doctool change later.

@asherkin asherkin merged commit cf0a498 into alliedmodders:master Jun 1, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Documentation a change only towards documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants