-
Notifications
You must be signed in to change notification settings - Fork 69
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
Main #313
Conversation
<!--- 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
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
|
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? |
Really no stress about the merge conflict :) |
I was actually right in the middle of fixing it 😊 |
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 😬 |
Let me fix the merge conflict and then we can merge this in with a minor edit I've thought of 😊 |
I was thinking when looking at it that adding in |
There was a problem hiding this 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
- Include the issue number you're working on, so
- 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 :)
Or how does Greek sound? I could make an issue for that for you? :) |
I.e. adding the language queries for Greek 😊 |
Thanks fo the tips! I will keep that on my mind. And yeah Let me look into #236 for now then I can move to queries for Greek. |
Sounds perfect :) |
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 😊 |
Contributor checklist
Description
Related issue