Skip to content

Comments

Parse the package manifest directly for toolchains#1564

Merged
djkoloski merged 1 commit intogoogle:mainfrom
djkoloski:pin_syn_with_matrix_toolchain
Jul 30, 2024
Merged

Parse the package manifest directly for toolchains#1564
djkoloski merged 1 commit intogoogle:mainfrom
djkoloski:pin_syn_with_matrix_toolchain

Conversation

@djkoloski
Copy link
Member

This prevents the lockfile for the workspace from being generated by a newer version of cargo, which may choose more up-to-date dependencies than our msrv toolchain can compile.

@djkoloski djkoloski force-pushed the pin_syn_with_matrix_toolchain branch from ad2ff41 to bc877de Compare July 29, 2024 15:38
@codecov-commenter
Copy link

codecov-commenter commented Jul 29, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 87.70%. Comparing base (1ff83c4) to head (2c6f242).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1564   +/-   ##
=======================================
  Coverage   87.70%   87.70%           
=======================================
  Files          15       15           
  Lines        5563     5563           
=======================================
  Hits         4879     4879           
  Misses        684      684           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@djkoloski djkoloski force-pushed the pin_syn_with_matrix_toolchain branch 4 times, most recently from 97c5d2d to a367053 Compare July 30, 2024 17:23
@djkoloski djkoloski changed the title Pin syn dependency in CI with the matrix toolchain Parse the package manifest directly for toolchains Jul 30, 2024
This prevents the lockfile for the workspace from being generated by a
newer version of cargo, which may choose more up-to-date dependencies
than our msrv toolchain can compile.
@djkoloski djkoloski force-pushed the pin_syn_with_matrix_toolchain branch from a367053 to 2c6f242 Compare July 30, 2024 17:56
@djkoloski djkoloski marked this pull request as ready for review July 30, 2024 18:09
@djkoloski djkoloski added this pull request to the merge queue Jul 30, 2024
Merged via the queue into google:main with commit d656fbd Jul 30, 2024
@djkoloski djkoloski deleted the pin_syn_with_matrix_toolchain branch July 30, 2024 18:55
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.

3 participants