Drop wrong api_wrappers from modules README.md #1212
Conversation
There is no api_wrappers folder inside this folder. The api_wrappers folder is in the sourced folder, not the modules folder. The AWS link is invalid, empty. Wolfram Alpha exists under the sourced/api_wrappers folder, where it was moved to from a root api_wrappers folder in c47ccd4.
| ## api_wrappers | ||
|
|
||
| Demo of various API wrappers: | ||
| - [Wolfram Alpha](../sourced/api_wrappers/wolframalpha.nu) |
There was a problem hiding this comment.
This one points to the correct folder, but it's outside of modules.
I would expect the modules README to show modules, not other things as well.
|
I see now there is more pointing to sourced - which is surprising and confusing to me. There's also many references to folders that exist in modules, next to the readme, missing. What's the intention here? How should it be? |
|
whatever works. we're not too picky. I'm fine with sourcing files or using them. |
|
And you're fine with calling them modules either way? And them being listed in a readme in a folder that doesn't contain all list items? |
|
No, scripts shouldn't be called modules. The proper fix is to move the script over to a module folder and tweak it to make it a module, if necessary. |
2 participants
There is no api_wrappers folder inside this folder.
The api_wrappers folder is in the sourced folder, not the modules folder.
The AWS link is invalid, empty. Wolfram Alpha exists under the sourced/api_wrappers folder, where it was moved to from a root api_wrappers folder in c47ccd4.