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

chore: resolve merge conflicts for upstream-58bf161 #708

Merged
merged 5 commits into from
Nov 7, 2024

Conversation

Jrigada
Copy link
Contributor

@Jrigada Jrigada commented Nov 6, 2024

What 💻

  • Resolves merge conflicts for 58bf161bc9dd6e74de8cb61e3ae23f701feb5512

zerosnacks and others added 3 commits November 6, 2024 15:08
…9269)

* soft-pin on minor version, prefer workspace

* use workspace

* prefer workspace
…ng remapping name (#9246)" (#9274)

This reverts commit 455ba9b.
@Jrigada Jrigada requested a review from a team as a code owner November 6, 2024 18:48
@Karrq
Copy link
Contributor

Karrq commented Nov 6, 2024

Shuldn't the base be something else? I don't think we want to merge this directly, right?

@Jrigada Jrigada changed the base branch from main to upstream-58bf161 November 6, 2024 19:02
@Jrigada
Copy link
Contributor Author

Jrigada commented Nov 6, 2024

@Karrq Yes! just changed the base. Like always I put the wrong branch name so I had to redo it

crates/common/src/compile.rs Show resolved Hide resolved
@Jrigada Jrigada merged commit abc6afe into upstream-58bf161 Nov 7, 2024
4 checks passed
@Jrigada Jrigada deleted the upstream-58bf161-merge-conflicts branch November 7, 2024 13:48
Jrigada added a commit that referenced this pull request Nov 8, 2024
* chore(deps): prefer soft pinning on minor version for dependencies (#9269)

* soft-pin on minor version, prefer workspace

* use workspace

* prefer workspace

* Revert "fix(remappings): check if remapping to add starts with existing remapping name (#9246)" (#9274)

This reverts commit 455ba9b.

* Fix build on upstream merge

* chore: resolve merge conflicts for upstream-58bf161 (#708)

* chore(deps): prefer soft pinning on minor version for dependencies (#9269)

* soft-pin on minor version, prefer workspace

* use workspace

* prefer workspace

* Revert "fix(remappings): check if remapping to add starts with existing remapping name (#9246)" (#9274)

This reverts commit 455ba9b.

---------

Co-authored-by: zerosnacks <[email protected]>
Co-authored-by: grandizzy <[email protected]>

* chore: fix build for upstream merge 58bf161 (#709)

* chore(deps): prefer soft pinning on minor version for dependencies (#9269)

* soft-pin on minor version, prefer workspace

* use workspace

* prefer workspace

* Revert "fix(remappings): check if remapping to add starts with existing remapping name (#9246)" (#9274)

This reverts commit 455ba9b.

* Fix build on upstream merge

---------

Co-authored-by: zerosnacks <[email protected]>
Co-authored-by: grandizzy <[email protected]>

* Apply format

* Update rust-toolchain

* Clippy

* chore: Update rust version on CI (#714)

update rust version on CI

* Add back verification logic

---------

Co-authored-by: zerosnacks <[email protected]>
Co-authored-by: grandizzy <[email protected]>
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.

5 participants