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

fix: Prettier Workflow run is failing for the recent PR merges #3638

Open
1 of 2 tasks
theAravinthM opened this issue Apr 3, 2024 · 1 comment
Open
1 of 2 tasks

Comments

@theAravinthM
Copy link
Collaborator

theAravinthM commented Apr 3, 2024

The Issue

As we can see in these images, Prettier Workflow run if failing for the recent PR merges. The code is not robust and not easy-to-read and this can cause linting errors.

Short term Solution

One can edit the whole index.html file by using the Prettier formatter and make a Pull Request to merge it.

Long term Solution

Although the above solution can be a temporary fix, this problem might arrive in the future as the new Contributors come in and make a PR.

  • So, it is better to mention it in the Contribution guidelines to check for this Prettier Code format as well - in order to not encounter the same issue in the future.

Tasks

  • Short Term Solution
  • Long Term Solution

image

image

@theAravinthM theAravinthM added good first issue changes requested Contributor is requested to make the required changes and removed changes requested Contributor is requested to make the required changes labels Apr 3, 2024
theAravinthM added a commit that referenced this issue Apr 4, 2024
fixed Prettier Formatting problem - fixes issue #3638 - short term solution
@theAravinthM
Copy link
Collaborator Author

Long term solution yet to be merged!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant