Skip to content

fix: add additional linting for html #624

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 5 commits into from
Closed

fix: add additional linting for html #624

wants to merge 5 commits into from

Conversation

@Revathyvenugopal162
Copy link
Contributor

@Revathyvenugopal162 Revathyvenugopal162 commented Feb 11, 2025 •
edited
Loading

  • Add precommit hook for html
  • update the prettier lint to a stable package

@github-actions github-actions bot added maintenance Generic maintenance related ui/ux Related with UI/UX design fix Pull requests related to resolving problems or errors labels Feb 11, 2025
@Revathyvenugopal162 Revathyvenugopal162 marked this pull request as ready for review February 11, 2025 10:35
@Revathyvenugopal162 Revathyvenugopal162 requested a review from a team as a code owner February 11, 2025 10:35
@Revathyvenugopal162
Copy link
Contributor Author

Revathyvenugopal162 commented Feb 11, 2025

#622 , #623 cam be merge after merging this PR

@Revathyvenugopal162 Revathyvenugopal162 marked this pull request as draft February 11, 2025 11:14
@Revathyvenugopal162
Copy link
Contributor Author

Revathyvenugopal162 commented Feb 17, 2025

Since #627 fixes the Prettier issue, we can reopen this once we receive confirmation on the package's license.

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

Reviewers

@jorgepiloto jorgepiloto Awaiting requested review from jorgepiloto

Assignees

No one assigned

Labels

fix Pull requests related to resolving problems or errors maintenance Generic maintenance related ui/ux Related with UI/UX design

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@Revathyvenugopal162 @pyansys-ci-bot