Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: damf context elements from local imports #150

Closed
wants to merge 3 commits into from

Conversation

innofarah
Copy link

relevant damf context elements (Kind, Type, Define, etc..) were not being considered and added to the compiled damf formulas contexts if they're introduced in locally imported files (for example, when we import "ccs_sig", all damf context elements in it were not being considered)

if a theorem uses a lemma that was locally imported (not previously existing as damf assertion, neither in the same file; but imported from a local file), this lemma was not being tracked as a damf dependency in the published theorem of the main file
relevant damf context elements (Kind, Type, Define, etc..) were not being considered and added to the compiled damf formulas contexts if they're introduced in locally imported files (for example, when we import "ccs_sig", all damf context elements in it were not being considered)
@chaudhuri
Copy link
Member

I am not sure this is correct. CImport will pull in the transitive dependencies.

@chaudhuri
Copy link
Member

Fixed in c7285e0

@chaudhuri chaudhuri closed this Nov 17, 2023
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