-
-
Notifications
You must be signed in to change notification settings - Fork 72
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
Update documentation on the league website to match the README on the ~3.0 branch #448
Comments
Duplicate of #362 |
Got fooled by that comment... |
The website is open source, and open to PRs. |
Yeah, for 3 years. If you're not going to fix your own release, and I don't have the time to give you that (otherwise I would have already, you've already suggested that I should), just redirect it. Stop fooling others into the same trap. |
Hardly a trap. v2 still exists, and you can use it exactly how the website says. It even tells you how to install it. |
So the suggestion is to use an old (unmaintained?) version just because
that's the one with documentation up to date, even if the next version was
released 5 years ago?
Seriously, what's wrong with getting the website down instead of leaving it
outdated forever there? If you're not going to update it, remove. README is
enough and we're used to reading long README files, just don't leave
misleading stuff on the web.
…On Mon, Jun 22, 2020 at 6:28 AM Graham Campbell ***@***.***> wrote:
Hardly a trap. v2 still exists, and you can use it exactly how the website
says. It even tells you how to install it.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#448 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAEB6S4MV52MB2FJ4BIWVYTRX4P33ANCNFSM4EBTHEZA>
.
|
I suppose I could also test version 2.1.x against PHP 7. It should work just fine. |
Not sure the existing deployment method from markdown to html on the league website, but the current site on the league website is showing documentation for v2. This is pretty confusing because it heavily relies on a Facade class that I think is no longer in use in v3.
The text was updated successfully, but these errors were encountered: