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

Feature/refactor formatter #111

Merged
merged 20 commits into from
Oct 3, 2024
Merged

Feature/refactor formatter #111

merged 20 commits into from
Oct 3, 2024

Conversation

kcinay055679
Copy link
Contributor

No description provided.

@kcinay055679 kcinay055679 force-pushed the feature/refactor-formatter branch from f97efba to 2ed2e36 Compare September 26, 2024 06:19
@kcinay055679 kcinay055679 linked an issue Sep 26, 2024 that may be closed by this pull request
2 tasks
Copy link
Collaborator

@ManuelMoeri ManuelMoeri left a comment

Choose a reason for hiding this comment

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

Looks fine.

@kcinay055679 kcinay055679 self-assigned this Sep 26, 2024
@kcinay055679 kcinay055679 force-pushed the feature/refactor-formatter branch from ea8bc1e to 47ecd0c Compare September 27, 2024 13:53
@kcinay055679 kcinay055679 marked this pull request as ready for review September 27, 2024 13:55
pizzi-cato
pizzi-cato previously approved these changes Oct 1, 2024
Copy link
Collaborator

@MasterEvarior MasterEvarior left a comment

Choose a reason for hiding this comment

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

Almost perfect, just two minor things 😃

MasterEvarior
MasterEvarior previously approved these changes Oct 2, 2024
Copy link
Collaborator

@MasterEvarior MasterEvarior left a comment

Choose a reason for hiding this comment

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

Looks good 👍

One thing that I think might be important is that as soon as this is merged, everybodies pre-commit hooks will stop working. It might be good idea to send a chat message to everybody informing them about that and what to do that they start working again

MasterEvarior
MasterEvarior previously approved these changes Oct 3, 2024
@MasterEvarior
Copy link
Collaborator

LGTM 😃
Just fix the conflicts and then I can merge it

@MasterEvarior MasterEvarior merged commit 5894cac into main Oct 3, 2024
3 checks passed
@MasterEvarior MasterEvarior deleted the feature/refactor-formatter branch October 4, 2024 08:55
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.

Frontendfiles formatter
6 participants