search

iTwin / eslint-plugin

MIT License
0 stars 0 forks source link

remove console.log; simplify logic a tad #65

Closed ben-polinsky closed 5 months ago

ben-polinsky commented 5 months ago

Remove console.logs causing messy lint run. Scale back the scope of the extensions plugin to only deal with extensions once again.

ben-polinsky commented 5 months ago

Holding for additional checks on core...

ben-polinsky commented 5 months ago

@aruniverse I slimmed-down the scope of this extensions rule - previously I had expanded it to flag any exports without a release tag, but looking at core that would cause a number of issues. Also, since only a few core packages would want the extensions rule, it doesn't make sense to combine the two ideas.

This is tested and ready to go. I'll open a PR on the core side once we're through.