Skip to content

Remove Vale and use AI instead#101

Draft
SergioGasquez wants to merge 10 commits into
esp-rs:feat/overhaulfrom
SergioGasquez:feat/remove-vale
Draft

Remove Vale and use AI instead#101
SergioGasquez wants to merge 10 commits into
esp-rs:feat/overhaulfrom
SergioGasquez:feat/remove-vale

Conversation

@SergioGasquez
Copy link
Copy Markdown
Member

No description provided.

Comment thread tools/ai-doc-review/review.mjs Outdated
Comment thread .github/workflows/ai-docs-review.yml Outdated
Comment thread .github/workflows/ai-docs-review.yml

- Use a clear, friendly, instructional voice for learners.
- Prefer active voice and direct instructions.
- Use `we` for guided walkthroughs and `you` when addressing actions the reader must take. Avoid switching voice within the same paragraph.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Nice, these are great - did you come up with these or analyze the first section to produce this?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Comment thread .github/doc-terms.json Outdated
@@ -0,0 +1,1273 @@
use std::collections::{BTreeMap, BTreeSet, HashMap};
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

❤️

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Thanks for the pointers!

Sign up for free to 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.

3 participants