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

[Auto] Update hlint and fourmolu image #55

Merged
merged 1 commit into from
Mar 13, 2024
Merged

[Auto] Update hlint and fourmolu image #55

merged 1 commit into from
Mar 13, 2024

Conversation

fossabot
Copy link
Collaborator

This PR should be approved once the latest HLS includes this fourmolu version.

@fossabot fossabot requested a review from a team as a code owner March 11, 2024 12:02
@fossabot fossabot requested review from a team and spatten March 11, 2024 12:02
@fossabot fossabot enabled auto-merge (squash) March 11, 2024 12:02
@fossabot fossabot requested a review from lackstein March 11, 2024 12:02
@lackstein lackstein removed their request for review March 11, 2024 17:03
@lackstein
Copy link
Member

Removing myself from review since I no longer have ownership over any Haskell codebases.

csasarak
csasarak approved these changes Mar 13, 2024
View reviewed changes
Copy link
Contributor

@csasarak csasarak left a comment

Choose a reason for hiding this comment

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

This update is fine. Currently HLS uses a local system cabal-fmt so it's not possible to keep this in sync with HLS like it is for fourmolu.

@fossabot fossabot merged commit d884199 into main Mar 13, 2024
4 checks passed
@fossabot fossabot deleted the auto/update-image branch March 13, 2024 16:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@csasarak csasarak csasarak approved these changes

@spatten spatten Awaiting requested review from spatten spatten is a code owner automatically assigned from fossas/analysis

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@fossabot @lackstein @csasarak