Skip to content
This repository has been archived by the owner on Feb 12, 2022. It is now read-only.

prevent uploading to beta releases older than the current release #1178

Merged
merged 2 commits into from
Aug 15, 2014

Conversation

jdx
Copy link
Contributor

@jdx jdx commented Aug 13, 2014

there are 2 minor bugs with beta releases. First, the updater currently thinks that beta releases are newer than the released version.

In other words, that 3.9.6-pre is newer than 3.9.6. That was resolved by adding the semantic gem to do the actual calculations.

Another bug was that it would update to the newest beta release. Now it will abort if it is about to install something older than the current release.

@jdx
Copy link
Contributor Author

jdx commented Aug 14, 2014

this might be a bad idea. It appears old rubygems versions will fail https://rails.lighthouseapp.com/projects/8994/tickets/2880-malformed-version-number-string-30pre

happened to me... might have to look for an alternative solution

@pedro
Copy link
Contributor

pedro commented Aug 14, 2014

damn! never ceases to amaze how even the most trivial things can be broken in some context, heh.

@jdx
Copy link
Contributor Author

jdx commented Aug 14, 2014

So this is actually a non-issue I think (at least part of it)

I think I was just using the pre version wrong, it's supposed to be something like 3.9.6.pre1 not 3.9.6.pre

I'll test to make sure it works that way.

Ensuring beta releases don't downgrade is important, however. So I'll probably change this PR to just include that.

@jdx jdx changed the title fixed bugs around beta releases prevent uploading to beta releases older than the current release Aug 14, 2014
@jdx
Copy link
Contributor Author

jdx commented Aug 14, 2014

updating to just include the changes for preventing updating to an older release

dickeyxxx added 2 commits August 15, 2014 14:00
This conversion is done by Transpec 2.3.6 with the following command:
    transpec

* 19 conversions
    from: == expected
      to: eq(expected)

* 19 conversions
    from: obj.should
      to: expect(obj).to

* 2 conversions
    from: obj.stub(:message)
      to: allow(obj).to receive(:message)

* 1 conversion
    from: obj.stub(:message => value)
      to: allow(obj).to receive_messages(:message => value)

For more details: https://github.com/yujinakayama/transpec#supported-conversions
jdx pushed a commit that referenced this pull request Aug 15, 2014
prevent uploading to beta releases older than the current release
@jdx jdx merged commit a8128d8 into master Aug 15, 2014
@jdx jdx deleted the beta-releases branch August 15, 2014 22:08
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants