Skip to content

Fix at lease instead of at least typo #22952

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

Closed
wants to merge 1 commit into from
Closed

Fix at lease instead of at least typo #22952

wants to merge 1 commit into from

Conversation

Greesb
Copy link

@Greesb Greesb commented Jan 2, 2023

Why:

There are multiple typos

Closes github/rest-api-description#2020

What's being changed (if available, include any code snippets, screenshots, or gifs):

Changed all occurrence of at lease to at least

Check off the following:

  • I have reviewed my changes in staging (look for the "Automatically generated comment" and click the links in the "Preview" column to view your latest changes).
  • For content changes, I have completed the self-review checklist.

Sorry, something went wrong.

Unverified

This commit is not signed, but one or more authors requires that any commit attributed to them is signed.
@welcome
Copy link

welcome bot commented Jan 2, 2023

Thanks for opening this pull request! A GitHub docs team member should be by to give feedback soon. In the meantime, please check out the contributing guidelines.

@github-actions
Copy link
Contributor

github-actions bot commented Jan 2, 2023

👋 Hey there spelunker. It looks like you've modified some files that we can't accept as contributions. The complete list of files we can't accept are:
.devcontainer/**
.github/actions-scripts/**
.github/workflows/**
.github/CODEOWNERS
assets/fonts/**
data/graphql/**
Dockerfile*
lib/graphql/**
lib/redirects/**
lib/rest/**
lib/webhooks/**
package*.json
scripts/**
translations/**
content/actions/deployment/security-hardening-your-deployments/**

You'll need to revert all of the files you changed in that list using GitHub Desktop or git checkout origin/main <file name>. Once you get those files reverted, we can continue with the review process. :octocat:

@github-actions github-actions bot added the triage Do not begin working on this issue until triaged by the team label Jan 2, 2023
@Greesb Greesb changed the title fix: Fix at lease instead of at least typo Fix at lease instead of at least typo Jan 2, 2023
Copy link

@ProKashif ProKashif left a comment

Choose a reason for hiding this comment

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

LGTM

@cmwilson21
Copy link
Contributor

@Greesb - Thanks for opening an PR! I've made a slight edit to your post to link it to your issue. For more information on how to do that, see "Linking a pull request to an issue". 👍

I'll get this triaged for review! ⚡

@cmwilson21 cmwilson21 added content This issue or pull request belongs to the Docs Content team waiting for review Issue/PR is waiting for a writer's review developers Content related to developers and removed triage Do not begin working on this issue until triaged by the team labels Jan 3, 2023
@felicitymay
Copy link
Contributor

felicitymay commented Jan 4, 2023

Hi @Greesb 👋🏻

Many thanks for identifying this typo in the REST API docs 🙇🏻‍♀️

Our docs for the REST API articles are generated automatically from an OpenAPI schema, which is why your pull request triggered the warning above. Having autogenerated docs makes it easier to ensure that they are updated whenever the API is updated, but does have the side effect that we'll need to fix this typo in a different repository.

I'll trigger a workflow that will transfer your issue to the API schema repository.

@felicitymay
Copy link
Contributor

Closing because this needs to be fixed in a different repository.

@felicitymay felicitymay closed this Jan 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
content This issue or pull request belongs to the Docs Content team developers Content related to developers waiting for review Issue/PR is waiting for a writer's review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Typo at lease instead of at least
4 participants