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

reflection warning in epsilon util, fixes #230 #237

Merged
merged 1 commit into from
Jun 10, 2023

Conversation

brandonstubbs
Copy link
Contributor

Just wanted to first say thank you for your hard work on this library. I started playing around with it today and it's awesome!

Fixed issue with reflection warning reported in #230.
I added bb reader conditionals before the clj ones which prevent us from trying to dynamically load the class.
Babashka Reader conditionals will read bb before clj.

I added cache in gitignore which clj-kondo & lsp-lisp use and is safe to ignore.
I added calva in gitignore for anyone who codes using calva on VSCode.

Please let me know if you have any questions or you want me to make any changes.

@brandonstubbs
Copy link
Contributor Author

brandonstubbs commented Jan 20, 2023

I just opened another PR #238 - It will be better to get that one in first so that you will be able to see all the tests that run in your ci workflow on the pull request before needing to merge.

Just wanted to add another little useful feature you may enjoy (Especially if you prefer github notification for management) https://joeprevite.com/auto-assign-pull-requests-github. This will allow any new PR's on the repo to be assigned to you so you can be notified.

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

2 participants