Skip to content

Conversation

@keyboardDrummer
Copy link
Member

@keyboardDrummer keyboardDrummer commented Jun 4, 2025

What was changed?

  • Stop depending on source location hierarchy for verification. There are several IDE related features that still depend on source location hierarchy, but that is mostly important when working with Dafny source code, so that's OK for downstream tools that compile to Dafny

How has this been tested?

  • Had to update some existing tests since now the location based API requires targeting the name of a symbol, not just any location inside the symbol.

By submitting this pull request, I confirm that my contribution is made under the terms of the MIT license.

@keyboardDrummer keyboardDrummer marked this pull request as ready for review June 4, 2025 12:11
@keyboardDrummer keyboardDrummer requested a review from robin-aws June 4, 2025 12:12
@keyboardDrummer keyboardDrummer enabled auto-merge (squash) June 4, 2025 12:57
Copy link
Member

@robin-aws robin-aws left a comment

Choose a reason for hiding this comment

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

🚀         🚀 
     🚀 
  🚀                    🚀 
             🚀 
🚀 

@keyboardDrummer keyboardDrummer merged commit 702ff27 into dafny-lang:master Jun 4, 2025
22 checks passed
@keyboardDrummer keyboardDrummer deleted the doNotUseSourceLocationHierarchyForVerification branch June 4, 2025 19:46
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