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

KB-45995: Add workspace config, plugins linting and formatting config. #960

Merged
merged 4 commits into from
May 16, 2024

Conversation

usantos-at-wiris
Copy link
Contributor

@usantos-at-wiris usantos-at-wiris commented May 6, 2024

Description

Introducing the latest updates to our workspace, including enhanced linting, formatting capabilities, and additional extensions.

Steps to reproduce

Explore the workspace and try formatting existing files or examining recent changes. Test out the newly integrated extensions to identify any additional features you may require.

Please, if you have a suggestion over a desired extension config or issue, let me know!


Also closes branch #KB-45995-fmt

#taskid 45995

Changes.md

feat: Add workspace config and plugins for linting, formating and utilities. #KB-45995

@xjiang-at-wiris
Copy link
Contributor

For me everything is ok, except Prettier don't format when saving

Is better if anyone else could try it

Copy link
Contributor

@carla-at-wiris carla-at-wiris left a comment

Choose a reason for hiding this comment

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

The linting README should be updated as well to reflect the current configuration.

There should be a place on documentation where it specifies that Prettier should be installed globally through npm, otherwise there's an error where it says that the package is not found. Ideally, it should be on the package.json instead of having to install it globally, if possible, let's try to go that way.

The autoloading does not autoformat de document and the linter does not show the linting errors on a file.

.eslintrc.js Show resolved Hide resolved
package.json Show resolved Hide resolved
.vscode/extensions.json Outdated Show resolved Hide resolved
usantos-at-wiris and others added 2 commits May 13, 2024 14:49
Co-authored-by: Jesus H. <79973396+jhernandez-at-wiris@users.noreply.github.com>
…ed yarn format for yarn lint, installed prettier
Copy link
Contributor

@jgonzalez-at-wiris jgonzalez-at-wiris left a comment

Choose a reason for hiding this comment

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

Todo perfecto a priori!

@usantos-at-wiris usantos-at-wiris merged commit e713d30 into master May 16, 2024
14 checks passed
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.

None yet

5 participants