Skip to content
This repository has been archived by the owner on Aug 7, 2024. It is now read-only.

data: l-legren #7335

Merged
merged 2 commits into from
Jun 3, 2023
Merged

Conversation

l-legren
Copy link
Contributor

@l-legren l-legren commented Jun 2, 2023

Fixes Issue

Changes proposed

Check List (Check all the applicable boxes)

  • My code follows the code style of this project.
  • My change requires changes to the documentation.
  • I have updated the documentation accordingly.
  • All new and existing tests passed.
  • This PR does not contain plagiarized content.
  • The title of my pull request is a short description of the requested changes.

Screenshots

Note to reviewers

@github-actions github-actions bot added the medium Pull request with changed lines between 10 and 30 label Jun 2, 2023
@github-actions
Copy link
Contributor

github-actions bot commented Jun 2, 2023

Welcome @l-legren! Thank you so much for your first pull request!

@github-actions github-actions bot added waiting-for-reviewers ✍ chore: profile This label will be added on PR with request of Data json addition LGTM labels Jun 2, 2023
Copy link
Member

@khushalarora khushalarora left a comment

Choose a reason for hiding this comment

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

Hey @l-legren, thank you for contributing 🎉

data/l-legren.json Outdated Show resolved Hide resolved
Co-authored-by: Khushal Arora <khushalcosmos26@gmail.com>
Copy link
Member

@SaraJaoude SaraJaoude 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

@SaraJaoude SaraJaoude merged commit cd42d17 into EddieHubCommunity:main Jun 3, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
✍ chore: profile This label will be added on PR with request of Data json addition LGTM medium Pull request with changed lines between 10 and 30 waiting-for-reviewers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants