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

fix: fix different import action #34

Closed
wants to merge 0 commits into from
Closed

fix: fix different import action #34

wants to merge 0 commits into from

Conversation

Zxilly
Copy link
Contributor

@Zxilly Zxilly commented Apr 22, 2021

This will keep legacy usage like const MongooseAdapter = require('casbin-mongoose-adapter');

@Sefriol
Copy link
Contributor

Sefriol commented Apr 22, 2021

lgtm

@hsluoyz
Copy link
Member

hsluoyz commented Apr 23, 2021

@Zxilly why do this PR? Maybe we can setup semantic-release first, then test it with this PR's merge.

@Zxilly
Copy link
Contributor Author

Zxilly commented Apr 23, 2021

@hsluoyz During the refactoring to Typescript, the import action was changed.
Yes, we can merge this after the auto release setup. But this may release a version with the different action.

@hsluoyz
Copy link
Member

hsluoyz commented Apr 23, 2021

Replaced by: #37

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.

3 participants