Skip to content
Snippets Groups Projects

Update internal state of the typer

Merged E. Rivas requested to merge er433/typer/context into dev
All threads resolved!

type:internal

The typer (src/pass/10-checking) keeps information around about the context of E_application in an ad-hoc way through variables last and args. This MR improves the representation by keeping an application context structure where things are pushed/popped.

Edited by E. Rivas

Merge request reports

Merge train pipeline #501636616 passed with warnings

Merge train pipeline passed with warnings for 1e913e96

Test coverage 47.92% (0.00%) from 1 job
Approved by

Merged by E. RivasE. Rivas 3 years ago (Mar 25, 2022 10:12pm UTC)

Merge details

  • Changes merged into dev with c37f3194 (commits were squashed).
  • Deleted the source branch.
  • Auto-merge enabled

Pipeline #501652603 passed with warnings

Pipeline: Ligo-Kubernetes-Infrastructure

#501659372

    Pipeline passed with warnings for c37f3194 on dev

    Test coverage 47.92% (0.00%) from 1 job

    Activity

    Filter activity
    • Approvals
    • Assignees & reviewers
    • Comments (from bots)
    • Comments (from users)
    • Commits & branches
    • Edits
    • Labels
    • Lock status
    • Mentions
    • Merge request status
    • Tracking
  • E. Rivas added 1 commit

    added 1 commit

    Compare with previous version

  • E. Rivas resolved all threads

    resolved all threads

  • Rémi
  • Rémi
  • E. Rivas added 1 commit

    added 1 commit

    Compare with previous version

  • E. Rivas added 1 commit

    added 1 commit

    • b50f99c4 - Move app_context to context.ml

    Compare with previous version

  • E. Rivas added 1 commit

    added 1 commit

    • cbd2f231 - Change from expression to type_expression in push/pop

    Compare with previous version

  • E. Rivas added 1 commit

    added 1 commit

    Compare with previous version

  • E. Rivas added 1 commit
  • E. Rivas added 1 commit

    added 1 commit

    • fda013d8 - Renaming app_context, typing_context, context

    Compare with previous version

  • E. Rivas added 1 commit

    added 1 commit

    • 3fd1eace - Abstract Typing_context inside Context

    Compare with previous version

  • E. Rivas resolved all threads

    resolved all threads

  • E. Rivas added 15 commits

    added 15 commits

    Compare with previous version

  • Rémi approved this merge request

    approved this merge request

  • E. Rivas enabled automatic add to merge train when the pipeline for ce2a91c9 succeeds

    enabled automatic add to merge train when the pipeline for ce2a91c9 succeeds

  • E. Rivas started a merge train

    started a merge train

  • merged

  • E. Rivas mentioned in commit c37f3194

    mentioned in commit c37f3194

  • unassigned @er433

  • Please register or sign in to reply
    Loading