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

Infer array and dictionary static types from expected type during import #1038

Conversation

turbolent
Copy link
Member

@turbolent turbolent commented Jun 30, 2021

⚠️ Depends on #1042

Work towards #870

Description

When importing values, provide the expected type (e.g. the parameter type for a script argument) to the import function, and use it to determine the static type for array values and dictionary values.


  • Targeted PR against master branch
  • Linked to Github issue with discussion and accepted design OR link to spec that describes this work
  • Code follows the standards mentioned here
  • Updated relevant documentation
  • Re-reviewed Files changed in the Github PR explorer
  • Added appropriate labels

@turbolent turbolent requested a review from SupunS June 30, 2021 00:41
@turbolent turbolent self-assigned this Jun 30, 2021
Base automatically changed from bastian/array-static-type to feature/container-static-types June 30, 2021 15:44
@turbolent turbolent force-pushed the bastian/container-value-import-type-inference branch from 7ab11b0 to ff69f6a Compare June 30, 2021 18:14
@turbolent turbolent changed the base branch from feature/container-static-types to bastian/finish-storage-v5 June 30, 2021 20:58
@turbolent turbolent force-pushed the bastian/container-value-import-type-inference branch from ff69f6a to 66fc4e9 Compare June 30, 2021 21:00
Base automatically changed from bastian/finish-storage-v5 to feature/container-static-types July 2, 2021 04:12
@@ -92,13 +92,22 @@ func (t InterfaceStaticType) Equal(other StaticType) bool {
otherInterfaceType.QualifiedIdentifier == t.QualifiedIdentifier
}

// ArrayStaticType

type ArrayStaticType interface {
Copy link
Member

Choose a reason for hiding this comment

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

👌

@SupunS SupunS merged commit b561c08 into feature/container-static-types Jul 2, 2021
@SupunS SupunS deleted the bastian/container-value-import-type-inference branch July 2, 2021 04:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants