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: Align code formatting in frontend projects #1863

Merged

Conversation

gorankokin
Copy link
Contributor

@gorankokin gorankokin commented Apr 7, 2023

Aligned formatting in shared and data-pipelines frontend using same configuration for prettier and .editorconfig Executed auto format on project data-pipelines.
No regression are expected, because only formatting is committed.
Tested local builds, linking built lib to UI and manual validatino on every screen, and seems there is no problems. ESLint executed successfully and there no issues.
Addressing issue #1801


Signed-off-by: gorankokin gorankokin@gmail.com

@gorankokin gorankokin added the area/frontend Related to changes in the folder projects/frontend for details about ui please see readme label Apr 7, 2023
Copy link
Contributor

@ivakoleva ivakoleva left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

May link the actual issue to the PR, so it is closed the moment change is merged

gorankokin and others added 2 commits April 10, 2023 17:09
Aligned formatting in shared and data-pipelines frontend
using same configuration for prettier and .editorconfig
Executed auto format on project data-pipelines.
No regression are expected, because only formatting
is committed.
Tested local builds, linking built lib to UI and manual
validatino on every screen, and seems there is no problems.
ESLint executed successfully and there no issues.

---------

Signed-off-by: gorankokin <gorankokin@gmail.com>
@gorankokin gorankokin force-pushed the person/gorankokin/align-formatting-frontend-projects-1801 branch from 5053f04 to e34fa21 Compare April 10, 2023 15:09
@gorankokin gorankokin enabled auto-merge (squash) April 10, 2023 16:07
@gorankokin gorankokin merged commit 70e4500 into main Apr 10, 2023
4 of 6 checks passed
@gorankokin gorankokin deleted the person/gorankokin/align-formatting-frontend-projects-1801 branch April 10, 2023 16:07
mivanov1988 pushed a commit that referenced this pull request Apr 13, 2023
Aligned formatting in shared and data-pipelines frontend using same
configuration for prettier and .editorconfig Executed auto format on
project data-pipelines.
No regression are expected, because only formatting is committed.
Tested local builds, linking built lib to UI and manual validatino on
every screen, and seems there is no problems. ESLint executed
successfully and there no issues.
Addressing issue #1801

---------

Signed-off-by: gorankokin <gorankokin@gmail.com>

---------

Signed-off-by: gorankokin <gorankokin@gmail.com>
Co-authored-by: Goran Kokinovski <gkokinovski@vmware.com>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/frontend Related to changes in the folder projects/frontend for details about ui please see readme cla-not-required
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants