Update readme.md links to new site #83
Conversation
Co-authored-by: Spiker985 <[email protected]>
|
This should also close #64 when merged |
|
@kaosine Since I don't have write access, I can't approve the PR to allow it to show it's merge status. You mind taking a look, and hopefully approving? |
|
I sync'd the branch to include your commit from #81 so that shouldn't cause a merge issue now, it was still pending when I made this PR. |
There was a problem hiding this comment.
Since I managed to miss it the first time, here's my approval.
Looks great, and I'll go ahead and remove @kaosine's changes since they seem to be addressed, and you are wanting to check the merge status.
confused-Techie
dismissed
kaosine’s stale review
Changes seem to be addressed. Contributor requesting merge status preview.
|
Alright, this PR has been active for over a month. Since I've already approved it, and it is a non-code changing PR, meaning we only need one approval, I'll go ahead and merge. Thanks for the contribution! |
Links lead to the repo docs, now the new site is live we can redirect the buttons to the general docs section, the installation section (which is yet to be completed) and the the build instructions.
This will clash with #81 and #74 (if the latter is to be merged) but both are easy conflicts to resolve.