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: add SecretStr and SecretBytes. #452

Merged
merged 9 commits into from Apr 4, 2019

Conversation

@Atheuz
Copy link
Contributor

@Atheuz Atheuz commented Apr 2, 2019

Change Summary

Added SecretStr and SecretBytes.

Related issue number

#443

Checklist

  • Unit tests for the changes exist
  • Tests pass on CI and coverage remains at 100%
  • Documentation reflects the changes where applicable
  • HISTORY.rst has been updated
@Atheuz
Copy link
Contributor Author

@Atheuz Atheuz commented Apr 2, 2019

@samuelcolvin is this better?

Loading

@samuelcolvin
Copy link
Owner

@samuelcolvin samuelcolvin commented Apr 2, 2019

Looks like you've changed the permissions on lots of files 100644 → 100755. 😈

Maybe you could set back to 644?

Loading

@codecov
Copy link

@codecov codecov bot commented Apr 2, 2019

Codecov Report

Merging #452 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master   #452   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files          14     14           
  Lines        2188   2220   +32     
  Branches      435    435           
=====================================
+ Hits         2188   2220   +32

Loading

docs/examples/ex_secret_types.py Show resolved Hide resolved
Loading
pydantic/schema.py Outdated Show resolved Hide resolved
Loading
pydantic/types.py Outdated Show resolved Hide resolved
Loading
pydantic/types.py Outdated Show resolved Hide resolved
Loading
pydantic/types.py Outdated Show resolved Hide resolved
Loading
pydantic/types.py Outdated Show resolved Hide resolved
Loading
tests/test_schema.py Outdated Show resolved Hide resolved
Loading
tests/test_schema.py Outdated Show resolved Hide resolved
Loading
tests/test_types.py Outdated Show resolved Hide resolved
Loading
@Atheuz Atheuz force-pushed the add-secret-types2 branch from e485d56 to d7845a4 Apr 2, 2019
@Atheuz
Copy link
Contributor Author

@Atheuz Atheuz commented Apr 2, 2019

@samuelcolvin fixed the issues you brought up, but didn't know about the format thing you mentioned.

Loading

@Atheuz Atheuz force-pushed the add-secret-types2 branch 2 times, most recently from af54f96 to fec9a1e Apr 2, 2019
@Atheuz Atheuz force-pushed the add-secret-types2 branch from fec9a1e to 6372516 Apr 2, 2019
@LasseGravesen
Copy link
Contributor

@LasseGravesen LasseGravesen commented Apr 3, 2019

@samuelcolvin are we mergable yet?

Loading

@samuelcolvin
Copy link
Owner

@samuelcolvin samuelcolvin commented Apr 3, 2019

Just waiting for a response from @tiangolo, on the question above, I've done some research but not entirely clear.

Is this particularly urgent for you?

Loading

@tiangolo
Copy link
Collaborator

@tiangolo tiangolo commented Apr 3, 2019

Sorry for the delay guys. I hadn't seen the notification. I added my comment in the conversation above.

Loading

@samuelcolvin
Copy link
Owner

@samuelcolvin samuelcolvin commented Apr 3, 2019

thanks @tiangolo, much appreciated.

No problem on the slight delay, still better than the response time for most paid services. :-)

Loading

Copy link
Collaborator

@tiangolo tiangolo left a comment

Great! This looks very useful.

I added a couple of suggestions related to JSON Schema.

Loading

pydantic/schema.py Outdated Show resolved Hide resolved
Loading
pydantic/schema.py Outdated Show resolved Hide resolved
Loading
docs/schema_mapping.py Outdated Show resolved Hide resolved
Loading
docs/schema_mapping.py Outdated Show resolved Hide resolved
Loading
Copy link
Collaborator

@tiangolo tiangolo left a comment

Just a couple of things I forgot to mention in docs.

Loading

docs/schema_mapping.py Outdated Show resolved Hide resolved
Loading
docs/schema_mapping.py Outdated Show resolved Hide resolved
Loading
Copy link
Collaborator

@tiangolo tiangolo left a comment

Great! LGTM

Loading

@samuelcolvin samuelcolvin merged commit 4a8faca into samuelcolvin:master Apr 4, 2019
7 checks passed
Loading
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants