Skip to content

Update changelog's data ranges #11027

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

Merged
merged 1 commit into from
Jun 26, 2023
Merged

Conversation

blyxyas
Copy link
Member

@blyxyas blyxyas commented Jun 25, 2023

Related to #10847
Instead of the non-descriptive commit ranges, now the changelog uses dates (much more illustrative, and cooler 😎 )
changelog:none

@rustbot
Copy link
Collaborator

rustbot commented Jun 25, 2023

r? @Manishearth

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

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Jun 25, 2023
@Manishearth
Copy link
Member

@bors r+

@bors
Copy link
Contributor

bors commented Jun 26, 2023

📌 Commit 7bc384f has been approved by Manishearth

It is now in the queue for this repository.

@bors
Copy link
Contributor

bors commented Jun 26, 2023

⌛ Testing commit 7bc384f with merge f8c2440...

bors added a commit that referenced this pull request Jun 26, 2023
Update changelog's data ranges

Related to #10847
Instead of the non-descriptive commit ranges, now the changelog uses dates (much more illustrative, and cooler 😎 )
changelog:none
@flip1995
Copy link
Member

@blyxyas Could you add a section/short note about this line to the book, please?

@bors
Copy link
Contributor

bors commented Jun 26, 2023

💥 Test timed out

@Manishearth
Copy link
Member

I'll wait for flip's comments to be addressed

@@ -56,6 +56,29 @@ and open that file in your editor of choice.
When updating the changelog it's also a good idea to make sure that `commit1` is
already correct in the current changelog.

#### From commit range to PR link

Raw commit ranges aren't used, they aren't very useful or illustrative, so we developed the concept of PR ranges to help the user understand the size of a new update.
Copy link
Member

Choose a reason for hiding this comment

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

hm, it's less useful to mention the old model here.

Let's just document what we do, rather than what we used to do

@Manishearth
Copy link
Member

@bors r+

@bors
Copy link
Contributor

bors commented Jun 26, 2023

📌 Commit e6d0f55 has been approved by Manishearth

It is now in the queue for this repository.

@bors
Copy link
Contributor

bors commented Jun 26, 2023

⌛ Testing commit e6d0f55 with merge efe447b...

bors added a commit that referenced this pull request Jun 26, 2023
Update changelog's data ranges

Related to #10847
Instead of the non-descriptive commit ranges, now the changelog uses dates (much more illustrative, and cooler 😎 )
changelog:none
@bors
Copy link
Contributor

bors commented Jun 26, 2023

💔 Test failed - checks-action_remark_test

@Manishearth
Copy link
Member

book/src/development/infrastructure/changelog_update.md
  61:232  warning  Line must be at most 120 characters  maximum-line-length  remark-lint

@Manishearth
Copy link
Member

@bors r+

@bors
Copy link
Contributor

bors commented Jun 26, 2023

📌 Commit aea731e has been approved by Manishearth

It is now in the queue for this repository.

@bors
Copy link
Contributor

bors commented Jun 26, 2023

⌛ Testing commit aea731e with merge 2a9cc31...

@bors
Copy link
Contributor

bors commented Jun 26, 2023

☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Approved by: Manishearth
Pushing 2a9cc31 to master...

@bors bors merged commit 2a9cc31 into rust-lang:master Jun 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants