Skip to content

Commit

Permalink
Merge branch 'master' into VT-29958-2
Browse files Browse the repository at this point in the history
  • Loading branch information
velichkin committed Dec 17, 2024
2 parents abcc114 + ac32751 commit b880c34
Show file tree
Hide file tree
Showing 28 changed files with 218 additions and 210 deletions.
15 changes: 9 additions & 6 deletions .circleci/config.yml
Original file line number Diff line number Diff line change
Expand Up @@ -2,9 +2,9 @@ version: 2.1

orbs:
browser-tools: circleci/[email protected]
codecov: codecov/codecov@4.2.0
codecov: codecov/codecov@5.0.0
macos: circleci/[email protected]
node: circleci/node@6.1.0
node: circleci/node@6.3.0

push_and_pr_builds: &push_and_pr_builds
filters:
Expand Down Expand Up @@ -65,7 +65,7 @@ executors:
resource_class: xlarge
jdk-docker-2xlarge:
docker:
- image: cimg/openjdk:21.0.2-node
- image: cimg/openjdk:21.0.5-node
resource_class: 2xlarge
macos-medium:
macos:
Expand Down Expand Up @@ -156,7 +156,8 @@ commands:
- save_cache:
name: '💾 Save node_modules/ Cache'
key: node-modules-cache-{{ arch }}-v3-{{ checksum "package-lock.json" }}
paths: node_modules/
paths:
- node_modules/
teardown_vm:
steps:
- persist_to_workspace:
Expand Down Expand Up @@ -335,7 +336,8 @@ jobs:
name: '⭐⭐⭐ All Unit Tests ⭐⭐⭐'
command: node build-system/pr-check/unit-tests.js
- codecov/upload:
file: test/coverage/lcov-unit.info
disable_search: true
files: test/coverage/lcov-unit.info
flags: unit_tests
- store_test_output
- store_filelist
Expand All @@ -350,7 +352,8 @@ jobs:
name: '⭐⭐⭐ Unminified Tests ⭐⭐⭐'
command: node build-system/pr-check/unminified-tests.js
- codecov/upload:
file: test/coverage/lcov-integration.info
disable_search: true
files: test/coverage/lcov-integration.info
flags: integration_tests
- store_test_output
- teardown_vm
Expand Down
6 changes: 3 additions & 3 deletions .github/workflows/codeql.yml
Original file line number Diff line number Diff line change
Expand Up @@ -34,16 +34,16 @@ jobs:
egress-policy: audit # TODO: change to 'egress-policy: block' after couple of runs

- name: Checkout repository
uses: actions/checkout@d632683dd7b4114ad314bca15554477dd762a938 # v4.2.0
uses: actions/checkout@eef61447b9ff4aafe5dcd4e0bbf5d482be7e7871 # v4.2.1

# Initializes the CodeQL tools for scanning.
- name: Initialize CodeQL
uses: github/codeql-action/init@6db8d6351fd0be61f9ed8ebd12ccd35dcec51fea # v3.26.11
uses: github/codeql-action/init@f779452ac5af1c261dce0346a8f964149f49322b # v3.26.13
with:
config-file: ./.github/codeql/config.yml
languages: ${{ matrix.language }}

- name: Perform CodeQL Analysis
uses: github/codeql-action/analyze@6db8d6351fd0be61f9ed8ebd12ccd35dcec51fea # v3.26.11
uses: github/codeql-action/analyze@f779452ac5af1c261dce0346a8f964149f49322b # v3.26.13
with:
category: '/language:${{matrix.language}}'
4 changes: 2 additions & 2 deletions .github/workflows/cross-platform-builds.yml
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ jobs:
egress-policy: audit # TODO: change to 'egress-policy: block' after couple of runs

- name: Checkout Repo
uses: actions/checkout@d632683dd7b4114ad314bca15554477dd762a938 # v4.2.0
uses: actions/checkout@eef61447b9ff4aafe5dcd4e0bbf5d482be7e7871 # v4.2.1
- name: Set Up Node
uses: actions/setup-node@0a44ba7841725637a19e28fa30b79a866c81b0a6 # v4.0.4
with:
Expand All @@ -48,7 +48,7 @@ jobs:
with:
egress-policy: audit # TODO: change to 'egress-policy: block' after couple of runs

