Skip to content

Generalize check-lurk-compiles.yml#61

Merged
samuelburnham merged 7 commits into
mainfrom
check-downstream-compiles
Apr 25, 2024
Merged

Generalize check-lurk-compiles.yml#61
samuelburnham merged 7 commits into
mainfrom
check-downstream-compiles

Conversation

@samuelburnham
Copy link
Copy Markdown
Member

@samuelburnham samuelburnham commented Mar 20, 2024

  • Generalizes check-lurk-compiles.yml to work with arbitrary downstream crates that import the workflow caller
  • Enables support for top-level, workspace, and subdirectory upstream dependencies. Patches always go in the top-level Cargo.toml of the downstream crate

TODO:

  • Fix SSH access for private repos
  • Open issue for adding crates.io patch support

Successful runs:
samuelburnham/workspace-test#1

Closes #13

Note

Requires a careful merge as the branch is load-bearing in some caller repositories

@samuelburnham samuelburnham force-pushed the check-downstream-compiles branch 6 times, most recently from 498ca6a to bd8a21e Compare March 21, 2024 23:11
@samuelburnham samuelburnham changed the title Check downstream compiles Generalize check-lurk-compiles.yml Mar 21, 2024
@samuelburnham samuelburnham marked this pull request as ready for review March 21, 2024 23:28
@huitseeker
Copy link
Copy Markdown
Contributor

This needs a rebase!

@samuelburnham samuelburnham force-pushed the check-downstream-compiles branch from bd8a21e to f542a2e Compare March 22, 2024 18:43
huitseeker
huitseeker previously approved these changes Mar 23, 2024
Copy link
Copy Markdown
Contributor

@huitseeker huitseeker left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks really nice, though at some point I wonder if we shouldn't move to https://crates.io/crates/cargo-edit.
Thanks a lot!

@samuelburnham samuelburnham force-pushed the check-downstream-compiles branch 10 times, most recently from 9602485 to 192ec2f Compare April 25, 2024 02:02
huitseeker
huitseeker previously approved these changes Apr 25, 2024
Copy link
Copy Markdown
Contributor

@huitseeker huitseeker left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@samuelburnham samuelburnham force-pushed the check-downstream-compiles branch from 56ec70e to 0760948 Compare April 25, 2024 14:21
@samuelburnham samuelburnham added this pull request to the merge queue Apr 25, 2024
Merged via the queue into main with commit cfd3093 Apr 25, 2024
@samuelburnham samuelburnham deleted the check-downstream-compiles branch April 25, 2024 14:33
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.

Transitive dependency support for check-lurk-compiles

2 participants