Skip to content

Don't overwrite config.theme.components entirely#4116

Merged
marlo-longley merged 1 commit intomainfrom
mirador-json-state-themes
Feb 27, 2025
Merged

Don't overwrite config.theme.components entirely#4116
marlo-longley merged 1 commit intomainfrom
mirador-json-state-themes

Conversation

@cbeer
Copy link
Collaborator

@cbeer cbeer commented Feb 27, 2025

... some of the function definitions can't be expressed in a JSON export.

@cbeer cbeer force-pushed the mirador-json-state-themes branch 5 times, most recently from 13b392e to ec925d8 Compare February 27, 2025 19:40
@cbeer cbeer force-pushed the mirador-json-state-themes branch from ec925d8 to a9af928 Compare February 27, 2025 19:42
@codecov
Copy link

codecov bot commented Feb 27, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 95.09%. Comparing base (fd75b6c) to head (a9af928).
Report is 4 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #4116   +/-   ##
=======================================
  Coverage   95.08%   95.09%           
=======================================
  Files         317      317           
  Lines       16006    16012    +6     
  Branches     2508     2512    +4     
=======================================
+ Hits        15220    15226    +6     
  Misses        782      782           
  Partials        4        4           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@cbeer cbeer marked this pull request as ready for review February 27, 2025 19:51
@marlo-longley marlo-longley merged commit 9992ca4 into main Feb 27, 2025
9 checks passed
@marlo-longley marlo-longley deleted the mirador-json-state-themes branch February 27, 2025 19:56
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