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.

Sign up for GitHub

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

Jump to bottom

Fixed spelling and grammar instance in index.rst #1229

Merged
merged 3 commits into from
Oct 17, 2023
Merged

Fixed spelling and grammar instance in index.rst #1229

merged 3 commits into from
Oct 17, 2023

Conversation

qstokkink
Copy link
Collaborator

@qstokkink qstokkink commented Oct 17, 2023 •
edited
Loading

This PR:

  • Adds a fallback to the "." directory if run_all_tests.py is run from the "" directory.
  • Fixes spelling and grammar instance in index.rst
  • Fixes the validation matrix still not using the PR head (this time using this solution).

This PR is mainly to test the recent validation Action fixes.

@qstokkink
Copy link
Collaborator Author

validate

@qstokkink
Copy link
Collaborator Author

validate

@qstokkink
Copy link
Collaborator Author

😕 it's still pulling the latest commit on master instead of the PR head:

 "C:\Program Files\Git\bin\git.exe" log -1 --format='%H'
'0f98cc5cbb95a794ae62f8767ebce036270532ee'

@qstokkink
Copy link
Collaborator Author

validate

@qstokkink qstokkink changed the title WIP: Fixed spelling and grammar instance in index.rst READY: Fixed spelling and grammar instance in index.rst Oct 17, 2023
@qstokkink qstokkink marked this pull request as ready for review October 17, 2023 14:13
@qstokkink qstokkink changed the title READY: Fixed spelling and grammar instance in index.rst Fixed spelling and grammar instance in index.rst Oct 17, 2023
@qstokkink qstokkink merged commit 7a2ef1b into Tribler:master Oct 17, 2023
8 checks passed
@qstokkink qstokkink deleted the fix_index_typo branch October 17, 2023 14:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@egbertbouman egbertbouman egbertbouman approved these changes

Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@qstokkink @egbertbouman