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

PR: Help People Get Started Faster! #74

Merged
merged 7 commits into from
Feb 14, 2023
Merged

PR: Help People Get Started Faster! #74

merged 7 commits into from
Feb 14, 2023

Conversation

nelsonic
Copy link
Member

@nelsonic nelsonic commented Jan 29, 2023 •
edited
Loading

Addresses: #75

@nelsonic nelsonic added documentation Improvements or additions to documentation enhancement New feature or request chore in-progress priority-2 user-feedback labels Jan 29, 2023
@nelsonic nelsonic self-assigned this Jan 29, 2023
@codecov
Copy link

codecov bot commented Jan 29, 2023 •
edited
Loading

Codecov Report

Merging #74 (eb916a9) into main (9f684aa) will not change coverage.
The diff coverage is n/a.

❗ Current head eb916a9 differs from pull request most recent head eae134f. Consider uploading reports for the commit eae134f to get more accurate results

@@            Coverage Diff            @@
##              main       #74   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            4         4           
  Lines           70        70           
=========================================
  Hits            70        70
Impacted Files Coverage Δ
demo_app/lib/services/todoService.dart
demo_app/lib/repository/todoRepository.dart
demo_app/lib/repository/todo_repository.dart 100.00% <0.00%> (ø)
demo_app/lib/services/todo_service.dart 100.00% <0.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Arhell
Arhell previously approved these changes Jan 29, 2023
View reviewed changes
@nelsonic nelsonic changed the title PR: Update info PR: Help People Get Started Faster! Jan 30, 2023
@nelsonic nelsonic mentioned this pull request Feb 3, 2023
Open
3 tasks
@nelsonic nelsonic assigned SimonLab and unassigned nelsonic Feb 6, 2023
@nelsonic
Copy link
Member Author

nelsonic commented Feb 6, 2023

@SimonLab please take a quick look at these docs additions and merge if ok. 🙏

SimonLab
SimonLab reviewed Feb 14, 2023
View reviewed changes
@nelsonic @SimonLab
Fill-in gap in command.
da94dc5 
Co-authored-by: Simon <simon@dwyl.com>
SimonLab
SimonLab reviewed Feb 14, 2023
View reviewed changes
SimonLab
SimonLab reviewed Feb 14, 2023
View reviewed changes
SimonLab
SimonLab approved these changes Feb 14, 2023
View reviewed changes
Copy link
Member

@SimonLab SimonLab left a comment

Choose a reason for hiding this comment

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

Looks good

@SimonLab SimonLab merged commit 1ebe634 into main Feb 14, 2023
@SimonLab SimonLab deleted the update-info branch February 14, 2023 09:24
@nelsonic
Copy link
Member Author

Thanks @SimonLab 👍

@SimonLab SimonLab removed their assignment Feb 14, 2023
@LuchoTurtle LuchoTurtle mentioned this pull request Feb 20, 2023
Closed
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@SimonLab SimonLab SimonLab approved these changes

@Arhell Arhell Arhell left review comments

Assignees
No one assigned
Labels
awaiting-review chore documentation Improvements or additions to documentation enhancement New feature or request priority-2 user-feedback
Projects
dwyl app kanban
Status: ✅ Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@nelsonic @SimonLab @Arhell