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

Update dependency handlebars to v5 for mdman. #13249

Merged
merged 1 commit into from
Jan 4, 2024

Conversation

WeiTheShinobi
Copy link
Contributor

@WeiTheShinobi WeiTheShinobi commented Jan 4, 2024

What does this PR try to resolve?

Fixes #13238

  • update dependency handlebars 4.5.0 -> 5.0.0
  • fix code to fit the changes of Handlebars API
  • RenderError::new() is deprecated. Use RenderErrorReason instead

How should we test and review this PR?

pass all tests in /crates/mdman/tests

- handlebars 4.5.0 -> 5.0.0
- fix code to fit the changes of Handlebars API
- RenderError::new() is deprecated. Use RenderErrorReason instead
@rustbot
Copy link
Collaborator

rustbot commented Jan 4, 2024

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

Please see the contribution instructions for more information. Namely, in order to ensure the minimum review times lag, PR authors and assigned reviewers should ensure that the review label (S-waiting-on-review and S-waiting-on-author) stays updated, invoking these commands when appropriate:

  • @rustbot author: the review is finished, PR author should check the comments and take action accordingly
  • @rustbot review: the author is ready for a review, this PR will be queued again in the reviewer's queue

@rustbot rustbot added A-cli-help Area: built-in command-line help S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Jan 4, 2024
@epage
Copy link
Contributor

epage commented Jan 4, 2024

Thanks!

@bors r+

@bors
Copy link
Contributor

bors commented Jan 4, 2024

📌 Commit 06201b9 has been approved by epage

It is now in the queue for this repository.

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Jan 4, 2024
@bors
Copy link
Contributor

bors commented Jan 4, 2024

⌛ Testing commit 06201b9 with merge d069c24...

bors added a commit that referenced this pull request Jan 4, 2024
…epage

Update dependency handlebars to v5 for mdman.

### What does this PR try to resolve?

issue #13238
- update dependency handlebars 4.5.0 -> 5.0.0
- fix code to fit the changes of Handlebars API
- RenderError::new() is deprecated. Use RenderErrorReason instead

### How should we test and review this PR?
pass all tests in /crates/mdman/tests
@bors
Copy link
Contributor

bors commented Jan 4, 2024

💔 Test failed - checks-actions

@bors bors added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Jan 4, 2024
@ehuss
Copy link
Contributor

ehuss commented Jan 4, 2024

@bors retry

macOS: The hosted runner encountered an error while running your job. (Error Type: Disconnect).

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Jan 4, 2024
@bors
Copy link
Contributor

bors commented Jan 4, 2024

⌛ Testing commit 06201b9 with merge 87eb374...

@bors
Copy link
Contributor

bors commented Jan 4, 2024

☀️ Test successful - checks-actions
Approved by: epage
Pushing 87eb374 to master...

@bors bors merged commit 87eb374 into rust-lang:master Jan 4, 2024
22 checks passed
@WeiTheShinobi WeiTheShinobi deleted the update_dependency_handlebars branch January 4, 2024 17:34
@WeiTheShinobi WeiTheShinobi restored the update_dependency_handlebars branch January 4, 2024 17:34
@WeiTheShinobi WeiTheShinobi deleted the update_dependency_handlebars branch January 4, 2024 17:34
@WeiTheShinobi WeiTheShinobi restored the update_dependency_handlebars branch January 4, 2024 17:38
bors added a commit to rust-lang-ci/rust that referenced this pull request Jan 6, 2024
Update cargo

10 commits in add15366eaf3f3eb84717d3b8b71902ca36a7c84..2ce45605d9db521b5fd6c1211ce8de6055fdb24e
2024-01-02 03:24:42 +0000 to 2024-01-04 18:04:13 +0000
- feat(embedded): Add prefix-char frontmatter syntax support (rust-lang/cargo#13247)
- Update dependency handlebars to v5 for mdman. (rust-lang/cargo#13249)
- Deprecate rustc plugin support in cargo (rust-lang/cargo#13248)
- feat(embedded): Add multiple experimental manifest syntaxes (rust-lang/cargo#13241)
- chore: update auto labels (rust-lang/cargo#13244)
- test: support publish package with a `public` field. (rust-lang/cargo#13245)
- `cargo fix`: Call rustc fewer times. (rust-lang/cargo#13243)
- chore: tracing be compat with rustc_log (rust-lang/cargo#13239)
- chore(deps): update compatible (rust-lang/cargo#13227)
- Contrib: Fix team HackMD links (rust-lang/cargo#13237)

r? ghost
@rustbot rustbot added this to the 1.77.0 milestone Jan 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-cli-help Area: built-in command-line help S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update dependency handlebars to v5 for mdman
5 participants