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.

Sign up for GitHub

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

Jump to bottom

Add Cairo links when multiple components are imported #427

Merged

ericglau merged 1 commit into OpenZeppelin:master from ericglau:cairolinks

Jan 17, 2025

Member

ericglau commented Jan 16, 2025

If multiple components are imported, link to the parent .cairo file with the same name as the last path segment
If a single component is imported, link to the file containing the component (if defined in the known mappings), otherwise link to the parent .cairo file with the same name as the last path segment

Fixes #423


          Add Cairo links when multiple components imported

0bd1d4f

ericglau requested review from ericnordelo, andrew-fleming and immrsd

January 16, 2025 22:27

ericnordelo approved these changes

View reviewed changes

Member

ericnordelo left a comment

LGTM! We should add more entries to the components mapping, but we can do that in a separate PR.

ericglau merged commit e1973f5 into OpenZeppelin:master

9 checks passed

ericglau deleted the cairolinks branch

January 17, 2025 14:24

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet