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

Do requirements analysis for literal text argument to use #2211

Merged
merged 4 commits into from
Nov 18, 2024

Conversation

byorgey
Copy link
Member

@byorgey byorgey commented Nov 17, 2024

In the special case that we have something like use "key" we can statically know that it requires a key device. For example, this means that now a program like

build { move; use "key"; move }

will automatically install a key device on the built robot.

Closes #1301.

@byorgey byorgey requested review from xsebek and kostmo November 17, 2024 23:27
Copy link
Member

@xsebek xsebek left a comment

Choose a reason for hiding this comment

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

Nice! 👍

@byorgey byorgey added the merge me Trigger the merge process of the Pull request. label Nov 18, 2024
@mergify mergify bot merged commit eb68fcf into main Nov 18, 2024
12 checks passed
@mergify mergify bot deleted the feature/static-use branch November 18, 2024 11:44
@byorgey byorgey added the CHANGELOG Once merged, this PR should be highlighted in the changelog for the next release. label Dec 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CHANGELOG Once merged, this PR should be highlighted in the changelog for the next release. merge me Trigger the merge process of the Pull request.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Perform requirements analysis on static argument to use command
2 participants