using the latest version of toml returns errors during build #9

New Issue

Open

opened 2023-12-07 15:00:53 +01:00 by rya · 3 comments

rya commented

2023-12-07 15:00:53 +01:00

Collaborator

the current newest version (commit db53c77) replaced TOML::Type with TOML::Any, although simply replacing all instances of TOML::Type with TOML::Any in the code doesn't seem to work, or at least I am not smart enough to replace it properly... I got stuck here if it's any helpful info

In src/endpoints/accounts/getURL.cr:10:6

 10 | "#{config_get("general.hostname").as(String)}/#{config_get("general.append_path").as(String)[0..-2]}"
         ^
Error: can't cast (Hash(String, TOML::Any) | TOML::Any | Nil) to String

the laziest hotfix I can think of is make the dependency use a commit before the TOML::Any changes

the current newest version (commit [db53c77](https://github.com/crystal-community/toml.cr/commit/db53c77b6973369c8d5575d20ee91dec971a6fee)) replaced `TOML::Type` with `TOML::Any`, although simply replacing all instances of TOML::Type with TOML::Any in the code doesn't seem to work, or at least I am not smart enough to replace it properly... I got stuck here if it's any helpful info ``` In src/endpoints/accounts/getURL.cr:10:6 10 | "#{config_get("general.hostname").as(String)}/#{config_get("general.append_path").as(String)[0..-2]}" ^ Error: can't cast (Hash(String, TOML::Any) | TOML::Any | Nil) to String ``` the laziest hotfix I can think of is make the dependency use a commit before the TOML::Any changes