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

Bump go version for playwright test #751

Merged
merged 1 commit into from
Mar 8, 2024
Merged

Bump go version for playwright test #751

merged 1 commit into from
Mar 8, 2024

Conversation

hanzei
Copy link
Contributor

@hanzei hanzei commented Mar 4, 2024

Summary

The playwright test are currenly failing on master with go: go.mod file indicates go 1.21, but maximum version supported by tidy is 1.19. (https://github.com/mattermost/mattermost-plugin-github/actions/runs/8134028209/job/22226371191) Let's bump the go version there and see if that fixes things.

Ticket Link

NONE

@hanzei hanzei added the 2: Dev Review Requires review by a core committer label Mar 4, 2024
@hanzei hanzei requested a review from ayusht2810 March 4, 2024 08:33
@hanzei hanzei requested a review from mickmister as a code owner March 4, 2024 08:33
@mickmister mickmister merged commit 417886a into master Mar 8, 2024
9 checks passed
@mickmister mickmister deleted the fix_playwright branch March 8, 2024 23:33
@hanzei hanzei added 4: Reviews Complete All reviewers have approved the pull request and removed 2: Dev Review Requires review by a core committer labels Mar 11, 2024
@hanzei
Copy link
Contributor Author

hanzei commented Mar 11, 2024

It's failing with a different error 🎉

https://github.com/mattermost/mattermost-plugin-github/actions/runs/8218101374

@mickmister Maybe we disable the job until it has been fixed?

@mickmister
Copy link
Contributor

@mickmister Maybe we disable the job until it has been fixed?

Yes sounds good 👍 Does simply removing the workflow file seem the right way?

@hanzei
Copy link
Contributor Author

hanzei commented Mar 15, 2024

I disabled the workflow manually via https://github.com/mattermost/mattermost-plugin-github/actions/workflows/playwright.yml.

@ayusht2810 ayusht2810 added this to the v2.3.0 milestone Apr 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4: Reviews Complete All reviewers have approved the pull request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants