Skip to content

Commit

Permalink
chore: remove touchbar support
Browse files Browse the repository at this point in the history
chore: force install python

chore: try running macos earlier version

chore: remove manual py install

chore: use macOS11 integration tests

chore: use set ubuntu to fixed test version

chore: update publish script

ci: try newer node version
  • Loading branch information
kyranjamie committed Nov 30, 2022
1 parent b30dce8 commit a7ba03e
Show file tree
Hide file tree
Showing 10 changed files with 45 additions and 46 deletions.
4 changes: 3 additions & 1 deletion .github/workflows/audit.yml
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,9 @@ jobs:
- name: Add required Linux dependencies
uses: ./.github/actions/linux-deps

- uses: actions/setup-node@v2
- uses: actions/setup-node@v3
with:
node-version: 14

- name: Cache node_modules
uses: actions/cache@v2
Expand Down
10 changes: 4 additions & 6 deletions .github/workflows/create-version.yml
Original file line number Diff line number Diff line change
Expand Up @@ -28,10 +28,9 @@ jobs:
- name: Add required Linux dependencies
uses: ./.github/actions/linux-deps

- name: Use Node.js
uses: actions/setup-node@v2
- uses: actions/setup-node@v3
with:
node-version: 12
node-version: 14

- name: Cache node_modules
id: cache
Expand Down Expand Up @@ -63,10 +62,9 @@ jobs:
- name: Add required Linux dependencies
uses: ./.github/actions/linux-deps

- name: Set up node
uses: actions/setup-node@v2
- uses: actions/setup-node@v3
with:
node-version: 12
node-version: 14

- name: Cache node_modules
id: cache
Expand Down
26 changes: 15 additions & 11 deletions .github/workflows/debug-build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ on: [pull_request]

jobs:
announce-start:
runs-on: ubuntu-latest
runs-on: ubuntu-20.04

steps:
- uses: lucasmotta/pull-request-sticky-header@1.0.0
Expand All @@ -25,16 +25,16 @@ jobs:

strategy:
matrix:
os: [macos-latest, ubuntu-latest, windows-latest]
os: [macos-11, ubuntu-20.04, windows-latest]
stx_network: [testnet, mainnet]
include:
- os: macos-latest
- os: macos-11
NPM_COMMAND: mac
UPLOAD_ASSETS: release/**/*.dmg
CSC_LINK_SECRET_NAME: CODE_SIGNING_CERTIFICATE_APPLE
CSC_KEY_PASSWORD_SECRET_NAME: CODE_SIGNING_PASSWORD_APPLE

- os: ubuntu-latest
- os: ubuntu-20.04
NPM_COMMAND: linux
UPLOAD_ASSETS: |
release/**/*.deb
Expand All @@ -57,7 +57,7 @@ jobs:

- name: Add required Linux dependencies
uses: ./.github/actions/linux-deps
if: matrix.os == 'ubuntu-latest'
if: matrix.os == 'ubuntu-20.04'

- name: Import GPG key
id: import_gpg_key
Expand All @@ -81,15 +81,19 @@ jobs:
- name: Install code-signing certs - Apple only
uses: apple-actions/import-codesign-certs@v1
if: matrix.os == 'macos-latest'
if: matrix.os == 'macos-11'
with:
p12-file-base64: ${{ secrets.CODE_SIGNING_CERTIFICATE_APPLE }}
p12-password: ${{ secrets.CODE_SIGNING_PASSWORD_APPLE }}

- name: Setup node
uses: actions/setup-node@v2
- uses: actions/setup-node@v3
with:
node-version: 12
node-version: 14

- uses: actions/setup-python@v4
with:
python-version: '2.7.18'
architecture: x64

#
# Windows build breaks when using the package.json script to install `app/` packages
Expand Down Expand Up @@ -135,7 +139,7 @@ jobs:
path: ${{ matrix.UPLOAD_ASSETS }}

asset-signatures:
runs-on: ubuntu-latest
runs-on: ubuntu-20.04
needs:
- debug-build

Expand Down Expand Up @@ -165,7 +169,7 @@ jobs:
path: shasum.txt.asc

announce-completion:
runs-on: ubuntu-latest
runs-on: ubuntu-20.04

if: always()

Expand Down
4 changes: 3 additions & 1 deletion .github/workflows/exact-versions.yml
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,9 @@ jobs:
steps:
- uses: actions/checkout@v2

- uses: actions/setup-node@v2
- uses: actions/setup-node@v3
with:
node-version: 14

- name: Add @actions/core
uses: nick-invision/retry@v2
Expand Down
5 changes: 2 additions & 3 deletions .github/workflows/gh-pages.yml
Original file line number Diff line number Diff line change
Expand Up @@ -17,10 +17,9 @@ jobs:
- name: Add required Linux dependencies
uses: ./.github/actions/linux-deps

- name: Set up node
uses: actions/setup-node@v2
- uses: actions/setup-node@v3
with:
node-version: 12
node-version: 14

- name: Cache node_modules
id: cache
Expand Down
12 changes: 6 additions & 6 deletions .github/workflows/integration-tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ jobs:

strategy:
matrix:
os: [macos-latest, ubuntu-latest]
os: [macos-11, ubuntu-20.04]
stx_network: [testnet, mainnet]

steps:
Expand All @@ -33,13 +33,13 @@ jobs:
path: '**/node_modules'
key: ${{ runner.os }}-modules-${{ hashFiles('**/yarn.lock') }}-${{ hashFiles('**/package.json') }}

