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

Add cargo doc generation and publishing. #3

Closed
wants to merge 0 commits into from
Closed

Add cargo doc generation and publishing. #3

wants to merge 0 commits into from

Conversation

d3zd3z
Copy link
Collaborator

@d3zd3z d3zd3z commented Sep 12, 2024

No description provided.

@d3zd3z d3zd3z force-pushed the docgen branch 13 times, most recently from f7025b2 to 7cac483 Compare September 12, 2024 20:36
@d3zd3z d3zd3z force-pushed the docgen branch 2 times, most recently from 7770f76 to 92433f5 Compare September 12, 2024 23:42
- name: Checkout
uses: actions/checkout@v4
with:
path: zephyr-rust-lang
Copy link

Choose a reason for hiding this comment

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

Why not use the module path?

Suggested change
path: zephyr-rust-lang
path: modules/lang/rust

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I'm actually not sure it needs any path, since it seems to result in it coming out in zephyr-rust-lang/zephyr-rust-lang. But, now that the module is "real", I wonder if this can just update the module instead of checking it out somewhere else. I'll investigate.

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.

2 participants