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.

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

Format Package.juvix when formatting project #3219

Merged
merged 3 commits into from
Dec 6, 2024
Merged

Format Package.juvix when formatting project #3219

merged 3 commits into from
Dec 6, 2024

Conversation

janmasrovira
Copy link
Collaborator

@janmasrovira janmasrovira commented Dec 3, 2024 •
edited
Loading

@janmasrovira janmasrovira added this to the 0.6.9 milestone Dec 3, 2024
@janmasrovira janmasrovira self-assigned this Dec 3, 2024
@janmasrovira janmasrovira linked an issue Dec 3, 2024 that may be closed by this pull request
juvix format should take Package.juvix into account #3217
Closed
@janmasrovira janmasrovira marked this pull request as ready for review December 3, 2024 11:53
@janmasrovira janmasrovira linked an issue Dec 4, 2024 that may be closed by this pull request
juvix format --in-place with no argument does not format the project Package.juvix #2898
Closed
@janmasrovira janmasrovira force-pushed the 3217-juvix-format-should-take-packagejuvix-into-account branch from 0e40f35 to d934e92 Compare December 4, 2024 14:14
paulcadman
paulcadman previously approved these changes Dec 5, 2024
View reviewed changes
@janmasrovira janmasrovira force-pushed the 3217-juvix-format-should-take-packagejuvix-into-account branch from d934e92 to 12bc3fb Compare December 5, 2024 13:50
@janmasrovira janmasrovira force-pushed the 3217-juvix-format-should-take-packagejuvix-into-account branch from 12bc3fb to 8dde6bc Compare December 6, 2024 08:30
@paulcadman paulcadman force-pushed the 3217-juvix-format-should-take-packagejuvix-into-account branch from 8dde6bc to 32878cc Compare December 6, 2024 09:47
@janmasrovira janmasrovira merged commit 3e367aa into main Dec 6, 2024
4 checks passed
@janmasrovira janmasrovira deleted the 3217-juvix-format-should-take-packagejuvix-into-account branch December 6, 2024 10:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@paulcadman paulcadman paulcadman approved these changes

@lukaszcz lukaszcz Awaiting requested review from lukaszcz

Assignees

@janmasrovira janmasrovira

Labels
fix:bug juvix-formatter
Projects
None yet
Milestone
0.6.9
2 participants
@janmasrovira @paulcadman