Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix grammar of the accepted feature warning #16840

Merged
merged 1 commit into from
Aug 31, 2014

Conversation

huonw
Copy link
Member

@huonw huonw commented Aug 29, 2014

No description provided.

alexcrichton added a commit to alexcrichton/rust that referenced this pull request Aug 29, 2014
@alexcrichton
Copy link
Member

This needs to update the relevant test as well (see alexcrichton@89163fc)

@huonw huonw force-pushed the feature-has-added branch from 35492f1 to b79930e Compare August 30, 2014 16:25
alexcrichton added a commit to alexcrichton/rust that referenced this pull request Aug 31, 2014
@bors bors merged commit b79930e into rust-lang:master Aug 31, 2014
lnicola pushed a commit to lnicola/rust that referenced this pull request Jun 23, 2024
Allow rust-project.json to include arbitrary shell commands for runnables

This is a follow-up on rust-lang#16135, resolving the feedback raised :)

Allow rust-project.json to include shell runnables, of the form:

```
{
  "build_info": {
    "label": "//project/foo:my-crate",
    "target_kind": "bin",
    "shell_runnables": [
      {
        "kind": "run",
        "program": "buck2",
        "args": ["run", "//project/foo:my-crate"]
      },
      {
        "kind": "test_one",
        "program": "test_runner",
        "args": ["--name=$$TEST_NAME$$"]
      }
    ]
  }
}

```

If these runnable configs are present for the current crate in rust-project.json, offer them as runnables in VS Code.

This PR required some boring changes to APIs that previously only handled cargo situations. I've split out these changes as commits labelled 'refactor', so it's easy to see the interesting changes.
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants