-
Notifications
You must be signed in to change notification settings - Fork 61
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
Broken Links #173
Comments
Do you know of a pre commit tool or anything to check spelling (I guess code spell) and broken links? We could add that to the code base. |
I tend to use a couple of tools, which, however, I let to be non-blocking in both pre-commit and GHA due to false positives that may be flagged.
|
Interesting. So you run them on PRs but they are non-blocking? |
Finally closed this. Thanks for reporting, Fra! I'll continue the checks/typos discussion in #202. |
There is a tiny list of broken links in the repo which I wouldn't know how to fix:
docs/forecasting.md
ln82 -> https://composingprograms.com/pages/16-higher-order-functions.html#curryingfunctime/llm/common
ln 122 -> https://github.com/openai/openai-python/blob/main/chatml.mdIf you know how to replace them, it could be good to do so in #172
The text was updated successfully, but these errors were encountered: