Skip to content

supplying missing brief description for driver functions.#926

Merged
thefallentree merged 6 commits intofluffos:masterfrom
gesslar:adding-missing-desc-to-name
Oct 8, 2022
Merged

supplying missing brief description for driver functions.#926
thefallentree merged 6 commits intofluffos:masterfrom
gesslar:adding-missing-desc-to-name

Conversation

@gesslar
Copy link
Copy Markdown
Contributor

@gesslar gesslar commented Oct 7, 2022

Can someone else do memory_summary()? I am not familiar enough with what it's providing.


process_input - provide a hook for the MudOS driver to send a copy of
what the player is typing to the player object.
process_input - provide a hook for the driver to send a copy of what the
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

All of the other docs include () after the function names.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

done! thanks! 🤗

#undef NO_ENVIRONMENT
#undef NO_WIZARDS
#undef NO_LIGHT
#define NO_LIGHT
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Shouldn’t change this file, can you revert this?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Copied the original back and recommitted under #764ca52

@thefallentree thefallentree merged commit 5adc7b4 into fluffos:master Oct 8, 2022
Amirani-al pushed a commit to Amirani-al/fluffos that referenced this pull request May 30, 2023
* supplying missing brief description for driver functions.

* parenthesising

* removing superfluous space 😡

* removing superfluous space 😡

* temp step to fix local_options
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants