Brian Smith

Results 358 comments of Brian Smith
trafficstars

> May I work on this? I am really interested in this issue @Vidit151003 I think you can go ahead and open a pull request to fix this, it doesn't...

I'm going to close this due to inactivity, if you'd like to come back to this, please reply or reopen it! Thanks :)

> > At best, a section advising against using third-party CDN hosts; at a minimum, an honest and natural stance which doesn’t describe any performance benefits. > > For this...

> I've successfully tested the workflow. Created Issue: [OnkarRuikar#29](https://github.com/OnkarRuikar/content/issues/29) The workflow run: https://github.com/OnkarRuikar/content/actions/runs/10591216755/job/29348284353#step:4:45 It looks good, tnx. I think we can merge this shortly. I would also echo the sentiment...

> I ran `sort file | uniq file > file` on the word list files. nice, thank you > How about names `project-words-list.txt` and `ignored-words-list.txt` for the files? IMO "words"...

> IMO this should not be good-first, unless we have the agreement that we can simply remove all of these notes. I'm not comfortable with doing that before this is...

Thanks, Chris. Done with a pass over everything. Mostly the form example is something open-ended if you want to allow validation in the page. Everything else is some prose suggestions...

> Do we really want to hand-maintain this? This seems destined to go out of date. Can it be a macro instead? No, I don't think we want to hand-maintain...

Thanks, all. Merging for now!

Thanks a lot for opening, I think you can go ahead and create a PR for this 👍🏻