Skip to content

[MSFNW26] Add help with missing env and author contact info#12

Merged
LucasHJin merged 2 commits intomainfrom
MFSN26-lucasonboarding
Feb 1, 2026
Merged

[MSFNW26] Add help with missing env and author contact info#12
LucasHJin merged 2 commits intomainfrom
MFSN26-lucasonboarding

Conversation

@LucasHJin
Copy link
Collaborator

@LucasHJin LucasHJin commented Feb 1, 2026

Notion ticket link

Dev Onboarding

Implementation description

  • Provides help with missing env error when running docker-compose up
  • Add Lucas Jin as an author

Steps to test

  1. Open README.md
  2. Read through help section

What should reviewers focus on?

  • Verify that the markdown worked

Checklist

  • My PR name is descriptive and in imperative tense
  • My commit messages are descriptive and in imperative tense. My commits are atomic and trivial commits are squashed or fixup'd into non-trivial commits
  • I have run the appropriate linter(s)
  • I have requested a review from the PL, as well as other devs who have background knowledge on this PR or who will be building on top of this PR

@LucasHJin LucasHJin requested a review from ericahan22 February 1, 2026 19:40
@ericahan22 ericahan22 requested a review from fiona-cai February 1, 2026 19:48
ericahan22
ericahan22 previously approved these changes Feb 1, 2026
Copy link
Member

@ericahan22 ericahan22 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 to me!

Copy link
Collaborator

@fiona-cai fiona-cai left a comment

Choose a reason for hiding this comment

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

LGTM!

@LucasHJin LucasHJin merged commit 85714d0 into main Feb 1, 2026
0 of 2 checks passed
@ericahan22 ericahan22 deleted the MFSN26-lucasonboarding branch February 1, 2026 21:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants