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

1.0 release notes and authors updates #25196

Merged
merged 2 commits into from
May 11, 2015
Merged

1.0 release notes and authors updates #25196

merged 2 commits into from
May 11, 2015

Conversation

brson
Copy link
Contributor

@brson brson commented May 7, 2015

cc @aturon

@brson brson added the beta-nominated Nominated for backporting to the compiler in the beta channel. label May 7, 2015
@rust-highfive
Copy link
Collaborator

r? @nikomatsakis

(rust_highfive has picked a reviewer for you, use r? to override)

* [Lifetime shadowing is an error][lt].
* [`Send` no longer implies `'static`][send-rfc], which made possible
the [`thread::scoped` API][scoped]. Scoped threads can borrow data
from their parent's stack frame -- safely!
Copy link
Member

Choose a reason for hiding this comment

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

thread::scoped is not in 1.0

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks, fixed.

@cjwelborn
Copy link

I noticed at least one duplicate in the AUTHORS. There may be others:

GlacJAY <glacjay@gmail.com>
...
Jay True <glacjay@gmail.com>

@nham
Copy link
Contributor

nham commented May 8, 2015

I'm in the list twice:

Nick Hamann <nick@wabbo.org>
nham <hamann.nick@gmail.com>

You can remove the second one if you'd like.

@cjwelborn
Copy link

There are 2 duplicated names (Alfie John and Jakub Wieczorek with different email addresses). I don't know if it's intentional or not, so I haven't tried to make a PR or anything. I found 23 duplicate email addresses (names are different, some use their middle name, some are 'handles').

@cjwelborn
Copy link

@brson, I made a PR on your repo that removes the duplicates in what I think is a sane way to go about it.

@brson
Copy link
Contributor Author

brson commented May 8, 2015

@cjwelborn Thanks! Merged.

@brson
Copy link
Contributor Author

brson commented May 8, 2015

@tshepang Under the normal release schedule, the beta of 4 weeks ago would not have its own release notes - those would be the release notes of the next stable release. So the thinking was to combine them to provide the full picture of what happened recently. The amount of new work that went into 1.0 this cycle was actually pretty small...

Beta's are not 'releases' even though we treated the last one as such.

@alfiedotwtf
Copy link
Contributor

Looks like I'm in there twice. Thanks @cjwelborn, but my preferred email for all things Rust is alfie@alfie.wtf :)

@cjwelborn
Copy link

@alfiedotwtf, sorry about that. I'm glad you caught it before it was finalized.

@daramos
Copy link
Contributor

daramos commented May 8, 2015

D'oh! Looks like I had my email as my name in gitconfig.

Any chance dan@daramos.com <dan@daramos.com> could be changed to Daniel Ramos <dan@daramos.com>?

Sorry about that...

@caipre
Copy link
Contributor

caipre commented May 8, 2015

If you're taking these last minute comments... s/caipre/Nick Platt

[new `dropck`][dropck]. This change is a major reduction in unsafe
code.
* Trait coherence was [revised again][fundamental], this time with an
eye toward API evolution over time.
Copy link
Member

Choose a reason for hiding this comment

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

This seems to be a duplicate.

@rpjohnst
Copy link
Contributor

rpjohnst commented May 8, 2015

Could you update "Russell rpjohnst@gmail.com" to "Russell Johnston rpjohnst@gmail.com"?

@Zoxc
Copy link
Contributor

Zoxc commented May 8, 2015

Can someone move John Kleint out of my names?

without breaking downsteam code.
* Digits of binary and octal literals are [lexed more eagerly][lex] to
improve error messages and macro behavior. For example, `0b1234` is
now lexed as `0b1234` instead of two tokens, `0b1` and `34`.
Copy link
Member

Choose a reason for hiding this comment

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

The 2 appears to be missing.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good eye.

@ogham
Copy link
Contributor

ogham commented May 8, 2015

Not sure why I decided to abbreviate my name. Could you change 'Ben S' to 'Ben Sago'? Ta

@thorncp
Copy link
Contributor

thorncp commented May 8, 2015

Please change Chris Thorn <thorn@thoughtbot.com> to Chris Thorn <chris@thorn.co>.

So much thanks to the Rust team for doing this! 💗

@sfackler
Copy link
Member

sfackler commented May 8, 2015

@ogham "Ben S" is the name associated with your github profile: https://github.com/ogham

@laumann
Copy link
Contributor

laumann commented May 8, 2015

@brson Is it any kind of code contribution that counts (ie commits in master)?

@TyOverby
Copy link
Contributor

TyOverby commented May 8, 2015

Could you change "TyOverby" to "Ty Overby"? Thanks!

@huonw
Copy link
Member

huonw commented May 8, 2015

@laumann yes, any commit!

@blackbeam
Copy link
Contributor

Could you please change "blackbeam" to "Anatoly Ikorsky"?

@laumann
Copy link
Contributor

laumann commented May 8, 2015

