Skip to content

Comments

[MINOR][DOCS] Rename Global to Glob#37484

Closed
khaledh wants to merge 2 commits intoapache:masterfrom
khaledh:patch-1
Closed

[MINOR][DOCS] Rename Global to Glob#37484
khaledh wants to merge 2 commits intoapache:masterfrom
khaledh:patch-1

Conversation

@khaledh
Copy link
Contributor

@khaledh khaledh commented Aug 12, 2022

This section is about the path glob filter. The word global seems to be a mistake.

What changes were proposed in this pull request?

Just a typo fix.

Why are the changes needed?

To be less confusing in what the section is about.

Does this PR introduce any user-facing change?

Apart from docs, no.

How was this patch tested?

No testing needed.

This section is about the path **glob** filter. The word **global** seems to be a mistake.
@github-actions github-actions bot added the DOCS label Aug 12, 2022
Copy link
Member

@srowen srowen left a comment

Choose a reason for hiding this comment

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

glob is actually short for global but you are right, it is called glob

@srowen srowen changed the title Rename Global to Glob [MINOR][DOCS] Rename Global to Glob Aug 13, 2022
@srowen
Copy link
Member

srowen commented Aug 13, 2022

I'm sure this change doesn't break any tests, but can you enable tests to run in your fork?
https://github.com/apache/spark/pull/37484/checks?check_run_id=7799006209

@AmplabJenkins
Copy link

Can one of the admins verify this patch?

@khaledh
Copy link
Contributor Author

khaledh commented Aug 14, 2022

I enabled GitHub Actions in my fork.

@srowen
Copy link
Member

srowen commented Aug 14, 2022

Hm, looks like it still didn't run, you're sure? needs a push too

@khaledh
Copy link
Contributor Author

khaledh commented Aug 14, 2022

Pushed an immaterial change and got a green build.

@srowen
Copy link
Member

srowen commented Aug 14, 2022

Merged to master

@srowen srowen closed this in 2fa5c3b Aug 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants