Skip to content
This repository has been archived by the owner on Oct 3, 2023. It is now read-only.

Update Core library README #216

Merged
merged 2 commits into from
Dec 4, 2018
Merged

Update Core library README #216

merged 2 commits into from
Dec 4, 2018

Conversation

mayurkale22
Copy link
Member

@mayurkale22 mayurkale22 commented Dec 3, 2018 •
edited
Loading

  1. Add badges (node, npm, license, dependencies and devDependencies).
  2. Add usage example.

bogdandrutu
bogdandrutu approved these changes Dec 3, 2018
View reviewed changes
Copy link
Contributor

@bogdandrutu bogdandrutu left a comment

Choose a reason for hiding this comment

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

For badges, for the installation and examples I would like others who know how to do that to review.

@mayurkale22 mayurkale22 force-pushed the core_readme branch 2 times, most recently from 4c01a87 to d7e3104 Compare December 4, 2018 18:11
@isaikevych isaikevych self-requested a review December 4, 2018 18:57
@mayurkale22 mayurkale22 merged commit e5281f4 into census-instrumentation:master Dec 4, 2018
@mayurkale22 mayurkale22 deleted the core_readme branch December 4, 2018 19:24
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@bogdandrutu bogdandrutu bogdandrutu approved these changes

@isaikevych isaikevych isaikevych approved these changes

@eduardoemery eduardoemery Awaiting requested review from eduardoemery

Assignees
No one assigned
Labels
cla: yes
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@mayurkale22 @bogdandrutu @isaikevych @googlebot