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

Allow manual_unwrap_or_default in all generated impls #296

Merged
merged 1 commit into from
Jul 9, 2024

Conversation

TedDriggs
Copy link
Owner

No description provided.

@TedDriggs TedDriggs merged commit 6a057da into master Jul 9, 2024
24 checks passed
@TedDriggs TedDriggs deleted the disable-manual-unwrap-or-default branch July 9, 2024 12:35
martin-g added a commit to apache/avro that referenced this pull request Jul 12, 2024
…ing dependency

Introduced-with: 2c484c6
Fixed upstream with: TedDriggs/darling#296

Signed-off-by: Martin Tzvetanov Grigorov <mgrigorov@apache.org>
martin-g added a commit to apache/avro that referenced this pull request Jul 12, 2024
…ing dependency (#3018)

Introduced-with: 2c484c6
Fixed upstream with: TedDriggs/darling#296

Signed-off-by: Martin Tzvetanov Grigorov <mgrigorov@apache.org>
martin-g added a commit to apache/avro that referenced this pull request Jul 12, 2024
…ing dependency (#3018)

Introduced-with: 2c484c6
Fixed upstream with: TedDriggs/darling#296

Signed-off-by: Martin Tzvetanov Grigorov <mgrigorov@apache.org>
(cherry picked from commit 69cd998)
andrei-ng added a commit to plotly/plotly.rs that referenced this pull request Jul 31, 2024
 - temporary patch introduced in
   3130f5a to bypass clippy errors
   due to darling package: TedDriggs/darling#296

Closes #210

Signed-off-by: Andrei Gherghescu <8067229+andrei-ng@users.noreply.github.com>
andrei-ng added a commit to plotly/plotly.rs that referenced this pull request Jul 31, 2024
* update contributing file with new contact details

Signed-off-by: Andrei Gherghescu <8067229+andrei-ng@users.noreply.github.com>

* provide alternative show_html method

 - the show() method by default creates a random named file in `/tmp`
   dir, however in some cases `/tmp` is not available, e.g., snap's Firefox
   runs in an isolated enviroment and cannot access the system's `/tmp`
 - provide a method that calls write_html followed by show for easier
   access

Closes #170

Signed-off-by: Andrei Gherghescu <8067229+andrei-ng@users.noreply.github.com>

* fix clippy & fmt warnings

Signed-off-by: Andrei Gherghescu <8067229+andrei-ng@users.noreply.github.com>

* revert CI clippy darling package command patch

 - temporary patch introduced in
   3130f5a to bypass clippy errors
   due to darling package: TedDriggs/darling#296

Closes #210

Signed-off-by: Andrei Gherghescu <8067229+andrei-ng@users.noreply.github.com>

---------

Signed-off-by: Andrei Gherghescu <8067229+andrei-ng@users.noreply.github.com>
martin-g added a commit to apache/avro-rs that referenced this pull request Sep 23, 2024
…ing dependency (#3018)

Introduced-with: apache/avro@2c484c6
Fixed upstream with: TedDriggs/darling#296

Signed-off-by: Martin Tzvetanov Grigorov <mgrigorov@apache.org>
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.

1 participant