Skip to content
This repository has been archived by the owner on May 25, 2022. It is now read-only.

the package is not compatible with quiver 0.22.0 #33

Closed
zoechi opened this issue Oct 18, 2016 · 5 comments
Closed

the package is not compatible with quiver 0.22.0 #33

zoechi opened this issue Oct 18, 2016 · 5 comments

Comments

@zoechi
Copy link

zoechi commented Oct 18, 2016

 quiver: '>=0.21.0 <0.22.0'
@davidmorgan
Copy link
Contributor

Should just be a matter of bumping the versions. I'll take a look tomorrow. Thanks for filing!

@davidmorgan
Copy link
Contributor

Now fixed, please re-open if you have any problems. Thanks!

@zoechi
Copy link
Author

zoechi commented Oct 19, 2016

Great, thanks!

A question. I tried your packages in one of my projects where I use an .analysis_options file with strong mode and most linter rules enabled. The generated code causes quite a lot of hints and warnings.

.analysis_options provides an option to exclude files but that doesn't yet work properly.

Do you plan to support strong mode (actually not tested if strong mode alone causes hints/warnings) and perhaps other linter rules?

@davidmorgan
Copy link
Contributor

Yes, we will definitely support strong mode. Could you file an issue with example hints/warnings please?

@zoechi
Copy link
Author

zoechi commented Oct 25, 2016

I saw you took the first steps already. When I find time to try it again and run into issues, I'll create a concrete issue. Thanks for your great work :)

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

No branches or pull requests

2 participants