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(macos): explicitly mention which fields are skipped #255

Conversation

kevinaboos
Copy link
Contributor

@kevinaboos kevinaboos commented Jun 28, 2024

New users may be confused when looking at the docs for Config and other config-like structs, as some fields cannot be specified in config files or via Cargo's package metadata.
This commit clarifies that those fields are intentionally excluded.

Also added an example of a macOS developer signing identity.

Notes

I added this notice to the docs of only 3 fields, those that were annotated with #[serde(skip)]. Let me know if there are any other fields that should have this notice too.

New users may be confused when looking at the docs for `Config`
and other config-like structs, as some fields cannot be specified
in config files or via Cargo's package metadata.
This commit clarifies that those fields are intentionally excluded.

Also added an example of a macOS developer signing identity.
@denjell-crabnebula
Copy link
Contributor

I approved the workflows because this is non-controversial. Still need @lucasfernog-crabnebula or @amr-crabnebula to approve.

Copy link
Member

@lucasfernog-crabnebula lucasfernog-crabnebula left a comment

Choose a reason for hiding this comment

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

nice catch, thanks
we skip because those are secrets so they should be defined programatically or via env vars

@lucasfernog-crabnebula lucasfernog-crabnebula merged commit 753ec77 into crabnebula-dev:main Jun 29, 2024
22 checks passed
@kevinaboos
Copy link
Contributor Author

nice catch, thanks we skip because those are secrets so they should be defined programatically or via env vars

Thanks, yeah I should've mentioned it here, but for the sake of giving credit, Amr is the one who informed me of this originally: #246 (comment)

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