Skip to content
This repository has been archived by the owner on May 22, 2024. It is now read-only.

feat: set default Rust toolchain #715

Merged
merged 4 commits into from
Oct 11, 2021
Merged

feat: set default Rust toolchain #715

merged 4 commits into from
Oct 11, 2021

Conversation

eduardoboucas
Copy link
Member

@eduardoboucas eduardoboucas commented Oct 8, 2021

- Summary

Before installing the Rust build target, sets the default toolchain to stable via the rustup default command. Because this is the one with less precedence out of all the override methods, users still have the option to specify a different toolchain using any of the other four methods.

- Test plan

Adjusted existing test.

- A picture of a cute animal (not mandatory but encouraged)

maxresdefault

@eduardoboucas eduardoboucas added the type: feature code contributing to the implementation of a feature and/or user facing functionality label Oct 8, 2021
@github-actions
Copy link
Contributor

github-actions bot commented Oct 8, 2021

⏱ Benchmark results

Comparing with 67961ee

largeDepsEsbuild: 12.9s

⬇️ 0.73% decrease vs. 67961ee

^                          14.6s                                                                          
│                           ┌──┐                                                                          
│          13.1s            |  |           12.9s                                                   12.9s  
│           ┌──┐            |  |   12.8s    ┌──┐           12.5s   12.8s                   12.6s    ┌──┐  
│ ──────────┼──┼────────────┼──┼────┌──┐────┼──┼───11.6s────┌──┐────┌──┐───11.7s────────────┌──┐────|▒▒|──
│  10.9s    |  |            |  |    |  |    |  |    ┌──┐    |  |    |  |    ┌──┐   11.1s    |  |    |▒▒|  
│   ┌──┐    |  |   10.4s    |  |    |  |    |  |    |  |    |  |    |  |    |  |    ┌──┐    |  |    |▒▒|  
│   |  |    |  |    ┌──┐    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
└───┴──┴────┴──┴────┴──┴────┴──┴────┴──┴────┴──┴────┴──┴────┴──┴────┴──┴────┴──┴────┴──┴────┴──┴────┴──┴──>
    T-12    T-11    T-10    T-9     T-8     T-7     T-6     T-5     T-4     T-3     T-2     T-1      T    
Legend

largeDepsZisi: 1m 11.7s

⬇️ 2.48% decrease vs. 67961ee

^                         1m 22.8s                                                                        
│                           ┌──┐                                                                          
│                           |  |                                                                          
│         1m 11.5s          |  |  1m 10.7s1m 10.2s                                                1m 11.7s
│           ┌──┐            |  |    ┌──┐    ┌──┐          1m 4.9s 1m 7.5s                 1m 7.7s   ┌──┐  
│ ──────────┼──┼────────────┼──┼────┼──┼────┼──┼────────────┌──┐────┌──┐────────────────────┌──┐────|▒▒|──
│  58.2s    |  |   55.9s    |  |    |  |    |  |   57.8s    |  |    |  |   59.5s    59s     |  |    |▒▒|  
│   ┌──┐    |  |    ┌──┐    |  |    |  |    |  |    ┌──┐    |  |    |  |    ┌──┐    ┌──┐    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
└───┴──┴────┴──┴────┴──┴────┴──┴────┴──┴────┴──┴────┴──┴────┴──┴────┴──┴────┴──┴────┴──┴────┴──┴────┴──┴──>
    T-12    T-11    T-10    T-9     T-8     T-7     T-6     T-5     T-4     T-3     T-2     T-1      T    
Legend

@eduardoboucas eduardoboucas changed the title feat: use RUSTUP_TOOLCHAIN env var feat: set default Rust toolchain Oct 10, 2021
@kodiakhq kodiakhq bot merged commit 65c32c2 into main Oct 11, 2021
@kodiakhq kodiakhq bot deleted the feat/rustup-toolchain-env branch October 11, 2021 08:50
Skn0tt pushed a commit to netlify/build that referenced this pull request May 21, 2024
* feat: use RUSTUP_TOOLCHAIN env var

* refactor: set default toolchain

Co-authored-by: kodiakhq[bot] <49736102+kodiakhq[bot]@users.noreply.github.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
automerge type: feature code contributing to the implementation of a feature and/or user facing functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants