Skip to content

Fix dialyzer errors#334

Merged
axelson merged 1 commit into
elixir-lsp:masterfrom
msaraiva:ms-fix-dialyzer-errors
Jul 26, 2020
Merged

Fix dialyzer errors#334
axelson merged 1 commit into
elixir-lsp:masterfrom
msaraiva:ms-fix-dialyzer-errors

Conversation

@msaraiva

Copy link
Copy Markdown
Collaborator

No description provided.

@axelson axelson left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

❤️

@axelson
axelson merged commit e34d0aa into elixir-lsp:master Jul 26, 2020
lukaszsamson added a commit that referenced this pull request Jul 4, 2026
…nts facade boundary

- Release gate: the absolute-path-dep check no longer has
  continue-on-error — the path dependency was replaced by a git pin, so a
  reintroduction must fail the workflow, not just show a red job. Stale
  development-phase comments updated.
- Pin elixir_sense to PR #334 head 702ff31a (Descr funnel delegates,
  property harness, short negative ExCk cache TTL, adaptor boundary test,
  TYPES_ROADMAP architecture contract).
- Add inlay_hints_boundary_test.exs: the InlayHints provider must consume
  types only through the ElixirSense.Core.TypeHints facade — direct calls to
  Binding or TypePresentation fail the test (checked against compiled BEAM
  import tables).

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01NvwMWQayfdxowBDrnt5iYM
lukaszsamson added a commit that referenced this pull request Jul 4, 2026
… branch

Pin conflict resolved keeping ours: master pins elixir_sense 9460c606, which
is an ancestor of our 9788101d (the #334 merge commit on elixir_sense
master). Gates: compile 0 warnings, format clean, language_server suite
1624 passed.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01NvwMWQayfdxowBDrnt5iYM
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