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

Fix missing array opening bracket. #238

Merged
merged 2 commits into from
Sep 6, 2022
Merged

Fix missing array opening bracket. #238

merged 2 commits into from
Sep 6, 2022

Conversation

homotechsual
Copy link
Contributor

Description

Readme syntax fix.

Motivation and Context

Error in readme config syntax.

Types of changes

  • Documentation only

Checklist

  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes if appropriate.
  • All new and existing tests passed.

@welcome-to-palo-alto-networks
Copy link

🎉 Thanks for opening this pull request! We really appreciate contributors like you! 🙌

@github-actions
Copy link

github-actions bot commented Sep 6, 2022

Visit the preview URL for this PR (updated for commit aa468fa):

https://docusaurus-openapi-36b86--pr238-2s4w243x.web.app

(expires Thu, 06 Oct 2022 10:09:25 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Copy link
Member

@sserrata sserrata left a comment

Choose a reason for hiding this comment

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

Thanks for the contribution!

@sserrata sserrata added the documentation Improvements or additions to documentation label Sep 6, 2022
@sserrata
Copy link
Member

sserrata commented Sep 6, 2022

FYI, we'll be sure to include this change in the upcoming documentation refresh.

@homotechsual
Copy link
Contributor Author

FYI, we'll be sure to include this change in the upcoming documentation refresh.

I think on the actual docs site the syntax is good.

@sserrata sserrata merged commit 8ff4a7d into PaloAltoNetworks:main Sep 6, 2022
@welcome-to-palo-alto-networks
Copy link

🎉 Congrats on getting your first pull request merged! We here at Palo Alto Networks are so grateful! ❤️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants