ci: run stylua via npx rather than via action #2

Closed

AdeAttwood wants to merge 2 commits from adeattwood/stylua-action into 0.x

pull from: adeattwood/stylua-action

merge into: AdeAttwood:0.x

AdeAttwood:0.x

AdeAttwood:adeattwood/rg-command-update

AdeAttwood:test

AdeAttwood:sapling-pr-archive-AdeAttwood

AdeAttwood:remote/0.x

AdeAttwood:sl-support

AdeAttwood:lua-iter

AdeAttwood:docs

AdeAttwood:backends

AdeAttwood:fix/41

AdeAttwood:better-delete-word

AdeAttwood:remove-cpp

AdeAttwood:issue-templates

AdeAttwood:fix/26

AdeAttwood:add-logo

AdeAttwood:fix/20

AdeAttwood:deps

AdeAttwood:feat/add-actions

AdeAttwood:refactor/10

AdeAttwood:fix/8

AdeAttwood:dot-files

AdeAttwood:better-ffi

Conversation 1 Commits 2 Files changed -

AdeAttwood commented 2024-09-03 17:22:07 +00:00

Owner

Copy link

Summary:

This is so we don't need the GITHUB_TOKEN that can not be added in forgejo. I
am giving it a go and the GITHUB_ prefix is reserved for secrets. There are
also issues with the action in a forgejo environment due to the API not
resolving to github.

This now should work on both platforms. So we can maintain github compatibility
for any pull requests that may come that way.

Test Plan:

Run the CI on github and forgejo, this will need to be a manual process to
validate this works.

Summary: This is so we don't need the GITHUB_TOKEN that can not be added in forgejo. I am giving it a go and the `GITHUB_` prefix is reserved for secrets. There are also issues with the action in a forgejo environment due to the API not resolving to github. This now should work on both platforms. So we can maintain github compatibility for any pull requests that may come that way. Test Plan: Run the CI on github and forgejo, this will need to be a manual process to validate this works.

AdeAttwood added 1 commit 2024-09-03 17:22:07 +00:00

ci: run stylua via npx rather than via action ... e82227c427

Summary:

This is so we don't need the GITHUB_TOKEN that can not be added in forgejo. I
am giving it a go and the `GITHUB_` prefix is reserved for secrets. There are
also issues with the action in a forgejo environment due to the API not
resolving to github.

This now should work on both platforms. So we can maintain github compatibility
for any pull requests that may come that way.

Test Plan:

Run the CI on github and forgejo, this will need to be a manual process to
validate this works.

AdeAttwood self-assigned this 2024-09-03 17:24:50 +00:00

AdeAttwood referenced this pull request from a commit 2024-09-03 18:20:26 +00:00

ci: run stylua via npx rather than via action

AdeAttwood added 1 commit 2024-09-03 18:28:22 +00:00

Merge branch '0.x' into adeattwood/stylua-action 75912732a0

AdeAttwood commented 2024-09-03 19:45:47 +00:00

Author

Owner

Copy link

Merged in f910955ad1

Merged in f910955ad190f133893c0e46a81bcb26337b063c

AdeAttwood closed this pull request 2024-09-03 19:45:47 +00:00

AdeAttwood deleted branch adeattwood/stylua-action 2024-09-03 19:46:04 +00:00

All checks were successful

Hide all checks

CI / Luacheck (pull_request) Successful in 18s

Details

CI / StyLua (pull_request) Successful in 13s

Details

CI / Cargo Format (pull_request) Successful in 18s

Details

CI / Build and test (pull_request) Successful in 1m49s

Details

Conventional Tools Commitlint / Commitlint (pull_request) Successful in 5s

Details

Pull request closed

This pull request cannot be reopened because the branch was deleted.

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels

No items

No labels

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

AdeAttwood

1 participant

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: AdeAttwood/ivy.nvim#2

No description provided.