osdir.com

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: calcite-avatica git commit: [CALCITE-2412] Add appveyor.yml to have tests on Windows against jdk1.8, jdk9, jdk10 Add Appveyor badge Add -DskipDockerCheck because of CALCITE-2385 and to make it sync with travis.yml


Sergey,

Can you please force-push to clean up the commit message. You didn’t leave a space between the first and second lines, so git created a multi-line summary. We need a simple, descriptive one-liner (ideally describing everything in the change, albeit at a high level). 

Sorry to harp on to everyone about commit messages. But a clear code history is essential for future contributors, and clear release notes are essential for our users.

Julian


> On Nov 19, 2018, at 5:38 AM, snuyanzin@xxxxxxxxxx wrote:
> 
> Repository: calcite-avatica
> Updated Branches:
>  refs/heads/master 9199fef02 -> 12eea22f1
> 
> 
> [CALCITE-2412] Add appveyor.yml to have tests on Windows against jdk1.8, jdk9, jdk10
> Add Appveyor badge
> Add -DskipDockerCheck because of CALCITE-2385 and to make it sync with travis.yml
> 
> Close apache/calcite-avatica#65
> 
> 
> Project: http://git-wip-us.apache.org/repos/asf/calcite-avatica/repo
> Commit: http://git-wip-us.apache.org/repos/asf/calcite-avatica/commit/12eea22f
> Tree: http://git-wip-us.apache.org/repos/asf/calcite-avatica/tree/12eea22f
> Diff: http://git-wip-us.apache.org/repos/asf/calcite-avatica/diff/12eea22f
> 
> Branch: refs/heads/master
> Commit: 12eea22f101780a68325a1595774e269616b4ec0
> Parents: 9199fef
> Author: snuyanzin <snuyanzin@xxxxxxxxx>
> Authored: Wed Jul 11 10:26:01 2018 +0300
> Committer: snuyanzin <snuyanzin@xxxxxxxxx>
> Committed: Mon Nov 19 16:33:57 2018 +0300
> 
> ----------------------------------------------------------------------
> README.md    |  1 +
> appveyor.yml | 43 +++++++++++++++++++++++++++++++++++++++++++
> 2 files changed, 44 insertions(+)
> ----------------------------------------------------------------------
> 
> 
> http://git-wip-us.apache.org/repos/asf/calcite-avatica/blob/12eea22f/README.md
> ----------------------------------------------------------------------
> diff --git a/README.md b/README.md
> index f5c84e7..ffd1ab0 100644
> --- a/README.md
> +++ b/README.md
> @@ -17,6 +17,7 @@ limitations under the License.
> {% endcomment %}
> -->
> [![Build Status](https://travis-ci.org/apache/calcite-avatica.svg?branch=master)](https://travis-ci.org/apache/calcite-avatica)
> +[![Build Status: Windows](https://ci.appveyor.com/api/projects/status/gjd9n5gjbldt0gg8/branch/master?svg=true)](https://ci.appveyor.com/project/ApacheSoftwareFoundation/calcite-avatica)
> 
> # Apache Calcite -- Avatica
> 
> 
> http://git-wip-us.apache.org/repos/asf/calcite-avatica/blob/12eea22f/appveyor.yml
> ----------------------------------------------------------------------
> diff --git a/appveyor.yml b/appveyor.yml
> new file mode 100644
> index 0000000..cd397a2
> --- /dev/null
> +++ b/appveyor.yml
> @@ -0,0 +1,43 @@
> +# Configuration file for Appveyor continuous integration.
> +#
> +# Licensed to the Apache Software Foundation (ASF) under one or more
> +# contributor license agreements.  See the NOTICE file distributed with
> +# this work for additional information regarding copyright ownership.
> +# The ASF licenses this file to you under the Apache License, Version 2.0
> +# (the "License"); you may not use this file except in compliance with
> +# the License.  You may obtain a copy of the License at
> +#
> +# http://www.apache.org/licenses/LICENSE-2.0
> +#
> +# Unless required by applicable law or agreed to in writing, software
> +# distributed under the License is distributed on an "AS IS" BASIS,
> +# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
> +# See the License for the specific language governing permissions and
> +# limitations under the License.
> +#
> +version: '{build}'
> +image: Visual Studio 2017
> +clone_depth: 10000
> +# branches to build
> +branches:
> +  # whitelist
> +  only:
> +    - master
> +    - new-master
> +    - javadoc
> +    - /^branch-.*$/
> +    - /^[0-9]+-.*$/
> +matrix:
> +  fast_finish: true  
> +environment:
> +  matrix:  
> +    - JAVA_HOME: C:\Program Files\Java\jdk1.8.0
> +    - JAVA_HOME: C:\Program Files\Java\jdk9
> +    - JAVA_HOME: C:\Program Files\Java\jdk10        
> +build_script:
> +  - mvn clean -V install -DskipTests -Dmaven.javadoc.skip=true -Djavax.net.ssl.trustStorePassword=changeit -DskipDockerCheck
> +test_script:
> +  - mvn -Dsurefire.useFile=false -Djavax.net.ssl.trustStorePassword=changeit verify javadoc:javadoc javadoc:test-javadoc -DskipDockerCheck
> +cache:
> +  - C:\maven\
> +  - C:\Users\appveyor\.m2
>