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

docs: add js document #813

Merged
merged 4 commits into from
Oct 14, 2022
Merged

docs: add js document #813

merged 4 commits into from
Oct 14, 2022

Conversation

LZHK1ng
Copy link
Contributor

@LZHK1ng LZHK1ng commented Oct 11, 2022

No description provided.

@mosn-community-bot
Copy link

Hi @LZHK1ng, welcome to mosn community, Please sign Contributor License Agreement!

After you signed CLA, we will automatically sync the status of this pull request in 3 minutes.

@seeflood
Copy link
Member

@LZHK1ng Thanks! Please sign the CLA first

Copy link
Member

@seeflood seeflood left a comment

Choose a reason for hiding this comment

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

lgtm. Thanks for your contribution and welcome to the community !

@seeflood seeflood merged commit c5009a6 into mosn:main Oct 14, 2022
seeflood added a commit that referenced this pull request Oct 14, 2022
@seeflood
Copy link
Member

@LZHK1ng Sorry, we found that the demo failed 😢
Could u help fix it ?
https://github.com/mosn/layotto/actions/runs/3247071797/jobs/5326577577
image

@seeflood
Copy link
Member

@LZHK1ng Sorry, we found that the demo failed 😢 Could u help fix it ? https://github.com/mosn/layotto/actions/runs/3247071797/jobs/5326577577

I know why. Let me fix it

seeflood added a commit that referenced this pull request Oct 14, 2022
@seeflood
Copy link
Member

@LZHK1ng Sorry, I don't know how to fix it, so I reverted this PR 😢 Could u submit this pr again and fix the CI error?

@LZHK1ng
Copy link
Contributor Author

LZHK1ng commented Oct 14, 2022

Ok, I will fix it.

@seeflood seeflood changed the title feat: add js document docs: add js document Oct 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants