mirror of
https://github.com/yairm210/Unciv.git
synced 2025-07-24 14:49:23 +07:00
Object.entries() apparently had a breaking change
Also updated setup-node to v4, latest
This commit is contained in:
@ -66,7 +66,7 @@ async function parseCommits() {
|
|||||||
}
|
}
|
||||||
);
|
);
|
||||||
|
|
||||||
Object.entries(ownerToCommits).forEach((author,commits) => {
|
Object.entries(ownerToCommits).forEach((author,index, commits) => {
|
||||||
if (commits.length === 1) {
|
if (commits.length === 1) {
|
||||||
commitSummary += "\n\n" + commits[0] + " - By " + author;
|
commitSummary += "\n\n" + commits[0] + " - By " + author;
|
||||||
} else {
|
} else {
|
||||||
|
8
.github/workflows/uncivbot.yml
vendored
8
.github/workflows/uncivbot.yml
vendored
@ -12,7 +12,7 @@ jobs:
|
|||||||
runs-on: ubuntu-latest
|
runs-on: ubuntu-latest
|
||||||
steps:
|
steps:
|
||||||
- uses: actions/checkout@v4
|
- uses: actions/checkout@v4
|
||||||
- uses: actions/setup-node@v2
|
- uses: actions/setup-node@v4
|
||||||
- name: Generate new buildConfig.kt and changelog.md files
|
- name: Generate new buildConfig.kt and changelog.md files
|
||||||
run: |
|
run: |
|
||||||
npm i @octokit/rest
|
npm i @octokit/rest
|
||||||
@ -40,7 +40,7 @@ jobs:
|
|||||||
steps:
|
steps:
|
||||||
|
|
||||||
- name: Merge Pull Request
|
- name: Merge Pull Request
|
||||||
uses: "actions/github-script@v5"
|
uses: actions/github-script@v5
|
||||||
with:
|
with:
|
||||||
github-token: ${{ secrets.ACTIONS_ACCESS_TOKEN }} # Required because default workflow token can't merge PRs
|
github-token: ${{ secrets.ACTIONS_ACCESS_TOKEN }} # Required because default workflow token can't merge PRs
|
||||||
script: |
|
script: |
|
||||||
@ -58,7 +58,7 @@ jobs:
|
|||||||
token: ${{ secrets.ACTIONS_ACCESS_TOKEN }}
|
token: ${{ secrets.ACTIONS_ACCESS_TOKEN }}
|
||||||
ref: master # By default will checkout *the version at time of trigger* which does *not* contain the PR we just merged!
|
ref: master # By default will checkout *the version at time of trigger* which does *not* contain the PR we just merged!
|
||||||
|
|
||||||
- uses: actions/setup-node@v2
|
- uses: actions/setup-node@v4
|
||||||
|
|
||||||
- name: Increment version and release
|
- name: Increment version and release
|
||||||
run: |
|
run: |
|
||||||
@ -76,7 +76,7 @@ jobs:
|
|||||||
runs-on: ubuntu-latest
|
runs-on: ubuntu-latest
|
||||||
steps:
|
steps:
|
||||||
- uses: actions/checkout@v4
|
- uses: actions/checkout@v4
|
||||||
- uses: actions/setup-node@v2
|
- uses: actions/setup-node@v4
|
||||||
- name: Add translation PRs to the new branch, and PR the new branch to master
|
- name: Add translation PRs to the new branch, and PR the new branch to master
|
||||||
|
|
||||||
# SO, the story is that when using the default access token you CANNOT merge PRs from forks.
|
# SO, the story is that when using the default access token you CANNOT merge PRs from forks.
|
||||||
|
Reference in New Issue
Block a user