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

[docs] fix typos, grammar issues and improve wording across the codebase, documentation and specs #19495

Merged
merged 146 commits into from
Dec 28, 2023

Conversation

jsoref
Copy link
Contributor

@jsoref jsoref commented Oct 20, 2021

Checklist

  • I've run bundle exec rspec from the root directory to see all new and existing tests pass
  • I've followed the fastlane code style and run bundle exec rubocop -a to ensure the code style is valid
  • I've read the Contribution Guidelines
  • I've updated the documentation if necessary.

Motivation and Context

Spelling errors result in all kinds of problems, including:

  • tests that don't test what authors think they're testing,
  • systems that are harder to use because the expected spelling doesn't work,
  • making it harder to search the code for an item

Description

This PR corrects misspellings identified by the check-spelling action.

Testing Steps

The misspellings have been reported at:

The action reports that the changes in this PR would make it happy:

@google-cla google-cla bot added the cla: yes label Oct 20, 2021
Copy link
Contributor Author

@jsoref jsoref left a comment

Choose a reason for hiding this comment

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

Most word corrections were picked by Google Sheets, but I stepped in at times, and all errors are my fault.

Testing.md Outdated Show resolved Hide resolved
deliver/lib/deliver/options.rb Show resolved Hide resolved
deliver/lib/deliver/options.rb Outdated Show resolved Hide resolved
deliver/lib/deliver/sync_screenshots.rb Show resolved Hide resolved
deliver/lib/deliver/upload_screenshots.rb Show resolved Hide resolved
spaceship/lib/spaceship/connect_api/client.rb Show resolved Hide resolved
spaceship/lib/spaceship/connect_api/tunes/tunes.rb Outdated Show resolved Hide resolved
spaceship/lib/spaceship/tunes/app_details.rb Outdated Show resolved Hide resolved
spaceship/spec/tunes/app_version_spec.rb Show resolved Hide resolved
@Kukurijek
Copy link
Contributor

Hi @jsoref!

I think this is a very good improvement 🚀

On the other hand PR is not small and there are a lot of things that need to be tested. Since there are currently conflicts in PR, can you rebase the master and resolve the conflicts please?

Thx 👍

jsoref added 26 commits March 7, 2022 09:07
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
.rubocop.yml Show resolved Hide resolved
Testing.md Show resolved Hide resolved
deliver/lib/deliver/options.rb Show resolved Hide resolved
deliver/spec/commands_generator_spec.rb Show resolved Hide resolved
match/lib/match/storage/git_storage.rb Show resolved Hide resolved
match/lib/match/storage/gitlab/client.rb Show resolved Hide resolved
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Copy link
Member

@rogerluan rogerluan left a comment

Choose a reason for hiding this comment

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

Just left a few more comments and this should be good to go 😊

deliver/lib/deliver/runner.rb Outdated Show resolved Hide resolved
fastlane/swift/Fastlane.swift Outdated Show resolved Hide resolved
fastlane_core/lib/fastlane_core/string_filters.rb Outdated Show resolved Hide resolved
gym/lib/gym/generators/package_command_generator_xcode7.rb Outdated Show resolved Hide resolved
@rogerluan
Copy link
Member

Also, could you check the CI failures? 🙏

Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Copy link
Contributor Author

@jsoref jsoref left a comment

Choose a reason for hiding this comment

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

I looked at the CI failures -- what I saw was a bizarre caching behavior. I've searched and people have been tripping on it for years. I tried to quickly reproduce it in a simple repository and failed. I filed a ticket to circleci asking for help.

fastlane/swift/Fastlane.swift Outdated Show resolved Hide resolved
@@ -55,7 +55,7 @@ def run
verify_version if options[:app_version].to_s.length > 0 && !options[:skip_app_version_update]

# Rejecting before upload meta
# Screenshots can not be update/deleted if in waiting for review
# Screenshots cannot be updated or deleted if the app is in the "waiting for review" state
Copy link
Contributor Author

Choose a reason for hiding this comment

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

fwiw, i have a rule (which I'm not using because it's too complicated) that generally complains about not...or because sometimes there should be a neither/nor pattern. But both (a) correctly applying a rule and (b) correctly describing the rule to catch incorrect cases are too hard, so I just cry a little bit when I see things like this.

I do agree that expanding the / is the right thing as is fixing the tense agreement.

I suspect this could be can be neither ... nor...while..., but that's really too much and I'd rather not deal w/ it in this PR.

Copy link
Member

Choose a reason for hiding this comment

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

I love your attention to detail, can we work together on more projects? 😆

Copy link
Member

@rogerluan rogerluan left a comment

Choose a reason for hiding this comment

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

This LGTM! 🚀

