Skip to content
This repository has been archived by the owner on Aug 31, 2023. It is now read-only.

chore(xtask): Part 8: Add line breaks before doc comments #3246

Merged
merged 2 commits into from
Sep 22, 2022

Conversation

MichaReiser
Copy link
Contributor

@MichaReiser MichaReiser commented Sep 16, 2022

This PR is part of the comments refactoring #3227.

Summary

This PR changes the binding code gen to emit leading line breaks before doc comments.

It contains a few more changes but I hope that's ok:

  • The codegen changes so that cargo codegen formatter generates files that use the FormatUnknownNodeRule for unknown nodes.
  • Adds a count feature to xtask bench to easily get the counts of any type using countme

Test Plan

Generated the bindings and formatted them with the new formatter.

@MichaReiser MichaReiser requested a review from a team September 16, 2022 13:56
@MichaReiser MichaReiser self-assigned this Sep 16, 2022
@MichaReiser MichaReiser added the A-Formatter Area: formatter label Sep 16, 2022
@MichaReiser MichaReiser added this to the 0.10.0 milestone Sep 16, 2022
@MichaReiser MichaReiser force-pushed the refactor/comments-infrastructure branch from 4327f88 to 1362393 Compare September 16, 2022 15:23
@MichaReiser MichaReiser force-pushed the refactor/js-syntax-comments branch 2 times, most recently from c2038ef to 9f33f98 Compare September 20, 2022 11:28
Base automatically changed from refactor/js-syntax-comments to refactor/comments September 22, 2022 06:13
@MichaReiser MichaReiser force-pushed the refactor/comments-infrastructure branch from beb55c1 to ba64695 Compare September 22, 2022 06:15
@MichaReiser MichaReiser merged commit f4dbc00 into refactor/comments Sep 22, 2022
@MichaReiser MichaReiser deleted the refactor/comments-infrastructure branch September 22, 2022 06:16
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A-Formatter Area: formatter
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants