You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@accumulo.apache.org by mikewalch <gi...@git.apache.org> on 2016/04/13 20:31:28 UTC

[GitHub] accumulo pull request: ACCUMULO-4189 - Added direct links to docum...

GitHub user mikewalch opened a pull request:

    https://github.com/apache/accumulo/pull/92

    ACCUMULO-4189 - Added direct links to documentation on Accumulo website

    #91 should be reviewed/merged first before this PR

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/mikewalch/accumulo readme-updates

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/accumulo/pull/92.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #92
    
----
commit 604df753ca108e1c347d2137bb52556785001462
Author: Mike Walch <mw...@gmail.com>
Date:   2016-04-11T21:24:23Z

    ACCUMULO-4189 - Added direct links to documentation on Accumulo website

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] accumulo pull request: ACCUMULO-4189 - Added direct links to docum...

Posted by mikewalch <gi...@git.apache.org>.
Github user mikewalch commented on the pull request:

    https://github.com/apache/accumulo/pull/92#issuecomment-210143136
  
    In 17288b37bab2b42, I fixed the README to point to latest pages that redirect to current version.  I also made the links to the Accumulo website use https. 


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] accumulo pull request: ACCUMULO-4189 - Added direct links to docum...

Posted by ctubbsii <gi...@git.apache.org>.
Github user ctubbsii commented on a diff in the pull request:

    https://github.com/apache/accumulo/pull/92#discussion_r59631999
  
    --- Diff: README.md ---
    @@ -110,5 +106,10 @@ with Accumulo 1.6.2 and 1.7.0 all API changes will follow [semver 2.0][12]
     [7]: http://thrift.apache.org/
     [8]: http://accumulo.apache.org/notable_features.html
     [9]: http://maven.apache.org/
    +[10]: http://accumulo.apache.org/user_manual.html
    --- End diff --
    
    These should use https wherever possible. I know some of the older links don't have it, but they should. At the very least, new ones should.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] accumulo pull request: ACCUMULO-4189 - Added direct links to docum...

Posted by ctubbsii <gi...@git.apache.org>.
Github user ctubbsii commented on a diff in the pull request:

    https://github.com/apache/accumulo/pull/92#discussion_r60314647
  
    --- Diff: README.md ---
    @@ -101,14 +97,19 @@ The Accumulo project maintains binary compatibility across this API within a
     major release, as defined in the Java Language Specification 3rd ed. Starting
     with Accumulo 1.6.2 and 1.7.0 all API changes will follow [semver 2.0][12]
     
    -[1]: http://accumulo.apache.org
    +[1]: https://accumulo.apache.org
     [2]: INSTALL.md
     [3]: TESTING.md
     [4]: http://research.google.com/archive/bigtable.html
     [5]: http://hadoop.apache.org
     [6]: http://zookeeper.apache.org
     [7]: http://thrift.apache.org/
    -[8]: http://accumulo.apache.org/notable_features.html
    +[8]: https://accumulo.apache.org/notable_features.html
     [9]: http://maven.apache.org/
    +[10]: https://accumulo.apache.org/user-manual/latest/
    +[11]: https://accumulo.apache.org/examples/latest/
    --- End diff --
    
    [nit] Adds trailing whitespace. (some above, also)


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] accumulo pull request: ACCUMULO-4189 - Added direct links to docum...

Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:

    https://github.com/apache/accumulo/pull/92


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] accumulo pull request: ACCUMULO-4189 - Added direct links to docum...

Posted by mikewalch <gi...@git.apache.org>.
Github user mikewalch commented on a diff in the pull request:

    https://github.com/apache/accumulo/pull/92#discussion_r60592421
  
    --- Diff: README.md ---
    @@ -101,14 +97,19 @@ The Accumulo project maintains binary compatibility across this API within a
     major release, as defined in the Java Language Specification 3rd ed. Starting
     with Accumulo 1.6.2 and 1.7.0 all API changes will follow [semver 2.0][12]
     
    -[1]: http://accumulo.apache.org
    +[1]: https://accumulo.apache.org
     [2]: INSTALL.md
     [3]: TESTING.md
     [4]: http://research.google.com/archive/bigtable.html
     [5]: http://hadoop.apache.org
     [6]: http://zookeeper.apache.org
     [7]: http://thrift.apache.org/
    -[8]: http://accumulo.apache.org/notable_features.html
    +[8]: https://accumulo.apache.org/notable_features.html
     [9]: http://maven.apache.org/
    +[10]: https://accumulo.apache.org/user-manual/latest/
    +[11]: https://accumulo.apache.org/examples/latest/
    --- End diff --
    
    Removed trailing whitespace in b3aad3d96


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---