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

test: reproduce fd leak #2830

Merged
merged 6 commits into from
Feb 10, 2025
Merged

test: reproduce fd leak #2830

merged 6 commits into from
Feb 10, 2025

Conversation

anmonteiro
Copy link
Member

@anmonteiro anmonteiro commented Feb 9, 2025

reproduces and fixes #2829

@anmonteiro anmonteiro force-pushed the anmonteiro/repro-eol-failure branch from 9ea2468 to d768cf0 Compare February 9, 2025 08:08
@anmonteiro anmonteiro merged commit 9455966 into master Feb 10, 2025
20 checks passed
@anmonteiro anmonteiro deleted the anmonteiro/repro-eol-failure branch February 10, 2025 00:20
anmonteiro added a commit to anmonteiro/opam-repository that referenced this pull request Feb 10, 2025
CHANGES:

- rtop: read `~/.config/rtop/init.re` configuration file (@anmonteiro,
  [reasonml/reason#2813](reasonml/reason#2813))
    - the `-init FILE` flag works as before
- rtop: ignore `~/.ocamlinit.ml` or `~/.config/utop/init.ml` config files
  (@anmonteiro, [reasonml/reason#2813](reasonml/reason#2813))
- Add support for raw identifier syntax (@anmonteiro,
  [reasonml/reason#2796](reasonml/reason#2796))
- Fix: display attributes in record field and JSX props under punning
  (@pedrobslisboa, [reasonml/reason#2824](reasonml/reason#2824))
- Support modest Unicode letters in identifiers
  (@anmonteiro, [reasonml/reason#2828](reasonml/reason#2828))
- refmt: fix file descriptor leak
  (@anmonteiro, [reasonml/reason#2830](reasonml/reason#2830))
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

File handle not closed in refmt lib function
1 participant