Denis Žoljom

Results 295 comments of Denis Žoljom

Should this be added in the ruleset? The ```xml ``` Or should we leave this out? @justintadlock

Would love to hear if there is some new info about this 🙂

I'll tag this as next release, so that we have 0.2.0 out before WCEU (and not to block Theme Sniffer new release) 🙂

This is good to filter out the obvious minified files. But we'll still have issues with minified files that don't end in `.min.js` or `.dev.js`. Definitely something we'd like for...

Hi! Do we have a decision on what CDN urls should be allowed? We should have a definite list so that we can update the #108 PR. We should brought...

Then we should remove the scripts that are in the whitelist section of this PR and only allow google fonts.

Since we have a list of things to allow, and disallow, I'll remove the decision needed tag, so that we know this can be worked on.

This is a known bug and Juliette mentioned how to override it in [her comment above](https://github.com/WPTRT/WPThemeReview/issues/228#issuecomment-523096201). Another thing you can do is what we've done in Twenty Twenty theme: https://github.com/WordPress/twentytwenty/blob/f257dfff755c415dadc56a2c70a412421f39aad7/composer.json#L25-L28:...

Hmmm, I haven't thought of `Templates` folder. I could place `Reserved Template Name Sniff` (PR #213) in there then...

I'm fine leaving this in `Templates` category. I even moved the reserved template name sniff to this category in the PR 🙂 :+1: As for the name, I was also...