Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Add the irq crate #245

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
bors merged 1 commit into rust-embedded:master from jonas-schievink:patch-2
Jan 26, 2020
Merged

Add the irq crate #245

bors merged 1 commit into rust-embedded:master from jonas-schievink:patch-2
Jan 26, 2020

Conversation

@jonas-schievink
Copy link
Contributor

@jonas-schievink jonas-schievink commented Jan 26, 2020

No description provided.

Copy link

r? @rahul-thakoor

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

@rust-highfive rust-highfive added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-resources labels Jan 26, 2020
Copy link
Member

@andre-richter andre-richter left a comment

Choose a reason for hiding this comment

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

bors r+

Copy link
Contributor Author

bors does not check reviews for commands, only normal comments

Copy link
Member

I'm sure bors does or used to check review comments for commands, it's how I've almost always r+'d PRs. Has it changed recently?

jamesmunns reacted with thumbs up emoji

Copy link
Member

bors r=andre-richter

bors bot added a commit that referenced this pull request Jan 26, 2020
245: Add the `irq` crate r=andre-richter a=jonas-schievink
Co-authored-by: Jonas Schievink <jonasschievink@gmail.com>
Copy link
Member

Yup done it this way since a long time. Strange...

bors r+

Copy link
Contributor

bors bot commented Jan 26, 2020

Already running a review

Copy link
Contributor Author

Huh, weird

Copy link
Contributor

bors bot commented Jan 26, 2020

@bors bors bot merged commit e03eda4 into rust-embedded:master Jan 26, 2020
Copy link
Member

If anyone is investigating, I also wonder if Github could have changed the way bors "sees" comments that originate from review comments. Ill do separate review and bors steps from now on though.

Copy link
Member

Looks like it's already filed: bors-ng/bors-ng#855

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@berkus berkus Awaiting requested review from berkus berkus is a code owner

@rahul-thakoor rahul-thakoor Awaiting requested review from rahul-thakoor rahul-thakoor is a code owner

@RandomInsano RandomInsano Awaiting requested review from RandomInsano RandomInsano is a code owner

1 more reviewer

@andre-richter andre-richter andre-richter approved these changes

Reviewers whose approvals may not affect merge requirements

Labels

S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-resources

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /