You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@arrow.apache.org by th...@apache.org on 2023/06/26 13:58:18 UTC

[arrow] branch main updated: GH-36299: [R][CI] Remove pkgdown check CI step (#36300)

This is an automated email from the ASF dual-hosted git repository.

thisisnic pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/arrow.git


The following commit(s) were added to refs/heads/main by this push:
     new bd8fd0cb1b GH-36299: [R][CI] Remove pkgdown check CI step (#36300)
bd8fd0cb1b is described below

commit bd8fd0cb1b2f61fa80891eb68fce23a062f48e11
Author: Nic Crane <th...@gmail.com>
AuthorDate: Mon Jun 26 14:58:10 2023 +0100

    GH-36299: [R][CI] Remove pkgdown check CI step (#36300)
    
    Fixes #36299
    * Closes: #36299
    
    Authored-by: Nic Crane <th...@gmail.com>
    Signed-off-by: Nic Crane <th...@gmail.com>
---
 .github/workflows/r.yml       |  2 --
 ci/scripts/r_pkgdown_check.sh | 41 -----------------------------------------
 2 files changed, 43 deletions(-)

diff --git a/.github/workflows/r.yml b/.github/workflows/r.yml
index f42c12b449..277080bec5 100644
--- a/.github/workflows/r.yml
+++ b/.github/workflows/r.yml
@@ -80,8 +80,6 @@ jobs:
           restore-keys: |
             ubuntu-${{ matrix.ubuntu }}-r-${{ matrix.r }}-${{ hashFiles('cpp/src/**/*.cc','cpp/src/**/*.h)') }}-
             ubuntu-${{ matrix.ubuntu }}-r-${{ matrix.r }}-
-      - name: Check pkgdown reference sections
-        run: ci/scripts/r_pkgdown_check.sh
       - name: Setup Python
         uses: actions/setup-python@v4
         with:
diff --git a/ci/scripts/r_pkgdown_check.sh b/ci/scripts/r_pkgdown_check.sh
deleted file mode 100755
index 327480a6b3..0000000000
--- a/ci/scripts/r_pkgdown_check.sh
+++ /dev/null
@@ -1,41 +0,0 @@
-#!/usr/bin/env bash
-#
-# 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.
-
-# Use this script to prevent errors in the pkgdown site being rendered due to missing YAML entries
-
-# all .Rd files in the repo
-all_rd_files=`find ./r/man -maxdepth 1 -name "*.Rd" | sed -e 's/.\/r\/man\///g' | sed -e 's/.Rd//g' | sort`
-
-# .Rd files to exclude from search (i.e. are internal)
-exclusions=`grep "\keyword{internal}" -rl ./r/man --include=*.Rd | sed -e 's/.\/r\/man\///g' | sed -e 's/.Rd//g' | sort`
-
-# .Rd files to check against pkgdown.yml
-rd_files=`echo ${exclusions[@]} ${all_rd_files[@]} | tr ' ' '\n' | sort | uniq -u`
-
-# pkgdown sections
-pkgdown_sections=`awk '/^[^ ]/{ f=/reference:/; next } f{ if (sub(/:$/,"")) pkg=$2; else print pkg, $2 }' ./r/_pkgdown.yml | grep -v "title:" | sort`
-
-# get things that appear in man files that don't appear in pkgdown sections
-pkgdown_missing=`echo ${pkgdown_sections[@]} ${pkgdown_sections[@]} ${rd_files[@]} | tr ' ' '\n' | sort | uniq -u`
-
-# if any sections are missing raise an error
-if ([ ${#pkgdown_missing} -ge 1 ]); then
-	echo "Error! $pkgdown_missing missing from ./r/_pkgdown.yml"
-  	exit 1 
-fi