Skip to content

Improve project documentation (Issue #153)#159

Merged
TooAngel merged 1 commit intoTooAngel:masterfrom
IanFindlay:docs/index
Oct 2, 2020
Merged

Improve project documentation (Issue #153)#159
TooAngel merged 1 commit intoTooAngel:masterfrom
IanFindlay:docs/index

Conversation

@IanFindlay
Copy link
Contributor

@IanFindlay IanFindlay commented Oct 1, 2020

Here is my proofread/edit of the index.html file addressing issue #153 . I hope it is useful to you, feel free to let me know if there are any changes you would like to see.

As a side note @TooAngel , your way of using line breaks instead of <p> tags in the html is, I think, a little bit dated. Converting that might be a worthwhile addition, but if you are happy with it as is I won't bother making the issue and/or changing it.

@worlddrivenbot
Copy link
Collaborator

This pull request will be automatically merged by worlddriven in 9 days and 23 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:

  1. Files changed
  2. Review changes
  • Speed up: Approve
  • Delay or stop: Request changes

Copy link
Owner

@TooAngel TooAngel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice, thx a lot.

Happy if you suggest also line break improvements, I'm pretty sure I'm a bit dated in regards to HTML.

On a side note: I think you understood the world driven idea. Are you aware that as soon your PR gets merged, you have also a certain 'power' to speed up or stop future PRs? :-)

@worlddrivenbot
Copy link
Collaborator

Thank you for the review.
This pull request will be automatically merged by worlddriven in 9 days, votes 0/175.

        Check the `worlddriven` status checks or the [dashboard](https://www.worlddriven.org/TooAngel/worlddriven/pull/159) for actual stats.

@IanFindlay
Copy link
Contributor Author

Happy if you suggest also line break improvements, I'm pretty sure I'm a bit dated in regards to HTML.

I think I may very well raise that as an issue and get going on it once this one is merged. It's a small change, but I think modernising the HTML a bit will also help with future additions.

On a side note: I think you understood the world driven idea. Are you aware that as soon your PR gets merged, you have also a certain 'power' to speed up or stop future PRs? :-)

I am, I'm looking forward to seeing how this works in practice as the theory seems rather interesting. You can almost certainly expect my input in the areas I have some knowledge of but much of this is currently unfamiliar to me.

@TooAngel
Copy link
Owner

TooAngel commented Oct 1, 2020

:-) Sounds good, both

@TooAngel TooAngel mentioned this pull request Oct 1, 2020
@TooAngel TooAngel merged commit 9191133 into TooAngel:master Oct 2, 2020
@TooAngel
Copy link
Owner

TooAngel commented Oct 2, 2020

This pull request was merged by worlddriven.

@IanFindlay IanFindlay deleted the docs/index branch October 2, 2020 13:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants