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

Updated Diving Into The Lake #636

Merged
merged 17 commits into from
Sep 4, 2023
Merged

Updated Diving Into The Lake #636

merged 17 commits into from
Sep 4, 2023

Conversation

JoshuaPoddoku
Copy link
Contributor

⚠️   Pre Checklist

Please complete ALL items in this checklist, and remove before submitting

  • I have npm run build and npm run serve locally before submitting this PR
  • I have read through the Contributing Documentation

Summary

  • Added Diving Into The Lake 02 ft. Ben Stiefel
  • I also added .vscode in .gitignore file as it was being generated while using DevChat

@JoshuaPoddoku
Copy link
Contributor Author

@Startrekzky I have merged the answers. Please help to review the PR.

Startrekzky
Startrekzky approved these changes Sep 4, 2023
View reviewed changes
Copy link
Contributor

@Startrekzky Startrekzky left a comment

Choose a reason for hiding this comment

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

LGTM

@Startrekzky Startrekzky merged commit a0e40bf into apache:main Sep 4, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Startrekzky Startrekzky Startrekzky approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@JoshuaPoddoku @Startrekzky