Skip to content
This repository has been archived by the owner on Apr 18, 2023. It is now read-only.

Quote github_user with double quotes #1975

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

25077667
Copy link

@25077667 25077667 commented May 3, 2022

Application to the Class of 2022🎓

This pull request template helps you complete an application to the Class of 2022. Use the checklist below to verify you have followed the instructions correctly.

Checklist ✅

  • I have read the instructions on the README file before submitting my application.
  • I made my submission by creating a folder on the _data folder and followed the naming convention mentioned in the instructions (<username>) and markdown file.
  • I have submitted a swag shipping form.
  • I have used the Markdown file template to add my information to the Year Book.
  • I understand that a reviewer will merge my pull request after examining it or ask for changes in case needed.
  • I understand I should not tag or add a reviewer to this Pull Request.
  • I have added the event to my Calendar.

Copy link
Collaborator

@github-education github-education left a comment

Choose a reason for hiding this comment

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

Hi 25077667,
Welcome to graduation!

Your Pull Request looks good! Thanks for your graduation submission. Next, one of our reviewers will come by to check your PR meets the Code of Conduct. You don’t need to do anything but wait. Once they've completed their review, they'll leave another comment asking for changes or merge your Pull Request.

If anyone's username is full numerical string the mdParser would
deduct it as a number rather than a string. Therefore there should
have a type-casting when it would be compared via type-sensitive
compare operator.
@25077667
Copy link
Author

Hi all,
May I ask what is the current status?
Thanks.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants