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

Add support for package field under dependencies. If unspecified, require that dependency name matches package name. #1002

Merged
merged 8 commits into from
Mar 23, 2022

Conversation

mitchmindtree
Copy link
Contributor

Closes #977 - see that issue for motivation. Unblocks #978.

A new should_pass/forc/dependency_package_field test has been added that tests the functionality.

All tests that depend on std and are pinned to a sway-lib-std commit older than FuelLabs/sway-lib-std@385c457 have been updated to the latest commit.

adlerjohn
adlerjohn previously approved these changes Mar 23, 2022
Copy link
Contributor

@adlerjohn adlerjohn left a comment

Choose a reason for hiding this comment

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

utACK

JoshuaBatty
JoshuaBatty previously approved these changes Mar 23, 2022
Copy link
Member

@JoshuaBatty JoshuaBatty left a comment

Choose a reason for hiding this comment

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

Nice, looks good to me. The test runs successfully on my end. just some conflicts with the master branch that need resolving.

This allows for the dependency name to differ from the dependency's
package name.

Closes #977.
It is no longer valid for packages to contain a manifest with a name
that does not match the name of the pinned package within the lock file.
As a result, the lock files of these tests have been invalidated and
regenerated.
@mitchmindtree mitchmindtree dismissed stale reviews from JoshuaBatty and adlerjohn via 3278751 March 23, 2022 23:18
@mitchmindtree mitchmindtree force-pushed the mitchmindtree/forc-package-names branch from 225c89a to 3278751 Compare March 23, 2022 23:18
@mitchmindtree
Copy link
Contributor Author

Okydoke, fixed the test conflicts, should be good to go 🟢

Copy link
Member

@JoshuaBatty JoshuaBatty left a comment

Choose a reason for hiding this comment

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

ship it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Either require that dependency name matches project name, or add a package field like cargo to support aliases
3 participants