[GitHub] [velocity-site] arkanovicz commented on pull request #1: Update docu about contributing patches/PRs

2021-01-28 Thread GitBox
arkanovicz commented on pull request #1: URL: https://github.com/apache/velocity-site/pull/1#issuecomment-76910 Sorry for the branches error. Yes, you have to use master. The default branch on github is still trunk but the fix is ongoing. The cloning of the production branch now

[GitHub] [velocity-site] arkanovicz merged pull request #4: Site builder

2021-01-28 Thread GitBox
arkanovicz merged pull request #4: URL: https://github.com/apache/velocity-site/pull/4 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go

[GitHub] [velocity-site] arkanovicz opened a new pull request #4: Site builder

2021-01-28 Thread GitBox
arkanovicz opened a new pull request #4: URL: https://github.com/apache/velocity-site/pull/4 Dockerized site builder. This is an automated message from the Apache Git Service. To respond to the message, please log on to G

[GitHub] [velocity-site] martin-g edited a comment on pull request #1: Update docu about contributing patches/PRs

2021-01-28 Thread GitBox
martin-g edited a comment on pull request #1: URL: https://github.com/apache/velocity-site/pull/1#issuecomment-769076194 It almost works! 1) I needed to use `trunk` branch. I am not sure whether this is correct. See my previous message above 2) The `get-involved.html` file is properly

[GitHub] [velocity-site] martin-g commented on pull request #1: Update docu about contributing patches/PRs

2021-01-28 Thread GitBox
martin-g commented on pull request #1: URL: https://github.com/apache/velocity-site/pull/1#issuecomment-769076194 It almost works! 1) I needed to use `trunk` branch. I am not sure whether this is correct. See my previous message above 2) The `get-involved.html` file is properly recrea

[GitHub] [velocity-site] martin-g commented on pull request #1: Update docu about contributing patches/PRs

2021-01-28 Thread GitBox
martin-g commented on pull request #1: URL: https://github.com/apache/velocity-site/pull/1#issuecomment-769069374 I am testing it now! But you pushed to `trunk` branch while the README file says: `Note: the branch trnk is obsolete. Please use the master branch.` ---

[GitHub] [velocity-site] arkanovicz commented on pull request #1: Update docu about contributing patches/PRs

2021-01-28 Thread GitBox
arkanovicz commented on pull request #1: URL: https://github.com/apache/velocity-site/pull/1#issuecomment-769053568 The dockerized site builder is functional. The single shell script `velocity-site/builder/bin/builder.sh` should do all the work. I think your setup was lacking

[GitHub] [velocity-site] arkanovicz merged pull request #3: Site builder

2021-01-28 Thread GitBox
arkanovicz merged pull request #3: URL: https://github.com/apache/velocity-site/pull/3 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go

[GitHub] [velocity-site] arkanovicz opened a new pull request #3: Site builder

2021-01-28 Thread GitBox
arkanovicz opened a new pull request #3: URL: https://github.com/apache/velocity-site/pull/3 Dockerized site builder. This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHu