Skip to content

Conversation

@NiloCK
Copy link
Collaborator

@NiloCK NiloCK commented Mar 26, 2025

  • default drawer to open
  • comments
  • remove unused login check
  • remove prior timer widget
  • improve 'hiding' of timer behind navbar
  • fix inlined css comment...
  • [common-ui] add cypress component testing infra
  • refactor: extract StudySessionTimer from Study
  • cypress updates to enable vuetify styling
  • add test for StudySessionTimer
  • lockfile
  • [common-ui] add ci

Comment on lines +22 to +81
runs-on: ubuntu-latest

steps:
- name: Checkout repository
uses: actions/checkout@v4

- name: Set up Node.js
uses: actions/setup-node@v4
with:
node-version: 18
cache: 'yarn'

- name: Install dependencies
run: yarn install --immutable

# Build dependencies first
- name: Build common package
run: |
cd packages/common
yarn build

- name: Build db package
run: |
cd packages/db
yarn build

# Lint common-ui
- name: Lint common-ui
run: |
cd packages/common-ui
yarn lint:check

# Build common-ui
- name: Build common-ui
run: |
cd packages/common-ui
yarn build

# Run unit tests
- name: Run unit tests
run: |
cd packages/common-ui
yarn test:unit

# Run component tests
- name: Run Cypress component tests
run: |
cd packages/common-ui
yarn cypress:run --component

# Archive test artifacts if tests fail
- name: Archive Cypress screenshots and videos
uses: actions/upload-artifact@v4
if: failure()
with:
name: cypress-artifacts
path: |
packages/common-ui/cypress/screenshots
packages/common-ui/cypress/videos
retention-days: 7

Check warning

Code scanning / CodeQL

Workflow does not contain permissions Medium

Actions job or workflow does not limit the permissions of the GITHUB_TOKEN. Consider setting an explicit permissions block, using the following as a minimal starting point: {contents: read}
@NiloCK NiloCK merged commit 36ac599 into master Mar 26, 2025
5 checks passed
@NiloCK NiloCK deleted the ui-tweaks branch March 26, 2025 18:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants