Skip to content

Type specification edits#168

Merged
RobertDober merged 27 commits intoRobertDober:masterfrom
bradhanks:type_specs
Mar 21, 2025
Merged

Type specification edits#168
RobertDober merged 27 commits intoRobertDober:masterfrom
bradhanks:type_specs

Conversation

@bradhanks
Copy link
Collaborator

With the exception of [one code refactor] (85d7c1e), these commits are the addition or correction of type specifications.

I tried to keep the commits very narrow in scope so you can pick and choose which edits are wanted.

Passed mix test
Passed mix dialyzer

lib/earmark_parser/parser/list_info.ex:29:21:unknown_type
Unknown type: EarmarkParser.Line.ListItem.t/0.

lib/earmark_parser/options.ex:55:37:unknown_type
Unknown type: Message.t/0.
…ype: it's a list not a MapSet; change type container_type to container
Copy link
Owner

@RobertDober RobertDober left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I like this PR a lot, and we do not need implement the ideas I pitched, but here are some more

should we define all our types with ..._t ?

But I am fine to merge with the excpetion of the current backtix type alias to String.t

@bradhanks bradhanks requested a review from RobertDober March 19, 2025 20:15
@RobertDober RobertDober merged commit 24fe96f into RobertDober:master Mar 21, 2025
7 checks passed
Copy link
Owner

@RobertDober RobertDober left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanx, merged

@bradhanks bradhanks deleted the type_specs branch March 21, 2025 07:38
RobertDober pushed a commit that referenced this pull request Mar 24, 2025
 - Fix rendering of multiple images with reflinks

 - Type specification edits (#168)
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.

2 participants