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

[3/x] Fix package manager integrations #263

Merged
merged 8 commits into from
Jan 10, 2022

Conversation

andrewchang-bird
Copy link
Contributor

@andrewchang-bird andrewchang-bird commented Jan 8, 2022

Stack

πŸ“š #266 [6/x] Update example projects to use 0.19.1
πŸ“š #265 [5/x] Fix support for Swift library evolution
πŸ“š #264 [4/x] Fix configurator relative path handling
πŸ“š #263 ← [3/x] Fix package manager integrations
πŸ“š #261 [2/x] Archive supporting sources in release workflow
πŸ“š #260 [1/x] Fix CLI archiving and distribution

Overview

Changes to the project structure and build pipeline caused most of the package manager integrations to break. The current package manager tests aren’t E2E (they only check if the spec / package definition validates), and the more robust example project tests don’t run against the current revision.

This PR fixes the regressions and improves the example project tests, allowing them to build against HEAD. The goal is to keep the example projects portable such that they can be used as a starting point, so the tests rewrite the Cartfile/Podfile/Package.swift manifest at run time to point to the local repo and CLI build.

Test Plan

  • The new E2E test jobs pass in CI.
  • Manually verified that all package managers work against staging as well.

@andrewchang-bird andrewchang-bird added this to the Release 0.19 milestone Jan 8, 2022
andrewchang-bird added a commit that referenced this pull request Jan 8, 2022
Changes to the project structure and build pipeline caused most of the
package manager integrations to break. This fixes the regressions and
improves the example project tests, allowing them to build against HEAD.
@andrewchang-bird andrewchang-bird force-pushed the dev/andrewchang-bird/archive-supporting-sources branch from 1e15f07 to 1b1dbf5 Compare January 10, 2022 19:54
Base automatically changed from dev/andrewchang-bird/archive-supporting-sources to master January 10, 2022 20:40
@andrewchang-bird andrewchang-bird force-pushed the dev/andrewchang-bird/fix-package-managers branch from e90e2d3 to 85ea3ac Compare January 10, 2022 20:44
@andrewchang-bird andrewchang-bird merged commit 2c90c82 into master Jan 10, 2022
@andrewchang-bird andrewchang-bird deleted the dev/andrewchang-bird/fix-package-managers branch January 10, 2022 21:30
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.

2 participants