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

Switch to GHA #215

Merged
merged 1 commit into from
May 13, 2022
Merged

Switch to GHA #215

merged 1 commit into from
May 13, 2022

Conversation

ihostage
Copy link
Member

No description provided.

@ihostage ihostage requested a review from mkurz May 13, 2022 16:48
@ihostage ihostage merged commit db8bbcf into main May 13, 2022
@mergify mergify bot deleted the gha branch May 13, 2022 16:51
"scalafmtSbtCheck",
"scalafmtCheckAll",
) ++
List(scalaChat, scalaMultiRoomChat, scalaClusteredChat, javaChat, javaMultiRoomChat, javaClusteredChat)
Copy link
Member

Choose a reason for hiding this comment

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

👍

apt:
packages:
- unzip
- dbus-x11
Copy link
Member

Choose a reason for hiding this comment

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

Very nice the GHA VM's provide these packages.

Copy link
Member Author

Choose a reason for hiding this comment

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

I'm not sure 😄 Just I migrated tests to webdrivermanager and used a headless mode

.addArguments("--headless", "--no-sandbox", "--disable-dev-shm-usage")

Copy link
Member Author

Choose a reason for hiding this comment

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

That why now we don't need install Chrome driver manually and use X11 for tests 😄

Copy link
Member

Choose a reason for hiding this comment

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

According to https://github.com/actions/virtual-environments/blob/ubuntu20/20220508.1/images/linux/Ubuntu2004-Readme.md there should be xvfb, unzip and dbus, however not sure about dbus-x11, maybe it's pulled in by another dep anyway. Whatever.

Copy link
Member

Choose a reason for hiding this comment

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

That why now we don't need install Chrome driver manually and use X11 for tests 😄

https://github.com/actions/virtual-environments/blob/ubuntu20/20220508.1/images/linux/Ubuntu2004-Readme.md#browsers-and-drivers 😉

lazy val runChromeWebDriver = taskKey[Unit]("Run the chromewebdriver tests")

val macwire = "com.softwaremill.macwire" %% "macros" % "2.3.4"
val lombok = "org.projectlombok" % "lombok" % "1.18.8"
val lombok = "org.projectlombok" % "lombok" % "1.18.8" % Provided
Copy link
Member

Choose a reason for hiding this comment

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

Good catch 👍

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