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

Create mentoring.md #2309

Merged
merged 4 commits into from
Feb 2, 2024
Merged

Create mentoring.md #2309

merged 4 commits into from
Feb 2, 2024

Conversation

RomaLetodiani
Copy link
Contributor

No description provided.

@github-actions github-actions bot added the track/typescript TypeScript track label Jan 6, 2024
@RomaLetodiani
Copy link
Contributor Author

anyone to review this PR?

Copy link
Member

@IsaacG IsaacG left a comment

Choose a reason for hiding this comment

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

  1. @SleeplessByte is the typescript track maintainer and probably the best to review this, assuming they have the availability for this. Mentor notes may not be a priority for them at the moment.
  2. Mentor notes typically provide talking points and common gotchas or questions. Mentors can find community solutions pretty easily so finding a solution is pretty easy. It's the rest that provides much of the value. Can you provide notes for mentors beyond paraphrasing the exercise and providing a solution?

tracks/typescript/exercises/etl/etl/mentoring.md Outdated Show resolved Hide resolved
tracks/typescript/exercises/etl/etl/mentoring.md Outdated Show resolved Hide resolved
@RomaLetodiani
Copy link
Contributor Author

@IsaacG
Considered your suggestion and updated file

SleeplessByte
SleeplessByte previously approved these changes Feb 2, 2024
tracks/typescript/exercises/etl/etl/mentoring.md Outdated Show resolved Hide resolved
tracks/typescript/exercises/etl/etl/mentoring.md Outdated Show resolved Hide resolved
tracks/typescript/exercises/etl/etl/mentoring.md Outdated Show resolved Hide resolved
tracks/typescript/exercises/etl/etl/mentoring.md Outdated Show resolved Hide resolved
tracks/typescript/exercises/etl/etl/mentoring.md Outdated Show resolved Hide resolved
tracks/typescript/exercises/etl/etl/mentoring.md Outdated Show resolved Hide resolved
@SleeplessByte SleeplessByte merged commit 2e96557 into exercism:main Feb 2, 2024
1 check passed
@SleeplessByte
Copy link
Member

Thank you very much. This is a good first version.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
track/typescript TypeScript track
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants