Skip to content

Commit

Permalink
Remove support for deprecated login
Browse files Browse the repository at this point in the history
  • Loading branch information
charlespwd committed Jan 5, 2024
1 parent 957eccd commit b06c2b5
Show file tree
Hide file tree
Showing 3 changed files with 0 additions and 35 deletions.
20 changes: 0 additions & 20 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -11,32 +11,14 @@ jobs:
include:
- job_name: 'staff store, custom app login'
store: shimmering-islands.myshopify.com
app_id: ''
app_password: ''
access_token: SHOP_ACCESS_TOKEN
password: ''

- job_name: 'staff store, private app login'
store: shimmering-islands.myshopify.com
app_id: SHOP_APP_ID
app_password: SHOP_APP_PASSWORD
access_token: ''
password: ''

- job_name: 'dev store, custom app login'
store: cpclermont-dev-store.myshopify.com
app_id: ''
app_password: DEV_APP_PASSWORD
access_token: DEV_ACCESS_TOKEN
password: DEV_PASSWORD

- job_name: 'dev store, private app login'
store: cpclermont-dev-store.myshopify.com
app_id: DEV_APP_ID
app_password: DEV_APP_PASSWORD
access_token: ''
password: DEV_PASSWORD

name: ${{ matrix.job_name }}

steps:
Expand All @@ -50,8 +32,6 @@ jobs:
uses: ./
id: lighthouse-ci-action
with:
app_id: ${{ secrets[matrix.app_id] }}
app_password: ${{ secrets[matrix.app_password] }}
access_token: ${{ secrets[matrix.access_token] }}
store: ${{ matrix.store }}
password: ${{ secrets[matrix.password] }}
Expand Down
7 changes: 0 additions & 7 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -66,10 +66,3 @@ For the GitHub Status Checks on PR. One of the two arguments is required:
* `lhci_github_token` - (optional) GitHub personal access token

For more details on the implications of choosing one over the other, refer to the [Lighthouse CI Getting Started Page](https://github.com/GoogleChrome/lighthouse-ci/blob/main/docs/getting-started.md#github-status-checks)

### Deprecated authentication configuration

The following were used to authenticate with private apps.

* `app_id` - (deprecated) Shopify store private app ID.
* `app_password` - (deprecated) Shopify store private app password.
8 changes: 0 additions & 8 deletions action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -43,14 +43,6 @@ inputs:
description: 'Minimum accessibility score'
required: false
default: 0.9
app_id:
description: '(deprecated) Shopify private app ID'
depreciationMessage: Shopify private apps are deprecated. Use a Custom App `access_token` instead.
required: false
app_password:
description: '(deprecated) Shopify private app password'
depreciationMessage: Shopify private apps are deprecated. Use a Custom App `access_token` instead.
required: false
runs:
using: 'docker'
image: 'Dockerfile'

0 comments on commit b06c2b5

Please sign in to comment.