- uses: actions/setup-node@v2
- uses: actions/setup-node@v3
with:
node-version: 15
node-version: 14

- name: Add required Linux dependencies
uses: ./.github/actions/linux-deps
if: matrix.os == 'ubuntu-latest'
if: matrix.os == 'ubuntu-20.04'

- name: Install packages
uses: nick-invision/retry@v2
Expand Down Expand Up @@ -78,10 +78,10 @@ jobs:
run: echo "::set-output name=date::$(date +'%Y-%m-%d')"

- run: yarn test:integration
if: matrix.os == 'macos-latest'
if: matrix.os == 'macos-11'

- run: xvfb-run --auto-servernum -- yarn test:integration
if: matrix.os == 'ubuntu-latest'
if: matrix.os == 'ubuntu-20.04'

- uses: actions/upload-artifact@v2
if: always()
Expand Down
22 changes: 10 additions & 12 deletions .github/workflows/publish-version.yml
Original file line number Diff line number Diff line change
Expand Up @@ -14,16 +14,16 @@ jobs:

strategy:
matrix:
os: [macos-latest, ubuntu-latest, windows-latest]
os: [macos-10.11, ubuntu-20.04, windows-latest]
stx_network: [testnet, mainnet]
include:
- os: macos-latest
- os: macos-10.11
UPLOAD_ASSETS: release/**/*.dmg
NPM_COMMAND: mac
CSC_LINK_SECRET_NAME: CODE_SIGNING_CERTIFICATE_APPLE
CSC_KEY_PASSWORD_SECRET_NAME: CODE_SIGNING_PASSWORD_APPLE

- os: ubuntu-latest
- os: ubuntu-20.04
NPM_COMMAND: linux
UPLOAD_ASSETS: |
release/**/*.deb
Expand All @@ -50,7 +50,7 @@ jobs:

- name: Add required Linux dependencies
uses: ./.github/actions/linux-deps
if: matrix.os == 'ubuntu-latest'
if: matrix.os == 'ubuntu-20.04'

- name: Variables
id: vars
Expand All @@ -60,16 +60,14 @@ jobs:
- name: Install code-signing certs - Apple only
uses: apple-actions/import-codesign-certs@v1
if: matrix.os == 'macos-latest'
if: matrix.os == 'macos-10.11'
with:
p12-file-base64: ${{ secrets.CODE_SIGNING_CERTIFICATE_APPLE }}
p12-password: ${{ secrets.CODE_SIGNING_PASSWORD_APPLE }}

- name: Install Node.js, NPM and Yarn
uses: actions/setup-node@v2
- uses: actions/setup-node@v3
with:
node-version: 12

node-version: 14
#
# Windows build breaks when using the package.json script to install `app/` packages
# using separate run to install those packages separately
Expand Down Expand Up @@ -107,7 +105,7 @@ jobs:
path: ${{ matrix.UPLOAD_ASSETS }}

create-release:
runs-on: ubuntu-latest
runs-on: ubuntu-20.04
needs:
- build

Expand Down Expand Up @@ -171,7 +169,7 @@ jobs:
body_path: release-body.txt

upload-release-assets:
runs-on: ubuntu-latest
runs-on: ubuntu-20.04
needs:
- build
- create-release
Expand Down Expand Up @@ -210,7 +208,7 @@ jobs:
asset_content_type: application/octet-stream

announce-release:
runs-on: ubuntu-latest
runs-on: ubuntu-20.04
env:
DISCORD_WEBHOOK: ${{ secrets.DISCORD_UX_NOTIFS_WEBHOOK }}

Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/typecheck.yml
Original file line number Diff line number Diff line change
Expand Up @@ -14,9 +14,9 @@ jobs:
- name: Add required Linux dependencies
uses: ./.github/actions/linux-deps

- uses: actions/setup-node@v2
- uses: actions/setup-node@v3
with:
node-version: '14'
node-version: 14

- name: Install dependencies
uses: nick-invision/retry@v2
Expand Down
3 changes: 0 additions & 3 deletions app/main.dev.ts
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,6 @@ import { getUserDataPath } from './main/get-user-data-path';
import { registerLedgerListeners } from './main/register-ledger-listeners';
import { registerIpcStoreHandlers } from './main/register-store-handlers';
import { registerIpcContextMenuHandlers } from './main/register-context-menus';
import { addMacOsTouchBarMenu } from './main/macos-touchbar-menu';
import { registerThemeModeHandlers } from './main/register-theme-mode-handlers';

// CSP enabled in production mode, don't warn in development
Expand Down Expand Up @@ -156,8 +155,6 @@ const createWindow = async () => {
registerThemeModeHandlers(mainWindow.webContents);

registerIpcContextMenuHandlers(mainWindow);

if (process.platform === 'darwin') addMacOsTouchBarMenu(mainWindow);
};

app.on('web-contents-created', (_event, contents) => {
Expand Down
1 change: 0 additions & 1 deletion app/tests/features/onboarding.feature.ts
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,6 @@ export function createOnboardingFeature(page: Page) {
async findTermsOfServiceTitle() {
// force top of page
await repeatAction(100)(() => page.keyboard.press('Shift+Space'));
await page.screenshot({ path: 'screenshots/terms-page.png' });
const selector = 'text="Terms of Service"';
return page.$(selector);
},
Expand Down

0 comments on commit a7ba03e

Please sign in to comment.