Skip to content

Add commands workflow#727

Merged
rom1504 merged 5 commits into
PrismarineJS:masterfrom
extremeheat:commands-workflow
Jul 2, 2023
Merged

Add commands workflow#727
rom1504 merged 5 commits into
PrismarineJS:masterfrom
extremeheat:commands-workflow

Conversation

@extremeheat
Copy link
Copy Markdown
Member

No description provided.

Comment thread .github/workflows/commands.yml Outdated
token: ${{ secrets.PAT_PASSWORD }}
# See `Options` section below for more info on these options
install-command: npm install
/fixlint.fix-command: npm run fix
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

are you sure this will work here?

mcdata is a bit different from all other repos

there's tools/js for package.json stuff

and for releases there's only HISTORY.md , no package.json

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Yeah, the package.json is optional, it can read the version from just the history extremeheat#17

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

how does it know where to run npm run fix ?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

It runs the commands specified in the workflow options at the root of the repo. But yeah it was missing the “cd tools/js” step, fixed that in the workflow options

@rom1504 rom1504 merged commit ecc3312 into PrismarineJS:master Jul 2, 2023
@rom1504
Copy link
Copy Markdown
Member

rom1504 commented Jul 2, 2023

nice, seems to work

maybe let's add it in prismarine template
and let's use https://github.com/PrismarineJS/prismarine-meta to open PRs to add it everywhere

@rom1504
Copy link
Copy Markdown
Member

rom1504 commented Jul 2, 2023

btw I created #728 to issue the commands

does that make sense? should we do this everywhere?

@extremeheat
Copy link
Copy Markdown
Member Author

Yeah, it can be added probably to most of the repos. It's not required to make a new issue, it should work in any issue or PR comment if the user is the author or has commit permissions on the repo.

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.

2 participants