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

[WIP] Upgrade rand 0.5 #258

Closed
wants to merge 2 commits into from

Conversation

gabisurita
Copy link

I'm submitting a ...

  • bug fix
  • feature enhancement
  • deprecation or removal
  • refactor

Description

Upgrades rand to 0.5, thus enabling stdweb support, among other upgrades. For the full changelog, check https://github.com/rust-lang-nursery/rand/blob/master/CHANGELOG.md#050---2018-05-21.

Related Issue(s)

#257

@gabisurita gabisurita changed the title Upgrade rand 0.5 [WIP] Upgrade rand 0.5 Jun 2, 2018
@kinggoesgaming
Copy link
Member

kinggoesgaming commented Jun 3, 2018

Thanks @gabisurita for the PR!!

There are few points to go over here:

  • 0.6.*(master) is currently locked from further updates, unless they are bug fixes.
  • 0.7.x(breaking) branch has gotten major refactors, including lib.rs(we removed cfg-if from our dependency tree)
  • rand@0.5.0 has a minimum supported rust version (msrv) of 1.22.0 while 0.6.x has a msrv of 1.18.0, which makes this update a breaking change. breaking already has enforced 1.22.0 as the new msrv in preparation for rand@0.5.0

As such I would suggest closing (master -> breaking merge conflict game is not fun) this PR, and opening another one against breaking branch

@Dylan-DPC-zz Dylan-DPC-zz changed the base branch from master to breaking June 3, 2018 06:26
@Dylan-DPC-zz
Copy link
Member

Dylan-DPC-zz commented Jun 3, 2018

@gabisurita thanks for the PR. However as pointed above, this needs to be targetted to the breaking branch (as mentioned here. Easier way out is to submit a new PR with the change.

PS: I have changed the branch to breaking so if you want you can solve the conflicts and get it updated.

@Dylan-DPC-zz Dylan-DPC-zz added breaking This causes a breaking change and will be merged to breaking branch. Release Notes labels Jun 3, 2018
@Dylan-DPC-zz Dylan-DPC-zz added this to the 0.7.0 milestone Jun 3, 2018
@kinggoesgaming
Copy link
Member

This has been superseded by #262

@kinggoesgaming kinggoesgaming added Wont Fix:Superseded and removed breaking This causes a breaking change and will be merged to breaking branch. Release Notes labels Jun 12, 2018
bors bot added a commit that referenced this pull request Jun 12, 2018
262: update rand to 0.5.x r=Dylan-DPC a=kinggoesgaming

# Description
Update `rand` to `0.5` and refactor code accordingly

# Motivation
Regular dependency updates

# Tests
Curent tests passing

# Related Issue(s)
supersedes #258 


Co-authored-by: Hunar Roop Kahlon <hunar.roop@gmail.com>
@gabisurita gabisurita deleted the upgrade-rand-0.5 branch June 19, 2018 14:30
LinusU pushed a commit to LinusU/uuid that referenced this pull request Feb 26, 2020
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.

None yet

3 participants