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

Remove rustversion dev-dependency #568

Merged
merged 3 commits into from
Jul 3, 2023

Conversation

Thomasdezeeuw
Copy link
Collaborator

Since the MSRV is now 1.60.

Since the MSRV is now 1.60.
When kv_unstable is not enabled.
@Thomasdezeeuw
Copy link
Collaborator Author

The minimal versions issue should be resolved by updating to proc-macro2 v1.0.60 (specifically in dtolnay/proc-macro2@e31d619). But I think this has to happen in serde_derive?

@Thomasdezeeuw
Copy link
Collaborator Author

The minimal versions issue should be resolved by updating to proc-macro2 v1.0.60 (specifically in dtolnay/proc-macro2@e31d619). But I think this has to happen in serde_derive?

Oh both serde_derive and sval_derive define proc-macro2 = "1.0" (https://github.com/serde-rs/serde/blob/015e39776fcd951d208d7fbb5f4a4de4efb4445f/serde_derive/Cargo.toml#L25 and https://github.com/sval-rs/sval/blob/3cbda0f19ac37260b44f16233944529f04666e05/derive/Cargo.toml#L25), so this is going to be a bit annoying.

@KodrAus
Copy link
Contributor

KodrAus commented Jul 3, 2023

Ah at first I was reading minimal versions as MSRV and wasn’t following what you meant. Now I get it.

I’m happy to add minimal versions to CI in sval and bump proc-macro2, but it might be easier for us to simply remove those derive dependencies in log. It might get a little tedious, depending on where we use them though.

@KodrAus
Copy link
Contributor

KodrAus commented Jul 3, 2023

Alternatively we could add proc-macro2 as a dev dependency with a comment on why it’s there. That might be the least painful.

@Thomasdezeeuw
Copy link
Collaborator Author

Alternatively we could add proc-macro2 as a dev dependency with a comment on why it’s there. That might be the least painful.

I've went with this option as it's the easiest, even though it's a little hacky.

Done to make the minimal (crate) version CI happy.
@KodrAus
Copy link
Contributor

KodrAus commented Jul 3, 2023

I’ll run through later and see if I can yank the derive crates out as dependencies altogether.

Copy link
Contributor

@KodrAus KodrAus left a comment

Choose a reason for hiding this comment

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

This looks good to me!

@KodrAus KodrAus merged commit e9123d6 into rust-lang:master Jul 3, 2023
@Thomasdezeeuw Thomasdezeeuw deleted the remove-rustversion-dep branch July 4, 2023 21:35
renovate bot referenced this pull request in ziyadedher/evm-bench Aug 12, 2023
[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Type | Update | Change |
|---|---|---|---|
| [log](https://togithub.com/rust-lang/log) | dependencies | patch |
`0.4.19` -> `0.4.20` |

---

### Release Notes

<details>
<summary>rust-lang/log (log)</summary>

###
[`v0.4.20`](https://togithub.com/rust-lang/log/blob/HEAD/CHANGELOG.md#0420---2023-07-11)

[Compare
Source](https://togithub.com/rust-lang/log/compare/0.4.19...0.4.20)

- Remove rustversion dev-dependency by
[@&#8203;Thomasdezeeuw](https://togithub.com/Thomasdezeeuw) in
[https://github.com/rust-lang/log/pull/568](https://togithub.com/rust-lang/log/pull/568)
- Remove `local_inner_macros` usage by
[@&#8203;EFanZh](https://togithub.com/EFanZh) in
[https://github.com/rust-lang/log/pull/570](https://togithub.com/rust-lang/log/pull/570)

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Enabled.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://developer.mend.io/github/ziyadedher/evm-bench).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNi40MC4zIiwidXBkYXRlZEluVmVyIjoiMzYuNDAuMyIsInRhcmdldEJyYW5jaCI6Im1haW4ifQ==-->

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
github-merge-queue bot referenced this pull request in knope-dev/knope Aug 12, 2023
[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Type | Update | Change |
|---|---|---|---|
| [log](https://togithub.com/rust-lang/log) | dependencies | patch |
`0.4.19` -> `0.4.20` |

---

### Release Notes

<details>
<summary>rust-lang/log (log)</summary>

###
[`v0.4.20`](https://togithub.com/rust-lang/log/blob/HEAD/CHANGELOG.md#0420---2023-07-11)

[Compare
Source](https://togithub.com/rust-lang/log/compare/0.4.19...0.4.20)

- Remove rustversion dev-dependency by
[@&#8203;Thomasdezeeuw](https://togithub.com/Thomasdezeeuw) in
[https://github.com/rust-lang/log/pull/568](https://togithub.com/rust-lang/log/pull/568)
- Remove `local_inner_macros` usage by
[@&#8203;EFanZh](https://togithub.com/EFanZh) in
[https://github.com/rust-lang/log/pull/570](https://togithub.com/rust-lang/log/pull/570)

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Enabled.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://developer.mend.io/github/knope-dev/knope).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNi40MC4zIiwidXBkYXRlZEluVmVyIjoiMzYuNDAuMyIsInRhcmdldEJyYW5jaCI6Im1haW4ifQ==-->

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
kbudde referenced this pull request in kbudde/bench-metrics Aug 12, 2023
[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Type | Update | Change |
|---|---|---|---|
| [log](https://togithub.com/rust-lang/log) | dependencies | patch |
`0.4.19` -> `0.4.20` |

---

### Release Notes

<details>
<summary>rust-lang/log (log)</summary>

###
[`v0.4.20`](https://togithub.com/rust-lang/log/blob/HEAD/CHANGELOG.md#0420---2023-07-11)

[Compare
Source](https://togithub.com/rust-lang/log/compare/0.4.19...0.4.20)

- Remove rustversion dev-dependency by
[@&#8203;Thomasdezeeuw](https://togithub.com/Thomasdezeeuw) in
[https://github.com/rust-lang/log/pull/568](https://togithub.com/rust-lang/log/pull/568)
- Remove `local_inner_macros` usage by
[@&#8203;EFanZh](https://togithub.com/EFanZh) in
[https://github.com/rust-lang/log/pull/570](https://togithub.com/rust-lang/log/pull/570)

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you
are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://developer.mend.io/github/kbudde/bench-metrics).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNi40MC4zIiwidXBkYXRlZEluVmVyIjoiMzYuNDAuMyIsInRhcmdldEJyYW5jaCI6Im1haW4ifQ==-->

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
fdkevin0 referenced this pull request in fdkevin0/fdkevin-cf-bot Aug 18, 2023
[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Type | Update | Change |
|---|---|---|---|
| [log](https://togithub.com/rust-lang/log) | dependencies | patch |
`0.4.17` -> `0.4.20` |

---

### Release Notes

<details>
<summary>rust-lang/log (log)</summary>

###
[`v0.4.20`](https://togithub.com/rust-lang/log/blob/HEAD/CHANGELOG.md#0420---2023-07-11)

[Compare
Source](https://togithub.com/rust-lang/log/compare/0.4.19...0.4.20)

- Remove rustversion dev-dependency by
[@&#8203;Thomasdezeeuw](https://togithub.com/Thomasdezeeuw) in
[https://github.com/rust-lang/log/pull/568](https://togithub.com/rust-lang/log/pull/568)
- Remove `local_inner_macros` usage by
[@&#8203;EFanZh](https://togithub.com/EFanZh) in
[https://github.com/rust-lang/log/pull/570](https://togithub.com/rust-lang/log/pull/570)

###
[`v0.4.19`](https://togithub.com/rust-lang/log/blob/HEAD/CHANGELOG.md#0419---2023-06-10)

[Compare
Source](https://togithub.com/rust-lang/log/compare/0.4.18...0.4.19)

- Use target_has_atomic instead of the old atomic_cas cfg by
[@&#8203;GuillaumeGomez](https://togithub.com/GuillaumeGomez) in
[https://github.com/rust-lang/log/pull/555](https://togithub.com/rust-lang/log/pull/555)
- Put MSRV into Cargo.toml by
[@&#8203;est31](https://togithub.com/est31) in
[https://github.com/rust-lang/log/pull/557](https://togithub.com/rust-lang/log/pull/557)

###
[`v0.4.18`](https://togithub.com/rust-lang/log/blob/HEAD/CHANGELOG.md#0418---2023-05-28)

[Compare
Source](https://togithub.com/rust-lang/log/compare/0.4.17...0.4.18)

- fix markdown links (again) by
[@&#8203;hellow554](https://togithub.com/hellow554) in
[https://github.com/rust-lang/log/pull/513](https://togithub.com/rust-lang/log/pull/513)
- add cargo doc to workflow by
[@&#8203;hellow554](https://togithub.com/hellow554) in
[https://github.com/rust-lang/log/pull/515](https://togithub.com/rust-lang/log/pull/515)
- Apply Clippy lints by
[@&#8203;hellow554](https://togithub.com/hellow554) in
[https://github.com/rust-lang/log/pull/516](https://togithub.com/rust-lang/log/pull/516)
- Replace ad-hoc eq_ignore_ascii_case with slice::eq_ignore_ascii_case
by [@&#8203;glandium](https://togithub.com/glandium) in
[https://github.com/rust-lang/log/pull/519](https://togithub.com/rust-lang/log/pull/519)
- fix up windows targets by
[@&#8203;KodrAus](https://togithub.com/KodrAus) in
[https://github.com/rust-lang/log/pull/528](https://togithub.com/rust-lang/log/pull/528)
- typo fix by [@&#8203;jiangying000](https://togithub.com/jiangying000)
in
[https://github.com/rust-lang/log/pull/529](https://togithub.com/rust-lang/log/pull/529)
- Remove dependency on cfg_if by
[@&#8203;EriKWDev](https://togithub.com/EriKWDev) in
[https://github.com/rust-lang/log/pull/536](https://togithub.com/rust-lang/log/pull/536)
- GitHub Workflows security hardening by
[@&#8203;sashashura](https://togithub.com/sashashura) in
[https://github.com/rust-lang/log/pull/538](https://togithub.com/rust-lang/log/pull/538)
- Fix build status badge by
[@&#8203;atouchet](https://togithub.com/atouchet) in
[https://github.com/rust-lang/log/pull/539](https://togithub.com/rust-lang/log/pull/539)
- Add call_logger to the documentation by
[@&#8203;a1ecbr0wn](https://togithub.com/a1ecbr0wn) in
[https://github.com/rust-lang/log/pull/547](https://togithub.com/rust-lang/log/pull/547)
- Use stable internals for key-value API by
[@&#8203;KodrAus](https://togithub.com/KodrAus) in
[https://github.com/rust-lang/log/pull/550](https://togithub.com/rust-lang/log/pull/550)
- Change wording of list of implementations by
[@&#8203;Thomasdezeeuw](https://togithub.com/Thomasdezeeuw) in
[https://github.com/rust-lang/log/pull/553](https://togithub.com/rust-lang/log/pull/553)
- Add std-logger to list of implementations by
[@&#8203;Thomasdezeeuw](https://togithub.com/Thomasdezeeuw) in
[https://github.com/rust-lang/log/pull/554](https://togithub.com/rust-lang/log/pull/554)
- Add `set_max_level_racy` and gate `set_max_level` by
[@&#8203;djkoloski](https://togithub.com/djkoloski) in
[https://github.com/rust-lang/log/pull/544](https://togithub.com/rust-lang/log/pull/544)
- \[doc] src/lib.rs : prefix an unused variable with an underscore by
[@&#8203;OccupyMars2025](https://togithub.com/OccupyMars2025) in
[https://github.com/rust-lang/log/pull/561](https://togithub.com/rust-lang/log/pull/561)
- \[doc] src/macros.rs : correct grammar errors of an example in lib
documentation by
[@&#8203;OccupyMars2025](https://togithub.com/OccupyMars2025) in
[https://github.com/rust-lang/log/pull/562](https://togithub.com/rust-lang/log/pull/562)

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you
are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://developer.mend.io/github/fdkevin0/fdkevin-cf-bot).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNS4xMDIuMTAiLCJ1cGRhdGVkSW5WZXIiOiIzNi40MC4zIiwidGFyZ2V0QnJhbmNoIjoibWFpbiJ9-->

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
kodiakhq bot referenced this pull request in X-oss-byte/Nextjs Sep 22, 2023
[![Mend Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Type | Update | Change |
|---|---|---|---|
| [log](https://togithub.com/rust-lang/log) | workspace.dependencies | patch | `0.4.17` -> `0.4.20` |

---

### Release Notes

<details>
<summary>rust-lang/log (log)</summary>

### [`v0.4.20`](https://togithub.com/rust-lang/log/blob/HEAD/CHANGELOG.md#0420---2023-07-11)

[Compare Source](https://togithub.com/rust-lang/log/compare/0.4.19...0.4.20)

-   Remove rustversion dev-dependency by [@&#8203;Thomasdezeeuw](https://togithub.com/Thomasdezeeuw) in [https://github.com/rust-lang/log/pull/568](https://togithub.com/rust-lang/log/pull/568)
-   Remove `local_inner_macros` usage by [@&#8203;EFanZh](https://togithub.com/EFanZh) in [https://github.com/rust-lang/log/pull/570](https://togithub.com/rust-lang/log/pull/570)

### [`v0.4.19`](https://togithub.com/rust-lang/log/blob/HEAD/CHANGELOG.md#0419---2023-06-10)

[Compare Source](https://togithub.com/rust-lang/log/compare/0.4.18...0.4.19)

-   Use target_has_atomic instead of the old atomic_cas cfg by [@&#8203;GuillaumeGomez](https://togithub.com/GuillaumeGomez) in [https://github.com/rust-lang/log/pull/555](https://togithub.com/rust-lang/log/pull/555)
-   Put MSRV into Cargo.toml by [@&#8203;est31](https://togithub.com/est31) in [https://github.com/rust-lang/log/pull/557](https://togithub.com/rust-lang/log/pull/557)

### [`v0.4.18`](https://togithub.com/rust-lang/log/blob/HEAD/CHANGELOG.md#0418---2023-05-28)

[Compare Source](https://togithub.com/rust-lang/log/compare/0.4.17...0.4.18)

-   fix markdown links (again) by [@&#8203;hellow554](https://togithub.com/hellow554) in [https://github.com/rust-lang/log/pull/513](https://togithub.com/rust-lang/log/pull/513)
-   add cargo doc to workflow by [@&#8203;hellow554](https://togithub.com/hellow554) in [https://github.com/rust-lang/log/pull/515](https://togithub.com/rust-lang/log/pull/515)
-   Apply Clippy lints by [@&#8203;hellow554](https://togithub.com/hellow554) in [https://github.com/rust-lang/log/pull/516](https://togithub.com/rust-lang/log/pull/516)
-   Replace ad-hoc eq_ignore_ascii_case with slice::eq_ignore_ascii_case by [@&#8203;glandium](https://togithub.com/glandium) in [https://github.com/rust-lang/log/pull/519](https://togithub.com/rust-lang/log/pull/519)
-   fix up windows targets by [@&#8203;KodrAus](https://togithub.com/KodrAus) in [https://github.com/rust-lang/log/pull/528](https://togithub.com/rust-lang/log/pull/528)
-   typo fix by [@&#8203;jiangying000](https://togithub.com/jiangying000) in [https://github.com/rust-lang/log/pull/529](https://togithub.com/rust-lang/log/pull/529)
-   Remove dependency on cfg_if by [@&#8203;EriKWDev](https://togithub.com/EriKWDev) in [https://github.com/rust-lang/log/pull/536](https://togithub.com/rust-lang/log/pull/536)
-   GitHub Workflows security hardening by [@&#8203;sashashura](https://togithub.com/sashashura) in [https://github.com/rust-lang/log/pull/538](https://togithub.com/rust-lang/log/pull/538)
-   Fix build status badge by [@&#8203;atouchet](https://togithub.com/atouchet) in [https://github.com/rust-lang/log/pull/539](https://togithub.com/rust-lang/log/pull/539)
-   Add call_logger to the documentation by [@&#8203;a1ecbr0wn](https://togithub.com/a1ecbr0wn) in [https://github.com/rust-lang/log/pull/547](https://togithub.com/rust-lang/log/pull/547)
-   Use stable internals for key-value API by [@&#8203;KodrAus](https://togithub.com/KodrAus) in [https://github.com/rust-lang/log/pull/550](https://togithub.com/rust-lang/log/pull/550)
-   Change wording of list of implementations by [@&#8203;Thomasdezeeuw](https://togithub.com/Thomasdezeeuw) in [https://github.com/rust-lang/log/pull/553](https://togithub.com/rust-lang/log/pull/553)
-   Add std-logger to list of implementations by [@&#8203;Thomasdezeeuw](https://togithub.com/Thomasdezeeuw) in [https://github.com/rust-lang/log/pull/554](https://togithub.com/rust-lang/log/pull/554)
-   Add `set_max_level_racy` and gate `set_max_level` by [@&#8203;djkoloski](https://togithub.com/djkoloski) in [https://github.com/rust-lang/log/pull/544](https://togithub.com/rust-lang/log/pull/544)
-   \[doc] src/lib.rs : prefix an unused variable with an underscore by [@&#8203;OccupyMars2025](https://togithub.com/OccupyMars2025) in [https://github.com/rust-lang/log/pull/561](https://togithub.com/rust-lang/log/pull/561)
-   \[doc] src/macros.rs :  correct  grammar errors of an example in lib documentation by [@&#8203;OccupyMars2025](https://togithub.com/OccupyMars2025) in [https://github.com/rust-lang/log/pull/562](https://togithub.com/rust-lang/log/pull/562)

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update again.

---

 - [ ] If you want to rebase/retry this PR, check this box

---

This PR has been generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View repository job log [here](https://developer.mend.io/github/sammyfilly/Nextjs).
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.

2 participants