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

Resolve absolute path to operatorsFile #116

Closed
wants to merge 2 commits into from

Conversation

colinwahl
Copy link
Collaborator

Fixes #115

I've verified this locally. To verify yourself you can follow the following steps:

Add an empty .tidyoperators file in the root of purescript-tidy. Modify the existing .tidyrc.json to have "operatorsFile": ".tidyoperators". Go to the parent directory for purescript-tidy. At this point, formatting purescript-tidy/src via the current purs-tidy will result in an error. Build this branch and retry with the local script - it should succeed.

@colinwahl colinwahl self-assigned this Dec 8, 2023
bin/Main.purs Outdated Show resolved Hide resolved
Co-authored-by: Nathan Faubion <[email protected]>
@natefaubion natefaubion closed this Dec 8, 2023
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.

Incorrect resolving of operatorsFile path
2 participants