Skip to content
This repository was archived by the owner on Sep 27, 2023. It is now read-only.

chore: Convert existing CI workflows to maven #376

Merged
merged 25 commits into from
Nov 29, 2022
Merged

Conversation

lqiu96
Copy link
Contributor

@lqiu96 lqiu96 commented Nov 17, 2022

This PR is to convert existing CI workflows to maven. There are a few changes in this PR:

  • Remove the old, unused CIs
  • Replace most of the jobs with Github Actions
  • Add linting, formatting, and checkstyles
  • Set the required CIs via sync-repo-settings.json

@lqiu96 lqiu96 requested a review from blakeli0 November 18, 2022 18:43
@lqiu96 lqiu96 marked this pull request as ready for review November 18, 2022 18:43
@lqiu96 lqiu96 requested a review from a team as a code owner November 18, 2022 18:43
@lqiu96
Copy link
Contributor Author

lqiu96 commented Nov 22, 2022

We are going to determine if the java.header file should have Google, Inc. vs Google, LLC.

Comment on lines +2 to +29
^ \* Copyright 20\d\d,? Google (Inc\.|LLC)( All [rR]ights [rR]eserved\.)?$
^ \*$
^ \* Licensed 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$
^ \* Redistribution and use in source and binary forms, with or without$
^ \* modification, are permitted provided that the following conditions are$
^ \* met:$
^ \*$
^ \*[ ]+https?://www.apache.org/licenses/LICENSE-2\.0$
^ \* \* Redistributions of source code must retain the above copyright$
^ \* notice, this list of conditions and the following disclaimer.$
^ \* \* Redistributions in binary form must reproduce the above$
^ \* copyright notice, this list of conditions and the following disclaimer$
^ \* in the documentation and/or other materials provided with the$
^ \* distribution.$
^ \* \* Neither the name of Google (Inc\.|LLC) nor the names of its$
^ \* contributors may be used to endorse or promote products derived from$
^ \* this software without specific prior written permission.$
^ \*$
^ \* 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\.$
^ \*/$
^ \* THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS$
^ \* "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT$
^ \* LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR$
^ \* A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT$
^ \* OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,$
^ \* SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES \(INCLUDING, BUT NOT$
^ \* LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,$
^ \* DATA, OR PROFITS; OR BUSINESS INTERRUPTION\) HOWEVER CAUSED AND ON ANY$
^ \* THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT$
^ \* \(INCLUDING NEGLIGENCE OR OTHERWISE\) ARISING IN ANY WAY OUT OF THE USE$
^ \* OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.$
^ \*/$
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is the same as license-header-javadoc.txt which gradle has been using. Nothing about the license changes except that maven uses this file.

I've updated it to accept Google Inc and LLC

@lqiu96
Copy link
Contributor Author

lqiu96 commented Nov 29, 2022

I'm going to temporarily re-add in the old ci/units and ci/format jobs to make the status pass. I'm also going to remove all the required status checks except for CLA.

I'll create a separate PR to remove the old workflows and update the status checks.

On second thought, I probably don't need to remove the required status checks..

@lqiu96 lqiu96 merged commit ff7db47 into main Nov 29, 2022
@lqiu96 lqiu96 deleted the main-maven_ci_jobs branch November 29, 2022 15:31
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants