mbox series

[ovs-dev,v2,0/4] Fix URLS and and associated markups.

Message ID 1524147432-3306-1-git-send-email-ian.stokes@intel.com
Headers show
Series Fix URLS and and associated markups. | expand

Message

Stokes, Ian April 19, 2018, 2:17 p.m. UTC 
Running make check-docs has caught a number of URLs that are no longer
active. This patch series updates the required URLs along with some mark-up
changes in the docs in relation to said URLs.

I originally looked at doing this in a single patch however as the URLs
are spread throughout the documentation and relate to different aspects
of OVS I thought it better to break this into 4 patches as the changes to
index and DocumentationStyle.rst may have more input than the others.

Ian Stokes (4):
  docs: Fix sflow documentation url and markup.
  docs: Fix style guide url in DocumentationStyle.rst.
  docs: Fix sphinx urls.
  docs: Fix urls in index.rst.

 Documentation/howto/sflow.rst                      |   9 +-
 .../internals/contributing/documentation-style.rst |   7 +-
 Documentation/intro/install/documentation.rst      |   4 +-
 Documentation/ref/index.rst                        | 214 ++++++++++-----------
 4 files changed, 117 insertions(+), 117 deletions(-)

Comments

Ben Pfaff April 19, 2018, 4:18 p.m. UTC | #1 
On Thu, Apr 19, 2018 at 03:17:08PM +0100, Ian Stokes wrote:
> Running make check-docs has caught a number of URLs that are no longer
> active. This patch series updates the required URLs along with some mark-up
> changes in the docs in relation to said URLs.
> 
> I originally looked at doing this in a single patch however as the URLs
> are spread throughout the documentation and relate to different aspects
> of OVS I thought it better to break this into 4 patches as the changes to
> index and DocumentationStyle.rst may have more input than the others.

These all seem good to me.

Do you want me to apply them directly or do you prefer to roll them into
your next pull request?
Stokes, Ian April 19, 2018, 6:46 p.m. UTC | #2 
> On Thu, Apr 19, 2018 at 03:17:08PM +0100, Ian Stokes wrote:
> > Running make check-docs has caught a number of URLs that are no longer
> > active. This patch series updates the required URLs along with some
> > mark-up changes in the docs in relation to said URLs.
> >
> > I originally looked at doing this in a single patch however as the
> > URLs are spread throughout the documentation and relate to different
> > aspects of OVS I thought it better to break this into 4 patches as the
> > changes to index and DocumentationStyle.rst may have more input than the
> others.
> 
> These all seem good to me.
> 
> Do you want me to apply them directly or do you prefer to roll them into
> your next pull request?

Thanks Ben, I've started prepping a pull request for tomorrow, I can roll these into it as well in that.

Thanks
Ian