Skip to content

Commit

Permalink
taking a more conservative approach
Browse files Browse the repository at this point in the history
  • Loading branch information
robaiken committed Feb 27, 2025
1 parent 050e88d commit 92cfe0a
Showing 1 changed file with 3 additions and 7 deletions.
10 changes: 3 additions & 7 deletions elm/lib/dependabot/elm/file_parser.rb
Original file line number Diff line number Diff line change
Expand Up @@ -162,8 +162,7 @@ def build_elm_json_dependency(name:, group:, requirement:, direct:)

sig { returns(String) }
def repo_type
type = parsed_elm_json.fetch("type")
T.must(type.is_a?(String) ? type : nil)
parsed_elm_json.fetch("type")
end

sig { override.void }
Expand All @@ -186,12 +185,9 @@ def version_for(version_requirement)
req.requirements.first.last
end

sig { returns(T::Hash[String, T.any(String, T::Hash[String, T.any(String, T::Hash[String, String])])]) }
sig { returns(T.untyped) }
def parsed_elm_json
@parsed_elm_json ||= T.let(JSON.parse(T.must(T.must(elm_json).content)),
T.nilable(T::Hash[String,
T.any(String,
T::Hash[String, T.any(String, T::Hash[String, String])])]))
@parsed_elm_json ||= T.let(JSON.parse(T.must(T.must(elm_json).content)), T.untyped)
rescue JSON::ParserError
raise Dependabot::DependencyFileNotParseable, elm_json&.path || MANIFEST_FILE
end
Expand Down

0 comments on commit 92cfe0a

Please sign in to comment.