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

Remove mention of CI scripts from docs #11257

Merged
merged 1 commit into from
Oct 29, 2019
Merged

Conversation

jryans
Copy link
Collaborator

@jryans jryans commented Oct 29, 2019

This removes fetch-develop.deps.sh from the docs, as it's designed more for
use on CI environments and ends up causing confusion when used for anything
else.

Fixes #11247

This removes `fetch-develop.deps.sh` from the docs, as it's designed more for
use on CI environments and ends up causing confusion when used for anything
else.

Fixes #11247
@jryans jryans requested a review from a team October 29, 2019 10:40
@jryans
Copy link
Collaborator Author

jryans commented Oct 29, 2019

There was some discussion on the issue of also renaming the script to make it clear it's for CI... but I am not really sure what naming would suffice, so I haven't done that here.

Copy link
Member

@turt2live turt2live left a comment

Choose a reason for hiding this comment

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

If we were to pick a new name, I'd vote for ci-fetch-deps.sh

@jryans jryans merged commit ddf733a into develop Oct 29, 2019
@t3chguy t3chguy deleted the jryans/rm-scripts-from-docs branch May 12, 2022 09:06
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.

check-i18n.pl is linked out of directory
2 participants