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

Update changed assets in subprojects #12721

Merged
merged 2 commits into from
Jun 10, 2024

Conversation

mkurz
Copy link
Member

@mkurz mkurz commented Jun 10, 2024

@mkurz mkurz added ci:skip-tests Skip 'tests' job in CI builds ci:skip-docs-tests Skip 'docs-tests' job in CI builds labels Jun 10, 2024
@mkurz mkurz force-pushed the fix-asset-changes-in-subprojects branch from 8444155 to 738cebc Compare June 10, 2024 08:45
@mkurz
Copy link
Member Author

mkurz commented Jun 10, 2024

As expected the newly added scripted test fails.

click for log
[info] Resource at /assets/stylesheets/main.css returned 200 as expected
[info] Resource at /assets/stylesheets/main.css contained margin-top: 50px;
[info] [success] Total time: 3 s, completed Jun 10, 2024, 9:00:34 AM
[info] Attempt 0 at /assets/stylesheets/main.css
[info] [info] LESS compiling on 1 source(s)
[info] Got exception: java.lang.RuntimeException: Resource at /assets/stylesheets/main.css didn't contain 'margin-top: 51px;':
[info] /*
[info]  * Copyright (C) from 2022 The Play Framework Contributors <https://github.com/playframework>, 2011-2021 Lightbend Inc. <https://www.lightbend.com>
[info]  */
[info] body {
[info]   margin-top: 50px;
[info] }
[info] . Cause was null
[info] Attempt 1 at /assets/stylesheets/main.css
[info] Got exception: java.lang.RuntimeException: Resource at /assets/stylesheets/main.css didn't contain 'margin-top: 51px;':
[info] /*
[info]  * Copyright (C) from 2022 The Play Framework Contributors <https://github.com/playframework>, 2011-2021 Lightbend Inc. <https://www.lightbend.com>
[info]  */
[info] body {
[info]   margin-top: 50px;
[info] }
[info] . Cause was null
[info] Attempt 2 at /assets/stylesheets/main.css
[info] Got exception: java.lang.RuntimeException: Resource at /assets/stylesheets/main.css didn't contain 'margin-top: 51px;':
[info] /*
[info]  * Copyright (C) from 2022 The Play Framework Contributors <https://github.com/playframework>, 2011-2021 Lightbend Inc. <https://www.lightbend.com>
[info]  */
[info] body {
[info]   margin-top: 50px;
[info] }
[info] . Cause was null
[info] Attempt 3 at /assets/stylesheets/main.css
[info] Got exception: java.lang.RuntimeException: Resource at /assets/stylesheets/main.css didn't contain 'margin-top: 51px;':
[info] /*
[info]  * Copyright (C) from 2022 The Play Framework Contributors <https://github.com/playframework>, 2011-2021 Lightbend Inc. <https://www.lightbend.com>
[info]  */
[info] body {
[info]   margin-top: 50px;
[info] }
[info] . Cause was null
[info] Attempt 4 at /assets/stylesheets/main.css
[info] Got exception: java.lang.RuntimeException: Resource at /assets/stylesheets/main.css didn't contain 'margin-top: 51px;':
[info] /*
[info] [error] 	at play.sbt.scriptedtools.ScriptedTools$.$anonfun$verifyResourceContainsImpl$1(ScriptedTools.scala:106)
[info] [error] 	at scala.collection.immutable.List.foreach(List.scala:431)
[info] [error] 	at play.sbt.scriptedtools.ScriptedTools$.verifyResourceContainsImpl(ScriptedTools.scala:104)
[info] [error] 	at play.sbt.scriptedtools.ScriptedTools$.verifyResourceContains(ScriptedTools.scala:78)
[info] [error] 	at $92c707ab781c35981ff2$.$anonfun$root$4(build.sbt:11)
[info] [error] 	at scala.runtime.java8.JFunction0$mcV$sp.apply(JFunction0$mcV$sp.java:23)
[info] [error] 	at sbt.std.Transform$$anon$3.$anonfun$apply$2(Transform.scala:47)
[info] [error] 	at sbt.std.Transform$$anon$4.work(Transform.scala:69)
[info] [error] 	at sbt.Execute.$anonfun$submit$2(Execute.scala:283)
[info] [error] 	at sbt.internal.util.ErrorHandling$.wideConvert(ErrorHandling.scala:24)
[info] [error] 	at sbt.Execute.work(Execute.scala:292)
[info] [error] 	at sbt.Execute.$anonfun$submit$1(Execute.scala:283)
[info] [error] 	at sbt.ConcurrentRestrictions$$anon$4.$anonfun$submitValid$1(ConcurrentRestrictions.scala:265)
[info] [error] 	at sbt.CompletionService$$anon$2.call(CompletionService.scala:65)
[info] [error] 	at java.base/java.util.concurrent.FutureTask.run(FutureTask.java:264)
[info] [error] 	at java.base/java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:539)
[info] [error] 	at java.base/java.util.concurrent.FutureTask.run(FutureTask.java:264)
[info] [error] 	at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1136)
[info] [error] 	at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:635)
[info] [error] 	at java.base/java.lang.Thread.run(Thread.java:833)
[info] [error] (verifyResourceContains) Resource at /assets/stylesheets/main.css didn't contain 'margin-top: 51px;':
[info] [error] /*
[info] [error]  * Copyright (C) from 2022 The Play Framework Contributors <https://github.com/playframework>, 2011-2021 Lightbend Inc. <https://www.lightbend.com>
[info] [error]  */
[info] [error] body {
[info] [error]   margin-top: 50px;
[info] [error] }
[info] [error] Total time: 16 s, completed Jun 10, 2024, 9:00:50 AM
Error:  x play-sbt-plugin/asset-changes-in-subprojects 
Error:   Cause of test exception: {line 4}  Command failed: verifyResourceContains failed

Pushing the latest webjars-locator-core should fix it.

@mkurz mkurz force-pushed the fix-asset-changes-in-subprojects branch from ca545f3 to 4a2f019 Compare June 10, 2024 09:11
@mkurz mkurz removed ci:skip-tests Skip 'tests' job in CI builds ci:skip-docs-tests Skip 'docs-tests' job in CI builds labels Jun 10, 2024
@mkurz mkurz force-pushed the fix-asset-changes-in-subprojects branch from 4a2f019 to 6b295a3 Compare June 10, 2024 09:11
@mkurz mkurz marked this pull request as ready for review June 10, 2024 09:13
@mkurz
Copy link
Member Author

mkurz commented Jun 10, 2024

It works (just like locally 😉)

@mkurz mkurz merged commit 7356db9 into playframework:main Jun 10, 2024
29 checks passed
@mkurz mkurz deleted the fix-asset-changes-in-subprojects branch June 10, 2024 09:39
@mkurz
Copy link
Member Author

mkurz commented Jun 10, 2024

@Mergifyio backport 3.0.x

@mkurz
Copy link
Member Author

mkurz commented Jun 10, 2024

@Mergifyio backport 2.9.x

Copy link
Contributor

mergify bot commented Jun 10, 2024

backport 3.0.x

✅ Backports have been created

Copy link
Contributor

mergify bot commented Jun 10, 2024

backport 2.9.x

✅ Backports have been created

mergify bot added a commit that referenced this pull request Jun 10, 2024
[3.0.x] Update changed assets in subprojects (backport #12721) by @mkurz
mergify bot added a commit that referenced this pull request Jun 10, 2024
[2.9.x] Update changed assets in subprojects (backport #12721) by @mkurz
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

1 participant