Skip to content

Add missing arg in TOML printvalue #57584

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

Merged
merged 4 commits into from
Mar 5, 2025
Merged

Conversation

qiaojunfeng
Copy link
Contributor

Otherwise error when calling print_inline_table

Otherwise error when calling `print_inline_table`
@KristofferC KristofferC added backport 1.10 Change should be backported to the 1.10 release backport 1.11 Change should be backported to release-1.11 backport 1.12 Change should be backported to release-1.12 TOML labels Feb 28, 2025
@IanButterworth IanButterworth added the needs tests Unit tests are required for this change label Feb 28, 2025
@IanButterworth
Copy link
Member

A test would be good

@IanButterworth IanButterworth removed the needs tests Unit tests are required for this change label Feb 28, 2025
@invokelatest print_inline_table(f, io, a)
@invokelatest print_inline_table(f, io, a, sorted)
Copy link
Member

Choose a reason for hiding this comment

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

I tested locally and this fix isn't covered by tests.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

OK added another test for this

@IanButterworth IanButterworth added the merge me PR is reviewed. Merge when all tests are passing label Mar 5, 2025
@giordano giordano merged commit f207224 into JuliaLang:master Mar 5, 2025
8 checks passed
@giordano giordano removed the merge me PR is reviewed. Merge when all tests are passing label Mar 5, 2025
@qiaojunfeng qiaojunfeng deleted the fix_toml branch March 5, 2025 10:53
@nsajko nsajko added the bugfix This change fixes an existing bug label Mar 5, 2025
KristofferC pushed a commit that referenced this pull request Mar 11, 2025
Otherwise error when calling `print_inline_table`

(cherry picked from commit f207224)
This was referenced Mar 11, 2025
KristofferC pushed a commit that referenced this pull request Mar 20, 2025
Otherwise error when calling `print_inline_table`

(cherry picked from commit f207224)
@KristofferC KristofferC removed the backport 1.12 Change should be backported to release-1.12 label Mar 24, 2025
KristofferC pushed a commit that referenced this pull request Mar 25, 2025
Otherwise error when calling `print_inline_table`

(cherry picked from commit f207224)
qiaojunfeng added a commit to qiaojunfeng/WannierIO.jl that referenced this pull request Mar 28, 2025
qiaojunfeng added a commit to qiaojunfeng/WannierIO.jl that referenced this pull request Mar 28, 2025
qiaojunfeng added a commit to qiaojunfeng/WannierIO.jl that referenced this pull request Mar 28, 2025
@KristofferC KristofferC removed the backport 1.11 Change should be backported to release-1.11 label Apr 10, 2025
@KristofferC KristofferC removed the backport 1.10 Change should be backported to the 1.10 release label Jun 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugfix This change fixes an existing bug TOML
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants