[Chore] Merge Argilla v2.8.0#73
Merged
Merged
Conversation
# Description <!-- Please include a summary of the changes and the related issue. Please also include relevant motivation and context. List any dependencies that are required for this change. --> Closes #<issue_number> **Type of change** <!-- Please delete options that are not relevant. Remember to title the PR according to the type of change --> - Bug fix (non-breaking change which fixes an issue) - New feature (non-breaking change which adds functionality) - Breaking change (fix or feature that would cause existing functionality to not work as expected) - Refactor (change restructuring the codebase without changing functionality) - Improvement (change adding some improvement to an existing functionality) - Documentation update **How Has This Been Tested** <!-- Please add some reference about how your feature has been tested. --> **Checklist** <!-- Please go over the list and make sure you've taken everything into account --> - I added relevant documentation - I followed the style guidelines of this project - I did a self-review of my code - I made corresponding changes to the documentation - I confirm My changes generate no new warnings - I have added tests that prove my fix is effective or that my feature works - I have added relevant notes to the CHANGELOG.md file (See https://keepachangelog.com/)
…ating `dataset.last_activity_at` column (#5656) # Description <!-- Please include a summary of the changes and the related issue. Please also include relevant motivation and context. List any dependencies that are required for this change. --> Closes #<issue_number> **Type of change** <!-- Please delete options that are not relevant. Remember to title the PR according to the type of change --> - Bug fix (non-breaking change which fixes an issue) **How Has This Been Tested** <!-- Please add some reference about how your feature has been tested. --> **Checklist** <!-- Please go over the list and make sure you've taken everything into account --> - I added relevant documentation - I followed the style guidelines of this project - I did a self-review of my code - I made corresponding changes to the documentation - I confirm My changes generate no new warnings - I have added tests that prove my fix is effective or that my feature works - I have added relevant notes to the CHANGELOG.md file (See https://keepachangelog.com/) --------- Co-authored-by: José Francisco Calvo <josefranciscocalvo@gmail.com>
Corrected "labelled" to "labeled" in [README.md]. This pull request addresses a minor typo found in repository. The typo has been corrected to improve clarity and maintain the quality of the documentation. This change is purely cosmetic and does not affect functionality. # Description <!-- Please include a summary of the changes and the related issue. Please also include relevant motivation and context. List any dependencies that are required for this change. --> Closes #<issue_number> **Type of change** <!-- Please delete options that are not relevant. Remember to title the PR according to the type of change --> - Bug fix (non-breaking change which fixes an issue) - New feature (non-breaking change which adds functionality) - Breaking change (fix or feature that would cause existing functionality to not work as expected) - Refactor (change restructuring the codebase without changing functionality) - Improvement (change adding some improvement to an existing functionality) - Documentation update **How Has This Been Tested** <!-- Please add some reference about how your feature has been tested. --> **Checklist** <!-- Please go over the list and make sure you've taken everything into account --> - I added relevant documentation - I followed the style guidelines of this project - I did a self-review of my code - I made corresponding changes to the documentation - I confirm My changes generate no new warnings - I have added tests that prove my fix is effective or that my feature works - I have added relevant notes to the CHANGELOG.md file (See https://keepachangelog.com/) Co-authored-by: Sara Han <127759186+sdiazlor@users.noreply.github.com>
# Description <!-- Please include a summary of the changes and the related issue. Please also include relevant motivation and context. List any dependencies that are required for this change. --> Closes #<issue_number> **Type of change** <!-- Please delete options that are not relevant. Remember to title the PR according to the type of change --> - Bug fix (non-breaking change which fixes an issue) - New feature (non-breaking change which adds functionality) - Breaking change (fix or feature that would cause existing functionality to not work as expected) - Refactor (change restructuring the codebase without changing functionality) - Improvement (change adding some improvement to an existing functionality) - Documentation update **How Has This Been Tested** <!-- Please add some reference about how your feature has been tested. --> **Checklist** <!-- Please go over the list and make sure you've taken everything into account --> - I added relevant documentation - I followed the style guidelines of this project - I did a self-review of my code - I made corresponding changes to the documentation - I confirm My changes generate no new warnings - I have added tests that prove my fix is effective or that my feature works - I have added relevant notes to the CHANGELOG.md file (See https://keepachangelog.com/) --------- Co-authored-by: Sara Han <127759186+sdiazlor@users.noreply.github.com>
# Description <!-- Please include a summary of the changes and the related issue. Please also include relevant motivation and context. List any dependencies that are required for this change. --> - [x] Use the newer version of python - [x] Use the latest version of core dependencies. (without code changes) - [x] Upgrade docker images - [x] Remove unnecessary dependencies. **Type of change** <!-- Please delete options that are not relevant. Remember to title the PR according to the type of change --> - Refactor (change restructuring the codebase without changing functionality) - Improvement (change adding some improvement to an existing functionality) **How Has This Been Tested** <!-- Please add some reference about how your feature has been tested. --> **Checklist** <!-- Please go over the list and make sure you've taken everything into account --> - I added relevant documentation - I followed the style guidelines of this project - I did a self-review of my code - I made corresponding changes to the documentation - I confirm My changes generate no new warnings - I have added tests that prove my fix is effective or that my feature works - I have added relevant notes to the CHANGELOG.md file (See https://keepachangelog.com/)
# Description Updated some of the missing German translations + a few aria updates **Type of change** - Improvement (change adding some improvement to an existing functionality) - **How Has This Been Tested** Local Build/Firefox **Checklist** <!-- Please go over the list and make sure you've taken everything into account --> - I added relevant documentation - I followed the style guidelines of this project - I did a self-review of my code - I made corresponding changes to the documentation - I confirm My changes generate no new warnings - I have added tests that prove my fix is effective or that my feature works - I have added relevant notes to the CHANGELOG.md file (See https://keepachangelog.com/) --------- Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
…#5661) # Add a high-contrast theme & improvements for the forced-colors mode - Invisible borders for key items to make them visible in forced colors mode via `border: 1px solid var(--bg-opacity-1)` - Set a button outline of 5px in forced colors to make selections visible via `@media (forced-colors: active) ` - add a new high contrast theme to the theme.scss & to the settings menu - all new colors in the new high-contrast theme should create enough contrast for WCAG AAA - replace some hard coded colors with the colors from the current theme - add hexToCssHsl to the Color.ts and resolveCssVariable to the RecordStatus.ts to use theme vars there too The high contrast theme is currently basically the light theme with stronger colors so that it complies with WCAG AAA  In the forced color mode you can now see certain features that where not visible before:  Forced contrast mode depends on the implementation of the OS & browser therefore the selected labels I cannot just highlight with a background color and I did it with a thick border. **Type of change** - New feature (non-breaking change which adds functionality) **How Has This Been Tested** Local Build - Firefox Nightly + Chromium **Checklist** <!-- Please go over the list and make sure you've taken everything into account --> - I added relevant documentation - I followed the style guidelines of this project - I did a self-review of my code - I made corresponding changes to the documentation - I confirm My changes generate no new warnings - I have added tests that prove my fix is effective or that my feature works - I have added relevant notes to the CHANGELOG.md file (See https://keepachangelog.com/) --------- Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
…lembic (#5672) # Description <!-- Please include a summary of the changes and the related issue. Please also include relevant motivation and context. List any dependencies that are required for this change. --> Alembic will use psycopy2 driver by default. We still need to install it to avoid startup errors. **Type of change** <!-- Please delete options that are not relevant. Remember to title the PR according to the type of change --> - Bug fix (non-breaking change which fixes an issue) **How Has This Been Tested** <!-- Please add some reference about how your feature has been tested. --> **Checklist** <!-- Please go over the list and make sure you've taken everything into account --> - I added relevant documentation - I followed the style guidelines of this project - I did a self-review of my code - I made corresponding changes to the documentation - I confirm My changes generate no new warnings - I have added tests that prove my fix is effective or that my feature works - I have added relevant notes to the CHANGELOG.md file (See https://keepachangelog.com/)
# Description Corrected the spelling mistakes and increased the grammatical continuity. **Type of change** These changes improve the quality of the documentations. They do not affect the system in any way. --------- Co-authored-by: David Berenstein <david.m.berenstein@gmail.com>
# Description <!-- Please include a summary of the changes and the related issue. Please also include relevant motivation and context. List any dependencies that are required for this change. --> Closes argilla-io/argilla#5627 **Type of change** <!-- Please delete options that are not relevant. Remember to title the PR according to the type of change --> - Improvement (change adding some improvement to an existing functionality) **How Has This Been Tested** <!-- Please add some reference about how your feature has been tested. --> **Checklist** <!-- Please go over the list and make sure you've taken everything into account --> - I added relevant documentation - I followed the style guidelines of this project - I did a self-review of my code - I made corresponding changes to the documentation - I confirm My changes generate no new warnings - I have added tests that prove my fix is effective or that my feature works - I have added relevant notes to the CHANGELOG.md file (See https://keepachangelog.com/)
# Description
<!-- Please include a summary of the changes and the related issue.
Please also include relevant motivation and context. List any
dependencies that are required for this change. -->
This PR allows users to change all the settings before creating the
dataset, and metadata and vector, when the dataset is created.
I included the `remove` method instead of a new
`settings.add(override=True)` since this change requires a lot of
internal refactor with the current design.
An example of how to use could be to change inferred settings from hub
before create the final dataset
```python
settings = rg.Settings.from_hub("google/frames-benchmark")
settings.fields.remove("answer")
for field in settings.fields:
if field.name.startswith("wiki"):
settings.fields.remove(field)
settings.metadata.add(rg.TermsMetadataProperty(field.name))
settings.questions.add(rg.TextQuestion(name="answer", title="Answer"))
dataset = rg.Dataset.from_hub("google/frames-benchmark", settings=settings)
```
Or adding new metadata or vector settings when the dataset is created:
```python
dataset = client.dataset("my-dataset")
dataset.metadata.add([rg.TermsMetadata(name="split")])
dataset.update() # this line sends the change to the argilla server
```
## Updated
You can remove a property by name:
```python
settings.fields.remove("text")
```
or by property instance
```python
for field in settings.fields:
if field.name.startwith("wiki"):
settings.fields.remove(field)
````
And you can override existing properties to change the property type,
using the new `settings.add` method:
```python
settings = Settings.from_hub(...)
# change some settings definitions before create the dataset
settings.add(rg.TermsMetadata("wikipedia_1")) # this will remove the existing wikipedia_1 property and will create a new terms metadata one
```
**Type of change**
<!-- Please delete options that are not relevant. Remember to title the
PR according to the type of change -->
- Refactor (change restructuring the codebase without changing
functionality)
- Improvement (change adding some improvement to an existing
functionality)
**How Has This Been Tested**
<!-- Please add some reference about how your feature has been tested.
-->
**Checklist**
<!-- Please go over the list and make sure you've taken everything into
account -->
- I added relevant documentation
- I followed the style guidelines of this project
- I did a self-review of my code
- I made corresponding changes to the documentation
- I confirm My changes generate no new warnings
- I have added tests that prove my fix is effective or that my feature
works
- I have added relevant notes to the CHANGELOG.md file (See
https://keepachangelog.com/)
---------
Co-authored-by: burtenshaw <ben@argilla.io>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Check the role and content properties to validate the chat field --------- Co-authored-by: Francisco Aranda <francis@argilla.io>
# Description <!-- Please include a summary of the changes and the related issue. Please also include relevant motivation and context. List any dependencies that are required for this change. --> Closes argilla-io/argilla#5633 **Type of change** <!-- Please delete options that are not relevant. Remember to title the PR according to the type of change --> - Bug fix (non-breaking change which fixes an issue) - New feature (non-breaking change which adds functionality) **How Has This Been Tested** <!-- Please add some reference about how your feature has been tested. --> **Checklist** <!-- Please go over the list and make sure you've taken everything into account --> - I added relevant documentation - I followed the style guidelines of this project - I did a self-review of my code - I made corresponding changes to the documentation - I confirm My changes generate no new warnings - I have added tests that prove my fix is effective or that my feature works - I have added relevant notes to the CHANGELOG.md file (See https://keepachangelog.com/) --------- Co-authored-by: José Francisco Calvo <jose@argilla.io> Co-authored-by: José Francisco Calvo <josefranciscocalvo@gmail.com> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
…xternal URL (#5675) # Description <!-- Please include a summary of the changes and the related issue. Please also include relevant motivation and context. List any dependencies that are required for this change. --> This PR fixes the `router.go` method and avoids calling `router.push` when the external flag is True **Type of change** <!-- Please delete options that are not relevant. Remember to title the PR according to the type of change --> - Bug fix (non-breaking change which fixes an issue) **How Has This Been Tested** <!-- Please add some reference about how your feature has been tested. --> **Checklist** <!-- Please go over the list and make sure you've taken everything into account --> - I added relevant documentation - I followed the style guidelines of this project - I did a self-review of my code - I made corresponding changes to the documentation - I confirm My changes generate no new warnings - I have added tests that prove my fix is effective or that my feature works - I have added relevant notes to the CHANGELOG.md file (See https://keepachangelog.com/)
This PR fixes the visualisation of highlighted text in text and chat fields adding an id
- [x] Rating values validate min / max - [x] Question name format and renaming - [x] Minor style adjustments - [x] Show required / not required in the form preview - [x] Redirect from /new/repoId to /repoId - [x] Redirect to error page when repoId is invalid --------- Co-authored-by: Francisco Aranda <francis@argilla.io> Co-authored-by: Paco Aranda <frascuchon@gmail.com> Co-authored-by: José Francisco Calvo <jose@argilla.io> Co-authored-by: José Francisco Calvo <josefranciscocalvo@gmail.com> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
# Description <!-- Please include a summary of the changes and the related issue. Please also include relevant motivation and context. List any dependencies that are required for this change. --> **Type of change** <!-- Please delete options that are not relevant. Remember to title the PR according to the type of change --> - Documentation update **How Has This Been Tested** <!-- Please add some reference about how your feature has been tested. --> **Checklist** <!-- Please go over the list and make sure you've taken everything into account --> - I added relevant documentation - I followed the style guidelines of this project - I did a self-review of my code - I made corresponding changes to the documentation - I confirm My changes generate no new warnings - I have added tests that prove my fix is effective or that my feature works - I have added relevant notes to the CHANGELOG.md file (See https://keepachangelog.com/)
# Description <!-- Please include a summary of the changes and the related issue. Please also include relevant motivation and context. List any dependencies that are required for this change. --> Closes argilla-io/argilla#5664 **Type of change** <!-- Please delete options that are not relevant. Remember to title the PR according to the type of change --> - Improvement (change adding some improvement to an existing functionality) **How Has This Been Tested** <!-- Please add some reference about how your feature has been tested. --> **Checklist** <!-- Please go over the list and make sure you've taken everything into account --> - I added relevant documentation - I followed the style guidelines of this project - I did a self-review of my code - I made corresponding changes to the documentation - I confirm My changes generate no new warnings - I have added tests that prove my fix is effective or that my feature works - I have added relevant notes to the CHANGELOG.md file (See https://keepachangelog.com/) --------- Co-authored-by: José Francisco Calvo <jose@argilla.io> Co-authored-by: José Francisco Calvo <josefranciscocalvo@gmail.com> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
fix #5659 --------- Co-authored-by: Damián Pumar <damianpumar@gmail.com>
# Description This is the feature branch for Webhooks implementation. Closes #1836 **Type of change** - New feature (non-breaking change which adds functionality) **How Has This Been Tested** - [ ] Manually tested locally and in HF spaces. **Checklist** - I added relevant documentation - I followed the style guidelines of this project - I did a self-review of my code - I made corresponding changes to the documentation - I confirm My changes generate no new warnings - I have added tests that prove my fix is effective or that my feature works - I have added relevant notes to the CHANGELOG.md file (See https://keepachangelog.com/) --------- Co-authored-by: Paco Aranda <frascuchon@gmail.com> Co-authored-by: Damián Pumar <damianpumar@gmail.com> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com> Co-authored-by: Francisco Aranda <francis@argilla.io> Co-authored-by: David Berenstein <david.m.berenstein@gmail.com> Co-authored-by: Sara Han <127759186+sdiazlor@users.noreply.github.com>
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
…hon -m argilla.cli'
There was a problem hiding this comment.
Pull Request Overview
This PR merges Argilla v2.8.0 into Extralit, updating dependencies, CI workflows, test coverage, and frontend features while preserving existing Extralit customizations.
- Upgraded Elasticsearch, Codecov actions, and Python/JS dependency tooling across workflows
- Enhanced frontend: added Japanese locale, refactored OAuth login UI, improved Jest mocks
- Improved test scripts in package.json and CI to collect coverage
Reviewed Changes
Copilot reviewed 86 out of 86 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| argilla-frontend/plugins/axios/axios-global-handler.ts | Updated copyright header |
| argilla-frontend/package.json | Enhanced test scripts with coverage and worker limits |
| argilla-frontend/nuxt.config.ts | Added Japanese locale entry |
| argilla-frontend/jest.config.js | Updated module name mapping and transform settings |
| argilla-frontend/components/features/login/OAuthLogin.vue | Refactored provider list to use semantic <ul>/<li> |
| argilla-frontend/components/features/login/components/*.vue | Added structured OAuth buttons and SVG logo component |
| argilla-frontend/components/base/base-render-table/renderTable.test.ts | Introduced Tabulator mock, skipped unstable tests |
| argilla-frontend/mocks/tabulator-tables.js | Added full mock implementation for tabulator-tables |
| argilla-frontend/CHANGELOG.md | Updated Extralit and Argilla version sections |
| .github/workflows/*.yml | Bumped versions, added UV setup, improved caching |
| .devcontainer/docker-compose/docker-compose.yml | Upgraded Elasticsearch image to 8.17.0 |
| .github/workflows/README.md | Added workflows documentation |
Comments suppressed due to low confidence (5)
argilla-frontend/nuxt.config.ts:130
- Ensure that the referenced locale file
ja.jsactually exists under yourlocalesdirectory; otherwise the new Japanese entry will cause a missing-file error at runtime.
{ code: "ja", name: "日本語", file: "ja.js", }
argilla-frontend/components/base/base-render-table/renderTable.test.ts:107
- Rather than skipping the
clearTablevalidation withexpect(true), consider providing a proper mock forclearDataor testing side effects (e.g.,tableJSONreset) to maintain meaningful coverage.
expect(true).toBe(true);
argilla-frontend/CHANGELOG.md:17
- The
Extralit 0.5.0section is empty—please add a brief summary of Extralit-specific changes introduced in this release or remove the header if no changes apply.
## [Extralit] [0.5.0](https://github.com/extralit/extralit/compare/v0.4.1...v0.5.0)
argilla-frontend/components/features/login/components/KeycloakLogo.vue:3
- [nitpick] This inline SVG is quite large and may affect readability; consider extracting it into a separate asset file or a dedicated SVG component to keep the template concise.
<svg
.github/workflows/argilla-frontend.deploy-environment.yml:9
- The default image name was updated—please verify and update any user-facing documentation or README references to match the new
extralit/argilla-quickstart-for-devimage.
default: extralit/argilla-quickstart-for-dev
| background: var(--color-black); | ||
| color: var(--color-white); | ||
| width: 100%; | ||
| min-height: $base-space * 6; |
There was a problem hiding this comment.
[nitpick] The min-height style is duplicated with the Keycloak button; consider extracting shared button styles into a common mixin or variable to avoid duplication.
jonnywireless
pushed a commit
that referenced
this pull request
Jul 31, 2025
This pull request introduces several updates across the codebase, including dependency upgrades, CI/CD workflow improvements, and new mock implementations for testing. The key changes focus on enhancing compatibility, improving test coverage, and streamlining workflows. ### Dependency and Configuration Updates: * Upgraded Elasticsearch image from `8.15.0` to `8.17.0` in `.devcontainer/docker-compose/docker-compose.yml` and `.github/workflows/argilla-server.yml` to ensure compatibility with the latest features and fixes. [[1]](diffhunk://#diff-989e458ca6018c442472aa47d71d0671edf1eec0a93d4f569ac6435e564022ecL21-R21) [[2]](diffhunk://#diff-d69af7b78e5cc7d03e327582d01e6cc2780c0cc8c91e45f98544349b2edf0e01L38-R38) * Updated the default Docker image in `.github/workflows/argilla-frontend.deploy-environment.yml` to align with the new project namespace (`extralit/argilla-quickstart-for-dev`). ### CI/CD Workflow Enhancements: * Added detailed documentation for GitHub Actions workflows in `.github/workflows/README.md`, including caching strategies, environment variables, and troubleshooting steps. * Introduced test coverage upload to Codecov in multiple workflows (`argilla-frontend.yml`, `argilla-server.yml`, and `argilla.yml`) to improve visibility into test quality. [[1]](diffhunk://#diff-05a21f9ed6d46c5b14e3c55a3da7fcb48c8b8a0616ca82d28b57e0176726fe19L68-R81) [[2]](diffhunk://#diff-d69af7b78e5cc7d03e327582d01e6cc2780c0cc8c91e45f98544349b2edf0e01L114-R126) [[3]](diffhunk://#diff-a91ef6c8ef0ab6b4fdf21f08b20730c3420fa2f78126ea0403ee7fa73af7d0c8R51-R143) * Enhanced Python dependency management by integrating `uv` for caching and special handling for Python 3.13 in workflows like `argilla.yml` and `argilla-server.yml`. [[1]](diffhunk://#diff-d69af7b78e5cc7d03e327582d01e6cc2780c0cc8c91e45f98544349b2edf0e01R91-R97) [[2]](diffhunk://#diff-a91ef6c8ef0ab6b4fdf21f08b20730c3420fa2f78126ea0403ee7fa73af7d0c8R51-R143) ### Testing Improvements: * Added a mock implementation for `tabulator-tables` in `argilla-frontend/__mocks__/tabulator-tables.js` to facilitate unit testing without relying on external libraries. * Updated tests in `argilla-frontend/components/base/base-render-table/renderTable.test.ts` to include mock functionality for `Tabulator` and adjusted test cases for better reliability. [[1]](diffhunk://#diff-beda1c0e26dd21e30b4d0523c659dc99e728035c5ba1dce2bf9b934458a3c959L5-R8) [[2]](diffhunk://#diff-beda1c0e26dd21e30b4d0523c659dc99e728035c5ba1dce2bf9b934458a3c959R90-R96) ### Miscellaneous Changes: * Added a new changelog entry in `argilla-frontend/CHANGELOG.md` for version `2.8.0`, highlighting the addition of Japanese language support. * Replaced `conda-incubator/setup-miniconda@v2` with `@v3` in `.github/workflows/argilla-v1.yml` for improved compatibility and updated the Miniforge variant. These updates collectively improve the project's maintainability, testing infrastructure, and compatibility with newer technologies.
JonnyTran
added a commit
that referenced
this pull request
Jul 31, 2025
This pull request introduces several updates across the codebase, including dependency upgrades, CI/CD workflow improvements, and new mock implementations for testing. The key changes focus on enhancing compatibility, improving test coverage, and streamlining workflows. ### Dependency and Configuration Updates: * Upgraded Elasticsearch image from `8.15.0` to `8.17.0` in `.devcontainer/docker-compose/docker-compose.yml` and `.github/workflows/argilla-server.yml` to ensure compatibility with the latest features and fixes. [[1]](diffhunk://#diff-989e458ca6018c442472aa47d71d0671edf1eec0a93d4f569ac6435e564022ecL21-R21) [[2]](diffhunk://#diff-d69af7b78e5cc7d03e327582d01e6cc2780c0cc8c91e45f98544349b2edf0e01L38-R38) * Updated the default Docker image in `.github/workflows/argilla-frontend.deploy-environment.yml` to align with the new project namespace (`extralit/argilla-quickstart-for-dev`). ### CI/CD Workflow Enhancements: * Added detailed documentation for GitHub Actions workflows in `.github/workflows/README.md`, including caching strategies, environment variables, and troubleshooting steps. * Introduced test coverage upload to Codecov in multiple workflows (`argilla-frontend.yml`, `argilla-server.yml`, and `argilla.yml`) to improve visibility into test quality. [[1]](diffhunk://#diff-05a21f9ed6d46c5b14e3c55a3da7fcb48c8b8a0616ca82d28b57e0176726fe19L68-R81) [[2]](diffhunk://#diff-d69af7b78e5cc7d03e327582d01e6cc2780c0cc8c91e45f98544349b2edf0e01L114-R126) [[3]](diffhunk://#diff-a91ef6c8ef0ab6b4fdf21f08b20730c3420fa2f78126ea0403ee7fa73af7d0c8R51-R143) * Enhanced Python dependency management by integrating `uv` for caching and special handling for Python 3.13 in workflows like `argilla.yml` and `argilla-server.yml`. [[1]](diffhunk://#diff-d69af7b78e5cc7d03e327582d01e6cc2780c0cc8c91e45f98544349b2edf0e01R91-R97) [[2]](diffhunk://#diff-a91ef6c8ef0ab6b4fdf21f08b20730c3420fa2f78126ea0403ee7fa73af7d0c8R51-R143) ### Testing Improvements: * Added a mock implementation for `tabulator-tables` in `argilla-frontend/__mocks__/tabulator-tables.js` to facilitate unit testing without relying on external libraries. * Updated tests in `argilla-frontend/components/base/base-render-table/renderTable.test.ts` to include mock functionality for `Tabulator` and adjusted test cases for better reliability. [[1]](diffhunk://#diff-beda1c0e26dd21e30b4d0523c659dc99e728035c5ba1dce2bf9b934458a3c959L5-R8) [[2]](diffhunk://#diff-beda1c0e26dd21e30b4d0523c659dc99e728035c5ba1dce2bf9b934458a3c959R90-R96) ### Miscellaneous Changes: * Added a new changelog entry in `argilla-frontend/CHANGELOG.md` for version `2.8.0`, highlighting the addition of Japanese language support. * Replaced `conda-incubator/setup-miniconda@v2` with `@v3` in `.github/workflows/argilla-v1.yml` for improved compatibility and updated the Miniforge variant. These updates collectively improve the project's maintainability, testing infrastructure, and compatibility with newer technologies.
jonnywireless
pushed a commit
that referenced
this pull request
Aug 2, 2025
This pull request introduces several updates across the codebase, including dependency upgrades, CI/CD workflow improvements, and new mock implementations for testing. The key changes focus on enhancing compatibility, improving test coverage, and streamlining workflows. ### Dependency and Configuration Updates: * Upgraded Elasticsearch image from `8.15.0` to `8.17.0` in `.devcontainer/docker-compose/docker-compose.yml` and `.github/workflows/argilla-server.yml` to ensure compatibility with the latest features and fixes. [[1]](diffhunk://#diff-989e458ca6018c442472aa47d71d0671edf1eec0a93d4f569ac6435e564022ecL21-R21) [[2]](diffhunk://#diff-d69af7b78e5cc7d03e327582d01e6cc2780c0cc8c91e45f98544349b2edf0e01L38-R38) * Updated the default Docker image in `.github/workflows/argilla-frontend.deploy-environment.yml` to align with the new project namespace (`extralit/argilla-quickstart-for-dev`). ### CI/CD Workflow Enhancements: * Added detailed documentation for GitHub Actions workflows in `.github/workflows/README.md`, including caching strategies, environment variables, and troubleshooting steps. * Introduced test coverage upload to Codecov in multiple workflows (`argilla-frontend.yml`, `argilla-server.yml`, and `argilla.yml`) to improve visibility into test quality. [[1]](diffhunk://#diff-05a21f9ed6d46c5b14e3c55a3da7fcb48c8b8a0616ca82d28b57e0176726fe19L68-R81) [[2]](diffhunk://#diff-d69af7b78e5cc7d03e327582d01e6cc2780c0cc8c91e45f98544349b2edf0e01L114-R126) [[3]](diffhunk://#diff-a91ef6c8ef0ab6b4fdf21f08b20730c3420fa2f78126ea0403ee7fa73af7d0c8R51-R143) * Enhanced Python dependency management by integrating `uv` for caching and special handling for Python 3.13 in workflows like `argilla.yml` and `argilla-server.yml`. [[1]](diffhunk://#diff-d69af7b78e5cc7d03e327582d01e6cc2780c0cc8c91e45f98544349b2edf0e01R91-R97) [[2]](diffhunk://#diff-a91ef6c8ef0ab6b4fdf21f08b20730c3420fa2f78126ea0403ee7fa73af7d0c8R51-R143) ### Testing Improvements: * Added a mock implementation for `tabulator-tables` in `argilla-frontend/__mocks__/tabulator-tables.js` to facilitate unit testing without relying on external libraries. * Updated tests in `argilla-frontend/components/base/base-render-table/renderTable.test.ts` to include mock functionality for `Tabulator` and adjusted test cases for better reliability. [[1]](diffhunk://#diff-beda1c0e26dd21e30b4d0523c659dc99e728035c5ba1dce2bf9b934458a3c959L5-R8) [[2]](diffhunk://#diff-beda1c0e26dd21e30b4d0523c659dc99e728035c5ba1dce2bf9b934458a3c959R90-R96) ### Miscellaneous Changes: * Added a new changelog entry in `argilla-frontend/CHANGELOG.md` for version `2.8.0`, highlighting the addition of Japanese language support. * Replaced `conda-incubator/setup-miniconda@v2` with `@v3` in `.github/workflows/argilla-v1.yml` for improved compatibility and updated the Miniforge variant. These updates collectively improve the project's maintainability, testing infrastructure, and compatibility with newer technologies.
jonnywireless
pushed a commit
that referenced
this pull request
Aug 2, 2025
This pull request introduces several updates across the codebase, including dependency upgrades, CI/CD workflow improvements, and new mock implementations for testing. The key changes focus on enhancing compatibility, improving test coverage, and streamlining workflows. ### Dependency and Configuration Updates: * Upgraded Elasticsearch image from `8.15.0` to `8.17.0` in `.devcontainer/docker-compose/docker-compose.yml` and `.github/workflows/argilla-server.yml` to ensure compatibility with the latest features and fixes. [[1]](diffhunk://#diff-989e458ca6018c442472aa47d71d0671edf1eec0a93d4f569ac6435e564022ecL21-R21) [[2]](diffhunk://#diff-d69af7b78e5cc7d03e327582d01e6cc2780c0cc8c91e45f98544349b2edf0e01L38-R38) * Updated the default Docker image in `.github/workflows/argilla-frontend.deploy-environment.yml` to align with the new project namespace (`extralit/argilla-quickstart-for-dev`). ### CI/CD Workflow Enhancements: * Added detailed documentation for GitHub Actions workflows in `.github/workflows/README.md`, including caching strategies, environment variables, and troubleshooting steps. * Introduced test coverage upload to Codecov in multiple workflows (`argilla-frontend.yml`, `argilla-server.yml`, and `argilla.yml`) to improve visibility into test quality. [[1]](diffhunk://#diff-05a21f9ed6d46c5b14e3c55a3da7fcb48c8b8a0616ca82d28b57e0176726fe19L68-R81) [[2]](diffhunk://#diff-d69af7b78e5cc7d03e327582d01e6cc2780c0cc8c91e45f98544349b2edf0e01L114-R126) [[3]](diffhunk://#diff-a91ef6c8ef0ab6b4fdf21f08b20730c3420fa2f78126ea0403ee7fa73af7d0c8R51-R143) * Enhanced Python dependency management by integrating `uv` for caching and special handling for Python 3.13 in workflows like `argilla.yml` and `argilla-server.yml`. [[1]](diffhunk://#diff-d69af7b78e5cc7d03e327582d01e6cc2780c0cc8c91e45f98544349b2edf0e01R91-R97) [[2]](diffhunk://#diff-a91ef6c8ef0ab6b4fdf21f08b20730c3420fa2f78126ea0403ee7fa73af7d0c8R51-R143) ### Testing Improvements: * Added a mock implementation for `tabulator-tables` in `argilla-frontend/__mocks__/tabulator-tables.js` to facilitate unit testing without relying on external libraries. * Updated tests in `argilla-frontend/components/base/base-render-table/renderTable.test.ts` to include mock functionality for `Tabulator` and adjusted test cases for better reliability. [[1]](diffhunk://#diff-beda1c0e26dd21e30b4d0523c659dc99e728035c5ba1dce2bf9b934458a3c959L5-R8) [[2]](diffhunk://#diff-beda1c0e26dd21e30b4d0523c659dc99e728035c5ba1dce2bf9b934458a3c959R90-R96) ### Miscellaneous Changes: * Added a new changelog entry in `argilla-frontend/CHANGELOG.md` for version `2.8.0`, highlighting the addition of Japanese language support. * Replaced `conda-incubator/setup-miniconda@v2` with `@v3` in `.github/workflows/argilla-v1.yml` for improved compatibility and updated the Miniforge variant. These updates collectively improve the project's maintainability, testing infrastructure, and compatibility with newer technologies.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR integrates Argilla v2.8.0 into Extralit while carefully preserving all Extralit-specific features and customizations. The merge maintains our document extraction functionality, table handling, and schema management while bringing in Argilla's new features.
Closes #65 #74
This pull request introduces several updates across various files to improve functionality, enhance testing, and update dependencies. Key changes include upgrading Elasticsearch versions, adding test coverage reporting, configuring
uvfor Python dependency management, and improving frontend testing with mock implementations.Dependency Updates:
.devcontainer/docker-compose/docker-compose.yml: Upgraded Elasticsearch image from version8.15.0to8.17.0..github/workflows/argilla-server.yml: Updated Elasticsearch image to8.17.0and upgraded Codecov action tov5.4.3. [1] [2].github/workflows/argilla-v1.yml: Updated Conda setup action tov3and switched toMiniforge3.CI Tests:
argilla/tests/unit/andargilla/tests/integration/.github/workflows/argilla-frontend.yml: Added frontend test coverage reporting using Codecov and updated test scripts to include coverage.argilla-frontend/__mocks__/tabulator-tables.js: Added a comprehensive mock implementation fortabulator-tablesto support frontend tests.argilla-frontend/components/base/base-render-table/renderTable.test.ts: Improved test coverage by mocking Tabulator functionality and skipping unreliable tests. [1] [2]Python Dependency Management:
.github/workflows/argilla-server.ymland.github/workflows/argilla.yml: Configureduvfor Python dependency management, enabling caching and optimized installation paths. [1] [2]Frontend Enhancements:
argilla-frontend/components/features/login/OAuthLogin.vue: Refactored the OAuth login UI to use a structured list for providers, improving accessibility and styling. [1] [2]argilla-frontend/components/features/login/components/HuggingFaceButton.vue: Added a minimum height to the HuggingFace button for consistent UI design.