@huonw Awesome, then I'm missing from the list (see d14109e):
Thomas Bracht Laumann Jespersen <laumann.thomas@gmail.com>.

@huonw
Copy link
Member

huonw commented May 8, 2015

@laumann, oh, I apologise because I realise I may have misspoke: this is the first release where the release isn't from the master branch, so some recent contributions may not be in it (I'm unsure if that commit is).

@brson
Copy link
Contributor Author

brson commented May 11, 2015

@laumann Anybody with commits to this repo.

@brson brson force-pushed the relnotes branch 4 times, most recently from 851ade0 to 5c26a7b Compare May 11, 2015 01:45
@brson
Copy link
Contributor Author

brson commented May 11, 2015

Updated with all comments addressed (I think) except @Zoxc's.

bar`, and Cargo now automatically translates "-" in *package* names
to underscore for the crate name.
* [Lifetime shadowing is an error][lt].
* [`Send` no longer implies `'static`][send-rfc], which made possible
Copy link
Member

Choose a reason for hiding this comment

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

, which made possible is a half sentence hanging here.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed.

@Zoxc
Copy link
Contributor

Zoxc commented May 11, 2015

@brson My complaint was addressed atleast. I'm no longer listed twice.

Version 1.0.0-beta (April 2015)
-------------------------------------

* ~1100 changes, numerous bugfixes
Copy link
Member

Choose a reason for hiding this comment

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

am curious why remove change count

Copy link
Contributor

Choose a reason for hiding this comment

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

Thanks for mentioning me!

This is me:
Heejong Ahn <heejongahn@gmail.com poga.bahamut@gmail.com>

p.s) To be frank, it's my first (and misc) open-source contribution and I
don't know what's really going on here so I just mocked what Poga's mail..
Plz let me know if I've done anything wrong.

2015-05-11 14:28 GMT+09:00 Tshepang Lekhonkhobe notifications@github.com:

In RELEASES.md
#25196 (comment):

@@ -1,79 +1,140 @@

-Version 1.0.0-beta (April 2015)

-* ~1100 changes, numerous bugfixes

am curious why remove change count


Reply to this email directly or view it on GitHub
https://github.com/rust-lang/rust/pull/25196/files#r30013677.

Copy link
Member

Choose a reason for hiding this comment

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

@brson I see you put the count back in 94caeb3... kool.

@gareins
Copy link
Contributor

gareins commented May 11, 2015

I am:
Ožbolt Menegatti <ozbolt.menegatti@gmail.com>

If it needs to be ascii, just use 'z' instead of 'ž'.

ps. Thanks everyone for an awesome language :)

@jest
Copy link
Contributor

jest commented May 11, 2015

If spellchecking is considered relevant contribution, please add

Przemek Wesołek <jest@go.art.pl>

due to #24812

@KokaKiwi
Copy link

If writing a postponed RFC is considered as a relevant contribution (maybe not, but I wanted to ask anyway :p), is it possible to add

KokaKiwi <kokakiwi+rust@kokakiwi.net>

Due to rust-lang/rfcs#257

@alexcrichton
Copy link
Member

r=me whenever this is ready @brson

@brson
Copy link
Contributor Author

brson commented May 11, 2015

@bors: r=alexcrichton

@bors
Copy link
Contributor

bors commented May 11, 2015

📌 Commit 94caeb3 has been approved by alexcrichton

@bors
Copy link
Contributor

bors commented May 11, 2015

⌛ Testing commit 94caeb3 with merge a90dd32...

bors added a commit that referenced this pull request May 11, 2015
@brson
Copy link
Contributor Author

brson commented May 11, 2015

I've already picked this into beta, so setting tag to beta-accepted

@brson brson added beta-accepted Accepted for backporting to the compiler in the beta channel. and removed beta-nominated Nominated for backporting to the compiler in the beta channel. labels May 11, 2015
@bors bors merged commit 94caeb3 into rust-lang:master May 11, 2015
@dhardy
Copy link
Contributor

dhardy commented May 16, 2015

I checked the authors file. It contains an old spam-infested email address, which is fine. Now somehow my github email appears in the 1.0 announcement, and right on queue I get a bunch of phishing emails. How'd that happen? I'm now going to have to replace the github email... don't put the new one in blog posts please.

@alfiedotwtf
Copy link
Contributor

@dhardy that's security through obscurity. All it takes is a bot to clone one of your repo to see your email address. Consider using temporary aliases for anything public facing.

@dhardy
Copy link
Contributor

dhardy commented May 16, 2015

Consider using temporary aliases for anything public facing.

That's precisely what it is, and mostly my system works (it seems bots are not in the habit of cloning repos). I only get spam on the gmail address and if anyone emails me there, they get an "out-of-office" reply notifying them that I don't read it frequently.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
beta-accepted Accepted for backporting to the compiler in the beta channel.
Projects
None yet
Development

Successfully merging this pull request may close these issues.