Skip to content

gitignore .tool-versions#460

Merged
PragTob merged 1 commit intomainfrom
actually-ignore-tool-versions
Apr 13, 2025
Merged

gitignore .tool-versions#460
PragTob merged 1 commit intomainfrom
actually-ignore-tool-versions

Conversation

@PragTob
Copy link
Member

@PragTob PragTob commented Apr 13, 2025

Kinda changed my opinion on this, it became annoying to upgrade patch versions on my other systems just to be in sync. Since this is a library that should work on many versions and runs CI that way there is no need for strict enforcement.

Kinda changed my opinion on this, it became annoying to upgrade patch versions on my other
systems just to be in sync. Since this is a library that should work on _many_ versions and
runs CI that way there is no need for strict enforcement.
@PragTob
Copy link
Member Author

PragTob commented Apr 13, 2025

Flakey where the runner was seemingly not fast enough:

   1) test warn when functions are evaluated warns when run in iex (BencheeTest)
Error:      test/benchee_test.exs:1042
     Assertion failed, no matching message after 20000ms
     The following variables were pinned:
       port = #Port<0.41>
     Showing 8 of 8 messages in the mailbox
     code: assert_receive {^port, {:data, "iex(2)> "}}
     mailbox:
       pattern: {^port, {:data, "iex(2)> "}}
       value:   {#Port<0.41>,
                 {:data,
                  "Interactive Elixir (1.15.8) - press Ctrl+C to exit (type h() ENTER for help)\n"}}

       pattern: {^port, {:data, "iex(2)> "}}
       value:   {#Port<0.41>,
                 {:data,
                  "Operating System: Linux\nCPU Information: AMD EPYC 7763 64-Core Processor\nNumber of Available Cores: 4\nAvailable memory: 15.62 GB\nElixir 1.15.8\nErlang 24.3.4.17\nJIT enabled: true\n\n"}}

       pattern: {^port, {:data, "iex(2)> "}}
       value:   {#Port<0.41>,
                 {:data,

@PragTob PragTob merged commit 011d9c5 into main Apr 13, 2025
43 of 44 checks passed
@PragTob PragTob deleted the actually-ignore-tool-versions branch April 13, 2025 09:45
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.

1 participant