Skip to content

Add github workflow - #219

Closed
todi1856 wants to merge 4 commits into
masterfrom
add-github-workflow
Closed

todi1856 wants to merge 4 commits into
masterfrom
add-github-workflow

Conversation

@todi1856

Copy link
Copy Markdown
Member

Please read and consider what information you want to pass on to people reviewing this PR

Purpose of PR

Type of change:

  • Bug fix
  • New feature
  • Code refactor
  • Breaking change
  • Documentation update
  • Other (please describe)
  • Description of the feature
    What it does, why was it created, how does it work and so on...
  • Links to screenshots, design docs, user docs, etc.
  • Links to Jira/Fogbugz cases
  • Halo effect
    What areas do you know/expect this feature to affect? Do you have any worries about possible breaking points? Write them down here.
  • Performance Impact
    Could this PR affect performance? If so, what has been done to measure time taken per frame, memory used, etc.?
  • Package requirements
    What Unity Editor version is required/supported? Any other dependancy packages needed?

Checklist for PR maker

  • Have you added a backport label? (if needed)
  • Have you updated the Changelog? Each package has a CHANGELOG.md file
  • Have you added or updated the Documentation to your PR? When you add a new feature, change a property name, or change the behaviour of a feature, it's best practice to include related documentation changes in the same PR

Testing status

  • Existing or new automation tests - what automation was added, changed
    Put any relevant Yamato links here
  • Detailed description of what testing was done, what projects were run
    Tell the reviewers what did you already test, so that they may have a better understanding when reviewing/testing this PR themselves
  • What environment did you test on?
    Your Operating System, Platforms you ran on, Graphics APIs and any other relevant info you can think of
  • Projects
    Any existing projects ready for testing should be noted here
  • Testing checklist
  • Built and run editor Locally or Katana
  • Built a player Android/iOS (if applicable)
  • Run on device Android/iOS (if applicable)
  • If new feature has UI, does it match Unity style? (Unity HIG?)
  • Tested Undo/Redo + Prefab overrides + Alignment in Preset
  • All items have tooltips?

Note to guardian

Changeset to use, if extra changesets on top. Any other notes to pass along

Comments to reviewers

Notes for the reviewers you have assigned

@u-pr u-pr Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

💡 Harness Review

Needs changes

The new automation cannot reliably update pull request descriptions in its current form.

Reviewed commit a3a73ae

🤖 Helpful? 👍/👎

Comment thread .github/workflows/prefill-pr.yml Outdated
name: Prefill PR Body

on:
pull_request:

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

P1

Because this public repository uses the pull_request event, workflows opened from forks receive a read-only GITHUB_TOKEN; the requested write permission cannot elevate it, and pulls.update will fail with 403 for external contributions. Use pull_request_target (without checking out or executing contributor-controlled code) or another privileged follow-up workflow if fork PRs must also be updated.

🤖 Helpful? 👍/👎

todi1856 and others added 3 commits September 21, 2026 15:26
Co-authored-by: u-pr[bot] <205906871+u-pr[bot]@users.noreply.github.com>
…om.unity.mobile.logcat into add-github-workflow
@todi1856 todi1856 closed this Sep 21, 2026
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.

1 participant