Skip to content

feat: specify external deps using minimal Swift package manifest #60

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 28 commits into from
Dec 13, 2022
Merged

feat: specify external deps using minimal Swift package manifest #60

merged 28 commits into from
Dec 13, 2022

Conversation

@cgrindel
Copy link
Owner

@cgrindel cgrindel commented Dec 13, 2022 •
edited
Loading

  • Support describing external deps using a minimal Swift package manifest.
  • Resolve the minimal package manifest and generate swift_package declarations.
  • The update command looks for swift_package declarations for external deps resolution.
  • Remove wspace Go package as it is not needed.

Closes #10.
Closes #54.

@cgrindel
Initial models for spreq
a12e5ca
@cgrindel
Add models for Swift requirements.
6505669
@cgrindel
Add plumbing for generating a manifest, resolving deps from the manifest
2738bd1 
and generating swift_package declarations from the resolved package.
@cgrindel
Add swiftbin.ResolvePackage
a4a457a
@cgrindel
Remove swift reqs plumbing for now
c1a995c
@cgrindel
Add pkg_manifest_minimal
e0408da
@cgrindel
Fix bug when checking for existence of the resolved package
06dcb29
@cgrindel
Refactor HTTPArchive processing to its own function
bf8e092
@cgrindel
Update deleted packages
1d443f6
@cgrindel
Ignore the Swift build folder
40cc860
@cgrindel
Add plumbing for including modules in swift_package declarations
5e09969
@cgrindel
Generating exported targets/modules as a dictionary
80f1e42
@cgrindel
Indexing modules from swift_package. Need to plumb through to resolve
4fbbd26
@cgrindel
Resolved external dep
6d9ea96
@cgrindel
Update tests for NewHTTPArchiveFromRule
2ff1b2f
@cgrindel
Add tests for BazelLabelFromTarget
4a4e233
@cgrindel
Fix test for RepoRuleFromPin
0f374db
@cgrindel
Add GHXXX comment
fefbe2f
@cgrindel
Remove exported_targets_tests.go for now
a424060
@cgrindel
Add pkg_manifest_minimal to integration tests
99a9e68
@cgrindel
Add pkg_manifest_minimal integration test
bedd9dd
@cgrindel
Clean up
d557ef7
@cgrindel
Remove spreq
1948b7d
@cgrindel
The pkg_manifest integration test works
5baffa3
@cgrindel
Ran tidy
811251c
@cgrindel
Ensure Gazelle uses correct naming convention for buildtools
8aaf0df
@cgrindel
Remove wspace Go package
9095792
@cgrindel
Clean up
1759c49
@cgrindel cgrindel self-assigned this Dec 13, 2022
@cgrindel cgrindel enabled auto-merge (squash) December 13, 2022 17:14
@cgrindel cgrindel merged commit 04c6bfa into main Dec 13, 2022
@cgrindel cgrindel deleted the gh010_real_specify_external_deps_new branch December 13, 2022 18:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@cgrindel cgrindel

Labels

None yet

Projects

None yet

Milestone

No milestone

2 participants

@cgrindel