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

feat: support update_forward_refs in scripts #1757

Closed

Conversation

PrettyWood
Copy link
Collaborator

@PrettyWood PrettyWood commented Jul 23, 2020

Change Summary

Currently update_forward_refs works only for modules. This adds a default behaviour for scripts

⚠️ It's probably better to first check #1686 as it probably covers also this case ⚠️

Related issue number

closes #1734

Checklist

  • Unit tests for the changes exist
  • Tests pass on CI and coverage remains at 100%
  • Documentation reflects the changes where applicable
  • changes/<pull request or issue id>-<github username>.md file added describing change
    (see changes/README.md for details)

@codecov
Copy link

codecov bot commented Jul 23, 2020

Codecov Report

Merging #1757 into master will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##            master     #1757   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           21        21           
  Lines         3895      3895           
  Branches       783       783           
=========================================
  Hits          3895      3895           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e985857...dc50258. Read the comment docs.

@remidebette
Copy link

Hi!

Anything blocking this?
I do not know the process of PR validation in the pydantic project.

Thanks

@samuelcolvin
Copy link
Member

I think best to wait for #1686, if it doesn't get merged we can use this.

@samuelcolvin
Copy link
Member

#1686 is now merged, could this be closed?

@PrettyWood
Copy link
Collaborator Author

Yes that was the plan thanks

@PrettyWood PrettyWood closed this Oct 19, 2020
@PrettyWood PrettyWood deleted the fix-forward-ref-script branch October 19, 2020 14:52
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.

Dynamically create_model that contains forward refs
3 participants