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

ReadMe.Remove non-working Typescript code that is very confusing. #233

Open
CookingWithCale opened this issue Sep 3, 2022 · 0 comments
Open

Comments

@CookingWithCale
Copy link

You should never, ever, copy and paste demo code and it doesn't compile. I write compilers, I've read RSS feeds for decades, but I can't figure out what you're code is even doing. You don't explain what 'baz' is, why I would need it. I understand that I can remove the non-working code and it will run and then maybe I'll figure out what you're talking about, but I can't really that when the code doesn't compile. How do I parse multiple RSS feeds? What is a Custom feed? Is that different than a regular feed? How is a custom feed different than a regular feed? The naming conventions here suck really bad. As an expert programmer, I should be able to look at your code and understand what does what, but I can't do that when you use generic placeholders and don't explain your nomiclature. When you use placeholders like Foo, Bar, they are supposed to be when you only require a generic placeholder. When you are dealing with concrete examples, you need to use ConcretePlaceholders. What does 'foo' mean?

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

No branches or pull requests

1 participant