-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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: OPTIC-277: Move LSE to Monorepo frontend #5184
Merged
Merged
Conversation
This file contains 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
* fix: LEAP-1: Start migration to monorepo * Fix assets and useEffects returning * add new rules to svg * remove duplicated assets * add back frontend folder * get old build * add back deleted files * remove diffs * remove default eslint rc * Remove package-lock, update yarn.lock * Remove yarnrc with wrong release * move files from label_studio/frontend to web/apps * remove files from frontend folder * fix package.json * fix yarn.lock * fix assets and bem after move files --------- Co-authored-by: hlomzik <hlomzik@gmail.com>
* fix: LEAP-1: Start migration to monorepo * Fix assets and useEffects returning * add new rules to svg * remove duplicated assets * feat: Move datamanager into the monorepo * add datamanager into ls repo * update datamanager * fix some packages that was missing * Update web/.env.example Co-authored-by: yyassi-heartex <104568407+yyassi-heartex@users.noreply.github.com> --------- Co-authored-by: yyassi-heartex <104568407+yyassi-heartex@users.noreply.github.com>
* fix: LEAP-1: Start migration to monorepo * Fix assets and useEffects returning * add new rules to svg * remove duplicated assets * feat: Move datamanager into the monorepo * add datamanager into ls repo * feat: LEAP-53: Move editor into the monorepo * migrate editor to monorepo * add some new packages and try to add different webpack configs * add submodules * remove github folder * add submodules as folders * Adjust docker config to work with monorepo * merge conflicts * Fix js libraries paths * delete npmrc file * update gitignore to add the files that shouldn't be on git * delete gitignore * add watch for libs * add env.build and unification of webpack config * add env.build --------- Co-authored-by: Nick Skriabin <nr@fenelon.ru>
* feat: LEAP-324: Return builds and source files to LS monorepo * remove source files * fix icons * fix output file * remove examples * change output * change output and add examples
* fix: LEAP-347: LSF running as standalone app * config editor to work standalone
* feat: LEAP-54: Move editor e2e tests to the new LS monorepo * move e2e folder to test folder
#5043) * chore: LEAP-50: Update makefile to work with current state of monorepo * Refactoring Co-authored-by: bmartel <brandonmartel@gmail.com> * Refactoring Co-authored-by: bmartel <brandonmartel@gmail.com> * Keep frontend-fetch for potentional commented rollback --------- Co-authored-by: bmartel <brandonmartel@gmail.com>
…sts work (#5051) * feat: LEAP-54: Move editor e2e tests to the new LS monorepo * feat: LEAP-239: Move ls-frontend-test to monorepo and make cypress tests work * add cypress * move e2e folder to test folder * remove nx cypress and config default cypress * add the frontend-test as a lib * add frontend-test as a lib * fix cypress tests * remove dev build * delete package lock
…n tests (#5052) * feat: LEAP-54: Move editor e2e tests to the new LS monorepo * feat: LEAP-239: Move ls-frontend-test to monorepo and make cypress tests work * add cypress * move e2e folder to test folder * remove nx cypress and config default cypress * add the frontend-test as a lib * add frontend-test as a lib * fix cypress tests * remove dev build * delete package lock * feat: LEAP-56: Update package scripts to run units e2e and integration tests * remove dist folder * remove nx cloud * merge conflicts * clear commands
…tructure and the monorepo setup (#5058) feat: LEAP-325: ADd mechanist to switch between current structure and monorepo
* [submodules] Copy src HumanSignal/dm2 Workflow run: https://github.com/HumanSignal/label-studio/actions/runs/6948415494 * [submodules] Build static HumanSignal/dm2 Workflow run: https://github.com/HumanSignal/label-studio/actions/runs/6948415494 --------- Co-authored-by: Nikita Belonogov <nikita.belonogov@humansignal.com>
* feat: LEAP-379: Add docs to monorepo * fix typos
* [submodules] Copy src HumanSignal/dm2 Workflow run: https://github.com/HumanSignal/label-studio/actions/runs/6959725389 * [submodules] Build static HumanSignal/dm2 Workflow run: https://github.com/HumanSignal/label-studio/actions/runs/6959725389 * fix: LEAP-409: Update files to match with monorepo * [submodules] Copy src HumanSignal/dm2 Workflow run: https://github.com/HumanSignal/label-studio/actions/runs/6960200640 * [submodules] Build static HumanSignal/dm2 Workflow run: https://github.com/HumanSignal/label-studio/actions/runs/6960200640 * fix editor changing the index name and merging diffs --------- Co-authored-by: Julio Sgarbi <julio.sgarbi@hotmail.com>
* [submodules] Build static HumanSignal/label-studio-frontend Workflow run: https://github.com/HumanSignal/label-studio/actions/runs/6970505860 * docs: LSF Update Workflow run: https://github.com/HumanSignal/label-studio/actions/runs/6970505860 * [submodules] Copy src HumanSignal/label-studio-frontend Workflow run: https://github.com/HumanSignal/label-studio/actions/runs/6971479552 * [submodules] Build static HumanSignal/label-studio-frontend Workflow run: https://github.com/HumanSignal/label-studio/actions/runs/6971479552 --------- Co-authored-by: Nikita Belonogov <nikita.belonogov@humansignal.com>
jombooth
approved these changes
Jan 11, 2024
yyassi-heartex
approved these changes
Jan 11, 2024
/docker build
|
MihajloHoma
approved these changes
Jan 12, 2024
nikitabelonogov
approved these changes
Jan 30, 2024
nikitabelonogov
approved these changes
Feb 8, 2024
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.
PR fulfills these requirements
[fix|feat|ci|chore|doc]: TICKET-ID: Short description of change made
ex.fix: DEV-XXXX: Removed inconsistent code usage causing intermittent errors
Change has impacts in these area(s)
(check all that apply)
Describe the reason for change
To better support continued development, all open source frontend repositories have been moved to being libraries within LabelStudio. All usages of these are going to be pointed here and CI jobs reflected to support this. This allows for a much improved workflow which alleviates having to move around repositories just to identify and make a frontend change. This not only complicates core team development, but makes the task of externally contributing very daunting. This step moves the codebase in the right direction of reduced cognitive burden, and cohesive tooling.
What libraries were added/updated?
Nearly everything is updated in some facet to support the Monorepo within LabelStudio. Versions of dependencies stay roughly the same for most, with some minor updates for compatibility with NX tooling.
Does this PR introduce a breaking change?
(check only one)
What level of testing was included in the change?
(check all that apply)
Which logical domain(s) does this change affect?
All