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(flutter_todos): include barrel files in tutorial for completeness #4350

Merged
merged 2 commits into from
Feb 17, 2025

Conversation

nkolatsis
Copy link
Contributor

Status

READY

Breaking Changes

NO

Description

Small documentation update on the flutter_todos. I am making my way through the tutorial and noticed the barrel files are missing from the tutorial. It is easy to overlook this and have import issues without checking the source code directly.

This update to the tutorial contains a small section with a description above code references to the two barrel files in the todos_api package.

Type of Change

  • ✨ New feature (non-breaking change which adds functionality)
  • 🛠️ Bug fix (non-breaking change which fixes an issue)
  • ❌ Breaking change (fix or feature that would cause existing functionality to change)
  • 🧹 Code refactor
  • ✅ Build configuration change
  • 📝 Documentation
  • 🗑️ Chore

@nkolatsis nkolatsis requested a review from felangel as a code owner February 6, 2025 12:21
Copy link
Owner

@felangel felangel left a comment

Choose a reason for hiding this comment

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

LGTM thanks for the contribution! 💙

@felangel felangel added the documentation Documentation requested label Feb 17, 2025
@felangel felangel changed the title Added the barrel file updates needed to get the todos_api package wor… docs(flutter_todos): include barrel files in tutorial for completeness Feb 17, 2025
@felangel felangel merged commit 1b9ba40 into felangel:master Feb 17, 2025
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Documentation requested
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants