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

Main #313

Merged
merged 3 commits into from
Oct 10, 2024
Merged

Main #313

merged 3 commits into from
Oct 10, 2024

Conversation

Nowshin1077
Copy link
Contributor

Contributor checklist


Description

Related issue

<!---
Thank you for your pull request! 🚀
-->

### Contributor checklist

<!-- Please replace the empty checkboxes [] below with checked ones [x] accordingly. -->

- [x] This pull request is on a [separate branch](https://docs.github.com/en/get-started/quickstart/github-flow) and not the main branch

---

### Description

<!--
Describe briefly what your pull request proposes to change. Especially if you have more than one commit, it is helpful to give a summary of what your contribution is trying to solve.

Also, please describe shortly how you tested that your change actually works.
-->

### Related issue

<!--- Scribe-Data prefers that pull requests be related to already open issues. -->
<!--- If applicable, please link to the issue by replacing ISSUE_NUMBER with the appropriate number below. -->
<!--- Feel free to delete this section if this does not apply. -->

- scribe-org#302
Copy link

Thank you for the pull request!

The Scribe team will do our best to address your contribution as soon as we can. The following is a checklist for maintainers to make sure this process goes as well as possible. Feel free to address the points below yourself in further commits if you realize that actions are needed :)

If you're not already a member of our public Matrix community, please consider joining! We'd suggest using Element as your Matrix client, and definitely join the General and Data rooms once you're in. Also consider joining our bi-weekly Saturday dev syncs. It'd be great to have you!

Maintainer checklist

  • The commit messages for the remote branch should be checked to make sure the contributor's email is set up correctly so that they receive credit for their contribution

    • The contributor's name and icon in remote commits should be the same as what appears in the PR
    • If there's a mismatch, the contributor needs to make sure that the email they use for GitHub matches what they have for git config user.email in their local Scribe-Data repo
  • The linting and formatting workflow within the PR checks do not indicate new errors in the files changed

  • The CHANGELOG has been updated with a description of the changes for the upcoming release and the corresponding issue (if necessary)

@andrewtavis
Copy link
Member

What do you want to do here, @Nowshin1077? I'm not even really sure if the Bengali adverbs query needs to be expanded 🤔 We could do a minor edit in here though?

@andrewtavis
Copy link
Member

Really no stress about the merge conflict :)

@andrewtavis
Copy link
Member

I was actually right in the middle of fixing it 😊

@Nowshin1077
Copy link
Contributor Author

Nowshin1077 commented Oct 10, 2024

Sorry, I noticed after making a PR that you already added "Adverbs" query 😬, we can do some minor edits though for other queries. Could you please give me some suggestions if possible? I don't know, I am just feeling lost as I am a noob 😬

@andrewtavis andrewtavis reopened this Oct 10, 2024
@andrewtavis
Copy link
Member

Let me fix the merge conflict and then we can merge this in with a minor edit I've thought of 😊

@andrewtavis
Copy link
Member

I was thinking when looking at it that adding in Bangla would make sense so it's easier for people to find the queries in the codebase if they're searching for this name of the language. Would you have interest in looking into #236? I can also help you find another!

Copy link
Member

@andrewtavis andrewtavis left a comment

Choose a reason for hiding this comment

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

A few more tips, @Nowshin1077:

  • Don't name your branch Main
    • Include the issue number you're working on, so 302-bengali-adverbs
  • Also make a branch for your work so that the any changes that are needed are not committed directly to the main of your fork :)

@andrewtavis andrewtavis merged commit 6b67a2e into scribe-org:main Oct 10, 2024
3 checks passed
@andrewtavis
Copy link
Member

Or how does Greek sound? I could make an issue for that for you? :)

@andrewtavis
Copy link
Member

I.e. adding the language queries for Greek 😊

@Nowshin1077
Copy link
Contributor Author

Thanks fo the tips! I will keep that on my mind. And yeah Bangla would make more sense. 😊

Let me look into #236 for now then I can move to queries for Greek.
What do you say?

@andrewtavis
Copy link
Member

Sounds perfect :)

@andrewtavis
Copy link
Member

Feel free to write in there and ask for adverbs and adjectives to get started for simple queries, and then we can move to something more difficult 😊

@andrewtavis andrewtavis added the hacktoberfest-accepted Accepted as a part of Hacktoberfest label Oct 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hacktoberfest-accepted Accepted as a part of Hacktoberfest
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants