fix: pin kcd-scripts at 6.5

[benmonro]

What:

Why:

How:

Checklist:

• Documentation
• Tests
• Ready to be merged
  

[benmonro]

@MichaelDeBoey could use your help if you have any time. for some reason the latest version of kcd-scripts is causing errors on ajv. only thing i've found to fix this is to pin kcd-scripts@5.

[codecov]

Codecov Report

Merging #88 into master will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##            master       #88   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           10        10           
  Lines          184       184           
  Branches        21        21           
=========================================
  Hits           184       184           

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 167137a...4d3558f. Read the comment docs.

[benmonro]

🎉 This PR is included in version 3.2.4 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀

[MichaelDeBoey]

@benmonro Do you have something more detailed that I can have a look at?

[benmonro]

Yeah just bump kcd scripts to latest and run yarn test

[MichaelDeBoey]

@kentcdodds I tried to fix the problem mentioned above (see #89 & #91), but can't figure out the cause.
Problem is also appearing with latest kcd-scripts (see #93 & #94)

[kentcdodds]

Based on those I have no idea what this could be. Doesn't look related to kcd-scripts 🤷‍♂️

[MichaelDeBoey]

Seems like it has something to do with dependencies of kcd-scripts as 6.5 is working, but newer isn't working anymore 🤔😕

[MichaelDeBoey]

Is rollup/plugins#640 going to fix this one too? 🤔

[kentcdodds]

Potentially. I haven't looked into the issues here. But if it is that issue, you can work around it by adding @types/estree to your devDependencies.

[MichaelDeBoey]

Nope, seems like adding @types/estree isn't fixing the TypeError: Ajv is not a constructor error 😢