Skip to content

test(metadata): Show behavior with TOML-specific types - #13703

Merged
bors merged 1 commit into
rust-lang:masterfrom
epage:toml
Apr 4, 2024
Merged

test(metadata): Show behavior with TOML-specific types#13703
bors merged 1 commit into
rust-lang:masterfrom
epage:toml

Conversation

@epage

@epage epage commented Apr 4, 2024

Copy link
Copy Markdown
Contributor

What does this PR try to resolve?

The goal is to know if/when these change so we can have a conversation about what to do.

How should we test and review this PR?

Additional information

The goal is to know if/when these change so we can have a conversation
about what to do.
@rustbot

rustbot commented Apr 4, 2024

Copy link
Copy Markdown
Collaborator

r? @ehuss

rustbot has assigned @ehuss.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Apr 4, 2024

@weihanglo weihanglo left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

How was this dicovered and do we have any prior discussion of it?

@epage

epage commented Apr 4, 2024

Copy link
Copy Markdown
Contributor Author

How was this dicovered and do we have any prior discussion of it?

I discovered it when I merged tomls functionality into toml_edit. I've previously glossed over this. In office hours, this came up for a couple of reasons

  • Caching of Cargo.toml parsing in alternative formats which could end up persisting this
  • Being prepared for a day when a cargo team member isn't maintaining toml

@arlosi

arlosi commented Apr 4, 2024

Copy link
Copy Markdown
Contributor

@bors r+

@bors

bors commented Apr 4, 2024

Copy link
Copy Markdown
Contributor

📌 Commit b930f7a has been approved by arlosi

It is now in the queue for this repository.

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Apr 4, 2024
@weihanglo

Copy link
Copy Markdown
Member

Being prepared for a day when a cargo team member isn't maintaining toml

I hope everything is okay 🙏🏾.

@bors

bors commented Apr 4, 2024

Copy link
Copy Markdown
Contributor

⌛ Testing commit b930f7a with merge a9a0746...

@epage

epage commented Apr 4, 2024

Copy link
Copy Markdown
Contributor Author

No announcements, more being prepared

@bors

bors commented Apr 4, 2024

Copy link
Copy Markdown
Contributor

☀️ Test successful - checks-actions
Approved by: arlosi
Pushing a9a0746 to master...

@bors
bors merged commit a9a0746 into rust-lang:master Apr 4, 2024
@epage
epage deleted the toml branch April 5, 2024 00:59
bors added a commit to rust-lang-ci/rust that referenced this pull request Apr 6, 2024
Update cargo

9 commits in 0637083df5bbdcc951845f0d2eff6999cdb6d30a..28e7b2bc0a812f90126be30f48a00a4ada990eaa
2024-04-02 23:55:05 +0000 to 2024-04-05 19:31:01 +0000
- refactor(toml): Decouple target discovery from Target creation (rust-lang/cargo#13701)
- Don't depend on `?` affecting type inference in weird ways (rust-lang/cargo#13706)
- test(metadata): Show behavior with TOML-specific types (rust-lang/cargo#13703)
- fix: adjust tracing verbosity in list_files_git (rust-lang/cargo#13704)
- doc: comments on `PackageRegistry` (rust-lang/cargo#13698)
- Switch to using gitoxide by default for listing files (rust-lang/cargo#13696)
- Allow precise update to prerelease. (rust-lang/cargo#13626)
- refactor(toml): Split out an explicit step to resolve `Cargo.toml` (rust-lang/cargo#13693)
- chore(deps): update rust crate base64 to 0.22.0 (rust-lang/cargo#13675)

r? ghost
@rustbot rustbot added this to the 1.79.0 milestone Apr 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants