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

Improve .fs filetype detection #12219

Closed
wants to merge 2 commits into from
Closed

Conversation

jkotlinski
Copy link
Contributor

@jkotlinski jkotlinski commented Apr 2, 2023

Reduces false Forth negatives to about 5%, while keeping the number of false Forth positives very low. Closes #11988

Reduces false Forth negatives to about 5%, while
keeping the number of false F# positives very low.
Closes vim#11988
@codecov
Copy link

codecov bot commented Apr 2, 2023

Codecov Report

Merging #12219 (28654de) into master (3cdd799) will decrease coverage by 81.65%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           master   #12219       +/-   ##
===========================================
- Coverage   81.94%    0.29%   -81.65%     
===========================================
  Files         164      154       -10     
  Lines      194111   179293    -14818     
  Branches    43831    40970     -2861     
===========================================
- Hits       159055      522   -158533     
- Misses      22216   178714   +156498     
+ Partials    12840       57    -12783     
Flag Coverage Δ
huge-clang-none ?
huge-gcc-none ?
huge-gcc-testgui ?
huge-gcc-unittests 0.29% <ø> (ø)
linux 0.29% <ø> (-82.11%) ⬇️
mingw-x64-HUGE ?
mingw-x86-HUGE ?
windows ?

Flags with carried forward coverage won't be shown. Click here to find out more.

see 158 files with indirect coverage changes

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@brammool brammool closed this in 065088d Apr 2, 2023
clason added a commit to clason/neovim that referenced this pull request Apr 3, 2023
Problem:    .fs files are falsely recognized as forth files.
Solution:   Check 100 lines for something that looks like forth. (Johan
            Kotlinski, closes vim/vim#12219, closes vim/vim#11988)

vim/vim@065088d

Co-authored-by: Johan Kotlinski <kotlinski@gmail.com>
clason added a commit to clason/neovim that referenced this pull request Apr 3, 2023
Problem:    .fs files are falsely recognized as forth files.
Solution:   Check 100 lines for something that looks like forth. (Johan
            Kotlinski, closes vim/vim#12219, closes vim/vim#11988)

vim/vim@065088d

Co-authored-by: Johan Kotlinski <kotlinski@gmail.com>
clason added a commit to clason/neovim that referenced this pull request Apr 3, 2023
Problem:    .fs files are falsely recognized as forth files.
Solution:   Check 100 lines for something that looks like forth. (Johan
            Kotlinski, closes vim/vim#12219, closes vim/vim#11988)

vim/vim@065088d

Co-authored-by: Johan Kotlinski <kotlinski@gmail.com>
clason added a commit to clason/neovim that referenced this pull request Apr 3, 2023
Problem:    .fs files are falsely recognized as forth files.
Solution:   Check 100 lines for something that looks like forth. (Johan
            Kotlinski, closes vim/vim#12219, closes vim/vim#11988)

vim/vim@065088d

Co-authored-by: Johan Kotlinski <kotlinski@gmail.com>
clason added a commit to neovim/neovim that referenced this pull request Apr 3, 2023
…22871)

Problem:    .fs files are falsely recognized as forth files.
Solution:   Check 100 lines for something that looks like forth. (Johan
            Kotlinski, closes vim/vim#12219, closes vim/vim#11988)

vim/vim@065088d

Co-authored-by: Johan Kotlinski <kotlinski@gmail.com>
theHamsta pushed a commit to theHamsta/neovim that referenced this pull request Apr 7, 2023
…eovim#22871)

Problem:    .fs files are falsely recognized as forth files.
Solution:   Check 100 lines for something that looks like forth. (Johan
            Kotlinski, closes vim/vim#12219, closes vim/vim#11988)

vim/vim@065088d

Co-authored-by: Johan Kotlinski <kotlinski@gmail.com>
Xevnar added a commit to Xevnar/filetype.nvim that referenced this pull request Apr 8, 2023
…m#22871)

Problem:    .fs files are falsely recognized as forth files.
Solution:   Check 100 lines for something that looks like forth. (Johan Kotlinski, closes vim/vim#12219, closes vim/vim#11988)

vim/vim@065088d
https://github.com/neovim/neovim/commit

Co-authored-by: Johan Kotlinski <kotlinski@gmail.com>
craigmac pushed a commit to craigmac/vim that referenced this pull request Apr 22, 2023
Problem:    .fs files are falsely recognized as forth files.
Solution:   Check 100 lines for something that looks like forth. (Johan
            Kotlinski, closes vim#12219, closes vim#11988)
folke pushed a commit to folke/neovim that referenced this pull request May 22, 2023
…eovim#22871)

Problem:    .fs files are falsely recognized as forth files.
Solution:   Check 100 lines for something that looks like forth. (Johan
            Kotlinski, closes vim/vim#12219, closes vim/vim#11988)

vim/vim@065088d

Co-authored-by: Johan Kotlinski <kotlinski@gmail.com>
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.

Forth/F# filetype detection gives up too early
1 participant