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

Make diag dict field entry a state-accepting func #554

Merged
merged 1 commit into from
Nov 10, 2021
Merged

Make diag dict field entry a state-accepting func #554

merged 1 commit into from
Nov 10, 2021

Conversation

charleskawczynski
Copy link
Member

This is a peel-off from #473. Basically, this PR changes the diagnostics dict field entry from a reference to the input state to a function that accepts the state. As a result, io_dictionary_diagnostics,io_dictionary_aux, etc. do not need the input state, and the dict acts in a more functional way.

@charleskawczynski
Copy link
Member Author

bors r+

bors bot added a commit that referenced this pull request Nov 10, 2021
554: Make diag dict field entry a state-accepting func r=charleskawczynski a=charleskawczynski

This is a peel-off from #473. Basically, this PR changes the diagnostics dict field entry from a reference to the input state to a function that accepts the state. As a result, `io_dictionary_diagnostics`,`io_dictionary_aux`, etc. do not need the input state, and the dict acts in a more functional way.

Co-authored-by: Charles Kawczynski <kawczynski.charles@gmail.com>
@bors
Copy link
Contributor

bors bot commented Nov 10, 2021

Build failed:

@charleskawczynski
Copy link
Member Author

bors r+

@charleskawczynski
Copy link
Member Author

bors r-

@bors
Copy link
Contributor

bors bot commented Nov 10, 2021

Canceled.

@charleskawczynski
Copy link
Member Author

bors r+

@bors
Copy link
Contributor

bors bot commented Nov 10, 2021

Build succeeded:

@bors bors bot merged commit e5e0d86 into main Nov 10, 2021
@bors bors bot deleted the ck/io_dict branch November 10, 2021 19:39
ilopezgp referenced this pull request Nov 17, 2021
570: Bump version to 0.6.0 r=ilopezgp a=ilopezgp



Co-authored-by: ilopezgp <ilopezgp@gmail.com>
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.

1 participant