Re: [PATCH v2 2/3] SubmittingPatches: replace discussion of Travis with GitHub Actions
On Mon, Jun 07, 2021 at 01:03:42PM +0200, Ævar Arnfjörð Bjarmason wrote:
Show 36 quoted lines
> Replace the discussion of Travis CI added in
> 0e5d028a7a0 (Documentation: add setup instructions for Travis CI,
> 2016-05-02) with something that covers the GitHub Actions added in
> 889cacb6897 (ci: configure GitHub Actions for CI/PR, 2020-04-11).
>
> The setup is trivial compared to using Travis, and it even works on
> Windows (that "hopefully soon" comment was probably out-of-date on
> Travis as well).
>
> Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
> ---
> Documentation/SubmittingPatches | 44 ++++++++++++---------------------
> 1 file changed, 16 insertions(+), 28 deletions(-)
>
> diff --git a/Documentation/SubmittingPatches b/Documentation/SubmittingPatches
> index 2643062624..e372d17673 100644
> --- a/Documentation/SubmittingPatches
> +++ b/Documentation/SubmittingPatches
> @@ -74,10 +74,9 @@ the feature triggers the new behavior when it should, and to show the
> feature does not trigger when it shouldn't. After any code change, make
> sure that the entire test suite passes.
>
> -If you have an account at GitHub (and you can get one for free to work
> -on open source projects), you can use their Travis CI integration to
> -test your changes on Linux, Mac (and hopefully soon Windows). See
> -GitHub-Travis CI hints section for details.
> +Pushing to a fork of https://github.com/git/git will use their CI
> +integration to test your changes on Linux, Mac and Windows. See the
> +GitHub CI section for details.
>
> Do not forget to update the documentation to describe the updated
> behavior and make sure that the resulting documentation set formats
> @@ -451,12 +450,12 @@ their trees themselves.
> the status of various proposed changes.
>
> [[travis]]
What about this occurence of "travis"?
Show 31 quoted lines
> -== GitHub-Travis CI hints
> +== GitHub CI
>
> -With an account at GitHub (you can get one for free to work on open
> -source projects), you can use Travis CI to test your changes on Linux,
> -Mac (and hopefully soon Windows). You can find a successful example
> -test build here: https://travis-ci.org/git/git/builds/120473209
> +With an account at GitHub, you can use GitHub CI to test your changes
> +on Linux, Mac and Windows. See
> +https://github.com/git/git/actions/workflows/main.yml for examples of
> +recent CI runs.
>
> Follow these steps for the initial setup:
>
> @@ -464,31 +463,20 @@ Follow these steps for the initial setup:
> You can find detailed instructions how to fork here:
> https://help.github.com/articles/fork-a-repo/
>
> -. Open the Travis CI website: https://travis-ci.org
> -
> -. Press the "Sign in with GitHub" button.
> -
> -. Grant Travis CI permissions to access your GitHub account.
> - You can find more information about the required permissions here:
> - https://docs.travis-ci.com/user/github-oauth-scopes
> -
> -. Open your Travis CI profile page: https://travis-ci.org/profile
> -
> . Enable Travis CI builds for your Git fork.
>
> After the initial setup, Travis CI will run whenever you push new changes
And what about these two mentions of Travis CI in the context?
> to your fork of Git on GitHub. You can monitor the test state of all your
> -branches here: https://travis-ci.org/__<Your GitHub handle>__/git/branches
> +branches here: https://github.com/<Your GitHub handle>/git/actions/workflows/main.yml
SZEDER Gábor <szeder.dev@gmail.com>