-
Notifications
You must be signed in to change notification settings - Fork 144
Introduce BuildInfo #2249
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
Introduce BuildInfo #2249
Conversation
874ae33
to
d4c6364
Compare
Based on discussion in #2051, I think that the |
ad2ded3
to
310c056
Compare
d6e526e
to
4736003
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Adding integration test for this feature would be beneficial.
modules/integration/src/test/scala/scala/cli/integration/RunTestDefinitions.scala
Outdated
Show resolved
Hide resolved
c202c8a
to
4b32166
Compare
modules/directives/src/main/scala/scala/build/preprocessing/directives/ComputeVersion.scala
Outdated
Show resolved
Hide resolved
modules/build/src/test/scala/scala/build/tests/SourceGeneratorTests.scala
Outdated
Show resolved
Hide resolved
3a9f0a2
to
71fb6d2
Compare
ComputeVersion is now moved to a different branch and will be a follow-up PR to this one |
56a06b4
to
2ad0dfc
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
modules/directives/src/main/scala/scala/build/preprocessing/directives/BuildInfo.scala
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Addresses #2106
As well as moves the wrapper generation to the step when Sources are created from ScopedSources