Skip to content

TFCond Descriptions#1005

Merged
asherkin merged 3 commits intoalliedmodders:masterfrom
Batfoxkid:tf2-cond-desc
Jun 1, 2019
Merged

TFCond Descriptions#1005
asherkin merged 3 commits intoalliedmodders:masterfrom
Batfoxkid:tf2-cond-desc

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.

@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