Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Disable alternate numbers format plugin temporary #2498

Merged
merged 6 commits into from
Dec 18, 2021

Conversation

jneira
Copy link
Member

@jneira jneira commented Dec 17, 2021

@jneira jneira requested a review from drsooch December 17, 2021 12:35
Copy link
Member

@Ailrun Ailrun left a comment

Choose a reason for hiding this comment

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

LGTM other than the hackage issue mentioned above. Here's some minor suggestion:

Copy link
Collaborator

@drsooch drsooch left a comment

Choose a reason for hiding this comment

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

Not sure if I actually need to approve. But I will anyways.... lgtm

@jneira jneira force-pushed the disable-alternate-numbers branch from e93ba89 to 7d99b25 Compare December 17, 2021 21:30
@jneira
Copy link
Member Author

jneira commented Dec 17, 2021

Not sure if I actually need to approve. But I will anyways.... lgtm

no needed technically but needed anyways, thanks!

@jneira jneira added the merge me Label to trigger pull request merge label Dec 17, 2021
@mergify mergify bot merged commit 15d7402 into haskell:master Dec 18, 2021
pepeiborra pushed a commit that referenced this pull request Dec 19, 2021
* Disable alternateNumberFormat temporary

* Disable alternateNumberFormat tests

* Add a warn in docs

* Add link to issue in docs

Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
drsooch added a commit that referenced this pull request Dec 22, 2021
jneira pushed a commit that referenced this pull request Dec 22, 2021
* Use `extQ` to choose parser.

- In the process of development, it was forgotten that the "parent" type
  of patterns and exprs is different. When traversal occurs, SYB would
  throw out `Pat` types as it was only expecting `HsExpr` types.

- Using `extQ` allows us to chain the expected types and we can then
  destructure patterns appropriately.

* Revert "Disable alternate numbers format plugin temporary (#2498)"

This reverts commit 15d7402.
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
merge me Label to trigger pull request merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants