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

feat: assist documentation #1387

Open
wants to merge 13 commits into
base: next
Choose a base branch
from
Open

feat: assist documentation #1387

wants to merge 13 commits into from

Conversation

ematipico
Copy link
Member

@ematipico ematipico commented Oct 29, 2024

Summary

Documentation for the Biome Assist.

Lint doc codegen

I refactored the lintdoc.rs to take the actions into account, too. Now we use RuleCategory across all functions to change the generation fo the code.

Since assist actions don't emit a diagnostic, I decided to create an ad-hoc diagnostic for the sole purpose of generating a nice diff for the website, emitted using HTML

For the time being, assist actions don't generate a "actions sources" page.

Navigation and sidebar

I refactored the navigation bar. I removed the top level "Tools" and moved the "Linter" under the "Analyzer".

The Assist main page is under /assist; the "rules" are called actions -> /assist/actions/use-sorted-keys.

Important

I am still writing some docs for the /assist page

@ematipico ematipico requested a review from a team as a code owner October 29, 2024 16:06
@ematipico ematipico marked this pull request as draft October 29, 2024 16:06
Copy link

netlify bot commented Oct 29, 2024

Deploy Preview for biomejs ready!

Name Link
🔨 Latest commit eb22f54
🔍 Latest deploy log https://app.netlify.com/sites/biomejs/deploys/6751c56dee944a0008722760
😎 Deploy Preview https://deploy-preview-1387--biomejs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@ematipico ematipico changed the base branch from main to next December 5, 2024 08:47
@ematipico ematipico force-pushed the feat/document-actions branch from 3c9cc94 to 9a49fa8 Compare December 5, 2024 10:09
@ematipico ematipico marked this pull request as ready for review December 5, 2024 10:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant