-
Notifications
You must be signed in to change notification settings - Fork 70
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
Implement normalization of manifests for publishing #710
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
mkaput
force-pushed
the
spr/main/fa87cc89
branch
from
September 25, 2023 19:44
bbb37b4
to
92bcba6
Compare
This was referenced Sep 25, 2023
Merged
mkaput
force-pushed
the
spr/main/fa87cc89
branch
from
September 26, 2023 10:20
92bcba6
to
465f3f0
Compare
mkaput
force-pushed
the
spr/main/1b03d139
branch
from
September 26, 2023 10:20
3a527d3
to
106b26b
Compare
mkaput
force-pushed
the
spr/main/fa87cc89
branch
from
September 26, 2023 10:21
465f3f0
to
d516f3e
Compare
github-merge-queue bot
pushed a commit
that referenced
this pull request
Sep 27, 2023
1. There were no reasons to disallow `[[target.lib]]`. 2. On the other hand, it would be cool to expect target names to have package-name-style validation. --- **Stack**: - #697 - #710 - #675 - #657 - #646 - #712 ⬅⚠️ *Part of a stack created by [spr](https://github.com/ejoffe/spr). Do not merge manually using the UI - doing so may have unexpected results.*
mkaput
force-pushed
the
spr/main/fa87cc89
branch
2 times, most recently
from
September 27, 2023 10:14
c5b93fb
to
3688e71
Compare
mkaput
force-pushed
the
spr/main/1b03d139
branch
from
September 27, 2023 10:14
ada868c
to
43c47f1
Compare
mkaput
force-pushed
the
spr/main/fa87cc89
branch
2 times, most recently
from
September 27, 2023 17:53
89a843c
to
da7f64a
Compare
mkaput
force-pushed
the
spr/main/1b03d139
branch
from
September 27, 2023 18:04
504aa3b
to
c1f45dc
Compare
mkaput
force-pushed
the
spr/main/fa87cc89
branch
from
September 27, 2023 18:04
da7f64a
to
6adac2a
Compare
mkaput
force-pushed
the
spr/main/1b03d139
branch
from
September 27, 2023 19:00
c1f45dc
to
f633d11
Compare
mkaput
force-pushed
the
spr/main/fa87cc89
branch
from
September 27, 2023 19:00
6adac2a
to
52be2f0
Compare
Merged
mkaput
force-pushed
the
spr/main/fa87cc89
branch
2 times, most recently
from
September 28, 2023 12:16
30f7875
to
3c3ced4
Compare
github-merge-queue bot
pushed a commit
that referenced
this pull request
Sep 28, 2023
**Stack**: - #697 - #710 - #726 - #675 ⬅⚠️ *Part of a stack created by [spr](https://github.com/ejoffe/spr). Do not merge manually using the UI - doing so may have unexpected results.*
mkaput
force-pushed
the
spr/main/fa87cc89
branch
from
September 28, 2023 12:28
3c3ced4
to
f6f68fa
Compare
mkaput
force-pushed
the
spr/main/57e55df8
branch
from
September 28, 2023 12:28
74792ef
to
891a55f
Compare
github-merge-queue bot
pushed a commit
that referenced
this pull request
Sep 28, 2023
This PR adds a new DSL for specifying dependencies in `ProjectBuilder` and `WorkspaceBuilder`. It is capable of expressing all cases for which we have been falling back to strings currently, and also it can now do things like amending auto-generated deps for e.g. paths. This PR updates all string- and TOML table-based usages of old API. It does not migrate explicit paths usage to auto-generation for `ChildPaths` whereverr we want to ensure paths are properly treated as relative. --- **Stack**: - #697 - #710 - #726 ⬅⚠️ *Part of a stack created by [spr](https://github.com/ejoffe/spr). Do not merge manually using the UI - doing so may have unexpected results.*
mkaput
force-pushed
the
spr/main/fa87cc89
branch
from
September 28, 2023 12:50
f6f68fa
to
56e4a56
Compare
maciektr
reviewed
Sep 28, 2023
maciektr
approved these changes
Sep 28, 2023
mkaput
force-pushed
the
spr/main/fa87cc89
branch
from
October 3, 2023 09:45
56e4a56
to
890c2cd
Compare
This was referenced Oct 3, 2023
maciektr
approved these changes
Oct 3, 2023
mkaput
force-pushed
the
spr/main/fa87cc89
branch
2 times, most recently
from
October 5, 2023 08:23
a267b4d
to
d86e6df
Compare
This was referenced Oct 5, 2023
mkaput
force-pushed
the
spr/main/fa87cc89
branch
2 times, most recently
from
October 6, 2023 08:10
326a3af
to
4fbbe50
Compare
szymmis
approved these changes
Oct 6, 2023
commit-id:fa87cc89
mkaput
force-pushed
the
spr/main/fa87cc89
branch
from
October 6, 2023 08:58
4fbbe50
to
b1a2d72
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Stack:
todo!
calls in packaging logic #758registry
field to dependency schema #757DetailedTomlDependency
into aBox
and useCow
to avoid cloning it #748.scarbignore
files when packaging #746