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

frontend: data-pipelines (#5) ui bundle sources #1632

Merged
merged 7 commits into from
Feb 20, 2023

Conversation

ivakoleva
Copy link
Contributor

This is a stacked PR #5 (follows #1631):
data-pipelines ui bundle sources.

Sources added.

Testing done: fetched the frontend/shared-components changes
(from person/mdilyan/os-shared-components feature branch in review),
and verified frontend/data-pipelines manual start + unit + e2e tests.

@ivakoleva ivakoleva changed the title frontend: data-pipelines(#5) ui bundle sources frontend: data-pipelines (#5) ui bundle sources Feb 13, 2023
.pre-commit-config.yaml Outdated Show resolved Hide resolved
This is a stacked PR #5:
data-pipelines ui bundle sources.

Sources added.

Testing done: fetched the `frontend/shared-components` changes
(from person/mdilyan/os-shared-components feature branch in review),
and verified `frontend/data-pipelines` manual start + unit + e2e tests.

Signed-off-by: ivakoleva <iva.koleva@clearcode.bg>
@ivakoleva ivakoleva force-pushed the person/ikoleva/frontend-data-pipelines-5 branch from 9de39ea to 4660fe5 Compare February 17, 2023 15:14
ivakoleva and others added 5 commits February 17, 2023 17:14
This is a stacked PR #5:
data-pipelines ui bundle sources.

Sources added.

Testing done: fetched the `frontend/shared-components` changes
(from person/mdilyan/os-shared-components feature branch in review),
and verified `frontend/data-pipelines` manual start + unit + e2e tests.

Signed-off-by: ivakoleva <iva.koleva@clearcode.bg>
This is a stacked PR #5:
data-pipelines ui bundle sources.

Sources added.

Testing done: fetched the `frontend/shared-components` changes
(from person/mdilyan/os-shared-components feature branch in review),
and verified `frontend/data-pipelines` manual start + unit + e2e tests.

Signed-off-by: ivakoleva <iva.koleva@clearcode.bg>
@ivakoleva
Copy link
Contributor Author

Added link to codacy static code analysis #1623
Fixes may be related to functional changes, we should have those in a separate effort

@ivakoleva ivakoleva enabled auto-merge (squash) February 20, 2023 08:20
@ivakoleva ivakoleva merged commit 8396662 into main Feb 20, 2023
@ivakoleva ivakoleva deleted the person/ikoleva/frontend-data-pipelines-5 branch February 20, 2023 08:28
ivakoleva added a commit that referenced this pull request Feb 20, 2023
This is a stacked PR #5 (follows
#1631):
data-pipelines ui bundle sources.

Sources added.

Testing done: fetched the `frontend/shared-components` changes 
(from person/mdilyan/os-shared-components feature branch in review),
and verified `frontend/data-pipelines` manual start + unit + e2e tests.

---------

Signed-off-by: ivakoleva <iva.koleva@clearcode.bg>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
ivakoleva added a commit that referenced this pull request Feb 22, 2023
This is a stacked PR #6 (follows
#1632):
data-pipelines e2e tests.

End-to-end tests sources added.

Testing done: fetched the `frontend/shared-components` changes 
(from person/mdilyan/os-shared-components feature branch in review),
and verified `frontend/data-pipelines` manual start + unit + e2e tests.

---------

Signed-off-by: ivakoleva <iva.koleva@clearcode.bg>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants