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: switch back to pull_request event trigger for gh actions #591

Merged
merged 1 commit into from
Jan 3, 2024

Conversation

mas-who
Copy link
Contributor

@mas-who mas-who commented Jan 3, 2024

Done

  • Switch to use pull_request event trigger instead of pull_request_target for the pr check action. The pull_request_target event trigger was put in place so that we can get write permissions with the GITHUB_TOKEN for foreign (forked) PRs as actions would be run in the context of the base repository. However, it seems actions are actually running in context of the main branch as well which will disregard incoming changes.
  • Will look for an alternative way to publish playwright reports.
  • Also fixed some formatting issues that cropped up due to linting dependency updates.

@webteam-app
Copy link

Demo starting at https://lxd-ui-591.demos.haus

Signed-off-by: Mason Hu <mason.hu@canonical.com>
Copy link
Collaborator

@edlerd edlerd left a comment

Choose a reason for hiding this comment

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

LGTM

@edlerd edlerd merged commit 9e8a909 into main Jan 3, 2024
11 of 12 checks passed
@edlerd edlerd deleted the remove-publish-report-job branch February 21, 2024 17:05
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.

None yet

3 participants