- uses: actions/checkout@d632683dd7b4114ad314bca15554477dd762a938 # v4.2.0
- uses: actions/checkout@eef61447b9ff4aafe5dcd4e0bbf5d482be7e7871 # v4.2.1

- name: Create issue on error
uses: JasonEtco/create-an-issue@1b14a70e4d8dc185e5cc76d3bec9eab20257b2c5 # v2.9.2
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/cut-nightly.yml
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ jobs:
egress-policy: audit # TODO: change to 'egress-policy: block' after couple of runs

- name: Checkout Repo
uses: actions/checkout@d632683dd7b4114ad314bca15554477dd762a938 # v4.2.0
uses: actions/checkout@eef61447b9ff4aafe5dcd4e0bbf5d482be7e7871 # v4.2.1
with:
fetch-depth: 100

Expand Down Expand Up @@ -59,7 +59,7 @@ jobs:
with:
egress-policy: audit # TODO: change to 'egress-policy: block' after couple of runs

- uses: actions/checkout@d632683dd7b4114ad314bca15554477dd762a938 # v4.2.0
- uses: actions/checkout@eef61447b9ff4aafe5dcd4e0bbf5d482be7e7871 # v4.2.1

- name: Create issue on error
uses: JasonEtco/create-an-issue@1b14a70e4d8dc185e5cc76d3bec9eab20257b2c5 # v2.9.2
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/dependency-review.yml
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,6 @@ jobs:
egress-policy: audit # TODO: change to 'egress-policy: block' after couple of runs

