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: 🔥 don't actually need the view action, have the README #1012

Merged
merged 1 commit into from
Jan 28, 2025

Conversation

lwjohnst86
Copy link
Member

Description

I mainly wrote the view and list functions in the context of the CLI or app, where it was more interactive and it was multi-user. But since we're focusing on core and lib, and since we have the information in human-readable form in the README, it doesn't make sense to have these view_ and list_ interfaces. When we do develop CLI, we can implement in cli if necessary, but I think the functionality with making the README will be sufficient for most purposes.

This PR needs a quick review.

Checklist

  • Updated documentation

@lwjohnst86 lwjohnst86 requested a review from a team as a code owner January 28, 2025 12:05
@lwjohnst86 lwjohnst86 merged commit 2488ed2 into main Jan 28, 2025
4 checks passed
@lwjohnst86 lwjohnst86 deleted the docs/remove-view-action branch January 28, 2025 12:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants