search

alastairtree / LazyCache

An easy to use thread safe in-memory caching service with a simple developer friendly API for c#
https://nuget.org/packages/LazyCache
MIT License
1.72k stars 159 forks source link

AppCacheExtenions #64

Closed JohnL404 closed 5 years ago

JohnL404 commented 5 years ago

This is pretty trivial so I apologize but I think the new AppCacheExtenions class should be called AppCacheExtensions? Thanks!

alastairtree commented 5 years ago

Thank you - and sorry my spelling is often terrible! Pretty sure you are not wrong. Feel free to submit a quick PR for it or i'll correct next time I work on it.

alastairtree commented 5 years ago

To submit a PR fork the repo, then create a branch on that repo. When you are happy create a Pull Request into LazyCache for me to review. See https://help.github.com/en/articles/creating-a-pull-request-from-a-fork.

On Tue, 26 Mar 2019 at 21:32, JohnL404 notifications@github.com wrote:

I'd created a branch locally with the correction but it's not letting me push it to GitHub in order to submit a PR (getting a 403). I'm new to GitHub as a contributor - is there a different process I should be following to create my branch?

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/alastairtree/LazyCache/issues/64#issuecomment-476861145, or mute the thread https://github.com/notifications/unsubscribe-auth/AF-5lY24dDq249JFwTqhdPyXnzSeQ2MTks5vapHlgaJpZM4cMJFt .

JohnL404 commented 5 years ago

Will do - thanks a ton!

alastairtree commented 5 years ago

Awesome, thanks.

image