Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(UI): Rename "Elements" to "Items" #538

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

rouk1
Copy link
Contributor

@rouk1 rouk1 commented Oct 18, 2024

No description provided.

@rouk1 rouk1 linked an issue Oct 18, 2024 that may be closed by this pull request
Copy link

Coverage Report for ./skore-ui

Status Category Percentage Covered / Total
🔵 Lines 82.65% 1759 / 2128
🔵 Statements 82.65% 1759 / 2128
🔵 Functions 45.45% 45 / 99
🔵 Branches 82.05% 128 / 156
File Coverage
File Stmts % Branch % Funcs % Lines Uncovered Lines
Changed Files
skore-ui/src/views/ProjectView.vue 44.75% 80% 0% 44.75% 1, 24-33, 35-37, 39-45, 47-53, 55-59, 61-65, 67-69, 71-73, 75-81, 84-90, 92-93, 95-121, 123-130, 135-138, 142
Generated in workflow #323 for commit 27a428d by the Vitest Coverage Report Action

@tuscland
Copy link
Member

There are more places that use "Elements". Are you sure you covered them all?
Note: #537 adds a place where I used Elements for consistency with existing code.

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.

UI: Rename "Elements" to "Items"
2 participants