Skip to content

Commit 9e32936

Browse files
authored
[CI] build-dev skip registry check and check-links tweak (#5569)
1 parent fc038bf commit 9e32936

File tree

6 files changed

+14
-12
lines changed

6 files changed

+14
-12
lines changed

.github/workflows/build-dev.yml

+1-1
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@ jobs:
3737

3838
- run: npm run log:test-and-fix
3939
env:
40-
CMD_SKIP: i18n
40+
CMD_SKIP: i18n|registry
4141
PIN_SKIP: ${{ inputs.submodule_path_regex }}
4242

4343
- uses: actions/upload-artifact@v4

.github/workflows/check-links.yml

+5
Original file line numberDiff line numberDiff line change
@@ -34,6 +34,11 @@ jobs:
3434
- run: npm install --omit=optional
3535
- run: npm run log:check:links
3636
continue-on-error: true
37+
- name: Any files need updating?
38+
run: |
39+
git restore package.json
40+
echo "If the following fails, then either run 'npm run fix:htmltest-config' locally or '/fix:htmltest-config' in GitHub"
41+
npm run _diff:fail
3742
- uses: actions/upload-artifact@v4
3843
with:
3944
name: build-log-etc

.github/workflows/pr-actions.yml

+3-3
Original file line numberDiff line numberDiff line change
@@ -29,9 +29,9 @@ jobs:
2929
- name: Extract action name
3030
id: extract_action_name
3131
run: |
32-
PR_ACTION=$(echo $COMMENT | grep -oP '/fix:\K\w+')
32+
PR_ACTION=$(echo $COMMENT | grep -oP '/fix:\K[-_0-9a-z]+')
3333
echo "Action is $PR_ACTION"
34-
ACTION_NAMES="all|dict|filenames|format|i18n|markdown|refcache|submodules?|text"
34+
ACTION_NAMES="all|dict|filenames|format|htmltest-config|i18n|markdown|refcache|submodules?|text"
3535
if [[ ! "$PR_ACTION" =~ ^($ACTION_NAMES)$ ]]; then
3636
echo "Invalid action name: $PR_ACTION"
3737
echo "Action name should be one of: $ACTION_NAMES"
@@ -72,7 +72,7 @@ jobs:
7272
all|refcache)
7373
npm install --omit=optional
7474
;&
75-
dict|filenames|format|markdown|submodule*)
75+
dict|filenames|format|htmltest-config|markdown|submodule*)
7676
npm run fix:$PR_ACTION
7777
;;
7878
esac

.warnings-skip-list.txt

+1
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
_filename-error

content/fr/docs/what-is-opentelemetry.md

-4
Original file line numberDiff line numberDiff line change
@@ -3,10 +3,6 @@ title: Qu'est-ce qu'OpenTelemetry ?
33
description:
44
Une brève explication de ce qu'est OpenTelemetry, et de ce qu'il n'est pas.
55
weight: 150
6-
htmltest:
7-
IgnoreDirs:
8-
# TODO drop next lines after https://github.com/open-telemetry/opentelemetry.io/issues/5555 is fixed for these pages:
9-
- ^fr/docs/concepts/glossary/
106
default_lang_commit: 71833a5f8b84110dadf1e98604b87a900724ac33
117
---
128

package.json

+4-4
Original file line numberDiff line numberDiff line change
@@ -55,16 +55,17 @@
5555
"check:text": "npm run _check:text -- ",
5656
"check": "npm run seq -- $(npm run -s _list:check:*)",
5757
"clean": "make clean",
58-
"code-excerpts": "rm -Rf tmp/excerpts/* && npm run seq -- code-excerpts:get code-excerpts:update-docs",
5958
"code-excerpts:get": "cd tools && dart run build_runner build --delete-conflicting-outputs --output ../tmp/excerpts",
6059
"code-excerpts:update-docs": "cd tools && dart run code_excerpt_updater --fragment-dir-path ../tmp/excerpts --src-dir-path examples --yaml --write-in-place content",
60+
"code-excerpts": "rm -Rf tmp/excerpts/* && npm run seq -- code-excerpts:get code-excerpts:update-docs",
6161
"cp:spec": "scripts/content-modules/cp-pages.sh",
6262
"diff:check": "npm run _diff:check || (echo; echo 'WARNING: the files above have not been committed'; echo)",
6363
"diff:fail": "npm run _diff:check || (echo; echo 'ERROR: the files above have changed. Locally rerun `npm run test-and-fix` and commit changes'; echo; exit 1)",
6464
"fix:all": "npm run all -- $(npm -s run _list:fix:*)",
6565
"fix:dict": "find content/en layouts -name \"*.md\" -print0 | xargs -0 scripts/normalize-cspell-front-matter.pl",
6666
"fix:filenames": "npm run _rename-to-kebab-case",
6767
"fix:format": "npm run format",
68+
"fix:htmltest-config": "scripts/htmltest-config.sh",
6869
"fix:i18n:all": "scripts/check-i18n.sh -a -c HEAD",
6970
"fix:i18n:drifted": "scripts/check-i18n.sh -c HEAD",
7071
"fix:i18n:new": "scripts/check-i18n.sh -n -c HEAD",
@@ -87,8 +88,8 @@
8788
"prebuild:preview": "npm run _prebuild",
8889
"prebuild:production": "npm run _prebuild",
8990
"prebuild": "npm run _prebuild",
90-
"precheck:links:internal": "npm run build && npm run update:htmltest-config",
91-
"precheck:links": "npm run build && npm run update:htmltest-config",
91+
"precheck:links:internal": "npm run build && npm run fix:htmltest-config",
92+
"precheck:links": "npm run build && npm run fix:htmltest-config",
9293
"prefix:submodule": "npm run update:submodule",
9394
"prepare": "npm run seq -- get:submodule _prepare:docsy",
9495
"preserve:hugo": "npm run _prebuild",
@@ -100,7 +101,6 @@
100101
"serve": "npm run serve:hugo --",
101102
"test-and-fix": "npm run seq -- fix:submodule $(npm -s run _list:fix:for-test-and-fix) $(npm -s run _list:check:for-test-and-fix)",
102103
"test": "npm run check",
103-
"update:htmltest-config": "scripts/htmltest-config.sh",
104104
"update:pkg:hugo": "npm install --save-dev --save-exact hugo-extended@latest",
105105
"update:pkgs": "npx npm-check-updates -u",
106106
"update:submodule": "set -x && git submodule update --remote ${DEPTH:- --depth 999}"

0 commit comments

Comments
 (0)