Closed IanFindlay closed 3 years ago
This pull request will be automatically merged by worlddriven in 9 days and 22 hours.
The start date is based on the latest Commit date / Pull Request created date / (force) Push date.
The time to merge is 5 days plus 5 days for each commit.
Check the worlddriven
status check or the dashboard for actual stats.
To speed up or delay the merge review the pull request:
@TooAngel I forgot to branch this PR meaning that my fork will be harder to sync when other commits occur. Would you rather I closed this PR and opened a new one from the right branch or should I just leave it as it? Is there an easy way to get the commit history of my master back to being the same as yours without moving these changes to a branch?
Actually, it doesn't matter (for me) from which branch you are sending the PR. Also, the commit history looks good from my perspective.
To make sure that your commits sit on top of the origin branch you can use git rebase
. (https://git-scm.com/book/en/v2/Git-Branching-Rebasing)
For this you need to have the tooangel/worlddriven
repository added as remote.
If you have questions let me know.
Thank you for the review. This pull request will be automatically merged by worlddriven in 9 days, votes 1/178.
Check the worlddriven
status checks or the dashboard for actual stats.
To make sure that your commits sit on top of the origin branch you can use
git rebase
. (https://git-scm.com/book/en/v2/Git-Branching-Rebasing)
Thanks for this link. I'll let this merge then rebase if required.
This pull request was merged by worlddriven.
Fixes #168 and also updates a few other elements, adds some CSS styling to the first list and corrects a few typos.
Also closes #153 .