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.

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

Remove IncludeDocsLink from bufconfig.BufYAMLFile #3240

Open
bufdev opened this issue Aug 15, 2024 · 0 comments
Open

Remove IncludeDocsLink from bufconfig.BufYAMLFile #3240

bufdev opened this issue Aug 15, 2024 · 0 comments
Assignees
@doriable
Labels
Cleanup Cleanup tasks

Comments

@bufdev
Copy link
Member

bufdev commented Aug 15, 2024

This is a write-only property that has nothing to do with the BufYAMLFile interface and what this object is. We need to rewrite #3181 so as to not add this method to BufYAMLFile, which shouldn't be that bad - it should just be an option PutBufYAMLFileForPrefixWithIncludeDocsLink.
@doriable doriable self-assigned this Aug 15, 2024
@bufdev bufdev added the Cleanup Cleanup tasks label Aug 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@doriable doriable

Labels
Cleanup Cleanup tasks
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@bufdev @doriable