It looks like you're the creator of the tool you're using, is that right? I'd love to integrate this tool in other projects as well as this one for a continuous spellchecking in PRs and such amaze would this be something you'd be interested in setting up, for this repo? 😊

Thanks for addressing all the comments and for bearing with the time it took for me to get to this PR 🙇 better late than never I guess? 😅

@@ -55,7 +55,7 @@ def run
verify_version if options[:app_version].to_s.length > 0 && !options[:skip_app_version_update]

# Rejecting before upload meta
# Screenshots can not be update/deleted if in waiting for review
# Screenshots cannot be updated or deleted if the app is in the "waiting for review" state
Copy link
Member

Choose a reason for hiding this comment

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

I love your attention to detail, can we work together on more projects? 😆

@jsoref
Copy link
Contributor Author

jsoref commented Dec 28, 2023

Yeah, I'm happy to create a PR for you once this merges. Do you have a real-time chat mechanism? it's helpful to talk over some of its knobs / show how it does things...

@rogerluan
Copy link
Member

@jsoref you can find me at t.me/rogerluan if you're on Telegram :)?

We have an internal Slack too but its history gets deleted after 90 days… if you're not on telegram, shoot me your email and I can add you to Slack! 🙏

@rogerluan rogerluan merged commit e4296f1 into fastlane:master Dec 28, 2023
2 checks passed
@jsoref jsoref deleted the spelling branch December 28, 2023 02:55
SubhrajyotiSen pushed a commit to KeepTruckin/fastlane that referenced this pull request Jan 17, 2024
…ase, documentation and specs (fastlane#19495)

* spelling: accept

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: acknowledge

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: adaptations

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: already

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: attempts

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: available

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: backed up

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: boolean

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: build

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: candidate

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: capability

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: catch/return

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: characters

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: collapsible

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: command-line

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: commands

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: contains

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: convenient

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: copies

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: default

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: deleted

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: deprecated

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: destination

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: directories

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: does

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: early

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: enqueued

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: error

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: evaluate

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: exclude

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: family

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: faraday

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: foo

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: function

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: groups

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: identifier

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: identifying

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: identities

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: ignore

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: individual

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: infrequent

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: integer

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: mapping

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: milliseconds

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: modified

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: multiple

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: nonexistent

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: normalized

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: original

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: originating

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: output

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: platform

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: portrait

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: practices

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: priority

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: probably

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: properly

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: provisioning

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: quote

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: reached

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: reenable

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: rejected

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: representing

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: represents

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: returns

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: screenshot

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: screenshots

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: secondary

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: set

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: shuts down

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: signature

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: simplify

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: spaceship

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: specified

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: string if

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: studio

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: succeeds

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: successfully

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: suppress

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: syntax

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: triggered

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: unauthorized

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: unique

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: unprocessable

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: upload

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: uploaded

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: verbose

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: version

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: whether

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: with

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* spelling: without

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>

* Use plural version of `env_names` and add backwards compatible env var identifiers.

* Revert changes made to `Fastlane.swift` autogenerated file.

* Improve wording.

* Use plural version of `env_names` and add backwards compatible env var identifiers.

* spelling: already

Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>

* spelling: anymore

Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>

* spelling: are

Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>

* spelling: backed up

Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>

* spelling: be

Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>

* spelling: cannot

Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>

* spelling: case-insensitive

Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>

* spelling: doubled

Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>

* spelling: emulate

Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>

* spelling: entry

Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>

* spelling: equal

Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>

* spelling: every time

Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>

* spelling: exclude

Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>

* spelling: existence

Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>

* spelling: far, far

Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>

* spelling: fine-grained

Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>

* spelling: for

Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>

* spelling: github

Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>

* spelling: gitlab

Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>

* spelling: identifying

Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>

* spelling: indicates

Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>

* spelling: macos

Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>

* spelling: matching

Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>

* spelling: minimally

Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>

* spelling: nonexistent

Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>

* spelling: parallel

Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>

* spelling: perform

Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>

* spelling: purchase

Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>

* spelling: resolution

Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>

* spelling: respond

Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>

* spelling: returns

Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>

* spelling: shellwords

Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>

* spelling: sierra

Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>

* spelling: squiggly

Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>

* spelling: still

Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>

* spelling: the

Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>

* spelling: through

Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>

* spelling: universal

Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>

* spelling: unsuitable

Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>

* spelling: using

Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>

* spelling: with

Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>

* spelling: work around

Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>

* spelling: workaround

Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>

* spelling: working

Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>

* spelling: data_for_app_store_versions

Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>

* Reverting changes to Fastlane.swift

* Improving comment

* spelling: time,

Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>

* spelling: of the dsym

Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>

---------

Signed-off-by: Josh Soref <jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Co-authored-by: Josh Soref <jsoref@users.noreply.github.com>
Co-authored-by: Roger Oba <rogerluan.oba@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants