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: amend CommonJS output to be compatible with require #36

Merged
merged 1 commit into from
Sep 22, 2022

Conversation

leosuncin
Copy link
Contributor

Add module.exports = module.exports.default; at the end of the CJS file.

Fix #32

Sorry to still adding PRs to fix the errors caused by #30 I wasn't able to detect the errors in the build results until a new release is published

Add `module.exports = module.exports.default;` at the end of the CJS file.
Example:

```js
const ruleset = require('@apisyouwonthate/style-guide');
```
@savage-alex
Copy link
Contributor

Thanks for the fix Hope we can get it merged soon as I have an OAS2 rule waiting to be pushed.

@philsturgeon philsturgeon merged commit d5828bc into apisyouwonthate:main Sep 22, 2022
@github-actions
Copy link

🎉 This PR is included in version 1.1.10 🎉

The release is available on:

Your semantic-release bot 📦🚀

@philsturgeon
Copy link
Contributor

Thanks @leosuncin, no worries that things haven’t gone smoothly, the main thing is you’re fixing it!

Sorry for delays merging but I’m back to work now and getting this working is a priority for me.

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

Successfully merging this pull request may close these issues.

Linting An OpenAPI 2 / Swagger file gives an error
3 participants