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

Use tabs consistently with all other files. #7330

Merged
merged 3 commits into from
Dec 30, 2024

Conversation

Moderocky
Copy link
Member

Description

Some pull requests over the years have introduced spaces rather than tabs in some lines, especially in test files (clearly the work of an enemy saboteur).

Usually, it's obvious when this happens because tabs are ninety metres wide on GitHub, and we spot them in review, but occasionally some slip through.

@Moderocky Moderocky added patch-ready A PR/issue that has been approved and is ready to be merged/closed for the next patch version. 2.10 Targeting a 2.10.X version release feature-ready A PR/issue that has been approved, tested and can be merged/closed in the next feature version. labels Dec 30, 2024
Comment on lines 1 to 6
test "item inventory conflict":
parse:
loop all players:
loop all items in the loop-player's inventory:
set {_item} to loop-item
loop all players:
loop all items in the loop-player's inventory:
set {_item} to loop-item
assert parse logs are not set with "failed to parse loop"
Copy link
Member

Choose a reason for hiding this comment

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

wee woo wee woo

@Moderocky Moderocky requested a review from sovdeeth December 30, 2024 07:47
@Moderocky Moderocky merged commit 48d0bbe into SkriptLang:dev/feature Dec 30, 2024
5 checks passed
@Moderocky Moderocky deleted the the-whitespace-incident branch December 30, 2024 07:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
2.10 Targeting a 2.10.X version release feature-ready A PR/issue that has been approved, tested and can be merged/closed in the next feature version. patch-ready A PR/issue that has been approved and is ready to be merged/closed for the next patch version.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants