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

docs: fixed path to crates, updated link to translating doc #2538

Merged
merged 2 commits into from
Apr 20, 2024

Conversation

viraxslot
Copy link
Contributor

@viraxslot viraxslot commented Apr 20, 2024

Summary

  • actualized contributing guide;
  • removed outdated gen-web command from justfile

Test Plan

Manually checked the changes.

@viraxslot
Copy link
Contributor Author

Looks like this info is outdated too so I can remove it if needed:

Because the website displays the changelog, you should update the website using the following command:

sh
just gen-web

Website is in the separate repo so the command cannot be found:

(fix-contributing)$ just gen-web
cargo codegen-website
error: no such command: `codegen-website`

        View all installed commands with `cargo --list`
        Find a package to install `codegen-website` with `cargo

@ematipico
Copy link
Member

True, it's outdated

@viraxslot
Copy link
Contributor Author

@ematipico removed, but I don't understand the current flow with the website so can't add any new information. If you could suggest what to write I'd love to add it.

@ematipico ematipico merged commit 7341677 into biomejs:main Apr 20, 2024
3 checks passed
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.

None yet

2 participants