Skip to content

Conversation

bartholomej
Copy link
Owner

What's new and why I made this pull request?

  • Fix live tests #kviff
  • Update all deps
  • Update CI deps

@bartholomej bartholomej requested a review from Copilot July 8, 2025 09:46
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

A concise update to ensure live tests pass and all dependencies are current.

  • Fixed live test expectations to match current CSFD data
  • Bumped devDependencies and CI action versions
  • Updated GitHub Actions workflows to use specific version tags

Reviewed Changes

Copilot reviewed 5 out of 6 changed files in this pull request and generated no comments.

Show a summary per file
File Description
tests/fetchers.test.ts Updated movie title constants and expectations
package.json Bumped versions of linting, testing, and build tools
.github/workflows/test.yml Upgraded codecov action from v4 to v5
.github/workflows/publish.yml Switched checkout/setup-node from master to v4 and updated codecov
.github/workflows/main.yml Upgraded codecov action from v4 to v5
Comments suppressed due to low confidence (1)

tests/fetchers.test.ts:11

  • These tests rely on live CSFD data and may break if the user's ratings change; consider mocking network responses or using recorded fixtures to make them deterministic.
  test('Fetch `912-bart` user and check some movie', async () => {

@bartholomej bartholomej merged commit 24794fc into master Jul 8, 2025
1 check passed
@bartholomej bartholomej deleted the update branch July 8, 2025 09:49
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.

1 participant