Try to fix CI

[ErikSchierboom]

No description provided.

[ErikSchierboom]

@kotp I've upgraded a bunch of stuff, but now I'm getting a different error. Would you mind looking into it?

[kotp]

Spent quite some time researching.

Without any success so far for a fix.

But pushing to see if the CI passes with this change...

[ErikSchierboom]

It looks like that did fix some things, as we now only have the knapsack error:

running tests for: knapsack
/tmp/knapsack20240406-1977-30tzgx/knapsack_test.rb:5:in `class:KnapsackTest': uninitialized constant KnapsackTest::Data (NameError)

Item = Data.define(:weight, :value)
^^^^
from /tmp/knapsack20240406-1977-30tzgx/knapsack_test.rb:4:in `

'

[kotp]

It looks like that did fix some things,

The fix is mostly "less detail" or "less specific" so the behavior may not be what was intended when it was written.

as we now only have the knapsack error:

running tests for: knapsack
/tmp/knapsack20240406-1977-30tzgx/knapsack_test.rb:5:in `class:KnapsackTest': uninitialized constant KnapsackTest::Data (NameError)

Item = Data.define(:weight, :value) ^^^^ from /tmp/knapsack20240406-1977-30tzgx/knapsack_test.rb:4:in `

'

This is because of Data not being available for the version of Ruby. A separate issue, really. If you want to bring it in even with the Ruby 3.1 failing, I can definitely work a solution that will pass the Data not being available with that version. (While still using Data as it is when it is provided by the language.)

[kotp]

The last commit does fix the knapsack, I think, but fails for where Data is being used for the zebra exercise.

[ErikSchierboom]

The last commit does fix the knapsack, I think, but fails for where Data is being used for the zebra exercise.

Any chance you could fix that too?

[kotp]

The last commit does fix the knapsack, I think, but fails for where Data is being used for the zebra exercise.

Any chance you could fix that too?

Ruby 3.0 has just transitioned to End of Life, Ruby 3.1 will be end of life in the next 11 months. Perhaps we make this "reportive" rather than required?

[ErikSchierboom]

Ruby 3.0 has just transitioned to End of Life, Ruby 3.1 will be end of life in the next 11 months. Perhaps we make this "reportive" rather than required?

Let's just test on Ruby 3.2 and Ruby 3.3 then.

[kotp]

Thank you @ErikSchierboom !

[ErikSchierboom]

You too!