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

Replace dprint prettier plugin with WASM plugins and yamlfmt #95

Merged
merged 6 commits into from
May 16, 2024

Conversation

kachick
Copy link
Owner

@kachick kachick commented May 13, 2024

Resolves #92

@@ -38,4 +38,4 @@ jobs:
- run: nix develop --command echo 'This step should be done before any other "nix develop" steps because of measuring Nix build time'
- run: nix develop --command task deps
- run: nix develop --command task check
# - run: nix run . -- --version
# - run: nix run . -- --version
Copy link
Owner Author

Choose a reason for hiding this comment

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

Looks like similar problem as google/yamlfmt#112

@@ -21,8 +21,7 @@ jobs:
- name: Judge
id: conclusion
if: >-
steps.dependabot.outputs.update-type != 'version-update:semver-major' ||
contains(steps.dependabot.outputs.dependency-names, 'DeterminateSystems')
steps.dependabot.outputs.update-type != 'version-update:semver-major' || contains(steps.dependabot.outputs.dependency-names, 'DeterminateSystems')
Copy link
Owner Author

Choose a reason for hiding this comment

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

I don't want this result, but yamlfmt can't keep the indent for now

Copy link
Owner Author

Choose a reason for hiding this comment

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

@kachick kachick force-pushed the drop-dprint-process-plugins branch from d2c44df to e7da4d0 Compare May 13, 2024 15:52
@kachick kachick changed the title Replace prettier with yamlfmt for yaml formatter Replace dprint prettier plugin with WASM plugins May 16, 2024
@kachick kachick changed the title Replace dprint prettier plugin with WASM plugins Replace dprint prettier plugin with WASM plugins and yamlfmt May 16, 2024
@kachick kachick marked this pull request as ready for review May 16, 2024 14:51
@kachick kachick merged commit 2e92325 into main May 16, 2024
10 checks passed
@kachick kachick deleted the drop-dprint-process-plugins branch May 16, 2024 14:52
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.

Do not depend on dprint process plugins
1 participant