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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add a section to the README for examples. #33

Merged
merged 1 commit into from
Feb 24, 2018
Merged

Add a section to the README for examples. #33

merged 1 commit into from
Feb 24, 2018

Conversation

bryanbraun
Copy link
Contributor

Hi there 馃憢

I was able to use this in a public project recently, and thought I'd say thanks by adding an "examples" section to the readme. If I end up making any fixes, I'm hopeful that I can contribute them upstream.

Thanks again for building and maintaining such a great library! 馃弳

@coveralls
Copy link

coveralls commented Feb 24, 2018

Coverage Status

Coverage remained the same at 97.451% when pulling e3efc1a on bryanbraun:add-examples into f6de179 on SheetJS:master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage remained the same at 97.451% when pulling e3efc1a on bryanbraun:add-examples into f6de179 on SheetJS:master.

@SheetJSDev
Copy link
Contributor

Glad the library was helpful. Your site looks very nice, and we'd love a little shoutout in your About page :)

@SheetJSDev SheetJSDev merged commit e3efc1a into SheetJS:master Feb 24, 2018
@bryanbraun
Copy link
Contributor Author

@SheetJSDev, sure thing! I've added a credit to the about page. 馃憤

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

Successfully merging this pull request may close these issues.

None yet

3 participants