waiting-for-dev / warden-jwt_auth

JWT token authentication with warden
MIT License
113 stars 56 forks source link

Relax dependencies on dry-* gems #52

Closed jwoudenberg closed 1 year ago

jwoudenberg commented 1 year ago

Summary

Hi! First of all, thank you for your work on this repo, it's super appreciated!

We've been updating some of our dependencies and found that the current constraints for the dry-auto_inject and dry-configurable gems are stricter then they need to be. Both gems recently released 1.x versions, and those to seem to work great. This PR relaxes those constraints to make warden-jwt_auth compatible with these newer versions.

Checklist

Check out our PR guidelines for more details.

The following are mandatory for all PRs:

The following are not always needed (~cross them out~ if they are not):

thomasdarde commented 1 year ago

This is a duplicate of #46 which also add zeitwerk loading

waiting-for-dev commented 1 year ago

Many thanks for your collaboration. I'd just to bump the version after merging this.

jwoudenberg commented 1 year ago

Many thanks for your collaboration. I'd just to bump the version after merging this.

Thank you for the quick response!