Skip to content

Update README.md#51

Open
lohith1511 wants to merge 1 commit into
drkrillo:mainfrom
lohith1511:patch-2
Open

Update README.md#51
lohith1511 wants to merge 1 commit into
drkrillo:mainfrom
lohith1511:patch-2

Conversation

@lohith1511
Copy link
Copy Markdown

#50 Spelling error solved

drkrillo#50 Spelling error solved
@drkrillo
Copy link
Copy Markdown
Owner

Hi @lohith1511 !

Sorry for the delay in this response.

You are right, there is a mispelling in that part.

But please, note this:

  • README.md is updated everyday and comes from templates/README.md.j2, thats the file you need to do the PR on.

Explanation

  • There is a GitHub Actions workflow that runs daily to fetch issues and update the README file.
  • That workflow uses templates/README.md.j2 to render de new daily README.md file with updated content.
  • If we do a modification on README.md, it will be once again replaced the day after with the template, that still has the mispelling that you spotted.

Solution

Do a PR with the fix in the template README.md.j2 file that lives in the template folder. That will ensure that it propagates correctly to the README.md file generated everyday.

Copy link
Copy Markdown
Owner

@drkrillo drkrillo left a comment

Choose a reason for hiding this comment

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

The change needs to be done in the template file.

Already solved, but other Issues opened for this kind of contributions

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants