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

Added OpenAPM Badge to the readme file #1493

Merged
merged 2 commits into from
Jul 26, 2018
Merged

Conversation

AlexanderWert
Copy link
Contributor

As discussed with @wu-sheng,

submitting an OpenAPM-Badge for the readme of this project, as Apache SkyWalking is included in the OpenAPM Landscape: https://openapm.io/landscape.

@wu-sheng wu-sheng requested a review from a team July 24, 2018 12:38
@wu-sheng wu-sheng added this to the 5.0.0-RC milestone Jul 24, 2018
@wu-sheng wu-sheng added the documentation Provide documents related issue or pull request only. label Jul 24, 2018
@coveralls
Copy link

Coverage Status

Coverage remained the same at 24.586% when pulling 22a4461 on AlexanderWert:master into 56d7fe4 on apache:master.

@wu-sheng
Copy link
Member

@apache/skywalking-committers Hi all. We have been listed in OpenAPM landscape. https://openapm.io

I want to start a public vote to decide whether we accept this. Open vote for at least 72 hours

Thanks, everyone.

@wu-sheng
Copy link
Member

My +1

1 similar comment
@liuhaoyang
Copy link
Member

My +1

@IluckySi
Copy link
Contributor

+1

1 similar comment
@ascrutae
Copy link
Member

+1

@hanahmily
Copy link
Contributor

Great! This's more professional landscape for APM.
+1

@peng-yongsheng
Copy link
Member

+1

4 similar comments
@clevertension
Copy link
Contributor

+1

@qiujiayu
Copy link
Member

+1

@zhangkewei
Copy link
Member

+1

@candyleer
Copy link
Member

+1

@wu-sheng
Copy link
Member

@AlexanderWert I am positive that the community could agree on this pr. I will try to merge this tomorrow. Also, will sync to 6.0 branch too.

@AlexanderWert
Copy link
Contributor Author

Thank you @wu-sheng !
Do you think I can now also submit a pull request with the badge for the web site?

@wu-sheng
Copy link
Member

Do you think I can now also submit a pull request with the badge for the web site?

I am still thinking about using a new template for our website. Just didn't find a good one. I am just not sure whether is a good time for you to update.

@wu-sheng wu-sheng merged commit 6753478 into apache:master Jul 26, 2018
wu-sheng pushed a commit that referenced this pull request Jul 26, 2018
* Added OpenAPM Badge to readme

* Update README.md
@wu-sheng
Copy link
Member

@AlexanderWert Done.

@AlexanderWert
Copy link
Contributor Author

@wu-sheng Got it!
Will wait then for the new template. Would be great if you could let me know!

Thanks!

@wu-sheng
Copy link
Member

Will wait then for the new template. Would be great if you could let me know!

Let me find out. I will put an official announcement on Twitter.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Provide documents related issue or pull request only.
Projects
None yet
Development

Successfully merging this pull request may close these issues.