Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Relax PhaseAssembly about loose constraints #10858

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

Merged
merged 1 commit into from
Sep 11, 2024
Merged

Conversation

lrytz
Copy link
Member

@lrytz lrytz commented Sep 10, 2024

Phases referenced in constraints (runsAfter, runsBefore) may not exist when running Scaladoc.

  • Don't warn about constraints on non-existing phases in Scaladoc
  • Drop invalid constraints (runsAfter / runsRightAfter / runsBefore)
  • Drop unreachabele phases (no valid runsAfter / runsRightAfter)

Minimal changes so we can move ahead with 2.13.15. Includes all tests from #10856 but not the code cleanups.

Fixes scala/bug#13028.

@scala-jenkins scala-jenkins added this to the 2.13.16 milestone Sep 10, 2024
Phases referenced in constraints (runsAfter, runsBefore) may not
exist when running Scaladoc.
  - Don't warn about constraints on non-existing phases in Scaladoc
  - Drop invalid constraints (runsAfter / runsRightAfter / runsBefore)
  - Drop unreachabele phases (no valid runsAfter / runsRightAfter)
@som-snytt
Copy link
Contributor

I don't think a surgeon is allowed to perform heart surgery on their own child, or in this case, in-utero heart surgery, so I will step outside the OR, pull off my mask in a demonstrative manner and throw my scrubs into the bin. Less metaphorically, I have a rib injury from trail running so I didn't sleep for a week, as happens, so I'm out of gas for this issue.

@SethTisue SethTisue merged commit 31538fb into scala:2.13.x Sep 11, 2024
3 checks passed
@SethTisue
Copy link
Member

rib injury from trail running

😱 – rest, feel better!

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.

pre-2.13.15 regression: phase assembly related failure during Scaladoc generation in Scala.js repo
4 participants