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

Fix Typo (report column and README) #1

Merged
merged 1 commit into from Apr 6, 2022
Merged

Conversation

parroty
Copy link
Contributor

@parroty parroty commented Apr 6, 2022

Change

Hi, it's a nice report🗒️ . Though it's a minor one..., I just noticed some typos while evaluating this action.

Testings Done

Build

$ node --version
v16.13.1
$ npm --version
8.1.2
$ npm install
$ npm run build

Behavior

  • Just tested on example with forked version.

Before

before

After

after

name: Organization Authorization Action

on:
  workflow_dispatch:

jobs:
  github-authorization-report:
    runs-on: ubuntu-latest
    steps:
      - name: Checkout
        uses: actions/checkout@v3
      - name: Get authorization report
        # uses: nicklegan/github-org-authorization-action@v1.0.0
        uses: parroty/github-org-authorization-action@fix/typo
        with:
          token: ${{ secrets.ORG_TOKEN }}
          organization: 'xxx-demo'

@parroty parroty requested a review from nicklegan as a code owner April 6, 2022 09:06
@nicklegan nicklegan merged commit 6de43e9 into nicklegan:main Apr 6, 2022
@nicklegan
Copy link
Owner

👋 @parroty thanks for the typo fixes! :shipit:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants