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

Remove @ from example to fix Documentation CI #719

Merged
merged 1 commit into from
Sep 15, 2023

Conversation

Spiker985
Copy link
Member

Coffeescript function calls are prefixed with an @, which is the same syntax that jsdoc uses for parsing - which meant that it thought the example was empty.

Coffeescript function calls are prefixed with an @, which is the same syntax that jsdoc uses for parsing - which meant that it thought the example was empty.
@Spiker985 Spiker985 added bug Something isn't working documentation Improvements or additions to documentation labels Sep 14, 2023
Copy link
Member

@confused-Techie confused-Techie left a comment

Choose a reason for hiding this comment

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

LGTM!

I detailed further how I broke this over on #707 so thanks a ton for fixing it!

@Spiker985
Copy link
Member Author

Seeing as how no code changes occurred, and the fact that the failed CI is due to a timeout - I'm gonna go ahead and merge this. Thanks @confused-Techie

@Spiker985 Spiker985 merged commit 218d26c into master Sep 15, 2023
98 of 99 checks passed
@Spiker985 Spiker985 deleted the update-jsdoc-tree-view-ci-error branch September 15, 2023 00:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants