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

Fix grammatical errors in config template #1062

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

thefury
Copy link

@thefury thefury commented Aug 5, 2024

This commit fixes a few small grammatical errors in the config/tasks template.

This commit fixes a few small grammatical errors in the `config/tasks`
template.
@theCyberTech theCyberTech added the documentation Improvements or additions to documentation label Aug 10, 2024
@pythonbyte
Copy link
Collaborator

Hey @thefury, nice catch.
Can you please resolve the conflict issue?

@thefury
Copy link
Author

thefury commented Aug 27, 2024

Hi @pythonbyte - thanks for getting back to me. I took a look back as I couldn't see how my text changes would affect any existing unit tests. To confirm, I reverted my changes and experienced the same errors. To check further, I've pulled a clean copy of origin and run the following: poetry lock, poetry install, and poetry run pytest and, once again, I'm seeing errors on main.

So, either I'm running this incorrectly, or there is an existing issue with the tests on main that I just happen to have based the changes on.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants