Skip to content

fix of #18181 issue - #18359

Merged
bors merged 1 commit into
rust-lang:masterfrom
1-more:feature
Oct 30, 2014
Merged

fix of #18181 issue#18359
bors merged 1 commit into
rust-lang:masterfrom
1-more:feature

Conversation

@1-more

Copy link
Copy Markdown
Contributor

No description provided.

@rust-highfive

Copy link
Copy Markdown
Contributor

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @huonw (or someone else) soon.

@alexcrichton

Copy link
Copy Markdown
Member

Thanks! Could you expand the commit message to have a short line at the top explaining what's being done, followed by a more detailed explanation of how it was accomplished, and finally with Closes #18181 to ensure the issue is closed when this is merged?

@1-more

Copy link
Copy Markdown
ContributorAuthor

Hi! I put a comment on my commit. Is it OK now? Or something else is to be done?

Technically speaking, negative duration is not valid ISO 8601, but we need to
print it anyway. If `d` is a positive duration with the output `xxxxxxx`, then
the expected output of negative `-d` value is `-xxxxxxx`. I.e. the idea is to
print negative durations as positive with a leading minus sign.
Closesrust-lang#18181.
@alexcrichton

Copy link
Copy Markdown
Member

Ah I mean in the commit message itself, you can use git commit --amend to edit it.

@1-more

Copy link
Copy Markdown
ContributorAuthor

OK, no problem

@borsbors closed this Oct 30, 2014
@bors
bors merged commit 9bf82fa into rust-lang:masterOct 30, 2014
@1-more
1-more deleted the feature branch October 30, 2014 08:20
lnicola pushed a commit to lnicola/rust that referenced this pull request Oct 22, 2024
…eykril
feat: support initializeStopped setting
See rust-lang#18356
Add option to start rust-analyzer in "stopped" state when the extension activates.
Sign up for freeto 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.

5 participants

@1-more@rust-highfive@alexcrichton@bors@huonw