Skip to content

Conversation

@tam-fr
Copy link

@tam-fr tam-fr commented Oct 1, 2025

Learners, PR Template

Self checklist

  • I have titled my PR with Region | Cohort | FirstName LastName | Sprint | Assignment Title
  • My changes meet the requirements of the task
  • I have tested my changes
  • My changes follow the style guide

Changelist

Briefly explain your PR.

Questions

Ask any questions you have for your reviewer.

<br>
<input type="text" name="text" id="name" minlength="3" required value="yourName">
<br>

Choose a reason for hiding this comment

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

Dont follow the good practices in the code style

<label for="name">What's your name?</label>
<br>
<input type="text" name="text" id="name" minlength="3" required value="yourName">
<br>

Choose a reason for hiding this comment

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

the <br> is not a group tag, I recommend you to use <div> or <p>

@talmurshidi
Copy link
Member

Please do the assignments in different branches and separated PRs
I've already left a feedback on your personal issues on GitHub

@talmurshidi talmurshidi self-requested a review October 27, 2025 13:55
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