update dev instructions #98

Merged
6543 merged 2 commits from pat-s/theme:update-dev-instructions into master 2021-09-12 17:08:33 +00:00
Member
  • Format markdown (one sentence per line & some more lints)
  • Update dev instructions (hopefully simplifying them)

fix #96

- Format markdown (one sentence per line & some more lints) - Update dev instructions (hopefully simplifying them) fix #96
pat-s added 1 commit 2021-08-07 14:08:25 +00:00
All checks were successful
continuous-integration/drone/pr Build is passing
6dd2206b0c
update dev instructions
lunny reviewed 2021-08-08 02:19:03 +00:00
README.md Outdated
@ -13,3 +12,1 @@
just download our prebuilt [theme](https://dl.gitea.io/theme/master.tar.gz), put
it into your `themes/gitea` folder and enable the theme with the
`theme = "gitea"` option of your website.
You need an existing [Hugo](https://github.com/spf13/hugo) website, than you can just download our prebuilt [theme](https://dl.gitea.io/theme/master.tar.gz), put it into your `themes/gitea` folder and enable the theme with the `theme = "gitea"` option of your website.

We need new lines by ourselves.

We need new lines by ourselves.
Author
Member

What do you mean with "we need need lines"?

What do you mean with "we need need lines"?

I mean you have to add new line characters on the article.

I mean you have to add new line characters on the article.
Author
Member

? Sorry, I still don't understand what you mean.

? Sorry, I still don't understand what you mean.

I think he revers to (two space) at the end of the line to enforce line break in markdown

I think he revers to ` ` (two space) at the end of the line to enforce line break in markdown
Author
Member

Two spaces also have an effect even if there is a line break.
The "one sentence per line" thing is somewhat unrelated to this and simplifies reading and review purposes.

No strong opinion though, this should not become a long discussion :)

Two spaces also have an effect even if there is a line break. The "one sentence per line" thing is somewhat unrelated to this and simplifies reading and review purposes. No strong opinion though, this should not become a long discussion :)
6543 reviewed 2021-08-10 00:40:11 +00:00
README.md Outdated
@ -63,3 +55,3 @@
## Copyright
```
```sh
-sh
+none
```diff -sh +none ```
6543 marked this conversation as resolved
pat-s added 1 commit 2021-08-11 06:17:38 +00:00
All checks were successful
continuous-integration/drone/pr Build is passing
803e3cafe1
sh -> none, remove whitespace
Author
Member

Summoning @techknowlogick and @6543 for review?

Summoning @techknowlogick and @6543 for review?
6543 approved these changes 2021-08-28 19:33:42 +00:00
axifive approved these changes 2021-09-12 17:03:46 +00:00
6543 merged commit 576e8a137f into master 2021-09-12 17:08:33 +00:00
6543 referenced this issue from a commit 2021-09-12 17:08:33 +00:00
pat-s deleted branch update-dev-instructions 2021-09-12 17:45:40 +00:00
This repo is archived. You cannot comment on pull requests.
No description provided.