diff mbox series

docs: replace 'Edit on GitLab' with 'View page source'

Message ID 20241209163506.2089961-1-berrange@redhat.com (mailing list archive)
State New
Headers show
Series docs: replace 'Edit on GitLab' with 'View page source' | expand

Commit Message

Daniel P. Berrangé Dec. 9, 2024, 4:35 p.m. UTC
QEMU takes contributions via the mailing list, so while you can edit a
file on gitlab and then switch to the terminal to send a patch, the
wording 'Edit on GitLab' strongly suggests we take merge requests.

Switching back to "View page source" is a more agnostic term that does
not imply a particular contribution approach, that we had used in QEMU
before:

  commit 73e6aec6522e1edd63f631c52577b49a39bc234f
  Author: Marc-André Lureau <marcandre.lureau@redhat.com>
  Date:   Tue Mar 23 15:53:28 2021 +0400

    sphinx: adopt kernel readthedoc theme

Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>
---
 docs/conf.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Philippe Mathieu-Daudé Dec. 9, 2024, 4:38 p.m. UTC | #1
On 9/12/24 17:35, Daniel P. Berrangé wrote:
> QEMU takes contributions via the mailing list, so while you can edit a
> file on gitlab and then switch to the terminal to send a patch, the
> wording 'Edit on GitLab' strongly suggests we take merge requests.
> 
> Switching back to "View page source" is a more agnostic term that does
> not imply a particular contribution approach, that we had used in QEMU
> before:
> 
>    commit 73e6aec6522e1edd63f631c52577b49a39bc234f
>    Author: Marc-André Lureau <marcandre.lureau@redhat.com>
>    Date:   Tue Mar 23 15:53:28 2021 +0400
> 
>      sphinx: adopt kernel readthedoc theme
> 

Is this:
Resolves: https://gitlab.com/qemu-project/qemu/-/issues/2709
?

> Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>
> ---
>   docs/conf.py | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/docs/conf.py b/docs/conf.py
> index c11a6ead8a..164a8ee8b2 100644
> --- a/docs/conf.py
> +++ b/docs/conf.py
> @@ -186,7 +186,7 @@
>   ]
>   
>   html_context = {
> -    "display_gitlab": True,
> +    "source_url_prefix": "https://gitlab.com/qemu-project/qemu/-/blob/master/docs/",
>       "gitlab_user": "qemu-project",
>       "gitlab_repo": "qemu",
>       "gitlab_version": "master",
Daniel P. Berrangé Dec. 9, 2024, 4:41 p.m. UTC | #2
On Mon, Dec 09, 2024 at 05:38:49PM +0100, Philippe Mathieu-Daudé wrote:
> On 9/12/24 17:35, Daniel P. Berrangé wrote:
> > QEMU takes contributions via the mailing list, so while you can edit a
> > file on gitlab and then switch to the terminal to send a patch, the
> > wording 'Edit on GitLab' strongly suggests we take merge requests.
> > 
> > Switching back to "View page source" is a more agnostic term that does
> > not imply a particular contribution approach, that we had used in QEMU
> > before:
> > 
> >    commit 73e6aec6522e1edd63f631c52577b49a39bc234f
> >    Author: Marc-André Lureau <marcandre.lureau@redhat.com>
> >    Date:   Tue Mar 23 15:53:28 2021 +0400
> > 
> >      sphinx: adopt kernel readthedoc theme
> > 
> 
> Is this:
> Resolves: https://gitlab.com/qemu-project/qemu/-/issues/2709
> ?

I won't really claim it resolves that, as that issues is showing
lots of user confusion, most of which is not in our control.

> 
> > Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>
> > ---
> >   docs/conf.py | 2 +-
> >   1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/docs/conf.py b/docs/conf.py
> > index c11a6ead8a..164a8ee8b2 100644
> > --- a/docs/conf.py
> > +++ b/docs/conf.py
> > @@ -186,7 +186,7 @@
> >   ]
> >   html_context = {
> > -    "display_gitlab": True,
> > +    "source_url_prefix": "https://gitlab.com/qemu-project/qemu/-/blob/master/docs/",
> >       "gitlab_user": "qemu-project",
> >       "gitlab_repo": "qemu",
> >       "gitlab_version": "master",
> 

With regards,
Daniel
Pierrick Bouvier Dec. 9, 2024, 6:57 p.m. UTC | #3
On 12/9/24 08:35, Daniel P. Berrangé wrote:
> QEMU takes contributions via the mailing list, so while you can edit a
> file on gitlab and then switch to the terminal to send a patch, the
> wording 'Edit on GitLab' strongly suggests we take merge requests.
> 
> Switching back to "View page source" is a more agnostic term that does
> not imply a particular contribution approach, that we had used in QEMU
> before:
> 
>    commit 73e6aec6522e1edd63f631c52577b49a39bc234f
>    Author: Marc-André Lureau <marcandre.lureau@redhat.com>
>    Date:   Tue Mar 23 15:53:28 2021 +0400
> 
>      sphinx: adopt kernel readthedoc theme
> 
> Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>
> ---
>   docs/conf.py | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/docs/conf.py b/docs/conf.py
> index c11a6ead8a..164a8ee8b2 100644
> --- a/docs/conf.py
> +++ b/docs/conf.py
> @@ -186,7 +186,7 @@
>   ]
>   
>   html_context = {
> -    "display_gitlab": True,
> +    "source_url_prefix": "https://gitlab.com/qemu-project/qemu/-/blob/master/docs/",
>       "gitlab_user": "qemu-project",
>       "gitlab_repo": "qemu",
>       "gitlab_version": "master",

Reviewed-by: Pierrick Bouvier <pierrick.bouvier@linaro.org>
diff mbox series

Patch

diff --git a/docs/conf.py b/docs/conf.py
index c11a6ead8a..164a8ee8b2 100644
--- a/docs/conf.py
+++ b/docs/conf.py
@@ -186,7 +186,7 @@ 
 ]
 
 html_context = {
-    "display_gitlab": True,
+    "source_url_prefix": "https://gitlab.com/qemu-project/qemu/-/blob/master/docs/",
     "gitlab_user": "qemu-project",
     "gitlab_repo": "qemu",
     "gitlab_version": "master",