- name: 'Checkout Repository'
uses: actions/checkout@d632683dd7b4114ad314bca15554477dd762a938 # v4.2.0
uses: actions/checkout@eef61447b9ff4aafe5dcd4e0bbf5d482be7e7871 # v4.2.1
- name: 'Dependency Review'
uses: actions/dependency-review-action@5a2ce3f5b92ee19cbb1541a4984c76d921601d7c # v4.3.4
4 changes: 2 additions & 2 deletions .github/workflows/release-tagger.yml
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@ jobs:
- uses: actions/setup-node@0a44ba7841725637a19e28fa30b79a866c81b0a6 # v4.0.4
with:
node-version: lts/*
- uses: actions/checkout@d632683dd7b4114ad314bca15554477dd762a938 # v4.2.0
- uses: actions/checkout@eef61447b9ff4aafe5dcd4e0bbf5d482be7e7871 # v4.2.1
- run: bash ./.github/workflows/install_dependencies.sh
- name: Run tagger
run: |
Expand All @@ -60,7 +60,7 @@ jobs:
with:
egress-policy: audit # TODO: change to 'egress-policy: block' after couple of runs

- uses: actions/checkout@d632683dd7b4114ad314bca15554477dd762a938 # v4.2.0
- uses: actions/checkout@eef61447b9ff4aafe5dcd4e0bbf5d482be7e7871 # v4.2.1

- name: Create issue on error
uses: JasonEtco/create-an-issue@1b14a70e4d8dc185e5cc76d3bec9eab20257b2c5 # v2.9.2
Expand Down
6 changes: 3 additions & 3 deletions .github/workflows/scorecard.yml
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@ jobs:
egress-policy: audit # TODO: change to 'egress-policy: block' after couple of runs

- name: 'Checkout code'
uses: actions/checkout@d632683dd7b4114ad314bca15554477dd762a938 # v4.2.0
uses: actions/checkout@eef61447b9ff4aafe5dcd4e0bbf5d482be7e7871 # v4.2.1
with:
persist-credentials: false

Expand All @@ -64,14 +64,14 @@ jobs:
# Upload the results as artifacts (optional). Commenting out will disable uploads of run results in SARIF
# format to the repository Actions tab.
- name: 'Upload artifact'
uses: actions/upload-artifact@50769540e7f4bd5e21e526ee35c689e35e0d6874 # v4.4.0
uses: actions/upload-artifact@b4b15b8c7c6ac21ea08fcf65892d2ee8f75cf882 # v4.4.3
with:
name: SARIF file
path: results.sarif
retention-days: 5

# Upload the results to GitHub's code scanning dashboard.
- name: 'Upload to code-scanning'
uses: github/codeql-action/upload-sarif@6db8d6351fd0be61f9ed8ebd12ccd35dcec51fea # v3.26.11
uses: github/codeql-action/upload-sarif@f779452ac5af1c261dce0346a8f964149f49322b # v3.26.13
with:
sarif_file: results.sarif
4 changes: 2 additions & 2 deletions .github/workflows/status-page.yml
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ jobs:
- uses: actions/setup-node@0a44ba7841725637a19e28fa30b79a866c81b0a6 # v4.0.4
with:
node-version: lts/*
- uses: actions/checkout@d632683dd7b4114ad314bca15554477dd762a938 # v4.2.0
- uses: actions/checkout@eef61447b9ff4aafe5dcd4e0bbf5d482be7e7871 # v4.2.1
- name: Add progress comment to cherry-pick issue for Stable and LTS
if: github.event_name == 'issues' && github.event.action == 'opened'
run: |
Expand Down Expand Up @@ -56,7 +56,7 @@ jobs:
with:
egress-policy: audit # TODO: change to 'egress-policy: block' after couple of runs

- uses: actions/checkout@d632683dd7b4114ad314bca15554477dd762a938 # v4.2.0
- uses: actions/checkout@eef61447b9ff4aafe5dcd4e0bbf5d482be7e7871 # v4.2.1

- name: Create issue on error
uses: JasonEtco/create-an-issue@1b14a70e4d8dc185e5cc76d3bec9eab20257b2c5 # v2.9.2
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/update-session-issues.yml
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ jobs:
egress-policy: audit # TODO: change to 'egress-policy: block' after couple of runs

- name: Checkout Repo
uses: actions/checkout@d632683dd7b4114ad314bca15554477dd762a938 # v4.2.0
uses: actions/checkout@eef61447b9ff4aafe5dcd4e0bbf5d482be7e7871 # v4.2.1
- name: Update Session Issues
run: |
node ./build-system/common/update-session-issues
Expand Down
9 changes: 8 additions & 1 deletion ads/vendors/navegg.js
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,11 @@ import {doubleclick} from '#ads/google/doubleclick';
* @param {!Object} data
*/
export function navegg(global, data) {
validateData(data, ['acc']);
const requiredParams = ['acc', 'wst', 'wct', 'wla'];
const optionalParams = [];

validateData(data, requiredParams, optionalParams);

const {acc} = data;
let seg,
nvg = function () {};
Expand All @@ -17,6 +21,9 @@ export function navegg(global, data) {
loadScript(global, 'https://tag.navdmp.com/amp.1.0.0.min.js', () => {
nvg = global[`nvg${acc}`] = new global['AMPNavegg']({
acc,
wst: data.wst ? '1' : '0',
wct: data.wct ? '1' : '0',
wla: data.wla ? '1' : '0',
});
nvg.getProfile((nvgTargeting) => {
for (seg in nvgTargeting) {
Expand Down
2 changes: 1 addition & 1 deletion ads/vendors/navegg.md
Original file line number Diff line number Diff line change
Expand Up @@ -18,4 +18,4 @@ To get Navegg integration working you only need to specify the `rtc-config` para

The Navegg adapter only supports DoubleClick for now. For the most up-to-date list of DoubleClick supported parameters and usage, refer to the [DoubleClick reference guide](https://github.com/ampproject/amphtml/blob/main/ads/google/doubleclick.md).

For any help, please contact [Navegg](https://www.navegg.com/en/institutional/#contact).
For any help, please contact [Navegg](https://www.navegg.com/en/about-us/contacts).
14 changes: 7 additions & 7 deletions build-system/tasks/e2e/package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion build-system/tasks/e2e/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@
"@types/selenium-webdriver": "4.1.26",
"babel-regenerator-runtime": "6.5.0",
"chromedriver": "119.0.1",
"geckodriver": "4.5.0",
"geckodriver": "4.5.1",
"selenium-webdriver": "4.25.0"
}
}
8 changes: 4 additions & 4 deletions build-system/tasks/storybook/env/amp/package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion build-system/tasks/storybook/env/amp/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,6 @@
"description": "packages specific to the amp storybook env",
"devDependencies": {
"@ampproject/storybook-addon": "1.1.10",
"preact": "10.24.2"
"preact": "10.24.3"
}
}
14 changes: 7 additions & 7 deletions build-system/tasks/storybook/env/preact/package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion build-system/tasks/storybook/env/preact/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,6 @@
"version": "0.1.0",
"description": "packages specific to the preact storybook env",
"devDependencies": {
"preact": "10.24.2"
"preact": "10.24.3"
}
}
Loading

0 comments on commit b880c34

Please sign in to comment.