Skip to content

ci: update github actions & fix UndefVarError #293

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

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

inkydragon
Copy link

ChainRulesTestUtils.jl not work with julia nightly

https://github.com/JuliaMath/AbstractFFTs.jl/actions/runs/14931159549/job/41947508400?pr=139#step:6:160

test_frule: fftshift on Vector{Float64},: Error During Test at /home/runner/.julia/packages/ChainRulesTestUtils/Ko1Wr/src/testers.jl:135
  Test threw exception
  Expression: 
  UndefVarError: `eval_test_comparison` not defined in `ChainRulesTestUtils`
  Suggestion: check for spelling errors or missing imports.
  Stacktrace:
    [1] macro expansion
      @ /opt/hostedtoolcache/julia/nightly/x64/share/julia/stdlib/v1.13/Test/src/Test.jl:727 [inlined]
    [2] macro expansion
      @ ~/.julia/packages/ChainRulesTestUtils/Ko1Wr/src/testers.jl:135 [inlined]
    [3] macro expansion
      @ /opt/hostedtoolcache/julia/nightly/x64/share/julia/stdlib/v1.13/Test/src/Test.jl:1833 [inlined]
    [4] test_frule(::RuleConfig, ::Any, ::Any, ::Vararg{Any}; output_tangent::Any, fdm::Any, frule_f::Any, check_inferred::Bool, fkwargs::NamedTuple, rtol::Real, atol::Real, testset_name::Any, kwargs...)
      @ ChainRulesTestUtils ~/.julia/packages/ChainRulesTestUtils/Ko1Wr/src/testers.jl:125
    [5] test_frule
      @ ~/.julia/packages/ChainRulesTestUtils/Ko1Wr/src/testers.jl:103 [inlined]
    [6] #test_frule#54
      @ ~/.julia/packages/ChainRulesTestUtils/Ko1Wr/src/testers.jl:100 [inlined]
    [7] test_frule(::Any, ::Any, ::Any)
      @ ChainRulesTestUtils ~/.julia/packages/ChainRulesTestUtils/Ko1Wr/src/testers.jl:99
    [8] top-level scope
      @ ~/work/AbstractFFTs.jl/AbstractFFTs.jl/test/runtests.jl:197
    [9] macro expansion
      @ /opt/hostedtoolcache/julia/nightly/x64/share/julia/stdlib/v1.13/Test/src/Test.jl:[183](https://github.com/JuliaMath/AbstractFFTs.jl/actions/runs/14931159549/job/41947508400?pr=139#step:6:186)3 [inlined]
   [10] macro expansion
      @ ~/work/AbstractFFTs.jl/AbstractFFTs.jl/test/runtests.jl:198 [inlined]
   [11] macro expansion
      @ /opt/hostedtoolcache/julia/nightly/x64/share/julia/stdlib/v1.13/Test/src/Test.jl:1833 [inlined]
   [12] macro expansion
      @ ~/work/AbstractFFTs.jl/AbstractFFTs.jl/test/runtests.jl:207 [inlined]
   [13] include(mapexpr::Function, mod::Module, _path::String)
      @ Base ./Base.jl:310
   [14] top-level scope
      @ none:6
   [15] eval(m::Module, e::Any)
      @ Core ./boot.jl:489
   [16] exec_options(opts::Base.JLOptions)
      @ Base ./client.jl:297
   [17] _start()
      @ Base ./client.jl:564

Copy link

codecov bot commented May 9, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 95.62%. Comparing base (f7388ec) to head (f7fc032).
Report is 5 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #293      +/-   ##
==========================================
+ Coverage   93.71%   95.62%   +1.91%     
==========================================
  Files          12       12              
  Lines         350      343       -7     
==========================================
  Hits          328      328              
+ Misses         22       15       -7     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@inkydragon inkydragon marked this pull request as ready for review May 9, 2025 15:54
# 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.

1 participant