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

Create Return Object page #495

Merged
merged 8 commits into from
Dec 8, 2024
Merged

Create Return Object page #495

merged 8 commits into from
Dec 8, 2024

Conversation

jumpsplat120
Copy link
Contributor

All over the wiki, "success" and "failure" objects are referenced, but it is not ever explained what they actually are. You can sort of piece it together by checking the .ok() page under scripts.lib, but instead I figured it would make more sense to just actually have a page that explained what it was and how it worked.

I've not created a page before, so the formatting is just based off what I observed from some of the other pages. As well, all the links on the wiki seem to be broken right now, so I'm not entirely sure if the link to the ok and not_impl pages have been formatted correctly, but I'm reasonably confident.

@seanmakesgames
Copy link
Member

We merged #498

Update your branch to get the test CI passing

@jumpsplat120
Copy link
Contributor Author

patch-4 and patch-5 branches updated.

@seanmakesgames
Copy link
Member

apparently wasn't enough. We got this one merged: #499

try an update again this time and let's see if the tests pass / run correctly now. :)

Copy link
Member

@tukib tukib left a comment

Choose a reason for hiding this comment

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

This is great! I've added some suggestions, and you will need to rename the filename to return_object.mdx for it to be registered as wiki page.

docs/scripting/syntax/return_object Outdated Show resolved Hide resolved
docs/scripting/syntax/return_object Outdated Show resolved Hide resolved
docs/scripting/syntax/return_object Outdated Show resolved Hide resolved
@tukib tukib merged commit 7a4271d into comcode-org:main Dec 8, 2024
4 checks passed
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